Skip to content

refactor: use reusable-dispatch with pre-computed matrix - #5

Merged
ralphbean merged 1 commit into
mainfrom
use-reusable-dispatch-matrix
Aug 21, 2026
Merged

ralphbean merged 1 commit into
mainfrom
use-reusable-dispatch-matrix

Conversation

@ralphbean

Copy link
Copy Markdown
Owner

Summary

Updates the Jira poll workflow to use reusable-dispatch.yml with the new matrix input (fullsend-ai/fullsend#6455) instead of the extracted reusable-harness-run.yml workflow.

Why This Change?

The original approach (PR fullsend-ai/fullsend#6356) extracted harness-run into a separate workflow, which conflicted with ADR 62's decision to inline stage workflows to avoid version skew.

The new approach (PR fullsend-ai/fullsend#6455):

  • ✅ Maintains ADR 62's inlining decision
  • ✅ No version skew introduced
  • ✅ Single entry point for both standard and custom polling flows
  • ✅ Simpler - uses the same workflow everyone else uses, just with different inputs

Changes

Workflow:

  • Renamed harness-run job to harness for clarity
  • Changed from reusable-harness-run.yml@extract-harness-run to reusable-dispatch.yml@reusable-dispatch-precomputed-matrix
  • Removed unnecessary inputs (install_mode, fullsend_version, runner_image) - using defaults
  • Cleaner comments explaining the flow

Plan document:

  • Updated Step 4 to describe the new reusable-dispatch.yml approach
  • Added explanation of how pre-computed matrix maintains ADR 62 compliance

Test Plan

  • Verify poll job builds matrix correctly
  • Confirm harness job invokes agents via reusable-dispatch.yml
  • Validate Jira secrets are properly passed through
  • Test with a real Jira issue

Related

🤖 Generated with Claude Code

Update the Jira poll workflow to use reusable-dispatch.yml with the new
matrix input (PR fullsend-ai/fullsend#6455) instead of the extracted
reusable-harness-run.yml workflow.

This approach:
- Maintains ADR 62's inlining decision (no version skew)
- Provides a single entry point for both standard and custom polling flows
- Simplifies the workflow by removing unnecessary input parameters

Changes:
- Rename harness-run job to harness for clarity
- Update to use reusable-dispatch.yml@reusable-dispatch-precomputed-matrix
- Remove install_mode, fullsend_version, and runner_image inputs (use defaults)
- Update plan document to describe the new approach

Assisted-by: Claude Sonnet 4.5 <noreply@anthropic.com>
Signed-off-by: Ralph Bean <rbean@redhat.com>
@ralphbean
ralphbean merged commit 8a0625d into main Aug 21, 2026
9 of 10 checks passed
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.

Extract harness-run from reusable-dispatch.yml into a reusable action

1 participant