principles-disciple 1.141.0 → 1.143.0

package/openclaw.plugin.json

@@ -2,7 +2,7 @@
   "id": "principles-disciple",
   "name": "Principles Disciple",
   "description": "Evolutionary programming agent framework with strategic guardrails and reflection loops.",
-  "version": "1.141.0",
+  "version": "1.143.0",
   "activation": {
     "onCapabilities": [
       "hook"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "principles-disciple",
-  "version": "1.141.0",
+  "version": "1.143.0",
   "description": "Native OpenClaw plugin for Principles Disciple",
   "type": "module",
   "main": "./dist/bundle.js",

package/scripts/sync-plugin.mjs

@@ -871,9 +871,9 @@ function copyDir(src, dest) {
 }
 
 /**
- * Inject local workspace packages (monorepo) into node_modules before npm install.
- * @principles/core is a workspace package, not published to npm — we must copy it
- * from the monorepo's node_modules so npm install --production doesn't 404 it.
+ * Inject local workspace packages (monorepo) into node_modules after npm install.
+ * @principles/core from the local monorepo is authoritative because the
+ * npm-published package line lacks exports required by the current pd-cli.
  */
 function injectLocalWorkspacePackages() {
     const monorepoModules = join(SOURCE_DIR, '..', '..', 'node_modules', '@principles', 'core');
@@ -885,24 +885,25 @@ function injectLocalWorkspacePackages() {
     }
 
     console.log('  📦 Injecting local workspace packages (@principles/core)...');
+
+    // Remove the target FIRST, because cp -rL on Windows copies the source
+    // directory INTO the existing target, creating a nested core/core/ that
+    // leaves the original npm-version package.json untouched. We must start
+    // with a clean slate so the local version is authoritative.
+    if (existsSync(targetModules)) {
+        rmSync(targetModules, { recursive: true, force: true });
+    }
+
     mkdirSync(dirname(targetModules), { recursive: true });
-    // cpSync creates symlinks on Windows for symlinked dirs — use cp -rL (dereference) via exec
-    let injected = false;
+    // Use copyDir (Node-based, no cp -rL semantics trap) for reliable
+    // cross-platform overwrite. cpSync creates symlinks on Windows for
+    // symlinked dirs — copyDir dereferences by reading file contents.
     try {
-        execSync(`cp -rL "${monorepoModules}" "${targetModules}"`, { stdio: 'ignore' });
-        injected = true;
-    } catch {
-        // Fallback: manual copy via node (Windows-compatible)
-        try {
-            copyDir(monorepoModules, targetModules);
-            injected = true;
-        } catch (copyErr) {
-            console.warn('  ⚠️ Failed to inject @principles/core from monorepo: ' + copyErr.message);
-            console.warn('  ⚠️ npm install --production may fail if @principles/core is not published');
-        }
-    }
-    if (injected && !existsSync(targetModules)) {
-        console.warn('  ⚠️ Injection reported success but target not found: ' + targetModules);
+        copyDir(monorepoModules, targetModules);
+        console.log('    ✅ @principles/core local build injected');
+    } catch (copyErr) {
+        console.warn('  ⚠️ Failed to inject @principles/core from monorepo: ' + copyErr.message);
+        console.warn('  ⚠️ npm version remains in place — pd-cli may fail if exports differ');
     }
 }
 
@@ -1082,7 +1083,7 @@ function restartGatewayWindows() {
                         execSync(`wmic process where "ProcessId=${pidTrim}" call terminate`, { stdio: 'ignore' });
                     } catch { /* ignore individual failures */ }
                 }
-                execSync('timeout /t 3 /nobreak > nul', { shell: true, stdio: 'ignore' });
+                execSync('timeout /t 3 /nobreak', { shell: true, stdio: 'ignore' });
             }
         } catch { /* no existing processes */ }
 
@@ -1146,7 +1147,7 @@ function restartGatewayWindows() {
                 }
             } catch { /* port not listening yet */ }
             try {
-                execSync('timeout /t 1 /nobreak > nul', { shell: true, stdio: 'ignore' });
+                execSync('timeout /t 1 /nobreak', { shell: true, stdio: 'ignore' });
             } catch { /* ignore */ }
         }
 
@@ -1309,8 +1310,18 @@ function main() {
     syncFilteredManifest(args.lang);
     syncPdCli();
 
-    injectLocalWorkspacePackages();
+    // IMPORTANT: npm install must run FIRST, then injectLocalWorkspacePackages
+    // overwrites @principles/core with the local monorepo version.
+    //
+    // Why: @principles/core on npm (v1.164.0-1.174.0, from a different publish
+    // line) is semantically "^1.74.1"-compatible but its exports field does NOT
+    // include ./principle-tree-ledger that pd-cli/health.js requires. The npm
+    // registry version must never be used — only the local monorepo build has
+    // the correct exports. Injecting BEFORE npm install is useless because npm
+    // overwrites the injected copy. The authoritative source is always the
+    // local monorepo, applied AFTER npm install.
     installTargetDependencies();
+    injectLocalWorkspacePackages();
     verifyInjectedWorkspaceDeps();
     verifyPdCliShim();