【架构优化】从 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 天内回复