feat(agentic): define AgenticModel component interface #511
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
mrh997
force-pushed
the
feat/responses_model
branch
from
0788f09 to
19d3bb9
Compare
mrh997
changed the title
mrh997
force-pushed
the
feat/responses_model
branch
3 times, most recently
from
d9473db to
ef740c2
Compare
mrh997
force-pushed
the
feat/responses_model
branch
7 times, most recently
from
555d774 to
39698ce
Compare
schema/agentic_message.go
Outdated
|
|
||
| type BlockReasoning struct { | ||
| // SummaryIndex is the index of the ReasoningSummary delta is associated with. | ||
| SummaryIndex *int |
There was a problem hiding this comment.
这个和 ContentBlock 中的 Index 和 State 重复了
There was a problem hiding this comment.
SummaryIndex 是给 Reasoning Block 中的 Summary array 用的，ContentBlock 是给 Block 用的
| SummaryIndex *int | ||
|
|
||
| // Summary is the reasoning content summary. | ||
| Summary []*ReasoningSummary |
There was a problem hiding this comment.
Summary 和 Content 的定位需要分清楚
There was a problem hiding this comment.
content 是模型 cot 过程，会包含 developer instruction 信息。summary 是可以直接透出给终端用户的思考内容。
https://cookbook.openai.com/articles/gpt-oss/handle-raw-cot
schema/agentic_message.go
Outdated
| } | ||
|
|
||
| type BaseGeneratedTextAnnotation interface { | ||
| AnnotationProvider() string |
There was a problem hiding this comment.
咱们没有暴露过 Provider/Vendor 概念。这个需要改一下
There was a problem hiding this comment.
Base 和 Generated 应该都不需要了
接口就表达了是 Base
已经处在 GenText Block 了，不用再冗余表达 Generated 了
There was a problem hiding this comment.
prefer name： TextAnnotation
schema/agentic_message.go
Outdated
| type BlockAssistantGenText struct { | ||
| Text string | ||
|
|
||
| Annotations []BaseGeneratedTextAnnotation |
There was a problem hiding this comment.
既然把 Annotations 抽象成接口了，没必要在这里限定成数组了。
更详细定义的 GetTextURLCitation 接口的返回，决定是数组，还是 map
mrh997
force-pushed
the
feat/responses_model
branch
2 times, most recently
from
886c0bd to
c8580de
Compare
schema/agentic_message.go
Outdated
| // Index is used for streaming to identify the chunk of the block for merging. | ||
| Index *int | ||
| // Streaming event type of this block. | ||
| EventType *StreamEventType |
There was a problem hiding this comment.
GPT 推荐：Stage *StreamStage
mrh997
force-pushed
the
feat/responses_model
branch
13 times, most recently
from
24eeda0 to
13259c3
Compare
mrh997
force-pushed
the
feat/responses_model
branch
from
13259c3 to
a01b5d5
Compare
What type of PR is this?
Check the PR title.
(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: