Skip to content

GH-2211: Wire native sub-issue linking into epic decomposition (internal/executor/)#2230

Merged
alekspetrov merged 1 commit intomainfrom
pilot/GH-2211
Apr 7, 2026
Merged

GH-2211: Wire native sub-issue linking into epic decomposition (internal/executor/)#2230
alekspetrov merged 1 commit intomainfrom
pilot/GH-2211

Conversation

@alekspetrov
Copy link
Copy Markdown
Collaborator

Summary

Automated PR created by Pilot for task GH-2211.

Closes #2211

Changes

GitHub Issue #2211: Wire native sub-issue linking into epic decomposition (internal/executor/)

Parent: GH-2209

In createSubIssuesViaGitHub() (epic.go ~line 556), after the gh issue create succeeds and the issue number is parsed:

  • Instantiate or receive a *github.Client — Runner currently has no ghClient field. Add one, initialized from the GitHub adapter token already available in config. Alternatively, accept a SubIssueLinker interface with just LinkSubIssue to keep the dependency narrow.
  • Call LinkSubIssue(ctx, owner, repo, parentNum, childNum) with a warn-level log on failure (non-fatal — the text marker remains as fallback).
  • Extract owner/repo from the execution context or executionPath (the worktree's git remote). Check how the existing code resolves these — plan.ParentTask.SourceIssueID and adapter metadata likely have them.
  • Keep the existing Parent: GH-N body text intact for backwards compatibility.
    Files: internal/executor/epic.go, internal/executor/runner.go (add field + constructor wiring)

…GH-2211)

Add SubIssueLinker interface to Runner. After each gh issue create in
createSubIssuesViaGitHub(), call LinkSubIssue(owner, repo, parentNum, childNum)
to establish the native GitHub parent→child relationship. Failures are
warn-level and non-fatal — the "Parent: GH-N" body text remains as fallback.
Wire *github.Client as the linker in main.go alongside the existing merge-wait setup.
@alekspetrov alekspetrov merged commit d026471 into main Apr 7, 2026
4 checks passed
@alekspetrov alekspetrov deleted the pilot/GH-2211 branch April 7, 2026 13:05
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.

Wire native sub-issue linking into epic decomposition (internal/executor/)

1 participant