Skip to content

Make each work plan task a commit-sized unit - #187

Merged
shinpr merged 1 commit into
mainfrom
fix/task-boundary-commit-unit
Sep 24, 2026
Merged

shinpr merged 1 commit into
mainfrom
fix/task-boundary-commit-unit

Conversation

@shinpr

@shinpr shinpr commented Sep 24, 2026

Copy link
Copy Markdown
Owner

Summary

Work plans could split changes that only pass the repository's commit checks together into separate tasks. The per-task commit cycle then could not commit those tasks one by one.

This PR fixes it mostly by removing rules. It adds one clause.

Changes

  • Removed: the phase-level "shared observable verification point" (work-planner.md, plan-template.md). The verification point now exists only at task level, so a phase can no longer act as the real commit unit while its tasks are fragments.
  • Removed: the Vertical/Horizontal/Hybrid slice guidance from plan-template.md. It duplicated implementation-approach.
  • Removed: the Rollback boundary field from the planner, the decomposer, and both templates. No downstream step read it.
  • Changed: the task grouping rule now ties the verification point to the repository's commit-time checks.
  • Changed: the task-decomposer description now matches its one-to-one mapping.

Also included: the patch version is bumped to 0.27.4, and the plugin copies are synced.

Verification

I ran work-planner → task-decomposer with claude -p and --plugin-dir on a fullstack Design Doc, once with main and once with this branch.

main This branch
Tasks 14 12
Commit checks Pass only at the end of each phase Pass at the end of every task

claude plugin validate passes for the marketplace and all plugins.

🤖 Generated with Claude Code

Work plans could split changes that only pass the repository's commit
checks together into separate tasks. The observable verification point
was defined at both task and phase level, so a phase could act as the
real commit unit while its tasks became non-committable fragments, and
the undefined rollback boundary field let plans record "phase commit
set" as a task boundary.

Remove the phase-level verification point, the slice-shape guidance in
the plan template (duplicated from implementation-approach), and the
rollback boundary field that no downstream step consumed. Tie the task
verification point to the repository's commit-time checks, and align
the task-decomposer description with its one-to-one mapping.

Bump the patch version to 0.27.4.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@shinpr shinpr self-assigned this Sep 24, 2026
@shinpr
shinpr merged commit 0c9856d into main Sep 24, 2026
1 check passed
@shinpr
shinpr deleted the fix/task-boundary-commit-unit branch September 24, 2026 10: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