principles-disciple 1.16.0 → 1.17.0

package/src/core/workspace-dir-validation.ts

@@ -1,152 +1,75 @@
 /**
  * WorkspaceDir Validation Utilities
  *
- * Provides runtime validation of workspaceDir to catch OpenClaw context bugs early.
- * When a hook receives an invalid workspaceDir, we warn immediately rather than
- * silently writing to the wrong directory.
+ * This module only validates candidate workspace directories and delegates
+ * actual resolution policy to workspace-dir-service.ts.
  */
 
-/* eslint-disable no-unused-vars -- Reason: type definitions require param names that implementations may not use */
-
 import * as os from 'os';
 import type { PluginLogger } from '../openclaw-sdk.js';
+import { resolveWorkspaceDir, type WorkspaceResolutionContext } from './workspace-dir-service.js';
 
-/**
- * Check if a path looks like a home directory (not a real workspace).
- * Returns the reason if suspicious, or null if it looks valid.
- */
 export function validateWorkspaceDir(dir: string | undefined): string | null {
   if (!dir) {
     return 'workspaceDir is undefined/null';
   }
-  
+
   const homeDir = os.homedir();
-  
-  // Home directory itself is not a valid workspace
+
   if (dir === homeDir) {
     return `workspaceDir equals home directory (${homeDir}), likely missing context field`;
   }
-  
-  // Root directory is definitely not a workspace
+
   if (dir === '/' || dir === '') {
     return `workspaceDir is root or empty: "${dir}"`;
   }
-  
-  // Check if it looks like a resolved '.' that went wrong
-  // Common bad patterns:
+
+  const escapedHome = homeDir.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
   const badPatterns = [
-    // Directly under home without a workspace subdirectory
-    { pattern: new RegExp(`^${homeDir.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}$`), desc: 'is home directory itself' },
-    { pattern: new RegExp(`^${homeDir.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}/$`), desc: 'is home directory with trailing slash' },
+    { pattern: new RegExp(`^${escapedHome}$`), desc: 'is home directory itself' },
+    { pattern: new RegExp(`^${escapedHome}/$`), desc: 'is home directory with trailing slash' },
   ];
-  
+
   for (const { pattern, desc } of badPatterns) {
     if (pattern.test(dir)) {
       return `workspaceDir ${desc}: "${dir}"`;
     }
   }
-  
-  return null; // Looks valid
+
+  return null;
 }
 
