Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

fix(validate): plain-text output has no leading pass/fail summary line

Cerrado
  • ¿Algo no está bien?
#116 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
3/5
Tiempo estimado
1-2 días
Aptitud para principiantes
68/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
rust
Área
cli

Línea de trabajo

Comienza en la ruta de salida de texto predeterminada del comando syscribe validate y ejecuta casos de validación limpios, con solo advertencias y con bloqueo. Añade el resumen inicial sin cambiar --json ni el comportamiento de los códigos de salida; se considera terminado cuando el resumen siempre aparece primero y refleja el estado efectivo de aprobado/fallido.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

enhancement

Summary

syscribe validate's default (non---json) text output has no leading pass/fail summary line. A clean run prints nothing; a warnings-only run jumps straight into Warnings (N): with no corresponding Errors (0) header ever printed. Answering "did this pass" from the plain-text mode means either noticing the absence of an Errors section (a fragile negative inference) or switching to --json and inspecting severities programmatically.

Motivation

Running syscribe validate after a batch of spec edits, the actual output was:

Warnings (51):

| Code | File | Message |
...

with no errors section at all — meaning it passed (0 errors), but the only way to confirm that from the text output was grep -c "^| E" for an empty result, since there's no explicit "0 errors" anywhere to read. --json solves this cleanly (structured severities), but the plain-text/human-facing mode — which is also what an LLM agent reads by default unless it specifically remembers to pass --json — leaves "did it pass" as an inference rather than a stated fact.

Proposed fix

Print a one-line summary banner at the very top of the default text output, always, regardless of whether there are findings:

0 errors, 51 warnings

or, when gated:

0 errors, 51 warnings (3 gated by --deny W308) — FAIL

Doesn't change --json's shape (already fine) or the exit-code contract — purely a legibility addition to the human/agent-default text path so "did this pass" is answerable by reading the first line, not by the absence of a section.

Acceptance criteria

  • A 0-error, 0-warning run prints an explicit 0 errors, 0 warnings line (not silence)
  • A warnings-only run's summary line appears before the Warnings (N): table, not after
  • The summary reflects gating flags (--deny/--max-warnings/--warnings-as-errors/--profile) when they change the effective pass/fail verdict
  • --json output is unchanged
Lenguaje dominante
JavaScript
Estrellas
8
Forks
1
Merge medio
55 min
PR fusionados (30 d)
65

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

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de sjames/syscribe

Todos los issues de sjames/syscribe

Issues similares

Más issues de JavaScript

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.