Skip to content

fix(docs): drop cross-repo symbol backticks tripping OC phantom-symbol gate #324

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ProtocolWarden merged 1 commit into from
Jun 18, 2026
Merged

fix(docs): drop cross-repo symbol backticks tripping OC phantom-symbol gate #324

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions .console/log.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6957,3 +6957,15 @@ WIRE/DELETE/KEEP dispositions, adversarially adjudicated (nothing deferred to a
human). Phase 1 (enforcement backbone) DONE: Custodian #46 closed the --only
silent-skip (gate now self-verifying). Phases 2 (WIRE 4 real gaps), 3 (DELETE
clean dead code), 4 (OC observer plane), 5 (ratchet cleanup) follow via /loop.

## 2026-06-18 — fix-forward: remediation doc tripped OC phantom-symbol gate

#323 merged (fleet reviewer LGTM; main is not branch-protected and the reviewer
does not gate on the advisory `audit` check) while the custodian-audit job was
red: my cross-repo roadmap doc backtick-referenced `p95_latency_ms` (a SwitchBoard
symbol), which OC's K1/OC8 phantom-symbol detectors flag (they only know OC src).
K1 has no per-file exclude (suppresses via known_values only), so the root-cause
fix is to drop the backticks on that one cross-repo symbol. Reworded line 64.
Audit now down to the sole environmental B2 (boundary artifact, materialized from
secret in CI). NOTE: the audit gate is advisory (main unprotected, reviewer
LGTM-merges over it) — a governance gap worth a follow-up.
2 changes: 1 addition & 1 deletion docs/design/INCOMPLETE_INTEGRATION_REMEDIATION.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ no external contract); KEEP = legitimate API/dispatch surface, do not touch.
|------|------|--------|
| TeamExecutor | `TeamExecutorRunner` never registered as an RxP runtime → the whole executor is undispatchable | WIRE: add the RxP runtime/entry-point registration |
| PlatformManifest | `parse_visibility_scope` — `visibility_scope` field shipped in `data/platform_manifest.yaml`, parsed nowhere | WIRE into `load_repo_graph`; surface on the graph |
| SwitchBoard | `p95_latency_ms` computed but the demote heuristic reads only `mean_latency_ms` (outlier-sensitive) | WIRE p95 into `AdjustmentEngine._evaluate` |
| SwitchBoard | p95 latency computed but the demote heuristic reads only mean latency (outlier-sensitive) | WIRE p95 into the AdjustmentEngine evaluate path |
| DAGExecutor | `NodeRunner` Protocol defined but the runner dispatch isn't typed to it (inert) | WIRE: type the runner registry as `dict[NodeType, NodeRunner]` |

### Phase 3 — DELETE provably-dead superseded code
Expand Down
Loading