Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@jonisjongithub
Copy link
Contributor

Summary

Fixes missing CLI support for Venice AI provider onboarding. PR #1666 added the Venice AI provider integration but the CLI onboarding flags were not wired up, causing users to be unable to configure Venice via clawdbot onboard.

Problem

After PR #1666, users trying to onboard with Venice experienced:

  • --auth-choice venice-api-key not shown in help text
  • --venice-api-key flag missing entirely
  • Credentials not being saved even when using workarounds

Solution

Added the missing CLI wiring by following the exact pattern used by other API key providers (synthetic, openrouter, moonshot, etc.):

  1. src/cli/program/register.onboard.ts

    • Added venice-api-key to --auth-choice help text
    • Added --venice-api-key <key> CLI option
    • Wired veniceApiKey through to onboardCommand
  2. src/commands/onboard-non-interactive/local/auth-choice.ts

    • Added imports for setVeniceApiKey and applyVeniceConfig
    • Added handler for authChoice === "venice-api-key"

Testing

  • pnpm build - passes
  • pnpm lint - passes
  • pnpm format - passes
  • Verified Venice API connectivity (curl test successful)
  • Code review: follows existing patterns exactly

Usage

After this fix, users can onboard with Venice via:

clawdbot onboard --non-interactive --accept-risk \
  --auth-choice venice-api-key \
  --venice-api-key "YOUR_VENICE_API_KEY"

Fixes issue with Venice onboarding discovered while testing #1666.

Fixes missing CLI support for Venice AI provider onboarding. PR moltbot#1666 added
the Venice AI provider integration but the CLI onboarding flags were not
wired up, causing users to be unable to configure Venice via clawdbot onboard.

Changes:
- Add venice-api-key to --auth-choice help text
- Add --venice-api-key CLI option
- Wire veniceApiKey through to onboardCommand
- Add handler for authChoice === 'venice-api-key'

Follows the exact pattern used by other API key providers (synthetic,
openrouter, moonshot, etc.)

Co-authored-by: Jon Shapiro <[email protected]>
Co-authored-by: Clawdbot <[email protected]>
@thewilloftheshadow
Copy link
Member

Squash-merged on main.

Commits:

Tests:

  • pnpm lint (fails: existing oxlint errors in src/line/*)
  • pnpm build (fails: missing @aws-sdk/client-bedrock and @line/bot-sdk)
  • pnpm test (fails: missing @aws-sdk/client-bedrock + many pre-existing failures)

steipete pushed a commit to mertcicekci0/clawdbot that referenced this pull request Jan 26, 2026
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