Skip to content

chore: use OPENHANDS_BOT_GITHUB_PAT_PUBLIC in sync workflows#473

Merged
all-hands-bot merged 2 commits intomainfrom
chore/use-openhands-bot-github-pat-public
Apr 23, 2026
Merged

chore: use OPENHANDS_BOT_GITHUB_PAT_PUBLIC in sync workflows#473
all-hands-bot merged 2 commits intomainfrom
chore/use-openhands-bot-github-pat-public

Conversation

@simonrosenberg
Copy link
Copy Markdown
Collaborator

Part of OpenHands/evaluation#428 (PAT blast-radius reduction).

Replaces secrets.PAT_TOKENsecrets.OPENHANDS_BOT_GITHUB_PAT_PUBLIC in the two auto-approve workflows:

  • .github/workflows/sync-docs-code-blocks.yml
  • .github/workflows/sync-agent-sdk-openapi.yml

The OPENHANDS_BOT_GITHUB_PAT_PUBLIC token is a fine-grained PAT scoped only to public OpenHands repos with the narrowest permissions needed to approve these bot-authored PRs — tighter than the shared PAT_TOKEN it replaces.

Prerequisites

  • OpenHands/docs added to the OPENHANDS_BOT_GITHUB_PAT_PUBLIC fine-grained PAT repo allowlist
  • OPENHANDS_BOT_GITHUB_PAT_PUBLIC secret added to OpenHands/docs repository secrets

@mintlify
Copy link
Copy Markdown

mintlify Bot commented Apr 23, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
all-hands-ai 🟢 Ready View Preview Apr 23, 2026, 7:31 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

Copy link
Copy Markdown
Contributor

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

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

Taste Rating: 🟢 Good taste - Clean, focused security improvement

[RISK ASSESSMENT]

  • [Overall PR] ⚠️ Risk Assessment: 🟢 LOW

This is a workflow-only change that improves security by replacing a shared PAT with a fine-grained token. Defensive conditionals ensure graceful degradation if the secret is missing. No code logic changes, easily revertible.

VERDICT:
⏸️ Ready to merge after prerequisites - Code changes are correct, but verify prerequisites are met before merging

KEY INSIGHT:
Textbook example of reducing security blast radius through fine-grained permissions - clean implementation with defensive fallbacks.

Comment thread .github/workflows/sync-agent-sdk-openapi.yml
@all-hands-bot all-hands-bot requested a review from enyst April 23, 2026 19:38
@all-hands-bot all-hands-bot self-assigned this Apr 23, 2026
Copy link
Copy Markdown
Collaborator

@enyst enyst left a comment

Choose a reason for hiding this comment

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

Let's do it!

@all-hands-bot all-hands-bot merged commit 54c78d6 into main Apr 23, 2026
7 checks passed
@all-hands-bot all-hands-bot deleted the chore/use-openhands-bot-github-pat-public branch April 23, 2026 19:41
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.

3 participants