【架构优化】从 httpapi.filterIssues 中拆出负责人与关键词匹配谓词
維護者通常 1 天內回覆
評估
研究方向
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.
由索引模型根據 Issue 內容生成。
描述
现状问题
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 的既有失败除外）。
- 主要語言
- Go
- 星號
- 141
- 分支
- 28
- 平均合併
- 1 天 15 小時
- 30 天內合併 PR
- 113
環境準備
- 提供 Dockerfile 或 Docker Compose 檔案
- 有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
Tencent/LoopForge 的其他 Issue
-
難度 2/5 1-3 小時 新手友好度 74/100
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 75/100
維護者通常 1 天內回覆
-
[Bug]: 官方镜像未安装 wecom-cli，企业微信通知报「通知服务未安装 wecom-cli」可能已有人在做 @newexploreor 於 1 天前認領。 未關閉bug
難度 2/5 1-3 小時 新手友好度 76/100
Tencent/LoopForge#156 · 1 則留言 · 已指派 1 人 ·
維護者通常 1 天內回覆
-
【架构优化】合并 gitwork/workbranch.go 中 push/pull 重复的默认提交身份注入为 withFallbackIdentity可能已有人在做 @fx-kurumi 今天認領。 未關閉
難度 2/5 1-3 小時 新手友好度 40/100
維護者通常 1 天內回覆
-
【架构优化】从 runtime.installNPMCLI 中拆出 npm 参数拼装 npmCLIInstallArgs 与安装结果启用 activateNPMStage可能已有人在做 @fx-kurumi 今天認領。 未關閉
難度 2/5 1-3 小時 新手友好度 30/100
維護者通常 1 天內回覆
查看 Tencent/LoopForge 的全部 Issue
相似的 Issue
-
team:library-maintainers
難度 2/5 1-3 小時 新手友好度 82/100
padok-team/burrito#1208 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 62/100
bazel-contrib/rules_go#4784 · 1 則留言 ·
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 75/100
-
難度 2/5 1-3 小時 新手友好度 62/100
POSIdev-community/aictl#40 ·
-
難度 1/5 1 小時以內 新手友好度 62/100
prime-radiant-inc/evener#6306 ·
維護者通常 1 天內回覆