【架构优化】拆分 runner.collectTaskOutputs 的输出文件读取、补丁目录推导与补丁采集

Aperta
  • Qualcosa non va?
#431 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

@fx-kurumi ci sta già lavorando.

Dal 13/11/2026.

  • #432 di @fx-kurumi — aperta

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
22/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
git, go
Ambito
devtools

Direzione di ricerca

Work is confined to internal/runner/outputs.go, where collectTaskOutputs mixes file reading, patch-directory derivation and patch collection. The payload names no entry point beyond that function. Done means three helpers are extracted with unchanged behaviour, the existing TestCollectTaskOutputsReportsOnlyRegularFilesInsideWorkspace still passes, and new tests cover the extracted pieces. However, a linked pull request (#432) is already open, so the work is likely claimed.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

现状问题

internal/runner/outputs.go 的 collectTaskOutputs 约 50 行，在一个函数里串行做了三件互不相关的事：

  1. 逐个读取 OutputFiles（路径安全校验、Lstat 只认普通文件、超过 maxArtifactUploadBytes 记入 TooLarge、读入 Files）；
  2. 推导需要采集补丁清单的仓库目录（无仓库时为工作区根 ""，否则为各仓库的 safeRunnerRepoDir(WorkspaceDir)），并逐个调用 gitwork.PatchInventory 填 Patches；
  3. 检测前端工作区事实 Facts。

其中「仓库目录推导」是纯逻辑却埋在 I/O 循环里，无法单独测试；现有测试也只覆盖了第 1 步。

重构方案

仅在 internal/runner/outputs.go 内拆函数，不改逻辑：

  • readOutputFiles(workspace, rels) (files map[string]string, tooLarge []string)：原第 1 段循环原样搬入；
  • patchInventoryDirs(inv) []string：原目录推导原样搬入；
  • collectPatchInventories(workspace, dirs) map[string]string：原补丁采集循环原样搬入；
  • collectTaskOutputs 只负责编排：恢复缺失输出 → 读文件 → 采补丁 → 采事实。

nil map / nil slice 的语义保持不变（没有文件时 Files/Patches 仍为 nil，JSON omitempty 输出不变）。

影响范围

仅 internal/runner/outputs.go（及新增测试文件），collectTaskOutputs 签名与三个调用方（daemon.go）不变，对外接口与行为不变。

验证方式

  • 新增测试覆盖 patchInventoryDirs（无仓库 / 多仓库）、collectPatchInventories（真实 git 仓库有改动 / 无改动 / 非仓库目录）与 readOutputFiles 的 nil 语义；
  • 原有 TestCollectTaskOutputsReportsOnlyRegularFilesInsideWorkspace 不改动且通过；
  • gofmt、go vet ./internal/runner/、go test ./internal/runner/。
Lingua principale
Go
Stelle
141
Fork
28
Merge medio
1g 15h
PR unite (30g)
113

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di Tencent/LoopForge

Tutte le issue di Tencent/LoopForge

Issue simili

Altre issue su Go

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.