@decohere/ci-tools 0.0.1-dev.127 → 0.0.1-dev.129

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@decohere/ci-tools",
-  "version": "0.0.1-dev.127",
+  "version": "0.0.1-dev.129",
   "description": "CI utility scripts for badge generation, coverage reporting, pages artifact assembly, and GitHub API tooling",
   "keywords": [
     "typescript",

package/src/github/smoke-create-launcher.mjs

@@ -10,9 +10,14 @@ import { pathToFileURL } from "node:url";
  * Must not use the repo cwd — after `@decohere/cli` is pinned, pnpm installs
  * `@decohere/create` into the workspace tree and npm's npx misses the bin.
  * No token: the packages are public, so this installs the launcher the way a user does.
+ * Anonymous reads go through npm's cache, which can lag minutes behind a publish, so it
+ * first waits until npm serves the version (`attempts` tries, `intervalMs` apart).
  *
  * @param {{
  *   createVersion: string,
+ *   attempts?: number,
+ *   intervalMs?: number,
+ *   sleep?: (ms: number) => void,
  *   mkdtempSync?: typeof mkdtempSync,
  *   writeFileSync?: typeof writeFileSync,
  *   spawnSync?: typeof spawnSync,
@@ -30,14 +35,31 @@ export function runCreateLauncherSmoke(options) {
 	const writeFile = options.writeFileSync ?? writeFileSync;
 	const spawn = options.spawnSync ?? spawnSync;
 	const tmp = options.tmpdir ?? tmpdir;
+	const sleep = options.sleep ?? sleepSync;
+	const attempts = options.attempts ?? 20;
+	const intervalMs = options.intervalMs ?? 15_000;
 
 	const workDir = mkdtemp(join(tmp(), "decohere-create-smoke-"));
 	const npmrc = join(workDir, ".npmrc");
 	writeFile(npmrc, "@decohere:registry=https://registry.npmjs.org/\n");
 
-	const result = spawn("npx", ["--yes", `@decohere/create@${version}`, "--help"], {
+	const env = { ...process.env, NPM_CONFIG_USERCONFIG: npmrc };
+	const spec = `@decohere/create@${version}`;
+
+	for (let attempt = 1; ; attempt++) {
+		const view = spawn("npm", ["view", spec, "version"], { cwd: workDir, env, stdio: "pipe" });
+		if (view.status === 0) break;
+		if (attempt >= attempts) {
+			console.error(`${spec} is still not served by npm after ${attempts} tries`);
+			return { workDir, status: 1 };
+		}
+		console.log(`${spec} not served by npm yet (try ${attempt} of ${attempts}); waiting`);
+		sleep(intervalMs);
+	}
+
+	const result = spawn("npx", ["--yes", spec, "--help"], {
 		cwd: workDir,
-		env: { ...process.env, NPM_CONFIG_USERCONFIG: npmrc },
+		env,
 		stdio: options.stdio ?? "inherit",
 	});
 
@@ -47,6 +69,11 @@ export function runCreateLauncherSmoke(options) {
 	};
 }
 
+/** @param {number} ms */
+function sleepSync(ms) {
+	Atomics.wait(new Int32Array(new SharedArrayBuffer(4)), 0, 0, ms);
+}
+
 const isCli =
 	process.argv[1] != null && import.meta.url === pathToFileURL(resolve(process.argv[1])).href;
 

package/tests/github/ci-workflow.test.mjs

@@ -0,0 +1,19 @@
+import { fileURLToPath } from "node:url";
+import { describe, expect, it, vi } from "vitest";
+
+const fs = await vi.importActual("node:fs");
+const CI_YML = fileURLToPath(new URL("../../../../.github/workflows/ci.yml", import.meta.url));
+
+describe("ci.yml and the merge queue", () => {
+	const yml = fs.readFileSync(CI_YML, "utf8");
+	const triggers = yml.slice(yml.indexOf("\non:"), yml.indexOf("\nconcurrency:"));
+	const gate = yml.slice(yml.indexOf("- id: gate"), yml.indexOf("# Static gates"));
+
+	it("runs for queued pull requests, or the queue waits forever for the required check", () => {
+		expect(triggers).toMatch(/^\s+merge_group:/m);
+	});
+
+	it("runs the whole suite in the queue: the docs-only shortcut is only for pull requests", () => {
+		expect(gate).toContain('"${{ github.event_name }}" == "merge_group"');
+	});
+});

package/tests/github/smoke-create-launcher.test.mjs

@@ -29,11 +29,11 @@ describe("runCreateLauncherSmoke", () => {
 
 		expect(result.workDir).toBe("/tmp/decohere-create-smoke-abc");
 		expect(result.workDir).not.toBe(process.cwd());
-		expect(calls).toHaveLength(1);
-		expect(calls[0].cmd).toBe("npx");
-		expect(calls[0].args).toEqual(["--yes", "@decohere/create@0.0.1-dev.80", "--help"]);
-		expect(calls[0].opts.cwd).toBe("/tmp/decohere-create-smoke-abc");
-		expect(calls[0].opts.cwd).not.toBe(process.cwd());
+		const npx = calls.at(-1);
+		expect(npx.cmd).toBe("npx");
+		expect(npx.args).toEqual(["--yes", "@decohere/create@0.0.1-dev.80", "--help"]);
+		expect(npx.opts.cwd).toBe("/tmp/decohere-create-smoke-abc");
+		expect(npx.opts.cwd).not.toBe(process.cwd());
 		expect(written).toEqual([
 			{
 				path: "/tmp/decohere-create-smoke-abc/.npmrc",
@@ -41,7 +41,57 @@ describe("runCreateLauncherSmoke", () => {
 			},
 		]);
 		// The packages are public: the smoke installs them the way a user does, with no token.
-		expect(calls[0].opts.env.NODE_AUTH_TOKEN).toBeUndefined();
+		expect(npx.opts.env.NODE_AUTH_TOKEN).toBeUndefined();
+	});
+
+	// npm's public cache can lag minutes behind a publish: the first token-free Publish ran npx
+	// 90 seconds after publishing and got ETARGET for a version the registry already had.
+	it("waits until npm serves the new version before running npx", () => {
+		/** @type {{ cmd: string, args: string[] }[]} */
+		const calls = [];
+		const sleeps = [];
+		let views = 0;
+
+		const result = runCreateLauncherSmoke({
+			createVersion: "0.0.1-dev.127",
+			tmpdir: () => "/tmp",
+			mkdtempSync: (prefix) => `${prefix}abc`,
+			writeFileSync: () => {},
+			spawnSync: (cmd, args) => {
+				calls.push({ cmd, args });
+				if (cmd === "npm") return { status: ++views < 3 ? 1 : 0 };
+				return { status: 0 };
+			},
+			sleep: (ms) => sleeps.push(ms),
+			stdio: "pipe",
+		});
+
+		expect(result.status).toBe(0);
+		expect(calls.map((call) => call.cmd)).toEqual(["npm", "npm", "npm", "npx"]);
+		expect(calls[0].args).toEqual(["view", "@decohere/create@0.0.1-dev.127", "version"]);
+		expect(sleeps).toHaveLength(2);
+	});
+
+	it("fails without running npx when npm never serves the version", () => {
+		/** @type {string[]} */
+		const commands = [];
+
+		const result = runCreateLauncherSmoke({
+			createVersion: "0.0.1-dev.127",
+			tmpdir: () => "/tmp",
+			mkdtempSync: (prefix) => `${prefix}abc`,
+			writeFileSync: () => {},
+			spawnSync: (cmd) => {
+				commands.push(cmd);
+				return { status: 1 };
+			},
+			sleep: () => {},
+			attempts: 3,
+			stdio: "pipe",
+		});
+
+		expect(result.status).toBe(1);
+		expect(commands).toEqual(["npm", "npm", "npm"]);
 	});
 
 	it("fails closed without a version", () => {