Skip to content

fix(ci): grant build-phar caller permission to fix release-please startup failure#37

Merged
kojiromike merged 1 commit into
mainfrom
kojiromike/fix-build-phar-permissions
May 8, 2026
Merged

fix(ci): grant build-phar caller permission to fix release-please startup failure#37
kojiromike merged 1 commit into
mainfrom
kojiromike/fix-build-phar-permissions

Conversation

@kojiromike
Copy link
Copy Markdown
Contributor

Summary

Fixes startup failure on the Release Please workflow (run 25553201816) introduced by #28.

The workflow has permissions: {} at the top level. The build-phar caller-job had no permissions: block of its own, so it inherited empty permissions. The local reusable workflow build-phar.yml declares permissions: contents: write at workflow level. GitHub rejects the call at startup because the caller cannot grant permissions it doesn't have, producing STARTUP_FAILURE with no logs.

Adding permissions: contents: write at the build-phar caller-job level gives it permission to grant to the called workflow.

The release-please job already has its own permissions: block, so it wasn't affected.

Test plan

  • actionlint clean
  • After merge: Release Please workflow no longer startup-fails on push to main
  • When a real release goes out, build-phar fires and attaches the PHAR to the release

Workflow-level permissions:{} means the build-phar caller-job inherited
no permissions. The local reusable workflow build-phar.yml declares
permissions: contents: write at workflow level. GitHub rejects the call
at startup because the caller cannot grant permissions it doesn't have,
producing STARTUP_FAILURE with no logs (run 25553201816).

Adding permissions: contents: write at the build-phar caller-job level
gives it permission to grant to the called reusable workflow.

Assisted-by: Claude Code
@kojiromike kojiromike merged commit b017d95 into main May 8, 2026
7 checks passed
@kojiromike kojiromike deleted the kojiromike/fix-build-phar-permissions branch May 8, 2026 13:16
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