Code review 2026-05-08 — 03307f3

[Luis85]

2026-05-08 — 03307f3

Reviewed: 49c60da..03307f3 (2 commits)
Blockers: 0 | Majors: 0 | Minors: 0 | Nits: 1

---

• [NIT] docs/specs/2026-05-06-strategy-ai-modding-review.md:54 — implementation task list in a spec file should live in a plan doc

  details

  Problem: Lines 54–67 enumerate four concrete implementation tasks under a "Suggested near-term implementation plan" heading inside a docs/specs/ file

  Why it matters: CLAUDE.md reserves docs/plans/ for "implementation roadmaps, chunked task lists"; items in specs won't receive ✅ shipped markers or PR cross-links and will silently rot untracked

  Fix:

  -## Suggested near-term implementation plan
  +## Suggested next steps
  +
  +(Concrete tasks to be tracked in a companion `docs/plans/` file when committed.)

  Or extract all four items into docs/plans/2026-05-06-strategy-ai-modding-plan.md so they can be linked to PRs and marked shipped.

---

• Reviewed range: 49c60da5611be873713af8e2c36819da373fa3f6..03307f3ce29c3966629278a1f2871b0d82b7e79a (2 commits, 2 files)
• Blockers: 0
• Majors: 0
• Minors: 0
• Nits: 1
• Counter-argument check: 03307f3.1 tested — section is labelled "Suggested", making it advisory brainstorm output that fits a spec; no tracking overhead is incurred. Counter holds for severity (retaining as [NIT] only), but the observation that items are invisible to the plan-tracking workflow stands; finding retained.
• Not reviewed: lockfiles, generated files, examples/, prior daily-review docs (immutable snapshots, out of scope)
• Last reviewed SHA: 03307f3ce29c3966629278a1f2871b0d82b7e79a