Improve provider attribution in request monitoring#90
Conversation
|
感谢这个 PR,provider attribution 这个方向我是认可的。多 provider / 多 prefix 场景下,监控页确实需要更清楚地展示请求来源。 不过结合 v1.2.1 已经引入的 api-key alias 设计,我不建议当前版本直接合并,需要调整后再合并。 v1.2.1 里 alias 的语义已经是: 所以这个 PR 里新增的 usage 另外,持久化方式也建议和 v1.2.1 的设计对齐。v1.2.1 是通过独立的 当前默认展示的信息也偏多了。按 v1.2.1 的设计,监控页主展示应该优先是用户设置的 api-key alias / account label,而不是 provider、baseUrl、usage alias 这些技术细节。建议展示层收敛为: 也就是说,provider prefix 可以展示,但不建议拼进账号主名称;baseUrl 和 usage alias 原文不建议默认铺在账号概览和实时请求列表里。 实现上也建议调整几个点:
gofmt -w usage-service/internal/...
go test ./...
go build ./...
npm run type-check
npm run lint
npm run build
npm test结论:功能方向可以接受,但当前实现和 v1.2.1 的 api-key alias 模型还没有完全对齐,尤其是命名、持久化方式和默认展示层级需要调整。建议先 Request changes,调整为“结构化持久化 + 轻量默认展示 + 详情中保留完整 provider attribution”后再合并。 |
Summary
Validation
npm run type-checknpm run lintnpm test -- src/pages/MonitoringCenterPage.test.tsx src/features/monitoring/hooks/useMonitoringData.test.ts src/features/monitoring/accountOverviewState.test.ts