-
Notifications
You must be signed in to change notification settings - Fork 0
refactor!: rename command #188
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
91d81cf
d10df86
7c26d1e
24888af
cf564bb
037ca67
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "@calycode/xano-skills": patch | ||
| --- | ||
|
|
||
| chore: improve installability |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| "@calycode/cli": minor | ||
| --- | ||
|
|
||
| refactor: **BREAKING CHANGE** rename CLI command from `xano` to `caly-xano`; update scripts, CI, and docs to use `caly-xano` | ||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -37,7 +37,7 @@ pnpm install | |||||||||||||||||||||||||||||||
| pnpm build | ||||||||||||||||||||||||||||||||
| # 4. Run any command | ||||||||||||||||||||||||||||||||
| xano | ||||||||||||||||||||||||||||||||
| caly-xano | ||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| > _(Optional)_ If you want the CLI globally available during development: | ||||||||||||||||||||||||||||||||
|
|
@@ -61,17 +61,17 @@ I have been astonished by the shadcn/ui CLI and the core principles of code dist | |||||||||||||||||||||||||||||||
| 1. Scaffold the registry or build it manually by obeying the schemas (https://calycode.com/schemas/registry/registry.json). | ||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||
| xano registry-scaffold | ||||||||||||||||||||||||||||||||
| caly-xano registry-scaffold | ||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||
| 2. Serve your registry locally or host it on an object storage (or [advanced] recreate a Xano api that would deliver the required JSON objects on demand --> this could allow you to add auth as well) | ||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||
| xano serve-registry | ||||||||||||||||||||||||||||||||
| caly-xano serve-registry | ||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||
| 3. Use the registry and it's content in `xano` | ||||||||||||||||||||||||||||||||
| 3. Use the registry and it's content in `caly-xano` | ||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||
|
Comment on lines
68
to
73
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add fenced code languages to satisfy markdownlint MD040. The updated fenced blocks in this section should declare a language. Proposed fix- ```
+ ```bash
caly-xano serve registry
```
@@
- ```
+ ```bash
caly-xano registry add <comma-separated component names> --registry <registry url>
```🧰 Tools🪛 markdownlint-cli2 (0.22.0)[warning] 68-68: Fenced code blocks should have a language specified (MD040, fenced-code-language) [warning] 73-73: Fenced code blocks should have a language specified (MD040, fenced-code-language) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||
| xano registry-add --components <coma separated component names> --registry <registry url> | ||||||||||||||||||||||||||||||||
| caly-xano registry-add --components <coma separated component names> --registry <registry url> | ||||||||||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||||||||||
|
Comment on lines
+64
to
75
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Use the current command syntax in registry examples (these forms are stale). The README still shows hyphenated forms ( Proposed fix- caly-xano registry-scaffold
+ caly-xano registry scaffold
@@
- caly-xano serve-registry
+ caly-xano serve registry
@@
-3. Use the registry and it's content in `caly-xano`
+3. Use the registry and its content in `caly-xano`
```
- caly-xano registry-add --components <coma separated component names> --registry <registry url>
+ caly-xano registry add <comma-separated component names> --registry <registry url>
```📝 Committable suggestion
Suggested change
🧰 Tools🪛 markdownlint-cli2 (0.22.0)[warning] 68-68: Fenced code blocks should have a language specified (MD040, fenced-code-language) [warning] 73-73: Fenced code blocks should have a language specified (MD040, fenced-code-language) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||||||||||
| > [!NOTE] > **Notes:** | ||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -41,7 +41,7 @@ pnpm install | |||||||||||||||||||||||
| pnpm build | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| # 4. Run any command | ||||||||||||||||||||||||
| xano | ||||||||||||||||||||||||
| caly-xano | ||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| > _(Optional)_ If you want the CLI globally available during development: | ||||||||||||||||||||||||
|
|
@@ -65,17 +65,17 @@ I have been astonished by the shadcn/ui CLI and the core principles of code dist | |||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| 1. Scaffold the registry or build it manually by obeying the schemas (https://calycode.com/schemas/registry/registry.json). | ||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||
| xano registry-scaffold | ||||||||||||||||||||||||
| caly-xano registry-scaffold | ||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||
| 2. Serve your registry locally or host it on an object storage (or [advanced] recreate a Xano api that would deliver the required JSON objects on demand --> this could allow you to add auth as well) | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||
| xano serve-registry | ||||||||||||||||||||||||
| caly-xano serve-registry | ||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| 3. Use the registry and it's content in `xano` | ||||||||||||||||||||||||
| 3. Use the registry and it's content in `caly-xano` | ||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||
| xano registry-add --components <coma separated component names> --registry <registry url> | ||||||||||||||||||||||||
| caly-xano registry-add --components <coma separated component names> --registry <registry url> | ||||||||||||||||||||||||
| ``` | ||||||||||||||||||||||||
|
Comment on lines
72
to
79
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Fix fenced code language and typo in registry example. Line 72/77 should include a fence language (MD040), and Line 78 should say “comma separated”. 💡 Suggested patch- ```
+ ```term
caly-xano serve-registry
```
3. Use the registry and it's content in `caly-xano`
- ```
- caly-xano registry-add --components <coma separated component names> --registry <registry url>
+ ```term
+ caly-xano registry-add --components <comma separated component names> --registry <registry url>
```📝 Committable suggestion
Suggested change
🧰 Tools🪛 markdownlint-cli2 (0.22.0)[warning] 72-72: Fenced code blocks should have a language specified (MD040, fenced-code-language) [warning] 77-77: Fenced code blocks should have a language specified (MD040, fenced-code-language) 🤖 Prompt for AI Agents |
||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| > [!NOTE] > **Notes:** | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -3,7 +3,7 @@ | |||||
| > #### Create a library based on the OpenAPI specification. If the openapi specification has not yet been generated, this will generate that as well as the first step. Supports **all** openapi tools generators + orval clients. | ||||||
|
|
||||||
| ```term | ||||||
| $ xano generate codegen [options] | ||||||
| $ caly-xano generate codegen [options] | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove shell prompt marker in command-only fenced block. Line 6 triggers MD014 ( 💡 Suggested patch- $ caly-xano generate codegen [options]
+ caly-xano generate codegen [options]📝 Committable suggestion
Suggested change
🧰 Tools🪛 markdownlint-cli2 (0.22.0)[warning] 6-6: Dollar signs used before commands without showing output (MD014, commands-show-output) 🤖 Prompt for AI Agents
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. That's just used for styling. The
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
✏️ Learnings added
🧠 Learnings used |
||||||
| ``` | ||||||
| ### Options | ||||||
|
|
||||||
|
|
@@ -26,10 +26,10 @@ $ xano generate codegen [options] | |||||
|
|
||||||
| ### generate codegen --help | ||||||
| ```term | ||||||
| $ xano generate codegen --help | ||||||
| $ caly-xano generate codegen --help | ||||||
| Create a library based on the OpenAPI specification. If the openapi specification has not yet been generated, this will generate that as well as the first step. Supports **all** openapi tools generators + orval clients. | ||||||
|
|
||||||
| Usage: xano generate codegen [options] [passthroughArgs...] | ||||||
| Usage: caly-xano generate codegen [options] [passthroughArgs...] | ||||||
|
|
||||||
| Arguments: | ||||||
| └─ passthroughArgs Additional arguments to pass to the generator. For options for each generator see https://openapi-generator.tech/docs/usage#generate this also accepts Orval additional arguments e.g. --mock etc. See Orval docs as well: https://orval.dev/reference/configuration/full-example | ||||||
|
|
@@ -45,6 +45,6 @@ Options: | |||||
| ├─ --debug Specify this flag in order to allow logging. Logs will appear in output/_logs. Default: false | ||||||
| └─ -h, --help display help for command | ||||||
|
|
||||||
| Run 'xano <command> --help' for detailed usage. | ||||||
| Run 'caly-xano <command> --help' for detailed usage. | ||||||
| https://github.com/calycode/xano-tools | https://links.calycode.com/discord | ||||||
| ``` | ||||||
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Repository: calycode/xano-tools
Length of output: 199
🏁 Script executed:
Repository: calycode/xano-tools
Length of output: 2290
Confirm whether breaking changes in pre-1.0 packages use
minorormajorbumps.This changeset marks the CLI command rename as a breaking change but declares
@calycode/cliasminor. While the repository has historically not usedmajorbumps (all changes arepatchorminor), this is the first entry with aBREAKING CHANGEmarker. If breaking changes should release asmajor, this should be corrected before publishing. Alternatively, if the repo intentionally treats pre-1.0 breaking changes asminor, this should be explicitly documented.🤖 Prompt for AI Agents
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's fine we are far from anything like a major release, and there's no actual user anyways just yet other than internal team members.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@MihalyToth20, understood! That's a perfectly reasonable approach for a pre-1.0 package with only internal consumers. I'll note this for future reviews.✏️ Learnings added
🧠 Learnings used