All items below have been addressed. Kept as a record of what was fixed and where.
- Per-app ADR paths (
docs/adr/api/001.md) —/architectresolves$ADR_DIR→docs/adr/<workspace>/per monorepo workspace,docs/adr/_root/for repo-wide. - Multiple mvps/adrs per whole-app refactor —
/architectuses one umbrella ADR directory (NNNN-<umbrella>/with children +research/) instead of many;/developbuilds a decided rollout inline/fan-out rather than re-architecting. - Show the source of a recommendation —
/architectappends(basis: …)to recommended options;/mvpdoes the same on roadmap recommendations + runs a sourcing subagent that web-verifies links into a## Referencessection. - Reading all mvp files when unrelated —
/architect,/develop,/syncnow scope roadmap reads to the single numbered file containing the target feature; only/status(global reporter) reads all. - ADR status "Accepted" too early — ADR lifecycle is now
Proposed→In Progress→Accepted(+Superseded), mirroring the feature:/architectcreates itProposed(confirmation ratifies content, does not flip status),/developadvances it as it builds,/syncreconciles,/statusflags ADR↔feature drift.Acceptedmeans the feature has shipped. - Unused code left after a refactor —
/developnow removes superseded code as part of the build (SKILL.md rule +logical-guide.mdPhase 6): delete dead functions/files/branches/imports, verify nothing references them, typecheck/build/lint clean with the old code gone.