Skip to content

refactor(cli): remove package manager commands from local cli#341

Merged
fengmk2 merged 1 commit intomainfrom
12-11-refactor_cli_remove_packages_commands_from_local_cli
Dec 19, 2025
Merged

refactor(cli): remove package manager commands from local cli#341
fengmk2 merged 1 commit intomainfrom
12-11-refactor_cli_remove_packages_commands_from_local_cli

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented Dec 11, 2025

TL;DR

Simplified CLI command structure by removing package manager related commands.

What changed?

  • Removed package manager related commands (add, remove, update, dedupe, outdated, why, link, unlink, and pm)
  • Added helpful command descriptions to the remaining commands
  • Simplified the is_package_manager_command() function to only check for the install command
  • Updated the help text to reflect the new command structure

How to test?

  1. Run vite -h to verify the updated command list
  2. Verify that only the core commands are available: run, lint, fmt, build, test, lib, dev, doc, cache, and install
  3. Check that each command has a descriptive help text

Why make this change?

This change streamlines the CLI interface by focusing on core functionality and removing rarely used package manager commands. This makes the CLI more approachable for new users and reduces maintenance overhead. The simplified command structure aligns better with the primary purpose of the tool while still maintaining the essential install command for backward compatibility.

@fengmk2 fengmk2 changed the title refactor(cli): remove packages commands from local cli refactor(cli): remove package manager commands Dec 11, 2025
@fengmk2 fengmk2 changed the title refactor(cli): remove package manager commands refactor(cli): remove package manager commands from local cli Dec 11, 2025
@fengmk2 fengmk2 self-assigned this Dec 11, 2025
@fengmk2 fengmk2 marked this pull request as ready for review December 11, 2025 09:35
@fengmk2 fengmk2 force-pushed the 12-10-refactor_global_global_cli_completely_independent branch from c658c82 to d917590 Compare December 11, 2025 09:45
@fengmk2 fengmk2 force-pushed the 12-11-refactor_cli_remove_packages_commands_from_local_cli branch from 4e0d3db to 12d92a5 Compare December 11, 2025 09:45
@fengmk2 fengmk2 force-pushed the 12-10-refactor_global_global_cli_completely_independent branch from d917590 to 9584148 Compare December 11, 2025 10:11
@fengmk2 fengmk2 force-pushed the 12-11-refactor_cli_remove_packages_commands_from_local_cli branch 2 times, most recently from 08c0d48 to c069be5 Compare December 11, 2025 10:48
@fengmk2 fengmk2 force-pushed the 12-10-refactor_global_global_cli_completely_independent branch 2 times, most recently from e3fe2fa to 67c4bca Compare December 12, 2025 03:58
@fengmk2 fengmk2 force-pushed the 12-11-refactor_cli_remove_packages_commands_from_local_cli branch 2 times, most recently from 9a893be to 85f1811 Compare December 12, 2025 09:00
@fengmk2 fengmk2 force-pushed the 12-10-refactor_global_global_cli_completely_independent branch from 67c4bca to 4ae86d3 Compare December 12, 2025 09:00
@fengmk2 fengmk2 force-pushed the 12-11-refactor_cli_remove_packages_commands_from_local_cli branch from 85f1811 to 522f43a Compare December 12, 2025 09:05
Comment thread .github/workflows/release.yml Outdated
@fengmk2 fengmk2 force-pushed the 12-11-refactor_cli_remove_packages_commands_from_local_cli branch 2 times, most recently from d2abf23 to 4743f4d Compare December 15, 2025 12:37
@fengmk2 fengmk2 force-pushed the 12-10-refactor_global_global_cli_completely_independent branch from 4ae86d3 to b4e29db Compare December 15, 2025 12:37
@fengmk2 fengmk2 force-pushed the 12-11-refactor_cli_remove_packages_commands_from_local_cli branch from 4743f4d to 4a06fee Compare December 16, 2025 06:15
@fengmk2 fengmk2 force-pushed the 12-10-refactor_global_global_cli_completely_independent branch from b4e29db to 5655019 Compare December 16, 2025 06:15
@fengmk2 fengmk2 changed the base branch from 12-10-refactor_global_global_cli_completely_independent to graphite-base/341 December 17, 2025 06:01
@fengmk2 fengmk2 force-pushed the 12-11-refactor_cli_remove_packages_commands_from_local_cli branch from 4a06fee to dfd9310 Compare December 17, 2025 06:16
@graphite-app graphite-app Bot changed the base branch from graphite-base/341 to main December 17, 2025 06:17
@fengmk2 fengmk2 force-pushed the 12-11-refactor_cli_remove_packages_commands_from_local_cli branch from dfd9310 to c8d3b03 Compare December 17, 2025 06:17
Copilot AI review requested due to automatic review settings December 17, 2025 07:52
@fengmk2 fengmk2 force-pushed the 12-11-refactor_cli_remove_packages_commands_from_local_cli branch from c8d3b03 to 17d6984 Compare December 17, 2025 07:52
Copy link
Copy Markdown
Contributor

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 simplifies the CLI command structure by removing package manager-related commands and retaining only core functionality. The change improves usability by reducing complexity and adding descriptive help text for each command.

Key changes:

  • Removed 9 package manager commands (add, remove, update, dedupe, outdated, why, link, unlink, pm) and their implementations
  • Added concise command descriptions to improve CLI help output
  • Simplified the is_package_manager_command() function to only check for the install command

Reviewed changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/cli/binding/src/commands/add.rs Deleted - removed add command implementation
packages/cli/binding/src/commands/dedupe.rs Deleted - removed dedupe command implementation
packages/cli/binding/src/commands/link.rs Deleted - removed link command implementation
packages/cli/binding/src/commands/mod.rs Updated module declarations to remove deleted command modules
packages/cli/binding/src/commands/outdated.rs Deleted - removed outdated command implementation
packages/cli/binding/src/commands/pm.rs Deleted - removed package manager utilities command
packages/cli/binding/src/commands/remove.rs Deleted - removed remove command implementation
packages/cli/binding/src/commands/unlink.rs Deleted - removed unlink command implementation
packages/cli/binding/src/commands/update.rs Deleted - removed update command implementation
packages/cli/binding/src/commands/why.rs Deleted - removed why command implementation
packages/cli/binding/src/cli.rs Removed imports and command definitions for deleted commands, added command descriptions, simplified command handling logic, and removed helper functions and tests for deleted commands
packages/cli/snap-tests/command-helper/snap.txt Added snapshot test output showing new help message with command descriptions
packages/cli/snap-tests/command-helper/steps.json Added test case for help message verification

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

Comment thread packages/cli/binding/src/cli.rs
Copy link
Copy Markdown
Member Author

fengmk2 commented Dec 19, 2025

Merge activity

  • Dec 19, 12:34 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Dec 19, 12:35 AM UTC: Graphite rebased this pull request as part of a merge.
  • Dec 19, 12:46 AM UTC: @fengmk2 merged this pull request with Graphite.

@fengmk2 fengmk2 force-pushed the 12-11-refactor_cli_remove_packages_commands_from_local_cli branch from 17d6984 to 8ec65ec Compare December 19, 2025 00:34
@fengmk2 fengmk2 merged commit b9acebf into main Dec 19, 2025
12 checks passed
@fengmk2 fengmk2 deleted the 12-11-refactor_cli_remove_packages_commands_from_local_cli branch December 19, 2025 00:46
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.

3 participants