Skip to content

fix: clarify custom provider configuration errors - #134

Open
EF1874 wants to merge 1 commit into
MemTensor:v1.0.5from
EF1874:v1.0.5
Open

fix: clarify custom provider configuration errors#134
EF1874 wants to merge 1 commit into
MemTensor:v1.0.5from
EF1874:v1.0.5

Conversation

@EF1874

@EF1874 EF1874 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • document the existing providers.*.apiBase override in the root README, Agent README, and English/Chinese BYOK docs
  • report the configured name for explicit unknown providers instead of provider 'null'
  • preserve existing known-provider credential errors and OAuth/local/direct/Bedrock exemptions

This addresses the first two items in #128; apiType and provider registry additions remain out of scope.

Testing

  • npm --prefix App/memmy-agent test -- tests/providers/providers-init.test.ts tests/providers/bedrock-provider.test.ts tests/providers/custom-provider.test.ts (21 passed)
  • npm --prefix App/memmy-agent run typecheck
  • built CLI validation with provider: not_registered confirmed the new error message

@ZongYue99
ZongYue99 requested a review from Wang-Daoji August 4, 2026 08: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.

1 participant