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

Ouverte
  • Quelque chose ne va pas ?
#431 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

@fx-kurumi y travaille déjà.

Depuis le 13/11/2026.

  • #432 par @fx-kurumi — ouverte

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
22/100
Type d'issue
Refactorisation
Clarté
Clairement spécifiée
Activité
Active
Stack technique
git, go
Domaine
devtools

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

现状问题

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/。
Langage dominant
Go
Étoiles
141
Forks
28
Merge moyen
1 j 15 h
PR mergées (30 j)
113

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de Tencent/LoopForge

Toutes les issues de Tencent/LoopForge

Issues similaires

Plus d'issues Go

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.