Skip to content

feat(agentic_ark): implement AgenticModel #587

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
mrh997 wants to merge 1 commit into
base: release/agentic_model
Choose a base branch
from
Open

feat(agentic_ark): implement AgenticModel #587

mrh997 wants to merge 1 commit into from

Conversation

@mrh997
Copy link
Contributor

@mrh997 mrh997 commented Dec 2, 2025

What type of PR is this?

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):

(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

@mrh997 mrh997 force-pushed the feat/mrh/ark_agentic branch 4 times, most recently from 1e414ce to 5d68538 Compare December 10, 2025 15:15
@mrh997 mrh997 force-pushed the feat/mrh/ark_agentic branch from 5d68538 to 23131b7 Compare January 5, 2026 14:39
@github-actions
Copy link

github-actions bot commented Jan 5, 2026

Need to create a new tag

The following modules have changes and may need version updates:

  • devops (Current: devops/v0.1.8)

  • components/model/arkbot (Current: components/model/arkbot/v0.1.2)

  • components/model/ollama (Current: components/model/ollama/v0.1.8)

  • components/model/gemini (Current: components/model/gemini/v0.1.23)

  • components/model/qwen (Current: components/model/qwen/v0.1.4)

  • components/model/ark (Current: components/model/ark/v0.1.61)

  • components/model/claude (Current: components/model/claude/v0.1.13)

  • components/model/openrouter (Current: components/model/openrouter/v0.1.2)

  • components/model/openai (Current: components/model/openai/v0.1.7)

  • components/model/deepseek (Current: components/model/deepseek/v0.1.2)

  • components/model/qianfan (Current: components/model/qianfan/v0.1.4)

  • components/tool/mcp (Current: components/tool/mcp/v0.0.8)

  • callbacks/cozeloop (Current: callbacks/cozeloop/v0.1.7)

  • libs/acl/openai (Current: libs/acl/openai/v0.1.12)

⚠️ Please create and push new version tags for these modules after merging this PR.

@mrh997 mrh997 force-pushed the release/agentic_model branch from 30377f9 to 388b5a2 Compare January 5, 2026 14:42
@mrh997 mrh997 force-pushed the feat/mrh/ark_agentic branch 10 times, most recently from b039e40 to 682d71b Compare January 6, 2026 14:06
@mrh997 mrh997 force-pushed the release/agentic_model branch from 388b5a2 to 4f236f1 Compare January 6, 2026 14:13
@mrh997 mrh997 force-pushed the feat/mrh/ark_agentic branch 6 times, most recently from 5b04437 to ed36a66 Compare January 6, 2026 15:17
meguminnnnnnnnn
meguminnnnnnnnn reviewed Jan 8, 2026
View reviewed changes
components/agentic/ark/examples/generate/main.go

log.Printf("request_id: %s\n", meta.ID)
respBody, _ := sonic.MarshalIndent(msg, " ", " ")
log.Printf(" body: %s\n", string(respBody))
Copy link
Contributor

@meguminnnnnnnnn meguminnnnnnnnn Jan 8, 2026

Choose a reason for hiding this comment

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

这个第二轮报错

2026/01/08 16:10:31 failed to generate, err: failed to create responses, err: Error code: 400 - {"code":"InvalidParameter","message":"The parameter `input[1].reasoning` specified in the request are not valid: Item reasoning is not supported for model: doubao-seed-1-6, version: 251015, please use another latest model. Request id: 0217678598314208d5b781e89b1033d1b98e0082083129b1df454","param":"input[1].reasoning","type":"BadRequest","request_id":"202601081610300000F94586E1DB651334"}

tool call 或者普通文本生成都是这个错

Copy link
Contributor Author

@mrh997 mrh997 Jan 8, 2026

Choose a reason for hiding this comment

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

和模型有关吗

meguminnnnnnnnn
meguminnnnnnnnn reviewed Jan 8, 2026
View reviewed changes
components/agentic/ark/examples/generate/main.go
meta := msg.ResponseMeta.Extension.(*ark.ResponseMetaExtension)

log.Printf("request_id: %s\n", meta.ID)
respBody, _ := sonic.MarshalIndent(msg, " ", " ")
Copy link
Contributor

@meguminnnnnnnnn meguminnnnnnnnn Jan 8, 2026

Choose a reason for hiding this comment

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

msg 有 string 方法，不需要 json 序列化

Copy link
Contributor Author

@mrh997 mrh997 Jan 8, 2026

Choose a reason for hiding this comment

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

string 方法信息不全，厂商的特化字段全都没有

@mrh997 mrh997 force-pushed the feat/mrh/ark_agentic branch from ed36a66 to c013075 Compare January 8, 2026 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@meguminnnnnnnnn meguminnnnnnnnn meguminnnnnnnnn left review comments

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mrh997 @meguminnnnnnnnn