Skip to content

refactor(apply-repo-settings): use github-app-auth + accept pre-gener… - #40

Merged
nsheaps merged 2 commits into
mainfrom
claude/practical-shannon-a2n2d
May 29, 2026
Merged

nsheaps merged 2 commits into
mainfrom
claude/practical-shannon-a2n2d

Conversation

@nsheaps

@nsheaps nsheaps commented May 29, 2026

Copy link
Copy Markdown
Owner

…ated token

Replace direct actions/create-github-app-token usage with the shared github-app-auth action (pinned SHA). This sets git user.name/email consistently with other nsheaps workflows (check.yaml, sync-files.yaml).

Add optional token input so callers that already authenticated via checkout-as-app can pass the pre-generated token instead of triggering a second auth round-trip. When token is provided, the auth step is skipped (if: inputs.token == ''). app-id remains required for the actor_id: -1 placeholder substitution in bypass_actors.

https://claude.ai/code/session_01XUJx6sKWxyKyoWdxNEsET1

…ated token

Replace direct actions/create-github-app-token usage with the shared
github-app-auth action (pinned SHA). This sets git user.name/email
consistently with other nsheaps workflows (check.yaml, sync-files.yaml).

Add optional `token` input so callers that already authenticated via
checkout-as-app can pass the pre-generated token instead of triggering
a second auth round-trip. When token is provided, the auth step is
skipped (if: inputs.token == ''). app-id remains required for the
actor_id: -1 placeholder substitution in bypass_actors.

https://claude.ai/code/session_01XUJx6sKWxyKyoWdxNEsET1
@nsheaps
nsheaps enabled auto-merge (squash) May 29, 2026 02:26
Checkov CKV2_GHA_1 flags reusable workflows (workflow_call) that have no
top-level permissions block, because without it the workflow inherits the
caller's full token permissions. Add permissions: {} at the workflow level
to restrict defaults; the sync-specs job already has permissions: contents:
write to grant only what it needs.

https://claude.ai/code/session_01XUJx6sKWxyKyoWdxNEsET1
@nsheaps
nsheaps merged commit 23d303b into main May 29, 2026
2 checks passed
@nsheaps
nsheaps deleted the claude/practical-shannon-a2n2d branch May 29, 2026 02:37
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