Skip to content

fix: pass createCommitOnBranch files via graphql --input - #24

Merged
xeaser merged 2 commits into
mainfrom
fix/packaging-graphql-input
Aug 26, 2026
Merged

xeaser merged 2 commits into
mainfrom
fix/packaging-graphql-input

Conversation

@xeaser

@xeaser xeaser commented Aug 26, 2026

Copy link
Copy Markdown
Owner

What

Dry-run on chore/packaging-dry-run (--skip=publish, tag v0.6.0) showed gh -F additions=... does not send a GraphQL [FileAddition!] list.

This posts the mutation with gh api graphql --input so fileChanges.additions/deletions are real JSON arrays. Also validate the checked-out tag SHA (not GITHUB_SHA) so workflow_dispatch from a non-main ref still checks the tag.

Throwaway dry-run passed after enabling Actions to create PRs: https://github.com/xeaser/squad-opencode/actions/runs/32942440031
Opened and closed dry-run pin PR #23 (verified github-actions[bot] commit). Latest release remains v0.6.0.

How to test

  • Throwaway release.yml dispatch with --skip=publish + tag=v0.6.0
  • go test ./internal/version

Checklist

  • No secrets
  • Does not add Copilot, Ink shell, Aspire, or npm install

xeaser added 2 commits August 26, 2026 12:49
gh -F additions=JSON does not send a [FileAddition!] list, so the
packaging-bump mutation failed with Expected to be a key-value object.

Build the query and fileChanges with jq and post them with
gh api graphql --input. Check the checked-out tag SHA, not GITHUB_SHA,
so workflow_dispatch from a non-main ref still validates the tag.
upload-artifact@v4 and download-artifact@v4 still run on Node 20,
which GitHub hosted runners now warn about. v6/v7 default to Node 24.
@xeaser
xeaser merged commit 16e449a into main Aug 26, 2026
10 checks passed
@xeaser
xeaser deleted the fix/packaging-graphql-input branch August 26, 2026 07:31
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.

1 participant