Skip to content

added tencent hy3 to opengateway available models - #1876

Merged
kevincodex1 merged 1 commit into
Twigpine:mainfrom
kevincodex1:feat/opengateway-tencent-hy3
Jul 7, 2026
Merged

kevincodex1 merged 1 commit into
Twigpine:mainfrom
kevincodex1:feat/opengateway-tencent-hy3

Conversation

@kevincodex1

@kevincodex1 kevincodex1 commented Jul 7, 2026 •

Copy link
Copy Markdown
Member

new model for opengateway

Summary by CodeRabbit

  • New Features

    • Added support for the Tencent HY3 model, including its availability in model selection.
    • Enabled Tencent HY3 through the available gateway catalog as a free option.
    • Exposed common capabilities for this model, including vision, streaming, function calling, JSON mode, and reasoning.
  • Tests

    • Updated model selection expectations to include the new Tencent HY3 option.

@coderabbitai

coderabbitai Bot commented Jul 7, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: c721da5c-3aac-4cd5-9266-256a1ebcd7d5

📥 Commits

Reviewing files that changed from the base of the PR and between 4be017b and d61bf95.

⛔ Files ignored due to path filters (1)
  • src/integrations/generated/integrationArtifacts.generated.ts is excluded by !**/generated/**, !src/integrations/generated/**
📒 Files selected for processing (4)
  • src/commands/model/model.test.tsx
  • src/integrations/brands/tencent.ts
  • src/integrations/gateways/gitlawb-opengateway.ts
  • src/integrations/models/tencent.ts
📜 Recent review details
⏰ Context from checks skipped due to timeout. (3)
  • GitHub Check: smoke-and-tests (22)
  • GitHub Check: typecheck
  • GitHub Check: smoke-and-tests (24.11.x)
🧰 Additional context used
📓 Path-based instructions (5)
src/**/*.{ts,tsx}

📄 CodeRabbit inference engine (AGENTS.md)

src/**/*.{ts,tsx}: Use TypeScript with strict mode
Use ESM imports in TypeScript source files

Files:

  • src/integrations/models/tencent.ts
  • src/integrations/gateways/gitlawb-opengateway.ts
  • src/integrations/brands/tencent.ts
  • src/commands/model/model.test.tsx
**

⚙️ CodeRabbit configuration file

**: # AGENTS.md - AI Agent Coding Guide

This guide is for AI coding agents working in the OpenClaude repository. Read it before changing code, and also follow CONTRIBUTING.md for contributor policy, PR expectations, review follow-up, and project scope.

Project Snapshot

OpenClaude is a coding-agent CLI for cloud and local model providers. It supports OpenAI-compatible APIs, Anthropic, Gemini, DeepSeek, Ollama, MCP, local backends, slash commands, tools, agents, and a React/Ink terminal UI.

The installed CLI runs on Node.js >=22.0.0. Bun is used for source builds, scripts, dependency management, and tests.

Work Style

  • Keep changes focused on one problem.
  • Prefer existing patterns in the file or nearby module.
  • Avoid unrelated formatting, renames, dependency changes, or broad rewrites.
  • Add or update tests when behavior changes.
  • Update docs when setup, commands, provider behavior, or user-facing behavior changes.
  • For new features, larger refactors, dependencies, or runtime changes, follow the issue-first guidance in CONTRIBUTING.md.

Stack And Conventions

  • TypeScript with strict mode and ESM imports.
  • React + Ink for terminal UI.
  • Bun lockfile and Bun scripts for development workflows.
  • Node runtime for the built CLI.

Common libraries and patterns:

  • chalk for terminal color.
  • commander for CLI argument parsing.
  • execa for child processes.
  • Existing service, provider, settings, permission, and UI patterns over new abstractions.

Repository Map

  • src/commands/ - slash and CLI command implementations.
  • src/components/ - React/Ink UI components.
  • src/services/ - API, MCP, OAuth, wiki, voice, and other service integrations.
  • src/tools/ - tool implementations.
  • src/utils/ - shared utilities.
  • src/integrations/ - provider and model integration metadata.
  • src/entrypoints/ - CLI, MCP, SDK, and generated public types.
  • src/tasks/ - local, remote, workflow, and monitor tas...

Files:

  • src/integrations/models/tencent.ts
  • src/integrations/gateways/gitlawb-opengateway.ts
  • src/integrations/brands/tencent.ts
  • src/commands/model/model.test.tsx
