Skip to content

fix(sdk): pass VERCEL_OIDC_TOKEN into Connect getToken - #107

Merged
HugoRCD merged 1 commit into
mainfrom
fix/connect-oidc-env-token
Aug 26, 2026
Merged

fix(sdk): pass VERCEL_OIDC_TOKEN into Connect getToken#107
HugoRCD merged 1 commit into
mainfrom
fix/connect-oidc-env-token

Conversation

@HugoRCD

@HugoRCD HugoRCD commented Aug 26, 2026

Copy link
Copy Markdown
Member

Summary

  • connectGithubToken forwards VERCEL_OIDC_TOKEN to Connect as vercelToken (explicit connectOptions.vercelToken still wins).
  • That skips @vercel/oidc's .vercel project-root walk, which fails inside eve/workflow snapshots (Unable to find project root directory).
  • Write-scoped mint (e.g. createIssue) was hitting this after a successful read, because Connect cache keys differ by scopes.

Stacked on #106.

Test plan

  • pnpm --filter @github-tools/sdk test (connect token tests)
  • In examples/eve with a pulled .env: createIssue after listIssues no longer throws VercelOidcTokenError about project root
  • Re-run vercel env pull if the OIDC token is expired (Connect will then 401 instead of looking for .vercel)

@vercel

vercel Bot commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
github-tools-docs Ready Ready Preview, v0 Aug 26, 2026 9:38am
github-tools-test-agent Ready Ready Preview, v0 Aug 26, 2026 9:38am

@github-actions

Copy link
Copy Markdown
Contributor

Thank you for following the naming conventions! 🙏

Workflow snapshots have no .vercel directory, so @vercel/oidc's project-root refresh fails on write-scoped mint. Forward the pulled env token as vercelToken instead.
@HugoRCD
HugoRCD force-pushed the fix/connect-oidc-env-token branch from db513b7 to df2d477 Compare August 26, 2026 09:38
@HugoRCD HugoRCD self-assigned this Aug 26, 2026
@HugoRCD
HugoRCD merged commit 2edd9ea into main Aug 26, 2026
14 checks passed
@HugoRCD
HugoRCD deleted the fix/connect-oidc-env-token branch August 26, 2026 09:41
@github-actions github-actions Bot mentioned this pull request Aug 26, 2026
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.

1 participant