Skip to content

Commit 301c3a6

Browse files
committed
Remove the standalone Plan phase
1 parent 32b4463 commit 301c3a6

30 files changed

Lines changed: 96 additions & 165 deletions

.agents/skills/act/SKILL.md

Lines changed: 17 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1,42 +1,44 @@
11
---
22
name: act
3-
description: Executes an explicitly authorized workflow Plan, gathers criterion-matched evidence, stops on every unresolved implementation doubt, and launches independent review before completion. Use automatically after the active workflow's Plan records user authorization, or when continuing implementation or closing review findings.
3+
description: Implements an explicitly authorized Ready Design, keeps a factual execution record, stops on every unresolved implementation doubt, and launches independent review before completion. Use automatically after the active Design records user authorization, or when continuing implementation or closing review findings.
44
---
55

66
# Workflow Act
77

8-
Execute the approved Plan without changing Design intent.
8+
Implement the approved Design without changing its intent. Choose mechanical execution order from repository facts; do not create a separate implementation Plan.
99

1010
## Start
1111

12-
Read active `design.md` and `plan.md` together. Confirm Design is Ready and Plan authorization is Approved. Inspect repository status and preserve unrelated user changes. If any precondition is false, stop.
12+
Read active `design.md`. Confirm Design is Ready, every coverage dimension is resolved or justified `N/A`, and `Implementation Authorization` is `Approved`. Inspect repository status and preserve unrelated user changes. If any precondition is false, stop.
13+
14+
Create or update `execution.md` from [assets/execution.md](assets/execution.md). It records work already started or completed, evidence, doubts, and deviations; it must not become a speculative task plan.
1315

1416
Read [references/doubt-protocol.md](references/doubt-protocol.md) before editing and use [references/validation-guide.md](references/validation-guide.md) throughout.
1517

16-
## Task loop
18+
## Execution loop
1719

18-
For each unchecked task in order:
20+
Until every Design success criterion is evidenced:
1921

20-
1. Re-read its Design anchors, constraints, affected system, and required evidence.
21-
2. Investigate unclear repository facts read-only.
22-
3. Before any mutation, confirm every Plan-required validation entrypoint and prerequisite exists and can be invoked. Inspect command/script availability without substituting or weakening the validation. If an entrypoint is unavailable, stop before implementation and apply the doubt protocol.
23-
4. If uncertainty remains, apply the doubt protocol and wait for the user.
24-
5. Implement the smallest coherent change uniquely entailed by Design and Plan.
25-
6. Run the planned validation and record raw results in `plan.md`.
26-
7. Mark the task complete only after its required evidence exists.
22+
1. Select an unmet success criterion or required production integration obligation and re-read its Design decisions, constraints, affected system, and required evidence.
23+
2. Investigate unclear repository facts read-only. Choose the next mechanical execution step from the actual dependency graph; do not ask the user about uniquely determined ordering.
24+
3. Before mutation, confirm the Design-required validation entrypoint and prerequisites exist and can be invoked. If unavailable, stop before implementation and apply the doubt protocol.
25+
4. If any implementation choice is not uniquely fixed by Design and repository facts, apply the doubt protocol and wait for the user.
26+
5. Implement the smallest coherent change uniquely entailed by Design.
27+
6. Run the applicable validation and append the raw result and production-path evidence to `execution.md`.
28+
7. Mark a success criterion covered only after its required evidence exists. Record deviations immediately; any deviation that changes Design requires user confirmation before further mutation.
2729

2830
Do not silently add dependencies, change APIs or formats, broaden scope, choose compatibility policy, introduce fallback behavior, skip evidence, or replace validation. Use required domain skills and follow the target project's version-control ownership rules.
2931

30-
Do not add or modify tests unless the user explicitly requested test work. Existing tests may be run when the authorized Plan names them as validation.
32+
Do not add or modify tests unless the user explicitly requested test work. Existing tests may be run when the approved Design names them as validation.
3133

3234
## Review and closure
3335

3436
After all tasks and agent-accessible validation finish:
3537

