Skip to content

feat(bazaarlink): add authHint to existing APIKEY_PROVIDERS entry - #4522

Merged
diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.33from
adivekar-utexas:feat/bazaarlink-auth-hint
Jun 21, 2026
Merged

diegosouzapw merged 1 commit into
cheaper-inference:release/v3.8.33from
adivekar-utexas:feat/bazaarlink-auth-hint

Conversation

@adivekar-utexas

@adivekar-utexas adivekar-utexas commented Jun 21, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Bazaarlink was already registered in APIKEY_PROVIDERS — connection creation
and validation already worked without any changes. This PR adds the missing
UX fields (authHint, improved freeNote, improved apiHint) so that users
see the expected key format and endpoint URL in the UI.

Changes

  • Added authHint to the existing bazaarlink entry (documents the sk-bl-
    key prefix and provider/model-name format requirement)
  • Updated freeNote to mention the 4M tokens/day free tier limit
  • Updated apiHint with clearer setup instructions and model format examples

Tests

Added tests/unit/bazaarlink-provider-integrity.test.ts covering:

  • Full provider entry integrity (all required fields, alias, icon, etc.)
  • Free tier and authHint presence
  • Registry registration (format, authType, model list completeness)
  • Static catalog resolution into the apikey category
  • isManagedProviderConnectionId returns true (gate check for connection creation)
  • Bulk API key support (not in exclusion list)

Result: 6 passing

Related

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@adivekar-utexas

Copy link
Copy Markdown
Contributor Author

CI status

All failures in this run are pre-existing on main — not introduced by this PR:

  • Lint: 3867 warnings (0 errors) from any-type usage across the codebase. Same result on main with identical count.
  • Docs Sync (Strict): env-doc contract drift — KIRO_VERIFY_FULL_CRC referenced in code but missing from .env.example. Pre-existing on main.
  • Coverage / Integration / Node 24: same root cause (env contract check in the coverage pipeline).

This PR only touches the existing bazaarlink entry in APIKEY_PROVIDERS (+5 lines for authHint, improved freeNote, improved apiHint) and adds a dedicated test file (6 tests, all passing).

PR #4497 was closed due to a duplicate-key defect — this branch (feat/bazaarlink-auth-hint) replaces it with the correct fix.

Adds authHint (sk-bl- key prefix + OpenAI-SDK base URL) and enriches
freeNote/apiHint on the existing bazaarlink entry. Integrity test included.
@diegosouzapw
diegosouzapw force-pushed the feat/bazaarlink-auth-hint branch from f1cdd1f to 0b353f0 Compare June 21, 2026 18:42
@diegosouzapw
diegosouzapw changed the base branch from main to release/v3.8.33 June 21, 2026 18:42
@diegosouzapw
diegosouzapw merged commit f91cf72 into cheaper-inference:release/v3.8.33 Jun 21, 2026
3 checks passed
@diegosouzapw diegosouzapw mentioned this pull request Jun 22, 2026
tkgo11 pushed a commit to tkgo11/OmniRoute that referenced this pull request Sep 23, 2026
…eaper-inference#4522)

Adds authHint + enriched freeNote/apiHint on the existing bazaarlink entry (distinct from the closed-redundant cheaper-inference#4497). Integrity test included.

Integrated into release/v3.8.33.
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