Skip to content

Allow custom sync workflow job IDs#60

Merged
adampoit merged 1 commit into
mainfrom
fix-58-custom-sync-job-id
Jul 19, 2026
Merged

Allow custom sync workflow job IDs#60
adampoit merged 1 commit into
mainfrom
fix-58-custom-sync-job-id

Conversation

@adampoit

Copy link
Copy Markdown
Owner

Summary

  • locate the sync workflow job by its patchlane sync command instead of requiring the fork-sync job ID
  • report clear errors when no sync job or multiple sync jobs are found
  • cover custom job IDs with direct GitHub App and wrapper token providers

Testing

  • npm test
  • npm run format:check

Closes #58

@not-adam

not-adam Bot commented Jul 19, 2026

Copy link
Copy Markdown
Contributor

Mira PR Walkthrough

Replaces the hard-coded fork-sync job ID lookup with a command-based job discovery that scans all jobs for a patchlane sync invocation. Adds inspectAuthenticatedCommandJob() to locate the sync job dynamically and reports clear errors when zero or multiple jobs match. Both direct GitHub App and wrapper token providers are covered with new tests.

Confidence: 5/5   ◉◉◉◉◉   Safe to merge
  • Well-scoped refactor: new helper and discovery function are additive, error handling covers empty/multiple matches, and the existing test suite is expanded with four targeted tests covering all code paths.

Blast Radius — 2 dependent repositories, 3 total references

adampoit/mira — 1 reference
adampoit/wezterm — 2 references

2 files reviewed


Comment @not-adam help to get the list of available commands and usage tips.

@adampoit
adampoit merged commit c8aab05 into main Jul 19, 2026
2 checks passed
@adampoit
adampoit deleted the fix-58-custom-sync-job-id branch July 19, 2026 01:20
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.

Allow adapted sync workflows to use a custom job ID

1 participant