Skip to content

fix(ci): Use GitHub App token for draft PR enforcement#5871

Merged
stephanie-anderson merged 2 commits intomasterfrom
sanderson/use-gh-app-to-enforce-draft-prs
Mar 25, 2026
Merged

fix(ci): Use GitHub App token for draft PR enforcement#5871
stephanie-anderson merged 2 commits intomasterfrom
sanderson/use-gh-app-to-enforce-draft-prs

Conversation

@stephanie-anderson
Copy link
Contributor

The default GITHUB_TOKEN lacks permission for the GraphQL convertPullRequestToDraft mutation, causing the enforce-draft-pr workflow to fail with "Resource not accessible by integration".

This switches to the new SDK Maintainer Bot GitHub App token (SDK_MAINTAINER_BOT_APP_ID / SDK_MAINTAINER_BOT_PRIVATE_KEY) which has the required pull-requests write scope for GraphQL mutations.

The default GITHUB_TOKEN lacks permission for the GraphQL
convertPullRequestToDraft mutation ("Resource not accessible by
integration"). Switch to the SDK Maintainer Bot GitHub App token
which has the required pull-requests write scope.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Contributor

github-actions bot commented Mar 25, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

Langchain

  • Set gen_ai.operation.name and gen_ai.pipeline.name on LLM spans by ericapisani in #5849
  • Broaden AI provider detection beyond OpenAI and Anthropic by ericapisani in #5707
  • Update LLM span operation to gen_ai.generate_text by ericapisani in #5796

Bug Fixes 🐛

  • (ci) Use GitHub App token for draft PR enforcement by stephanie-anderson in #5871

Internal Changes 🔧

Langchain

  • Add text completion test by alexander-alderman-webb in #5740
  • Add tool execution test by alexander-alderman-webb in #5739
  • Add basic agent test with Responses call by alexander-alderman-webb in #5726
  • Replace mocks with httpx types by alexander-alderman-webb in #5724
  • Consolidate span origin assertion by alexander-alderman-webb in #5723
  • Consolidate available tools assertion by alexander-alderman-webb in #5721

Other

  • (asyncpg) Normalize query whitespace in integration by ericapisani in #5855
  • Add note to coordinate with assignee before PR submission by sentrivana in #5868
  • Temporarily stop running litellm tests by alexander-alderman-webb in #5851

Other

  • ci+docs: Add draft PR enforcement by stephanie-anderson in #5867

🤖 This preview updates automatically when you update the PR.

@stephanie-anderson stephanie-anderson marked this pull request as ready for review March 25, 2026 18:14
@stephanie-anderson stephanie-anderson requested a review from a team as a code owner March 25, 2026 18:14
@github-actions
Copy link
Contributor

github-actions bot commented Mar 25, 2026

Codecov Results 📊

13 passed | Total: 13 | Pass Rate: 100% | Execution Time: 7.60s

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 14389 uncovered lines.
✅ Project coverage is 30.24%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    25.32%    30.24%    +4.92%
==========================================
  Files          189       189         —
  Lines        20625     20625         —
  Branches      6738      6738         —
==========================================
+ Hits          5223      6236     +1013
- Misses       15402     14389     -1013
- Partials       429       474       +45

Generated by Codecov Action

Co-authored-by: sentry-warden[bot] <258096371+sentry-warden[bot]@users.noreply.github.com>
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Copy link
Contributor

@dingsdax dingsdax left a comment

Choose a reason for hiding this comment

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

🚢 it

@stephanie-anderson stephanie-anderson merged commit 11b8262 into master Mar 25, 2026
158 checks passed
@stephanie-anderson stephanie-anderson deleted the sanderson/use-gh-app-to-enforce-draft-prs branch March 25, 2026 18:35
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