Skip to content

fix(finish): fall back to GitHubService for PRs on Linear/Jira projects#983

Merged
acreeger merged 1 commit into
mainfrom
fix/finish-pr-linear-fallback
Apr 17, 2026
Merged

fix(finish): fall back to GitHubService for PRs on Linear/Jira projects#983
acreeger merged 1 commit into
mainfrom
fix/finish-pr-linear-fallback

Conversation

@acreeger
Copy link
Copy Markdown
Collaborator

Summary

  • il finish <pr-number> on a Linear-configured project failed with Issue tracker does not support pull requests because finish guarded the PR path on issueTracker.supportsPullRequests.
  • For projects that track issues in Linear/Jira but still use GitHub as the VCS, PRs live on GitHub — not on the issue tracker.
  • Mirrored the fallback pattern already used by StartCommand: when the configured issue tracker doesn't support PRs, fetch the PR via a locally-instantiated GitHubService.
  • Tightened numeric validation on the parsed PR number in both code paths.

Test plan

  • New unit tests in finish.pr-workflow.test.ts cover both entry points on Linear projects:
    • il finish 4008 (explicit numeric identifier detected as PR via worktree suffix)
    • il finish --pr 4008 (explicit PR flag)
  • Full finish suite passes (146/146)
  • Verified live on the Linear-configured project that originally failed

`il finish <pr-number>` on a Linear-configured project failed with
"Issue tracker does not support pull requests" because the finish command
checked `issueTracker.supportsPullRequests` to decide whether to fetch a PR.
For projects that use Linear or Jira for issue tracking but GitHub as the
VCS, PRs live on GitHub — not on the issue tracker.

Mirror the fallback pattern already used in `start.ts`: when the configured
issue tracker doesn't support PRs, fetch the PR via a locally-instantiated
`GitHubService` instead.
@acreeger acreeger merged commit a7ebae2 into main Apr 17, 2026
4 checks passed
@acreeger acreeger deleted the fix/finish-pr-linear-fallback branch April 17, 2026 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant