Skip to content

Conversation

TensorNull
Copy link

@TensorNull TensorNull commented Oct 13, 2025

@Copilot Copilot AI review requested due to automatic review settings October 13, 2025 06:14
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds CometAPI as a new inference provider to the Hugging Face inference package, enabling support for chat completion, text generation, and feature extraction tasks through CometAPI's service.

Key changes:

  • Added CometAPI provider implementation with support for conversational, text generation, and feature extraction tasks
  • Integrated CometAPI into the provider registry and configuration
  • Added comprehensive test coverage for all supported CometAPI functionality

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/inference/src/types.ts Added "cometapi" to the list of supported inference providers
packages/inference/src/providers/consts.ts Added empty cometapi entry to the hardcoded model inference mapping
packages/inference/src/providers/cometapi.ts New provider implementation with task classes for conversational, text generation, and feature extraction
packages/inference/src/lib/getProviderHelper.ts Registered CometAPI task implementations in the provider registry
packages/inference/test/InferenceClient.spec.ts Added comprehensive test suite for CometAPI functionality
packages/inference/README.md Added CometAPI to provider documentation and supported models list

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@TensorNull
Copy link
Author

@julien-c
Hello, we sincerely hope to integrate CometAPI into Hugging Face. We have activated the official CometAPI account with a “Subscription Team Plan” and have specifically prepared the billing interface in the CometAPI backend. Could you help us review this PR? By the way, it would be great if you could also give us some detailed suggestions on integration implementation.

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