【架构优化】从 githubCombinedChecks 中拆出 commit status/check runs 读取与整体状态汇总 githubChecksOverall

Offen
  • Stimmt etwas nicht?
#483 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.

  • #484 von @fx-kurumi — offen

Bewertung

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

Rechercherichtung

The work is confined to githubCombinedChecks in internal/httpapi/agent_tool_github.go, which should be split into githubCommitStatuses, githubCheckRuns and githubChecksOverall without changing URLs, warnings or outcome rules. Read that function and its caller githubPRStatus, then run go test ./internal/httpapi/ to get a baseline. Note that pull request #484 is already open against this issue, so check it before starting. Done means the existing tests pass and a new table-driven test covers githubChecksOverall.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

现状问题

internal/httpapi/agent_tool_github.go 中的 githubCombinedChecks 约 80 行，一个函数里依次做了三件事：

  1. 请求 commit status API、解析、追加结果和截断告警；
  2. 请求 check-runs API、解析（status/conclusion 取值）、追加结果和截断告警；
  3. 按 githubCheckOutcome 汇总 overall，并在有告警时把 none/success 降级为 unknown。

两段读取结构几乎一样，又和汇总逻辑写在同一个函数里，读起来要来回对照；汇总规则只能通过 mock HTTP 间接测试。

重构方案

在同文件内拆出三个包级函数，githubCombinedChecks 只负责按顺序调用并拼接结果：

  • githubCommitStatuses(ctx, client, token, path, sha) ([]githubCheckSummary, []string)：读取 commit status；
  • githubCheckRuns(ctx, client, token, path, sha) ([]githubCheckSummary, []string)：读取 check runs；
  • githubChecksOverall(checks, warnings) string：汇总整体状态。

请求 URL、告警文案、状态取值、汇总规则全部原样保留。

影响范围

仅 internal/httpapi/agent_tool_github.go；githubCombinedChecks 签名及唯一调用方 githubPRStatus 不变，对外行为不变。

验证方式

  • 新增 githubChecksOverall 的表驱动单元测试（none / success / pending / failure / 有告警降级 unknown / 失败不被告警覆盖）；
  • 现有 githubCombinedChecks 相关测试保持通过；
  • gofmt、go vet ./internal/httpapi/、go test ./internal/httpapi/。
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.