@stacksjs/actions 0.75.275 → 0.75.277

package/dist/project-scripts.js

@@ -57,6 +57,8 @@ public static class StacksOwnedCheck {
   [DllImport("kernel32.dll", SetLastError = true)]
   static extern bool PeekNamedPipe(IntPtr pipe, IntPtr buffer, uint size, IntPtr read,
     out uint available, IntPtr remaining);
+  [DllImport("kernel32.dll", SetLastError = true)]
+  static extern bool ReadFile(IntPtr pipe, byte[] buffer, uint count, out uint read, IntPtr overlapped);
   [DllImport("kernel32.dll", CharSet = CharSet.Unicode, SetLastError = true)]
   static extern bool CreateProcessW(string application, StringBuilder command, IntPtr processSecurity,
     IntPtr threadSecurity, bool inherit, uint flags, IntPtr environment, string directory,
@@ -91,6 +93,27 @@ public static class StacksOwnedCheck {
     if (error == 109 || error == 232) return true; // Closed control pipe.
     throw new Win32Exception(error);
   }
+  static bool AwaitStart() {
+    // No consumer can exist until the live Bun host grants permission. It may
+    // safely kill a compiling or waiting helper when cancellation wins first.
+    Console.Out.WriteLine("STACKS_OWNED_CHECK_READY");
+    Console.Out.Flush();
+    while (true) {
+      uint available;
+      if (!PeekNamedPipe(GetStdHandle(-10), IntPtr.Zero, 0, IntPtr.Zero, out available, IntPtr.Zero)) {
+        int error = Marshal.GetLastWin32Error();
+        if (error == 109 || error == 232) return false;
+        throw new Win32Exception(error);
+      }
+      if (available != 0) {
+        byte[] command = new byte[1];
+        uint read;
+        Require(ReadFile(GetStdHandle(-10), command, 1, out read, IntPtr.Zero));
+        return read == 1 && command[0] == 1;
+      }
+      Thread.Sleep(10);
+    }
+  }
   static IntPtr Job() {
     IntPtr job = CreateJobObjectW(IntPtr.Zero, null);
     Require(job != IntPtr.Zero);
@@ -101,8 +124,8 @@ public static class StacksOwnedCheck {
     return job;
   }
   public static int Run(string bun, string script) {
-    // A queued stop must prevent startup even when Add-Type took most of the
-    // check's budget. The control pipe is never inherited as consumer stdin.
+    // The control pipe is never inherited as consumer stdin. Prepare both
+    // ownership jobs before reporting readiness or accepting startup permission.
     if (StopRequested()) return 1;
     // The outer job owns this host before it creates anything. If cancellation
     // races suspended-child assignment below, host death still kills that child.
@@ -113,6 +136,7 @@ public static class StacksOwnedCheck {
     IntPtr commandJob = Job();
     ProcessInfo child = new ProcessInfo();
     try {
+      if (!AwaitStart() || StopRequested()) return 1;
       Startup startup = new Startup();
       startup.Size = (uint)Marshal.SizeOf(typeof(Startup));
       startup.Flags = 0x100; // STARTF_USESTDHANDLES
@@ -175,6 +199,7 @@ exit [StacksOwnedCheck]::Run('${bun.replaceAll("'","''")}', '${script}')
 setInterval(() => {}, 60_000)
 let command
 let stopRequested = false
+let consumerPermitted = false
 let start
 const permitted = new Promise(resolve => { start = resolve })
 process.on('message', async (message) => {
@@ -183,8 +208,16 @@ process.on('message', async (message) => {
     start()
     if (command) {
       try {
-        command.stdin.write(new Uint8Array([1]))
-        await command.stdin.flush()
+        if (!consumerPermitted) {
+          // Preparation may itself spawn the C# compiler. Before permission,
+          // the entire helper tree is disposable without interrupting user code.
+          const killed = Bun.spawnSync([${JSON.stringify(windowsTaskkill)}, '/PID', String(command.pid), '/T', '/F'], { stdout: 'ignore', stderr: 'ignore', timeout: 2000 })
+          if (killed.exitCode !== 0 && command.exitCode === null) throw new Error('Native preparation cleanup failed')
+        }
+        else {
+          command.stdin.write(new Uint8Array([2]))
+          await command.stdin.flush()
+        }
       } catch (error) { process.send({ error: String(error) }) }
     }
   } else if (message?.start) start()
@@ -194,10 +227,48 @@ await permitted
 if (stopRequested) process.send({ exitCode: 1 })
 else {
   try {
-    command = Bun.spawn(${JSON.stringify(command)}, { stdin: 'pipe', stdout: 'inherit', stderr: 'inherit' })
+    command = Bun.spawn(${JSON.stringify(command)}, { stdin: 'pipe', stdout: 'pipe', stderr: 'inherit' })
     process.send({ helperPid: command.pid })
-    process.send({ exitCode: await command.exited })
-  } catch (error) { process.send({ error: String(error) }) }
+    const forward = async () => {
+      const reader = command.stdout.getReader()
+      let prefix = Buffer.alloc(0)
+      let ready = false
+      try {
+        while (true) {
+          const next = await reader.read()
+          if (next.done) break
+          if (ready) { await Bun.write(Bun.stdout, next.value); continue }
+          prefix = Buffer.concat([prefix, Buffer.from(next.value)])
+          const line = prefix.indexOf(10)
+          if (line < 0) {
+            if (prefix.length > 128) throw new Error('Native ownership readiness was malformed')
+            continue
+          }
+          if (prefix.subarray(0, line).toString().trimEnd() !== 'STACKS_OWNED_CHECK_READY')
+            throw new Error('Native ownership readiness was malformed')
+          ready = true
+          if (!stopRequested) {
+            // This assignment and write share one event-loop turn. A later
+            // stop therefore uses cooperative Job Object cleanup, never the
+            // preparation-only kill once user code could be running.
+            consumerPermitted = true
+            command.stdin.write(new Uint8Array([1]))
+            await command.stdin.flush()
+          }
+          if (prefix.length > line + 1) await Bun.write(Bun.stdout, prefix.subarray(line + 1))
+          prefix = Buffer.alloc(0)
+        }
+        if (!ready && !stopRequested) throw new Error('Native ownership readiness was not received')
+      } finally { reader.releaseLock() }
+    }
+    const [code] = await Promise.all([command.exited, forward()])
+    process.send({ exitCode: code })
+  } catch (error) {
+    if (command && !consumerPermitted) {
+      try { command.kill('SIGKILL'); await command.exited } catch {}
+    }
+    process.send({ error: String(error) })
+  }
 }
 `:`
 setInterval(() => {}, 60_000)

package/package.json

@@ -2,7 +2,7 @@
   "name": "@stacksjs/actions",
   "type": "module",
   "sideEffects": false,
-  "version": "0.75.275",
+  "version": "0.75.277",
   "description": "The Stacks actions.",
   "author": "Chris Breuer",
   "contributors": [
@@ -66,58 +66,58 @@
     "prepublishOnly": "bun run build"
   },
   "dependencies": {
-    "@stacksjs/action-runner": "0.75.275",
-    "@stacksjs/ai": "0.75.275",
-    "@stacksjs/analytics": "0.75.275",
-    "@stacksjs/api": "0.75.275",
-    "@stacksjs/auth": "0.75.275",
-    "@stacksjs/build": "0.75.275",
+    "@stacksjs/action-runner": "0.75.277",
+    "@stacksjs/ai": "0.75.277",
+    "@stacksjs/analytics": "0.75.277",
+    "@stacksjs/api": "0.75.277",
+    "@stacksjs/auth": "0.75.277",
+    "@stacksjs/build": "0.75.277",
     "@stacksjs/bumpx": "^0.2.26",
     "@stacksjs/bunpress": "^0.2.15",
-    "@stacksjs/cli": "0.75.275",
-    "@stacksjs/cloud": "0.75.275",
-    "@stacksjs/cms": "0.75.275",
+    "@stacksjs/cli": "0.75.277",
+    "@stacksjs/cloud": "0.75.277",
+    "@stacksjs/cms": "0.75.277",
     "@stacksjs/components": "0.2.670",
-    "@stacksjs/config": "0.75.275",
-    "@stacksjs/database": "0.75.275",
-    "@stacksjs/desktop-build": "0.75.275",
-    "@stacksjs/dns": "0.75.275",
-    "@stacksjs/enums": "0.75.275",
-    "@stacksjs/env": "0.75.275",
-    "@stacksjs/error-handling": "0.75.275",
-    "@stacksjs/github": "0.75.275",
+    "@stacksjs/config": "0.75.277",
+    "@stacksjs/database": "0.75.277",
+    "@stacksjs/desktop-build": "0.75.277",
+    "@stacksjs/dns": "0.75.277",
+    "@stacksjs/enums": "0.75.277",
+    "@stacksjs/env": "0.75.277",
+    "@stacksjs/error-handling": "0.75.277",
+    "@stacksjs/github": "0.75.277",
     "@stacksjs/gitit": "^0.2.5",
-    "@stacksjs/i18n": "0.75.275",
-    "@stacksjs/image": "0.75.275",
-    "@stacksjs/logging": "0.75.275",
+    "@stacksjs/i18n": "0.75.277",
+    "@stacksjs/image": "0.75.277",
+    "@stacksjs/logging": "0.75.277",
     "@stacksjs/logsmith": "^0.2.11",
-    "@stacksjs/mobile": "0.75.275",
-    "@stacksjs/orm": "0.75.275",
-    "@stacksjs/path": "0.75.275",
-    "@stacksjs/payments": "0.75.275",
-    "@stacksjs/queue": "0.75.275",
-    "@stacksjs/registry": "0.75.275",
-    "@stacksjs/router": "0.75.275",
+    "@stacksjs/mobile": "0.75.277",
+    "@stacksjs/orm": "0.75.277",
+    "@stacksjs/path": "0.75.277",
+    "@stacksjs/payments": "0.75.277",
+    "@stacksjs/queue": "0.75.277",
+    "@stacksjs/registry": "0.75.277",
+    "@stacksjs/router": "0.75.277",
     "@stacksjs/rpx": "^0.11.54",
-    "@stacksjs/scheduler": "0.75.275",
-    "@stacksjs/search-engine": "0.75.275",
-    "@stacksjs/security": "0.75.275",
-    "@stacksjs/server": "0.75.275",
-    "@stacksjs/sites": "0.75.275",
-    "@stacksjs/storage": "0.75.275",
-    "@stacksjs/strings": "0.75.275",
+    "@stacksjs/scheduler": "0.75.277",
+    "@stacksjs/search-engine": "0.75.277",
+    "@stacksjs/security": "0.75.277",
+    "@stacksjs/server": "0.75.277",
+    "@stacksjs/sites": "0.75.277",
+    "@stacksjs/storage": "0.75.277",
+    "@stacksjs/strings": "0.75.277",
     "@stacksjs/stx": "0.2.670",
     "@stacksjs/tlsx": "^0.13.19",
     "@stacksjs/ts-cloud": "^0.16.58",
     "@stacksjs/ts-md": "^0.1.1",
-    "@stacksjs/tunnel": "0.75.275",
-    "@stacksjs/types": "0.75.275",
-    "@stacksjs/utils": "0.75.275",
-    "@stacksjs/whois": "0.75.275",
+    "@stacksjs/tunnel": "0.75.277",
+    "@stacksjs/types": "0.75.277",
+    "@stacksjs/utils": "0.75.277",
+    "@stacksjs/whois": "0.75.277",
     "bun-plugin-stx": "0.2.670"
   },
   "peerDependencies": {
-    "craft-native": ">=0.0.296",
+    "craft-native": ">=0.0.301",
     "pickier": "^0.1.75"
   },
   "peerDependenciesMeta": {
@@ -129,7 +129,7 @@
     }
   },
   "devDependencies": {
-    "@stacksjs/validation": "0.75.275",
+    "@stacksjs/validation": "0.75.277",
     "better-dx": "^0.2.24"
   }
 }