gsd-antigravity-kit 1.32.0 → 2.0.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/.agent/skills/gsd/SKILL.md +152 -123
- package/.agent/skills/gsd/VERSION +1 -0
- package/.agent/skills/gsd/assets/templates/user-profile.md +8 -8
- package/.agent/skills/gsd/bin/gsd-tools.cjs +81 -3
- package/.agent/skills/gsd/bin/help-manifest.json +24 -1
- package/.agent/skills/gsd/bin/hooks/gsd-check-update.js +15 -5
- package/.agent/skills/gsd/bin/hooks/gsd-context-monitor.js +1 -1
- package/.agent/skills/gsd/bin/hooks/gsd-phase-boundary.sh +27 -0
- package/.agent/skills/gsd/bin/hooks/gsd-prompt-guard.js +2 -1
- package/.agent/skills/gsd/bin/hooks/gsd-read-guard.js +1 -1
- package/.agent/skills/gsd/bin/hooks/gsd-session-state.sh +33 -0
- package/.agent/skills/gsd/bin/hooks/gsd-statusline.js +5 -5
- package/.agent/skills/gsd/bin/hooks/gsd-validate-commit.sh +47 -0
- package/.agent/skills/gsd/bin/hooks/gsd-workflow-guard.js +1 -1
- package/.agent/skills/gsd/bin/lib/config.cjs +31 -10
- package/.agent/skills/gsd/bin/lib/core.cjs +48 -13
- package/.agent/skills/gsd/bin/lib/frontmatter.cjs +34 -2
- package/.agent/skills/gsd/bin/lib/intel.cjs +660 -0
- package/.agent/skills/gsd/bin/lib/learnings.cjs +378 -0
- package/.agent/skills/gsd/bin/lib/milestone.cjs +13 -4
- package/.agent/skills/gsd/bin/lib/model-profiles.cjs +17 -17
- package/.agent/skills/gsd/bin/lib/profile-output.cjs +31 -31
- package/.agent/skills/gsd/bin/lib/security.cjs +119 -0
- package/.agent/skills/gsd/bin/lib/verify.cjs +15 -15
- package/.agent/skills/gsd/migration_report.md +7 -0
- package/.agent/skills/gsd/references/agents/gsd-code-fixer.md +516 -0
- package/.agent/skills/gsd/references/agents/gsd-code-reviewer.md +355 -0
- package/.agent/skills/gsd/references/agents/gsd-debugger.md +10 -1
- package/.agent/skills/gsd/references/agents/gsd-executor.md +3 -0
- package/.agent/skills/gsd/references/agents/gsd-intel-updater.md +314 -0
- package/.agent/skills/gsd/references/agents/gsd-phase-researcher.md +3 -0
- package/.agent/skills/gsd/references/agents/gsd-plan-checker.md +16 -4
- package/.agent/skills/gsd/references/agents/gsd-planner.md +7 -0
- package/.agent/skills/gsd/references/agents/gsd-user-profiler.md +5 -5
- package/.agent/skills/gsd/references/agents/gsd-verifier.md +55 -1
- package/.agent/skills/gsd/references/agents/profiles/dev.md +21 -0
- package/.agent/skills/gsd/references/agents/profiles/research.md +22 -0
- package/.agent/skills/gsd/references/agents/profiles/review.md +22 -0
- package/.agent/skills/gsd/references/commands/{gsd-add-todo.md → atomic/add-todo.md} +5 -4
- package/.agent/skills/gsd/references/commands/{gsd-check-todos.md → atomic/check-todos.md} +5 -4
- package/.agent/skills/gsd/references/commands/{gsd-cleanup.md → atomic/cleanup.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-do.md → atomic/do.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-help.md → atomic/help.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-join-discord.md → atomic/join-discord.md} +21 -19
- package/.agent/skills/gsd/references/commands/{gsd-note.md → atomic/note.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-session-report.md → atomic/session-report.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-ship.md → atomic/ship.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-stats.md → atomic/stats.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-thread.md → atomic/thread.md} +7 -6
- package/.agent/skills/gsd/references/commands/atomic/undo.md +36 -0
- package/.agent/skills/gsd/references/commands/{gsd-add-backlog.md → milestone/add-backlog.md} +8 -7
- package/.agent/skills/gsd/references/commands/{gsd-audit-milestone.md → milestone/audit-milestone.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-complete-milestone.md → milestone/complete-milestone.md} +6 -4
- package/.agent/skills/gsd/references/commands/{gsd-milestone-summary.md → milestone/milestone-summary.md} +5 -3
- package/.agent/skills/gsd/references/commands/{gsd-new-milestone.md → milestone/new-milestone.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-plan-milestone-gaps.md → milestone/plan-milestone-gaps.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-plant-seed.md → milestone/plant-seed.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-review-backlog.md → milestone/review-backlog.md} +6 -5
- package/.agent/skills/gsd/references/commands/misc/audit-fix.md +35 -0
- package/.agent/skills/gsd/references/commands/{gsd-audit-uat.md → misc/audit-uat.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-next.md → misc/next.md} +6 -3
- package/.agent/skills/gsd/references/commands/{gsd-progress.md → misc/progress.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-verify-work.md → misc/verify-work.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-add-phase.md → phase/add-phase.md} +5 -4
- package/.agent/skills/gsd/references/commands/{gsd-add-tests.md → phase/add-tests.md} +8 -3
- package/.agent/skills/gsd/references/commands/{gsd-discuss-phase.md → phase/discuss-phase.md} +5 -4
- package/.agent/skills/gsd/references/commands/{gsd-execute-phase.md → phase/execute-phase.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-insert-phase.md → phase/insert-phase.md} +5 -4
- package/.agent/skills/gsd/references/commands/{gsd-list-phase-assumptions.md → phase/list-phase-assumptions.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-plan-phase.md → phase/plan-phase.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-remove-phase.md → phase/remove-phase.md} +5 -4
- package/.agent/skills/gsd/references/commands/{gsd-research-phase.md → phase/research-phase.md} +7 -6
- package/.agent/skills/gsd/references/commands/{gsd-secure-phase.md → phase/secure-phase.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-ui-phase.md → phase/ui-phase.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-ui-review.md → phase/ui-review.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-validate-phase.md → phase/validate-phase.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-workstreams.md → phase/workstreams.md} +71 -70
- package/.agent/skills/gsd/references/commands/{gsd-analyze-dependencies.md → project/analyze-dependencies.md} +5 -4
- package/.agent/skills/gsd/references/commands/project/explore.md +29 -0
- package/.agent/skills/gsd/references/commands/project/import.md +38 -0
- package/.agent/skills/gsd/references/commands/project/intel.md +181 -0
- package/.agent/skills/gsd/references/commands/{gsd-list-workspaces.md → project/list-workspaces.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-map-codebase.md → project/map-codebase.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-new-project.md → project/new-project.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-new-workspace.md → project/new-workspace.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-remove-workspace.md → project/remove-workspace.md} +4 -3
- package/.agent/skills/gsd/references/commands/project/scan.md +28 -0
- package/.agent/skills/gsd/references/commands/{gsd-autonomous.md → system/autonomous.md} +4 -3
- package/.agent/skills/gsd/references/commands/system/code-review-fix.md +54 -0
- package/.agent/skills/gsd/references/commands/system/code-review.md +57 -0
- package/.agent/skills/gsd/references/commands/{gsd-debug.md → system/debug.md} +7 -6
- package/.agent/skills/gsd/references/commands/{gsd-docs-update.md → system/docs-update.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-fast.md → system/fast.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-forensics.md → system/forensics.md} +5 -3
- package/.agent/skills/gsd/references/commands/{gsd-health.md → system/health.md} +5 -4
- package/.agent/skills/gsd/references/commands/{gsd-manager.md → system/manager.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-pause-work.md → system/pause-work.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-pr-branch.md → system/pr-branch.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-profile-user.md → system/profile-user.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-quick.md → system/quick.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-reapply-patches.md → system/reapply-patches.md} +25 -7
- package/.agent/skills/gsd/references/commands/{gsd-resume-work.md → system/resume-work.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-review.md → system/review.md} +4 -3
- package/.agent/skills/gsd/references/commands/system/set-profile.md +14 -0
- package/.agent/skills/gsd/references/commands/{gsd-settings.md → system/settings.md} +4 -3
- package/.agent/skills/gsd/references/commands/{gsd-update.md → system/update.md} +4 -3
- package/.agent/skills/gsd/references/docs/agent-contracts.md +79 -0
- package/.agent/skills/gsd/references/docs/common-bug-patterns.md +114 -0
- package/.agent/skills/gsd/references/docs/context-budget.md +49 -0
- package/.agent/skills/gsd/references/docs/domain-probes.md +125 -0
- package/.agent/skills/gsd/references/docs/few-shot-examples/plan-checker.md +73 -0
- package/.agent/skills/gsd/references/docs/few-shot-examples/verifier.md +109 -0
- package/.agent/skills/gsd/references/docs/gate-prompts.md +100 -0
- package/.agent/skills/gsd/references/docs/gates.md +70 -0
- package/.agent/skills/gsd/references/docs/model-profile-resolution.md +2 -0
- package/.agent/skills/gsd/references/docs/model-profiles.md +20 -14
- package/.agent/skills/gsd/references/docs/planning-config.md +216 -1
- package/.agent/skills/gsd/references/docs/revision-loop.md +97 -0
- package/.agent/skills/gsd/references/docs/thinking-models-debug.md +44 -0
- package/.agent/skills/gsd/references/docs/thinking-models-execution.md +50 -0
- package/.agent/skills/gsd/references/docs/thinking-models-planning.md +62 -0
- package/.agent/skills/gsd/references/docs/thinking-models-research.md +50 -0
- package/.agent/skills/gsd/references/docs/thinking-models-verification.md +55 -0
- package/.agent/skills/gsd/references/docs/thinking-partner.md +96 -0
- package/.agent/skills/gsd/references/docs/universal-anti-patterns.md +58 -0
- package/.agent/skills/gsd/references/docs/user-profiling.md +10 -10
- package/.agent/skills/gsd/references/docs/verification-overrides.md +227 -0
- package/.agent/skills/gsd/references/docs/workstream-flag.md +2 -2
- package/.agent/skills/gsd/references/mapping.md +11 -21
- package/.agent/skills/gsd/references/workflows/analyze-dependencies.md +3 -3
- package/.agent/skills/gsd/references/workflows/audit-fix.md +157 -0
- package/.agent/skills/gsd/references/workflows/autonomous.md +22 -1
- package/.agent/skills/gsd/references/workflows/code-review-fix.md +497 -0
- package/.agent/skills/gsd/references/workflows/code-review.md +515 -0
- package/.agent/skills/gsd/references/workflows/discuss-phase-power.md +3 -3
- package/.agent/skills/gsd/references/workflows/discuss-phase.md +20 -0
- package/.agent/skills/gsd/references/workflows/execute-phase.md +103 -0
- package/.agent/skills/gsd/references/workflows/explore.md +139 -0
- package/.agent/skills/gsd/references/workflows/import.md +274 -0
- package/.agent/skills/gsd/references/workflows/inbox.md +384 -0
- package/.agent/skills/gsd/references/workflows/manager.md +5 -5
- package/.agent/skills/gsd/references/workflows/new-milestone.md +1 -1
- package/.agent/skills/gsd/references/workflows/next.md +56 -0
- package/.agent/skills/gsd/references/workflows/plan-phase.md +48 -1
- package/.agent/skills/gsd/references/workflows/quick.md +96 -2
- package/.agent/skills/gsd/references/workflows/review.md +23 -3
- package/.agent/skills/gsd/references/workflows/scan.md +102 -0
- package/.agent/skills/gsd/references/workflows/settings.md +1 -1
- package/.agent/skills/gsd/references/workflows/ui-review.md +2 -2
- package/.agent/skills/gsd/references/workflows/undo.md +312 -0
- package/.agent/skills/gsd/references/workflows/update.md +5 -5
- package/.agent/skills/gsd-converter/SKILL.md +67 -59
- package/.agent/skills/gsd-converter/assets/migration-manifest.json +74 -0
- package/.agent/skills/gsd-converter/references/mapping.md +6 -16
- package/.agent/skills/gsd-converter/scripts/convert.py +419 -80
- package/.agent/skills/gsd-converter/scripts/regression_test.py +33 -0
- package/.agent/skills/selectpaste-update/SKILL.md +46 -0
- package/.agent/skills/selectpaste-update/scripts/sync-commands.py +317 -0
- package/README.md +4 -2
- package/bin/install.js +116 -116
- package/package.json +1 -1
- package/.agent/skills/gsd/references/commands/gsd-set-profile.md +0 -12
- /package/.agent/skills/gsd/references/commands/{gsd-tools.md → system/gsd-tools.md} +0 -0
|
@@ -1,12 +1,12 @@
|
|
|
1
1
|
<purpose>
|
|
2
|
-
Analyze ROADMAP.md phases for dependency relationships before execution. Detect file overlap between phases, semantic API/data-flow dependencies, and suggest `Depends on` entries to prevent merge conflicts during parallel execution by `/gsd
|
|
2
|
+
Analyze ROADMAP.md phases for dependency relationships before execution. Detect file overlap between phases, semantic API/data-flow dependencies, and suggest `Depends on` entries to prevent merge conflicts during parallel execution by `/gsd-manager`.
|
|
3
3
|
</purpose>
|
|
4
4
|
|
|
5
5
|
<process>
|
|
6
6
|
|
|
7
7
|
## 1. Load ROADMAP.md
|
|
8
8
|
|
|
9
|
-
Read `.planning/ROADMAP.md`. If it does not exist, error: "No ROADMAP.md found — run `/gsd
|
|
9
|
+
Read `.planning/ROADMAP.md`. If it does not exist, error: "No ROADMAP.md found — run `/gsd-new-project` first."
|
|
10
10
|
|
|
11
11
|
Extract all phases. For each phase capture:
|
|
12
12
|
- Phase number and name
|
|
@@ -91,6 +91,6 @@ When writing to ROADMAP.md:
|
|
|
91
91
|
- Preserve all other phase content unchanged
|
|
92
92
|
- Do not reorder phases
|
|
93
93
|
|
|
94
|
-
After applying: "ROADMAP.md updated. Run `/gsd
|
|
94
|
+
After applying: "ROADMAP.md updated. Run `/gsd-manager` to execute phases in the correct order."
|
|
95
95
|
|
|
96
96
|
</process>
|
|
@@ -0,0 +1,157 @@
|
|
|
1
|
+
<purpose>
|
|
2
|
+
Autonomous audit-to-fix pipeline. Runs an audit, parses findings, classifies each as
|
|
3
|
+
auto-fixable vs manual-only, spawns executor agents for fixable issues, runs tests
|
|
4
|
+
after each fix, and commits atomically with finding IDs for traceability.
|
|
5
|
+
</purpose>
|
|
6
|
+
|
|
7
|
+
<available_agent_types>
|
|
8
|
+
- gsd-executor — executes a specific, scoped code change
|
|
9
|
+
</available_agent_types>
|
|
10
|
+
|
|
11
|
+
<process>
|
|
12
|
+
|
|
13
|
+
<step name="parse-arguments">
|
|
14
|
+
Extract flags from the user's invocation:
|
|
15
|
+
|
|
16
|
+
- `--max N` — maximum findings to fix (default: **5**)
|
|
17
|
+
- `--severity high|medium|all` — minimum severity to process (default: **medium**)
|
|
18
|
+
- `--dry-run` — classify findings without fixing (shows classification table only)
|
|
19
|
+
- `--source <audit>` — which audit to run (default: **audit-uat**)
|
|
20
|
+
|
|
21
|
+
Validate `--source` is a supported audit. Currently supported:
|
|
22
|
+
- `audit-uat`
|
|
23
|
+
|
|
24
|
+
If `--source` is not supported, stop with an error:
|
|
25
|
+
```
|
|
26
|
+
Error: Unsupported audit source "{source}". Supported sources: audit-uat
|
|
27
|
+
```
|
|
28
|
+
</step>
|
|
29
|
+
|
|
30
|
+
<step name="run-audit">
|
|
31
|
+
Invoke the source audit command and capture output.
|
|
32
|
+
|
|
33
|
+
For `audit-uat` source:
|
|
34
|
+
```bash
|
|
35
|
+
.agent/skills/gsd/bin/gsd-tools.cjs" init audit-uat 2>/dev/null || echo "{}")
|
|
36
|
+
if [[ "$INIT" == @file:* ]]; then INIT=$(cat "${INIT#@file:}"); fi
|
|
37
|
+
```
|
|
38
|
+
|
|
39
|
+
Read existing UAT and verification files to extract findings:
|
|
40
|
+
- Glob: `.planning/phases/*/*-UAT.md`
|
|
41
|
+
- Glob: `.planning/phases/*/*-VERIFICATION.md`
|
|
42
|
+
|
|
43
|
+
Parse each finding into a structured record:
|
|
44
|
+
- **ID** — sequential identifier (F-01, F-02, ...)
|
|
45
|
+
- **description** — concise summary of the issue
|
|
46
|
+
- **severity** — high, medium, or low
|
|
47
|
+
- **file_refs** — specific file paths referenced in the finding
|
|
48
|
+
</step>
|
|
49
|
+
|
|
50
|
+
<step name="classify-findings">
|
|
51
|
+
For each finding, classify as one of:
|
|
52
|
+
|
|
53
|
+
- **auto-fixable** — clear code change, specific file referenced, testable fix
|
|
54
|
+
- **manual-only** — requires design decisions, ambiguous scope, architectural changes, user input needed
|
|
55
|
+
- **skip** — severity below the `--severity` threshold
|
|
56
|
+
|
|
57
|
+
**Classification heuristics** (err on manual-only when uncertain):
|
|
58
|
+
|
|
59
|
+
Auto-fixable signals:
|
|
60
|
+
- References a specific file path + line number
|
|
61
|
+
- Describes a missing test or assertion
|
|
62
|
+
- Missing export, wrong import path, typo in identifier
|
|
63
|
+
- Clear single-file change with obvious expected behavior
|
|
64
|
+
|
|
65
|
+
Manual-only signals:
|
|
66
|
+
- Uses words like "consider", "evaluate", "design", "rethink"
|
|
67
|
+
- Requires new architecture or API changes
|
|
68
|
+
- Ambiguous scope or multiple valid approaches
|
|
69
|
+
- Requires user input or design decisions
|
|
70
|
+
- Cross-cutting concerns affecting multiple subsystems
|
|
71
|
+
- Performance or scalability issues without clear fix
|
|
72
|
+
|
|
73
|
+
**When uncertain, always classify as manual-only.**
|
|
74
|
+
</step>
|
|
75
|
+
|
|
76
|
+
<step name="present-classification">
|
|
77
|
+
Display the classification table:
|
|
78
|
+
|
|
79
|
+
```
|
|
80
|
+
## Audit-Fix Classification
|
|
81
|
+
|
|
82
|
+
| # | Finding | Severity | Classification | Reason |
|
|
83
|
+
|---|---------|----------|---------------|--------|
|
|
84
|
+
| F-01 | Missing export in index.ts | high | auto-fixable | Specific file, clear fix |
|
|
85
|
+
| F-02 | No error handling in payment flow | high | manual-only | Requires design decisions |
|
|
86
|
+
| F-03 | Test stub with 0 assertions | medium | auto-fixable | Clear test gap |
|
|
87
|
+
```
|
|
88
|
+
|
|
89
|
+
If `--dry-run` was specified, **stop here and exit**. The classification table is the
|
|
90
|
+
final output — do not proceed to fixing.
|
|
91
|
+
</step>
|
|
92
|
+
|
|
93
|
+
<step name="fix-loop">
|
|
94
|
+
For each **auto-fixable** finding (up to `--max`, ordered by severity desc):
|
|
95
|
+
|
|
96
|
+
**a. Spawn executor agent:**
|
|
97
|
+
```
|
|
98
|
+
Task(
|
|
99
|
+
prompt="Fix finding {ID}: {description}. Files: {file_refs}. Make the minimal change to resolve this specific finding. Do not refactor surrounding code.",
|
|
100
|
+
subagent_type="gsd-executor"
|
|
101
|
+
)
|
|
102
|
+
```
|
|
103
|
+
|
|
104
|
+
**b. Run tests:**
|
|
105
|
+
```bash
|
|
106
|
+
npm test 2>&1 | tail -20
|
|
107
|
+
```
|
|
108
|
+
|
|
109
|
+
**c. If tests pass** — commit atomically:
|
|
110
|
+
```bash
|
|
111
|
+
git add {changed_files}
|
|
112
|
+
git commit -m "fix({scope}): resolve {ID} — {description}"
|
|
113
|
+
```
|
|
114
|
+
The commit message **must** include the finding ID (e.g., F-01) for traceability.
|
|
115
|
+
|
|
116
|
+
**d. If tests fail** — revert changes, mark finding as `fix-failed`, and **stop the pipeline**:
|
|
117
|
+
```bash
|
|
118
|
+
git checkout -- {changed_files} 2>/dev/null
|
|
119
|
+
```
|
|
120
|
+
Log the failure reason and stop processing — do not continue to the next finding.
|
|
121
|
+
A test failure indicates the codebase may be in an unexpected state, so the pipeline
|
|
122
|
+
must halt to avoid cascading issues. Remaining auto-fixable findings will appear in the
|
|
123
|
+
report as `not-attempted`.
|
|
124
|
+
</step>
|
|
125
|
+
|
|
126
|
+
<step name="report">
|
|
127
|
+
Present the final summary:
|
|
128
|
+
|
|
129
|
+
```
|
|
130
|
+
## Audit-Fix Complete
|
|
131
|
+
|
|
132
|
+
**Source:** {audit_command}
|
|
133
|
+
**Findings:** {total} total, {auto} auto-fixable, {manual} manual-only
|
|
134
|
+
**Fixed:** {fixed_count}/{auto} auto-fixable findings
|
|
135
|
+
**Failed:** {failed_count} (reverted)
|
|
136
|
+
|
|
137
|
+
| # | Finding | Status | Commit |
|
|
138
|
+
|---|---------|--------|--------|
|
|
139
|
+
| F-01 | Missing export | Fixed | abc1234 |
|
|
140
|
+
| F-03 | Test stub | Fix failed | (reverted) |
|
|
141
|
+
|
|
142
|
+
### Manual-only findings (require developer attention):
|
|
143
|
+
- F-02: No error handling in payment flow — requires design decisions
|
|
144
|
+
```
|
|
145
|
+
</step>
|
|
146
|
+
|
|
147
|
+
</process>
|
|
148
|
+
|
|
149
|
+
<success_criteria>
|
|
150
|
+
- Auto-fixable findings processed sequentially until --max reached or a test failure stops the pipeline
|
|
151
|
+
- Tests pass after each committed fix (no broken commits)
|
|
152
|
+
- Failed fixes are reverted cleanly (no partial changes left)
|
|
153
|
+
- Pipeline stops after the first test failure (no cascading fixes)
|
|
154
|
+
- Every commit message contains the finding ID
|
|
155
|
+
- Manual-only findings are surfaced for developer attention
|
|
156
|
+
- --dry-run produces a useful standalone classification table
|
|
157
|
+
</success_criteria>
|
|
@@ -356,6 +356,27 @@ Store the agent task_id. The workflow can now start discussing the next phase wh
|
|
|
356
356
|
Skill(skill="gsd-execute-phase", args="${PHASE_NUM} --no-transition")
|
|
357
357
|
```
|
|
358
358
|
|
|
359
|
+
**3c.5. Code Review and Fix**
|
|
360
|
+
|
|
361
|
+
Auto-invoke code review and fix chain. Autonomous mode chains both review and fix (unlike execute-phase/quick which only suggest fix).
|
|
362
|
+
|
|
363
|
+
**Config gate:**
|
|
364
|
+
```bash
|
|
365
|
+
.agent/skills/gsd/bin/gsd-tools.cjs" config-get workflow.code_review 2>/dev/null || echo "true")
|
|
366
|
+
```
|
|
367
|
+
If `"false"`: display "Code review skipped (workflow.code_review=false)" and proceed to 3d.
|
|
368
|
+
|
|
369
|
+
```
|
|
370
|
+
Skill(skill="gsd:code-review", args="${PHASE_NUM}")
|
|
371
|
+
```
|
|
372
|
+
|
|
373
|
+
Parse status from REVIEW.md frontmatter. If "clean" or "skipped": proceed to 3d. If findings found: auto-invoke:
|
|
374
|
+
```
|
|
375
|
+
Skill(skill="gsd:code-review-fix", args="${PHASE_NUM} --auto")
|
|
376
|
+
```
|
|
377
|
+
|
|
378
|
+
**Error handling:** If either Skill fails, catch the error, display as non-blocking, and proceed to 3d.
|
|
379
|
+
|
|
359
380
|
**3d. Post-Execution Routing**
|
|
360
381
|
|
|
361
382
|
**If `INTERACTIVE` is set:** Wait for the execute agent to complete before reading verification results.
|
|
@@ -784,7 +805,7 @@ Decisions captured: {count} across {area_count} areas
|
|
|
784
805
|
Completed through phase ${TO_PHASE} as requested.
|
|
785
806
|
Remaining phases were not executed.
|
|
786
807
|
|
|
787
|
-
Resume with: /gsd
|
|
808
|
+
Resume with: /gsd-autonomous --from ${next_incomplete_phase}
|
|
788
809
|
```
|
|
789
810
|
|
|
790
811
|
Proceed directly to lifecycle step (which handles partial completion — skips audit/complete/cleanup since not all phases are done). Exit cleanly.
|