-/**
- * Try to resolve workspaceDir from agentId.
- * Returns the resolved path if successful and valid, or null.
- */
-function tryResolveFromAgentId(
-  agentId: string,
+export function resolveValidWorkspaceDir(
+  ctx: WorkspaceResolutionContext,
   api: {
     runtime: { agent: { resolveAgentWorkspaceDir: (config: unknown, agentId: string) => string } };
     config: unknown;
+    logger: PluginLogger;
   },
-  onWarning: (msg: string) => void,
-): string | null {
-  try {
-    const resolved = api.runtime.agent.resolveAgentWorkspaceDir(api.config, agentId);
-    const issue = validateWorkspaceDir(resolved);
-    if (issue) {
-      onWarning(`agentId resolution returned invalid: "${resolved}" (${issue})`);
-      return null;
-    }
-    return resolved;
-  } catch (err) {
-    onWarning(`failed to resolve from agentId: ${String(err)}`);
-    return null;
-  }
+  options?: { source?: string; fallbackAgentId?: string },
+): string | undefined {
+  return resolveWorkspaceDir(api as never, ctx, {
+    source: options?.source,
+    fallbackAgentId: options?.fallbackAgentId,
+    logger: api.logger,
+  });
 }
 
-/**
- * Validate a fallback workspaceDir and warn if invalid.
- * Returns the path regardless (it's the last resort).
- */
-function validateFallback(path: string, onWarning: (msg: string) => void): string {
-  const issue = validateWorkspaceDir(path);
-  if (issue) {
-    onWarning(`FINAL FALLBACK "${path}" is also invalid: ${issue}. Events will be written to wrong location!`);
-  }
-  return path;
-}
-
-/**
- * Resolve workspaceDir with validation and warning.
- *
- * Usage:
- *   const workspaceDir = resolveValidWorkspaceDir(ctx, api, { source: 'after_tool_call' });
- *
- * Fallback chain:
- *   1. ctx.workspaceDir (validated)
- *   2. api.runtime.agent.resolveAgentWorkspaceDir(config, ctx.agentId)
- *   3. api.resolvePath('.') (last resort, warns loudly)
- */
-export function resolveValidWorkspaceDir(
-  ctx: { workspaceDir?: string; agentId?: string },
+export function logWorkspaceDirHealth(
+  ctx: WorkspaceResolutionContext,
+  source: string,
   api: {
     runtime: { agent: { resolveAgentWorkspaceDir: (config: unknown, agentId: string) => string } };
     config: unknown;
-    resolvePath: (input: string) => string;
     logger: PluginLogger;
   },
-  options?: { source?: string; onWarning?: (msg: string) => void },
-): string {
-  const source = options?.source || 'unknown';
-  const onWarning = options?.onWarning || ((msg: string) => api.logger.warn(`[PD:workspaceDir] ${msg}`));
-
-  // 1. Try ctx.workspaceDir
-  if (ctx.workspaceDir) {
-    const issue = validateWorkspaceDir(ctx.workspaceDir);
-    if (issue) {
-      onWarning(`${source}: ctx.workspaceDir="${ctx.workspaceDir}" is invalid: ${issue}`);
-    } else {
-      return ctx.workspaceDir;
-    }
-  }
-
-  // 2. Try agentId resolution
-  if (ctx.agentId) {
-    const fromAgent = tryResolveFromAgentId(ctx.agentId, api, onWarning);
-    if (fromAgent) return fromAgent;
-  }
-
-  // 3. Final fallback
-  return validateFallback(api.resolvePath('.'), onWarning);
-}
-
-/**
- * Log workspaceDir resolution for debugging.
- * Call this once during plugin startup to verify hook contexts.
- */
-export function logWorkspaceDirHealth(ctx: { workspaceDir?: string; agentId?: string }, source: string, api: {
-  runtime: { agent: { resolveAgentWorkspaceDir: (config: unknown, agentId: string) => string } };
-  config: unknown;
-  resolvePath: (input: string) => string;
-  logger: PluginLogger;
-}): void {
-  const resolved = resolveValidWorkspaceDir(ctx, api, { source });
+): void {
+  const resolved = resolveValidWorkspaceDir(ctx, api, { source, fallbackAgentId: 'main' });
   const issue = validateWorkspaceDir(resolved);
-  
+
   if (issue) {
     api.logger.error(`[PD:health] ${source}: workspaceDir="${resolved}" - ${issue}`);
   } else {
-    api.logger.info(`[PD:health] ${source}: workspaceDir="${resolved}" ✓`);
+    api.logger.info(`[PD:health] ${source}: workspaceDir="${resolved}" OK`);
   }
 }

package/src/hooks/bash-risk.ts

@@ -38,12 +38,12 @@ export type BashRiskLevel = 'safe' | 'dangerous' | 'normal';
  * @param logger - Optional logger for warnings about invalid patterns
  * @returns The risk level: 'safe', 'dangerous', or 'normal'
  */
-/* eslint-disable @typescript-eslint/max-params -- Reason: Bash risk analysis requires command + pattern lists - refactoring to options object would be breaking API change */
+ 
 export function analyzeBashCommand(
   command: string,
   safePatterns: string[],
   dangerousPatterns: string[],
-  logger?: { warn?: (/* eslint-disable-line no-unused-vars -- Reason: callback parameter, unused by design */_message: string) => void }
+  logger?: { warn?: ( _message: string) => void }
 ): BashRiskLevel {
   let normalizedCmd = command.trim().toLowerCase();
 
@@ -65,7 +65,7 @@ export function analyzeBashCommand(
   // - Zero-width joiner (U+200D)
   // - Word joiner (U+2060)
   // - Zero-width invisible separator (U+FEFF)
-  // eslint-disable-next-line no-misleading-character-class -- Reason: zero-width character class ranges are intentional - documented in comment above
+   
   const ZERO_WIDTH_CHARS = /[\u200B\u200C\u200D\u2060\uFEFF]/g;
   if (ZERO_WIDTH_CHARS.test(command)) {
     logger?.warn?.(`[PD_GATE] Bash command contains zero-width characters — blocking as dangerous`);

package/src/hooks/edit-verification.ts

@@ -126,7 +126,7 @@ This is enforced by P-03 (精确匹配前验证原则).`;
  * Handle edit tool verification before allowing operation
  * This enforces P-03 at the tool layer
  */
-// eslint-disable-next-line @typescript-eslint/max-params -- Reason: Hook handler signature requires event + context + config - refactoring would break plugin interface
+ 
 export function handleEditVerification(
   event: PluginHookBeforeToolCallEvent,
   wctx: WorkspaceContext,
@@ -155,11 +155,11 @@ export function handleEditVerification(
   }
 
   // 2. Resolve and read file
-  // eslint-disable-next-line @typescript-eslint/init-declarations -- assigned in try, catch has early return
+   
   let absolutePath: string;
   try {
     absolutePath = wctx.resolve(filePath);
-  } catch (_error) { // eslint-disable-line @typescript-eslint/no-unused-vars, no-unused-vars -- Reason: intentionally unused - let it fail naturally on path resolution error
+  } catch (_error) { // eslint-disable-line @typescript-eslint/no-unused-vars -- Reason: intentionally unused - let it fail naturally on path resolution error
     // Path resolution error, let it fail naturally
     return;
   }
@@ -222,7 +222,7 @@ export function handleEditVerification(
     }
 
     // 3. Read current file content with improved error handling
-    // eslint-disable-next-line @typescript-eslint/init-declarations -- assigned in try, all catch paths return early
+     
     let currentContent: string;
     try {
       currentContent = fs.readFileSync(absolutePath, 'utf-8');

package/src/hooks/gate-block-helper.ts

@@ -48,9 +48,9 @@ export interface BlockContext {
 export function recordGateBlockAndReturn(
   wctx: WorkspaceContext,
   blockCtx: BlockContext,
-  /* eslint-disable no-unused-vars -- Reason: type-only callback parameters in logger type */
+   
   logger: { warn?: (_message: string) => void; error?: (_message: string) => void; info?: (_message: string) => void }
-  /* eslint-enable no-unused-vars */
+   
 ): PluginHookBeforeToolCallResult {
   const { filePath, reason, toolName, sessionId, blockSource } = blockCtx;
 
@@ -93,7 +93,7 @@ export function recordGateBlockAndReturn(
     wctx.trajectory?.recordGateBlock?.(trajectoryPayload);
   } catch (error: unknown) {
     logWarn(`[PD_GATE] Failed to record trajectory gate block: ${String(error)}`);
-    // eslint-disable-next-line @typescript-eslint/no-use-before-define -- Reason: function is defined later but called in this helper for retry logic
+     
     scheduleTrajectoryGateBlockRetry(wctx, trajectoryPayload, 1, logWarn, logError);
   }
 
@@ -133,7 +133,7 @@ This is a mandatory security gate. The operation was blocked because the modific
  * Uses exponential backoff with max retries.
  * Failures are logged but do not affect the runtime block decision.
  */
-/* eslint-disable @typescript-eslint/max-params, no-unused-vars -- Reason: Function requires all params for retry scheduling */
+ 
 function scheduleTrajectoryGateBlockRetry(
   wctx: WorkspaceContext,
   payload: {

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)) {