CSV fields are interpreted as console markup, changing data or aborting rendering
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 72/100
Línea de trabajo
Empieza en render_csv() en src/rich_cli/main.py, donde las cadenas sin procesar se pasan a Table.add_column() y Table.add_row(). Ejecuta los dos comandos printf del issue para ver cómo se consume el markup y el código de salida 255. Está terminado cuando los valores CSV entre corchetes, como [red]literal[/red] y [/bold], se muestran como texto literal y el comando sale con 0.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
CSV cells containing bracketed text are passed through Rich markup parsing. Valid CSV data can therefore disappear or abort the command rather than render literally.
Environment
- rich-cli distribution 1.8.1, default-branch commit
46f4d2469097be395558768714a5f07ebccf1412 - Rich 12.6.0, Click 8.5.0
- Linux x86_64, Python 3.10.12
Reproduction
printf 'name,value\n[red]literal[/red],2\n' | rich --csv -
printf 'name,value\n[/bold],2\n' | rich --csv -
echo "exit=$?"
I also reproduced the first case from a .csv file and inspected its SVG export.
Actual behavior
The first table shows only literal, in red. The actual CSV value [red]literal[/red] is changed, even though this is CSV mode, not --print mode.
The second command fails with:
failed to print resource: closing tag '[/bold]' at position 0 doesn't match any
open tag
exit=255
A standalone field [not-a-style] also renders as an empty cell.
Expected behavior
CSV header and data values should render as literal text. Brackets and strings resembling markup should not remove data or cause a valid CSV to fail.
Cause
render_csv() in src/rich_cli/__main__.py passes raw strings to Table.add_column() and Table.add_row(). Rich's table/console path interprets those strings as markup. Wrapping the values in Text or escaping them before handing them to the table would keep the cells literal.
These commands were run locally with the versions above. This is separate from empty-CSV handling in #143 and delimiter detection: both fixtures parse into ordinary two-column CSV rows.
- Lenguaje dominante
- Python
- Estrellas
- 3.8k
- Forks
- 125
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 Textualize/rich-cli
-
HTTP error responses are rendered as successful input with exit status 0Posiblemente ocupada @ac1982 la tomó hace 7 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Textualize/rich-cli#148 ·
-
README is missing several documented CLI options (--head/--tail, --emoji, --rst, --inspect, --max-width, --panel-style, --title/--caption, --export-svg, --version)Posiblemente ocupada @philipgadsden3 la tomó hace 79 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Textualize/rich-cli#136 ·
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
Textualize/rich-cli#19 · 4 comentarios · 4 reacciones ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
Textualize/rich-cli#135 ·
-
RFE: allow pager animation to be disabledQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
Textualize/rich-cli#132 ·
Todos los issues de Textualize/rich-cli
Issues similares
-
[Resource]: glyphcss-diagramsAbiertoresource-submission validation-passed
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
hesreallyhim/awesome-claude-code#3214 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
room-commons replace accepts an empty old string and silently prepends the new text to the pageAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
doobidoo/mcp-memory-service#1650 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 82/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
ml-explore/mlx-lm#2002 ·
Los mantenedores suelen responder en 2 días