-
Notifications
You must be signed in to change notification settings - Fork 0
chore: CLI-only passthrough architecture + version 2.2.0 #29
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
Conversation
BREAKING CHANGE: Package is now a pure CLI wrapper (no programmatic API). - Passthrough design delegating ALL commands to capiscio-core binary - Matches capiscio-python wrapper exactly - Added --wrapper-version and --wrapper-clean commands - Removed programmatic API (CoreValidator, A2AValidator, types) - Removed unused dependencies (commander, glob, inquirer, jose) - All core commands (badge, key, gateway, validate) work via passthrough
- Add comprehensive tests for BinaryManager - Test platform detection (darwin, linux, win32) - Test architecture handling (x64, arm64) - Test getBinaryPath with env var override - Test install error handling (404, network, generic errors) - Test findPackageRoot fallback behavior - Coverage: 53% → 91% - Tests: 4 → 24
Align with capiscio-core for unified versioning. All capiscio packages now share the same version number.
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.
Pull request overview
This PR implements a major architectural refactoring that strips the npm package down to a pure CLI wrapper (matching capiscio-python), removing all programmatic validation/scoring logic. The package now downloads and delegates to the capiscio-core v2.2.0 binary. Version alignment brings all CapiscIO packages to 2.2.0.
Key Changes:
- Removed all TypeScript validation, scoring, and type exports - now purely a CLI passthrough wrapper
- Updated
capiscio-corebinary version from v1.0.2 to v2.2.0 - Improved test coverage from 53% to 91% with new CLI wrapper tests
Reviewed changes
Copilot reviewed 52 out of 54 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
src/utils/binary-manager.ts |
Updated core version to v2.2.0 |
src/cli.ts |
Converted to pure passthrough architecture |
src/index.ts |
Exports only version and BinaryManager |
package.json |
Version 2.2.0, removed unused dependencies |
vitest.config.ts |
Added 70% coverage thresholds |
tsup.config.ts |
Updated externals list |
CHANGELOG.md |
Documented breaking changes |
docs/** |
Updated documentation for CLI-only usage |
tests/** |
Removed validator/scoring tests, added CLI wrapper tests |
| Deleted files | Removed validator, scorer, output formatter modules |
No issues found. The refactoring is well-documented, properly tested, and maintains backward compatibility for CLI users while clearly documenting breaking changes for programmatic API users.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Pull request overview
Copilot reviewed 52 out of 54 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (1)
src/utils/binary-manager.ts:1
- The version 'v2.2.0' is being set as the default version for the capiscio-core binary. Since my knowledge cutoff is January 2025 and it's currently December 11, 2025, I cannot verify if this version exists. Please ensure that capiscio-core v2.2.0 has been released and is available for download before merging this PR.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Pull request overview
Copilot reviewed 53 out of 55 changed files in this pull request and generated no new comments.
Comments suppressed due to low confidence (1)
src/utils/binary-manager.ts:1
- The version 'v2.2.0' appears to reference a release date of December 10, 2025 (per CHANGELOG.md line 10), which is in the future relative to the current date (December 11, 2025). This suggests the date in the CHANGELOG may be incorrect.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
Major refactoring to CLI-only passthrough architecture matching
capiscio-python, plus version alignment to 2.2.0.Changes
Architecture (CLI-Only)
CoreValidator,A2AValidator, types, scorers, output formattersBinaryManagerfor downloading/caching capiscio-core binaryVersion Alignment
2.2.0v2.2.0Testing
Removed Dependencies
commander,glob,inquirer,joseBreaking Changes
@capiscio/sdk(coming soon)Version Alignment