principles-disciple 1.43.0 → 1.45.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.43.0",
+  "version": "1.45.0",
   "skills": [
     "./skills"
   ],

package/package.json

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

package/src/commands/capabilities.ts

@@ -14,7 +14,7 @@ const TOOLS_TO_SCAN = [
   { name: 'shellcheck', cmd: ['shellcheck', '--version'] },
 ];
 
-// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Reason: third-party API execSync returns dynamic output - type structure unknown at call site
+ 
 function scanEnvironment(wctx: WorkspaceContext): any {
   const tools: Record<string, { available: boolean; version?: string }> = {};
 

package/src/commands/context.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import * as fs from 'fs';
 import * as path from 'path';
 import type { PluginCommandContext, PluginCommandResult } from '../openclaw-sdk.js';

package/src/commands/export.ts

@@ -46,12 +46,12 @@ export function handleExportCommand(ctx: PluginCommandContext): PluginCommandRes
        
       return {
         text: zh
-          ? `已导出 ORPO 决策点样本到 ${result.manifest!.exportPath}，` + // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: caller guarantees manifest exists via success check
-              `共 ${result.manifest!.sampleCount} 条，模型家族: ${result.manifest!.targetModelFamily}，` + // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: caller guarantees manifest exists via success check
-              `数据集指纹: ${result.manifest!.datasetFingerprint.substring(0, 16)}...` // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: caller guarantees manifest exists via success check
-          : `Exported ORPO decision-point samples to ${result.manifest!.exportPath}, ` + // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: caller guarantees manifest exists via success check
-              `${result.manifest!.sampleCount} samples, target: ${result.manifest!.targetModelFamily}, ` + // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: caller guarantees manifest exists via success check
-              `dataset fingerprint: ${result.manifest!.datasetFingerprint.substring(0, 16)}...`, // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: caller guarantees manifest exists via success check
+          ? `已导出 ORPO 决策点样本到 ${result.manifest!.exportPath}，` +  
+              `共 ${result.manifest!.sampleCount} 条，模型家族: ${result.manifest!.targetModelFamily}，` +  
+              `数据集指纹: ${result.manifest!.datasetFingerprint.substring(0, 16)}...`  
+          : `Exported ORPO decision-point samples to ${result.manifest!.exportPath}, ` +  
+              `${result.manifest!.sampleCount} samples, target: ${result.manifest!.targetModelFamily}, ` +  
+              `dataset fingerprint: ${result.manifest!.datasetFingerprint.substring(0, 16)}...`,  
       };
     }
 

package/src/commands/nocturnal-train.ts

@@ -538,7 +538,7 @@ Next steps:
         }
       }
 
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Reason: JSON.parse returns dynamic JSON - type unknown at parse time, narrowed via type narrowing below
+       
       let result: any;
       try {
         result = JSON.parse(resultJson);

package/src/commands/thinking-os.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import * as fs from 'fs';
 import type { PluginCommandContext, PluginCommandResult } from '../openclaw-sdk.js';
 import { WorkspaceContext } from '../core/workspace-context.js';

package/src/core/adaptive-thresholds.ts

@@ -441,7 +441,7 @@ export function adjustThresholdsFromSignals(
       currentThresholds.principleAlignmentMin + adjustment,
       `High arbiter reject rate (${signals.arbiterRejectRate.toFixed(2)}) → tightening alignment threshold`
     );
-    if (result.changed && (!bestResult.changed || ((result.newValue! - result.oldValue!) > 0))) { // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: changed flag guarantees newValue/oldValue are defined
+    if (result.changed && (!bestResult.changed || ((result.newValue! - result.oldValue!) > 0))) {  
       bestResult = result;
     }
   }
@@ -455,7 +455,7 @@ export function adjustThresholdsFromSignals(
       currentThresholds.executabilityMin + adjustment,
       `High executability reject rate (${signals.executabilityRejectRate.toFixed(2)}) → tightening executability threshold`
     );
-    if (result.changed && (!bestResult.changed || ((result.newValue! - result.oldValue!) > 0))) { // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: changed flag guarantees newValue/oldValue are defined
+    if (result.changed && (!bestResult.changed || ((result.newValue! - result.oldValue!) > 0))) {  
       bestResult = result;
     }
   }
@@ -469,7 +469,7 @@ export function adjustThresholdsFromSignals(
       Math.max(currentThresholds.aggregateMin - reward, THRESHOLD_MIN),
       `Positive quality delta (${signals.qualityDelta.toFixed(2)}) → rewarding with slightly lower aggregate threshold`
     );
-    if (result.changed && (!bestResult.changed || ((result.oldValue! - result.newValue!) > 0))) { // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: changed flag guarantees newValue/oldValue are defined
+    if (result.changed && (!bestResult.changed || ((result.oldValue! - result.newValue!) > 0))) {  
       bestResult = result;
     }
   }

package/src/core/config.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import * as fs from 'fs';
 import * as path from 'path';
 import { atomicWriteFileSync } from '../utils/io.js';
@@ -266,7 +266,7 @@ export class PainConfig {
         }
     }
 
