fix(cli): publish under fallback npm package#393
Conversation
mrgoonie
left a comment
There was a problem hiding this comment.
Summary: PR #393 applies the maintainer fallback for issue #353 by moving the npm package identity to \ while preserving the installed \ binary.
Risk level: Medium. This is a package identity migration, so docs/update paths must move together, but the runtime command remains backward-compatible after install.
Mandatory gates:
- Duplicate / prior implementation: clear. PR #384 and #392 fixed workflow/bin metadata; neither solved npm ownership. No other PR changes to \ were found.
- Project standards: docs found. README/CLAUDE sync rules and no-direct-main workflow followed.
- Strategic necessity: clear value. Current token authenticates as , while \ is owned only by ; fallback package unblocks npm distribution.
Findings:
- Critical: none.
- Important: none.
- Suggestions: none.
Verification:
- \
ui-ux-pro-max-cli@2.5.0 check:assets
node scripts/sync-assets.mjs --check
CLI assets are in sync. passed.
- \
ui-ux-pro-max-cli@2.5.0 typecheck
tsc --noEmit passed.
- \ passed.
- {
"error": {
"code": "ENOENT",
"summary": "Could not read package.json: Error: ENOENT: no such file or directory, open '/Volumes/GOON/www/nlb/ui-ux-pro-max/skill/package.json'",
"detail": "This is related to npm not being able to find a file."
}
} passed from \ and reported package \ with \ included. - PR merge state is clean; GitHub reports no checks on the branch.
Verdict: Approve. This is ready to merge, then release. After release, close #353 only when , {
"error": {
"code": "E404",
"summary": "Not Found - GET https://registry.npmjs.org/ui-ux-pro-max-cli - Not found",
"detail": "The requested resource 'ui-ux-pro-max-cli@*' could not be found or you do not have permission to access it.\n\nNote that you can also install from a\ntarball, folder, http url, or git url."
}
}, and clean \ all pass.
Posted by /ck:review-pr at 2026-06-26T04:00:39Z
mrgoonie
left a comment
There was a problem hiding this comment.
Supersedes the previous malformed review comment; the earlier body was corrupted by local markdown shell expansion before posting.
Summary: PR #393 applies the maintainer fallback for issue #353 by moving the npm package identity to ui-ux-pro-max-cli while preserving the installed uipro binary.
Risk level: Medium. This is a package identity migration, so docs/update paths must move together, but the runtime command remains backward-compatible after install.
Mandatory gates:
- Duplicate / prior implementation: clear. PR #384 and #392 fixed workflow/bin metadata; neither solved npm ownership. No other PR changing to
ui-ux-pro-max-cliwas found. - Project standards: docs found. README/CLAUDE sync rules and no-direct-main workflow followed.
- Strategic necessity: clear value. Current
NPM_TOKENauthenticates asmrgoonie, whileuipro-cliis owned only byviettranx; fallback package unblocks npm distribution.
Findings:
- Critical: none.
- Important: none.
- Suggestions: none.
Verification:
npm view uipro-cli versionstill returns2.2.3.npm owner ls uipro-clistill lists onlyviettranx <viettranx@gmail.com>.npm view ui-ux-pro-max-cli versionreturns 404, so the fallback package is available.npm --prefix cli run check:assetspassed.npm --prefix cli run typecheckpassed.npm exec --yes --package bun -- bun build src/index.ts --outdir dist --target nodepassed.npm publish --dry-run --jsonpassed fromcli/and reported packageui-ux-pro-max-cli@2.5.0withdist/index.jsincluded.- PR merge state is clean; GitHub reports no checks on the branch.
Verdict: Approve. This is ready to merge, then release. After release, close #353 only when npm view ui-ux-pro-max-cli version, npm view ui-ux-pro-max-cli bin --json, and clean npx --yes ui-ux-pro-max-cli@<new-version> init --ai claude all pass.
Posted by /ck:review-pr at 2026-06-26T04:01:19Z
Summary
Switches the npm package identity from
uipro-clitoui-ux-pro-max-cliwhile preserving the installeduiprobinary command.Why
Issue #353 is still blocked on npm ownership for the official
uipro-clipackage. The repository release pipeline can build the correct package, but the currentNPM_TOKENauthenticates asmrgooniewhilenpm owner ls uipro-clistill lists onlyviettranx. This PR applies the maintainer fallback decision to publish under a different package name.Changes
cli/package.jsonandcli/package-lock.jsontoui-ux-pro-max-cli.uipro updateso automatic/manual updates installui-ux-pro-max-cli@<version>.skill.json, and contributor/docs references to useui-ux-pro-max-cli.bin.uipro = dist/index.js, so user commands remainuipro ...after install.Verification
npm view uipro-cli versionstill returns2.2.3.npm owner ls uipro-clistill lists onlyviettranx <viettranx@gmail.com>.NPM_TOKENidentity check returnsmrgoonie, so it cannot publishuipro-cli.npm view ui-ux-pro-max-cli versionreturns 404, so the fallback package name is available.npm --prefix cli run check:assetspasses.npm --prefix cli run typecheckpasses.npm exec --yes --package bun -- bun build src/index.ts --outdir dist --target nodepasses.npm publish --dry-run --jsonfromcli/passes and reportsui-ux-pro-max-cli@2.5.0withdist/index.jsincluded.Closes #353 after a release proves npm publish and clean install succeed.