Fuzzer goodies (#104)

* html tokenizer: fix path that failed to report a tag name

* fuzzer fixes

* add duplicate id detection

also implements correctly "namespacing" for `<template>` elements

* support both zig 0.15 and 0.16.0-dev

Author: kristoff-it

build.zig

@@ -22,6 +22,7 @@ pub fn build(b: *std.Build) !void {
     const superhtml = b.addModule("superhtml", .{
         .root_source_file = b.path("src/root.zig"),
         .target = target,
+        .optimize = optimize,
     });
     superhtml.addImport("scripty", scripty.module("scripty"));
     superhtml.addImport("tracy", tracy.module("tracy"));
@@ -77,6 +78,22 @@ pub fn build(b: *std.Build) !void {
     if (version == .tag) {
         setupReleaseStep(b, options, superhtml, folders, lsp);
     }
+
+    setupGeneratorStep(b, target);
+}
+
+fn setupGeneratorStep(b: *std.Build, target: std.Build.ResolvedTarget) void {
+    const gen = b.step("generator", "Build generator executable for reproing fuzz cases");
+    const supergen = b.addExecutable(.{
+        .name = "generator",
+        .root_module = b.createModule(.{
+            .root_source_file = b.path("src/generator.zig"),
+            .target = target,
+            .optimize = .ReleaseSafe,
+        }),
+    });
+
+    gen.dependOn(&b.addInstallArtifact(supergen, .{}).step);
 }
 
 fn setupCheckStep(
@@ -119,8 +136,7 @@ fn setupTestStep(
 
     const unit_tests = b.addTest(.{
         .root_module = superhtml,
-        // .strip = true,
-        // .filter = "if-else-loop",
+        .filters = b.args orelse &.{},
     });
 
     const run_unit_tests = b.addRunArtifact(unit_tests);

build.zig.zon

@@ -13,8 +13,8 @@
             .hash = "tracy-0.0.0-4Xw-1pwwAABTfMgoDP1unCbZDZhJEfict7XCBGF6IdIn",
         },
         .lsp_kit = .{
-            .url = "git+https://github.com/kristoff-it/lsp-kit?ref=zig-0.15#01c14e592d25dc57dfebba27b8bd2b4aa91c1140",
-            .hash = "lsp_kit-0.1.0-bi_PL5YyCgA2QFEza6llr2Uy08QUQsWBu2wKvtr8tbLx",
+            .url = "git+https://github.com/zigtools/lsp-kit#fe98e895ca3bd1b39965ab30f0f252f7b7e83ee6",
+            .hash = "lsp_kit-0.1.0-bi_PLzAyCgClDh8_M0U9Q50ysdsQBuRuBTZfwg6rZPd6",
         },
         .scripty = .{
             .url = "git+https://github.com/kristoff-it/scripty#50dbab8945440089384f26ec165d870c29555247",

src/cli.zig

@@ -31,9 +31,8 @@ pub fn panic(
 ) noreturn {
     if (lsp_mode) {
         std.log.err("\n{s}\n\n{?f}", .{ msg, trace });
-    } else {
-        std.debug.print("\n{s}\n\n{?f}", .{ msg, trace });
     }
+
     blk: {
         const out: std.fs.File = if (!lsp_mode) std.fs.File.stderr() else logging.log_file orelse break :blk;
         var writer = out.writerStreaming(&.{});
@@ -43,17 +42,13 @@ pub fn panic(
             w.print("Unable to dump stack trace: debug info stripped\n", .{}) catch {};
             break :blk;
         }
-        const debug_info = std.debug.getSelfDebugInfo() catch |err| {
-            w.print(
-                "Unable to dump stack trace: Unable to open debug info: {s}\n",
-                .{@errorName(err)},
-            ) catch {};
-            break :blk;
-        };
-        std.debug.writeCurrentStackTrace(w, debug_info, .no_color, ret_addr) catch |err| {
-            w.print("Unable to dump stack trace: {t}\n", .{err}) catch {};
-            break :blk;
-        };
+
+        if (builtin.zig_version.minor != 15) {
+            std.debug.writeCurrentStackTrace(.{ .first_address = ret_addr }, w, .no_color) catch |err| {
+                w.print("Unable to dump stack trace: {t}\n", .{err}) catch {};
+                break :blk;
+            };
+        }
     }
 
     if (builtin.mode == .Debug) @breakpoint();

src/cli/check.zig

@@ -1,9 +1,10 @@
+const builtin = @import("builtin");
 const std = @import("std");
 const super = @import("superhtml");
 
 const FileType = enum { html, super };
 
-pub fn run(gpa: std.mem.Allocator, args: []const []const u8) !void {
+pub fn run(gpa: std.mem.Allocator, args: []const []const u8) !noreturn {
     const cmd = Command.parse(args);
     var any_error = false;
     switch (cmd.mode) {
@@ -64,6 +65,7 @@ pub fn run(gpa: std.mem.Allocator, args: []const []const u8) !void {
     if (any_error) {
         std.process.exit(1);
     }
+    std.process.exit(0);
 }
 
 fn checkDir(
@@ -106,13 +108,19 @@ fn checkFile(
     defer _ = arena_impl.reset(.retain_capacity);
     const arena = arena_impl.allocator();
 
-    const in_bytes = try base_dir.readFileAllocOptions(
+    const in_bytes = if (builtin.zig_version.minor == 15) try base_dir.readFileAllocOptions(
         arena,
         sub_path,
         super.max_size,
         null,
         .of(u8),
         0,
+    ) else try base_dir.readFileAllocOptions(
+        sub_path,
+        arena,
+        .limited(super.max_size),
+        .of(u8),
+        0,
     );
 
     const file_type: FileType = blk: {