36-
1. Launch an independent subagent with `$agent-workflow-kit:review`, passing only the active record, repository root, and raw validation entrypoints. Do not pass your conclusions or suspected gaps.
38+
1. Launch an independent subagent with `$agent-workflow-kit:review`, passing only `design.md`, `execution.md`, the repository root, and raw validation entrypoints. Do not pass your conclusions or suspected gaps.
3739
2. If review returns P0/P1 or an unmet criterion, fix only when the required correction is uniquely determined. Otherwise ask the user before editing.
3840
3. Ask the reviewer to re-check fixes until it returns `PASS`, or report a genuine `BLOCKED` decision/evidence dependency.
39-
4. Reconcile the Completion Record, final Design behavior, validation results, deviations, and review verdict.
41+
4. Reconcile `execution.md`, final Design behavior, validation results, deviations, and review verdict.
4042
5. Apply required session log/ADR/architecture/feature updates. Move the entire change folder from `workflow/active/` to `workflow/completed/` only after `PASS`.
4143

4244
Never claim completion before independent `PASS`.
Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
interface:
22
display_name: "Workflow Act"
3-
short_description: "Execute an authorized plan and stop on unresolved doubt"
4-
default_prompt: "Use $act to execute the authorized plan with evidence and stop-on-doubt discipline."
3+
short_description: "Implement an authorized Design and record real evidence"
4+
default_prompt: "Use $act to implement the authorized Design, record execution evidence, and stop on unresolved doubt."
55

66
policy:
77
allow_implicit_invocation: true
Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# <change-name> Execution Record
2+
3+
**Status:** Active
4+
5+
## Design Success Criteria Coverage
6+
7+
| Success criterion | Production path / change | Evidence | Result |
8+
| --- | --- | --- | --- |
9+
10+
## Activity Log
11+
12+
<!-- Record work when it starts or completes. Do not pre-populate future tasks. -->
13+
14+
## Validation Evidence
15+
16+
## Doubts and User Decisions During Act
17+
18+
## Deviations from Design
19+
20+
<!-- Write "None" only when implementation is complete and no deviation occurred. -->
21+
22+
## Independent Review
23+

.agents/skills/act/references/doubt-protocol.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ Use read-only inspection to resolve discoverable facts: repository code and docs
66

77
## Stop when uncertainty remains
88

9-
Stop before further mutation whenever there are multiple reasonable implementations or uncertainty about Design/Plan conflict, scope, API/format, compatibility, persistence, dependency choice, fallback behavior, validation substitution, failure handling, task ordering, or a reviewer finding.
9+
Stop before further mutation whenever there are multiple reasonable implementations or uncertainty about a Design conflict or omission, scope, API/format, compatibility, persistence, dependency choice, fallback behavior, validation substitution, failure handling, non-mechanical task ordering, or a reviewer finding.
1010

1111
Tell the user:
1212

@@ -17,4 +17,4 @@ Tell the user:
1717

1818
Wait for the response. Until then, continue only safe read-only investigation. Do not implement the recommendation first.
1919

20-
Mechanical details may proceed only when Design, Plan, and existing project patterns admit one safe interpretation. If you doubt uniqueness, stop.
20+
Mechanical details may proceed only when Design and existing project patterns admit one safe interpretation. If you doubt uniqueness, stop.

.agents/skills/act/references/validation-guide.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,6 @@
66
- Editor/UI interaction: existing deterministic checks where useful, plus explicit manual interaction/appearance checks; create test coverage only when explicitly requested.
77
- Resource/version-control changes: follow the target project's ownership rules and record the relevant revision/signature.
88

9-
If planned evidence cannot be obtained, do not silently substitute a weaker check. Stop and ask the user.
9+
If Design-required evidence cannot be obtained, do not silently substitute a weaker check. Stop and ask the user.
1010

11-
Before modifying implementation files, verify that every Plan-required validation command, script, tool, fixture, and required external input is present and invocable. This availability preflight does not replace the post-change validation. If availability cannot be established, stop before mutation.
11+
Before modifying implementation files, verify that every Design-required validation command, script, tool, fixture, and required external input is present and invocable. This availability preflight does not replace the post-change validation. If availability cannot be established, stop before mutation.

