Develop Issue
Quick Start
Invoke with exactly one same-repository GitHub issue reference:
text
/develop-issue #123
/develop-issue https://github.com/<owner>/<repo>/issues/123
This skill coordinates existing workflow skills. It does not replace their
contracts, loosen their guardrails, or merge pull requests.
Required Child Skills
Before branch setup or implementation, confirm these installed skills are
available in the agent environment:
If any are missing, halt before implementation. Report the missing skill names
and install guidance:
sh
npm_config_ignore_scripts=true npx skills@latest add patinaproject/skills --skill new-branch --skill review-code --skill finish-pr -y
npm_config_ignore_scripts=true npx skills@latest add mattpocock/skills@tdd -y
npm_config_ignore_scripts=true npx skills@latest add mattpocock/skills@diagnose -y
The
,
,
,
, and
install
hints intentionally track their source catalog's default branch. Consumers who
need a frozen install can add
to those sources.
Conditional Routes
Conditional routes are not blanket prerequisites. Check that the named skill is
available only when the issue triggers that route; halt with the missing skill
name and install guidance only for a triggered missing route.
Install guidance for triggered conditional routes:
sh
npm_config_ignore_scripts=true npx skills@latest add mattpocock/skills@write-a-skill -y
npm_config_ignore_scripts=true npx skills@latest add mattpocock/skills@zoom-out -y
npm_config_ignore_scripts=true npx skills@latest add mattpocock/skills@prototype -y
- Route through when the issue changes an installable skill
package surface: skill entry instructions, frontmatter or description,
workflow contract text, examples, reference material, or bundled helper
scripts. For skill-package changes that include executable helper scripts,
run before , then use for executable behavior.
- Use for ad-hoc, read-only discovery when the agent cannot yet
explain the relevant modules, callers, and domain vocabulary. It may run in a
background explorer when the host supports that, but the main workflow must
consume the result before choosing an implementation route.
- Use only when the issue explicitly asks for throwaway exploration,
state-model sanity checks, UI direction exploration, or equivalent prototype
work. Delete or absorb prototype output before local review unless the issue
explicitly asks to commit prototype artifacts.
Do not add normal
routes for upstream planning, triage,
architecture review, handoff, or conversation-mode skills unless the issue
explicitly asks for them.
Input Contract
- Accept one bare issue number, , or same-repository GitHub issue
URL.
- Reject missing issue references.
- Reject multiple issue references.
- Reject cross-repository issue URLs.
- Resolve the issue through the current working directory's default
repository.
- Treat the issue as prior approval for implementation only when acceptance
criteria, scope, repository rules, and design decisions are actionable.
Pause for a human when the issue lacks actionable acceptance criteria, conflicts
with repository rules, requires a design decision, depends on external access,
or otherwise needs judgment not recorded in the issue.
Workflow
- Read and if present, plus any docs they import.
- Validate the single same-repository issue reference and required child skills.
- Delegate branch setup to ; inherit every halt.
- Apply any triggered conditional route:
- Route installable skill package surface changes through .
- Use before implementation routing when discovery is needed.
- Use only for explicit throwaway exploration requests.
- Implement one behavior at a time through ; stays in the main
thread so the issue controller keeps ownership of implementation decisions.
- Route to when root cause is unclear, reproduction is missing,
behavior is flaky, or performance has regressed.
- Run repository-documented verification before local review.
- Check for reviewable local changes after verification: committed branch diff
from the default-branch merge base, staged changes, unstaged changes, or
untracked files.
- When reviewable changes exist, invoke and inherit its full
contract. owns fresh reviewer dispatch, isolation
requirements, read-only boundary, cleanup, and halt reporting.
Explicit use of is sufficient approval for this required
local review gate: dispatch the fresh read-only reviewer without asking for
another user confirmation. Preserve the boundary exactly:
no same-thread fallback and no file edits, staging, commits, pushes, PR
comments, review-thread mutation, or other worktree mutation. Halt if fresh
reviewer dispatch is unavailable or if reports a halt
condition.
When no reviewable changes exist, skip and report that no
local changes required review.
- Triage every local review finding with the router below.
- Repeat implementation, verification, reviewable-change detection, and
until no actionable local findings remain or a human-owned
blocker appears.
- Delegate final publishing and PR readiness to only after local
verification and are clean, skipped because no reviewable
local changes exist, or every local finding has a recorded
, , or disposition; inherit
every halt. Never merge the pull request.
Review Finding Router
Classify findings into exactly one of these outcomes:
| Outcome | Use When | Next Action |
|---|
| The expected behavior is clear or evidence can be gathered locally | Route clear behavior changes to ; route unclear root cause, missing reproduction, flaky behavior, or performance regression to |
| The finding needs judgment, external access, manual testing, design input, missing information, changed scope, product decisions, permissions, conflicting direction, or valid work outside the issue | Stop the loop and report the blocker with evidence checked |
| The finding is stale, incorrect, conflicts with repository rules, or is intentionally rejected | Explain politely in the report; add concise code comments only when future reviewers would otherwise re-raise the same concern |
There is no
state in v1. Insufficient information maps to
.
Final Report
When the workflow stops, report:
- Issue reference and URL
- Branch name
- Child skills invoked, with halt reason if any
- Verification commands and results
- Latest result, or that it was skipped because no reviewable
local changes existed
- Human-owned blockers, if any
- explanations, if any
- PR URL and readiness status, when runs