Skip to content

chore(ci): drop secrets: inherit from PR verify#11

Open
apokusin wants to merge 1 commit into
mainfrom
chore/pr-drop-secrets-inherit
Open

chore(ci): drop secrets: inherit from PR verify#11
apokusin wants to merge 1 commit into
mainfrom
chore/pr-drop-secrets-inherit

Conversation

@apokusin
Copy link
Copy Markdown
Contributor

Summary

  • PR builds called verify.yml with secrets: inherit, exposing NPM_TOKEN and GPG_PRIVATE_KEY to same-repo branch PRs even though verify.yml never references them.
  • Motivated by the TanStack npm supply-chain compromise, where a malicious PR exfiltrated CI tokens to plant poisoned files used by the later release run.
  • Release (changesets.yml) is unaffected — it sets its own secrets.NPM_TOKEN / secrets.GPG_PRIVATE_KEY explicitly and runs only on push: main.

Test plan

  • Confirm the PR check (Verify) still passes on this PR — i.e. nothing in verify actually needed the inherited secrets.

🤖 Generated with Claude Code

The verify workflow does not reference any secrets, but inheriting them
exposed NPM_TOKEN and GPG_PRIVATE_KEY to PR builds from same-repo
branches — an unnecessary attack surface highlighted by the TanStack
npm supply-chain compromise (poisoned PR build exfiltrating release
tokens).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 12, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
moonwell-ai cec5076 May 12 2026, 04:54 PM

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 12, 2026

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
moonwell-api cec5076 May 12 2026, 04:54 PM

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