ag-ui-validate 0.1.0 → 0.2.0

package/src/cli-args.ts

@@ -20,6 +20,8 @@ export interface CliConfig {
   sarifFile?: string
   junitFile?: string
   jsonFile?: string
+  /** Pretty output only: one line per rule with a count, not one per finding. */
+  group: boolean
   help: boolean
   version: boolean
 }
@@ -40,6 +42,7 @@ Output (default: human-readable):
   --json-file <path>              additionally write the JSON report to a file
   --sarif-file <path>             additionally write a SARIF log to a file
   --junit-file <path>             additionally write JUnit XML to a file
+  --group                         one line per rule with a count (large streams)
   --no-color                      disable ANSI colors
 
 Rules:
@@ -67,6 +70,7 @@ const FLAGS: Record<string, Flag> = {
   "--help": { takesValue: false, apply: (c) => ((c.help = true), null) },
   "--version": { takesValue: false, apply: (c) => ((c.version = true), null) },
   "--no-color": { takesValue: false, apply: (c) => ((c.color = false), null) },
+  "--group": { takesValue: false, apply: (c) => ((c.group = true), null) },
   "--json": { takesValue: false, apply: (c) => setFormat(c, "json") },
   "--sarif": { takesValue: false, apply: (c) => setFormat(c, "sarif") },
   "--junit": { takesValue: false, apply: (c) => setFormat(c, "junit") },
@@ -150,6 +154,7 @@ export function parseCliArgs(argv: string[]): ParseResult {
     color: null,
     headers: {},
     severityOverrides: {},
+    group: false,
     help: false,
     version: false,
   }
@@ -182,6 +187,12 @@ export function parseCliArgs(argv: string[]): ParseResult {
     if (error !== null) return { ok: false, error }
   }
 
+  if (config.group && config.format !== "pretty") {
+    return {
+      ok: false,
+      error: "--group only applies to the default pretty output, not --json/--sarif/--junit",
+    }
+  }
   if (targets.length > 1) {
     return { ok: false, error: `expected exactly one target, got ${targets.length}` }
   }

package/src/cli.ts

@@ -7,7 +7,7 @@ import { createReadStream, readFileSync, writeFileSync } from "node:fs"
 import process from "node:process"
 import { decideExitCode, parseCliArgs, USAGE } from "./cli-args.js"
 import type { CliConfig } from "./cli-args.js"
-import { formatDiagnosticLine, formatReportSummary, toJsonReport, toJUnit, toSarif } from "./report/index.js"
+import { formatDiagnosticLine, formatGroupedDiagnostics, formatReportSummary, toJsonReport, toJUnit, toSarif } from "./report/index.js"
 import { TransportError, validateBody, validateEndpoint } from "./transport/index.js"
 import type { TransportOptions, TransportResult } from "./transport/index.js"
 import type { Diagnostic, ValidatorOptions } from "./types.js"
@@ -62,8 +62,9 @@ async function main(): Promise<number> {
 
   // Pretty mode streams findings as they are detected; machine formats emit
   // one document at the end, so nothing else may touch stdout before it.
+  // --group defers everything to one collapsed block after the run.
   let printed = 0
-  const onDiagnostic = pretty
+  const onDiagnostic = pretty && !config.group
     ? (d: Diagnostic): void => {
         printed += 1
         process.stdout.write(`${formatDiagnosticLine(d, { color })}\n`)
@@ -116,7 +117,11 @@ async function main(): Promise<number> {
   }
 
   if (pretty) {
-    if (printed > 0) process.stdout.write("\n")
+    if (config.group && report.diagnostics.length > 0) {
+      process.stdout.write(`${formatGroupedDiagnostics(report.diagnostics, { color })}\n\n`)
+    } else if (printed > 0) {
+      process.stdout.write("\n")
+    }
     process.stdout.write(`${formatReportSummary(report, { color })}\n`)
   } else if (config.format === "json") {
     const doc = toJsonReport(report, { tool: { name: TOOL_NAME, version }, target: targetLabel })

package/src/report/index.ts

@@ -1,6 +1,6 @@
 // ag-ui-validate/report: pure reporters over a core Report. No I/O — callers
 // decide where the strings go.
-export { formatDiagnosticLine, formatReportSummary } from "./pretty.js"
+export { formatDiagnosticLine, formatGroupedDiagnostics, formatReportSummary } from "./pretty.js"
 export type { PrettyOptions } from "./pretty.js"
 export { toJsonReport } from "./json.js"
 export type { JsonReportDocument, JsonReportOptions } from "./json.js"

package/src/report/pretty.ts

@@ -1,5 +1,6 @@
 // Human-readable output. Pure string formatting — the CLI decides where it
 // goes and whether a TTY wants color.
+import { RULES } from "../rules/catalog.js"
 import type { Diagnostic, Report, Severity } from "../types.js"
 
 export interface PrettyOptions {
@@ -25,6 +26,48 @@ function count(n: number, noun: string): string {
   return `${n} ${noun}${n === 1 ? "" : "s"}`
 }
 
+const SEVERITY_RANK: Record<Severity, number> = { error: 0, warning: 1, info: 2 }
+const SAMPLE_INDEXES = 3
+
+/**
+ * One line per rule instead of one per occurrence — for large streams where
+ * the same violation repeats. Totals stay honest in the summary; this only
+ * changes what is listed.
+ */
+export function formatGroupedDiagnostics(diagnostics: Diagnostic[], opts: PrettyOptions): string {
+  const groups = new Map<string, Diagnostic[]>()
+  for (const d of diagnostics) {
+    const list = groups.get(d.rule)
+    if (list === undefined) groups.set(d.rule, [d])
+    else list.push(d)
+  }
+
+  const sorted = [...groups.values()].sort((a, b) => {
+    const bySeverity = SEVERITY_RANK[a[0]!.severity] - SEVERITY_RANK[b[0]!.severity]
+    return bySeverity !== 0 ? bySeverity : a[0]!.rule.localeCompare(b[0]!.rule)
+  })
+
+  const lines: string[] = []
+  for (const group of sorted) {
+    const first = group[0]!
+    const title = RULES.get(first.rule)?.title ?? first.message
+    const indexes = group.map((d) => d.eventIndex).filter((i) => i >= 0)
+    let where: string
+    if (indexes.length === 0) {
+      where = "stream-level"
+    } else {
+      const sample = indexes.slice(0, SAMPLE_INDEXES).join(", ")
+      const rest = indexes.length - SAMPLE_INDEXES
+      where = `events ${sample}${rest > 0 ? ` (+${rest} more)` : ""}`
+    }
+    const head = paint(SGR[first.severity], `${SYMBOL[first.severity]} ${first.rule}`, opts.color)
+    const meta = paint("2", `${first.severity.padEnd(7)} ×${group.length}`, opts.color)
+    const cite = paint("2", `  ↳ ${first.specUrl}`, opts.color)
+    lines.push(`${head}  ${meta}  ${title} — ${where}`, cite)
+  }
+  return lines.join("\n")
+}
+
 export function formatReportSummary(report: Report, opts: PrettyOptions): string {
   const { errors, warnings, info } = report.summary
   const lines: string[] = []