[Bug][linker] LinkPrToIssue never matches lowercase issue keys (e.g. in branch names)
Les mainteneurs répondent en général sous 2 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 78/100
Piste de recherche
Commence par backend/plugins/linker/tasks/link_pr_and_issue.go et suis la façon dont LinkPrToIssue extrait et compare les clés des champs de pull request avec issues.issue_key. Consulte les tests existants pour cette tâche et ajoute un cas où une clé de branche en minuscules est associée à une clé stockée en majuscules. Le résultat attendu est que le pipeline crée la ligne prévue dans pull_request_issues pour ce cas.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Search before asking
- I had searched in the issues and found no similar issues.
What happened
The linker plugin's LinkPrToIssue subtask compares the keys extracted from a PR (title, description or head ref) against issues.issue_key with an exact, case-sensitive match. normalizeIssueKey only strips # and whitespace.
Some issue trackers generate lowercase branch names from the issue key, e.g. user/proj-123-short-title for issue PROJ-123. Since #8777, head refs are searched, but even with a case-insensitive prToIssueRegexp such as (?i)\b(PROJ)-\d+\b, the extracted key is proj-123. It never equals the stored key PROJ-123, so the PR is silently left unlinked.
In my instance, about 5% of PRs that contain a key are affected, namely those where the key appears only in the branch name.
What do you expect to happen
Matching should be case-insensitive, or the extracted key should be normalized to match the stored issue_key, so that proj-123 links to PROJ-123.
How to reproduce
- Sync an issue tracker with an issue keyed
PROJ-123and a GitHub repo into one project. - Open a PR whose head ref is
user/proj-123-short-title, with no key in the title or description. - Enable the
linkerplugin withprToIssueRegexp=(?i)\b(PROJ)-\d+\b. - Run the project pipeline.
pull_request_issueshas no row for this PR.
Anything else
Suggested fix in backend/plugins/linker/tasks/link_pr_and_issue.go: compare case-insensitively, e.g. uppercase the extracted keys and match with UPPER(issues.issue_key) IN ?. Uppercasing in normalizeIssueKey alone would only work for trackers with uppercase keys.
A related limitation is that the enricher stops at the first field (title, then description, then head ref) with any regex match. A PR with an unrelated match in the title, such as CVE-2024-0000, never has its description or branch checked, so a real key there is ignored. I can open that as a separate issue if preferred.
Version
v1.0.3-beta18
Are you willing to submit PR?
- Yes I am willing to submit a PR!
Code of Conduct
- I agree to follow this project's Code of Conduct
- Langage dominant
- Go
- Étoiles
- 3.2k
- Forks
- 830
- Merge moyen
- 2 j 6 h
- PR mergées (30 j)
- 62
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- Aucun Dockerfile ni fichier Docker Compose
- Propose un modèle de pull request
- Aucun 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 apache/devlake
-
Difficulté 2/5 1-3 heures Accessibilité débutants 60/100
Les mainteneurs répondent en général sous 2 jours
-
[Bug][claude_code] Incremental collection stops after the first runPeut-être pris @clementsen l’a pris aujourd’hui. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 32/100
Les mainteneurs répondent en général sous 2 jours
-
type/bug
Difficulté 3/5 Une demi-journée Accessibilité débutants 73/100
Les mainteneurs répondent en général sous 2 jours
-
[Refactor][Build] Build the core and all plugins into a single binary by defaultPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertecomponent/plugins type/refactor
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
apache/devlake#9276 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
[Feature][Plugin] Add a GLPI data source pluginPeut-être pris @rabelenda l’a pris il y a 2 jours. Ouvertetype/feature-request
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 38/100
apache/devlake#9273 · 1 réaction ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de apache/devlake
Issues similaires
-
automation doc-healer documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
Les mainteneurs répondent en général sous 1 jour
-
Anthropic /v1/messages: only the last thinking block of an assistant message is keptPeut-être pris @Shayan12456 l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
ollama/ollama#19415 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
area:recipes event:contribfest format:human-first source:agent status:needs-triage type:recipe
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
cadence-workflow/cadence-ai-samples#140 ·
Les mainteneurs répondent en général sous 1 jour
-
customer issue
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
dolthub/doltgresql#3716 ·
Les mainteneurs répondent en général sous 1 jour
-
ux
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
evcc-io/evcc#35605 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour