Skip to content

fix(github): Preserve installation permissions on write tokens#903

Merged
dcramer merged 1 commit into
mainfrom
fix/github-write-credential-envelope
Jul 14, 2026
Merged

fix(github): Preserve installation permissions on write tokens#903
dcramer merged 1 commit into
mainfrom
fix/github-write-credential-envelope

Conversation

@dcramer

@dcramer dcramer commented Jul 14, 2026

Copy link
Copy Markdown
Member

GitHub App write leases now inherit the installation's complete permission
envelope while remaining scoped to the target repository. Previously,
appPermissions could downscope installation-write tokens, so a push could
succeed while a related pull request operation failed with an incomplete token.

The write credential input is now role-shaped: repository-scoped writes cannot
include a permissions body, while read leases retain explicit downscoping.
Local raw issue and pull request creation denials now identify themselves as
Junior routing failures, and the docs, skills, eval, and regression coverage
reflect automatic credentials. The obsolete split-grant OpenSpec plan is
removed so it cannot reintroduce the retired model.

Repository write leases could still be downscoped by appPermissions,
recreating incomplete tokens across related push and pull request operations.
Make the write issuance variant repository-only so it cannot include an
operation-specific permission body.

Align routing guidance, skills, docs, and regression coverage with automatic
GitHub App credentials. Remove the completed plan for the retired split-grant
model.

Co-Authored-By: OpenAI GPT-5 <noreply@openai.com>
@vercel

vercel Bot commented Jul 14, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
junior-docs Ready Ready Preview Jul 14, 2026 4:17pm

Request Review

@dcramer dcramer marked this pull request as ready for review July 14, 2026 16:23
@github-actions github-actions Bot added the risk: medium PR risk score: medium label Jul 14, 2026
@dcramer dcramer merged commit 3350d53 into main Jul 14, 2026
34 checks passed
@dcramer dcramer deleted the fix/github-write-credential-envelope branch July 14, 2026 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: medium PR risk score: medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant