Skip to content

docs: track remaining tracker + plugin open gaps post-rebase#51

Draft
dordor12 wants to merge 1 commit into
mainfrom
claude/find-missing-implementations-fpNmy
Draft

docs: track remaining tracker + plugin open gaps post-rebase#51
dordor12 wants to merge 1 commit into
mainfrom
claude/find-missing-implementations-fpNmy

Conversation

@dordor12

Copy link
Copy Markdown
Owner

Summary

Captures the punch lists from a 10-agent parallel spec-vs-implementation audit as tracked plan documents under docs/superpowers/plans/. Each gap includes file:line, symptom, and a remediation plan. The "Recently closed" sections reference PRs #43#49 so the docs serve as both a status snapshot and a working backlog.

Tracker — 8 open gaps (2026-05-11-tracker-open-gaps.md)

  • T8 — Settlement consumer-sig verification deferred to T17.5 (wire-format amendment needed)
  • T9session/inflight.go:54 hardcodes time.Now() instead of injected clock
  • T10Ledger.AssertChainIntegrity never invoked at runtime (only tests)
  • T13 — Settlement pre-transition state guard missing (silently swallows ErrIllegalTransition)
  • T19 / T20admission.FetchHeadroomToS and broker.SeederPostAcceptDisconnect metrics registered but never .Inc()'d
  • M4shared/proto/canonical.go:37 bypasses signing.DeterministicMarshal choke point
  • T21–T24 — Documented MVP scope (TEE PCR whitelist, reputation signal derivation, FirstSeenAt decay, evaluator perf)

Plugin — 14 open gaps (2026-05-11-plugin-open-gaps.md)

PR #43 closed the entire consumer-side composition (consumerflow.Coordinator + 8 collaborators) but explicitly deferred two items:

  • P1 — Hook subprocess entry point: nothing reads stdin or invokes hooks.Dispatcher.Handle
  • P2plugin.json still has commands: [] / hooks: []; plugin/commands/ and plugin/hooks/ empty

Plus correctness/safety:

  • P3 — Seeder still uses NopAcceptor (blocked on tunnel-metadata wire format)
  • P4 — Runtime compatibility probe (§5.3 step 2) not implemented
  • P5trackerclient.balanceCache.invalidate //nolint:unused on reconnect
  • P6 — ccproxy NetworkRouter has no SessionModeStore reference
  • P7 — SSE translator wired only seeder-side; verify direction
  • P13 — Exhaustion proof built at hook time, cached for 15-min session (spec §5.6 wants ≤60s freshness)
  • P14 — ccproxy PassThroughRouter must preserve Authorization header verbatim (CLAUDE.md rule 3)

Plus polish: audit log rotation, config hot-reload (§9 TODO), Bedrock/Vertex/Foundry enrollment rejection, revert confirmation, config field consumption spot-check.

Test plan

  • Docs render correctly on GitHub (tables, anchors)
  • File-path references resolve to current main head
  • No code changes — no test/lint impact

https://claude.ai/code/session_018LM5gu14njmMi9TexizQCk


Generated by Claude Code

Capture the punch lists from the 10 parallel spec-vs-impl audits as
tracked plan documents. Each gap includes file:line, symptom, and
planned remediation. Recently-closed sections reference PRs #43#49 so
the docs are useful as both a status snapshot and a backlog.

- tracker: T8 settlement consumer-sig (T17.5), T9 Inflight clock,
  T10 AssertChainIntegrity runtime, T13 settlement state guard,
  T19/T20 unwired metrics, M4 canonical helper choke-point bypass,
  plus T21-T24 documented MVP follow-ups.
- plugin: P1 hook subprocess entry + P2 plugin.json declarations
  (deferred by #43), P3 seeder NopAcceptor, P4 runtime compat probe,
  P5 balanceCache.invalidate on reconnect, P6 NetworkRouter ↔
  SessionModeStore, P7 SSE translator direction, P13 request-time
  proof rebuild, P14 Authorization preservation, plus P8-P12 polish.

https://claude.ai/code/session_018LM5gu14njmMi9TexizQCk
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.

2 participants