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

Aperta
  • Qualcosa non va?
#444 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.

  • #445 di @fx-kurumi — aperta

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
15/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Ferma
Stack tecnologico
go
Ambito
backend

Direzione di ricerca

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.

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

Descrizione

现状问题

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 一致）。
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.