Skip to content

Replace implement-epic-sequence with implement-epic - #21

Merged
shaug merged 1 commit into
mainfrom
scott/issue-19-implement-epic
Jul 20, 2026
Merged

Replace implement-epic-sequence with implement-epic#21
shaug merged 1 commit into
mainfrom
scott/issue-19-implement-epic

Conversation

@shaug

@shaug shaug commented Jul 20, 2026

Copy link
Copy Markdown
Owner

TL;DR

Replaces implement-epic-sequence with a smaller implement-epic orchestrator that delegates every selected child to the canonical implement-ticket workflow.

Summary

  • Rename the public epic skill to implement-epic and remove the unsupported former name after confirming there are no repository-owned compatibility consumers.
  • Reduce epic execution to live graph discovery, child selection, one implement-ticket invocation per selected child, terminal-result verification, graph refresh, and separately authorized closeout.
  • Keep single-ticket readiness, implementation, validation, review, PR, merge, transition, and cleanup mechanics under implement-ticket.
  • Split GitHub and Linear references along graph/parent versus ticket/PR ownership boundaries.
  • Add contract tests and forward-evaluation scenarios for result handling, scope, authority, isolation, graph refresh, safe parallelism, and conservative closeout.

Why

The prior skill duplicated the single-ticket workflow and encoded its graph-walking strategy in the public name. This composition creates one canonical execution layer, makes the epic contract easier to audit, and preserves dependency and authority gates without introducing a workflow framework or compatibility alias.

Validation

  • just format
  • just lint
  • just test
  • just test-implement-epic
  • just test-implement-ticket
  • Skill validation for skills/implement-epic
  • Fresh-context forward scenarios covering graph refresh, terminal states, scope boundaries, mutation isolation, missing dependencies, late feedback, and umbrella closeout
  • Fresh repository-owned aggregate review: clean across solution simplicity, correctness, and code simplicity

Tickets

Fixes #19

## Summary

- Replace `implement-epic-sequence` with the canonical `implement-epic` skill
- Delegate one-ticket execution and review mechanics to `implement-ticket`
- Add focused graph, result, authority, closeout, and forward-evaluation contracts

## Why

- Keep epic orchestration focused on live dependency selection and refresh
- Establish one canonical owner for ticket implementation, review, merge, and cleanup
- Preserve scope and authority boundaries while allowing future safe parallel execution
@shaug
shaug marked this pull request as ready for review July 20, 2026 14:06
@shaug
shaug merged commit 0700b1f into main Jul 20, 2026
1 check passed
@shaug
shaug deleted the scott/issue-19-implement-epic branch July 20, 2026 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace implement-epic-sequence with implement-epic as an implement-ticket orchestrator

1 participant