Skip to content

Conversation

@beonde
Copy link
Member

@beonde beonde commented Dec 11, 2025

Summary

Major refactoring to CLI-only passthrough architecture matching capiscio-python, plus version alignment to 2.2.0.

Changes

Architecture (CLI-Only)

  • Stripped to pure CLI wrapper - removed all programmatic API
  • Removed: CoreValidator, A2AValidator, types, scorers, output formatters
  • Kept: BinaryManager for downloading/caching capiscio-core binary
  • All commands passthrough to capiscio-core binary

Version Alignment

  • Package version: 2.2.0
  • Core version downloaded: v2.2.0
  • Updated CHANGELOG.md

Testing

  • Improved test coverage from 53% to 91%
  • Added 70% coverage threshold to vitest config
  • 24 tests passing

Removed Dependencies

  • commander, glob, inquirer, jose

Breaking Changes

  • Removed programmatic API exports
  • For programmatic usage, use @capiscio/sdk (coming soon)

Version Alignment

Package Version
capiscio-core 2.2.0
capiscio (npm) 2.2.0
capiscio (PyPI) 2.2.0

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.
Copilot AI review requested due to automatic review settings December 11, 2025 02:30
Copy link

Copilot AI left a 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-core binary 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.

Copilot AI review requested due to automatic review settings December 11, 2025 04:04
Copy link

Copilot AI left a 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.

Copilot AI review requested due to automatic review settings December 11, 2025 04:16
Copy link

Copilot AI left a 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.

@beonde beonde merged commit fa39ac4 into main Dec 11, 2025
4 checks passed
@beonde beonde deleted the chore/passthrough-architecture branch December 11, 2025 04:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants