Skip to content

refactor: project status through Hydra (COD-488) - #21

Merged
shivros merged 1 commit into
mainfrom
runner/COD-488-generated-status
Sep 19, 2026
Merged

shivros merged 1 commit into
mainfrom
runner/COD-488-generated-status

Conversation

@shivros

@shivros shivros commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Moves the existing GET /status endpoint into Rite's generated Hydra operation surface as HTTP-only get_status.
  • Routes the live process-local snapshot through shared execute_operation dispatch and removes the handwritten /status route.
  • Adds generated-route/dispatch parity and ownership-regression tests; documents exact counters and restart semantics.

Verification

  • cargo build --all-targets
  • cargo test --all-targets (50 tests)
  • cargo test -p rite-server --lib (14 tests)
  • cargo clippy --all-targets -- -D warnings
  • cargo fmt --all -- --check
  • cargo run -p rite-codegen -- check
  • creed diff
  • git diff --check
  • Ran codegen write twice; all generated artifacts were byte-identical on the second write. generated/cli.rs and generated/mcp.json retained their pre-change hashes.

Boundary

/status remains an HTTP-only, process-local snapshot. No CLI/MCP telemetry, metric semantics, authentication, deployment, configuration, release, or live-service behavior changes.

Linear: COD-488

@shivros

shivros commented Sep 19, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Automated Review Panel

Both reviewers received the COD-488 task goal, Rite project context/conventions, and the complete branch diff.

Reviewer A — GPT-5.6 Terra

Verdict: APPROVE

No substantive correctness or goal-alignment findings. It independently verified generated-surface freshness, the full test suite, strict Clippy, formatting, and that generated/cli.rs plus generated/mcp.json remain byte-identical to origin/main.

Reviewer B — Gemini 3 Flash

Verdict: APPROVE

The reviewer found that the operation is correctly HTTP-only, preserves the six nonnegative status fields, uses shared execute_operation dispatch, and removes the handwritten route while leaving /health as the sole handwritten endpoint. It specifically approved the real-router/dispatch parity test and the generated-route ownership regression guard. Ordering::Relaxed remains appropriate for this best-effort process-metrics snapshot.

@shivros
shivros marked this pull request as ready for review September 19, 2026 06:22
@shivros

shivros commented Sep 19, 2026

Copy link
Copy Markdown
Contributor Author

Auto-Merge Gate

Confidence: 0.94

COD-488's bounded objective is met: the existing HTTP-only /status snapshot is declared as Hydra get_status, routed through shared dispatch, and no longer registered by hand. The six-field process-local contract and the explicit no-CLI/no-MCP boundary are preserved.

Evidence: GitHub CI test and Creed context drift succeeded. Independent gate verification passed: cargo test -p rite-server --lib — 14 passed. The review panel approved both correctness and the generated-route ownership/parity coverage.

Scope checked: five coherent files; no secrets, auth redesign, deployment/cutover, billing, destructive behavior, or manual-review marker. Squash merge is authorized by this gate.

@shivros
shivros merged commit 5d500e8 into main Sep 19, 2026
2 checks passed
@shivros
shivros deleted the runner/COD-488-generated-status branch September 19, 2026 06:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant