Skip to content

feat: 请求日志记录新增'推理强度'字段#1506

Merged
looplj merged 1 commit into
looplj:unstablefrom
deqiying:feature/unstable-request-reasoning-effort-column
Apr 27, 2026
Merged

feat: 请求日志记录新增'推理强度'字段#1506
looplj merged 1 commit into
looplj:unstablefrom
deqiying:feature/unstable-request-reasoning-effort-column

Conversation

@deqiying

Copy link
Copy Markdown
Contributor

背景

这是对 v0.9.x 分支网关功能的优化,用于在请求日志中记录并展示 reasoning effort,方便排查和分析推理模型请求的实际配置。

改动内容

  • 为 Request 记录新增可选字段 reasoningEffort / reasoning_effort
  • 在网关创建请求日志时保存 llmRequest.ReasoningEffort
  • 更新 Ent 与 GraphQL 生成代码,暴露请求日志中的推理强度字段及查询过滤能力。
  • 前端请求日志列表新增“推理强度”列。
  • 补充中英文 i18n 文案。

验证方式

  • 已执行 git diff --check,排除 .gitignore 后未发现空白符问题。
  • 未运行 lint/build/test,遵循仓库规则:除非明确要求,否则不主动执行。

影响与风险

  • 该改动会为请求日志增加一个 nullable 数据库字段,已有历史记录不受影响。
  • 未设置 reasoning effort 的请求会在前端显示为 -
  • 回退时移除 Request schema 字段、请求创建赋值、生成代码以及前端展示相关改动即可。

关联 Issue

Related to #1489

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a new reasoningEffort field to the Request entity across the stack. Changes include updating the database schema via Ent, exposing the field in the GraphQL API, and displaying it as a new column in the frontend request logs table with corresponding translations. I have no feedback to provide.

@looplj looplj merged commit 236ddc0 into looplj:unstable Apr 27, 2026
4 checks passed
gaoyuyun pushed a commit to gaoyuyun/axonhub that referenced this pull request May 10, 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.

2 participants