Skip to content

Conversation

@kingston
Copy link
Collaborator

@kingston kingston commented Oct 23, 2025

Summary by CodeRabbit

  • Chores
    • Updated Vite build tool to version 7.1.12 across projects and example apps.
    • Simplified and restored workspace configuration settings (publish branch, workspace linking/protocol, and minimum release age).
    • Removed several now-unused npm registry configuration lines.

@changeset-bot
Copy link

changeset-bot bot commented Oct 23, 2025

🦋 Changeset detected

Latest commit: b77de31

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 18 packages
Name Type
@baseplate-dev/react-generators Patch
@baseplate-dev/project-builder-server Patch
@baseplate-dev/plugin-auth Patch
@baseplate-dev/plugin-storage Patch
@baseplate-dev/project-builder-cli Patch
@baseplate-dev/project-builder-common Patch
@baseplate-dev/project-builder-test Patch
@baseplate-dev/project-builder-web Patch
@baseplate-dev/create-project Patch
@baseplate-dev/code-morph Patch
@baseplate-dev/core-generators Patch
@baseplate-dev/fastify-generators Patch
@baseplate-dev/project-builder-lib Patch
@baseplate-dev/sync Patch
@baseplate-dev/tools Patch
@baseplate-dev/ui-components Patch
@baseplate-dev/utils Patch
@baseplate-dev/plugin-queue Patch

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

@vercel
Copy link

vercel bot commented Oct 23, 2025

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

Project Deployment Preview Comments Updated (UTC)
baseplate-project-builder-web Ready Ready Preview Comment Oct 24, 2025 7:44am

@socket-security
Copy link

socket-security bot commented Oct 23, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedvite@​7.1.12981008299100

View full report

@coderabbitai
Copy link

coderabbitai bot commented Oct 23, 2025

Walkthrough

Bumps Vite from 7.1.5 to 7.1.12 across example app package.json files and the REACT_PACKAGES constant, adds pnpm workspace configuration keys, removes several lines from .npmrc, and adds a changeset documenting the patch update to @baseplate-dev/react-generators.

Changes

Cohort / File(s) Summary
Changeset
.changeset/silly-tires-relax.md
Added changeset noting patch bump for @baseplate-dev/react-generators and Vite upgrade.
Example app devDependencies
examples/blog-with-auth/apps/admin/package.json, examples/todo-with-auth0/apps/admin/package.json, examples/todo-with-auth0/apps/web/package.json
Updated devDependencies.vite from 7.1.5 to 7.1.12.
Generator constant
packages/react-generators/src/constants/react-packages.ts
Updated REACT_PACKAGES mapping: vite value changed from '7.1.5' to '7.1.12'.
Monorepo workspace config
pnpm-workspace.yaml
Updated catalog entry for Vite to 7.1.12; added top-level keys: publishBranch: main, linkWorkspacePackages: true, saveWorkspaceProtocol: rolling, minimumReleaseAge: 1440.
NPM config
.npmrc
Removed several workspace/publish-related configuration lines and associated comments (publish-branch, link-workspace-packages, save-workspace-protocol, minimumReleaseAge).

Sequence Diagram(s)

(omitted — changes are dependency and configuration edits with no control-flow modifications)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "feat: Upgrade vite to 7.1.12" is directly related to the main change in the changeset. The raw summary confirms that vite is being upgraded from 7.1.5 to 7.1.12 across multiple files, including package.json files, the react-packages constants, and the pnpm-workspace catalog. While the PR also includes configuration reorganization changes (moving pnpm settings from .npmrc to pnpm-workspace.yaml), the vite upgrade appears to be the primary focus of this changeset, and the title clearly and specifically identifies this main change. The title is concise, clear, and provides meaningful context that a teammate scanning the history would understand.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch kingston/upgrade-vite

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between 3d12b17 and b77de31.

