Skip to content

fix(cli): include custom providers in provider ls#459

Open
chenmijiang wants to merge 1 commit intogetpaseo:mainfrom
chenmijiang:fix/cli-provider-visibility
Open

fix(cli): include custom providers in provider ls#459
chenmijiang wants to merge 1 commit intogetpaseo:mainfrom
chenmijiang:fix/cli-provider-visibility

Conversation

@chenmijiang
Copy link
Copy Markdown

Summary

Before this change, the CLI connected to the daemon without an appVersion. The server then treated it as a legacy client and filtered provider snapshot entries down to legacy built-in providers only, so custom providers could appear in the app settings but not in paseo provider ls.

This change:

  • sends the CLI appVersion during daemon connection
  • restores custom provider visibility in paseo provider ls
  • adds a regression test for daemon-backed custom providers
  • extracts CLI version resolution into a shared helper reused by both the CLI entrypoint and daemon client

Test plan

  • npm run build --workspace=@getpaseo/cli
  • npx tsx packages/cli/tests/15-provider.test.ts
  • npx tsx packages/cli/tests/33-cli-version.test.ts

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.

1 participant