From a436a9edccee8688ec916f4e7ba223532a5183c4 Mon Sep 17 00:00:00 2001
From: Jon Parise <jon@indelible.org>
Date: Wed, 19 Aug 2026 18:50:53 -0400
Subject: [PATCH] macos: avoid temporary path component allocation

The common directory helper previously allocated a temporary slice to
prepend the base directory before joining path components. Pass the
three known components directly to std.fs.path.join, leaving only the
allocation for the returned path.
---
 src/os/macos.zig | 17 +++++++----------
 1 file changed, 7 insertions(+), 10 deletions(-)

diff --git a/src/os/macos.zig b/src/os/macos.zig
index fcd1c3e5a..b59882d77 100644
--- a/src/os/macos.zig
+++ b/src/os/macos.zig
@@ -28,7 +28,7 @@ pub fn appSupportDir(
     return try commonDir(
         alloc,
         .NSApplicationSupportDirectory,
-        &.{ build_config.bundle_id, sub_path },
+        sub_path,
     );
 }
 
@@ -43,7 +43,7 @@ pub fn cacheDir(
     return try commonDir(
         alloc,
         .NSCachesDirectory,
-        &.{ build_config.bundle_id, sub_path },
+        sub_path,
     );
 }
 
@@ -110,7 +110,7 @@ pub const NSSearchPathDomainMask = enum(c_ulong) {
 fn commonDir(
     alloc: Allocator,
     directory: NSSearchPathDirectory,
-    sub_paths: []const []const u8,
+    sub_path: []const u8,
 ) (error{AppleAPIFailed} || Allocator.Error)![]const u8 {
     comptime assert(builtin.target.os.tag.isDarwin());
 
@@ -140,13 +140,10 @@ fn commonDir(
         return error.AppleAPIFailed;
     const base_dir = std.mem.sliceTo(c_str, 0);
 
-    // Create a new array with base_dir as the first element
-    var paths = try alloc.alloc([]const u8, sub_paths.len + 1);
-    paths[0] = base_dir;
-    @memcpy(paths[1..], sub_paths);
-    defer alloc.free(paths);
-
-    return try std.fs.path.join(alloc, paths);
+    return try std.fs.path.join(
+        alloc,
+        &.{ base_dir, build_config.bundle_id, sub_path },
+    );
 }
 
 test "cacheDir paths" {