Skip to content

Slim spec-to-plan master plan.md; lock the lean shape in evals#25

Merged
kreneskyp merged 1 commit into
mainfrom
slim-spec-to-plan-master-plan
Jun 27, 2026
Merged

Slim spec-to-plan master plan.md; lock the lean shape in evals#25
kreneskyp merged 1 commit into
mainfrom
slim-spec-to-plan-master-plan

Conversation

@kreneskyp

Copy link
Copy Markdown
Contributor

Why

The /spec-to-plan master plan.md had become an over-stated mirror of the bundle: the same task↔track↔FR↔TC↔status matrix was restated 3–4 times in prose (Requirements Summary, Dependency Graph, a duplicate Execution Tracks section, Remaining Work per-task Scope, Test Plan, Task File Mapping) — re-deriving data the task frontmatter already holds authoritatively (references/depends_on/verifies). On a 6-task plan that's ~160 lines, and it scales badly.

Root cause was the skill's own step-1/2/3 output templates, which stacked the redundant sections; the generation then piled on more (a standalone Execution Tracks section the template never asked for, the gate stated twice, free-form The seams prose).

What

Make plan.md an orchestration overview, not a mirror — state each fact once.

Skill templates (skills/spec-to-plan/):

  • step-3: Remaining Work collapses to one-liners (no per-task Scope, no TC-id exit lists — exit criteria are behavioral); a single Task File Mapping table is the only per-task matrix; no separate Execution Tracks section; gates stated once, inline in their track; new "Mirroring frontmatter in prose" Common Mistake.
  • step-1: The seams promoted to an official optional subsection; noted as the single dependency-graph representation.
  • step-2: Test Plan is the single TC enumeration.
  • SKILL.md: states the orchestration-overview principle up front.

Evals: EV-027 gains a fileContains shape check — a single Task File Mapping table is present and there is no separate Execution Tracks section — to lock in the slimming. Eval-matrix row (spec/evals.md) updated to match.

Verification

  • New EV-027 assertion verified against real fixtures: it catches the old duplicated shape (Plan-004, which carries ## Execution Tracks → fails) and passes the lean shape. (Scenario file parses; regex semantics mirror assert.mjs.)
  • eslint clean, prettier --check . clean, build present.
  • Live agent-pty EV-027 not run in CI/this PR: the harness's inner agent (--permission-mode bypassPermissions) blocks on Claude Code's bypass-permissions accept interstitial in environments where it isn't pre-accepted, so it must be run where that's configured. The assertion logic itself is verified offline against real plan files (above). Skill template changes are prose-only and don't touch src/.

🤖 Generated with Claude Code

@kreneskyp
kreneskyp requested a review from a team as a code owner June 27, 2026 17:06
The /spec-to-plan plan.md was an over-stated mirror of the bundle: the same
task↔track↔FR↔TC↔status matrix appeared 3-4 times (Requirements Summary,
Dependency Graph, a duplicate Execution Tracks section, Remaining Work
per-task Scope, Test Plan, Task File Mapping), restating data the task
frontmatter already holds authoritatively.

Skill templates now make plan.md an orchestration overview, not a mirror —
state each fact once:
- step-3: Remaining Work collapses to one-liners (no per-task Scope, no
  TC-id exit lists); a single Task File Mapping table is the only per-task
  matrix; no separate Execution Tracks section; gates stated once; new
  "Mirroring frontmatter in prose" Common Mistake.
- step-1: "The seams" promoted to an official optional subsection; noted as
  the single dependency-graph representation.
- step-2: Test Plan is the single TC enumeration.
- SKILL.md: states the orchestration-overview principle up front.

Evals: EV-027 gains a fileContains shape check (single Task File Mapping
table present; no Execution Tracks section) to lock in the slimming; spec
matrix row updated to match.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@kreneskyp
kreneskyp force-pushed the slim-spec-to-plan-master-plan branch from 5cbed94 to f4a07a3 Compare June 27, 2026 19:17
@kreneskyp
kreneskyp merged commit ba961fe into main Jun 27, 2026
6 checks passed
@kreneskyp
kreneskyp deleted the slim-spec-to-plan-master-plan branch June 27, 2026 20:21
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