【架构优化】合并 repo/analytics.go 中重复的用户/项目/需求来源累加器取用或新建逻辑

Offen
  • Stimmt etwas nicht?
#473 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

@fx-kurumi arbeitet bereits daran.

Seit 13.11.2026.

  • #474 von @fx-kurumi — offen

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
35/100
Issue-Typ
Refactoring
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
go
Bereich
backend

Rechercherichtung

The duplicated get-or-create logic lives in internal/repo/analytics.go, in accumulateUserUsage, accumulateProjectUsage, accumulateIssueSource and accumulateConversationRun. Read those four functions first to see the shared default values and key formats. The issue names the three helper functions to add, but pull request #474 is already open against this work, so check it before starting. Done means the helpers are used by both callers, the new analytics_accumulator_entry_test.go passes, and go test ./internal/repo/ passes.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

现状问题

internal/repo/analytics.go 中「按键取累加器、不存在就新建并写回 map」的代码在两处各写了一遍：

  • 责任人 × 自然日：accumulateUserUsage 与 accumulateConversationRun（含空责任人归为 unassigned、key 拼接）
  • 项目：accumulateProjectUsage 与 accumulateConversationRun（含空项目归为 unassigned）
  • 需求来源：accumulateIssueSource 与 accumulateConversationRun

accumulateConversationRun 因此长达约 90 行，且默认值、key 格式、初始字段（如 CostKnown: true、issues map）需要在两处同步维护，容易漂移。

重构方案

新增三个包内函数，两处调用方都改为使用它们：

  • analyticsUserEntry(values, userID, day)
  • analyticsProjectEntry(values, projectID)
  • analyticsSourceEntry(values, source)（空来源归为 local，与 accumulateIssueSource 原逻辑一致；analyticsIssueSource 本身不会返回空串，会话侧行为不变）

只抽取「取或建」部分，累加逻辑保持原样。

影响范围

仅 internal/repo/analytics.go 内部私有函数，不改对外接口、统计口径和返回结果。

验证方式

  • 新增 internal/repo/analytics_accumulator_entry_test.go，覆盖默认值、复用同一累加器、不同键新建
  • gofmt、go vet ./internal/repo/、go test ./internal/repo/（含现有 analytics 全部用例）
Vorherrschende Sprache
Go
Sterne
141
Forks
28
Ø Merge
1 T. 15 Std.
Gemergte PRs (30 T.)
113

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus Tencent/LoopForge

Alle Issues in Tencent/LoopForge

Ähnliche Issues

Weitere Issues zu Go

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.