Skip to content

chore: Refactor onboarding workflow to improve security and clarity. Changed…#84

Merged
witmicko merged 2 commits intomainfrom
migrate-onboarding-action-to-gh-app
Mar 12, 2026
Merged

chore: Refactor onboarding workflow to improve security and clarity. Changed…#84
witmicko merged 2 commits intomainfrom
migrate-onboarding-action-to-gh-app

Conversation

@witmicko
Copy link
Contributor

@witmicko witmicko commented Mar 12, 2026

… permissions to read for contents, updated steps to use generated GitHub App token for authentication, and enhanced variable handling for repository and branch detection.

ran successfully on previously failed repos:


Note

Medium Risk
Moderate risk because it changes authentication/permissions and default-branch detection for an automated workflow that writes to other repositories, which could cause onboarding failures if token scopes or outputs are miswired.

Overview
Refactors .github/workflows/onboard-new-repo.yml to reduce permissions (contents: read) and replace the shared ONBOARDING_TOKEN with a per-target GitHub App installation token for all gh/git operations.

The workflow now parses and validates org/repo inputs into explicit step outputs, adds a dedicated default-branch detection step, and updates downstream steps (opt-out check, checkout, empty-repo init, PR creation, and messaging) to consistently use the detected base branch and new token.

Written by Cursor Bugbot for commit 435d357. This will update automatically on new commits. Configure here.

… permissions to read for contents, updated steps to use generated GitHub App token for authentication, and enhanced variable handling for repository and branch detection.
@witmicko witmicko requested a review from a team as a code owner March 12, 2026 11:17
@witmicko witmicko requested a review from mmenoxx March 12, 2026 11:18
@witmicko witmicko changed the title Refactor onboarding workflow to improve security and clarity. Changed… chore: Refactor onboarding workflow to improve security and clarity. Changed… Mar 12, 2026
…y by grouping echo statements into a single block.
Copy link

@mmenoxx mmenoxx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@witmicko witmicko merged commit 834b682 into main Mar 12, 2026
22 of 23 checks passed
@witmicko witmicko deleted the migrate-onboarding-action-to-gh-app branch March 12, 2026 11: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