chore(deps): Upgrade vulnerable dependencies#872
Conversation
🦋 Changeset detectedLatest commit: bcd1d3d The changes in this PR will be included in the next version bump. This PR includes changesets to release 24 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
📝 WalkthroughWalkthroughDependency and tooling bumps across the monorepo: AWS S3 SDKs to 3.1020.0, Fastify to 5.8.4, Prisma packages to 7.6.0, YAML to 2.8.3, pnpm to 10.33.0; Claude skill frontmatter renamed to kebab-case; runner environment handling adjusted to pass a controlled base env to child processes. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Deploying baseplate-storybook with
|
| Latest commit: |
b4ea38c
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://49c730c7.baseplate-storybook.pages.dev |
| Branch Preview URL: | https://kingston-upgrade-vulnerable.baseplate-storybook.pages.dev |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
There was a problem hiding this comment.
🧹 Nitpick comments (1)
packages/project-builder-dev/src/e2e-runner/environment.ts (1)
16-41: Consider addingTMPDIRto passthrough variables.The list includes Windows temp directory variables (
TMP,TEMP) but omitsTMPDIR, which is the POSIX standard used by macOS and many Linux tools. Package managers, native module builds, and various tools rely onTMPDIRto locate the temporary directory.Proposed fix
const PASSTHROUGH_ENV_VARS = [ // Essential for package management 'HOME', 'PATH', 'NODE_OPTIONS', 'COREPACK_HOME', 'TMP', 'TEMP', + 'TMPDIR', 'APPDATA',🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@packages/project-builder-dev/src/e2e-runner/environment.ts` around lines 16 - 41, PASSTHROUGH_ENV_VARS is missing the POSIX temp var TMPDIR; update the PASSTHROUGH_ENV_VARS array to include 'TMPDIR' alongside 'TMP' and 'TEMP' so macOS/Linux tools and native builds that rely on TMPDIR inherit the temp directory value when spawning e2e processes.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@packages/project-builder-dev/src/e2e-runner/environment.ts`:
- Around line 16-41: PASSTHROUGH_ENV_VARS is missing the POSIX temp var TMPDIR;
update the PASSTHROUGH_ENV_VARS array to include 'TMPDIR' alongside 'TMP' and
'TEMP' so macOS/Linux tools and native builds that rely on TMPDIR inherit the
temp directory value when spawning e2e processes.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: f6347c8c-7af5-403a-81c7-d751210f8e9e
⛔ Files ignored due to path filters (11)
examples/blog-with-auth/apps/admin/baseplate/generated/package.jsonis excluded by!**/generated/**,!**/generated/**examples/blog-with-auth/apps/backend/baseplate/generated/package.jsonis excluded by!**/generated/**,!**/generated/**examples/blog-with-auth/baseplate/generated/package.jsonis excluded by!**/generated/**,!**/generated/**examples/blog-with-auth/libs/transactional/baseplate/generated/package.jsonis excluded by!**/generated/**,!**/generated/**examples/todo-with-better-auth/apps/admin/baseplate/generated/package.jsonis excluded by!**/generated/**,!**/generated/**examples/todo-with-better-auth/apps/backend/baseplate/generated/package.jsonis excluded by!**/generated/**,!**/generated/**examples/todo-with-better-auth/apps/web/baseplate/generated/package.jsonis excluded by!**/generated/**,!**/generated/**examples/todo-with-better-auth/baseplate/generated/package.jsonis excluded by!**/generated/**,!**/generated/**examples/todo-with-better-auth/libs/transactional/baseplate/generated/package.jsonis excluded by!**/generated/**,!**/generated/**pnpm-lock.yamlis excluded by!**/pnpm-lock.yamltests/simple/snapshots/root/diffs/pnpm-lock.yaml.diffis excluded by!tests/**
📒 Files selected for processing (15)
.changeset/afraid-meals-talk.mdexamples/blog-with-auth/apps/admin/package.jsonexamples/blog-with-auth/apps/backend/package.jsonexamples/blog-with-auth/libs/transactional/package.jsonexamples/blog-with-auth/package.jsonexamples/todo-with-better-auth/apps/admin/package.jsonexamples/todo-with-better-auth/apps/backend/package.jsonexamples/todo-with-better-auth/apps/web/package.jsonexamples/todo-with-better-auth/libs/transactional/package.jsonexamples/todo-with-better-auth/package.jsonmise.tomlpackage.jsonpackages/core-generators/src/constants/node.tspackages/project-builder-dev/src/e2e-runner/environment.tspnpm-workspace.yaml
✅ Files skipped from review due to trivial changes (14)
- examples/todo-with-better-auth/apps/web/package.json
- pnpm-workspace.yaml
- examples/todo-with-better-auth/libs/transactional/package.json
- examples/blog-with-auth/apps/admin/package.json
- examples/todo-with-better-auth/apps/admin/package.json
- .changeset/afraid-meals-talk.md
- mise.toml
- examples/blog-with-auth/libs/transactional/package.json
- packages/core-generators/src/constants/node.ts
- examples/todo-with-better-auth/package.json
- package.json
- examples/blog-with-auth/apps/backend/package.json
- examples/todo-with-better-auth/apps/backend/package.json
- examples/blog-with-auth/package.json
Summary
pnpm dedupeDetails
Summary by CodeRabbit
Chores
Documentation