Skip to content

refactor(cli): rename the internal orbit-cli to orbit-release-cli - #662

Open
MRmarioruci wants to merge 1 commit into
mainfrom
release-5-cli-rename
Open

MRmarioruci wants to merge 1 commit into
mainfrom
release-5-cli-rename

Conversation

@MRmarioruci

@MRmarioruci MRmarioruci commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Renames the internal release tool from orbit-cli to orbit-release-cli, which frees orbit-cli for the CLI we ship to users. Requested in #658 (comment).

Covers the workspace package and its bin, the commander program name, the prepare-cli build and link script, the release publish call in release.yaml, the e2e helper, two error messages, two temp-file prefixes, the audit README and the station-audit skill.

orbit-cli stays as a second bin entry. The deploy stack adds three more call sites that still use that name, and those files are not on main yet, so the alias lets this PR and that stack merge in either order. A follow-up renames them and drops it.

The package has no entry in .release.json, so it has never been tagged or published and no release history changes. Not marked breaking: there are no external consumers, and a ! would risk nx forcing a major bump on whichever released project it attributes these files to.

One commit, because every intermediate state would have a package name its own callers do not use.

Verified: pnpm install runs the renamed prepare-cli end to end, and the built CLI reports orbit-release-cli with all subcommands intact.

@MRmarioruci
MRmarioruci requested a review from a team as a code owner September 4, 2026 13:08
@zeropath-ai

zeropath-ai Bot commented Sep 4, 2026

Copy link
Copy Markdown

No security or compliance issues detected. Reviewed everything up to 0a50bc8.

Security Overview
Detected Code Changes
Change Type Relevant files
Enhancement ► .claude/skills/orbit-station-audit/SKILL.md
• Update references from orbit-cli to orbit-release-cli in descriptions and commands
► .claude/skills/orbit-station-audit/scripts/run-audit.sh
• Update script to invoke orbit-release-cli audit instead of orbit-cli audit
► .github/workflows/release.yaml
• Update release command from orbit-cli release publish to orbit-release-cli release publish
► cli/package.json
• Rename CLI package to orbit-release-cli and expose dist/cli.js under new bin name
► cli/src/audit/README.md
• Update references from orbit-cli audit to orbit-release-cli audit in usage/docs
► cli/src/audit/identity-icp.ts
• Update error/message strings to refer to orbit-release-cli audit and session file naming
► cli/src/cli.ts
• Update CLI name from orbit-cli to orbit-release-cli
► cli/src/registry/publish.ts
• Update temporary argument file naming from orbit-cli-argument- to orbit-release-cli-argument-
► package.json (root)\n> • Prepare CLI now targets orbit-release-cli by updating prepare-cli script to build/expose orbit-release-cli
► tests/e2e/utils/orbit.utils.ts
• Update publish command to orbit-release-cli registry publish --app

Frees the name orbit-cli for the CLI we ship to users, which is what
dfx-orbit is proposed to become. Renames the workspace package, the
commander program name, the prepare-cli build and link script, the
release.yaml call site, the e2e helper, the audit README and the station
audit skill.

Keeps orbit-cli as a second bin entry for now. The release workflows on
the deploy stack still call that name, so the alias means this PR and
that stack can merge in either order without breaking the pipeline. A
follow-up drops the alias once those call sites are renamed.

Not marked as a breaking change on purpose. The package is not in
.release.json, so it has never been tagged or published and there are no
external consumers, and a breaking marker would risk nx forcing a major
bump on whichever released project it attributes these files to.
@MRmarioruci
MRmarioruci changed the base branch from release-4-docs to main September 4, 2026 13:28
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.

1 participant