principles-disciple 1.16.0 → 1.18.0

package/src/hooks/gate.ts

@@ -134,7 +134,7 @@ export function handleBeforeToolCall(
     const mutationMatch = /(?:>|>>|sed\s+-i|rm|mv|mkdir|touch|cp)\s+(?:-[a-zA-Z]+\s+)*([^\s;&|<>]+)/.exec(command);
 
     if (mutationMatch) {
-      // eslint-disable-next-line @typescript-eslint/prefer-destructuring -- Reason: mutationMatch[1] assigned to reassignable outer let - destructuring would shadow outer variable
+       
       filePath = mutationMatch[1];
     } else {
       const hasRiskPath = profile.risk_paths.some(rp => command.includes(rp));
@@ -167,30 +167,30 @@ export function handleBeforeToolCall(
       action: {
         toolName: event.toolName,
         normalizedPath: relPath,
-        // eslint-disable-next-line @typescript-eslint/no-use-before-define -- Reason: mutual recursion between helper functions - reordering would break logical grouping
+         
         paramsSummary: _extractParamsSummary(event.params),
       },
       workspace: {
         isRiskPath: risky,
-        // eslint-disable-next-line @typescript-eslint/no-use-before-define -- Reason: mutual recursion between helper functions - reordering would break logical grouping
+         
         planStatus: _getPlanStatus(ctx.workspaceDir),
-        // eslint-disable-next-line @typescript-eslint/no-use-before-define -- Reason: mutual recursion between helper functions - reordering would break logical grouping
+         
         hasPlanFile: _hasPlanFile(ctx.workspaceDir),
       },
       session: {
         sessionId: ctx.sessionId,
-        // eslint-disable-next-line @typescript-eslint/no-use-before-define -- Reason: mutual recursion between helper functions - reordering would break logical grouping
+         
         currentGfi: _getCurrentGfi(ctx.sessionId),
-        // eslint-disable-next-line @typescript-eslint/no-use-before-define -- Reason: mutual recursion between helper functions - reordering would break logical grouping
+         
         recentThinking: _hasRecentThinking(ctx.sessionId),
       },
       evolution: {
-        // eslint-disable-next-line @typescript-eslint/no-use-before-define -- Reason: mutual recursion between helper functions - reordering would break logical grouping
+         
         epTier: _getEpTier(wctx.workspaceDir),
       },
       derived: {
         estimatedLineChanges: estimateLineChanges({ toolName: event.toolName, params: event.params }),
-        // eslint-disable-next-line @typescript-eslint/no-use-before-define -- Reason: mutual recursion between helper functions - reordering would break logical grouping
+         
         bashRisk: _getBashRisk(event, profile),
       },
     };
@@ -337,12 +337,12 @@ function _getEpTier(workspaceDir: string): number {
   }
 }
 
-/* eslint-disable no-unused-vars, @typescript-eslint/no-unused-vars -- Reason: type-only parameter not used at runtime */
+ 
 function _getBashRisk(
   event: PluginHookBeforeToolCallEvent,
   _profile: { risk_paths: string[] }
 ): 'safe' | 'normal' | 'dangerous' | 'unknown' {
-/* eslint-enable no-unused-vars, @typescript-eslint/no-unused-vars */
+ 
   if (!BASH_TOOLS_SET.has(event.toolName)) return 'unknown';
   try {
     const command = String(event.params.command || event.params.args || '');

package/src/hooks/gfi-gate.ts

@@ -47,16 +47,16 @@ export interface GfiGateConfig {
 /**
  * Internal helper to call the shared block helper with gfi-gate source tag.
  */
-/* eslint-disable @typescript-eslint/max-params -- Reason: Helper function requires all params for block recording */
+ 
 function block(
   wctx: WorkspaceContext,
   filePath: string,
   reason: string,
   toolName: string,
   sessionId: string | undefined,
-  /* eslint-disable no-unused-vars -- Reason: type-only callback params in logger type */
+   
   logger?: { info?: (message: string) => void; warn?: (message: string) => void; error?: (message: string) => void }
-  /* eslint-enable no-unused-vars */
+   
 ): PluginHookBeforeToolCallResult {
   return recordGateBlockAndReturn(wctx, {
     filePath,
@@ -64,19 +64,19 @@ function block(
     toolName,
     sessionId,
     blockSource: 'gfi-gate',
-  }, logger || // eslint-disable-next-line @typescript-eslint/no-empty-function -- empty warn/error no-op
+  }, logger ||  
   { warn: () => {}, error: () => {} } as const);
 }
 
-/* eslint-disable @typescript-eslint/max-params -- Reason: Gate function requires all params for comprehensive checks */
+ 
 export function checkGfiGate(
   event: PluginHookBeforeToolCallEvent,
   wctx: WorkspaceContext,
   sessionId: string | undefined,
   config: GfiGateConfig,
-  /* eslint-disable no-unused-vars -- Reason: type-only callback params in logger type */
+   
   logger?: { info?: (message: string) => void; warn?: (message: string) => void }
-  /* eslint-enable no-unused-vars */
+   
 ): PluginHookBeforeToolCallResult | undefined {
   if (!config || config.enabled === false || !sessionId) {
     return undefined;

package/src/hooks/lifecycle.ts

@@ -122,7 +122,7 @@ export async function extractPainFromSessionFile(sessionFile: string, ctx: Plugi
     try {
       rl.close();
       fileStream.destroy();
-    } catch (_e) { // eslint-disable-line @typescript-eslint/no-unused-vars, no-unused-vars -- Reason: intentionally unused - cleanup errors ignored
+    } catch (_e) { // eslint-disable-line @typescript-eslint/no-unused-vars -- Reason: intentionally unused - cleanup errors ignored
       // Ignore cleanup errors
     }
   }
@@ -196,7 +196,7 @@ export async function handleBeforeCompaction(
     await extractPainFromSessionFile(event.sessionFile, ctx);
 
     // 新增：提取并保存工作记忆
-    /* eslint-disable @typescript-eslint/no-use-before-define -- Reason: extractAndSaveWorkingMemory is defined later in this file */
+     
     await extractAndSaveWorkingMemory(event.sessionFile, ctx, wctx);
   }
 }

package/src/hooks/llm.ts

@@ -255,7 +255,7 @@ export function handleLlmOutput(
     }
 
     // ═══ Thinking OS: Mental Model Usage Tracking ═══
-    /* eslint-disable @typescript-eslint/no-use-before-define -- Reason: trackThinkingModelUsage is defined later in this file */
+     
     trackThinkingModelUsage({
         text,
         wctx,

package/src/hooks/pain.ts

@@ -2,7 +2,7 @@ import * as fs from 'fs';
 import { isRisky, normalizePath } from '../utils/io.js';
 import { normalizeProfile } from '../core/profile.js';
 import { computePainScore, buildPainFlag, writePainFlag, trackPrincipleValue } from '../core/pain.js';
-import { getSession, trackFriction, resetFriction, getInjectedProbationIds, clearInjectedProbationIds } from '../core/session-tracker.js';
+import { getSession, trackFriction, resetFriction, getInjectedProbationIds, clearInjectedProbationIds, type SessionState } from '../core/session-tracker.js';
 import { denoiseError, computeHash } from '../utils/hashing.js';
 import { SystemLogger } from '../core/system-logger.js';
 import { WorkspaceContext } from '../core/workspace-context.js';
@@ -10,6 +10,8 @@ import { getEvolutionLogger, createTraceId } from '../core/evolution-logger.js';
 import { recordEvolutionSuccess, recordEvolutionFailure } from '../core/evolution-engine.js';
 import type { EvolutionLoopEvent } from '../core/evolution-types.js';
 import type { PluginHookAfterToolCallEvent, PluginHookToolContext, OpenClawPluginApi } from '../openclaw-sdk.js';
+import { validateWorkspaceDir } from '../core/workspace-dir-validation.js';
+import { resolveWorkspaceDir } from '../core/workspace-dir-service.js';
 
 /**
  * Interface for tool parameters to avoid 'any'
@@ -49,7 +51,9 @@ export function handleAfterToolCall(
   ctx: PluginHookToolContext & { workspaceDir?: string; pluginConfig?: Record<string, unknown> },
   api?: OpenClawPluginApi
 ): void {
-  const effectiveWorkspaceDir = ctx.workspaceDir || (api as unknown as { workspaceDir?: string })?.workspaceDir || api?.resolvePath?.('.');
+  const effectiveWorkspaceDir = api
+    ? resolveWorkspaceDir(api, ctx, { source: 'after_tool_call' })
+    : validateWorkspaceDir(ctx.workspaceDir) ? undefined : ctx.workspaceDir;
   if (!effectiveWorkspaceDir) {
     return;
   }
@@ -123,10 +127,10 @@ export function handleAfterToolCall(
     const hash = computeHash(denoised);
     
     const deltaF = config.get('scores.tool_failure_friction') || 30;
-    const updatedState = trackFriction(sessionId, deltaF, hash, effectiveWorkspaceDir);
+    const updatedState = trackFriction(sessionId, deltaF, hash, effectiveWorkspaceDir, { source: 'tool_failure' });
     
     // ── Trust Engine: Record failure ──
-    /* eslint-disable @typescript-eslint/no-use-before-define -- Reason: extractErrorType is defined later in this file */
+     
     const errorType = extractErrorType(event.error || errorText);
     const filePath = params.file_path || params.path || params.file;
     const relPath = typeof filePath === 'string' ? normalizePath(filePath, effectiveWorkspaceDir) : 'unknown';
@@ -184,7 +188,23 @@ export function handleAfterToolCall(
     clearInjectedProbationIds(sessionId, effectiveWorkspaceDir);
   } else {
     // ── SUCCESS BRANCH ──
-    const resetState = resetFriction(sessionId, effectiveWorkspaceDir);
+    // Only reduce tool_failure source GFI by 50%, preserve user_empathy and other sources
+    // This prevents "read file success" from wiping user frustration signals
+    const session = getSession(sessionId);
+    const toolFailureGfi = session?.gfiBySource?.['tool_failure'] || 0;
+    
+    let resetState: SessionState;
+    if (toolFailureGfi > 0) {
+      // Reduce tool_failure source by 50% (relief from successful tool execution)
+      const reliefAmount = toolFailureGfi * 0.5;
+      resetState = resetFriction(sessionId, effectiveWorkspaceDir, {
+        source: 'tool_failure',
+        amount: reliefAmount,
+      });
+    } else {
+      // No tool_failure GFI to reduce, just get current state
+      resetState = session || resetFriction(sessionId, effectiveWorkspaceDir);
+    }
     
     recordEvolutionSuccess(effectiveWorkspaceDir, event.toolName, {
         sessionId,

package/src/hooks/progressive-trust-gate.ts

@@ -86,15 +86,15 @@ export function buildEvolutionGateReason(
 /**
  * Internal helper to call the shared block helper with progressive-trust-gate source tag.
  */
-/* eslint-disable @typescript-eslint/max-params -- Reason: Helper function requires all parameters for block recording */
+ 
 function block(
   filePath: string,
   reason: string,
   wctx: WorkspaceContext,
   toolName: string,
-  /* eslint-disable no-unused-vars -- Reason: type-only callback params in logger type */
+   
   logger: { warn?: (message: string) => void; error?: (message: string) => void },
-  /* eslint-enable no-unused-vars */
+   
   sessionId?: string
 ): PluginHookBeforeToolCallResult {
   return recordGateBlockAndReturn(wctx, {
@@ -119,16 +119,16 @@ function block(
  * @param profile - Gate profile containing risk_paths config
  * @returns PluginHookBeforeToolCallResult to block, or undefined to allow
  */
-/* eslint-disable @typescript-eslint/max-params -- Reason: Gate function requires all parameters for comprehensive checks */
+ 
 export function checkProgressiveTrustGate(
   event: PluginHookBeforeToolCallEvent,
   wctx: WorkspaceContext,
   relPath: string,
   risky: boolean,
   lineChanges: number,
-  /* eslint-disable no-unused-vars -- Reason: type-only callback params in logger type */
+   
   logger: { warn?: (message: string) => void; error?: (message: string) => void; info?: (message: string) => void },
-  /* eslint-enable no-unused-vars */
+   
   ctx: { workspaceDir?: string; sessionId?: string },
   profile?: { risk_paths: string[]; core_governance_files?: string[] }
 ): PluginHookBeforeToolCallResult | void {
@@ -151,7 +151,7 @@ export function checkProgressiveTrustGate(
   });
 
   const currentTier = epDecision.currentTier ?? 1;
-  // eslint-disable-next-line @typescript-eslint/no-use-before-define -- Reason: mutual recursion between helper functions - reordering would break logical grouping
+   
   const tierName = getTierName(currentTier);
 
   logger.info?.(`[PD_GATE] EP Gate: Tier ${currentTier} (${tierName}), Tool: ${event.toolName}, Risk: ${risky}, Allowed: ${epDecision.allowed}`);

package/src/hooks/prompt.ts

@@ -1,7 +1,7 @@
 import * as fs from 'fs';
 import * as path from 'path';
 import type { PluginHookBeforePromptBuildEvent, PluginHookAgentContext, PluginHookBeforePromptBuildResult, PluginLogger, OpenClawPluginApi } from '../openclaw-sdk.js';
-import { clearInjectedProbationIds, getSession, resetFriction, setInjectedProbationIds, trackFriction } from '../core/session-tracker.js';
+import { clearInjectedProbationIds, getSession, resetFriction, setInjectedProbationIds, trackFriction, decayGfi, getGfiDecayElapsed } from '../core/session-tracker.js';
 import { WorkspaceContext } from '../core/workspace-context.js';
 import type { ContextInjectionConfig} from '../types.js';
 import { defaultContextConfig } from '../types.js';
@@ -308,6 +308,13 @@ export async function handleBeforePromptBuild(
     return;
   }
 
+  // ──── DEBUG: Verify subagent availability in this context ────
+  const subagent = ctx.api?.runtime?.subagent;
+  logger?.info?.(`[PD:DEBUG:SubagentCheck] trigger=${ctx.trigger}, subagent_exists=${!!subagent}, subagent.run_exists=${!!subagent?.run}`);
+  if (subagent?.run) {
+    logger?.info?.('[PD:DEBUG:SubagentCheck] run entrypoint is callable');
+  }
+
   const wctx = WorkspaceContext.fromHookContext(ctx);
   const { trigger, sessionId, api } = ctx;
   if (sessionId) {
@@ -358,7 +365,7 @@ export async function handleBeforePromptBuild(
   // appendSystemContext: Principles + Thinking OS + reflection_log + project_context (cacheable, WebUI-hidden)
   // prependContext: Only short dynamic directives: evolutionDirective + heartbeat
 
-  // eslint-disable-next-line no-useless-assignment -- Reason: initial value unused due to immediate reassignment
+   
   let prependSystemContext = '';
   let prependContext = '';
   let appendSystemContext = '';
@@ -612,6 +619,18 @@ The empathy observer subagent handles pain detection independently.
 
   // ──── 4. Heartbeat-specific checklist ────
   if (trigger === 'heartbeat') {
+    // ──── 4a. GFI Time-based Decay ────
+    // Apply segmented exponential decay to GFI on each heartbeat
+    if (sessionId) {
+      const elapsedMinutes = getGfiDecayElapsed(sessionId);
+      if (elapsedMinutes >= 1) {
+        const decayedState = decayGfi(sessionId, elapsedMinutes);
+        if (decayedState) {
+          logger?.info?.(`[PD:GFI] Heartbeat decay applied: ${elapsedMinutes}min elapsed, GFI now ${decayedState.currentGfi.toFixed(1)}`);
+        }
+      }
+    }
+    
     const heartbeatPath = wctx.resolve('HEARTBEAT');
     if (fs.existsSync(heartbeatPath)) {
       try {
@@ -628,7 +647,7 @@ ACTION: Run self-audit. If stable, reply ONLY with "HEARTBEAT_OK".
   }
 
   // ──── 6. Dynamic Attitude Matrix (based on GFI) ────
-  // eslint-disable-next-line no-useless-assignment -- Reason: initial value unused due to immediate reassignment
+   
   let attitudeDirective = '';
   const currentGfi = session?.currentGfi || 0;
   
@@ -853,10 +872,10 @@ ACTION: Run self-audit. If stable, reply ONLY with "HEARTBEAT_OK".
         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[] = [];
-          // eslint-disable-next-line @typescript-eslint/init-declarations -- assigned in while loop condition
+           
           let _m;
           const r = new RegExp(pattern.source, pattern.flags);
-          /* eslint-disable @typescript-eslint/no-unused-vars, no-unused-vars -- Reason: regex exec side effect used, match variable intentionally unused */
+          /* eslint-disable @typescript-eslint/no-unused-vars -- Reason: regex exec side effect used, match variable intentionally unused */
           while ((_m = r.exec(latestUserText)) !== null) matches.push(tool);
           return matches;
         });

package/src/hooks/subagent.ts

@@ -12,7 +12,7 @@ import type { WorkflowManager } from '../service/subagent-workflow/types.js';
  * Factory to create the appropriate WorkflowManager by workflow_type string.
  * Used by the subagent_ended hook to dispatch lifecycle recovery to the right manager.
  */
-/* eslint-disable @typescript-eslint/max-params -- Reason: Factory function requires workflowType, workspaceDir, logger, and subagent */
+ 
 function createWorkflowManagerForType(
     workflowType: string,
     workspaceDir: string,
@@ -23,7 +23,7 @@ function createWorkflowManagerForType(
         info: (m: string) => logger.info(String(m)),
         warn: (m: string) => logger.warn(String(m)),
         error: (m: string) => logger.error(String(m)),
-        // eslint-disable-next-line @typescript-eslint/no-empty-function -- debug logger no-op
+         
         debug: () => {},
     } as unknown as PluginLogger;
 

package/src/hooks/thinking-checkpoint.ts

@@ -40,12 +40,12 @@ export interface ThinkingCheckpointConfig {
  * @param logger - Optional logger for info messages
  * @returns Block result if thinking required, undefined otherwise
  */
-/* eslint-disable @typescript-eslint/max-params -- Reason: Checkpoint function requires event, config, sessionId, and logger */
+ 
 export function checkThinkingCheckpoint(
   event: PluginHookBeforeToolCallEvent,
   config: ThinkingCheckpointConfig,
   sessionId: string | undefined,
-  /* eslint-disable no-unused-vars -- Reason: callback parameter in type annotation */
+   
   logger?: { info?: (message: string) => void }
 ): PluginHookBeforeToolCallResult | undefined {
   const enabled = config.enabled ?? false;

package/src/hooks/trajectory-collector.ts

@@ -228,7 +228,7 @@ export function handleBeforeMessageWrite(
   // 提取文本内容
   let content = '';
   if (typeof msg.content === 'string') {
-    /* eslint-disable @typescript-eslint/prefer-destructuring */
+     
     // Reason: msg.content is string | ContentPart[]; destructuring would require renaming in the else branch
     content = msg.content;
   } else if (Array.isArray(msg.content)) {

package/src/http/principles-console-route.ts

@@ -9,6 +9,7 @@ import { TrajectoryRegistry } from '../core/trajectory.js';
 import { getCentralDatabase } from '../service/central-database.js';
 import { CentralOverviewService } from '../service/central-overview-service.js';
 import { CentralHealthService } from '../service/central-health-service.js';
+import { resolveRequiredWorkspaceDir } from '../core/workspace-dir-service.js';
 
 const ROUTE_PREFIX = '/plugins/principles';
 const API_PREFIX = `${ROUTE_PREFIX}/api`;
@@ -90,11 +91,11 @@ function serveFile(res: ServerResponse, filePath: string): boolean {
 }
 
 function createService(api: OpenClawPluginApi): ControlUiQueryService {
-  const workspaceDir = api.resolvePath('.');
+  const workspaceDir = resolveRequiredWorkspaceDir(api, { agentId: 'main' }, { source: 'principles_console.control_ui', fallbackAgentId: 'main' });
   return new ControlUiQueryService(workspaceDir);
 }
 
-/* eslint-disable @typescript-eslint/max-params -- Reason: Route handler requires api, pathname, req, and res */
+ 
 function handleApiRoute(
   api: OpenClawPluginApi,
   pathname: string,
@@ -102,13 +103,20 @@ function handleApiRoute(
   res: ServerResponse,
 ): Promise<boolean> | boolean {
   // Check authentication for API routes
-  /* eslint-disable @typescript-eslint/no-use-before-define -- Reason: validateGatewayAuth is defined later in the file */
+   
   if (!validateGatewayAuth(req)) {
     json(res, 401, { error: 'unauthorized', message: 'Valid Gateway token required.' });
     return true;
   }
 
-  const service = createService(api);
+  let service: ControlUiQueryService;
+  try {
+    service = createService(api);
+  } catch (error) {
+    api.logger.warn(`[PD:ControlUI] Failed to resolve workspace for ${pathname}: ${String(error)}`);
+    json(res, 500, { error: 'internal_error', message: String(error) });
+    return true;
+  }
   const url = new URL(req.url || pathname, 'http://127.0.0.1');
   const method = (req.method || 'GET').toUpperCase();
 
@@ -338,7 +346,7 @@ function handleApiRoute(
 
   // === Evolution API ===
   const evolutionService = () => {
-    const workspaceDir = api.resolvePath('.');
+    const workspaceDir = resolveRequiredWorkspaceDir(api, { agentId: 'main' }, { source: 'principles_console.evolution', fallbackAgentId: 'main' });
     const trajectory = TrajectoryRegistry.get(workspaceDir);
     return getEvolutionQueryService(trajectory);
   };
@@ -398,7 +406,7 @@ function handleApiRoute(
 
   // === Health Query API (v1.1 new endpoints) ===
   const healthService = () => {
-    const workspaceDir = api.resolvePath('.');
+    const workspaceDir = resolveRequiredWorkspaceDir(api, { agentId: 'main' }, { source: 'principles_console.health', fallbackAgentId: 'main' });
     return new HealthQueryService(workspaceDir);
   };
 

package/src/i18n/commands.ts

@@ -37,6 +37,10 @@ export const commandDescriptions: Record<string, Record<SupportedLanguage, strin
     zh: '管理思维模型 [status|propose|audit]',
     en: 'Manage Thinking OS [status|propose|audit]'
   },
+  'pd-reflect': {
+    zh: '手动触发 Nocturnal 睡眠反射（跳过 idle 检测）',
+    en: 'Manually trigger Nocturnal sleep reflection (bypass idle check)'
+  },
   'pd-daily': {
     zh: '配置并发送进化日报',
     en: 'Configure and send daily evolution report'