Add skill definitions for resolving-issues and general-audit#410
Merged
Conversation
Captures the prompts used to schedule recurring Claude web tasks so they live under version control. Seeds the directory with the master-orchestrator audit routine.
Routine targets a single repo when run, so the multi-repo orchestration caveats don't apply.
Move the audit prompt from docs/routines/ into .claude/skills/audit/SKILL.md so it can be invoked directly (`/audit` in Claude web or Code) instead of being copy-pasted. Drop docs/routines/ — future routines should use the same skill format.
Skill now files findings only — opens master + child issues, no auto-fix PRs and no closing of existing issues. Resolution will live in a separate routine. Renamed to general-audit to leave room for narrower audit skills.
Companion to general-audit. Coordinator-only skill that sweeps open issues and PRs, dispatches up to 3 parallel subagents in git worktrees to fix small-scope items, and drafts PRs with questions when scope is unclear. GH comms stay caveman; large features and refactors out of scope.
resolving-issues: cap 10 issues per run, sequential not parallel, fresh subagent + fresh worktree per issue. general-audit: require git worktrees in setup so each subagent runs isolated from main checkout. Caveman speech applied to resolving-issues body.
Caveman lives in its own plugin, not superpowers.
Sub-fixes land in a single long-running draft PR per run instead of opening N independent PRs against main. Implementer agent owns the full issue lifecycle: optional research subagents, worktree off master PR branch, sub-PR into master PR, CI watch, merge on green. Human reviews the master PR holistically and Fixes #N keywords there auto-close all linked issues on merge.
Lessons feedback now opens a dedicated issue per run instead of being appended to the report. Human edits the skill file directly to incorporate the suggestions, keeping the loop in version control.
Lists superpowers + caveman dependencies up front so the orchestrator loads them at run start rather than discovering them mid-flow.
Each /resolving-issues run creates its own master PR with a timestamped branch name. Avoids tangled state from prior runs and keeps each batch reviewable on its own.
End-of-run lessons land in the master PR body as a Lessons Learned section. Mirrors general-audit's lessons routine but writes to the PR instead of opening a separate issue, since the master PR is already the report for this routine. Also fixes a stale 'create or reuse' line that contradicted the no-reuse rule from the prior commit.
- resolving-issues: reorder Core Loop so master PR is only created when at least one in-scope issue exists, avoiding empty draft PRs on noop runs. Lessons Learned step explicit at end of run. - resolving-issues: 'drafts sub-PR' (which implied creating it) replaced with 'mark sub-PR as draft' (post-open) and 'park work as draft sub-PR' (mid-fix block, sub-PR may not exist yet). - general-audit: clarify 'last report' to 'most recent general-audit master issue' so the skip-if-same-commit check has a concrete anchor. - general-audit: drop redundant '/general-audit in PR' section; same trigger already covered in When to Use.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR introduces two new skill definition files that document the orchestration patterns and workflows for automated issue resolution and codebase auditing. These skills define how Claude agents coordinate to systematically address issues and identify technical debt.
Changes
.claude/skills/resolving-issues/SKILL.md— Defines the coordinator pattern for batch issue resolution:.claude/skills/general-audit/SKILL.md— Defines the orchestration pattern for comprehensive codebase audits:Notable Details
Both skills emphasize:
https://claude.ai/code/session_01VBGfb4shnH88vSKHdnHWTY