【架构优化】合并 gitwork/workbranch.go 中 push/pull 重复的默认提交身份注入为 withFallbackIdentity

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

  • #489 von @fx-kurumi — offen

Bewertung

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

Rechercherichtung

Both copies live in internal/gitwork/workbranch.go: one in pushWorkBranch before the commit args are built, one in pullWorkBranch before the rebase args. Add package-level withFallbackIdentity(ctx, workspace, args) beside them and swap both call sites without changing argument order. Add a test file using a real temp repository for the configured and unconfigured cases, then run gofmt, go vet ./internal/gitwork/ and go test ./internal/gitwork/. PR #489 already touches this file, so check it before starting.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

现状问题

internal/gitwork/workbranch.go 里有两处一模一样的“仓库没配 user.email 时补默认提交身份”的代码：

  • pushWorkBranch：拼 commit 参数前
  • pullWorkBranch：拼 rebase 参数前
if email, _ := C(ctx, workspace, "config", "user.email"); strings.TrimSpace(email) == "" {
	args = append([]string{"-c", "user.name=LoopForge", "-c", "user.email=loopforge@local"}, args...)
}

两份代码都写死了 LoopForge / loopforge@local，以后改了一处容易漏掉另一处。

重构方案

在 workbranch.go 里新增包内函数 withFallbackIdentity(ctx, workspace, args) []string：仓库没配 user.email 时，在 args 前加上 -c user.name=LoopForge -c user.email=loopforge@local，配了就原样返回。两处调用改用这个函数，其余逻辑不动。

影响范围

  • 只改 internal/gitwork/workbranch.go 一个文件，另加一个测试文件；
  • 不改导出函数，不改 git 命令参数和执行顺序，行为不变；
  • 和 PR #452（同文件、verifyOffWorkBranch）改的不是同一段代码。

验证方式

  • 新增单元测试：用真实临时仓库验证“没配身份时加前缀”和“已配身份时原样返回”两种情况；
  • gofmt、go vet ./internal/gitwork/、go test ./internal/gitwork/ 全部通过。
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.