Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .changeset/cruel-rings-play.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
"@calycode/cli": patch

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🟠 Major | ⚡ Quick win

Use a minor changeset for this command removal.

Line 2 marks this as patch, but removing the xano command is a breaking CLI contract change and should follow the repo’s pre-1.0 policy as minor.

Suggested fix
-"`@calycode/cli`": patch
+"`@calycode/cli`": minor

Based on learnings, breaking changes for pre-1.0 internal packages in this repo are intentionally released as minor, and non-minor is treated as incorrect.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"@calycode/cli": patch
"`@calycode/cli`": minor
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In @.changeset/cruel-rings-play.md at line 2, The changeset currently lists the
`@calycode/cli` update as "patch" but removing the xano CLI command is a breaking
pre-1.0 change and must be released as "minor"; update the entry for
"`@calycode/cli`" in the .changeset/cruel-rings-play.md file to use minor instead
of patch so the release follows the repo's pre-1.0 policy.

Source: Learnings

---

chore: remove the legacy 'xano' command handling completely to avoid collisions with xano cli
37 changes: 27 additions & 10 deletions packages/cli/esbuild.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,16 +13,34 @@ function defineEnvValue(name: string): string {
}

const buildEnvDefines: Record<string, string> = {
'process.env.CALY_BUILD_OC_EXT_DISCOVERY_MODE': defineEnvValue('CALY_BUILD_OC_EXT_DISCOVERY_MODE'),
'process.env.CALY_BUILD_OC_EXT_DISCOVERY_MODE': defineEnvValue(
'CALY_BUILD_OC_EXT_DISCOVERY_MODE',
),
'process.env.CALY_BUILD_OC_EXT_NAME': defineEnvValue('CALY_BUILD_OC_EXT_NAME'),
'process.env.CALY_BUILD_OC_EXT_TRUSTED_AUTHORS': defineEnvValue('CALY_BUILD_OC_EXT_TRUSTED_AUTHORS'),
'process.env.CALY_BUILD_OC_EXT_TRUSTED_HOMEPAGES': defineEnvValue('CALY_BUILD_OC_EXT_TRUSTED_HOMEPAGES'),
'process.env.CALY_BUILD_OC_EXT_TRUSTED_UPDATE_URLS': defineEnvValue('CALY_BUILD_OC_EXT_TRUSTED_UPDATE_URLS'),
'process.env.CALY_BUILD_OC_EXT_REQUIRE_NATIVE_MESSAGING': defineEnvValue('CALY_BUILD_OC_EXT_REQUIRE_NATIVE_MESSAGING'),
'process.env.CALY_BUILD_OC_EXT_PUBLIC_KEY_B64': defineEnvValue('CALY_BUILD_OC_EXT_PUBLIC_KEY_B64'),
'process.env.CALY_BUILD_OC_EXT_DISCOVERY_ENABLED': defineEnvValue('CALY_BUILD_OC_EXT_DISCOVERY_ENABLED'),
'process.env.CALY_BUILD_OC_EXT_INCLUDE_KNOWN_IDS': defineEnvValue('CALY_BUILD_OC_EXT_INCLUDE_KNOWN_IDS'),
'process.env.CALY_BUILD_OC_WRITE_ALL_BROWSER_MANIFESTS': defineEnvValue('CALY_BUILD_OC_WRITE_ALL_BROWSER_MANIFESTS'),
'process.env.CALY_BUILD_OC_EXT_TRUSTED_AUTHORS': defineEnvValue(
'CALY_BUILD_OC_EXT_TRUSTED_AUTHORS',
),
'process.env.CALY_BUILD_OC_EXT_TRUSTED_HOMEPAGES': defineEnvValue(
'CALY_BUILD_OC_EXT_TRUSTED_HOMEPAGES',
),
'process.env.CALY_BUILD_OC_EXT_TRUSTED_UPDATE_URLS': defineEnvValue(
'CALY_BUILD_OC_EXT_TRUSTED_UPDATE_URLS',
),
'process.env.CALY_BUILD_OC_EXT_REQUIRE_NATIVE_MESSAGING': defineEnvValue(
'CALY_BUILD_OC_EXT_REQUIRE_NATIVE_MESSAGING',
),
'process.env.CALY_BUILD_OC_EXT_PUBLIC_KEY_B64': defineEnvValue(
'CALY_BUILD_OC_EXT_PUBLIC_KEY_B64',
),
'process.env.CALY_BUILD_OC_EXT_DISCOVERY_ENABLED': defineEnvValue(
'CALY_BUILD_OC_EXT_DISCOVERY_ENABLED',
),
'process.env.CALY_BUILD_OC_EXT_INCLUDE_KNOWN_IDS': defineEnvValue(
'CALY_BUILD_OC_EXT_INCLUDE_KNOWN_IDS',
),
'process.env.CALY_BUILD_OC_WRITE_ALL_BROWSER_MANIFESTS': defineEnvValue(
'CALY_BUILD_OC_WRITE_ALL_BROWSER_MANIFESTS',
),
};

(async () => {
Expand All @@ -36,7 +54,6 @@ const buildEnvDefines: Record<string, string> = {
const result = await build({
entryPoints: {
index: resolve(rootDir, 'src/index.ts'),
'legacy-xano': resolve(rootDir, 'src/legacy-xano-command.ts'),
},
bundle: true,
platform: 'node',
Expand Down
4 changes: 1 addition & 3 deletions packages/cli/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
"developer-tools"
],
"bin": {
"xano": "dist/legacy-xano.cjs",
"caly-xano": "dist/index.cjs"
},
"main": "dist/index.cjs",
Expand Down Expand Up @@ -52,7 +51,6 @@
"commander": "^14.0.0",
"js-yaml": "^4.1.0",
"posthog-node": "^5.9.2",
"postject": "1.0.0-alpha.6",
"shx": "^0.4.0",
"tar": "^7.4.3"
},
Expand All @@ -62,7 +60,7 @@
"test:coverage": "jest --coverage",
"clean": "trash dist tsconfig.tsbuildinfo",
"build:js": "tsx esbuild.config.ts",
"build:chmod": "shx chmod +x dist/index.cjs dist/legacy-xano.cjs",
"build:chmod": "shx chmod +x dist/index.cjs",
"build": "pnpm clean && pnpm build:js && pnpm build:chmod && pnpm link -g",
"bundle:ncc": "pnpm ncc build dist/index.cjs -o dist/cli.cjs && shx cp README.md LICENSE dist/cli.cjs/",
"bundle:sea": "tsx build-sea.ts",
Expand Down
4 changes: 0 additions & 4 deletions packages/cli/src/legacy-xano-command.ts

This file was deleted.