chore(beads): complete native campaign migration - #4062
Conversation
Record the guarded live application of native campaign controls, bindings, and blocker edges; close the completed control-plane migration; and stage the source-cited AgentCTL adapter boundary without changing product routes or deletion authority. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013Euo3GTawxdtdRgob9B5ke
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 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 |
Record the source-proven retirement of the deleted incident-coverage ledger while preserving the executable archive-verification registry as the remaining verification owner. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_013Euo3GTawxdtdRgob9B5ke
💡 Codex Reviewhttps://github.com/Sinity/polylogue/blob/0792f9fd23a757add617d68111dc0e9153213f57/.beads/issues.jsonl#L199 The successful 151-member parity validation recorded on the migration Bead predates this adapter ( AGENTS.md reference: AGENTS.md:L410-L415 https://github.com/Sinity/polylogue/blob/0792f9fd23a757add617d68111dc0e9153213f57/.beads/issues.jsonl#L1 After the old campaign plane is deleted, no executable campaign validator remains in the repository: a repo-wide search finds the WIP-limit keys and parity requirements only in Beads prose, while this note records only a one-time validation run. Subsequent label, edge, batch, lane, or merge-train changes can therefore violate acceptance criterion 8 without any gate detecting them; leave this Bead partial until the stateless validator is checked in and wired into verification. AGENTS.md reference: AGENTS.md:L410-L415 https://github.com/Sinity/polylogue/blob/0792f9fd23a757add617d68111dc0e9153213f57/.beads/issues.jsonl#L1 A fresh checkout contains no thematic-batch formula definition under AGENTS.md reference: AGENTS.md:L410-L415 https://github.com/Sinity/polylogue/blob/0792f9fd23a757add617d68111dc0e9153213f57/.beads/issues.jsonl#L1 This closure dismisses acceptance criterion 7 because the old AGENTS.md reference: AGENTS.md:L345-L350 ℹ️ 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". |
|
Actionable. I reopened |
|
The migration state from this PR has been reconciled into master at e41d7195f, including the current Beads graph and the reviewed partial dispositions. The PR branch is now stale relative to master and its quick-gate result is not an admission signal for the current tree. The underlying native-control and AgentCTL Beads remain open for their explicitly listed acceptance criteria; those residuals are recorded in live Beads notes. This PR is therefore being closed as superseded by the reconciled master state. |
Summary
Record the native Beads campaign migration and retire the deleted incident ledger. This PR is a migration record and staged adapter boundary, not campaign completion.
Problem
The original PR body claimed the native-control migration was satisfied, but the authoritative Beads state reopened that Bead after independent review. The current graph exists and carries the historical scope, yet its provenance and validator contracts remain incomplete.
Solution
Whole-Bead disposition matrix
polylogue-reindex-native-control-planepolylogue-incident-coverage-ledgerpolylogue-agentctl-adapterVerification
The migration and graph checks recorded by the original implementation remain useful evidence. They do not prove the reopened native-control acceptance criteria. No live archive action or AgentCTL route cutover is claimed by this PR.
The authoritative Beads export and current independent review findings supersede the earlier completion wording.