@herb-tools/linter 0.4.3 → 0.5.0

package/dist/types/src/cli/formatters/index.d.ts

@@ -1,3 +1,5 @@
 export { BaseFormatter } from "./base-formatter.js";
 export { SimpleFormatter } from "./simple-formatter.js";
 export { DetailedFormatter } from "./detailed-formatter.js";
+export { JSONFormatter, type JSONOutput } from "./json-formatter.js";
+export { GitHubActionsFormatter } from "./github-actions-formatter.js";

package/dist/types/src/cli/formatters/json-formatter.d.ts

@@ -0,0 +1,42 @@
+import { BaseFormatter } from "./base-formatter.js";
+import type { Diagnostic, SerializedDiagnostic } from "@herb-tools/core";
+import type { ProcessedFile } from "../file-processor.js";
+interface JSONOffense extends SerializedDiagnostic {
+    filename: string;
+}
+interface JSONSummary {
+    filesChecked: number;
+    filesWithOffenses: number;
+    totalErrors: number;
+    totalWarnings: number;
+    totalOffenses: number;
+    ruleCount: number;
+}
+interface JSONTiming {
+    startTime: string;
+    duration: number;
+}
+export interface JSONOutput {
+    offenses: JSONOffense[];
+    summary: JSONSummary | null;
+    timing: JSONTiming | null;
+    completed: boolean;
+    clean: boolean | null;
+    message: string | null;
+}
+interface JSONFormatOptions {
+    files: string[];
+    totalErrors: number;
+    totalWarnings: number;
+    filesWithOffenses: number;
+    ruleCount: number;
+    startTime: number;
+    startDate: Date;
+    showTiming: boolean;
+}
+export declare class JSONFormatter extends BaseFormatter {
+    format(allOffenses: ProcessedFile[]): Promise<void>;
+    formatWithSummary(allOffenses: ProcessedFile[], options: JSONFormatOptions): Promise<void>;
+    formatFile(_filename: string, _offenses: Diagnostic[]): void;
+}
+export {};

package/dist/types/src/cli/formatters/simple-formatter.d.ts

@@ -2,6 +2,6 @@ import { BaseFormatter } from "./base-formatter.js";
 import type { Diagnostic } from "@herb-tools/core";
 import type { ProcessedFile } from "../file-processor.js";
 export declare class SimpleFormatter extends BaseFormatter {
-    format(allDiagnostics: ProcessedFile[]): Promise<void>;
-    formatFile(filename: string, diagnostics: Diagnostic[]): void;
+    format(allOffenses: ProcessedFile[]): Promise<void>;
+    formatFile(filename: string, offenses: Diagnostic[]): void;
 }

package/dist/types/src/cli/output-manager.d.ts

@@ -0,0 +1,31 @@
+import type { ThemeInput } from "@herb-tools/highlighter";
+import type { FormatOption } from "./argument-parser.js";
+import type { ProcessingResult } from "./file-processor.js";
+interface OutputOptions {
+    formatOption: FormatOption;
+    theme: ThemeInput;
+    wrapLines: boolean;
+    truncateLines: boolean;
+    showTiming: boolean;
+    startTime: number;
+    startDate: Date;
+}
+interface LintResults extends ProcessingResult {
+    files: string[];
+}
+export declare class OutputManager {
+    private summaryReporter;
+    /**
+     * Output successful lint results
+     */
+    outputResults(results: LintResults, options: OutputOptions): Promise<void>;
+    /**
+     * Output informational message (like "no files found")
+     */
+    outputInfo(message: string, options: OutputOptions): void;
+    /**
+     * Output error message
+     */
+    outputError(message: string, options: OutputOptions): void;
+}
+export {};

package/dist/types/src/cli/summary-reporter.d.ts

