docs: update CLI naming to capiscio (consolidation)#34
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates repository references from capiscio-cli to capiscio-node and bumps the version to 2.4.0. However, there are critical issues: the PR description incorrectly states it updates naming from capiscio-cli to capiscio (when it's actually updating to capiscio-node), and the CHANGELOG claims to update to capiscio-core v2.4.0 without actually modifying the code to do so.
Changes:
- Bumped package version from 2.3.1 to 2.4.0
- Updated repository URLs from
capiscio/capiscio-clitocapiscio/capiscio-nodein CONTRIBUTING.md and workflow README - Added CHANGELOG entry for version 2.4.0
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
| package.json | Version bumped from 2.3.1 to 2.4.0 |
| CONTRIBUTING.md | Updated repository URLs and added clarification about repository vs package naming |
| CHANGELOG.md | Added entry for version 2.4.0 claiming update to capiscio-core v2.4.0 |
| .github/workflows/README.md | Updated repository references and added clarification about package naming |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| 1. Fork the repository | ||
| 2. Clone your fork: `git clone https://github.com/your-username/capiscio-cli.git` | ||
| 2. Clone your fork: `git clone https://github.com/your-username/capiscio-node.git` |
There was a problem hiding this comment.
The PR description states "Updates CLI naming references from capiscio-cli to capiscio", but the actual changes are updating repository URLs from capiscio/capiscio-cli to capiscio/capiscio-node. The repository name is being changed to capiscio-node, not capiscio. The npm package name is capiscio, but that's different from the repository name. Please update the PR description to accurately reflect that this is updating repository references from capiscio-cli to capiscio-node.
| ## [2.4.0] - 2026-01-18 | ||
|
|
||
| ### Changed | ||
| - **CORE VERSION**: Now downloads `capiscio-core` v2.4.0 |
There was a problem hiding this comment.
The CHANGELOG entry for version 2.4.0 only mentions updating the core version to v2.4.0, but doesn't document the repository naming changes made in this PR (updating references from capiscio-cli to capiscio-node). The CHANGELOG should comprehensively document all changes made in this release, including the repository URL updates in CONTRIBUTING.md and the workflow README.
| - **CORE VERSION**: Now downloads `capiscio-core` v2.4.0 | |
| - **CORE VERSION**: Now downloads `capiscio-core` v2.4.0 | |
| - **REPOSITORY RENAME**: Updated project naming from `capiscio-cli` to `capiscio-node`. | |
| - Updated repository URLs and references in `CONTRIBUTING.md`. | |
| - Updated workflow documentation and badges in the workflow README to point to the new repository name. |
| ## [2.4.0] - 2026-01-18 | ||
|
|
||
| ### Changed | ||
| - **CORE VERSION**: Now downloads `capiscio-core` v2.4.0 |
There was a problem hiding this comment.
The CHANGELOG entry claims "Now downloads capiscio-core v2.4.0", but the DEFAULT_VERSION in src/utils/binary-manager.ts is still set to 'v2.2.0'. The code needs to be updated to match the CHANGELOG, or the CHANGELOG entry should be corrected to reflect the actual version being used.
| - **CORE VERSION**: Now downloads `capiscio-core` v2.4.0 | |
| - **CORE VERSION**: Still downloads `capiscio-core` v2.2.0 (no core binary change in this wrapper release). |
1 participant
Updates CLI naming references from
capiscio-clitocapisciofor consistency.Part of the CLI naming consolidation effort.