principles-disciple 1.34.0 → 1.34.1

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.34.0",
+  "version": "1.34.1",
   "skills": [
     "./skills"
   ],

package/package.json

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

package/src/core/correction-cue-learner.ts

@@ -13,13 +13,15 @@
 
 import * as fs from 'fs';
 import * as path from 'path';
-import {
+import type {
   CorrectionKeyword,
   CorrectionKeywordStore,
-  CorrectionMatchResult,
+  CorrectionMatchResult} from './correction-types.js';
+import {
   CORRECTION_SEED_KEYWORDS,
   MAX_CORRECTION_KEYWORDS,
 } from './correction-types.js';
+import { checkCooldown, recordCooldown } from '../service/nocturnal-runtime.js';
 
 const KEYWORD_STORE_FILE = 'correction_keywords.json';
 
@@ -128,8 +130,8 @@ export function _resetCorrectionCueLearnerInstance(): void {
 // =========================================================================
 
 export class CorrectionCueLearner {
-  private store: CorrectionKeywordStore;
-  private stateDir: string;
+  private readonly store: CorrectionKeywordStore;
+  private readonly stateDir: string;
 
   constructor(stateDir: string) {
     this.stateDir = stateDir;
@@ -142,7 +144,7 @@ export class CorrectionCueLearner {
    * Checks whether text contains a correction cue (D-11).
    * Normalisation is equivalent to the original detectCorrectionCue():
    *   trim → lowercase → strip punctuation
-   * Returns the first matched term only (first-match semantics).
+   * Returns weighted score based on keyword accuracy (D-39-03, D-39-04).
    */
   match(text: string): CorrectionMatchResult {
     const normalized = text
@@ -150,13 +152,98 @@ export class CorrectionCueLearner {
       .toLowerCase()
       .replace(/[.,!?;:，。！？；：]/g, '');
 
+    const matchedTerms: string[] = [];
+    let totalScore = 0;
+
     for (const keyword of this.store.keywords) {
       if (normalized.includes(keyword.term.toLowerCase())) {
-        return { matched: true, matchedTerms: [keyword.term], score: keyword.weight, confidence: 0.9 };
+        // D-39-03, D-39-04: Weighted score formula
+        // score = weight x ((TP + 1) / (TP + FP + 2))
+        // +2 smoothing: new keywords (TP=0, FP=0) get accuracy=0.5
+        const tp = keyword.truePositiveCount ?? 0;
+        const fp = keyword.falsePositiveCount ?? 0;
+        const accuracy = (tp + 1) / (tp + fp + 2);
+        const score = keyword.weight * accuracy;
+
+        totalScore += score;
+        matchedTerms.push(keyword.term);
+
+        // Increment hitCount
+        keyword.hitCount = (keyword.hitCount ?? 0) + 1;
+        keyword.lastHitAt = new Date().toISOString();
       }
     }
 
-    return { matched: false, matchedTerms: [], score: 0.0, confidence: 0.0 };
+    const cappedScore = Math.min(1, totalScore);
+    const isMatched = matchedTerms.length > 0;
+
+    // D-39-04: Confidence derived from multiple signals
+    const termConfidence = Math.min(1, matchedTerms.length / 3);
+    const scoreConfidence = Math.min(1, cappedScore / 0.8);
+    const confidence = Math.max(termConfidence, scoreConfidence);
+
+    return {
+      matched: isMatched,
+      matchedTerms: matchedTerms.slice(0, 5),
+      score: cappedScore,
+      confidence,
+    };
+  }
+
+  /**
+   * Records a confirmed true positive for the given keyword term.
+   * Increments both hitCount and truePositiveCount.
+   */
+  recordTruePositive(term: string): void {
+    const keyword = this.store.keywords.find(k => k.term.toLowerCase() === term.toLowerCase());
+    if (!keyword) return;
+
+    keyword.truePositiveCount = (keyword.truePositiveCount ?? 0) + 1;
+    keyword.hitCount = (keyword.hitCount ?? 0) + 1;
+    keyword.lastHitAt = new Date().toISOString();
+
+    this.flush();
+  }
+
+  /**
+   * Records a confirmed false positive for the given keyword term.
+   * CORR-10: Decreases keyword weight by 20% (x0.8 multiplicative factor).
+   */
+  recordFalsePositive(term: string): void {
+    const keyword = this.store.keywords.find(k => k.term.toLowerCase() === term.toLowerCase());
+    if (!keyword) return;
+
+    keyword.falsePositiveCount = (keyword.falsePositiveCount ?? 0) + 1;
+    keyword.hitCount = (keyword.hitCount ?? 0) + 1;
+
+    // D-39-15: Multiplicative weight decay x0.8 on confirmed FP
+    keyword.weight = Math.max(0.1, keyword.weight * 0.8);
+    keyword.lastHitAt = new Date().toISOString();
+
+    this.flush();
+  }
+
+  /**
+   * Returns true if optimization is allowed (within daily throttle limit).
+   * CORR-08: Max 4 optimizations per day across all triggers.
+   */
+  canRunKeywordOptimization(): boolean {
+    // D-39-12, D-39-13: Per-workspace throttle, 4 calls/day
+    const cooldown = checkCooldown(this.stateDir, 'keyword_optimization', {
+      maxRunsPerWindow: 4,
+      quotaWindowMs: 24 * 60 * 60 * 1000,
+    });
+    return !cooldown.globalCooldownActive && !cooldown.quotaExhausted;
+  }
+
+  /**
+   * Records that an optimization was performed.
+   * Increments the daily throttle counter and updates lastOptimizedAt.
+   */
+  async recordOptimizationPerformed(): Promise<void> {
+    await recordCooldown(this.stateDir, 24 * 60 * 60 * 1000);
+    this.store.lastOptimizedAt = new Date().toISOString();
+    this.flush();
   }
 
   /**
@@ -177,11 +264,48 @@ export class CorrectionCueLearner {
     this.flush();
   }
 
+  /**
+   * Updates the weight of an existing keyword.
+   * Weight is clamped to 0.1-0.9 range.
+   * Throws if keyword not found.
+   */
+  updateWeight(term: string, weight: number): void {
+    const idx = this.store.keywords.findIndex(
+      k => k.term.toLowerCase() === term.toLowerCase()
+    );
+    if (idx < 0) {
+      throw new Error(`Keyword not found: ${term}`);
+    }
+
+    this.store.keywords[idx].weight = Math.max(0.1, Math.min(0.9, weight));
+    this.flush();
+  }
+
+  /**
+   * Removes a keyword from the store by term.
+   * Throws if keyword not found.
+   */
+  remove(term: string): void {
+    const idx = this.store.keywords.findIndex(
+      k => k.term.toLowerCase() === term.toLowerCase()
+    );
+    if (idx < 0) {
+      throw new Error(`Keyword not found: ${term}`);
+    }
+    this.store.keywords.splice(idx, 1);
+    this.flush();
+  }
+
   /** Returns a reference to the in-memory store. */
   getStore(): CorrectionKeywordStore {
     return this.store;
   }
 
+  /** Returns the lastOptimizedAt timestamp. */
+  getLastOptimizedAt(): string {
+    return this.store.lastOptimizedAt;
+  }
+
   /** Persists the current in-memory store to disk atomically. */
   flush(): void {
     saveCorrectionKeywordStore(this.stateDir, this.store);

package/src/core/trajectory-types.ts

@@ -117,7 +117,7 @@ export interface TrajectorySessionInput {
 }
 
 // V2: Task kind and priority types for queue schema
-export type TaskKind = 'pain_diagnosis' | 'sleep_reflection' | 'model_eval';
+export type TaskKind = 'pain_diagnosis' | 'sleep_reflection' | 'model_eval' | 'keyword_optimization';
 export type TaskPriority = 'high' | 'medium' | 'low';
 
 // V2: EvolutionTaskInput with all V2 fields

package/src/service/correction-observer-types.ts

@@ -29,6 +29,17 @@ export interface CorrectionObserverPayload {
   };
   /** Recent user messages for pattern analysis */
   recentMessages: string[];
+
+  /**
+   * Trajectory history: user turns where correctionDetected=true (D-40-08).
+   * Includes term matched, timestamp, sessionId for FPR trend analysis.
+   */
+  trajectoryHistory: Array<{
+    sessionId: string;
+    timestamp: string;
+    term: string;
+    userMessage: string;
+  }>;
 }
 
 /**

package/src/service/correction-observer-workflow-manager.ts

@@ -21,7 +21,6 @@ import { isSubagentRuntimeAvailable } from '../utils/subagent-probe.js';
 import type {
     CorrectionObserverPayload,
     CorrectionObserverResult,
-    CorrectionObserverWorkflowSpec,
 } from './correction-observer-types.js';
 
 const WORKFLOW_SESSION_PREFIX = 'agent:main:subagent:workflow-correction-';
@@ -92,7 +91,7 @@ export const correctionObserverWorkflowSpec: SubagentWorkflowSpec<CorrectionObse
 
     buildPrompt(taskInput: unknown, _metadata: WorkflowMetadata): string {
         const payload = taskInput as CorrectionObserverPayload;
-        const { keywordStoreSummary, recentMessages } = payload;
+        const { keywordStoreSummary, recentMessages, trajectoryHistory } = payload;
 
         const termsList = keywordStoreSummary.terms
             .map(t => `  - term="${t.term}", weight=${t.weight}, hits=${t.hitCount}, TP=${t.truePositiveCount}, FP=${t.falsePositiveCount}`)
@@ -102,6 +101,11 @@ export const correctionObserverWorkflowSpec: SubagentWorkflowSpec<CorrectionObse
             ? recentMessages.map(m => `  - ${JSON.stringify(m)}`).join('\n')
             : '  (none)';
 
+        const trajectory = trajectoryHistory.length > 0
+            ? trajectoryHistory.map(t => `  - [${t.sessionId}] ${t.term} (${t.timestamp}): ${t.userMessage.substring(0, 80)}`)
+              .join('\n')
+            : '  (none)';
+
         return [
             'You are a correction keyword optimizer.',
             '',
@@ -115,6 +119,9 @@ export const correctionObserverWorkflowSpec: SubagentWorkflowSpec<CorrectionObse
             '## Recent User Messages (' + recentMessages.length + ' messages):',
             messages,
             '',
+            '## Correction Trajectory (recent confirmed corrections, D-40-08):',
+            trajectory,
+            '',
             '## Rules:',
             '- ADD: If a correction pattern is detected in messages but not in store',
             '- UPDATE: If a term\'s weight should change based on TP/FP ratio',
@@ -187,7 +194,7 @@ export class CorrectionObserverWorkflowManager extends WorkflowManagerBase {
         return super.startWorkflow(spec, options);
     }
 
-    // eslint-disable-next-line @typescript-eslint/class-methods-use-this
+     
     protected override createWorkflowMetadata<TResult>(
         spec: SubagentWorkflowSpec<TResult>,
         options: {
@@ -207,6 +214,28 @@ export class CorrectionObserverWorkflowManager extends WorkflowManagerBase {
             ...options.metadata,
         };
     }
+
+    /**
+     * Retrieves and parses the workflow result for a completed workflow.
+     * Called by evolution-worker.ts after getWorkflowDebugSummary reports state=completed.
+     */
+    async getWorkflowResult(workflowId: string): Promise<CorrectionObserverResult | null> {
+        const workflow = this.store.getWorkflow(workflowId);
+        if (!workflow) return null;
+
+        const result = await this.driver.getResult({
+            sessionKey: workflow.child_session_key,
+            limit: 20,
+        });
+
+        const metadata = JSON.parse(workflow.metadata_json) as WorkflowMetadata;
+        return correctionObserverWorkflowSpec.parseResult({
+            messages: result.messages,
+            assistantTexts: result.assistantTexts,
+            metadata,
+            waitStatus: 'ok',
+        });
+    }
 }
 
 // ── Factory ─────────────────────────────────────────────────────────────────