Skip to content

Feat/volcengine provider#3994

Open
Nioolek wants to merge 4 commits intoagentscope-ai:mainfrom
Nioolek:feat/volcengine-provider
Open

Feat/volcengine provider#3994
Nioolek wants to merge 4 commits intoagentscope-ai:mainfrom
Nioolek:feat/volcengine-provider

Conversation

@Nioolek
Copy link
Copy Markdown
Contributor

@Nioolek Nioolek commented May 1, 2026

Description

[Describe what this PR does and why]

添加火山引擎和其coding plan provider

Related Issue: Fixes #(issue_number) or Relates to #(issue_number)

Security Considerations: [If applicable, e.g. channel auth, env/config handling]

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation
  • Refactoring

Component(s) Affected

  • Core / Backend (app, agents, config, providers, utils, local_models)
  • Console (frontend web UI)
  • Channels (DingTalk, Feishu, QQ, Discord, iMessage, etc.)
  • Skills
  • CLI
  • Documentation (website)
  • Tests
  • CI/CD
  • Scripts / Deploy

Checklist

  • I ran pre-commit run --all-files locally and it passes
  • If pre-commit auto-fixed files, I committed those changes and reran checks
  • I ran tests locally (pytest or as relevant) and they pass
  • Documentation updated (if needed)
  • Ready for review

For Channel Changes (DingTalk, Feishu, QQ, Console, etc.)

  • I ran ./scripts/check-channels.sh (or ./scripts/check-channels.sh --changed) and it passes
  • Contract test exists in tests/contract/channels/test_<channel>_contract.py (REQUIRED)
  • Contract test implements create_instance() with proper channel initialization
  • All 19 contract verification points pass (see tests/contract/channels/__init__.py)
  • Optional: Unit tests in tests/unit/channels/test_<channel>.py for complex internal logic

Testing

[How to test these changes]

Local Verification Evidence

image

本地自测通过

pre-commit run --all-files
# paste summary result

pytest
# paste summary result

Additional Notes

    case "volcengine-cn":
    case "volcengine-cn-codingplan":
      return "https://VOLCENGINE_LOGO_CDN_URL_PLACEHOLDER";

logo内容目测需要你们上传到CDN上,获取到url后填充到这里。

Nioolek and others added 4 commits May 1, 2026 18:17
…lan modes

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-project-automation github-project-automation Bot moved this to Todo in QwenPaw May 1, 2026
@Nioolek Nioolek temporarily deployed to maintainer-approved May 1, 2026 10:27 — with GitHub Actions Inactive
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 1, 2026

Welcome to QwenPaw! 🐾

Hi @Nioolek, this is your 3rd Pull Request.

🙌 Join Developer Community

Thanks so much for your contribution! We'd love to invite you to join the official QwenPaw developer group! You can find the Discord and DingTalk group links under the "Developer Community" section on our docs page:
https://qwenpaw.agentscope.io/docs/community

We truly appreciate your enthusiasm—and look forward to your future contributions! 😊

We'll review your PR soon.

Copy link
Copy Markdown
Member

@xieyxclack xieyxclack left a comment

Choose a reason for hiding this comment

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

@Nioolek Thank you for your contribution! Please see inline comments

"""Verify Volcano Engine model definitions."""
model_ids = [m.id for m in VOLCENGINE_MODELS]
assert "Doubao-Seed-2.0-Code" in model_ids
assert len(VOLCENGINE_MODELS) == 1
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

will fail in the unit tests

name="Volcano Engine Coding Plan",
base_url="https://ark.cn-beijing.volces.com/api/coding/v3",
api_key_prefix="",
models=VOLCENGINE_MODELS,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

VOLCENGINE_CODINGPLAN_MODELS?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants