Repository navigation
feat(cli): add transitrix bin alias, deprecate cervin (P1) - #133
Merged
Merged
Conversation
First phase of the Cervin → Transitrix CLI rename (CLAUDE.md §Cervin naming, P1). No breaking change. - package.json: add `transitrix` bin entry and `npm run transitrix` script, both resolving to the same dist/cli.js; keep `cervin` bin. - cli.ts: usage/--help now recommend `transitrix`; emit a one-line deprecation notice to stderr when launched under the legacy `cervin` name (best-effort detection from argv[1], degrades silently on Windows .cmd shims). - cli-parse.ts: add pure `invokedAsCervin()` helper and CERVIN_DEPRECATION_NOTICE, with unit tests. - CHANGELOG: note transitrix CLI added, cervin deprecated. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Signed-off-by: Valerii Korobeinikov <vkgeorgia@icloud.com>
📊 Metrics Regression Report✅ All metrics within tolerance Summary
Detailsai-expense-approval.bpmn.transitrix.yaml
feature-release.bpmn.transitrix.yaml
large-cyclic-workflow.bpmn.transitrix.yaml
order-fulfillment.bpmn.transitrix.yaml
simple-approval.bpmn.transitrix.yaml
simple-linear.bpmn.transitrix.yaml
small-dense-approval.bpmn.transitrix.yaml
xlarge-stress-test.bpmn.transitrix.yaml
|
8 tasks done
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
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
First phase of the Cervin → Transitrix gradual CLI rename (CLAUDE.md §Cervin naming, P1). No breaking change —
cervinkeeps working.Closes the acceptance criteria of strategy hub issue #188.
Changes
package.json—bingains atransitrixentry pointing at the samedist/cli.jsascervin; added annpm run transitrixscript alongsidenpm run cervin. Thecervinbin is kept (no removal this phase).src/cli.ts— usage/--helptext now recommendstransitrix; a one-line deprecation notice is printed to stderr when the tool is launched under the legacycervinname (so it never pollutes--jsonoutput).src/cli-parse.ts— new pureinvokedAsCervin(argv1)helper +CERVIN_DEPRECATION_NOTICEconstant. Detection is best-effort fromargv[1]: fires on POSIX symlink shims (.../bin/cervin), degrades silently on Windows.cmdshims (which invoke node with the resolvedcli.jspath).tests/cli-parse.test.ts— unit tests for the detector and notice (POSIX/Windows paths, case-insensitivity, negative cases).CHANGELOG.md— Added/Deprecated entries: "cervin CLI is deprecated, use transitrix".Verification
npm run compile(tsc --noEmit) — clean.npx vitest run— 242 tests green.--helpshowstransitrix …usage; deprecation notice fires for an argv[1] basename ofcervin.Notes
Internal error-message prefixes (
cervin: …) are intentionally left untouched — they're part of the later opportunistic internal rename, not the user-facing P1 surface. Next phase: P2 (extension settings, #189).🤖 Generated with Claude Code