Upgrade to Zig 0.15.2 (#439)

---------

Co-authored-by: Sebastian <[email protected]>

Author: remorses

.github/workflows/build-core.yml

@@ -8,7 +8,7 @@ on:
 jobs:
   build:
     name: Core - Build and Test
-    runs-on: ubuntu-latest
+    runs-on: macos-latest
     steps:
       - name: Checkout code
         uses: actions/checkout@v4
@@ -21,7 +21,7 @@ jobs:
       - name: Setup Zig
         uses: goto-bus-stop/setup-zig@v2
         with:
-          version: 0.14.1
+          version: 0.15.2
 
       - name: Install dependencies
         run: bun install

.github/workflows/build-native.yml

@@ -14,7 +14,7 @@ on:
         default: false
 
 env:
-  ZIG_VERSION: 0.14.1
+  ZIG_VERSION: 0.15.2
 
 jobs:
   build-native:
@@ -39,7 +39,10 @@ jobs:
         run: bun install
 
       - name: Build packages (cross-compile for all platforms)
-        run: bun run build
+        run: |
+          cd packages/core
+          bun run build:native --all
+          bun run build:lib
 
       - name: Verify build outputs
         run: |

.github/workflows/build-react.yml

@@ -21,7 +21,7 @@ jobs:
       - name: Setup Zig
         uses: goto-bus-stop/setup-zig@v2
         with:
-          version: 0.14.1
+          version: 0.15.2
 
       - name: Install dependencies
         run: bun install

.github/workflows/build-solid.yml

@@ -21,7 +21,7 @@ jobs:
       - name: Setup Zig
         uses: goto-bus-stop/setup-zig@v2
         with:
-          version: 0.14.1
+          version: 0.15.2
 
       - name: Install dependencies
         run: bun install

.github/workflows/npm-release.yml

@@ -26,7 +26,7 @@ jobs:
       - name: Setup Zig
         uses: goto-bus-stop/setup-zig@v2
         with:
-          version: 0.14.1
+          version: 0.15.2
 
       - name: Extract version from tag
         id: extract_version
@@ -52,8 +52,13 @@ jobs:
       - name: Prepare release versions
         run: bun run prepare-release "${{ steps.version_scheme.outputs.version }}"
 
-      - name: Build packages
-        run: bun run build
+      - name: Build packages (cross-compile for all platforms)
+        run: |
+          cd packages/core
+          bun run build:native --all
+          bun run build:lib
+          cd ../solid && bun run build
+          cd ../react && bun run build
 
       - name: Publish packages
         run: bun run publish

.zig-version

@@ -1 +1 @@
-0.14.1
+0.15.2

packages/core/scripts/build.ts

@@ -40,6 +40,7 @@ const args = process.argv.slice(2)
 const buildLib = args.find((arg) => arg === "--lib")
 const buildNative = args.find((arg) => arg === "--native")
 const isDev = args.includes("--dev")
+const buildAll = args.includes("--all") // Build for all platforms
 
 const variants: Variant[] = [
   { platform: "darwin", arch: "x64" },
@@ -78,16 +79,17 @@ if (missingRequired.length > 0) {
 }
 
 if (buildNative) {
-  console.log(`Building native ${isDev ? "dev" : "prod"} binaries...`)
+  console.log(`Building native ${isDev ? "dev" : "prod"} binaries${buildAll ? " for all platforms" : ""}...`)
 
-  const zigBuild: SpawnSyncReturns<Buffer> = spawnSync(
-    "zig",
-    ["build", `-Doptimize=${isDev ? "Debug" : "ReleaseFast"}`],
-    {
-      cwd: join(rootDir, "src", "zig"),
-      stdio: "inherit",
-    },
-  )
+  const zigArgs = ["build", `-Doptimize=${isDev ? "Debug" : "ReleaseFast"}`]
+  if (buildAll) {
+    zigArgs.push("-Dall")
+  }
+
+  const zigBuild: SpawnSyncReturns<Buffer> = spawnSync("zig", zigArgs, {
+    cwd: join(rootDir, "src", "zig"),
+    stdio: "inherit",
+  })
 
   if (zigBuild.error) {
     console.error("Error: Zig is not installed or not in PATH")
@@ -124,16 +126,10 @@ if (buildNative) {
     }
 
     if (copiedFiles === 0) {
-      console.error(`Error: No dynamic libraries found for ${platform}-${arch} in ${libDir}`)
-      console.error(`Expected to find files like: libopentui.so, libopentui.dylib, opentui.dll`)
-      console.error(`Found files in ${libDir}:`)
-      if (existsSync(libDir)) {
-        const files = spawnSync("ls", ["-la", libDir], { stdio: "pipe" })
-        if (files.stdout) console.error(files.stdout.toString())
-      } else {
-        console.error("Directory does not exist")
-      }
-      process.exit(1)
+      // Skip platforms that weren't built
+      console.log(`Skipping ${platform}-${arch}: no libraries found`)
+      rmSync(nativeDir, { recursive: true, force: true })
+      continue
     }
 
     const indexTsContent = `const module = await import("./${libraryFileName}", { with: { type: "file" } })

packages/core/src/renderables/Code.test.ts

@@ -1692,6 +1692,16 @@ test("CodeRenderable - streaming with conceal and drawUnstyledText=false should
 
   currentRenderer.root.add(codeRenderable)
 
+  const waitForHighlightingCycle = async (timeout = 2000) => {
+    const start = Date.now()
+    await renderOnce()
+    await new Promise((resolve) => setTimeout(resolve, 10))
+    while (codeRenderable.isHighlighting && Date.now() - start < timeout) {
+      await new Promise((resolve) => setTimeout(resolve, 10))
+    }
+    await renderOnce()
+  }
+
   // Use TestRecorder to capture frames
   const { TestRecorder } = await import("../testing/test-recorder")
   const recorder = new TestRecorder(currentRenderer)
@@ -1701,13 +1711,13 @@ test("CodeRenderable - streaming with conceal and drawUnstyledText=false should
   recorder.rec()
 
   // Wait for initial highlighting to complete
-  await new Promise((resolve) => setTimeout(resolve, 100))
+  await waitForHighlightingCycle()
 
   // Now simulate streaming: add more content including fenced code block
   codeRenderable.content = `# Example\n\nHere's some code:\n\n\`\`\`typescript\nconst x = 1;\n\`\`\``
 
   // Wait for highlighting to process the update
-  await new Promise((resolve) => setTimeout(resolve, 150))
+  await waitForHighlightingCycle()
 
   // Stop everything
   currentRenderer.stop()

packages/core/src/tests/wrap-resize-perf.test.ts

@@ -176,15 +176,16 @@ describe("Word wrap algorithmic complexity", () => {
     const widths = [60, 70, 80, 90, 100]
     const times: number[] = []
 
-    view.setWrapWidth(widths[0])
-    view.measureForDimensions(widths[0], 100)
+    // Warmup
+    view.setWrapWidth(50)
+    view.measureForDimensions(50, 100)
 
+    // Measure first (uncached) call for each width
     for (const width of widths) {
       view.setWrapWidth(width)
-      const time = measureMedian(() => {
-        view.measureForDimensions(width, 100)
-      })
-      times.push(time)
+      const start = performance.now()
+      view.measureForDimensions(width, 100)
+      times.push(performance.now() - start)
     }
 
     view.destroy()

packages/core/src/zig/ansi.zig

@@ -21,15 +21,15 @@ pub const ANSI = struct {
 
     // Direct writing to any writer - the most efficient option
     pub fn moveToOutput(writer: anytype, x: u32, y: u32) AnsiError!void {
-        std.fmt.format(writer, "\x1b[{d};{d}H", .{ y, x }) catch return AnsiError.WriteFailed;
+        writer.print("\x1b[{d};{d}H", .{ y, x }) catch return AnsiError.WriteFailed;
     }
 
     pub fn fgColorOutput(writer: anytype, r: u8, g: u8, b: u8) AnsiError!void {
-        std.fmt.format(writer, "\x1b[38;2;{d};{d};{d}m", .{ r, g, b }) catch return AnsiError.WriteFailed;
+        writer.print("\x1b[38;2;{d};{d};{d}m", .{ r, g, b }) catch return AnsiError.WriteFailed;
     }
 
     pub fn bgColorOutput(writer: anytype, r: u8, g: u8, b: u8) AnsiError!void {
-        std.fmt.format(writer, "\x1b[48;2;{d};{d};{d}m", .{ r, g, b }) catch return AnsiError.WriteFailed;
+        writer.print("\x1b[48;2;{d};{d};{d}m", .{ r, g, b }) catch return AnsiError.WriteFailed;
     }
 
     // Text attribute constants
@@ -51,11 +51,11 @@ pub const ANSI = struct {
     pub const cursorUnderlineBlink = "\x1b[3 q";
 
     pub fn cursorColorOutputWriter(writer: anytype, r: u8, g: u8, b: u8) AnsiError!void {
-        std.fmt.format(writer, "\x1b]12;#{x:0>2}{x:0>2}{x:0>2}\x07", .{ r, g, b }) catch return AnsiError.WriteFailed;
+        writer.print("\x1b]12;#{x:0>2}{x:0>2}{x:0>2}\x07", .{ r, g, b }) catch return AnsiError.WriteFailed;
     }
 
     pub fn explicitWidthOutput(writer: anytype, width: u32, text: []const u8) AnsiError!void {
-        std.fmt.format(writer, "\x1b]66;w={d};{s}\x1b\\", .{ width, text }) catch return AnsiError.WriteFailed;
+        writer.print("\x1b]66;w={d};{s}\x1b\\", .{ width, text }) catch return AnsiError.WriteFailed;
     }
 
     pub const resetCursorColor = "\x1b]112\x07";
@@ -131,12 +131,15 @@ pub const ANSI = struct {
     pub const setTerminalTitle = "\x1b]0;{s}\x07";
 
     pub fn setTerminalTitleOutput(writer: anytype, title: []const u8) AnsiError!void {
-        std.fmt.format(writer, setTerminalTitle, .{title}) catch return AnsiError.WriteFailed;
+        writer.print(setTerminalTitle, .{title}) catch return AnsiError.WriteFailed;
     }
 
     pub fn makeRoomForRendererOutput(writer: anytype, height: u32) AnsiError!void {
         if (height > 1) {
-            writer.writeByteNTimes('\n', height - 1) catch return AnsiError.WriteFailed;
+            var i: u32 = 0;
+            while (i < height - 1) : (i += 1) {
+                writer.writeByte('\n') catch return AnsiError.WriteFailed;
+            }
         }
     }
 };