Skip to content

Commit a49226a

Browse files
bpamiriclaude
andauthored
fix(view): packagelist UI shows canonical 'add' verb, not 'install' (#2508)
The Packages dev UI rendered an install hint: wheels packages install <name> LuCLI's built-in extension installer intercepts the literal subcommand `install` across all modules, so `wheels packages install <name>` never reaches Module.cfc. PR #2374 already renamed the verb to `add` for this reason; the dev UI hadn't been updated. Users following the UI's suggestion saw "Command 'wheels packages install <name>' cannot be resolved" (CommandBox) or LuCLI's generic dependency-installer output ("No git or extension dependencies to install"). Switch the rendered command to the canonical `add` verb. Update CLAUDE.md's package-install snippet to match. Resolves #2467. Co-authored-by: Claude <noreply@anthropic.com>
1 parent 6988a26 commit a49226a

2 files changed

Lines changed: 8 additions & 5 deletions

File tree

‎CLAUDE.md‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -382,9 +382,9 @@ Use the `wheels packages` CLI. Resolves names against the `wheels-dev/wheels-pac
382382
wheels packages list # browse the registry
383383
wheels packages search <query> # name/description/tag match
384384
wheels packages show <name> # detail page
385-
wheels packages install <name> # latest compat version
386-
wheels packages install <name>@<version> # pin
387-
wheels packages install <name> --force # overwrite an existing vendor/<name>
385+
wheels packages add <name> # latest compat version (canonical verb)
386+
wheels packages add <name>@<version> # pin
387+
wheels packages add <name> --force # overwrite an existing vendor/<name>
388388
wheels packages update <name> --yes # explicit update
389389
wheels packages update --all --yes # update every installed package
390390
wheels packages remove <name> # delete vendor/<name>

‎vendor/wheels/public/views/packagelist.cfm‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,8 +161,11 @@ for (local.key in packageMeta) {
161161
<span class="ui label"><i class="check icon"></i> Installed</span>
162162
<cfelse>
163163
<!--- rp.name is registry-schema-constrained to ^[a-z0-9][a-z0-9-]*$, so both id and JS string are already safe.
164-
JSStringFormat() is applied in the onclick defensively in case that invariant ever loosens. --->
165-
<code id="install-#HTMLEditFormat(local.rpKey)#">wheels packages install #HTMLEditFormat(local.rp.name)#</code>
164+
JSStringFormat() is applied in the onclick defensively in case that invariant ever loosens.
165+
`add` (not `install`) is the canonical verb — LuCLI's built-in extension
166+
installer intercepts the literal subcommand `install`, so `wheels packages
167+
install <name>` never reaches Module.cfc. See PR #2374 / cli/lucli/services/packages/PackagesMainCli.cfc. --->
168+
<code id="install-#HTMLEditFormat(local.rpKey)#">wheels packages add #HTMLEditFormat(local.rp.name)#</code>
166169
<button type="button"
167170
class="ui tiny button"
168171
aria-label="Copy install command for #HTMLEditFormat(local.rp.name)#"

0 commit comments

Comments
 (0)