docs(remediation): cross-repo incomplete-integration audit + roadmap - #323
Merged
Merged
Conversation
Plan of record for the #313-class debt across the platform. Headline finding (adversarial): the claimed-complete-but-inert pattern is NOT systemic — only OC's observer plane (#247/#279/#250) shows it; the other 10 src repos' "unwired" symbols are honestly-deferred cross-repo API, framework dispatch, or benign superseded wrappers. Per-item WIRE/DELETE/KEEP dispositions adjudicated. Phase 1 (Custodian #46, --only silent-skip) done; Phases 2-5 follow. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
ProtocolWarden
added a commit
that referenced
this pull request
Jun 18, 2026
…l gate (#324) The cross-repo remediation roadmap backtick-referenced `p95_latency_ms` (a SwitchBoard symbol). OC's K1/OC8 phantom-symbol detectors only know OC's own src, so they flagged it — turning the custodian-audit job red on main after #323 merged (the reviewer LGTM-merged over the advisory, non-required check). K1 suppresses via known_values only (no per-file exclude), so the clean fix is to drop the backticks on the cross-repo symbol. Audit now down to the sole environmental B2 (boundary artifact, materialized from a secret in CI). Co-authored-by: ProtocolWarden <ProtocolWarden@users.noreply.github.com> Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
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
1 participant
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.
Plan of record for the #313-class ("built + tested but never wired") debt across the platform, and the phased remediation now running autonomously.
Headline (adversarial)
Audited all 11 src-bearing repos (excluding VideoFoundry + PrivateManifest). The #313 claimed-complete-but-inert pattern is NOT systemic — only OC's observer plane (#247/#279/#250) shows a numbered PR claiming "Complete/Verified/integrated" over inert code. Everywhere else, "unwired" symbols are honestly-deferred cross-repo library API, framework-dispatched handlers, or benign superseded wrappers.
Roadmap
--onlysilent-skip (a gate naming an absent detector passed green). Gate is self-verifying.parse_visibility_scope, SwitchBoard p95, DAGExecutorNodeRunner).d12_baselineas debt burns down).Full per-item dispositions + the audited KEEP list are in the doc.
🤖 Generated with Claude Code