fix superhtml cli tool

the update to zig 0.14.0 fixed using superhtml as
a library, but failed to update the CLI tool code,
mainly because the test step would only run unit
tests without trying to build the full library.

this commit fixes the few issues that remained
and also made the `test` step depend on `check`,
ensuring that going forward the CLI tool doesn't
lag behind

Author: kristoff-it

build.zig

@@ -51,10 +51,10 @@ pub fn build(b: *std.Build) !void {
     const folders = b.dependency("known_folders", .{});
     const lsp = b.dependency("lsp_kit", .{});
 
-    setupTestStep(b, target, superhtml);
+    const check = setupCheckStep(b, target, optimize, options, superhtml, folders, lsp);
+    setupTestStep(b, target, superhtml, check);
     setupCliTool(b, target, optimize, options, superhtml, folders, lsp);
     setupWasmStep(b, optimize, options, superhtml, lsp);
-    setupCheckStep(b, target, optimize, options, superhtml, folders, lsp);
     if (version == .tag) {
         setupReleaseStep(b, options, superhtml, folders, lsp);
     }
@@ -76,7 +76,8 @@ fn setupCheckStep(
     superhtml: *std.Build.Module,
     folders: *std.Build.Dependency,
     lsp: *std.Build.Dependency,
-) void {
+) *std.Build.Step {
+    const check = b.step("check", "Check if the SuperHTML CLI compiles");
     const super_cli_check = b.addExecutable(.{
         .name = "superhtml",
         .root_source_file = b.path("src/cli.zig"),
@@ -92,15 +93,17 @@ fn setupCheckStep(
     super_cli_check.root_module.addImport("lsp", lsp.module("lsp"));
     super_cli_check.root_module.addOptions("build_options", options);
 
-    const check = b.step("check", "Check if the SuperHTML CLI compiles");
     check.dependOn(&super_cli_check.step);
+    return check;
 }
 fn setupTestStep(
     b: *std.Build,
     target: std.Build.ResolvedTarget,
     superhtml: *std.Build.Module,
+    check: *std.Build.Step,
 ) void {
     const test_step = b.step("test", "Run unit tests");
+    test_step.dependOn(check);
 
     const unit_tests = b.addTest(.{
         .root_source_file = b.path("src/root.zig"),
@@ -273,7 +276,7 @@ fn setupReleaseStep(
         super_exe_release.root_module.addImport("superhtml", superhtml);
         super_exe_release.root_module.addImport(
             "known_folders",
-            folders.module("known_folders"),
+            folders.module("known-folders"),
         );
         super_exe_release.root_module.addImport("lsp", lsp.module("lsp"));
         super_exe_release.root_module.addOptions("build_options", options);

src/cli.zig

@@ -1,7 +1,7 @@
 const std = @import("std");
 const builtin = @import("builtin");
 const build_options = @import("build_options");
-const known = @import("known-folders");
+const known = @import("known_folders");
 const super = @import("super");
 const logging = @import("cli/logging.zig");
 const interface_exe = @import("cli/interface.zig");

src/cli/logging.zig

@@ -1,7 +1,7 @@
 const std = @import("std");
 const builtin = @import("builtin");
 const build_options = @import("build_options");
-const folders = @import("known-folders");
+const folders = @import("known_folders");
 
 pub var log_file: ?std.fs.File = switch (builtin.target.os.tag) {
     .linux, .macos => std.io.getStdErr(),

src/cli/lsp.zig

@@ -284,6 +284,14 @@ pub const Handler = struct {
         };
     }
 
+    pub fn documentSymbol(
+        _: Handler,
+        _: std.mem.Allocator,
+        _: types.DocumentSymbolParams,
+    ) !ResultType("textDocument/documentSymbol") {
+        return null;
+    }
+
     pub fn references(
         _: Handler,
         arena: std.mem.Allocator,
@@ -353,7 +361,7 @@ pub const Handler = struct {
         _ = self;
         const id: []const u8 = switch (_response.id) {
             .string => |id| id,
-            .integer => |id| {
+            .number => |id| {
                 log.warn("received response from client with id '{d}' that has no handler!", .{id});
                 return;
             },