Skip to content

Comments

chore: update pnpm settings#61

Merged
witoszekdev merged 2 commits intomainfrom
update-pnpm-settings
Jan 23, 2026
Merged

chore: update pnpm settings#61
witoszekdev merged 2 commits intomainfrom
update-pnpm-settings

Conversation

@witoszekdev
Copy link
Member

Summary

  • Update pnpm to version 10.28.1
  • Add pnpm-workspace.yaml security settings:
    • blockExoticSubdeps: true
    • minimumReleaseAge: 1440 (24h)
    • trustPolicy: no-downgrade

@vercel
Copy link

vercel bot commented Jan 21, 2026

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

Project Deployment Review Updated (UTC)
dummy-payment-app Ready Ready Preview, Comment Jan 22, 2026 3:45pm

Request Review

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the project's pnpm package manager configuration and fixes a typo in import paths. The changes include upgrading pnpm from version 9.2.0 to 10.28.1, adding security-focused workspace settings, correcting the filename from create-graphq-client.ts to create-graphql-client.ts, and adding project documentation files for AI coding agents.

Changes:

  • Updated pnpm version from 9.2.0 to 10.28.1 with new security settings in pnpm-workspace.yaml
  • Fixed typo in import paths: create-graphq-clientcreate-graphql-client
  • Added AGENTS.md documentation and supporting files (CLAUDE.md, .gitallowed)

Reviewed changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
package.json Updated packageManager field to pnpm@10.28.1
pnpm-workspace.yaml Added security settings: blockExoticSubdeps, minimumReleaseAge, and trustPolicy
src/lib/create-graphql-client.ts New file with corrected spelling (previously create-graphq-client.ts)
src/server/procedure/procedure-with-graphql-client.ts Updated import path to use corrected filename
src/providers/GraphQLProvider.tsx Updated import path to use corrected filename
AGENTS.md Added comprehensive project documentation for AI coding agents
CLAUDE.md Added reference file pointing to AGENTS.md
.gitallowed Added configuration to allow AGENTS.md

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

"zod": "3.23.8"
},
"packageManager": "pnpm@9.2.0",
"packageManager": "pnpm@10.28.1",
Copy link

Copilot AI Jan 21, 2026

Choose a reason for hiding this comment

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

The packageManager field specifies pnpm version 10.28.1, but the engines.pnpm field in package.json still requires >=9. Consider updating the engines.pnpm field to >=10 or >=10.28.1 to ensure consistency with the packageManager version and prevent potential version mismatches.

Copilot uses AI. Check for mistakes.
lkostrowski
lkostrowski previously approved these changes Jan 22, 2026
@witoszekdev
Copy link
Member Author

Fixed build issue by enabling corepack: https://vercel.com/docs/builds/configure-a-build#corepack

@witoszekdev witoszekdev merged commit 2587af6 into main Jan 23, 2026
8 checks passed
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