⛔ Files ignored due to path filters (9)
  • examples/blog-with-auth/apps/admin/baseplate/generated/package.json is excluded by !**/generated/**, !**/generated/**
  • examples/blog-with-auth/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
  • examples/todo-with-auth0/apps/admin/baseplate/generated/package.json is excluded by !**/generated/**, !**/generated/**
  • examples/todo-with-auth0/apps/web/baseplate/generated/package.json is excluded by !**/generated/**, !**/generated/**
  • examples/todo-with-auth0/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
  • tests/simple/apps/web/baseplate/generated/package.json is excluded by !**/generated/**, !tests/**, !**/generated/**
  • tests/simple/apps/web/package.json is excluded by !tests/**
  • tests/simple/pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml, !tests/**
📒 Files selected for processing (6)
  • .changeset/silly-tires-relax.md (1 hunks)
  • examples/blog-with-auth/apps/admin/package.json (1 hunks)
  • examples/todo-with-auth0/apps/admin/package.json (1 hunks)
  • examples/todo-with-auth0/apps/web/package.json (1 hunks)
  • packages/react-generators/src/constants/react-packages.ts (1 hunks)
  • pnpm-workspace.yaml (2 hunks)
🚧 Files skipped from review as they are similar to previous changes (4)
  • examples/todo-with-auth0/apps/web/package.json
  • packages/react-generators/src/constants/react-packages.ts
  • .changeset/silly-tires-relax.md
  • examples/blog-with-auth/apps/admin/package.json
🧰 Additional context used
📓 Path-based instructions (2)
**/*

📄 CodeRabbit inference engine (.cursor/rules/code-style.mdc)

Use kebab-case for file names

Files:

  • examples/todo-with-auth0/apps/admin/package.json
  • pnpm-workspace.yaml
examples/todo-with-auth0/{package.json,apps/{admin,backend}/package.json}

📄 CodeRabbit inference engine (examples/todo-with-auth0/CLAUDE.md)

Use ESM only by setting "type": "module" in package.json

Files:

  • examples/todo-with-auth0/apps/admin/package.json
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (8)
  • GitHub Check: Lint Examples (blog-with-auth)
  • GitHub Check: Lint Examples (todo-with-auth0)
  • GitHub Check: Verify Sync (todo-with-auth0)
  • GitHub Check: Verify Sync (blog-with-auth)
  • GitHub Check: Socket Security: Pull Request Alerts
  • GitHub Check: Lint
  • GitHub Check: Test E2E
  • GitHub Check: test
🔇 Additional comments (4)
examples/todo-with-auth0/apps/admin/package.json (1)

92-92: LGTM! Vite dependency updated correctly.

The version bump from 7.1.5 to 7.1.12 is straightforward, and the file correctly has "type": "module" set on line 7 per ESM requirements.

pnpm-workspace.yaml (3)

21-21: Vite version in catalog updated consistently.

The catalog version matches the version bump in package.json files across the workspace.


35-38: Workspace configuration added with valid pnpm settings.

The new top-level configuration keys are all recognized pnpm options:

  • publishBranch: main and minimumReleaseAge: 1440 support changesets publishing flow
  • linkWorkspacePackages: true enables workspace package linking
  • saveWorkspaceProtocol: rolling controls workspace protocol persistence

According to the AI summary, these settings align with removals from .npmrc, consolidating workspace configuration in a single place.


21-21: Verify PR title aligns with actual Vite version.

The PR title states "Upgrade vite to 7.1.11", but both package.json and pnpm-workspace.yaml show version 7.1.12. Please confirm that 7.1.12 is the intended target version, or update the PR title if 7.1.11 was the original intent.


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

@kingston kingston changed the title feat: Upgrade vite to 7.1.11 feat: Upgrade vite to 7.1.12 Oct 24, 2025
@kingston kingston merged commit a6274e9 into main Oct 24, 2025
17 of 18 checks passed
@kingston kingston deleted the kingston/upgrade-vite branch October 24, 2025 07:48
@github-actions github-actions bot mentioned this pull request Oct 24, 2025
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