feat(pm): vite add command for package installation

[fengmk2]

TL;DR

Added a new vite add command to simplify adding packages to dependencies with automatic package manager detection.

What changed?

• Added a new AddCommand implementation in vite_task/src/add.rs that handles adding packages to dependencies
• Implemented build_add_args method in PackageManager to generate the correct command arguments for different package managers (npm, yarn, pnpm)
• Added support for workspace filtering, workspace root, and workspace-only flags
• Added new error types for package manager detection and package specification
• Added comprehensive tests for the new functionality

How to test?

Test the new command with different package managers:

# Basic usage
vite add react react-dom

# Add dev dependencies
vite add typescript -D

# Add to specific workspace packages
vite add lodash --filter app

# Add to workspace root
vite add eslint -w

# Add to multiple workspaces
vite add jest --filter app --filter web

Why make this change?

This change simplifies the process of adding dependencies to projects by automatically detecting the package manager and translating the command to the appropriate syntax. It eliminates the need for users to remember different command formats for npm, yarn, and pnpm, especially in monorepo setups where workspace flags differ significantly between package managers.

---

Note

Introduces a unified vite add command that detects pnpm/yarn/npm, builds correct args (filters, workspace/root/global, save flags), aliases install <pkgs> to add, and adds error handling with extensive tests/snapshots.

• CLI (vite task):
  • Add Add subcommand in crates/vite_task/src/lib.rs with flags (-D/-P/-O/--save-peer, -E, --filter, -w, --workspace, -g, pass-through --).
  • Implement execution in crates/vite_task/src/add.rs; prints resolved command and runs via task graph.
  • Treat install <PACKAGES> as alias to add via parse_install_as_add.
• Package Manager Adapter:
  • New crates/vite_package_manager/src/add.rs with AddCommandOptions and SaveDependencyType and PackageManager::resolve_add_command that maps to pnpm/yarn/npm args (filters ordering, workspace/root, save flags, exact, catalogs, --allow-build, global via npm).
  • Export in crates/vite_package_manager/src/lib.rs.
• Errors:
  • Add Error::NoPackagesSpecified in crates/vite_error/src/lib.rs.
• Tests/Fixtures:
  • Comprehensive unit tests for arg resolution and command building across pnpm (v9/10), yarn (v4), npm (v10/11), including workspaces, catalogs, pass-through, and global.
  • Update CLI help/snapshots to include add and new outputs.
  • packages/tools: normalize CI output; strip Yarn YN0013 in replaceUnstableOutput with new test.
    ^{Written by Cursor Bugbot for commit b868089. This will update automatically on new commits. Configure here.}

[fengmk2]

• feat(pm): vite add command for package installation #221 : 3 dependent PRs (#237 , #242 , #243 ) 👈 (View in Graphite)
• refactor(pm): rename resolve_commmand to resolve_install_command #229
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Copilot]

Pull Request Overview

Copilot reviewed 57 out of 57 changed files in this pull request and generated 3 comments.

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 59 out of 60 changed files in this pull request and generated 4 comments.

Comments suppressed due to low confidence (2)

crates/vite_package_manager/src/add.rs:1

• The args parameter should be &[String] instead of &Vec<String>. Using a slice is more idiomatic and flexible in Rust.

use std::{collections::HashMap, process::ExitStatus};

packages/cli/binding/src/cli.rs:1

• This debug statement appears to be leftover from development. Consider removing it or clarifying if it's intentionally added for debugging purposes.

//! CLI types and logic moved from vite_task

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 62 out of 64 changed files in this pull request and generated 4 comments.

Files not reviewed (1)

• pnpm-lock.yaml: Language not supported

Comments suppressed due to low confidence (2)

packages/tools/src/utils.ts:1

• Corrected spelling of 'passth rough' to 'passthrough'.

import { Minimatch } from 'minimatch';

packages/tools/src/utils.ts:1

• Corrected spelling of 'passth rough' to 'passthrough'.

import { Minimatch } from 'minimatch';

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 62 out of 64 changed files in this pull request and generated 6 comments.

Files not reviewed (1)

• pnpm-lock.yaml: Language not supported

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 62 out of 64 changed files in this pull request and generated 5 comments.

Files not reviewed (1)

• pnpm-lock.yaml: Language not supported

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Pull Request Overview

Copilot reviewed 62 out of 64 changed files in this pull request and generated 2 comments.

Files not reviewed (1)

• pnpm-lock.yaml: Language not supported

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[fengmk2]

Merge activity

• Oct 23, 11:25 AM UTC: @fengmk2 merged this pull request with Graphite.