fix: advertise wheels packages add (not install) on docs and help - #3382
Merged
Conversation
LuCLI intercepts `packages install` before Module.cfc, so user-facing copy, --help, and the packages website now lead with `add`. The Basecoat chapter also copies the showcase from vendor/ after add, not GitHub. Signed-off-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Peter Amiri <peter@alurium.com>
The packages index and wheels-sentry canaries now screenshot `wheels packages add` instead of the intercepted `install` verb. Signed-off-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Peter Amiri <peter@alurium.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Users type
wheels packages install <name>and get LuCLI's extension installer ([INFO] No git or extension dependencies…or alucee.jsonerror) instead of a Wheels package. The official verb iswheels packages add. This PR makes that discoverable on every user-facing surface in this repo. This PR does not address #3378 (Basecoat mixin / showcase loading is out of scope) and does not changePackageLoadermixin behavior.Related Issue
Related to #3378 (docs/help only — this is not the Basecoat mixin / UIBADGE work).
Type of Change
What changed
web/sites/packages/): copy-to-clipboard snippets and the index subhead now saywheels packages add. The detail page also states thatinstallis not the verb. Visual baselines forpackages-indexandpackages-wheels-sentryrefreshed to match./commands/packages/install/to/commands/packages/add/(old URL redirects). Index, packages howto, and the Basecoat bonus chapter already usedadd; the chapter now has an optional showcase section: add the package, then copy fromvendor/wheels-basecoat/, not the raw GitHub tree.packages()hint now leads withAdd, matchingwheels --help.wheels packages add.$packagesHelp()already documented the LuCLI intercept; left as-is.LuCLI limit (documented, not hooked)
wheels packages installis swallowed by LuCLI's built-in extension installer beforeModule.cfcruns. There is no safe in-module alias for the shell path. MCP / in-process callers already fall throughcase "install":toadd(#2786). This PR does not invent a fragile pre-dispatch hook.Feature Completeness Checklist
Signed-off-by:installslug redirects toadd.ai/wheels/packages install verb)wheels packages addchangelog.d/3378-packages-add-verb.fixed.mdtools/test-local.shcould not start here (nowheelsCLI / Docker). Source-scan assertions were re-checked independently.Test Plan
wheels packages installremains as a recommended command (warnings that say "not the verb" are intentional)