Keep quoted string contents intact in CI condition expressions
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- python
- Área
- ci-cd, testing-qa, tooling
Línea de trabajo
Comienza en tools/ci/common.py:39 y ejecuta la reproducción proporcionada de evaluate() para a!b. Rastrea cómo se tratan los literales entre comillas y los operadores reales, y luego añade pruebas de regresión positivas y negativas enfocadas para las grafías indicadas de operadores, funciones y contextos. Completarlo significa que las cadenas entre comillas idénticas se comparan como iguales mientras que los operadores booleanos, la búsqueda en el contexto y la evaluación en cortocircuito siguen funcionando.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
The plan evaluator rewrites operators and function spellings before parsing quoted strings. Equality against the identical values a!b, a&&b, a||b, and always() returns false. Context-looking text inside a string needs the same protection.
Starting point: tools/ci/common.py:39.
Validation
Audited canonical Wave 540728c2eb6cc12f5c69c3fb3fdb96e07ba059a4 (0.0.1-alpha-dev), whose tree matches the local checkout. Validation: Executed Python evaluate() reproductions. Executed compiler probes use the matching checkout-built compiler and std on Linux amd64 (Rust 1.89.0 / LLVM 21.1.8). Source/header or mocked checks are not native execution on another OS.
from tools.ci.common import evaluate
print(evaluate("inputs.value == 'a!b'", {"inputs.value": "a!b"}))
# False; the comparison should be True.
Scope
Make expression token handling respect quoted literals while preserving the existing small checked expression language; do not broaden it into unrestricted evaluation.
Completion criteria
- Identical quoted strings containing operators/function/context spellings compare equal.
- Actual boolean operators, context lookup and short-circuiting retain their behavior.
- Add focused positive/negative evaluator regressions.
This is a bounded, unassigned task with the starting point and focused checks above. Keep unrelated refactoring out of the change.
- Lenguaje dominante
- Rust
- Estrellas
- 56
- Forks
- 21
- Merge medio
- 7 h 55 min
- PR fusionados (30 d)
- 56
Preparar el entorno
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 wavefnd/Wave
-
Validate CI command vectors before indexing Cargo subcommandsPosiblemente ocupada @amandeavor la tomó hace 1 día. Abiertobug ci good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
build good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
Todos los issues de wavefnd/Wave
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
FavioVazquez/mollify#62 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 61/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
fendouai/awesome-deepseek-harness#125 ·
Los mantenedores suelen responder en 4 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
rust-embedded/heapless#694 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día