@@ -2,12 +2,12 @@ export interface SummaryData {
     files: string[];
     totalErrors: number;
     totalWarnings: number;
-    filesWithViolations: number;
+    filesWithOffenses: number;
     ruleCount: number;
     startTime: number;
     startDate: Date;
     showTiming: boolean;
-    ruleViolations: Map<string, {
+    ruleOffenses: Map<string, {
         count: number;
         files: Set<string>;
     }>;
@@ -15,7 +15,7 @@ export interface SummaryData {
 export declare class SummaryReporter {
     private pluralize;
     displaySummary(data: SummaryData): void;
-    displayMostViolatedRules(ruleViolations: Map<string, {
+    displayMostViolatedRules(ruleOffenses: Map<string, {
         count: number;
         files: Set<string>;
     }>, limit?: number): void;

package/dist/types/src/cli.d.ts

@@ -1,6 +1,8 @@
 export declare class CLI {
     private argumentParser;
     private fileProcessor;
-    private summaryReporter;
+    private outputManager;
+    private exitWithError;
+    private exitWithInfo;
     run(): Promise<void>;
 }

package/dist/types/src/rules/erb-no-empty-tags.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class ERBNoEmptyTagsRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/erb-no-output-control-flow.d.ts

@@ -1,7 +1,7 @@
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
 export declare class ERBNoOutputControlFlowRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/erb-prefer-image-tag-helper.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class ERBPreferImageTagHelperRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/erb-require-whitespace-inside-tags.d.ts

@@ -1,7 +1,7 @@
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
 export declare class ERBRequireWhitespaceRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-anchor-require-href.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLAnchorRequireHrefRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-aria-attribute-must-be-valid.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLAriaAttributeMustBeValid extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-aria-level-must-be-valid.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLAriaLevelMustBeValidRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-aria-role-heading-requires-level.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLAriaRoleHeadingRequiresLevelRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-aria-role-must-be-valid.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLAriaRoleMustBeValidRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-attribute-double-quotes.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLAttributeDoubleQuotesRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-attribute-values-require-quotes.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLAttributeValuesRequireQuotesRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-boolean-attributes-no-value.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLBooleanAttributesNoValueRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-img-require-alt.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLImgRequireAltRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-no-block-inside-inline.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLNoBlockInsideInlineRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-no-duplicate-attributes.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLNoDuplicateAttributesRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-no-duplicate-ids.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 import type { LintOffense, LintContext } from "../types";
 export declare class HTMLNoDuplicateIdsRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-no-empty-headings.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLNoEmptyHeadingsRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-no-nested-links.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLNoNestedLinksRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/html-tag-name-lowercase.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class HTMLTagNameLowercaseRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/rules/parser-no-errors.d.ts

@@ -0,0 +1,8 @@
+import { ParserRule } from "../types.js";
+import type { LintOffense } from "../types.js";
+import type { ParseResult } from "@herb-tools/core";
+export declare class ParserNoErrorsRule extends ParserRule {
+    name: string;
+    check(result: ParseResult): LintOffense[];
+    private herbErrorToLintOffense;
+}

package/dist/types/src/rules/svg-tag-name-capitalization.d.ts

@@ -1,7 +1,7 @@
 import { ParserRule } from "../types.js";
 import type { LintOffense, LintContext } from "../types.js";
-import type { Node } from "@herb-tools/core";
+import type { ParseResult } from "@herb-tools/core";
 export declare class SVGTagNameCapitalizationRule extends ParserRule {
     name: string;
-    check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }

package/dist/types/src/types.d.ts

@@ -1,6 +1,6 @@
-import { Node, Diagnostic, LexResult } from "@herb-tools/core";
+import { Diagnostic, LexResult, ParseResult } from "@herb-tools/core";
 import type { defaultRules } from "./default-rules.js";
-export type LintSeverity = "error" | "warning";
+export type LintSeverity = "error" | "warning" | "info" | "hint";
 /**
  * Automatically inferred union type of all available linter rule names.
  * This type extracts the 'name' property from each rule class instance.
@@ -18,7 +18,7 @@ export interface LintResult {
 export declare abstract class ParserRule {
     static type: "parser";
     abstract name: string;
-    abstract check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    abstract check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }
 export declare abstract class LexerRule {
     static type: "lexer";

package/dist/types/types.d.ts

@@ -1,6 +1,6 @@
-import { Node, Diagnostic, LexResult } from "@herb-tools/core";
+import { Diagnostic, LexResult, ParseResult } from "@herb-tools/core";
 import type { defaultRules } from "./default-rules.js";
-export type LintSeverity = "error" | "warning";
+export type LintSeverity = "error" | "warning" | "info" | "hint";
 /**
  * Automatically inferred union type of all available linter rule names.
  * This type extracts the 'name' property from each rule class instance.
@@ -18,7 +18,7 @@ export interface LintResult {
 export declare abstract class ParserRule {
     static type: "parser";
     abstract name: string;
-    abstract check(node: Node, context?: Partial<LintContext>): LintOffense[];
+    abstract check(result: ParseResult, context?: Partial<LintContext>): LintOffense[];
 }
 export declare abstract class LexerRule {
     static type: "lexer";

package/docs/rules/README.md

@@ -23,6 +23,7 @@ This page contains documentation for all Herb Linter rules.
 - [`html-no-duplicate-ids`](./html-no-duplicate-ids.md) - Prevents duplicate IDs within a document
 - [`html-no-nested-links`](./html-no-nested-links.md) - Prevents nested anchor tags
 - [`html-tag-name-lowercase`](./html-tag-name-lowercase.md) - Enforces lowercase tag names in HTML
+- [`parser-no-errors`](./parser-no-errors.md) - Disallow parser errors in HTML+ERB documents
 - [`svg-tag-name-capitalization`](./svg-tag-name-capitalization.md) - Enforces proper camelCase capitalization for SVG elements
 
 ## Contributing

package/docs/rules/parser-no-errors.md

@@ -0,0 +1,84 @@
+# Linter Rule: Disallow parser errors in HTML+ERB documents
+
+**Rule:** `parser-no-errors`
+
+## Description
+
+Report parser errors as linting offenses. This rule surfaces syntax errors, malformed HTML, and other parsing issues that prevent the document from being correctly parsed.
+
+## Rationale
+
+Parser errors indicate fundamental structural problems in HTML+ERB documents that can lead to unexpected rendering behavior, accessibility issues, and maintenance difficulties. These errors should be fixed before addressing other linting concerns as they represent invalid markup that browsers may interpret inconsistently.
+
+By surfacing parser errors through the linter, developers can catch these critical issues when running lint checks directly, without needing to switch to the language server or other tools.
+
+## Examples
+
+### ✅ Good
+
+```html
+<h2>Welcome to our site</h2>
+<p>This is a paragraph with proper structure.</p>
+
+<div class="container">
+  <img src="image.jpg" alt="Description">
+</div>
+```
+
+```erb
+<h2><%= @page.title %></h2>
+<p><%= @page.description %></p>
+
+<% if user_signed_in? %>
+  <div class="user-section">
+    <%= current_user.name %>
+  </div>
+<% end %>
+```
+
+### 🚫 Bad
+
+```html
+<!-- Mismatched closing tag -->
+<h2>Welcome to our site</h3>
+
+<!-- Unclosed element -->
+<div>
+  <p>This paragraph is never closed
+</div>
+
+<!-- Missing opening tag -->
+Some content
+</div>
+```
+
+```erb
+<!-- Invalid Ruby syntax in ERB -->
+<%= 1 + %>
+
+<!-- Mismatched quotes -->
+<div class="container'>Content</div>
+
+<!-- Void element with closing tag -->
+<img src="image.jpg" alt="Description"></img>
+```
+
+## Error Types
+
+This rule reports various parser error types:
+
+- **`UNCLOSED_ELEMENT_ERROR`**: Elements that are opened but never closed
+- **`MISSING_CLOSING_TAG_ERROR`**: Opening tags without matching closing tags
+- **`MISSING_OPENING_TAG_ERROR`**: Closing tags without matching opening tags
+- **`TAG_NAMES_MISMATCH_ERROR`**: Opening and closing tags with different names
+- **`QUOTES_MISMATCH_ERROR`**: Mismatched quotation marks in attributes
+- **`VOID_ELEMENT_CLOSING_TAG_ERROR`**: Void elements (like `<img>`) with closing tags
+- **`RUBY_PARSE_ERROR`**: Invalid Ruby syntax within ERB tags
+- **`UNEXPECTED_TOKEN_ERROR`**: Unexpected tokens during parsing
+- **`UNEXPECTED_ERROR`**: Other unexpected parsing issues
+
+## References
+
+* [HTML Living Standard - Parsing](https://html.spec.whatwg.org/multipage/parsing.html)
+* [W3C HTML Validator](https://validator.w3.org/)
+* [ERB Template Guide](https://guides.rubyonrails.org/layouts_and_rendering.html)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@herb-tools/linter",
-  "version": "0.4.3",
+  "version": "0.5.0",
   "description": "HTML+ERB linter for validating HTML structure and enforcing best practices",
   "license": "MIT",
   "homepage": "https://herb-tools.dev",
@@ -33,9 +33,9 @@
     }
   },
   "dependencies": {
-    "@herb-tools/core": "0.4.3",
-    "@herb-tools/highlighter": "0.4.3",
-    "@herb-tools/node-wasm": "0.4.3",
+    "@herb-tools/core": "0.5.0",
+    "@herb-tools/highlighter": "0.5.0",
+    "@herb-tools/node-wasm": "0.5.0",
     "glob": "^11.0.3"
   },
   "files": [

package/src/cli/argument-parser.ts

@@ -11,9 +11,11 @@ import type { ThemeInput } from "@herb-tools/highlighter"
 
 import { name, version } from "../../package.json"
 
+export type FormatOption = "simple" | "detailed" | "json" | "github"
+
 export interface ParsedArguments {
   pattern: string
-  formatOption: 'simple' | 'detailed'
+  formatOption: FormatOption
   showTiming: boolean
   theme: ThemeInput
   wrapLines: boolean
@@ -32,9 +34,11 @@ export class ArgumentParser {
     Options:
       -h, --help       show help
       -v, --version    show version
-      --format         output format (simple|detailed) [default: detailed]
+      --format         output format (simple|detailed|json|github) [default: detailed]
       --simple         use simple output format (shortcut for --format simple)
-      --theme          syntax highlighting theme (${THEME_NAMES.join('|')}) or path to custom theme file [default: ${DEFAULT_THEME}]
+      --json           use JSON output format (shortcut for --format json)
+      --github         use GitHub Actions output format (shortcut for --format github)
+      --theme          syntax highlighting theme (${THEME_NAMES.join("|")}) or path to custom theme file [default: ${DEFAULT_THEME}]
       --no-color       disable colored output
       --no-timing      hide timing information
       --no-wrap-lines  disable line wrapping
@@ -45,15 +49,17 @@ export class ArgumentParser {
     const { values, positionals } = parseArgs({
       args: argv.slice(2),
       options: {
-        help: { type: 'boolean', short: 'h' },
-        version: { type: 'boolean', short: 'v' },
-        format: { type: 'string' },
-        simple: { type: 'boolean' },
-        theme: { type: 'string' },
-        'no-color': { type: 'boolean' },
-        'no-timing': { type: 'boolean' },
-        'no-wrap-lines': { type: 'boolean' },
-        'truncate-lines': { type: 'boolean' }
+        help: { type: "boolean", short: "h" },
+        version: { type: "boolean", short: "v" },
+        format: { type: "string" },
+        simple: { type: "boolean" },
+        json: { type: "boolean" },
+        github: { type: "boolean" },
+        theme: { type: "string" },
+        "no-color": { type: "boolean" },
+        "no-timing": { type: "boolean" },
+        "no-wrap-lines": { type: "boolean" },
+        "truncate-lines": { type: "boolean" }
       },
       allowPositionals: true
     })
@@ -69,8 +75,8 @@ export class ArgumentParser {
       process.exit(0)
     }
 
-    let formatOption: 'simple' | 'detailed' = 'detailed'
-    if (values.format && (values.format === "detailed" || values.format === "simple")) {
+    let formatOption: FormatOption = "detailed"
+    if (values.format && (values.format === "detailed" || values.format === "simple" || values.format === "json" || values.format === "github")) {
       formatOption = values.format
     }
 
@@ -78,21 +84,29 @@ export class ArgumentParser {
       formatOption = "simple"
     }
 
-    if (values['no-color']) {
+    if (values.json) {
+      formatOption = "json"
+    }
+
+    if (values.github) {
+      formatOption = "github"
+    }
+
+    if (values["no-color"]) {
       process.env.NO_COLOR = "1"
     }
 
-    const showTiming = !values['no-timing']
+    const showTiming = !values["no-timing"]
 
-    let wrapLines = !values['no-wrap-lines']
+    let wrapLines = !values["no-wrap-lines"]
     let truncateLines = false
 
-    if (values['truncate-lines']) {
+    if (values["truncate-lines"]) {
       truncateLines = true
       wrapLines = false
     }
 
-    if (!values['no-wrap-lines'] && values['truncate-lines']) {
+    if (!values["no-wrap-lines"] && values["truncate-lines"]) {
       console.error("Error: Line wrapping and --truncate-lines cannot be used together. Use --no-wrap-lines with --truncate-lines.")
       process.exit(1)
     }