e81592878e
* feat(#2789): trim skill description anti-patterns; enforce 100-char budget - Trim descriptions in all commands/gsd/*.md files over 100 chars - Remove flag documentation from descriptions (belongs in argument-hint) - Remove Triggers: keyword stuffing - Add scripts/lint-descriptions.cjs — fails on descriptions > 100 chars - Add npm script: lint:descriptions - Add tests/enh-2789-description-budget.test.cjs Closes #2789 Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * docs(#2789): add CHANGELOG entry for description budget lint * docs(#2789): update COMMANDS.md descriptions; add skill description standards note Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
53 lines
2.0 KiB
Markdown
53 lines
2.0 KiB
Markdown
---
|
|
name: gsd:code-review-fix
|
|
description: Auto-fix issues found by code review in REVIEW.md; commits each fix atomically.
|
|
argument-hint: "<phase-number> [--all] [--auto]"
|
|
allowed-tools:
|
|
- Read
|
|
- Bash
|
|
- Glob
|
|
- Grep
|
|
- Write
|
|
- Edit
|
|
- Task
|
|
---
|
|
<objective>
|
|
Auto-fix issues found by code review. Reads REVIEW.md from the specified phase, spawns gsd-code-fixer agent to apply fixes, and produces REVIEW-FIX.md summary.
|
|
|
|
Arguments:
|
|
- Phase number (required) — which phase's REVIEW.md to fix (e.g., "2" or "02")
|
|
- `--all` (optional) — include Info findings in fix scope (default: Critical + Warning only)
|
|
- `--auto` (optional) — enable fix + re-review iteration loop, capped at 3 iterations
|
|
|
|
Output: {padded_phase}-REVIEW-FIX.md in phase directory + inline summary of fixes applied
|
|
</objective>
|
|
|
|
<execution_context>
|
|
@~/.claude/get-shit-done/workflows/code-review-fix.md
|
|
</execution_context>
|
|
|
|
<context>
|
|
Phase: $ARGUMENTS (first positional argument is phase number)
|
|
|
|
Optional flags parsed from $ARGUMENTS:
|
|
- `--all` — Include Info findings in fix scope. Default behavior fixes Critical + Warning only.
|
|
- `--auto` — Enable fix + re-review iteration loop. After applying fixes, re-run code-review at same depth. If new issues found, iterate. Cap at 3 iterations total. Without this flag, single fix pass only.
|
|
|
|
Context files (CLAUDE.md, REVIEW.md, phase state) are resolved inside the workflow via `gsd-sdk query init.phase-op` and delegated to agent via config blocks.
|
|
</context>
|
|
|
|
<process>
|
|
This command is a thin dispatch layer. It parses arguments and delegates to the workflow.
|
|
|
|
Execute the code-review-fix workflow from @~/.claude/get-shit-done/workflows/code-review-fix.md end-to-end.
|
|
|
|
The workflow (not this command) enforces these gates:
|
|
- Phase validation (before config gate)
|
|
- Config gate check (workflow.code_review)
|
|
- REVIEW.md existence check (error if missing)
|
|
- REVIEW.md status check (skip if clean/skipped)
|
|
- Agent spawning (gsd-code-fixer)
|
|
- Iteration loop (if --auto, capped at 3 iterations)
|
|
- Result presentation (inline summary + next steps)
|
|
</process>
|