Sync from the Ink Studio AI repository (da88bcd)

Author: github-actions[bot]

test/cli.test.mjs

@@ -3,17 +3,15 @@ import { existsSync, mkdtempSync, readFileSync, readdirSync, rmSync, statSync, w
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { Readable } from "node:stream";
-import { after, afterEach, before, beforeEach, describe, test } from "node:test";
+import { after, afterEach, beforeEach, describe, test } from "node:test";
 import { main } from "../src/cli.js";
 import { IMAGE_BYTES, OTHER_KEY, PRICING, VALID_KEY, startMockServer } from "./mock-server.mjs";
 
-/** @type {Awaited<ReturnType<typeof startMockServer>>} */
-let api;
+// Top-level await, not before(): Node 18 can run a top-level before() hook
+// after the beforeEach() hooks of a describe() suite.
+const api = await startMockServer();
 let home = "";
 
-before(async () => {
-  api = await startMockServer();
-});
 after(() => api.close());
 beforeEach(() => {
   home = mkdtempSync(join(tmpdir(), "inkstudioai-cli-"));

test/mcp.test.mjs

@@ -4,20 +4,16 @@ import { chmodSync, mkdtempSync, readFileSync, rmSync, statSync } from "node:fs"
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { createInterface } from "node:readline";
-import { after, before, test } from "node:test";
+import { after, test } from "node:test";
 import { fileURLToPath } from "node:url";
 import { IMAGE_BYTES, VALID_KEY, startMockServer } from "./mock-server.mjs";
 
 const BIN = fileURLToPath(new URL("../bin/inkstudioai.js", import.meta.url));
 
-/** @type {Awaited<ReturnType<typeof startMockServer>>} */
-let api;
-let home = "";
+// Top-level await, not before(): see the note in cli.test.mjs.
+const api = await startMockServer();
+const home = mkdtempSync(join(tmpdir(), "inkstudioai-mcp-"));
 
-before(async () => {
-  api = await startMockServer();
-  home = mkdtempSync(join(tmpdir(), "inkstudioai-mcp-"));
-});
 after(async () => {
   await api.close();
   rmSync(home, { recursive: true, force: true });

test/mock-server.mjs

@@ -226,6 +226,9 @@ export async function startMockServer() {
   });
 
   await new Promise((done) => server.listen(0, "127.0.0.1", () => done(undefined)));
+  // Node 18 runs top-level after() hooks only once the event loop is empty, so
+  // a listening server would keep the test process alive forever.
+  server.unref();
   const address = /** @type {import("node:net").AddressInfo} */ (server.address());
   origin = `http://127.0.0.1:${address.port}`;
   return {