Skip to content

security: stricter pnpm config blockExoticSubdeps & trustPolicy - #10719

Merged
TkDodo merged 2 commits into
TanStack:mainfrom
Sheraff:stricter-pnpm-deps-config
May 22, 2026
Merged

security: stricter pnpm config blockExoticSubdeps & trustPolicy#10719
TkDodo merged 2 commits into
TanStack:mainfrom
Sheraff:stricter-pnpm-deps-config

Conversation

@Sheraff

@SheraffSheraff commented May 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Enable pnpm's no-downgrade trust policy.
  • Block exotic transitive dependencies with blockExoticSubdeps.
  • Remove the redundant danielroe/provenance-action downgrade CI job.

pnpm 11 compatibility is expected via #10699, which will be merged separately.

Summary by CodeRabbit

  • Chores
    • Added version preview information to pull requests for improved visibility into upcoming version changes.
    • Updated workspace configuration with enhanced dependency management policies for better security and consistency.

Review Change Stack

@Sheraff
Sheraff requested a review from a team as a code ownerMay 17, 2026 14:05
@coderabbitai

coderabbitaiBot commented May 17, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 56de6f07-532b-4ac5-96ac-4fe78174fc9e

📥 Commits

Reviewing files that changed from the base of the PR and between 98fd431 and 6756022.

📒 Files selected for processing (2)
  • .github/workflows/pr.yml
  • pnpm-workspace.yaml
💤 Files with no reviewable changes (1)
  • .github/workflows/pr.yml

📝 Walkthrough

Walkthrough

This PR updates repository configuration across two systems: the CI workflow replaces the provenance job with a version-preview job for displaying changeset information, and the pnpm workspace configuration adds dependency integrity constraints via blockExoticSubdeps and trustPolicy settings.

Changes

Repository Configuration Updates

Layer / File(s)Summary
CI Workflow: version-preview job replacement
.github/workflows/pr.yml
The provenance job is removed and replaced with a new version-preview job that runs the changeset-preview action with pull request and issue write permissions for displaying version-related preview information.
pnpm workspace dependency policies
pnpm-workspace.yaml
Workspace-level pnpm configuration adds blockExoticSubdeps: true and trustPolicy: 'no-downgrade' to enforce dependency integrity constraints across the monorepo.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A rabbit hops through config files so neat,
Removing provenance, version-preview's sweet,
Dependencies guarded with trust and care,
Workspace integrity blooms everywhere!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check nameStatusExplanationResolution
Description check⚠️ WarningThe PR description clearly summarizes the changes but does not follow the required template structure with mandatory sections like checklist and release impact.Restructure the description to include the required template sections: Changes section, Checklist with contributing guide and test confirmations, and Release Impact declaration.
✅ Passed checks (4 passed)
Check nameStatusExplanation
Title check✅ PassedThe title directly addresses the main changes: enabling stricter pnpm security configuration with blockExoticSubdeps and trustPolicy settings.
Docstring Coverage✅ PassedNo functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check✅ PassedCheck skipped because no linked issues were found for this pull request.
Out of Scope Changes check✅ PassedCheck skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@nx-cloud

nx-cloudBot commented May 17, 2026

Copy link
Copy Markdown

🤖 Nx Cloud AI Fix Eligible

An automatically generated fix could have helped fix failing tasks for this run, but Self-healing CI is disabled for this workspace. Visit workspace settings to enable it and get automatic fixes in future runs.

To disable these notifications, a workspace admin can disable them in workspace settings.


View your CI Pipeline Execution ↗ for commit 6756022

CommandStatusDurationResult
nx affected --targets=test:sherif,test:knip,tes...❌ Failed4m 25sView ↗
nx run-many --target=build --exclude=examples/*...✅ Succeeded1m 56sView ↗

☁️ Nx Cloud last updated this comment at 2026-05-22 16:26:28 UTC

@pkg-pr-new

pkg-pr-newBot commented May 17, 2026

Copy link
Copy Markdown
More templates

@tanstack/angular-query-experimental

npm i https://pkg.pr.new/@tanstack/angular-query-experimental@10719

@tanstack/eslint-plugin-query

npm i https://pkg.pr.new/@tanstack/eslint-plugin-query@10719

@tanstack/lit-query

npm i https://pkg.pr.new/@tanstack/lit-query@10719

@tanstack/preact-query

npm i https://pkg.pr.new/@tanstack/preact-query@10719

@tanstack/preact-query-devtools

npm i https://pkg.pr.new/@tanstack/preact-query-devtools@10719

@tanstack/preact-query-persist-client

npm i https://pkg.pr.new/@tanstack/preact-query-persist-client@10719

@tanstack/query-async-storage-persister

npm i https://pkg.pr.new/@tanstack/query-async-storage-persister@10719

@tanstack/query-broadcast-client-experimental

npm i https://pkg.pr.new/@tanstack/query-broadcast-client-experimental@10719

@tanstack/query-core

npm i https://pkg.pr.new/@tanstack/query-core@10719

@tanstack/query-devtools

npm i https://pkg.pr.new/@tanstack/query-devtools@10719

@tanstack/query-persist-client-core

npm i https://pkg.pr.new/@tanstack/query-persist-client-core@10719

@tanstack/query-sync-storage-persister

npm i https://pkg.pr.new/@tanstack/query-sync-storage-persister@10719

@tanstack/react-query

npm i https://pkg.pr.new/@tanstack/react-query@10719

@tanstack/react-query-devtools

npm i https://pkg.pr.new/@tanstack/react-query-devtools@10719

@tanstack/react-query-next-experimental

npm i https://pkg.pr.new/@tanstack/react-query-next-experimental@10719

@tanstack/react-query-persist-client

npm i https://pkg.pr.new/@tanstack/react-query-persist-client@10719

@tanstack/solid-query

npm i https://pkg.pr.new/@tanstack/solid-query@10719

@tanstack/solid-query-devtools

npm i https://pkg.pr.new/@tanstack/solid-query-devtools@10719

@tanstack/solid-query-persist-client

npm i https://pkg.pr.new/@tanstack/solid-query-persist-client@10719

@tanstack/svelte-query

npm i https://pkg.pr.new/@tanstack/svelte-query@10719

@tanstack/svelte-query-devtools

npm i https://pkg.pr.new/@tanstack/svelte-query-devtools@10719

@tanstack/svelte-query-persist-client

npm i https://pkg.pr.new/@tanstack/svelte-query-persist-client@10719

@tanstack/vue-query

npm i https://pkg.pr.new/@tanstack/vue-query@10719

@tanstack/vue-query-devtools

npm i https://pkg.pr.new/@tanstack/vue-query-devtools@10719

commit: 6756022

@TkDodo
TkDodo merged commit f0865fd into TanStack:mainMay 22, 2026
9 of 10 checks passed
@coderabbitaicoderabbitaiBot mentioned this pull request Jun 24, 2026
Sign up for freeto 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

@Sheraff@TkDodo