【架构优化】从 httpapi.filterIssues 中拆出负责人与关键词匹配谓词
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 3/5
- Temps estimé
- Une demi-journée
- Accessibilité débutants
- 25/100
Piste de recherche
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.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
现状问题
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 的既有失败除外）。
- Langage dominant
- Go
- Étoiles
- 141
- Forks
- 28
- Merge moyen
- 1 j 15 h
- PR mergées (30 j)
- 113
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Propose un modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de Tencent/LoopForge
-
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: 官方镜像未安装 wecom-cli，企业微信通知报「通知服务未安装 wecom-cli」Peut-être pris @newexploreor l’a pris il y a 1 jour. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Tencent/LoopForge#156 · 1 commentaire · 1 personne assignée ·
Les mainteneurs répondent en général sous 1 jour
-
【架构优化】合并 gitwork/workbranch.go 中 push/pull 重复的默认提交身份注入为 withFallbackIdentityPeut-être pris @fx-kurumi l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 40/100
Les mainteneurs répondent en général sous 1 jour
-
【架构优化】从 runtime.installNPMCLI 中拆出 npm 参数拼装 npmCLIInstallArgs 与安装结果启用 activateNPMStagePeut-être pris @fx-kurumi l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 30/100
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de Tencent/LoopForge
Issues similaires
-
team:library-maintainers
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
padok-team/burrito#1208 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
bazel-contrib/rules_go#4784 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
Уязвимые компоненты в GoOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
POSIdev-community/aictl#40 ·
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 62/100
prime-radiant-inc/evener#6306 ·
Les mainteneurs répondent en général sous 1 jour