Skip to content

fix(cli): remove unsupported catalogMode for yarn monorepo#873

Merged
fengmk2 merged 4 commits intovoidzero-dev:mainfrom
Salnika:fix-cli-monorepo-yarn-catalog
Mar 15, 2026
Merged

fix(cli): remove unsupported catalogMode for yarn monorepo#873
fengmk2 merged 4 commits intovoidzero-dev:mainfrom
Salnika:fix-cli-monorepo-yarn-catalog

Conversation

@Salnika
Copy link
Copy Markdown
Contributor

@Salnika Salnika commented Mar 14, 2026

When creating a monorepo with yarn as package manager, the default template uses the catalogMode field which is not supported by yarn. This PR removes the field in the _yarnrc.yml template file.

vp create  --verbose
VITE+ - The Unified Toolchain for the Web

  Vite+ Monorepo

◇ Package name:
  yarn-monorepo

◇ Which package manager would you like to use?
  yarn

◇ yarn@4.12.0 installed

◇ Which agents are you using?
  ChatGPT (Codex)

◇ Which editor are you using?
  VSCode

◇ Set up pre-commit hooks to run formatting, linting, and type checking with auto-fixes?
  Yes

◇ Initialize git repository:
  Yes

Target directory: ./yarn-monorepo
Creating Vite+ monorepo...
Monorepo template created
Git repository initialized
Creating default application in apps/website...

Running: yarn dlx create-vite@latest apps/website --template vanilla-ts --no-interactive --no-immediate --no-rolldown
Usage Error: Unrecognized or legacy configuration settings found: catalogMode - run "yarn config" to see the list of settings supported in Yarn (in /private/var/folders/h4/nx53m7310nl_w6v5568pjb480000gq/T/xfs-4f51d873/dlx-38907/.yarnrc.yml)

$ yarn add [--json] [-F,--fixed] [-E,--exact] [-T,--tilde] [-C,--caret] [-D,--dev] [-P,--peer] [-O,--optional] [--prefer-dev] [-i,--interactive] [--cached] [--mode #0] ...

Failed to create default application: 1
Failed to create monorepo, exit code: 1

@netlify
Copy link
Copy Markdown

netlify bot commented Mar 14, 2026

Deploy Preview for viteplus-preview canceled.

Name Link
🔨 Latest commit ed508b7
🔍 Latest deploy log https://app.netlify.com/projects/viteplus-preview/deploys/69b697fbee362d0008c783dc

Copy link
Copy Markdown
Member

@fengmk2 fengmk2 left a comment

Choose a reason for hiding this comment

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

The test in packages/cli/src/create/tests/monorepo-template.spec.ts is a bit redundant, I will remove it. Thank you for contributing to open source.

@fengmk2 fengmk2 self-assigned this Mar 15, 2026
Copy link
Copy Markdown
Member

fengmk2 commented Mar 15, 2026

Merge activity

  • Mar 15, 11:38 AM UTC: Graphite couldn't merge this PR because it failed for an unknown reason (Fast-forward merges are not supported for forked repositories. Please create a branch in the target repository in order to merge).
  • Mar 15, 11:42 AM UTC: A user started a stack merge that includes this pull request via Graphite.

@fengmk2 fengmk2 merged commit a9b57f8 into voidzero-dev:main Mar 15, 2026
21 checks passed
@Salnika Salnika deleted the fix-cli-monorepo-yarn-catalog branch March 16, 2026 16:17
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