executing-github-task
Executing GitHub Task
This skill is the per-task execution orchestrator for the GitHub workflow. It does exactly three things: validate that the selected task is ready, dispatch the right specialist for each phase, and decide whether to advance, run a targeted fix cycle, or escalate.
Execution kickoff is the first point at which the workflow may mutate GitHub issue state or otherwise cross from critique-only work into active execution. Everything before kickoff remains critique and planning on disk; kickoff is the first execution mutation boundary after the user approves implementation. The orchestrator running this skill keeps only concise summaries in memory; subagents do the heavy work in isolation.
Inputs
| Input | Required | Example | Notes |
|---|---|---|---|
ISSUE_SLUG |
Yes | acme-app-42 |
Workflow key; derives artifact paths. |
TASK_NUMBER |
Yes | 3 |
Exactly one task per invocation. |
Required artifacts
| Artifact | Phase | Required | Purpose |
|---|---|---|---|
docs/<ISSUE_SLUG>.md |
1 | Yes | Issue snapshot and GitHub context. |
docs/<ISSUE_SLUG>-tasks.md |
2–4 | Yes | Task plan, ## GitHub Task Issues, statuses. |
docs/<ISSUE_SLUG>-task-<N>-brief.md |
5 | Yes | Scope, DoD, execution constraints. |
docs/<ISSUE_SLUG>-task-<N>-execution-plan.md |
5 | Yes | Approved implementation approach. |
docs/<ISSUE_SLUG>-task-<N>-test-spec.md |
5 | Yes | Required behavior coverage. |
docs/<ISSUE_SLUG>-task-<N>-refactoring-plan.md |
5 | Yes | Approved structural prep and cleanup. |
docs/<ISSUE_SLUG>-task-<N>-critique.md |
6 | Yes | Task-level critique record. |
docs/<ISSUE_SLUG>-task-<N>-decisions.md |
6 | Yes | Critique outcomes and confirmed decisions. |
Read ./references/contracts.md when validating task readiness or artifact
shape.
Workflow Overview
These are the execution skill's ordered internal stages. Read
./references/pipeline.md for the full run -> check -> fix -> re-check order
and ./references/contracts.md for readiness and handoff contracts.
| Internal stage | Goal | Primary result |
|---|---|---|
| 0. Readiness | Validate prerequisites and task readiness | Ready-to-run task or explicit blocker |
| 1. Kickoff | Apply first side effects and establish active execution state | KICKOFF_REPORT |
| 2. Execution | Implement the planned change | EXECUTION_REPORT |
| 3. Documentation | Add in-code docs, commit Category B work, and update tracking | DOCUMENTATION_REPORT |
| 4. Requirements Verification | Confirm Definition of Done coverage before review gates | VERIFICATION_RESULT |
| 5. Quality Gates | Run clean-code, architecture, and security review in order | Review verdicts and actionable feedback |
| 6. Targeted Fix Cycle | Re-run only the failing verification or review path | Re-validated task or escalation |
| 7. Final Report | Report the selected task's final outcome | One concise task completion summary |
Requirements gaps are resolved after internal step 4 and before internal step 5. Quality-gate fix cycles happen after internal step 5.
Subagent Registry
| Subagent | Path | Purpose |
|---|---|---|
execution-starter |
./subagents/execution-starter.md |
Execution kickoff: readiness, workspace checks, first gh-based GitHub updates (labels, assignees, comments, child-issue fields) when appropriate. |
task-executor |
./subagents/task-executor.md |
Implements the scoped change and tests from the approved planning artifacts. |
documentation-writer |
./subagents/documentation-writer.md |
In-code docs, commits Category B, updates docs/<ISSUE_SLUG>-tasks.md, optional gh completion updates on the task issue. |
requirements-verifier |
./subagents/requirements-verifier.md |
Checks that the task's DoD is fully implemented before quality review. |
clean-code-reviewer |
./subagents/clean-code-reviewer.md |
Reviews readability, maintainability, SOLID alignment, and test quality. |
architecture-reviewer |
./subagents/architecture-reviewer.md |
Reviews domain boundaries, composition, and architectural fit. |
security-auditor |
./subagents/security-auditor.md |
Audits the committed change set for exploitable security weaknesses. |
Use this registry as a lookup table. Read exactly one subagent definition per dispatch, then pass only the inputs that subagent needs.
How This Skill Works
The orchestrator uses direct reads only to load this skill, the reference file for the current phase, and the specific subagent it is about to dispatch. Everything else is delegated. Pass file paths and short summaries between subagents instead of raw file contents or command output.
In practice, the orchestrator does only three kinds of work directly: load the current instructions, dispatch the next specialist, and carry forward the smallest summary needed for the next decision. Execution, mutation, analysis, and artifact updates stay inside the specialist subagents.
Every task run follows the same validation loop: confirm readiness, cross the kickoff mutation boundary deliberately after critique approval, verify Definition of Done coverage before the review gates, and re-run only the failing phase when a targeted fix is needed.
Treat artifacts in two categories:
- Category A:
docs/<ISSUE_SLUG>*.md, progress files, briefs, plans, test specs, refactoring plans, critique, and decisions. Stay on disk, never committed, never deleted. - Category B: source code, tests, config, in-code documentation. Committed normally.
If a selected task is already complete, blocked by unmet prerequisites, or produces a repeated unresolved blocker, stop and report that state instead of forcing the pipeline forward.
Output Contract
After a successful run, this skill leaves behind these deliverables:
- Category B implementation artifacts: committed source code, tests, config changes, and in-code documentation.
- Category A orchestration artifacts: updated on disk but left uncommitted, including task status, implementation summary, file list, and optional GitHub tracking updates recorded in the plan.
- Kickoff summary: a returned
KICKOFF_REPORTcovering readiness, workspace state, and GitHub kickoff actions (or documented skips). - Task-only completion report: a concise user-facing report summarising the selected task's execution, commits, and gate verdicts.
Phase Guide
| When you need... | Read... |
|---|---|
| Artifact contracts and task readiness checks | ./references/contracts.md |
| Normal execution flow, kickoff, fix-loop order | ./references/pipeline.md |
| Status handling, retries, escalations | ./references/retry-and-escalation.md |
| Shared reviewer expectations | ./references/review-gate-policy.md |
Execution Steps
- Read
./references/contracts.mdand confirm the task is ready to cross the execution boundary. - Read
./references/pipeline.mdand follow its normal run order exactly. - Dispatch only the next required subagent, passing explicit inputs and keeping only structured summaries in orchestration context.
- When a blocker, missing prerequisite, or failing gate appears, read the
relevant recovery reference from
## Phase Guideand run only the targeted retry or escalation path. - Report the selected task's outcome once the pipeline finishes or stops. Do not auto-continue to the next task.
Operating Constraints
- Execute one task per invocation, then stop and wait for a new invocation for the next task.
- Keep the task plan as the source of truth. If execution reveals a plan change is needed, escalate instead of silently rewriting the plan.
- Preserve Category A artifacts on disk and out of git history.
- Keep fix cycles targeted. Re-run only the failing verification or review steps, not the entire pipeline.
- Treat missing required skills, missing tracker capability, or unresolved ambiguity as orchestration decisions. Surface them clearly and stop.
Example
Input:
ISSUE_SLUG=acme-app-42TASK_NUMBER=3
- Validate the required per-task artifacts exist; Task 3 is not already complete.
- Dispatch
execution-starterfor kickoff (firstghmutations after critique).KICKOFF_REPORT->READY
- Dispatch
task-executorwith artifact paths underdocs/acme-app-42-task-3-*.md.EXECUTION_REPORT->COMPLETE
- Dispatch
documentation-writerwithEXECUTION_REPORT,ISSUE_SLUG,TASK_NUMBER. - Dispatch
requirements-verifier.VERIFICATION_RESULT->PASS
- Run
clean-code-reviewer, thenarchitecture-reviewer, thensecurity-auditor.- All gates return a pass variant
- Report kickoff outcome, final verdicts, commits, files changed, and any skipped GitHub updates.
Input:
ISSUE_SLUG=acme-app-42TASK_NUMBER=3
execution-starterreturnsREADY.task-executorreturnsCOMPLETE.documentation-writerreturnsCOMPLETE.requirements-verifierreturnsFAILbecause one DoD item is untested.- Re-dispatch
task-executorwith only the verifier gap summary. - Re-dispatch
documentation-writerfor the new Category B delta. - Re-run
requirements-verifier.VERIFICATION_RESULT->PASS
- Continue into the review gates, and if one gate returns
NEEDS FIXES, re-run only that gate's targeted fix cycle instead of the whole pipeline.