Skip to content

Feature/add hpc ai provider#1586

Open
lioZ129 wants to merge 3 commits intoPortkey-AI:mainfrom
lioZ129:feature/add-hpc-ai-provider
Open

Feature/add hpc ai provider#1586
lioZ129 wants to merge 3 commits intoPortkey-AI:mainfrom
lioZ129:feature/add-hpc-ai-provider

Conversation

@lioZ129
Copy link
Copy Markdown

@lioZ129 lioZ129 commented Mar 30, 2026

Description: (required)

  • Add native provider hpc-ai with OpenAI-compatible chat completions: default base URL https://api.hpc-ai.com/inference/v1, Authorization: Bearer, endpoints /chat/completions (non-stream + stream), response transforms for OpenAI-style errors, optional FastAPI-style detail errors, and provider tagging on success/stream chunks.
  • Support optional Node overrides: custom_host / x-portkey-custom-host, and env HPC_AI_BASE_URL after customHost; document HPC_AI_API_KEY / HPC_AI_BASE_URL in .env.example and integration test vars in testVariables.ts.
  • Register HPC_AI in VALID_PROVIDERS; add models minimax/minimax-m2.5 and moonshotai/kimi-k2.5 to models.json under provider hpc-ai.
  • Update local Gateway Console (public/index.html): provider option, modelMap / docsMap for HPC-AI.
  • README / Japanese README table: HPC-AI row with link to https://www.hpc-ai.com/ and logo https://company.hpc-ai.com/hubfs/logos/colossal-ai_logo_vertical.svg.

Tests Run/Test cases added: (required)

  • npm run build — passed
  • npm run format:check — passed
  • No new automated test cases added (existing npm run test:gateway has pre-existing failures unrelated to this change in this environment)

Type of Change:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)

Copilot AI review requested due to automatic review settings March 30, 2026 09:41
Copy link
Copy Markdown
Contributor

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

Adds a new native hpc-ai provider to the Gateway with OpenAI-compatible /chat/completions support, wiring it through provider registration, model catalog, local console UI, and documentation.

Changes:

  • Implement hpc-ai provider configuration (base URL, auth header, endpoint mapping, response + stream chunk transforms).
  • Register hpc-ai across the gateway (providers index, VALID_PROVIDERS), add HPC-AI models to models.json, and add integration-test variables.
  • Update local Gateway Console UI and docs (README + JP README) to expose/select HPC-AI.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
src/tests/resources/testVariables.ts Adds hpc-ai env var hook-up for integration tests.
src/public/index.html Adds HPC-AI to provider dropdown plus model/docs maps in the local console.
src/providers/index.ts Registers the new hpc-ai provider config.
src/providers/hpc-ai/index.ts New provider entry-point wiring API + transforms.
src/providers/hpc-ai/chatComplete.ts New chat completion request config and response/stream transforms with provider tagging.
src/providers/hpc-ai/api.ts New base URL resolution (customHost/env/default), auth header, and endpoint mapping.
src/globals.ts Adds HPC_AI constant and registers it in VALID_PROVIDERS.
src/data/models.json Adds two HPC-AI models to the global model list.
README.md Adds HPC-AI to supported providers and documents usage/overrides.
.github/README.jp.md Adds HPC-AI to the Japanese supported providers table.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@lioZ129
Copy link
Copy Markdown
Author

lioZ129 commented Apr 7, 2026

@narengogi could you please review this & docs?

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