D61 — Startup-scripted human-gate progression - #83
Conversation
Decision 061 (STARTUP-SCRIPTED HUMAN-GATE PROGRESSION, Tier 2). Adds a bounded, one-shot startup submission of exactly one HUMAN_GATE_OPENED event after the authorized startup workflow-open. This is NOT a post-start external event source and NOT autonomous orchestration. - AutoflowOrchestrator gains a structurally narrow openHumanGate(): it mints one HUMAN_GATE_OPENED event bound to the workflow's own boundCommitSha and delegates once to the runtime. No generic apply(event); CLOSE_REQUESTED and the other kinds stay unconstructible in production (domain legality is not production authority). - readStartupHumanGateConfig reads AGENTBRIDGE_STARTUP_OPEN_HUMAN_GATE once with strict "1"-or-throw semantics (no trim/case-fold/coercion). - runStartupProgression is the single boot call site: a gate without a valid startup-open, a non-APPLIED open, or a non-APPLIED gate all fail closed before serving. No loop/timer/poll/callback/post-start path. Cockpit stays read-only; AutoflowRuntime remains sole state owner; no persistence/replay. Six strong boundaries byte-identical. Full suite 1891/1891; typecheck/lint/build/diff-check clean. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01H2mKZV7PWy3uRh6m81RRrX
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (6)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughThe change adds ChangesStartup Human-Gate Progression
Estimated code review effort: 3 (Moderate) | ~20 minutes Merge Risk: ⚪ Minimal · up to When explicitly enabled, startup now opens a human gate after opening the workflow, moving it to awaiting human decision. Invalid configuration and rejected transitions stop startup rather than proceeding with an invalid state, and no concrete merge-blocking risk remains. Sequence Diagram(s)sequenceDiagram
participant Environment
participant runStartupProgression
participant AutoflowOrchestrator
participant AutoflowRuntime
Environment->>runStartupProgression: read startup configuration
runStartupProgression->>AutoflowOrchestrator: open workflow binding
AutoflowOrchestrator->>AutoflowRuntime: apply workflow-open event
runStartupProgression->>AutoflowOrchestrator: openHumanGate()
AutoflowOrchestrator->>AutoflowRuntime: apply HUMAN_GATE_OPENED event
AutoflowRuntime-->>runStartupProgression: return transition result
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. Can't wait for the next one! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@coderabbitai review |
✅ Action performedReview finished.
|
Decision 061 — STARTUP-SCRIPTED HUMAN-GATE PROGRESSION
This Tier-2 milestone adds one structurally narrow production Autoflow progression:
startup workflow open
→ HUMAN_GATE_OPENED
→ AWAITING_HUMAN_DECISION
Production authority added:
Explicitly not added:
Validated candidate:
Candidate was independently validated before commit and the committed bytes were proven identical to the validated candidate.
This PR intentionally remains Draft pending exact-head CI and substantive review.
Human merge authority is preserved.
🤖 Generated with Claude Code
https://claude.ai/code/session_01H2mKZV7PWy3uRh6m81RRrX
Summary by CodeRabbit
New Features
Bug Fixes
Tests