Skip to content

Use pinned bun binary for post-steps when allowed_non_write_users is set#1190

Open
OctavianGuzu wants to merge 1 commit intomainfrom
oct/pin-bun-for-post-steps
Open

Use pinned bun binary for post-steps when allowed_non_write_users is set#1190
OctavianGuzu wants to merge 1 commit intomainfrom
oct/pin-bun-for-post-steps

Conversation

@OctavianGuzu
Copy link
Copy Markdown
Collaborator

Copies bun into $GITHUB_ACTION_PATH/bin before the claude step runs, then uses that copy in the two post-steps (Cleanup SSH signing key, Post buffered inline comments). Falls back to the PATH-resolved bun when allowed_non_write_users is unset, so this is a no-op for the default configuration.

@OctavianGuzu OctavianGuzu force-pushed the oct/pin-bun-for-post-steps branch from 17ddc03 to 4a3c115 Compare April 7, 2026 10:00
@OctavianGuzu OctavianGuzu marked this pull request as ready for review April 7, 2026 10:02
@OctavianGuzu OctavianGuzu requested a review from ashwin-ant April 7, 2026 10:06
Copies the bun binary into $GITHUB_ACTION_PATH/bin before the claude step
runs and uses that copy in the two post-steps that invoke bun. Falls back
to PATH-resolved bun when allowed_non_write_users is empty.

:house: Remote-Dev: homespace
@OctavianGuzu OctavianGuzu force-pushed the oct/pin-bun-for-post-steps branch from 4a3c115 to fdeb85f Compare April 7, 2026 10:13
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.

2 participants