【架构优化】从 httpapi.filterIssues 中拆出负责人与关键词匹配谓词
Maintainer antworten meist innerhalb von 1 Tag
Bewertung
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- Ein halber Tag
- Anfängerfreundlichkeit
- 25/100
Rechercherichtung
Start in internal/httpapi/issues.go and read filterIssues, paying attention to the assignee and keyword branches inside its loop. The issue asks for two private predicates, issueMatchesAssignee and issueMatchesQuery, with the original matching behaviour kept intact. Run go test ./internal/httpapi/ to confirm the existing filter behaviour before changing anything. Note that pull request #439 is already open against this issue, so check that work before starting.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
现状问题
internal/httpapi/issues.go 的 filterIssues 在一个循环里同时处理项目、状态、来源、负责人、关键词五类过滤条件，其中：
- 负责人过滤用了
unassigned分支 +matched临时变量的两层嵌套 if/else； - 关键词过滤内联拼接检索文本（标题/ID/编号/TAPD ID/外部 ID），并在每条记录上重复计算
needle。
循环体被这两段嵌套逻辑撑长，难以单独阅读和测试。
重构方案
- 拆出
issueMatchesAssignee(item, assignee) bool：保持unassigned（大小写不敏感）、按用户 ID 数字匹配、按 AssigneeID 大小写不敏感匹配的原语义； - 拆出
issueMatchesQuery(item, needle) bool：保持原检索文本拼接与fuzzyIssueTitleMatch回退； filterIssues循环体改为调用上述两个谓词，needle在循环外计算一次；排序逻辑不动。
影响范围
仅 internal/httpapi/issues.go 中 filterIssues 及新增两个包内私有函数；对外接口、查询参数语义、排序规则均不变。
验证方式
- 新增
issue_filter_predicates_test.go覆盖负责人（空、unassigned、用户 ID、AssigneeID 大小写）、关键词（标题、编号、外部 ID、模糊匹配、不匹配）以及filterIssues组合过滤结果； gofmt、go vet ./internal/httpapi/、go test ./internal/httpapi/通过（本机缺 Chromium 的既有失败除外）。
- Vorherrschende Sprache
- Go
- Sterne
- 141
- Forks
- 28
- Ø Merge
- 1 T. 15 Std.
- Gemergte PRs (30 T.)
- 113
Entwicklungsumgebung
- Enthält ein Dockerfile oder eine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Tencent/LoopForge
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Maintainer antworten meist innerhalb von 1 Tag
-
[Bug]: 官方镜像未安装 wecom-cli，企业微信通知报「通知服务未安装 wecom-cli」Evtl. vergeben @newexploreor hat das vor 1 Tag übernommen. Offenbug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 76/100
Tencent/LoopForge#156 · 1 Kommentar · 1 zugewiesene Person ·
Maintainer antworten meist innerhalb von 1 Tag
-
【架构优化】合并 gitwork/workbranch.go 中 push/pull 重复的默认提交身份注入为 withFallbackIdentityEvtl. vergeben @fx-kurumi hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 40/100
Maintainer antworten meist innerhalb von 1 Tag
-
【架构优化】从 runtime.installNPMCLI 中拆出 npm 参数拼装 npmCLIInstallArgs 与安装结果启用 activateNPMStageEvtl. vergeben @fx-kurumi hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 30/100
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in Tencent/LoopForge
Ähnliche Issues
-
team:library-maintainers
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
padok-team/burrito#1208 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
bazel-contrib/rules_go#4784 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Уязвимые компоненты в GoOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
POSIdev-community/aictl#40 ·
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 62/100
prime-radiant-inc/evener#6306 ·
Maintainer antworten meist innerhalb von 1 Tag