principles-disciple 1.72.0 → 1.74.0

package/INSTALL.md

@@ -236,9 +236,7 @@ After installation, enable the PLAN whitelist feature:
 }
 ```
 
-2. Ensure `docs/PLAN.md` has `STATUS: READY`
-
-3. Restart your agent session
+2. Restart your agent session
 
 Now even Stage 1 agents can edit files when a READY plan exists!
 

package/openclaw.plugin.json

@@ -2,7 +2,12 @@
   "id": "principles-disciple",
   "name": "Principles Disciple",
   "description": "Evolutionary programming agent framework with strategic guardrails and reflection loops.",
-  "version": "1.72.0",
+  "version": "1.74.0",
+  "activation": {
+    "onCapabilities": [
+      "hook"
+    ]
+  },
   "skills": [
     "templates/langs/en/skills",
     "templates/langs/zh/skills"
@@ -37,7 +42,7 @@
         },
         "default": [],
         "description": "自定义高危目录（例如 .git/, prod_db/）。AI 试图修改这些目录前，将被强制拦截并要求出具安全计划。"
-      },
+      }
     }
   },
   "uiHints": {
@@ -53,8 +58,8 @@
     }
   },
   "buildFingerprint": {
-    "gitSha": "0d6db62f8866",
-    "bundleMd5": "9cf803e170837b148578f78ff3c77dbd",
-    "builtAt": "2026-04-18T23:04:51.214Z"
+    "gitSha": "6a8d91ab50bb",
+    "bundleMd5": "0351a5287c1f3fbf74b3988e04a2541e",
+    "builtAt": "2026-05-20T11:40:04.688Z"
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "principles-disciple",
-  "version": "1.72.0",
+  "version": "1.74.0",
   "description": "Native OpenClaw plugin for Principles Disciple",
   "type": "module",
   "main": "./dist/bundle.js",
@@ -14,6 +14,9 @@
   },
   "author": "csuzngjh",
   "license": "MIT",
+  "publishConfig": {
+    "access": "public"
+  },
   "bugs": {
     "url": "https://github.com/csuzngjh/principles/issues"
   },
@@ -21,42 +24,40 @@
   "openclaw": {
     "id": "principles-disciple",
     "name": "Principles Disciple",
+    "setupEntry": "./dist/bundle.js",
     "extensions": [
       "./dist/bundle.js"
     ]
   },
   "scripts": {
     "build": "tsc",
-    "build:web": "node scripts/build-web.mjs",
-    "build:bundle": "node esbuild.config.js && node scripts/build-web.mjs",
-    "build:production": "node esbuild.config.js --production && node scripts/build-web.mjs --production && node scripts/verify-build.mjs",
+    "build:bundle": "node esbuild.config.js",
+    "build:production": "node esbuild.config.js --production && node scripts/verify-build.mjs",
     "test": "vitest run",
     "test:unit": "vitest run tests/core tests/service tests/hooks tests/commands tests/utils tests/scripts --exclude tests/commands/evolver.test.ts",
     "test:integration": "vitest run tests/integration/",
     "test:coverage": "vitest run --coverage",
     "test:all": "vitest run",
-    "lint": "eslint src/",
+    "lint": "eslint \"src/**/*.ts\"",
     "bootstrap-rules": "node scripts/bootstrap-rules.mjs",
     "compile-principles": "node scripts/compile-principles.mjs",
-    "validate-live-path": "tsx scripts/validate-live-path.ts"
+    "validate-live-path": "tsx scripts/validate-live-path.ts",
+    "sync-plugin": "node scripts/sync-plugin.mjs"
   },
   "devDependencies": {
-    "@testing-library/react": "^16.3.0",
     "@types/better-sqlite3": "^7.6.13",
     "@types/js-yaml": "^4.0.9",
     "@types/micromatch": "^4.0.10",
-    "@types/node": "^25.6.0",
-    "@types/react": "^19.2.2",
-    "@types/react-dom": "^19.2.2",
+    "@types/node": "^25.6.2",
     "@types/ws": "^8.5.13",
     "@typescript-eslint/eslint-plugin": "^8.58.2",
     "@typescript-eslint/parser": "^8.58.0",
-    "@vitest/coverage-v8": "^4.1.0",
+    "@vitest/coverage-v8": "^4.1.5",
     "esbuild": "^0.28.0",
     "eslint": "^10.2.1",
-    "jsdom": "^29.0.1",
+    "jsdom": "^29.1.1",
     "typescript": "^6.0.3",
-    "vitest": "^4.1.0",
+    "vitest": "^4.1.5",
     "ws": "^8.18.0"
   },
   "peerDependencies": {
@@ -68,14 +69,11 @@
     }
   },
   "dependencies": {
-    "@principles/core": "^0.1.0",
+    "@principles/core": "^1.73.0",
     "@sinclair/typebox": "^0.34.48",
     "better-sqlite3": "^12.9.0",
+    "commander": "^12.0.0",
     "js-yaml": "^4.1.1",
-    "lucide-react": "^1.7.0",
-    "micromatch": "^4.0.8",
-    "react": "^19.2.0",
-    "react-dom": "^19.2.0",
-    "react-router-dom": "^7.9.4"
+    "micromatch": "^4.0.8"
   }
 }

package/scripts/acceptance-test.mjs

@@ -114,48 +114,23 @@ function main() {
   assert(uniqueReasons.size === reasons.length, 'All pain reasons are unique', `${uniqueReasons.size}/${reasons.length} unique`);
 
   // ═══════════════════════════════════════════════
-  // SECTION 2: write_pain_flag Tool
+  // SECTION 2: PainSignalBridge + Runtime v2
   // ═══════════════════════════════════════════════
-  console.log('\n── 2. write_pain_flag Tool ──');
-
-  // 2.1 Test tool registered in source code
-  const indexSource = join(__dirname, '..', 'src', 'index.ts');
-  if (existsSync(indexSource)) {
-    const indexContent = readFileSync(indexSource, 'utf-8');
-    const hasImport = indexContent.includes("write-pain-flag");
-    const hasRegister = indexContent.includes('createWritePainFlagTool');
-    assert(hasImport && hasRegister, 'write_pain_flag registered in index.ts');
-  } else {
-    warn('write_pain_flag tool check', 'index.ts not found');
+  console.log('\n── 2. PainSignalBridge + Runtime v2 ──');
+
+  // 2.1 Verify PainSignalBridge is imported from runtime-v2 in pain.ts
+  const painHookSource = join(__dirname, '..', 'src', 'hooks', 'pain.ts');
+  if (existsSync(painHookSource)) {
+    const painContent = readFileSync(painHookSource, 'utf-8');
+    assert(painContent.includes('PainToPrincipleService'), 'pain.ts uses PainToPrincipleService from runtime-v2');
+    assert(!painContent.includes('createPainSignalBridge'), 'pain.ts no longer imports createPainSignalBridge');
   }
 
-  // 2.2 Verify tool source file exists
-  const toolSource = join(__dirname, '..', 'src', 'tools', 'write-pain-flag.ts');
-  assert(existsSync(toolSource), 'write-pain-flag.ts source exists');
-
-  // 2.3 Verify atomic write function
-  const toolContent = readFileSync(toolSource, 'utf-8');
-  assert(toolContent.includes('renameSync'), 'Uses atomic write (renameSync)', 'renameSync not found');
-
-  // 2.4 Verify KV serialization
-  assert(toolContent.includes('serializeKvLines'), 'Uses serializeKvLines for KV format', 'serializeKvLines not found');
-
-  // 2.5 Verify [object Object] protection
-  assert(toolContent.includes('buildPainFlag'), 'Uses buildPainFlag factory', 'buildPainFlag not found');
-
-  // 2.6 Test actual write end-to-end
-  const testFlagPath = join(stateDir, '.pain_flag_acceptance_test');
-  const testStateDir = join(stateDir, '.state_test_' + Date.now());
-  mkdirSync(testStateDir, { recursive: true });
-  try {
-    const testPath = join(testStateDir, '.pain_flag');
-    writeFileSync(testPath + '.tmp', 'source: manual\nscore: 80\ntime: 2026-04-13T00:00:00.000Z\nreason: acceptance test\n', 'utf-8');
-    execFileSync('mv', [testPath + '.tmp', testPath]);
-    const content = readFileSync(testPath, 'utf-8');
-    assert(content.includes('source: manual'), 'Manual pain flag write works');
-    assert(!content.includes('[object Object]'), 'No [object Object] corruption');
-  } finally {
-    rmSync(testStateDir, { recursive: true, force: true });
+  // 2.2 Verify write_pain_flag tool is NOT registered in index.ts
+  const indexSource2 = join(__dirname, '..', 'src', 'index.ts');
+  if (existsSync(indexSource2)) {
+    const indexContent = readFileSync(indexSource2, 'utf-8');
+    assert(!indexContent.includes('createWritePainFlagTool'), 'write_pain_flag NOT registered in index.ts');
   }
 
   // ═══════════════════════════════════════════════
@@ -214,45 +189,13 @@ function main() {
   // SECTION 5: Shell Injection Safety
   // ═══════════════════════════════════════════════
   console.log('\n── 5. Shell Injection Safety ──');
-
-  const seedScript = join(__dirname, '..', 'scripts', 'seed-nocturnal-scenarios.mjs');
-  const diagnoseScript = join(__dirname, '..', 'scripts', 'diagnose-nocturnal.mjs');
-
-  if (existsSync(seedScript)) {
-    const seedContent = readFileSync(seedScript, 'utf-8');
-    const execSyncCalls = (seedContent.match(/execSync\s*\(/g) || []).length;
-    const execFileCalls = (seedContent.match(/execFileSync\s*\(/g) || []).length;
-    assert(execSyncCalls === 0 || seedContent.includes("execSync('git") || seedContent.includes('execSync("git'), 
-           'Seed script: no sqlite3 in execSync', 
-           `found ${execSyncCalls} execSync calls`);
-    assert(execFileCalls > 0, 'Seed script: uses execFileSync', `found ${execFileCalls} calls`);
-    
-    // 5.1 Pre-flight check
-    assert(seedContent.includes('ensureSqlite3') || seedContent.includes('sqlite3 --version'), 'Seed script: sqlite3 pre-flight check');
-  }
-
-  if (existsSync(diagnoseScript)) {
-    const diagContent = readFileSync(diagnoseScript, 'utf-8');
-    // 5.2 Diagnose script uses execFileSync for sqlite3
-    const diagExecFileCalls = (diagContent.match(/execFileSync\s*\(\s*['"]sqlite3/g) || []).length;
-    assert(diagExecFileCalls > 0, 'Diagnose script: uses execFileSync for sqlite3', `found ${diagExecFileCalls} calls`);
-
-    // 5.3 No shell-interpolated sqlite3 calls
-    const shellSqliteCalls = (diagContent.match(/execSync\s*\(\s*['"]sqlite3/g) || []).length;
-    assert(shellSqliteCalls === 0, 'Diagnose script: no sqlite3 in execSync', `found ${shellSqliteCalls} unsafe calls`);
-  }
+  // (Nocturnal scripts removed in PRI-231 — section retained for future script checks)
 
   // ═══════════════════════════════════════════════
   // SECTION 6: Pending Review Warning Fix
   // ═══════════════════════════════════════════════
   console.log('\n── 6. Pending Review Warning Fix ──');
-
-  if (existsSync(diagnoseScript)) {
-    const diagContent = readFileSync(diagnoseScript, 'utf-8');
-    // 6.1 Pending case returns warn object
-    const pendingWarnPattern = /if\s*\(\s*pending\s*>\s*0\s*\)\s*\{[\s\S]*?status:\s*['"]warn['"]/;
-    assert(pendingWarnPattern.test(diagContent), 'Pending review returns {status:"warn"} object');
-  }
+  // (Nocturnal scripts removed in PRI-231 — section retained for future checks)
 
   // ═══════════════════════════════════════════════
   // SECTION 7: Path Resolver Fallback