gentle-pi 3.7.0 → 4.1.0

package/tests/verify-package-files.test.ts

@@ -1,16 +1,30 @@
 import assert from "node:assert/strict";
+import { spawnSync } from "node:child_process";
 import { mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
+import { fileURLToPath } from "node:url";
 import test from "node:test";
 import {
 	gentleAiVersionPinMismatches,
+	installerPaths,
+	missingRequiredPaths,
 	reconcileContractsOnDisk,
 	reconcileGeneratedRuntimeSources,
+	requiredPaths,
 } from "../scripts/verify-package-files.mjs";
 import { INSTALLER_VERSION, RELEASE_BASE_URL, GENTLE_AI_WINDOWS_SOURCE_TAG } from "../scripts/gentle-ai-installer.mjs";
 import { GENTLE_AI_VERSION } from "../lib/gentle-ai-binary.ts";
 
+test("package resource CLI reconciles the STATUS diagnostic runtime with the generated inventory", () => {
+	const result = spawnSync(process.execPath, [join(process.cwd(), "scripts/verify-package-files.mjs")], {
+		cwd: process.cwd(), encoding: "utf8", env: { ...process.env, NODE_NO_WARNINGS: "1" },
+	});
+	assert.equal(result.status, 0, result.stderr || result.stdout);
+	assert.equal(result.stderr, "");
+	assert.match(result.stdout, /^gentle-pi package resource check passed \(/);
+});
+
 function makeFixtureRoot(): string {
 	return mkdtempSync(join(tmpdir(), "gentle-pi-verify-package-files-"));
 }
@@ -179,3 +193,45 @@ test("both walks report no drift when sources, runtime/*.mjs, requiredPaths, and
 		rmSync(fixtureRoot, { recursive: true, force: true });
 	}
 });
+
+test("required package paths include every browser installation wizard file", () => {
+	const expected = [
+		"bin/gentle-shell-install.mjs",
+		"scripts/installer-server.mjs",
+		"scripts/installer-runner.mjs",
+		"scripts/installer-preflight.mjs",
+		"scripts/installer-probes.mjs",
+		"scripts/installer-downloads.mjs",
+		"scripts/main-channel.mjs",
+		"scripts/installer-windows.mjs",
+		"scripts/installer-windows-artifacts.json",
+		// The bundled install the wizard runs for a new release installation (T1c).
+		"scripts/bundled-wizard.mjs",
+		"scripts/bundled-install.mjs",
+		"scripts/bootstrap.sh",
+		"scripts/bootstrap.cmd",
+		"assets/install-wizard/index.html",
+		"assets/install-wizard/wizard.js",
+		"assets/install-wizard/wizard.css",
+	];
+	assert.deepEqual([...installerPaths].sort(), [...expected].sort());
+	for (const relativePath of expected) assert.ok(requiredPaths.includes(relativePath), relativePath);
+	const repositoryRoot = fileURLToPath(new URL("..", import.meta.url));
+	assert.deepEqual(missingRequiredPaths(repositoryRoot, installerPaths), []);
+});
+
+test("missing required paths are reported, including directories in place of files", () => {
+	const fixtureRoot = makeFixtureRoot();
+	try {
+		mkdirSync(join(fixtureRoot, "bin"), { recursive: true });
+		mkdirSync(join(fixtureRoot, "assets/install-wizard/index.html"), { recursive: true });
+		writeFileSync(join(fixtureRoot, "bin/gentle-shell-install.mjs"), "// entry\n");
+
+		assert.deepEqual(
+			missingRequiredPaths(fixtureRoot, ["bin/gentle-shell-install.mjs", "assets/install-wizard/index.html", "scripts/installer-server.mjs"]),
+			["assets/install-wizard/index.html", "scripts/installer-server.mjs"],
+		);
+	} finally {
+		rmSync(fixtureRoot, { recursive: true, force: true });
+	}
+});

package/tests/vim-editor-adapter-host-resolution.test.ts

@@ -0,0 +1,37 @@
+import assert from "node:assert/strict";
+import test from "node:test";
+import { spawnSync } from "node:child_process";
+import { copyFileSync, mkdirSync, mkdtempSync, rmSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { fileURLToPath, pathToFileURL } from "node:url";
+
+// A git install (`pi install git:...`) has no node_modules next to the
+// extension: Pi aliases ES imports of host packages, but nothing else
+// resolves them (#1586). Reproduce that layout with an ESM-only alias.
+test("vim adapter loads without an extension-local pi-tui and fails closed without a verified version", (t) => {
+  const root = mkdtempSync(join(tmpdir(), "gentle-vim-host-resolution-"));
+  t.after(() => rmSync(root, { recursive: true, force: true }));
+  mkdirSync(join(root, "lib"));
+  copyFileSync(fileURLToPath(new URL("../lib/vim-editor-adapter.ts", import.meta.url)), join(root, "lib", "vim-editor-adapter.ts"));
+  const tuiUrl = import.meta.resolve("@earendil-works/pi-tui");
+  writeFileSync(join(root, "alias-hooks.mjs"), `export async function resolve(specifier, context, next) {
+  if (specifier === "@earendil-works/pi-tui") return { url: ${JSON.stringify(tuiUrl)}, shortCircuit: true };
+  return next(specifier, context);
+}
+`);
+  writeFileSync(join(root, "register.mjs"), `import { register } from "node:module";
+register("./alias-hooks.mjs", import.meta.url);
+`);
+  writeFileSync(join(root, "probe.mjs"), `import { Editor } from "@earendil-works/pi-tui";
+const { createVimEditorAdapter } = await import("./lib/vim-editor-adapter.ts");
+const editor = new Editor({ terminal: { rows: 24 }, requestRender() {} }, { borderColor: (s) => s });
+let unverified = "admitted";
+try { createVimEditorAdapter(editor, "0.99.1"); } catch (error) { unverified = error.message; }
+const verified = typeof createVimEditorAdapter(editor, "0.99.1", Editor, "0.99.1").move;
+process.stdout.write(JSON.stringify({ unverified, verified }));
+`);
+  const result = spawnSync(process.execPath, ["--experimental-strip-types", "--import", pathToFileURL(join(root, "register.mjs")).href, join(root, "probe.mjs")], { cwd: root, encoding: "utf8" });
+  assert.equal(result.status, 0, result.stderr);
+  assert.deepEqual(JSON.parse(result.stdout), { unverified: "Unsupported Pi editor layout/version", verified: "function" });
+});