gentle-pi 0.10.4 → 0.10.6

package/extensions/pi-pretty.ts

@@ -0,0 +1,16 @@
+import { realpathSync } from "node:fs";
+import { createRequire } from "node:module";
+import { fileURLToPath } from "node:url";
+
+const packageJsonPath = realpathSync(
+	fileURLToPath(new URL("../package.json", import.meta.url)),
+);
+const requireFromRealPackage = createRequire(packageJsonPath);
+const piPrettyModule = requireFromRealPackage("@heyhuynhgiabuu/pi-pretty");
+
+const piPrettyExtension =
+	typeof piPrettyModule === "function"
+		? piPrettyModule
+		: piPrettyModule.default;
+
+export default piPrettyExtension;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gentle-pi",
-  "version": "0.10.4",
+  "version": "0.10.6",
   "description": "Turn Pi into el Gentleman: a senior-architect development harness with SDD/OpenSpec, subagents, strict TDD evidence, review guardrails, and skill discovery.",
   "license": "MIT",
   "type": "module",
@@ -37,8 +37,7 @@
   "pi": {
     "image": "https://cdn.jsdelivr.net/npm/gentle-pi/assets/gentle-logo-only.png",
     "extensions": [
-      "./extensions",
-      "./node_modules/@heyhuynhgiabuu/pi-pretty/dist/index.js"
+      "./extensions"
     ],
     "themes": [
       "./themes"

package/tests/package-manifest.test.ts

@@ -1,5 +1,5 @@
 import assert from "node:assert/strict";
-import { readFileSync } from "node:fs";
+import { existsSync, readFileSync } from "node:fs";
 import { dirname, join } from "node:path";
 import test from "node:test";
 import { fileURLToPath } from "node:url";
@@ -27,7 +27,7 @@ function readPackageJson(): PackageJson {
 	}
 }
 
-test("package manifest installs and loads pi-pretty without bundling native optional dependencies", () => {
+test("package manifest installs pi-pretty through a wrapper without bundling native optional dependencies", () => {
 	const packageJson = readPackageJson();
 
 	assert.equal(
@@ -36,10 +36,18 @@ test("package manifest installs and loads pi-pretty without bundling native opti
 		"gentle-pi must install the tested pi-pretty version as a normal dependency",
 	);
 	assert.ok(
-		packageJson.pi?.extensions?.includes(
+		packageJson.pi?.extensions?.includes("./extensions"),
+		"gentle-pi must load packaged extension wrappers",
+	);
+	assert.ok(
+		!packageJson.pi?.extensions?.includes(
 			"./node_modules/@heyhuynhgiabuu/pi-pretty/dist/index.js",
 		),
-		"gentle-pi must load the pi-pretty extension from its installed dependency",
+		"gentle-pi must not reference pnpm-unportable nested node_modules paths",
+	);
+	assert.ok(
+		existsSync(join(PACKAGE_ROOT, "extensions", "pi-pretty.ts")),
+		"gentle-pi must expose pi-pretty through a packaged wrapper extension",
 	);
 	assert.ok(
 		!packageJson.bundledDependencies?.includes("@heyhuynhgiabuu/pi-pretty"),
@@ -50,3 +58,15 @@ test("package manifest installs and loads pi-pretty without bundling native opti
 		"pi-pretty must not be bundled because its native optional dependencies are platform-specific",
 	);
 });
+
+
+test("pi-pretty wrapper uses real package path resolution for pnpm symlink installs", () => {
+	const wrapper = readFileSync(
+		join(PACKAGE_ROOT, "extensions", "pi-pretty.ts"),
+		"utf8",
+	);
+
+	assert.match(wrapper, /realpathSync/);
+	assert.match(wrapper, /createRequire/);
+	assert.match(wrapper, /@heyhuynhgiabuu\/pi-pretty/);
+});