feat(streaming): expose graph-level operational status - #442
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This slice adds no recovery-format or coordinator changes.
Validation
cargo test --workspace --lockedscripts/check-agents-md.shgit diff --checkGreptile Summary
This PR exposes a checked, graph-redacted streaming operational-status cut through the engine, shared API types, HTTP/OpenAPI route, and remote CLI.
Cache-Control: no-store.omnigraph stream statuswith JSON and human-readable output, plus API, server, CLI, concurrency, and documentation coverage.Confidence Score: 5/5
The PR appears safe to merge; no concrete blocking or independently actionable non-blocking defect remains.
The new route is protected by graph-read authorization, status capture is bounded and serialized with stream authority using a consistent gate order, movement is rejected rather than combined into a partial cut, and transport projection excludes physical and recovery identities.
Important Files Changed
Sequence Diagram
sequenceDiagram participant CLI as Remote CLI / HTTP client participant Server as OmniGraph server participant Policy as Graph read policy participant Gate as Process status gate participant Engine as Checked status capture participant Manifest as Manifest/token authority CLI->>Server: "GET /graphs/{graph_id}/stream/status" Server->>Policy: authorize graph read Policy-->>Server: allow Server->>Gate: try acquire single process slot Gate-->>Server: permit Server->>Engine: capture served graph status Engine->>Manifest: preflight immutable authority Engine->>Engine: fence fold/profile/lane movement Engine->>Manifest: verify trailing cut Engine-->>Server: graph-redacted operational status Server-->>CLI: 200 JSON + Cache-Control: no-storeReviews (1): Last reviewed commit: "feat(streaming): expose graph-level oper..." | Re-trigger Greptile
Context used (5)