Skip to content

Conversation

bhouston
Copy link
Member

Description

This PR introduces the initial implementation of citty as a replacement for yargs in the CLI package. It sets up a parallel implementation that can be toggled using an environment variable (USE_CITTY), allowing for gradual migration and testing.

Changes

  • Added citty as a dependency
  • Created a parallel implementation of the CLI using citty
  • Set up a feature flag (USE_CITTY=true) to switch between implementations
  • Preserved the original yargs implementation
  • Added start:citty script to test the new implementation
  • Created a detailed migration plan

Testing

The implementation can be tested by running:

pnpm -C packages/cli start:citty

This will run the CLI with the citty implementation. The original yargs implementation remains the default.

Next Steps

  • Complete test coverage for the citty implementation
  • Fix any remaining type issues
  • Address performance and usability concerns
  • Make citty the default implementation
  • Remove yargs implementation when stable

Related Issues

Closes #258

This commit:
- Adds citty as a dependency
- Creates a parallel implementation using citty
- Sets up a feature flag (USE_CITTY) to switch between implementations
- Preserves the original yargs implementation
- Adds start:citty script to test the new implementation

Refs #258
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.

Migrate CLI package from yargs to citty

1 participant