【架构优化】从 validateInsightExecution 中拆出单条用量 Token/成本校验 validateInsightUsageAmounts
維護者通常 1 天內回覆
還沒有人認領這個 Issue。
評估
研究方向
工作在 internal/httpapi/insights.go 中的 validateInsightExecution 內進行，這裡的 per-usage 迴圈混合了 provider/model 檢查、去重以及 Token/成本檢查。先閱讀該迴圈，然後把 Token 和成本檢查移入套件層級函式 validateInsightUsageAmounts(u domain.InsightUsage) string，保持訊息和檢查順序不變。完成條件是：TestInsightsValidationAndHTTP 仍然通過，一個新的表驅動測試涵蓋有效邊界值和每個錯誤分支，並且 gofmt、go vet ./internal/httpapi/ 和 go test ./internal/httpapi/ 都乾淨。
由索引模型根據 Issue 內容生成。
描述
现状问题
internal/httpapi/insights.go 的 validateInsightExecution（圈复杂度 43）同时承担两层校验：执行汇总本身（类型、ID、状态、时间、时长、URL）和 usage 数组里每一项模型用量。单项用量的 Token 范围、缓存/推理包含关系、成本状态与金额校验约 20 行嵌在 for 循环里，和 provider/model 必填、去重混在一起，阅读和单独测试都不方便。
重构方案
- 把循环体中 Token 与成本的校验抽成包级函数
validateInsightUsageAmounts(u domain.InsightUsage) string，错误文案与判断顺序保持原样。 validateInsightExecution的循环只保留：provider/model 必填 → 同一 provider/model 去重 → 调用validateInsightUsageAmounts，三者顺序与原实现完全一致，因此同时命中多条规则时返回的错误也不变。
影响范围
仅 internal/httpapi/insights.go 一个函数内部拆分 + 新增测试文件；/api/insights/v1/ingest 接口行为、错误文案、返回码均不变，无对外接口变化。
验证方式
- 原有
TestInsightsValidationAndHTTP全部通过； - 新增
validateInsightUsageAmounts表驱动测试，覆盖合法输入（含边界值）与每一条错误分支； gofmt、go vet ./internal/httpapi/、go test ./internal/httpapi/。
- 主要語言
- Go
- 星號
- 141
- 分支
- 28
- 平均合併
- 1 天 15 小時
- 30 天內合併 PR
- 113
環境準備
- 提供 Dockerfile 或 Docker Compose 檔案
- 有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
Tencent/LoopForge 的其他 Issue
-
難度 2/5 1-3 小時 新手友好度 75/100
維護者通常 1 天內回覆
-
[Bug]: 官方镜像未安装 wecom-cli，企业微信通知报「通知服务未安装 wecom-cli」可能已有人在做 @newexploreor 於 1 天前認領。 未關閉bug
難度 2/5 1-3 小時 新手友好度 76/100
Tencent/LoopForge#156 · 1 則留言 · 已指派 1 人 ·
維護者通常 1 天內回覆
-
【架构优化】合并 gitwork/workbranch.go 中 push/pull 重复的默认提交身份注入为 withFallbackIdentity可能已有人在做 @fx-kurumi 今天認領。 未關閉
難度 2/5 1-3 小時 新手友好度 40/100
維護者通常 1 天內回覆
-
【架构优化】从 runtime.installNPMCLI 中拆出 npm 参数拼装 npmCLIInstallArgs 与安装结果启用 activateNPMStage可能已有人在做 @fx-kurumi 今天認領。 未關閉
難度 2/5 1-3 小時 新手友好度 30/100
維護者通常 1 天內回覆
-
【架构优化】从 githubCombinedChecks 中拆出 commit status/check runs 读取与整体状态汇总 githubChecksOverall可能已有人在做 @fx-kurumi 今天認領。 未關閉
難度 2/5 1-3 小時 新手友好度 18/100
維護者通常 1 天內回覆
查看 Tencent/LoopForge 的全部 Issue
相似的 Issue
-
難度 2/5 1-3 小時 新手友好度 62/100
bazel-contrib/rules_go#4784 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 62/100
POSIdev-community/aictl#40 ·
-
難度 2/5 1-3 小時 新手友好度 62/100
nebari-dev/nebari-infrastructure-core#774 ·
維護者通常 1 天內回覆
-
bug
難度 2/5 1-3 小時 新手友好度 72/100
oracle/terraform-provider-oci#2662 ·
-
bug needs-triage
難度 2/5 1-3 小時 新手友好度 62/100
valkey-io/valkey-operator#644 ·
維護者通常 3 天內回覆