[issue #101] Document pinned-launcher operational caveat and post-commissioning evidence checklist#121
Merged
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe runbook documents the pinned launcher required for enforcement and adds a post-commissioning checklist covering assurance reports, sealed-generation validation, identifier cross-checks, runtime acceptance, and claim suppression conditions. ChangesMain-agent enforcement operations
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related issues
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 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 |
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
Repo-side deliverable for issue #101 (milestone M1 of the deterministic-harness migration, epic #120). Documents the operational caveat that day-to-day operation of the enforced Codex main agent must go through the pinned launcher, and adds a post-commissioning evidence checklist for the human administrator. The host-side deployment/commissioning itself is human-only (sudo/root) and intentionally not performed by this change.
Changes
docs/runbooks/main-agent-enforcement.mdcodexinvocations / the legacy wrapper remain out of enforcement scope with noingress_enforcedclaim; the implemented ingress is Codex-only for now while the ingress contract stays launch-source-agnostic (refs [orchestrator] Make the frontdoor ingress contract launch-source-agnostic (extensible to Claude/Cursor/Grok) #119 andfrontdoor-orchestrator-protocol.md).decision,profiles[0].claim_results.action_enforced,profiles[0].effective_claims,profiles[0].attestation_digest) and the authoritative sources forcommissioning_id/ sealedgeneration_id/ selectedgeneration_id/activation_id, requiring the generation ids to match across sources.Testing
python3 scripts/validate_all.py→result: pass(41 suites), re-run after each commit.organization/runtime/workflows/frontdoor-orchestrator-protocol.md,organization/runtime/workflows/profiles/verify_enforcement.md); profile idcodex-main-agent-a-primematches the shipped profiles.Review
tech-devopssecrole review (independent session): approve, zero blocking findings. One minor evidence-clarity defect (report JSON nesting and generation-id sources left implicit) was fixed in the second commit; field paths were re-verified againstagent_integration_assurance.pyand the generation/activation state layout.Human handoff (host work, out of scope here)
docs/runbooks/main-agent-enforcement.mdend to end as the human administrator (root), commission perprofiles/verify_enforcement.md, seal and select an active generation.agent_integration_assurance.py reportand record the report output and generation ids as evidence in the Agents-Vault task record — this is the issue close condition, so this PR intentionally does not auto-close the issue.Refs #101
🤖 Generated with Claude Code
Summary by CodeRabbit