Skip to content

docs: document workflow OAuth scope requirement for sl pr submit#1299

Open
Marukome0743 wants to merge 1 commit into
facebook:mainfrom
Marukome0743:pr1299
Open

docs: document workflow OAuth scope requirement for sl pr submit#1299
Marukome0743 wants to merge 1 commit into
facebook:mainfrom
Marukome0743:pr1299

Conversation

@Marukome0743
Copy link
Copy Markdown

The default scopes requested by gh auth login (repo, read:org,
gist) do not include workflow. As a result, sl pr submit aborts
with an UNAUTHORIZED-WRITE error from the GraphQL mergeBranch
mutation whenever a commit touches .github/workflows/. Because the
SSH push and PR creation succeed first, the PR exists by the time the
command exits non-zero, which makes the failure mode hard to debug.

Add a :::note near the auth instructions pointing at
gh auth refresh -s workflow, and a Troubleshooting subsection that
walks through the full error and the order of operations. Cross-link
issue #787, which is the same root cause surfaced via HTTPS push.

The default scopes requested by `gh auth login` (`repo`, `read:org`,
`gist`) do not include `workflow`. As a result, `sl pr submit` aborts
with an `UNAUTHORIZED-WRITE` error from the GraphQL `mergeBranch`
mutation whenever a commit touches `.github/workflows/`. Because the
SSH push and PR creation succeed first, the PR exists by the time the
command exits non-zero, which makes the failure mode hard to debug.

Add a `:::note` near the auth instructions pointing at
`gh auth refresh -s workflow`, and a Troubleshooting subsection that
walks through the full error and the order of operations. Cross-link
issue facebook#787, which is the same root cause surfaced via HTTPS push.
@meta-cla meta-cla Bot added the CLA Signed label May 8, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented May 8, 2026

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this in D104483725. (Because this pull request was imported automatically, there will not be any future comments.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant