Skip to content

Recover knowledge activation with verified owner-merge reconciliation - #324

Merged
tzhouam merged 2 commits into
mainfrom
codex/kb-reviewed-reconciliation-20261006
Oct 6, 2026
Merged

tzhouam merged 2 commits into
mainfrom
codex/kb-reviewed-reconciliation-20261006

Conversation

@tzhouam

@tzhouam tzhouam commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

The knowledge service blocks snapshot activation after owner-merged onboarding and administration PRs because those commits were not produced by its automated proposal ledger. Add an explicit, signed reconciliation command that verifies the exact merged history before admitting it.

The operator prepares a plan for the full current main SHA, inspects it, and applies it after a second verification. Admission requires an allowlisted authenticated operator and merger, matching PR merge/head commits, passing named checks on each exact head, both knowledge validators, and a fully verified snapshot. Immutable receipts are rechecked by audit, activation, status, and publisher control; future unknown commits still block activation. Automatic proposal gates, disposal records, pauses, and publication modes retain their existing behavior.

Validation: all 17 focused recovery tests pass, independent implementation review is clear, and real read-only evidence passes for all 15 historical merges and the complete snapshot. The full local provider suite completed: 3,855 passed, 18 skipped, and four failures from the unsupported system dependency described below. Documentation links, citations, and strict SPEC freshness checks also pass. Four tests hit an incompatible system MCP 2 installation; all 166 tests in the affected files pass in an isolated environment with the declared MCP 1 dependency. Exact-head historical CI reruns pass.

Final revision CI: both push and PR suites pass on 86e6ab5afd42324477dc652516ebe06e2f41a91d, including complete unit, CLI, packaged runtime, documentation, SPEC, and knowledge checks.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@tzhouam
tzhouam merged commit 4f98143 into main Oct 6, 2026
2 checks passed
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