Skip to content

[safe-output-integrator] fix: use canonical dispatch-repository key in test fixture#44614

Merged
pelikhan merged 1 commit into
mainfrom
fix/dispatch-repository-canonical-key-d382bd6d0247976e
Jul 10, 2026
Merged

[safe-output-integrator] fix: use canonical dispatch-repository key in test fixture#44614
pelikhan merged 1 commit into
mainfrom
fix/dispatch-repository-canonical-key-d382bd6d0247976e

Conversation

@github-actions

@github-actions github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Summary

Fixes an incorrect key name in the test-copilot-dispatch-repository workflow test fixture. The safe-outputs key was using an underscore (dispatch_repository) instead of the canonical hyphen form (dispatch-repository) required by the safe-output schema.

Changes

  • pkg/cli/workflows/test-copilot-dispatch-repository.md: Renamed safe-outputs key from dispatch_repository to dispatch-repository

Details

  • Type: Bug fix (test fixture correction)
  • Scope: Single-line YAML frontmatter change in a workflow test file
  • The trigger_ci tool definition and all other workflow content are unchanged
  • Aligns the test fixture with the canonical dispatch-repository key format

Verification

No functional logic was changed. The fix ensures the test fixture correctly exercises the dispatch-repository safe-output type with its canonical key name.

Generated by PR Description Updater for #44614 · 22.8 AIC · ⌖ 4.34 AIC · ⊞ 4.7K ·

The test-copilot-dispatch-repository.md fixture used the deprecated
underscore form (dispatch_repository:) instead of the canonical
hyphenated key (dispatch-repository:).

Update the frontmatter to use the canonical form so the fixture is
consistent with current parser expectations and coverage checks.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@gh-aw-bot

Copy link
Copy Markdown
Collaborator

pr-sous-chef
@copilot please run the pr-finisher skill, ensure the branch is up to date, address any remaining review feedback, and rerun checks.

Run: https://github.com/github/gh-aw/actions/runs/29050994278

Generated by 👨‍🍳 PR Sous Chef · 9.81 AIC · ⌖ 6.1 AIC · ⊞ 7.1K ·
Comment /souschef to run again

Copilot AI commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

pr-sous-chef
@copilot please run the pr-finisher skill, ensure the branch is up to date, address any remaining review feedback, and rerun checks.
...

Done. I ran the PR-finisher pass on this branch and verified it is up to date with origin/main (ahead by one commit, not behind). There are no unresolved review threads to address, local checks passed (make fmt, make lint, make test-unit, make test, make recompile), and no additional code changes were needed beyond 719ce8e.

@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot please run the pr-finisher skill, address unresolved review comments, refresh the branch if needed, and rerun checks.

Generated by 👨‍🍳 PR Sous Chef · 7.75 AIC · ⌖ 9.11 AIC · ⊞ 7.1K ·
Comment /souschef to run again

Copilot AI commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

@copilot please run the pr-finisher skill, address unresolved review comments, refresh the branch if needed, and rerun checks.

...

Completed. I re-ran PR-finisher for this branch, confirmed there are no unresolved review threads/comments, verified the branch is up to date with origin/main (not behind), and reran checks locally (make fmt, make lint, make test-unit, make test, make recompile). No additional code changes were needed beyond 719ce8e.

@github-actions

Copy link
Copy Markdown
Contributor Author

🤖 PR Triage — Run §29061171191

Field Value
Category bug
Risk 🟢 Low
Score 40/100 (Impact 15 + Urgency 15 + Quality 10)
Action auto_merge

Canonical dispatch-repository key fix in test fixture. 1 file, 1 addition, 1 deletion. Minimal surgical fix, not a draft, no review required. Auto-merge when CI passes.

Generated by 🔧 PR Triage Agent · 155.8 AIC · ⌖ 8.06 AIC · ⊞ 5.4K ·

@gh-aw-bot

Copy link
Copy Markdown
Collaborator

@copilot please run the pr-finisher skill, address unresolved review comments, ensure the branch is up to date, and rerun checks.

Generated by 👨‍🍳 PR Sous Chef · 7.94 AIC · ⌖ 6.73 AIC · ⊞ 7.1K ·
Comment /souschef to run again

Copilot AI commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

@copilot please run the pr-finisher skill, address unresolved review comments, ensure the branch is up to date, and rerun checks.

...

Completed PR-finisher pass.

  • No unresolved in-scope review threads/comments were found.
  • The branch is up to date with origin/main (behind 0, ahead 1).
  • Local checks were rerun and passed: make fmt, make lint, make test-unit, make test, make recompile.
  • No additional code changes were needed beyond 719ce8e.

@gh-aw-bot

Copy link
Copy Markdown
Collaborator

pr-sous-chef
@copilot please run the pr-finisher skill, address unresolved review comments, ensure the branch is up to date, and rerun checks.

Run: https://github.com/github/gh-aw/actions/runs/29065468296

Generated by 👨‍🍳 PR Sous Chef · 9.71 AIC · ⌖ 6.72 AIC · ⊞ 7.1K ·
Comment /souschef to run again

Copilot AI commented Jul 10, 2026

Copy link
Copy Markdown
Contributor

pr-sous-chef
@copilot please run the pr-finisher skill, address unresolved review comments, ensure the branch is up to date, and rerun checks.
...

Completed another PR-finisher pass.

  • No unresolved in-scope review threads/comments were found.
  • Branch is up to date with origin/main (behind 0, ahead 1).
  • Local checks passed: make fmt, make lint, make test-unit, make test, make recompile.
  • Reviewed the referenced run 29065468296: completed successfully with no failed jobs.
  • No additional code changes were needed beyond 719ce8e.

@pelikhan pelikhan merged commit b4081bf into main Jul 10, 2026
4 checks passed
@pelikhan pelikhan deleted the fix/dispatch-repository-canonical-key-d382bd6d0247976e branch July 10, 2026 04:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants