principles-disciple 1.126.0 → 1.127.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.126.0",
+  "version": "1.127.0",
   "activation": {
     "onCapabilities": [
       "hook"

package/package.json

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

package/src/core/principle-compiler/code-validator.ts

@@ -44,7 +44,10 @@ export function validateGeneratedCode(code: string): ValidationResult {
   }
 
   // --- Check 2: Forbidden patterns (delegated to core) ---
-  const forbiddenLabels = checkForbiddenPatterns(code);
+  // PRI-439: checkForbiddenPatterns forbids `export` (canonical dialect is bare function).
+  // The openclaw-plugin legacy compiler still accepts `export` (the sandbox strips it),
+  // so we pass the normalized code (export keywords removed) to the checker.
+  const forbiddenLabels = checkForbiddenPatterns(normalized);
   for (const label of forbiddenLabels) {
     errors.push(`Forbidden pattern: ${label}`);
   }

package/tests/core/template-generator.test.ts

@@ -10,8 +10,8 @@ describe('generateFromTemplate', () => {
     const result = generateFromTemplate('P_066', 'Writing to secrets directory', patterns);
 
     expect(result).not.toBeNull();
-    expect(result).toContain("export const meta");
-    expect(result).toContain("export function evaluate(input)");
+    expect(result).toContain("const meta");
+    expect(result).toContain("function evaluate(input, helpers)");
     expect(result).toContain('name: "Auto_P_066"');
     expect(result).toContain('ruleId: "R_P_066_auto"');
     expect(result).toContain('sourcePrincipleId: "P_066"');
@@ -82,7 +82,7 @@ describe('generateFromTemplate', () => {
     expect(result).not.toBeNull();
 
     // Check meta structure
-    expect(result).toMatch(/export const meta = \{/);
+    expect(result).toMatch(/const meta = \{/);
     expect(result).toMatch(/name: "Auto_P_050"/);
     expect(result).toMatch(/version: '1\.0\.0'/);
     expect(result).toMatch(/ruleId: "R_P_050_auto"/);
@@ -90,10 +90,10 @@ describe('generateFromTemplate', () => {
     expect(result).toMatch(/compiledAt: "\d{4}-\d{2}-\d{2}T/);
 
     // Check evaluate function structure
-    expect(result).toMatch(/export function evaluate\(input\)/);
+    expect(result).toMatch(/function evaluate\(input, helpers\)/);
     expect(result).toMatch(/decision: 'block'/);
     expect(result).toMatch(/matched: true/);
-    expect(result).toMatch(/return \{ matched: false \}/);
+    expect(result).toMatch(/return \{ decision: 'allow', matched: false, reason: 'rule not applicable' \}/);
 
     // Check edit tool uses new_string fallback
     expect(result).toContain("input.action.paramsSummary.content || input.action.paramsSummary.new_string");