Skip to content

fix: clarify custom provider configuration errors - #134

Closed
EF1874 wants to merge 17 commits into
MemTensor:mainfrom
EF1874:v1.0.5
Closed

fix: clarify custom provider configuration errors#134
EF1874 wants to merge 17 commits into
MemTensor:mainfrom
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
@syzsunshine219
syzsunshine219 changed the base branch from v1.0.5 to main August 6, 2026 11:37
@EF1874 EF1874 closed this Aug 7, 2026
@EF1874
EF1874 deleted the v1.0.5 branch August 7, 2026 01: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.

4 participants