Skip to content

feat: add qiniucloud provider#202

Open
huaiyuWangh wants to merge 2 commits intocharmbracelet:mainfrom
huaiyuWangh:feat/add-qiniucloud-provider
Open

feat: add qiniucloud provider#202
huaiyuWangh wants to merge 2 commits intocharmbracelet:mainfrom
huaiyuWangh:feat/add-qiniucloud-provider

Conversation

@huaiyuWangh
Copy link

@huaiyuWangh huaiyuWangh commented Mar 3, 2026

Summary

  • add QiniuCloud inference provider (qiniucloud)
  • register provider in embedded providers registry
  • add provider ID to known providers
  • sync internal/providers/configs/qiniucloud.json with the latest model list

Validation

  • go test ./...

@huaiyuWangh huaiyuWangh requested a review from a team as a code owner March 3, 2026 15:23
@huaiyuWangh huaiyuWangh requested review from meowgorithm and raphamorim and removed request for a team March 3, 2026 15:23
@huaiyuWangh huaiyuWangh changed the title feat: add qiniucloud provider and sync model list feat: add qiniucloud provider Mar 3, 2026
Copy link
Member

@andreynering andreynering left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @huaiyuWangh,

Added a comment below.

Also, can you confirm that you tried this on Crush to ensure it's working correctly? Let me know if you need help knowing how.

Comment on lines +13 to +16
"cost_per_1m_in": 0,
"cost_per_1m_out": 0,
"cost_per_1m_in_cached": 0,
"cost_per_1m_out_cached": 0,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed you kept all cots zeroed. Don't we know them?

Is this a coding plan or a token credit based provider?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the review — I’ve updated this based on the official QiniuCloud model catalog/pricing data and pushed the fixes to this PR.

@huaiyuWangh
Copy link
Author

huaiyuWangh commented Mar 5, 2026

@andreynering I tested it locally and the integration works
image
image

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