Execution Skill
This is the main orchestration skill that coordinates the entire fix implementation process after a plan has been approved.
Purpose
Execute approved fix plans by coordinating all other skills (git-operations, tdd-workflow, code-review) to implement, test, and commit the fix.
When to Use
- •After a plan has been approved via
@agent approvecomment - •When an execution task is received from the planning agent
- •For implementing any approved code changes
Cloud & Headless Support
When running in cloud environments (e.g., Docker, CI/CD) without browser access:
- •GitHub MCP: Configured to use HTTP mode with
GITHUB_TOKENto avoid OAuth browser flows.
Prerequisites
- •Approved PLAN.md exists in the PR
- •Repository access via Git
- •Test framework is configured in the project
Process
Step 1: Setup Workspace
Using git-operations skill:
- •Clone the repository (if not already cloned)
- •Checkout the PR branch
- •Pull latest changes
- •Verify clean working state
Step 2: Read and Parse Plan
- •Fetch PLAN.md from the PR
- •Parse implementation steps
- •Identify affected files
- •Identify test requirements
Step 3: Execute TDD Workflow
Using tdd-workflow skill:
RED Phase:
- •Write failing tests as specified in plan
- •Run tests to verify they fail
- •Commit tests:
test: add tests for [issue]
GREEN Phase:
- •Implement the fix as specified in plan
- •Make minimal changes to pass tests
- •Run tests to verify they pass
- •Commit fix:
fix: [description] ([issue-key])
Step 4: Verification
Using code-review skill:
- •Run full test suite
- •Run linters and type checks
- •Perform self-review of changes
- •Ensure no regressions
Step 5: Commit and Push
Using git-operations skill:
- •Stage all changes
- •Create commits with conventional messages
- •Push to PR branch
- •Verify push succeeded
Step 6: Update External Systems
- •Comment on PR with implementation summary
- •Mark PR as ready for review (remove draft)
- •Update Jira ticket status to "In Review"
- •Send Slack notification (if configured)
Output Format
json
{
"status": "success|failed",
"steps_completed": [
"workspace_setup",
"plan_parsed",
"tests_written",
"fix_implemented",
"tests_passing",
"code_reviewed",
"committed",
"pushed",
"jira_updated",
"slack_notified"
],
"commits": [
{
"sha": "abc123",
"message": "test: add tests for null check in auth service"
},
{
"sha": "def456",
"message": "fix: add null check for user session (PROJ-123)"
}
],
"test_results": {
"total": 150,
"passed": 150,
"failed": 0,
"skipped": 2
},
"pr_url": "https://github.com/org/repo/pull/123",
"errors": []
}
Error Handling
| Error | Action |
|---|---|
| Plan unclear | Comment on PR asking for clarification |
| Tests fail after fix | Report failure, do NOT force a fix |
| Merge conflicts | Report conflict, request human intervention |
| CI fails | Report CI failure with logs |
Important
- •Follow the plan exactly - don't improvise or add scope
- •If tests fail, report failure - don't force a fix
- •Use conventional commit messages -
fix:,test:,feat: - •Always run tests before marking complete
- •If blocked, report the blocker clearly