Skip to content

Skip discussions in the issue_lock workflow - #98146

Merged
eps1lon merged 1 commit into
canaryfrom
sebbie/issue-lock-skip-discussions
Sep 1, 2026
Merged

Skip discussions in the issue_lock workflow#98146
eps1lon merged 1 commit into
canaryfrom
sebbie/issue-lock-skip-discussions

Conversation

@eps1lon

@eps1lon eps1lon commented Sep 1, 2026

Copy link
Copy Markdown
Member

The last remaining Lock Threads failures are likely due to missing permissions for discussions which dessant/lock-threads handles by default.

This disables discussion handling to get the workflow green to start tracking regressions. We can discuss if we want to lock inactive discussions in a follow-up (which would be new behavior this workflow never performed).

The `Lock Threads` workflow failed with `Resource not accessible by integration` once the `dessant/lock-threads` v6.0.2 upgrade fixed the token-length validation crash (https://github.com/vercel/next.js/actions/runs/33529115631). The action processes issues, pull requests, and discussions by default, and discussions are handled exclusively through GraphQL (`search(type: DISCUSSION)`, then `lockLockable`). The workflow token only has `issues: write` and `pull-requests: write`, so every discussion call is rejected and the job fails after issues and PRs have already been locked.

This change sets `process-only: 'issues, prs'` so the action never touches discussions, which this workflow was never meant to lock. Granting `discussions: write` would also fix the failure, but skipping discussions keeps the token permissions minimal and matches the workflow's original scope.

Co-Authored-By: Claude Code (kimi-k3[1m]) <noreply@anthropic.com>
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Tests Passed

Commit: 39c9983

@eps1lon eps1lon changed the title sebbie/issue lock skip discussions Skip discussions in the issue_lock workflow Sep 1, 2026
@eps1lon eps1lon changed the title Skip discussions in the issue_lock workflow Skip discussions in the issue_lock workflow Sep 1, 2026
@eps1lon
eps1lon marked this pull request as ready for review September 1, 2026 16:41
@eps1lon
eps1lon requested a review from timneutkens September 1, 2026 16:42
@eps1lon
eps1lon enabled auto-merge (squash) September 1, 2026 17:06
@eps1lon
eps1lon merged commit 52cd5b9 into canary Sep 1, 2026
327 of 332 checks passed
@eps1lon
eps1lon deleted the sebbie/issue-lock-skip-discussions branch September 1, 2026 18:06
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