.agents/skills/design/SKILL.md

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,11 @@
11
---
22
name: design
3-
description: Explores a change against the real repository and maintains a living, decision-complete Design before implementation planning. Use only when the user explicitly invokes $agent-workflow-kit:design; never start the workflow automatically from task complexity, risk, or ambiguity.
3+
description: Explores a change against the real repository and maintains a living, decision-complete Design before authorized implementation. Use only when the user explicitly invokes $agent-workflow-kit:design; never start the workflow automatically from task complexity, risk, or ambiguity.
44
---
55

66
# Workflow Design
77

8-
Turn an emerging change into a Ready Design through repository-grounded discussion. Do not plan implementation tasks or modify product code in this phase.
8+
Turn an emerging change into a Ready Design through repository-grounded discussion. Do not create an implementation task sequence or modify product code in this phase.
99

1010
## Start
1111

@@ -26,7 +26,7 @@ Work through the unresolved Design Coverage dimensions in dependency order. Do n
2626
4. Ask only for the remaining choice, one question at a time. A technical recommendation is not permission to choose for the user.
2727
5. Immediately update the affected Design section, `Decisions`, `Open Questions`, and Design Coverage status after each material resolution.
2828

29-
Design-level implementation mechanics include algorithms, state transitions, ownership, dependencies, integration points, and how existing production paths will consume the change. Decide these before Plan; leave file-by-file task sequencing and command execution to Plan.
29+
Design-level implementation mechanics include algorithms, state transitions, ownership, dependencies, integration points, and how existing production paths will consume the change. Decide these before Act. Do not prescribe a file-by-file task sequence; Act may choose mechanical execution order from the current repository.
3030

3131
Reject unrelated refactors and speculative requirements. If a question is inherently visual, follow [references/visual-companion.md](references/visual-companion.md). Offer the companion just in time and start it only after explicit consent.
3232

@@ -38,6 +38,7 @@ When the gate passes:
3838

3939
1. Set Design status to `Ready` and write `Open Questions: None`.
4040
2. Briefly tell the user why discussion is complete.
41-
3. Invoke `$agent-workflow-kit:plan` directly. There is no separate Design approval gate.
41+
3. Ask for explicit implementation authorization. Do not modify product code while waiting.
42+
4. After the user authorizes implementation, set `Implementation Authorization: Approved` in `design.md` and invoke `$agent-workflow-kit:act` directly.
4243

43-
The only terminal state is transition to Workflow Plan.
44+
The only terminal state is transition to Workflow Act after explicit authorization.
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
interface:
22
display_name: "Workflow Design"
33
short_description: "Explore and record a decision-complete change design"
4-
default_prompt: "Use $design to explore this change and maintain its living Design before planning."
4+
default_prompt: "Use $design to explore this change and maintain its living Design before authorized implementation."
55

66
policy:
77
allow_implicit_invocation: false

.agents/skills/design/assets/design.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
# <change-name> Design
22

33
**Status:** Exploring
4+
**Implementation Authorization:** Pending
45

56
## Summary
67

.agents/skills/design/references/design-contract.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ Design is Ready only when all are true:
1919
- Relevant current implementation and references have been inspected.
2020
- Every Design Coverage row is `Resolved` or a justified `N/A`; none is merely inferred from a filled heading.
2121
- Architecture boundaries, components and responsibilities, interfaces/contracts, data ownership and flow, implementation mechanics and integration points, failure/edge/compatibility behavior, and validation are defined or explicitly not applicable.
22-
- The implementation approach fixes the mechanisms and production integration path strongly enough that Plan does not need to invent architecture, ownership, contracts, algorithms, or compatibility policy. Plan may still choose task decomposition and execution order.
22+
- The implementation approach fixes the mechanisms and production integration path strongly enough that Act does not need to invent architecture, ownership, contracts, algorithms, or compatibility policy. Act may choose only mechanical execution order from repository facts.
2323
- Every design choice with multiple viable approaches records the compared approaches, causal tradeoffs, recommendation, and user decision. Do not manufacture alternatives when repository evidence leaves only one valid approach.
2424
- Success criteria are observable and testable with the chosen evidence type.
2525
- Every implementation-shaping decision is recorded and no such question remains open.

.agents/skills/plan/SKILL.md

Lines changed: 0 additions & 30 deletions
This file was deleted.

0 commit comments

Comments
 (0)