**/*

⚙️ CodeRabbit configuration file

**/*: Apply the OpenClaude maintainer review rubric from AGENTS.md. Review the current diff, not stale discussion context. Separate real blockers from suggestions. Do not request changes for vague style churn. Treat approval as merge-ready from CodeRabbit's side, pending required human review and GitHub Checks. If checks are failing or unavailable, say so clearly instead of implying the PR is fully ready.

Files:

  • src/integrations/models/tencent.ts
  • src/integrations/gateways/gitlawb-opengateway.ts
  • src/integrations/brands/tencent.ts
  • src/commands/model/model.test.tsx
{src/services/api/**,src/integrations/**,src/utils/model/**,src/utils/provider*.ts,src/commands/provider/**}

⚙️ CodeRabbit configuration file

{src/services/api/**,src/integrations/**,src/utils/model/**,src/utils/provider*.ts,src/commands/provider/**}: Review provider routing, model selection, env precedence, auth/token handling, OpenAI-compatible shims, retries, proxy behavior, and outbound HTTP behavior with high scrutiny. Block on silent default changes, hidden fallback expansion, credential reuse mistakes, hardcoded provider assumptions, or new network reach that is not intentional and documented.

Files:

  • src/integrations/models/tencent.ts
  • src/integrations/gateways/gitlawb-opengateway.ts
  • src/integrations/brands/tencent.ts
{src/**/*.test.ts,src/**/*.test.tsx,tests/**,scripts/**/*.test.ts,vscode-extension/**/*.test.js}

⚙️ CodeRabbit configuration file

{src/**/*.test.ts,src/**/*.test.tsx,tests/**,scripts/**/*.test.ts,vscode-extension/**/*.test.js}: Review tests for meaningful coverage of the changed behavior, isolation of global/env/config state, async cleanup, fake timers, provider profile leaks, and Windows-compatible assumptions. Block when risky runtime changes lack focused regression coverage or tests assert implementation details while missing the user-visible behavior.

Files:

  • src/commands/model/model.test.tsx
🔇 Additional comments (4)
src/integrations/brands/tencent.ts (1)

1-16: LGTM!

src/integrations/models/tencent.ts (1)

1-22: LGTM!

src/integrations/gateways/gitlawb-opengateway.ts (1)

136-142: LGTM!

src/commands/model/model.test.tsx (1)

1283-1283: LGTM!


📝 Walkthrough

Walkthrough

Adds a new Tencent brand and model integration (tencent/hy3) with defined capabilities, context window, and token limits, registers it as a free model entry in the gitlawb-opengateway catalog, and updates a model picker test to expect the new option.

Changes

Tencent HY3 model integration

Layer / File(s) Summary
Brand and model definitions
src/integrations/brands/tencent.ts, src/integrations/models/tencent.ts
New default exports define the Tencent brand (canonical vendor mapping, capability flags, model ID list) and the tencent/hy3 model (classification, capabilities, context window, max output tokens).
Catalog wiring and test update
src/integrations/gateways/gitlawb-opengateway.ts, src/commands/model/model.test.tsx
Adds a "Tencent HY3 Free" entry to the Opengateway catalog models list and updates the model picker test's expected options to include tencent/hy3.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

  • Gitlawb/openclaude#1515: Both PRs extend the same Opengateway catalog.models list and update the corresponding model picker test.
  • Gitlawb/openclaude#1539: Both PRs add a new "free" Opengateway catalog model entry and update the same test assertions.
  • Gitlawb/openclaude#1772: Both PRs add a new catalog model entry to the same gateway file and adjust the same test expectations.

Suggested labels: enhancement

Suggested reviewers: jatmn

🚥 Pre-merge checks | ✅ 5 | ❌ 2

❌ Failed checks (2 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is too vague and misses the required Summary, Impact, Testing, and Notes sections. Expand it to match the template with what changed, why, impact, testing, and any notes or limitations.
Risk Surface Disclosed ❓ Inconclusive PR adds OpenGateway/auth-routing model entries, but no review text is provided, so I can’t verify a risk-surface or blocker callout. Provide the actual review/comments (or approval note) so I can confirm whether the auth/provider-routing risk was explicitly addressed.
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title is concise and matches the diff by describing the new Tencent HY3 OpenGateway model entry.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
No Hidden Policy Change ✅ Passed PR only adds Tencent HY3 model/brand wiring and a test expectation; no telemetry, permission, trust-model, or routing-default fields changed, and gateway defaults stay the same.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands.

@kevincodex1
kevincodex1 merged commit 2a506f9 into Twigpine:main Jul 7, 2026
5 checks passed
hotmanxp added a commit to hotmanxp/openclaude that referenced this pull request Jul 7, 2026
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