feat: tweak short aliases: remove ddp, hide un/uninstall from help (#577)

- [x] Remove `ddp` alias from `dedupe` command (no longer works)
- [x] Keep `un` and `uninstall` as visible aliases on `remove` (per
reviewer feedback)
- [x] Update custom help text to reflect alias changes
- [x] Update snap tests that use `vp ddp` to use `vp dedupe`
- [x] Update snap.txt files to reflect new help output
- [x] Remove `ddp` references from RFC documentation
- [x] Code review passed
- [x] Security check passed (no alerts)

<!-- START COPILOT ORIGINAL PROMPT -->



<details>

<summary>Original prompt</summary>

> 
> ----
> 
> *This section details on the original issue you should resolve*
> 
> <issue_title>Tweak short alias</issue_title>
> <issue_description>> That said, I think we should support a few very
common aliases like vp i - this is mostly to cater to established habits
from users with existing package managers and ease migration. But only
the common ones. i, rm, ln, ls and up should be enough (this aligns with
pnpm). We should get rid of ddp for sure.
> > side note: pnpm supports uninstall and un but doesn't list it in its
help. We can probably do that too.</issue_description>
> 
> ## Comments on the Issue (you are @copilot in this section)
> 
> <comments>
> </comments>
> 


</details>



<!-- START COPILOT CODING AGENT SUFFIX -->

- Fixes #576

<!-- START COPILOT CODING AGENT TIPS -->
---

✨ Let Copilot coding agent [set things up for
you](https://github.com/voidzero-dev/vite-plus/issues/new?title=✨+Set+up+Copilot+instructions&body=Configure%20instructions%20for%20this%20repository%20as%20documented%20in%20%5BBest%20practices%20for%20Copilot%20coding%20agent%20in%20your%20repository%5D%28https://gh.io/copilot-coding-agent-tips%29%2E%0A%0A%3COnboard%20this%20repo%3E&assignees=copilot)
— coding agent works faster and does higher quality work when set up for
your repo.

Co-authored-by: Yunfei He <i.heyunfei@gmail.com>

Author: Copilot

crates/vite_global_cli/src/cli.rs

@@ -311,7 +311,6 @@ pub enum Commands {
     },
 
     /// Deduplicate dependencies
-    #[command(visible_alias = "ddp")]
     Dedupe {
         /// Check if deduplication would make changes
         #[arg(long)]
@@ -1617,7 +1616,7 @@ fn apply_custom_help(cmd: clap::Command) -> clap::Command {
   {bold}install, i{reset}                     Install all dependencies, or add packages if package names are provided
   {bold}add{reset}                            Add packages to dependencies
   {bold}remove, rm, un, uninstall{reset}      Remove packages from dependencies
-  {bold}dedupe, ddp{reset}                    Deduplicate dependencies by removing older versions
+  {bold}dedupe{reset}                         Deduplicate dependencies by removing older versions
   {bold}dlx{reset}                            Execute a package binary without installing it as a dependency
   {bold}info, view, show{reset}               View package information from the registry
   {bold}link, ln{reset}                       Link packages for local development

packages/cli/snap-tests-todo/exit-non-zero-on-cmd-not-exists/snap.txt

@@ -1,6 +1,6 @@
 [2]> vite command-not-exists # should exit with non-zero code
 error: 'vite' requires a subcommand but one was not provided
-  [subcommands: run, lint, fmt, build, test, lib, dev, doc, cache, install, i, add, remove, rm, un, uninstall, update, up, dedupe, ddp, outdated, why, explain, link, ln, unlink, pm, help]
+  [subcommands: run, lint, fmt, build, test, lib, dev, doc, cache, install, i, add, remove, rm, un, uninstall, update, up, dedupe, outdated, why, explain, link, ln, unlink, pm, help]
 
 Usage: vite [OPTIONS] [TASK] [-- <TASK_ARGS>...] <COMMAND>
 

packages/global/snap-tests/cli-helper-message/snap.txt

@@ -21,7 +21,7 @@ Package Manager Commands:
   install, i                     Install all dependencies, or add packages if package names are provided
   add                            Add packages to dependencies
   remove, rm, un, uninstall      Remove packages from dependencies
-  dedupe, ddp                    Deduplicate dependencies by removing older versions
+  dedupe                         Deduplicate dependencies by removing older versions
   dlx                            Execute a package binary without installing it as a dependency
   info, view, show               View package information from the registry
   link, ln                       Link packages for local development

packages/global/snap-tests/command-dedupe-npm10/snap.txt

@@ -34,7 +34,7 @@ up to date in <variable>ms
   "packageManager": "npm@<semver>"
 }
 
-> vp ddp -- --loglevel=warn && cat package.json # support pass through arguments
+> vp dedupe -- --loglevel=warn && cat package.json # support pass through arguments
 
 up to date in <variable>ms
 {

packages/global/snap-tests/command-dedupe-npm10/steps.json

@@ -5,6 +5,6 @@
   "commands": [
     "vp dedupe && cat package.json # should dedupe dependencies",
     "vp dedupe --check && cat package.json # should check if deduplication would make changes",
-    "vp ddp -- --loglevel=warn && cat package.json # support pass through arguments"
+    "vp dedupe -- --loglevel=warn && cat package.json # support pass through arguments"
   ]
 }

packages/global/snap-tests/command-dedupe-pnpm10/snap.txt

@@ -56,7 +56,7 @@ Progress: resolved <variable>, reused <variable>, downloaded <variable>, added <
   "packageManager": "pnpm@<semver>"
 }
 
-> vp ddp -- --loglevel=warn && cat package.json # support pass through arguments
+> vp dedupe -- --loglevel=warn && cat package.json # support pass through arguments
 {
   "name": "command-dedupe-pnpm10",
   "version": "1.0.0",

packages/global/snap-tests/command-dedupe-pnpm10/steps.json

@@ -6,7 +6,7 @@
     "vp dedupe --help # should show help",
     "vp dedupe && cat package.json # should dedupe dependencies",
     "vp dedupe --check && cat package.json # should check if deduplication would make changes",
-    "vp ddp -- --loglevel=warn && cat package.json # support pass through arguments",
+    "vp dedupe -- --loglevel=warn && cat package.json # support pass through arguments",
     "json-edit package.json '_.dependencies = {}' && cat package.json && vp dedupe --check # should check fails because no dependencies",
     "vp dedupe && cat package.json && vp dedupe --check # should dedupe fix the change by removing the dependencies"
   ]

packages/global/snap-tests/command-dedupe-yarn4/snap.txt

@@ -45,7 +45,7 @@
   "packageManager": "yarn@<semver>"
 }
 
-> vp ddp -- --json && cat package.json # support pass through arguments
+> vp dedupe -- --json && cat package.json # support pass through arguments
 {
   "name": "command-dedupe-yarn4",
   "version": "1.0.0",

packages/global/snap-tests/command-dedupe-yarn4/steps.json

@@ -5,6 +5,6 @@
   "commands": [
     "vp dedupe && cat package.json # should dedupe dependencies",
     "vp dedupe --check && cat package.json # should check if deduplication would make changes",
-    "vp ddp -- --json && cat package.json # support pass through arguments"
+    "vp dedupe -- --json && cat package.json # support pass through arguments"
   ]
 }

rfcs/dedupe-package-command.md

@@ -2,7 +2,7 @@
 
 ## Summary
 
-Add `vite dedupe` (alias: `vite ddp`) command that automatically adapts to the detected package manager (pnpm/npm/yarn) for optimizing dependency trees by removing duplicate packages and upgrading older dependencies to newer compatible versions in the lockfile. This helps reduce redundancy and improve project efficiency.
+Add `vite dedupe` command that automatically adapts to the detected package manager (pnpm/npm/yarn) for optimizing dependency trees by removing duplicate packages and upgrading older dependencies to newer compatible versions in the lockfile. This helps reduce redundancy and improve project efficiency.
 
 ## Motivation
 
@@ -40,7 +40,6 @@ yarn dedupe --check            # yarn@2+ - check without modifying
 ```bash
 # Works for all package managers
 vite dedupe                    # Deduplicate dependencies
-vite ddp                       # Alias
 
 # Check mode (dry-run)
 vite dedupe --check            # Check if deduplication would make changes
@@ -54,15 +53,13 @@ vite dedupe --check            # Check if deduplication would make changes
 
 ```bash
 vite dedupe [OPTIONS]
-vite ddp [OPTIONS]        # Alias
 ```
 
 **Examples:**
 
 ```bash
 # Basic deduplication
 vite dedupe
-vite ddp
 
 # Check mode (preview changes without modifying)
 vite dedupe --check
@@ -97,10 +94,6 @@ vite dedupe --check
 - pnpm uses `--check` for dry-run, npm uses `--dry-run`, yarn@2+ uses `--check`
 - yarn@1 does not have dedupe command and is not supported
 
-**Aliases:**
-
-- `vite ddp` = `vite dedupe` (matches npm's `ddp` alias)
-
 ### Dedupe Behavior Differences Across Package Managers
 
 #### pnpm
@@ -164,7 +157,7 @@ pub enum Commands {
     // ... existing commands
 
     /// Deduplicate dependencies by removing older versions
-    #[command(disable_help_flag = true, alias = "ddp")]
+    #[command(disable_help_flag = true)]
     Dedupe {
         /// Check if deduplication would make changes (pnpm: --check, npm: --dry-run)
         #[arg(long)]
@@ -662,8 +655,6 @@ Deduplicate dependencies by removing older versions
 
 Usage: vite dedupe [OPTIONS] [-- <PASS_THROUGH_ARGS>...]
 
-Aliases: ddp
-
 Options:
   --check                    Check if deduplication would make changes
                              (pnpm: --check, npm: --dry-run, yarn@2+: --check)
@@ -677,7 +668,6 @@ Note: yarn@1 does not have a dedupe command and is not supported
 
 Examples:
   vite dedupe                          # Deduplicate all dependencies
-  vite ddp                             # Same as above (alias)
   vite dedupe --check                  # Check if changes would occur
   vite dedupe -- --some-flag           # Pass custom flags to package manager
 ```