【架构优化】合并 catalog/assistant handler 中重复的资产引用查找为包级 helper

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

  • #445 von @fx-kurumi — offen

Bewertung

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

Rechercherichtung

The three duplicated lookups are the agentNamesReferencing, sandboxNamesReferencing and workflowNamesReferencingSandbox methods on catalogHandlers in internal/httpapi/catalog.go and assistantHandlers in internal/httpapi/assistant_approval.go. Pull request #445 is already open against this issue, so check it before starting. Done means the package-level helpers in asset_references.go are used by both handlers, the new asset_references_test.go passes, and go test ./internal/httpapi/ is green.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

现状问题

internal/httpapi/catalog.go（catalogHandlers）与 internal/httpapi/assistant_approval.go（assistantHandlers）各自定义了一份逐字相同的资产引用查找方法：

  • agentNamesReferencing(kind, id)：列出引用某规则/技能的角色名
  • sandboxNamesReferencing(kind, id)：列出引用某 Dockerfile/环境脚本的沙箱名
  • workflowNamesReferencingSandbox(id)：列出引用某沙箱的流程名

两份实现共约 45 行重复，只依赖 *repo.Store 的只读查询，与各自 handler 的其它状态无关。目录删除校验（catalog）和助手删除审批预警（assistant）判断"是否仍被引用"的口径本应一致，重复实现容易在后续修改时只改一边而出现偏差。

重构方案

  • 新建包级 helper 文件 internal/httpapi/asset_references.go，提供三个无 receiver 的函数 agentNamesReferencing(st *repo.Store, kind, id)、sandboxNamesReferencing(st, kind, id)、workflowNamesReferencingSandbox(st, id)，实现与现有代码逐行一致（符合 AGENTS.md「纯共享的包级 helper 可以放独立 helper 文件」）。
  • 删除两个 handler 上的重复方法，调用处改为传 h.repo。
  • 现有测试中对旧方法的调用同步改为调用包级函数。

影响范围

仅 internal/httpapi 包内部未导出函数；不涉及 HTTP 接口、错误文案、返回值或排序，行为不变。

验证方式

  • 新增 asset_references_test.go 覆盖三个函数（重复引用只计一次、kind 区分、未引用返回空）。
  • gofmt、go vet ./internal/httpapi/、go test ./internal/httpapi/ 通过（本机缺 Chromium 的浏览器用例除外，与 main 一致）。
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.