-    /* eslint-disable @typescript-eslint/no-explicit-any */
+     
     // Reason: deepMerge handles arbitrary nested object structures where static typing cannot precisely capture recursive object shapes
     private deepMerge(target: any, source: any): any {
         const output = { ...target };

package/src/core/dictionary.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import * as fs from 'fs';
 import * as path from 'path';
 import { atomicWriteFileSync } from '../utils/io.js';

package/src/core/evolution-engine.ts

@@ -557,7 +557,7 @@ export function getEvolutionEngine(workspaceDir: string): EvolutionEngine {
   if (!_instances.has(resolved)) {
     _instances.set(resolved, new EvolutionEngine(resolved));
   }
-  // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: set() above guarantees get() returns non-null
+   
   return _instances.get(resolved)!;
 }
 

package/src/core/nocturnal-compliance.ts

@@ -384,7 +384,7 @@ function detectT05Opportunity(session: SessionEvents): OpportunityMatch {
     if (RISKY_TOOLS.has(call.toolName)) return true;
     // Check bash for dangerous patterns
     if (call.toolName === 'bash' && call.errorMessage) {
-      // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: call.errorMessage guard above ensures truthiness
+       
       return DANGEROUS_BASH_PATTERNS.some((p) => p.test(call.errorMessage!));
     }
     return false;
@@ -432,7 +432,7 @@ function detectT06Opportunity(session: SessionEvents): OpportunityMatch {
 function detectT07Opportunity(session: SessionEvents): OpportunityMatch {
   const filePaths = session.toolCalls
     .filter((call) => call.filePath !== undefined)
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: filter ensures filePath is defined
+     
     .map((call) => normalizePathPosix(call.filePath!));
   const uniqueFiles = new Set(filePaths);
   if (uniqueFiles.size >= 3) {
@@ -478,7 +478,7 @@ function detectT09Opportunity(session: SessionEvents): OpportunityMatch {
   const uniqueFiles = new Set(
     session.toolCalls
       .filter((call) => call.filePath !== undefined)
-      // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: filter ensures filePath is defined
+       
       .map((call) => normalizePathPosix(call.filePath!))
   );
   const hasComplexity = toolCallCount >= 5 || uniqueFiles.size >= 3;
@@ -592,7 +592,7 @@ function detectT01Violation(session: SessionEvents): ViolationMatch {
   const readFiles = new Set(
     session.toolCalls
       .filter((call) => READ_TOOLS.has(call.toolName) && call.filePath !== undefined)
-      // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: filter ensures filePath is defined
+       
       .map((call) => normalizePathPosix(call.filePath!))
   );
 
@@ -786,7 +786,7 @@ function detectT07Violation(session: SessionEvents): ViolationMatch {
   const modifiedFiles = new Set(
     session.toolCalls
       .filter((call) => EDIT_TOOLS.has(call.toolName) && call.filePath !== undefined)
-      // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: filter ensures filePath is defined
+       
       .map((call) => normalizePathPosix(call.filePath!))
   );
 
@@ -843,7 +843,7 @@ function detectT09Violation(session: SessionEvents): ViolationMatch {
   const uniqueFiles = new Set(
     session.toolCalls
       .filter((call) => call.filePath !== undefined)
-      // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: filter ensures filePath is defined
+       
       .map((call) => normalizePathPosix(call.filePath!))
   );
 
@@ -1086,7 +1086,7 @@ export function groupEventsIntoSessions(events: RawEventEntry[]): Map<string, Se
       });
     }
 
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: set() above guarantees get() returns non-null
+     
     const session = sessionMap.get(sessionId)!;
 
     switch (event.type) {

package/src/core/path-resolver.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import * as path from 'path';
 import * as os from 'os';
 import * as fs from 'fs';

package/src/core/pd-task-reconciler.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import * as fs from 'fs';
 import * as path from 'path';
 import * as os from 'os';

package/src/core/pd-task-store.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import * as fs from 'fs';
 import * as path from 'path';
 import type { PDTaskSpec } from './pd-task-types.js';

package/src/core/profile.ts

@@ -57,7 +57,7 @@ export const PROFILE_DEFAULTS = {
   custom_guards: [] as { pattern: string; message: string; severity: string }[],
 };
 
-/* eslint-disable @typescript-eslint/no-explicit-any */
+ 
 // Reason: normalizeProfile handles arbitrary JSON profile shapes where static typing cannot capture runtime field existence
 export function normalizeProfile(rawProfile: any): any {
   const defaults = JSON.parse(JSON.stringify(PROFILE_DEFAULTS));

package/src/core/promotion-gate.ts

@@ -323,8 +323,6 @@ export function evaluatePromotionGate(
 ): PromotionGateResult {
   const {
     checkpointId,
-     
-    targetProfile: _targetProfile,
     baselineMetrics,
     minDelta = DEFAULT_MIN_DELTA,
     allowedMargin = DEFAULT_ALLOWED_MARGIN,

package/src/core/replay-engine.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import * as fs from 'fs';
 import * as path from 'path';
 import { withLock } from '../utils/file-lock.js';

package/src/core/session-tracker.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import * as path from 'path';
 import * as fs from 'fs';
 import { atomicWriteFileSync } from '../utils/io.js';

package/src/core/trajectory.ts

@@ -209,10 +209,6 @@ export class TrajectoryDatabase {
   recordToolCall(input: TrajectoryToolCallInput): number {
     this.recordSession({ sessionId: input.sessionId, startedAt: input.createdAt });
     const createdAt = input.createdAt ?? nowIso();
-    // Extract filePath from paramsJson if provided and is an object with filePath
-    const paramsObj = input.paramsJson as Record<string, unknown> | undefined;
-     
-    const _filePath = paramsObj && typeof paramsObj.filePath === 'string' ? paramsObj.filePath : null;
     const rowId = this.withWrite(() => {
       const result = this.db.prepare(`
         INSERT INTO tool_calls (

package/src/core/workspace-context.ts

@@ -173,7 +173,7 @@ export class WorkspaceContext {
      * Uses PathResolver to handle path normalization and fallback logic.
      * @throws Error if workspaceDir is missing and no fallback available.
      */
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Reason: OpenClaw plugin framework hook context has dynamic shape - type not available
+     
     static fromHookContext(ctx: any): WorkspaceContext {
         const {logger} = ctx;
         const log = (msg: string) => logger?.info?.(msg);

package/src/hooks/edit-verification.ts

@@ -131,7 +131,7 @@ This is enforced by P-03 (精确匹配前验证原则).`;
 export function handleEditVerification(
   event: PluginHookBeforeToolCallEvent,
   wctx: WorkspaceContext,
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Reason: logger is typed as any by plugin framework - type not available
+   
   ctx: { logger?: any; sessionId?: string },
   config: EditVerificationConfig = {}
 ): PluginHookBeforeToolCallResult | void {

package/src/hooks/prompt.ts

@@ -402,9 +402,10 @@ export async function handleBeforePromptBuild(
             learner.flush();
           }
         }
-      } catch {
-        // Fallback to hardcoded detection if learner fails
+      } catch (learnerErr) {
+        // Fallback to hardcoded detection if learner fails — log for observability
         correctionCue = detectCorrectionCue(userText);
+        logger?.warn?.(`[PD:Prompt] CorrectionCueLearner.match() failed (${String(learnerErr)}), fallback=${correctionCue ? `matched="${correctionCue}"` : 'no-match'}`);
       }
       let referencesAssistantTurnId: number | null = null;
       const hasPriorAssistant = event.messages
@@ -981,12 +982,8 @@ ${taskBlocks}${processingNote}
         const filePattern = /\b([a-zA-Z]:\\?[^\s,]+\.[a-z]{2,10}|[./][^\s,]+\.[a-z]{2,10})\b/gi;
         const toolMatches = toolPatterns.flatMap(({ pattern, tool }) => {
           const matches: string[] = [];
-           
-           
-          let _m;
           const r = new RegExp(pattern.source, pattern.flags);
-           
-          while ((_m = r.exec(latestUserText)) !== null) matches.push(tool);
+          while (r.exec(latestUserText) !== null) matches.push(tool);
           return matches;
         });
         const fileMatches = latestUserText.match(filePattern) ?? [];

package/src/index.ts

@@ -283,7 +283,7 @@ const plugin = {
 
           if (shouldRecordShadow) {
             const observation = recordShadowRouting(workspaceDir, {
-              checkpointId: decision.activeCheckpointId!, // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: !!decision.activeCheckpointId guard above ensures truthiness
+              checkpointId: decision.activeCheckpointId!,  
               workerProfile: agentId as WorkerProfile,
               taskFingerprint: computeRuntimeShadowTaskFingerprint(event),
             });
@@ -362,7 +362,7 @@ const plugin = {
 
     // ── Slash Commands ──
     // Register command with optional short alias
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+     
     const registerCommandWithAlias = (name: string, alias: string | null, desc: string, handler: any, opts?: { acceptsArgs?: boolean }) => {
       const base = {
         name,
@@ -380,17 +380,17 @@ const plugin = {
       }
     };
 
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+     
     registerCommandWithAlias('pd-init', 'pdi', getCommandDescription('pd-init', language), (ctx: any) => handleInitStrategy(ctx));
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+     
     registerCommandWithAlias('pd-okr', 'pdk', getCommandDescription('pd-okr', language), (ctx: any) => handleManageOkr(ctx));
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+     
     registerCommandWithAlias('pd-bootstrap', 'pdb', getCommandDescription('pd-bootstrap', language), (ctx: any) => handleBootstrapTools(ctx));
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+     
     registerCommandWithAlias('pd-research', 'pdr', getCommandDescription('pd-research', language), (ctx: any) => handleResearchTools(ctx));
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+     
     registerCommandWithAlias('pd-thinking', 'pdt', getCommandDescription('pd-thinking', language), (ctx: any) => handleThinkingOs(ctx), { acceptsArgs: true });
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+     
     registerCommandWithAlias('pd-reflect', 'pdrl', getCommandDescription('pd-reflect', language), (ctx: any) => {
       try {
         // Resolve agentId from sessionKey (if available), fallback to 'main'

package/src/service/central-health-service.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import { getCentralDatabase } from './central-database.js';
 import { HealthQueryService } from './health-query-service.js';
 

package/src/service/central-overview-service.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import { getCentralDatabase, type CentralDatabase } from './central-database.js';
 import { getThinkingModelDefinitions } from '../core/thinking-models.js';
 import type { OverviewResponse } from './control-ui-query-service.js';

package/src/service/evolution-query-service.ts

@@ -349,13 +349,13 @@ export class EvolutionQueryService {
     for (const task of recentTasks) {
       const [createdDay] = task.createdAt.split('T');
       if (activityByDay.has(createdDay)) {
-        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: has() check guarantees entry exists
+         
         activityByDay.get(createdDay)!.created++;
       }
       if (task.completedAt) {
         const [completedDay] = task.completedAt.split('T');
         if (activityByDay.has(completedDay)) {
-          // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: has() check guarantees entry exists
+           
           activityByDay.get(completedDay)!.completed++;
         }
       }

package/src/service/evolution-worker.ts

@@ -233,8 +233,6 @@ function buildFallbackNocturnalSnapshot(
     };
 }
 
-const PAIN_QUEUE_DEDUP_WINDOW_MS = 30 * 60 * 1000;
-
 // Queue lock constants and requireQueueLock are imported from queue-io.ts
 
 export function extractEvolutionTaskId(task: string): string | null {
@@ -284,13 +282,6 @@ export function purgeStaleFailedTasks(
     return { purged: purged.length, remaining: queue.length, byReason };
 }
 
-function normalizePainDedupKey(source: string, preview: string, reason?: string): string {
-    // Include reason in dedup key to match createEvolutionTaskId() behavior
-    // Different reasons for the same source/preview should create different tasks
-    const normalizedReason = (reason || '').trim().toLowerCase();
-    return `${source.trim().toLowerCase()}::${preview.trim().toLowerCase()}::${normalizedReason}`;
-}
-
  
  
 export function hasRecentDuplicateTask(queue: EvolutionQueueItem[], source: string, preview: string, now: number, reason?: string): boolean {
@@ -667,7 +658,7 @@ async function processEvolutionQueue(wctx: WorkspaceContext, logger: PluginLogge
                             workspaceDir: wctx.workspaceDir,
                             stateDir: wctx.stateDir,
                             logger: api?.logger || logger,
-                            // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: api is guaranteed non-null in this recovery path where runtimeAdapter is required
+                             
                             runtimeAdapter: new OpenClawTrinityRuntimeAdapter(api!),
                             subagent: api?.runtime?.subagent,
                         });
@@ -1185,7 +1176,7 @@ async function processEvolutionQueue(wctx: WorkspaceContext, logger: PluginLogge
                     let snapshotData: NocturnalSessionSnapshot | undefined;
 
                     if (isPollingTask) {
-                        // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: polling path requires existing resultRef
+                         
                         workflowId = sleepTask.resultRef!;
                     } else {
                         // Phase 1: Build trajectory snapshot for Nocturnal pipeline
@@ -1556,7 +1547,7 @@ async function processEvolutionQueue(wctx: WorkspaceContext, logger: PluginLogge
                     const manager = new CorrectionObserverWorkflowManager({
                         workspaceDir: wctx.workspaceDir,
                         logger,
-                        subagent: api?.runtime?.subagent!, /* eslint-disable-line @typescript-eslint/no-non-null-assertion */
+                        subagent: api?.runtime?.subagent!,  
                         agentSession: api?.runtime?.agent?.session,
                     });
 
@@ -1570,7 +1561,7 @@ async function processEvolutionQueue(wctx: WorkspaceContext, logger: PluginLogge
                         workflowId = handle.workflowId;
                         koTask.resultRef = workflowId;
                     } else {
-                        workflowId = koTask.resultRef!; /* eslint-disable-line @typescript-eslint/no-non-null-assertion */
+                        workflowId = koTask.resultRef!;  
                     }
 
                     // Poll workflow state

package/src/service/health-query-service.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import * as fs from 'fs';
 import * as path from 'path';
 import { readPainFlagData } from '../core/pain.js';

package/src/service/keyword-optimization-service.ts

@@ -31,12 +31,13 @@ export class KeywordOptimizationService {
   applyResult(result: CorrectionObserverResult): void {
     const learner = CorrectionCueLearner.get(this.stateDir);
 
-    if (!result.updated || !result.updates) {
+    const updates = result.updates ?? {};
+    if (!result.updated || Object.keys(updates).length === 0) {
       this.logger?.info?.('[KeywordOptimizationService] No updates to apply');
       return;
     }
 
-    for (const [term, update] of Object.entries(result.updates)) {
+    for (const [term, update] of Object.entries(updates)) {
       try {
         switch (update.action) {
           case 'add': {
@@ -74,6 +75,27 @@ export class KeywordOptimizationService {
         this.logger?.warn?.(`[KeywordOptimizationService] ${update.action.toUpperCase()} failed for term="${term}": ${String(opErr)}`);
       }
     }
+
+    // H-1: Record confirmed false positives — terms where correctionDetected fired
+    // but trajectory analysis shows user wasn't actually expressing frustration.
+    if (result.fpAnalysisStatus === 'completed' && result.fpTerms && result.fpTerms.length > 0) {
+      // Normalize: trim, lowercase, dedupe, sanity cap
+      const MAX_FP_TERMS = 20;
+      const normalizedFpTerms = [...new Set(
+        result.fpTerms
+          .map(t => t.trim().toLowerCase())
+          .filter(t => t.length > 0)
+      )].slice(0, MAX_FP_TERMS);
+
+      for (const term of normalizedFpTerms) {
+        try {
+          learner.recordFalsePositive(term);
+          this.logger?.info?.(`[KeywordOptimizationService] FP recorded for term="${term}" (weight x0.8)`);
+        } catch (fpErr) {
+          this.logger?.warn?.(`[KeywordOptimizationService} recordFalsePositive failed for term="${term}": ${String(fpErr)}`);
+        }
+      }
+    }
   }
 
   /**

package/src/service/nocturnal-runtime.ts

@@ -390,9 +390,6 @@ export function checkCooldown(
     } = {}
 ): CooldownCheckResult {
     const {
-         
-        globalCooldownMs: _globalCooldownMs = DEFAULT_GLOBAL_COOLDOWN_MS,
-        principleCooldownMs: _principleCooldownMs = DEFAULT_PRINCIPLE_COOLDOWN_MS,
         maxRunsPerWindow = DEFAULT_MAX_RUNS_PER_WINDOW,
         quotaWindowMs = DEFAULT_QUOTA_WINDOW_MS,
     } = options;

package/src/service/nocturnal-service.ts

@@ -116,7 +116,7 @@ function incrementGeneratedSampleCount(stateDir: string, principleId: string): v
     state.generatedSampleCount += 1;
     setPrincipleState(stateDir, state);
   } catch (err) {
-    // eslint-disable-next-line no-console -- Non-critical warning in helper function
+     
     console.warn(`[nocturnal-service] Failed to sync generatedSampleCount for ${principleId}:`, err instanceof Error ? err.stack : err);
   }
 }
@@ -522,7 +522,7 @@ function persistCodeCandidate(
     try {
       refreshPrincipleLifecycle(workspaceDir, stateDir);
     } catch (err) {
-      // eslint-disable-next-line no-console -- Non-critical warning in helper function
+       
       console.warn('[nocturnal-service] Lifecycle refresh failed after code candidate persistence:', err instanceof Error ? err.stack : err);
     }
     return {
@@ -702,7 +702,7 @@ export function executeNocturnalReflection(
 ): NocturnalRunResult {
   // Use provided logger or fallback to console
   const logger = options.logger;
-  // eslint-disable-next-line no-console -- Intentional console fallback when no logger provided
+   
   const warn = logger?.warn?.bind(logger) ?? console.warn.bind(console);
 
   const diagnostics: NocturnalRunDiagnostics = {
@@ -882,7 +882,7 @@ export function executeNocturnalReflection(
           diagnostics,
         };
       }
-      trinityArtifact = trinityResult.artifact!; // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: artifact is validated by validateTrinityDraft which returns valid: true when artifact exists
+      trinityArtifact = trinityResult.artifact!;  
       // Convert Trinity draft to arbiter-compatible artifact
       const artifactData = draftToArtifact(trinityArtifact);
       rawJson = JSON.stringify(artifactData);
@@ -933,7 +933,7 @@ export function executeNocturnalReflection(
             diagnostics,
           };
         }
-        trinityArtifact = trinityResult.artifact!; // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: artifact is validated by validateTrinityDraft which returns valid: true when artifact exists
+        trinityArtifact = trinityResult.artifact!;  
         // Convert Trinity draft to arbiter-compatible artifact
         const artifactData = draftToArtifact(trinityArtifact);
         rawJson = JSON.stringify(artifactData);
@@ -1187,7 +1187,7 @@ async function executeNocturnalReflectionWithAdapter(
 ): Promise<NocturnalRunResult> {
   // Use provided logger or fallback to console
   const logger = options.logger;
-  // eslint-disable-next-line no-console -- Intentional console fallback when no logger provided
+   
   const warn = logger?.warn?.bind(logger) ?? console.warn.bind(console);
 
   const diagnostics: NocturnalRunDiagnostics = {
@@ -1387,7 +1387,7 @@ async function executeNocturnalReflectionWithAdapter(
       adjustThresholdsFromSignals(stateDir, { malformedRate: 1.0, arbiterRejectRate: 0.0, executabilityRejectRate: 0.0, qualityDelta: 0.0 });
       return { success: false, noTargetSelected: false, validationFailed: true, validationFailures: [`Trinity override failed: ${failures.join('; ')}`], snapshot, diagnostics };
     }
-    trinityArtifact = trinityResult.artifact!; // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: artifact is validated by validateTrinityDraft which returns valid: true when artifact exists
+    trinityArtifact = trinityResult.artifact!;  
     const artifactData = draftToArtifact(trinityArtifact);
     rawJson = JSON.stringify(artifactData);
   } else {
@@ -1414,7 +1414,7 @@ async function executeNocturnalReflectionWithAdapter(
           adjustThresholdsFromSignals(stateDir, { malformedRate: 1.0, arbiterRejectRate: 0.0, executabilityRejectRate: 0.0, qualityDelta: 0.0 });
           return { success: false, noTargetSelected: false, validationFailed: true, validationFailures: failures, snapshot, diagnostics };
         }
-        trinityArtifact = trinityResult.artifact!; // eslint-disable-line @typescript-eslint/no-non-null-assertion -- Reason: artifact is validated by validateTrinityDraft which returns valid: true when artifact exists
+        trinityArtifact = trinityResult.artifact!;  
         const artifactData = draftToArtifact(trinityArtifact);
         rawJson = JSON.stringify(artifactData);
       } else {

package/src/service/queue-io.ts

@@ -160,9 +160,9 @@ export function readRecentPainContext(wctx: WorkspaceContext): RecentPainContext
         }
     } catch (err) {
       // Best effort — non-fatal, but surface unexpected errors
-      /* eslint-disable no-console */
+       
       console.warn(`[queue-io] Failed to read pain context (non-fatal): ${String(err)}`);
-      /* eslint-enable no-console */
+       
     }
 
     return { mostRecent: null, recentPainCount: 0, recentMaxPainScore: 0 };
@@ -185,7 +185,7 @@ export function shouldSkipForDedup(
     const recentSimilarReflection = hasRecentSimilarReflection(queue, painSourceKey, now);
 
     if (recentSimilarReflection) {
-        const completedTime = new Date(recentSimilarReflection.completed_at!).getTime(); /* eslint-disable-line @typescript-eslint/no-non-null-assertion */
+        const completedTime = new Date(recentSimilarReflection.completed_at!).getTime();  
         logger?.debug?.(`[PD:EvolutionWorker] Skipping sleep_reflection — similar reflection completed ${Math.round((now - completedTime) / 60000)}min ago (same pain pattern: ${painSourceKey})`);
         return true;
     }
@@ -358,9 +358,9 @@ export function loadEvolutionQueue(queuePath: string): EvolutionQueueItem[] {
             rawQueue = [];
         } else {
             // Corrupted JSON or other read error — warn and recover with empty queue
-            /* eslint-disable no-console */
+             
             console.warn(`[queue-io] Failed to load evolution queue (recovering with empty): ${String(err)}`);
-            /* eslint-enable no-console */
+             
             rawQueue = [];
         }
     }

package/src/service/sleep-cycle.ts

@@ -71,7 +71,7 @@ export interface CycleOptions {
  * @param options.heartbeatCounterRef — mutable counter, incremented by runCycle
  */
 export async function runCycle(options: CycleOptions): Promise<WorkerStatusReport> {
-    const { wctx, logger, eventLog: _eventLog, api: _api, heartbeatCounterRef } = options;
+    const { wctx, logger, api: _api, heartbeatCounterRef } = options;
     const cycleStart = Date.now();
     heartbeatCounterRef.value++;
 

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

@@ -55,6 +55,19 @@ export interface CorrectionObserverResult {
     falsePositiveRate?: number;
     reasoning: string;
   }>;
+  /**
+   * Terms identified as false positives — user message didn't actually indicate
+   * frustration/correction despite correctionDetected firing for these terms.
+   * CORR-10 / H-1: Calling recordFalsePositive() decays weight by x0.8 per term.
+   * Only meaningful when fpAnalysisStatus='completed'.
+   */
+  fpTerms?: string[];
+  /**
+   * Whether FP analysis was performed. 'skipped' means the LLM did not run
+   * trajectory analysis (e.g., trajectory was empty). 'completed' means fpTerms
+   * contains the LLM's FP findings (may be empty if no FPs were found).
+   */
+  fpAnalysisStatus?: 'completed' | 'skipped';
   /** Human-readable summary */
   summary: string;
 }

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

@@ -115,6 +115,7 @@ export const correctionObserverWorkflowSpec: SubagentWorkflowSpec<CorrectionObse
             '## TASK',
             'Analyze the current correction keyword store and recent user messages.',
             'Recommend ADD/UPDATE/REMOVE actions to improve correction cue accuracy.',
+            'Also identify terms that triggered false positives (correctionDetected fired but user message doesn\'t indicate actual frustration).',
             '',
             '## Current Keyword Store (' + keywordStoreSummary.totalKeywords + ' terms):',
             termsList,
@@ -129,11 +130,14 @@ export const correctionObserverWorkflowSpec: SubagentWorkflowSpec<CorrectionObse
             '- 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',
             '- REMOVE: If a term has 0 hits after many uses AND high false positive rate (>0.3)',
+            '- FALSE POSITIVE: If a term appears in trajectory but the user message doesn\'t actually express frustration (e.g., user said "wrong" but in a factual context, not emotional)',
+            '- fpAnalysisStatus: set to "completed" if you performed trajectory analysis (even if no FPs found), or "skipped" if trajectory was empty/unavailable',
             '- Keep reasoning concise (max 100 chars)',
             '- Weight range: 0.1-0.9',
             '',
             'Return strict JSON (no markdown):',
-            '{"updated": boolean, "updates": {...}, "summary": string}',
+            '{"updated": boolean, "updates": {...}, "fpTerms": ["term1", ...], "fpAnalysisStatus": "completed" | "skipped", "summary": string}',
+            'Note: fpTerms is optional — only include if you identified clear false positives.',
         ].join('\n');
     },
 

package/src/service/subagent-workflow/deep-reflect-workflow-manager.ts

@@ -130,7 +130,7 @@ export const deepReflectWorkflowSpec: SubagentWorkflowSpec<DeepReflectResult> =
             } else if (Array.isArray(lastMessage?.content)) {
                 insights = (lastMessage.content as { type?: string; text?: string }[])
                     .filter((c) => c?.type === 'text' && typeof c.text === 'string')
-                    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: filter ensures c.text is a string
+                     
                     .map((c) => c.text!)
                     .join('\n');
             }

package/src/service/subagent-workflow/empathy-observer-workflow-manager.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import type { PluginLogger } from '../../openclaw-sdk.js';
 import type {
     SubagentWorkflowSpec,

package/src/service/subagent-workflow/nocturnal-workflow-manager.ts

@@ -453,7 +453,7 @@ export class NocturnalWorkflowManager implements WorkflowManager {
 
     async sweepExpiredWorkflows(
         maxAgeMs = 30 * 60 * 1000,
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Reason: subagentRuntime param is intentionally any for backward compatibility with callers
+         
         subagentRuntime?: any,
          
         agentSession?: {

package/src/service/subagent-workflow/workflow-store.ts

@@ -1,4 +1,4 @@
-/* eslint-disable no-console */
+ 
 import Database from 'better-sqlite3';
 import * as fs from 'fs';
 import * as path from 'path';

package/src/utils/file-lock.ts

@@ -352,7 +352,7 @@ export async function withAsyncLock<T>(
   try {
     return await fn();
   } finally {
-    // eslint-disable-next-line @typescript-eslint/no-non-null-assertion -- Reason: resolveRelease is assigned in Promise constructor before finally runs
+     
     resolveRelease!();
     // 清理已完成的队列
     if (asyncLockQueues.get(lockPath) === currentQueue) {

package/src/utils/io.ts

@@ -126,7 +126,7 @@ export function parseKvLines(text: string): Record<string, string> {
   return result;
 }
 
-// eslint-disable-next-line @typescript-eslint/no-explicit-any -- Reason: serializeKvLines handles arbitrary object shapes for kv line serialization
+ 
 export function serializeKvLines(data: Record<string, any>): string {
   const lines: string[] = [];
   const keys = Object.keys(data).sort();

package/src/utils/retry.ts

@@ -424,7 +424,7 @@ export function computeDynamicTimeout(
   if (history.length < MIN_SAMPLES) {
     // Not enough data — use the spec's static timeout
     const fallback = clampTimeout(defaultTimeout);
-    // eslint-disable-next-line no-console -- Monitoring output for fallback diagnostics
+     
     console.info(`[PD:DynamicTimeout] Insufficient samples (${history.length} < ${MIN_SAMPLES}) for '${workflowType}', falling back to static timeout: ${fallback}ms`);
     return fallback;
   }
@@ -432,7 +432,7 @@ export function computeDynamicTimeout(
   const p95 = percentile(history, 95);
   const adaptive = p95 * SAFETY_MULTIPLIER;
   const result = clampTimeout(adaptive);
-  // eslint-disable-next-line no-console -- Monitoring output for adaptive timeout diagnostics
+   
   console.info(`[PD:DynamicTimeout] Computed adaptive timeout for '${workflowType}': P95=${p95}ms (from ${history.length} samples) × ${SAFETY_MULTIPLIER} = ${result}ms`);
   return result;
 }

package/tests/core/model-deployment-registry.test.ts

@@ -378,8 +378,15 @@ describe('ModelDeploymentRegistry getDeployment / listDeployments', () => {
 
     const deployments = listDeployments(tmpDir);
     expect(deployments).toHaveLength(2);
-    // Most recently updated is last in the list (sorted asc by updatedAt in memory)
-    expect(deployments[0].workerProfile).toBe('local-editor'); // bound second
+    // Verify sort order is descending by updatedAt (most recent first)
+    const [first, second] = deployments;
+    const firstUpdated = new Date(first.updatedAt).getTime();
+    const secondUpdated = new Date(second.updatedAt).getTime();
+    expect(firstUpdated).toBeGreaterThanOrEqual(secondUpdated);
+    // Also verify both expected profiles are present (order-independent)
+    const profiles = deployments.map(d => d.workerProfile);
+    expect(profiles).toContain('local-reader');
+    expect(profiles).toContain('local-editor');
   });
 
   it('listDeployments filters by workerProfile', () => {