Skip to content

docs(roadmap): verify and correct #434 sub-item status#442

Merged
BaseInfinity merged 1 commit into
mainfrom
roadmap-434-status-verify
Jul 10, 2026
Merged

docs(roadmap): verify and correct #434 sub-item status#442
BaseInfinity merged 1 commit into
mainfrom
roadmap-434-status-verify

Conversation

@BaseInfinity

Copy link
Copy Markdown
Owner

Summary

  • ROADMAP API features to review (28 pending as of 2026-07-01) #434's sub-items (b)-(f) status text hadn't been updated since the row was written, even though (b) and (c) actually shipped back in v1.84.0.
  • Corrected all five against verified evidence (file reads, git log, CHANGELOG/ROADMAP citations) rather than re-asserting stale text:
    • (b)/(c): confirmed DONE — hooks/model-effort-check.sh's model-aware floor and skills/sdlc/SKILL.md's removed effort: max frontmatter both landed in v1.84.0.
    • (d): reclassified from "still pending" to "partially superseded" — the failure-signal half of the original scope was built once (ROADMAP [MEDIUM] Auto-Update: Claude Code v2.1.114 #195), explicitly kept as valuable in the v1.69.0 bloat audit, then deleted 2026-07-06 as unfired. The context-size-trigger and /model opus-escalation half was never attempted.
    • (e): hooks/token-spike-check.sh (roadmap(#224-227): follow-ups from Codex review of auto-PRs #220) does adjacent but distinct post-hoc anomaly detection against past sessions, not the live Max-cap-aware monitor (e) originally asked for.
    • (f): reworded from "partially done" to "tracked, not started" — today's sibling sync issues (claude-gdlc-wizard#15, claude-rdlc-wizard#10) are tracking only; neither sibling repo has actually changed.

No code, hooks, or live guidance changed — this is a documentation-only status correction to an internal tracking row.

Review history

Codex (GPT-5.6 Sol, xhigh) — 3 rounds to CERTIFIED, not rubber-stamped:

  • Round 1: 4 findings (2 P1, 2 P2) — (f) overstated issue-filing as "partially done"; (d) overstated [MEDIUM] Auto-Update: Claude Code v2.1.114 #195 as covering the "exact mechanism" and misattributed the deletion evidence to CHANGELOG (which only records the addition); (b) overstated that all max configs are silent (settings-only max still warns); (e)'s "zero hits" grep claim was false (the concept is discussed as manual guidance, just no automated detector). All 4 fixed.
  • Round 2: 1 more finding — (e) still claimed "no automated detector exists" when hooks/token-spike-check.sh (roadmap(#224-227): follow-ups from Codex review of auto-PRs #220) already does adjacent token-burn detection. Fixed by precisely distinguishing post-hoc anomaly detection from the live cap-aware monitor never built.
  • Round 3: CERTIFIED, no further findings.

Test plan

@BaseInfinity BaseInfinity merged commit 586796a into main Jul 10, 2026
3 checks passed
@BaseInfinity BaseInfinity deleted the roadmap-434-status-verify branch July 10, 2026 23:34
Sub-items (b)-(f) of ROADMAP #434 hadn't been updated since the row
was written, even though (b) and (c) shipped in v1.84.0. Corrected
all five against verified evidence rather than re-asserting stale text:

- (b)/(c): confirmed DONE via current hook/skill content and git history
- (d): was built once as ROADMAP #195, kept as valuable in the v1.69.0
  bloat audit, then deleted 2026-07-06 as unfired — only the
  failure-signal half of the original scope ever existed
- (e): hooks/token-spike-check.sh (#220) does adjacent but distinct
  post-hoc anomaly detection, not the live cap-aware monitor (d) asked for
- (f): today's sibling sync issues (claude-gdlc-wizard#15,
  claude-rdlc-wizard#10) are tracking only — neither repo has changed

Codex (GPT-5.6 Sol, xhigh) reviewed 3 rounds to CERTIFIED, catching a
wrong CHANGELOG citation and two overstated absence claims along the way.

ROADMAP #434
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