docs(reviews): daily review 2026-05-08

docs/daily-reviews/2026-05-08.md

@@ -0,0 +1,48 @@
+---
+date: 2026-05-08
+range: 49c60da5611be873713af8e2c36819da373fa3f6..03307f3ce29c3966629278a1f2871b0d82b7e79a
+commits: 2
+blockers: 0
+majors: 0
+minors: 0
+nits: 1
+issue: 194
+---
+
+## 2026-05-08 — 03307f3
+
+Reviewed: 49c60da5611be873713af8e2c36819da373fa3f6..03307f3ce29c3966629278a1f2871b0d82b7e79a (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 <!-- f:03307f3.1 -->
+  <details><summary>details</summary>
+
+  **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:**
+
+  ```diff
+  -## 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.
+
+  </details>
+
+---
+
+- 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`