Repository navigation
feat: add Tencent TokenHub direct API support - #900
Merged
Merged
Conversation
57 tasks done
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
hy3/hy3-previewsnapshot fallback, and cover Tencent's two-stage reasoning/tool-call replay contract deterministically.Why
Refs #860. Tencent TokenHub direct API access was missing from Maka's provider catalog and runtime contract. This slice adds only the direct API identity and preserves the existing single-registry ownership, discovery fallback, provider ordering, and brand-governance mechanisms introduced by preceding provider PRs.
Scope
This PR is limited to Tencent TokenHub direct API support. It does not add Tencent Coding Plan, Tencent Token Plan, any Phase 7 work, or any newly expanded provider-matrix item.
Verification
npm run typecheck— passed for all workspaces on the final NVIDIA-based branch.node --test scripts/sync-model-metadata.test.mjs— 11/11 passed.npm --workspace @maka/core test— passed, including the Tencent registry, credential URL, ordering, exact model snapshot, and brand contracts.reasoning_contentand completetool_callswithout real secrets.THIRD_PARTY_LICENSESchecks passed.git diff --checkpassed; the feature worktree is clean.npm testsuite passed before the final NVIDIA rebase, including 2,414 Desktop tests. The final affected suites above were rerun after rebasing.Impact
Users can configure Tencent TokenHub with persisted provider ID
tencent-tokenhub, an API key, and the defaulthttps://tokenhub.tencentmaas.com/v1endpoint, with optionalTENCENT_TOKENHUB_API_KEY,TENCENT_TOKENHUB_API_KEY_FILE, andTENCENT_TOKENHUB_BASE_URLheadless configuration. Existing persisted providers need no migration. Release notes should mention Tencent TokenHub direct API support.Reviewer notes
GET /v1/modelsdiscovery, exacthy3/hy3-previewIDs, and distinct OpenAI versus Anthropic compatibility paths. This implementation deliberately uses only the OpenAI protocol.@lobehub/icons-static-svg@1.91.0, immutable commite4302041fbb3039608d25f9f618bd462783b875e, pathpackages/static-svg/icons/hunyuan.svg, under the MIT license. SHA-256:7306a65eb71c4de61e21a637e5c4fef94afde823678e225c46f891cc783f6531.reasoning_contentreplay. Both were fixed with regression tests; the repeated review reported no P0–P3 findings and explicitly passed the first-principles/Occam's-razor standard.Ready for review
Verificationexplains why it is not applicable