Plugin: planning · Categories: side-effect surface (3), minor hardcoded opinion (1)
Source: work-readiness sweep (read-only audit vs docs/PLUGIN-PHILOSOPHY.md + docs/MIGRATION-PLAYBOOK.md)
Findings
plugins/planning/skills/plan/SKILL.md:109 — tracker-write phases → gh issue create; :313-316 — close-out promotes durable outcomes ("Actionable follow-ups go through the work-item tracker seam") and hardcodes git mv targets docs/adr/ / docs/specs/
plugins/planning/skills/plan/context/plan-template.md:300-325 — phase-entry check hardcodes gh issue create / gh issue comment (:319)
Assessment
The create paths are gated — plan approved at Step 5, close-out explicitly invoked, search-before-create dedup mandated (plan-template.md:309-315). Remaining gaps: (a) still tracker writes, so they need the same consume-only guard as wayfind (sibling issue) for read-only environments; (b) GitHub/gh hardcoded — same backend coupling as the wayfind issue; (c) minor category-1: close-out hardcodes docs/adr/ / docs/specs/ while interview.md:125 correctly defers ("if none is declared, offer and defer — never prescribe a location") — internal inconsistency.
Fix direction
Fold tracker-write phases and close-out follow-up filing under the same tracker-write userConfig gate + work-item-tracker seam as wayfind; let ADR/specs subpaths be consumer-declared, as interview already does.
Plugin:
planning· Categories: side-effect surface (3), minor hardcoded opinion (1)Source: work-readiness sweep (read-only audit vs docs/PLUGIN-PHILOSOPHY.md + docs/MIGRATION-PLAYBOOK.md)
Findings
plugins/planning/skills/plan/SKILL.md:109— tracker-write phases →gh issue create;:313-316— close-out promotes durable outcomes ("Actionable follow-ups go through the work-item tracker seam") and hardcodesgit mvtargetsdocs/adr//docs/specs/plugins/planning/skills/plan/context/plan-template.md:300-325— phase-entry check hardcodesgh issue create/gh issue comment(:319)Assessment
The create paths are gated — plan approved at Step 5, close-out explicitly invoked, search-before-create dedup mandated (
plan-template.md:309-315). Remaining gaps: (a) still tracker writes, so they need the same consume-only guard as wayfind (sibling issue) for read-only environments; (b) GitHub/ghhardcoded — same backend coupling as the wayfind issue; (c) minor category-1: close-out hardcodesdocs/adr//docs/specs/whileinterview.md:125correctly defers ("if none is declared, offer and defer — never prescribe a location") — internal inconsistency.Fix direction
Fold tracker-write phases and close-out follow-up filing under the same tracker-write userConfig gate + work-item-tracker seam as wayfind; let ADR/specs subpaths be consumer-declared, as interview already does.