AI false-positive summary is not translatable and doubles punctuation
Los mantenedores suelen responder en 1 día
Ya se ha fusionado un pull request relacionado.
- #1505 de @ekamran — fusionado
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- Medio día
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- javascript, php, wordpress
- Área
- backend, frontend, internationalization
Línea de trabajo
Start with the message assembly in assets/js/plugin-check-admin.js, where the AI summary is built from hard-coded English and a . is appended unless the text ends in ., ! or ?. Then check includes/CLI/Plugin_Check_Command.php, where ' AI detected %1$d %2$s' is appended to "Checks complete. No errors found." and the count sits outside the _n() plural string. Done when each part is a translatable sentence with its own punctuation and plural forms follow locale rules. A pull request (#1505) is already open against this issue, so check it before starting.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
When AI analysis runs, the summary of possible false positives cannot be translated correctly, and it can end up with doubled punctuation.
Admin screen. assets/js/plugin-check-admin.js builds the AI part of the results message from hard-coded English ("AI detected … possible false positives", "Input tokens:", "Output tokens:", "Tokens spent:", "Model:"), so it stays in English on a translated site. It then adds . after the localized message unless that message ends in ., ! or ?. A Hindi message ending in । therefore gets a second stop: कोई त्रुटि नहीं मिली।. AI detected 5 possible false positives. …
WP-CLI. When a run with --ai has no errors or warnings left to show (for example with --ignore-errors), includes/CLI/Plugin_Check_Command.php appends . plus ' AI detected %1$d %2$s' to "Checks complete. No errors found.", so the line reads:
Success: Checks complete. No errors found.. AI detected 3 possible false positives (Tokens spent: 1,234)
The number also sits outside the plural string (' AI detected %1$d %2$s' with a separate _n( 'possible false positive', 'possible false positives', … )), which cannot be translated correctly in languages where the whole phrase changes with the count.
The CLI string was flagged by polyglots lint (https://github.com/emreerkan/polyglots) as plural-without-count.
Expected
Each part of the summary is a complete translatable sentence with its own punctuation, plural forms follow the locale's plural rules, and the parts are joined with a space.
AI Assisted: Claude Code helped find and reproduce this.
- Lenguaje dominante
- PHP
- Estrellas
- 366
- Forks
- 128
- Merge medio
- 1 d 9 h
- PR fusionados (30 d)
- 22
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 WordPress/plugin-check
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
WordPress/plugin-check#1509 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
False positive: native auto_update_plugins option triggers update_modification_detectedPosiblemente ocupada @KhushalSainS la tomó hace 3 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
WordPress/plugin-check#1506 ·
Los mantenedores suelen responder en 1 día
-
Accept PolyFillsPosiblemente ocupada @sanketio la tomó hace 10 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
WordPress/plugin-check#1496 ·
Los mantenedores suelen responder en 1 día
-
Question about a PCP WarningAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
WordPress/plugin-check#1475 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 55/100
WordPress/plugin-check#1474 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de WordPress/plugin-check
Issues similares
-
sync-en
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
coollabsio/coolify#12299 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Automattic/blocks-engine#3369 ·
Los mantenedores suelen responder en 1 día
-
Protect against "unlikely RCE"Abierto0. to triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 67/100
roundcube/roundcubemail-docker#478 ·
Los mantenedores suelen responder en 1 día
-
0. Needs triage 35-feedback bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día