fix(source-control): ready hold check treats hyphenated identifiers containing wip as a WIP marker
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 77/100
Línea de trabajo
Lee plugins/source-control/skills/pull-request/scripts/check-ready-holds.sh, especialmente la línea 77, y sigue cómo el matcher del título identifica los marcadores WIP. Añade el título reportado como una prueba stay-quiet que falle antes de la corrección y, después, restringe el matcher para que los identificadores con guiones no cuenten como marcadores WIP. El trabajo estará terminado cuando la comprobación de disponibilidad pase con el título de reproducción y siga detectando los marcadores WIP independientes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
/source-control:pull-request ready refuses to flip a draft whose title names a hyphenated identifier containing wip, reporting a WIP marker.
plugins/source-control/skills/pull-request/scripts/check-ready-holds.sh:77 tests the title with test("\\bwip\\b"; "i"). A hyphen is a word boundary, so check-wip-cap matches.
Reproduction
PR #8967, title refactor(pr-pipeline): fold the admission window into check-wip-cap with a queued-work count:
title: a WIP marker in "refactor(pr-pipeline): fold the admission window into check-wip-cap with a queued-work count"
exit=3
Expected
A WIP marker is a title token such as a leading WIP, [WIP] or WIP:, not wip inside a hyphenated identifier. The ready check should pass this title.
Fix
Narrow the matcher, for example to a leading WIP/[WIP]/(WIP) token or wip that has no -, _ or alphanumeric character on either side. Per the hook-precision convention, land it with a stay-quiet test (the title above) that fails before the fix.
- Lenguaje dominante
- Shell
- Estrellas
- 22
- Forks
- 2
- Merge medio
- 6 h 58 min
- PR fusionados (30 d)
- 951
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de melodic-software/claude-code-plugins
-
priority: medium status: needs-decision work-class: scoped
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
melodic-software/claude-code-plugins#9339 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
pipeline-canary priority: low status: ready work-class: mechanical
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
melodic-software/claude-code-plugins#9263 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
good first issue priority: low
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
melodic-software/claude-code-plugins#9251 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
area: security needs-triage priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
melodic-software/claude-code-plugins#9211 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
priority: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
melodic-software/claude-code-plugins#9202 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de melodic-software/claude-code-plugins
Issues similares
-
Security: Vulnerable PHP version (CVE-2026-109618 / CVE-2026-109619) in current Docker imagesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
MISP/misp-docker#478 ·
Los mantenedores suelen responder en 1 día
-
roadmap-100
Dificultad 2/5 1-3 horas Aptitud para principiantes 80/100
Los mantenedores suelen responder en 1 día
-
数据界面的横向滚动条缺失Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
zilliztech/attu#1037 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
community-scripts/ProxmoxVE#18073 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Uptime "Test Monitor" always fails with an empty HTTP 400 (uptime-checker API endpoint not configured)Posiblemente ocupada @ayoubjemai la tomó hoy. AbiertoWaiting for: Product Owner
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
getsentry/self-hosted#4582 ·
Los mantenedores suelen responder en 1 día