principles-disciple 1.28.0 → 1.28.2

package/src/core/pain-context-extractor.ts

@@ -203,6 +203,7 @@ function extractTurn(msg: ParsedMessage): string | null {
  * SAFETY: Tail-only read, skip oversized lines, cap output.
  * Returns empty string on any failure — caller should use pain reason as fallback.
  */
+    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
 export async function extractRecentConversation(
   sessionId: string,
   agentId = 'main',
@@ -235,11 +236,9 @@ export async function extractRecentConversation(
 /**
  * Extracts failed tool call context with argument correlation.
  */
-  // Reason: breaking API change - default param must precede required params for type inference compatibility
- 
 export async function extractFailedToolContext(
   sessionId: string,
-  agentId = 'main',
+  agentId: string,
   toolName: string,
   filePath?: string,
 ): Promise<string> {

package/src/core/pain.ts

@@ -279,6 +279,7 @@ export function readPainFlagContract(projectDir: string): PainFlagContractResult
  * Errors are silently ignored to avoid disrupting the pain pipeline.
  */
  
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
 export function trackPrincipleValue(
   workspaceDir: string,
   painData: { reason?: string; source?: string; score?: string },

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

@@ -103,6 +103,7 @@ async function writeCronStore(store: CronStoreFile): Promise<void> {
   });
 }
 
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
 function diff(declared: PDTaskSpec[], actual: CronJob[]): DiffAction[] {
   const actions: DiffAction[] = [];
   const actualByName = new Map<string, CronJob>();

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

@@ -4,6 +4,7 @@ import { reconcilePDTasks } from './pd-task-reconciler.js';
 export const PDTaskService: OpenClawPluginService = {
   id: 'principles-disciple-task-manager',
 
+    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
   async start(ctx: OpenClawPluginServiceContext): Promise<void> {
     const {workspaceDir} = ctx;
     if (!workspaceDir) {

package/src/core/principle-internalization/deprecated-readiness.ts

@@ -27,6 +27,7 @@ function clampScore(value: number): number {
   return Math.max(0, Math.min(100, Number(value.toFixed(2))));
 }
 
+    // eslint-disable-next-line complexity -- complexity 15, refactor candidate
 export function assessDeprecatedReadiness(
   principle: PrincipleLifecycleEvidence,
   precomputedRuleMetrics?: Record<string, RuleMetricResult>,

package/src/core/principle-internalization/principle-lifecycle-service.ts

@@ -38,6 +38,7 @@ export interface PrincipleLifecycleAssessment {
   routeRecommendation: InternalizationRouteRecommendation;
 }
 
+    // eslint-disable-next-line complexity -- complexity 15, refactor candidate
 function createValueMetrics(
   principle: PrincipleLifecycleEvidence,
   adherence: PrincipleAdherenceResult,

package/src/core/principle-tree-migration.ts

@@ -9,8 +9,6 @@
  *   - Or run manually: node scripts/migrate-principle-tree.mjs <workspace-dir>
  */
 
-import * as fs from 'fs';
-import * as path from 'path';
 import {
   loadLedger,
   saveLedger,
@@ -23,11 +21,11 @@ export interface PrincipleTreeMigrationResult {
   migratedCount: number;
   skippedCount: number;
   errorCount: number;
-  details: Array<{
+  details: {
     principleId: string;
     status: 'migrated' | 'skipped' | 'error';
     reason?: string;
-  }>;
+  }[];
 }
 
 /**
@@ -94,6 +92,7 @@ function mapInternalizationStatusToPrincipleStatus(
  * This function is idempotent: it only migrates principles that don't exist
  * in tree.principles yet.
  */
+    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
 export function migratePrincipleTree(
   stateDir: string,
   workspaceDir?: string

package/src/core/replay-engine.ts

@@ -236,6 +236,7 @@ export class ReplayEngine {
     };
   }
 
+    // eslint-disable-next-line complexity -- complexity 13, refactor candidate
   private _buildRuleHostInput(sample: ReplaySample): RuleHostInput | null {
     const snapshot = getNocturnalSessionSnapshot(
       TrajectoryRegistry.get(this.workspaceDir),
@@ -294,6 +295,7 @@ export class ReplayEngine {
   }
 
    
+    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
   private _selectToolCall(
     snapshot: NocturnalSessionSnapshot,
     classification: SampleClassification,
@@ -380,7 +382,9 @@ export class ReplayEngine {
     return toolCall.outcome === 'success' ? 'safe' : 'normal';
   }
 
+    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
    
+    // eslint-disable-next-line complexity -- complexity 11
   private _scoreEvaluation(
     sample: ReplaySample,
     result: RuleHostResult,

package/src/core/risk-calculator.ts

@@ -9,6 +9,7 @@ export interface FileModification {
     params: Record<string, unknown>;
 }
 
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
 export function estimateLineChanges(modification: FileModification): number {
     const { toolName, params } = modification;
     

package/src/core/rule-host.ts

@@ -59,6 +59,7 @@ export class RuleHost {
    *   - { decision: 'block', ... } when any implementation returns block (short-circuits)
    *   - { decision: 'requireApproval', ... } when any implementation returns requireApproval
    */
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
   evaluate(input: RuleHostInput): RuleHostResult | undefined {
     try {
       // Load active code implementations from the ledger
@@ -181,6 +182,7 @@ export class RuleHost {
    * Uses the shared isolated runtime loader so candidate code does not execute
    * in the host global realm.
    */
+    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
   private _loadSingleImplementation(
     impl: Implementation
   ): LoadedImplementation | null {

package/src/core/session-tracker.ts

@@ -243,6 +243,7 @@ export function trackToolRead(sessionId: string, filePath: string, workspaceDir?
 }
 
  
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
 export function trackLlmOutput(sessionId: string, usage: TokenUsage | undefined, config?: PainConfig, workspaceDir?: string, sessionKey?: string, trigger?: string): SessionState {
     const state = getOrCreateSession(sessionId, workspaceDir, sessionKey, trigger);
     state.llmTurns += 1;
@@ -282,6 +283,7 @@ export function trackLlmOutput(sessionId: string, usage: TokenUsage | undefined,
  * Tracks physical friction based on tool execution failures.
  */
  
+    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
 export function trackFriction(
     sessionId: string,
     deltaF: number,

package/src/core/thinking-models.ts

@@ -202,6 +202,7 @@ let _cachedWorkspace: string | null = null;
  *
  * @param workspaceDir Optional. If provided, loads from that workspace's THINKING_OS.md.
  */
+    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
 export function listThinkingModels(workspaceDir?: string): ThinkingModelDefinition[] {
   const cacheKey = workspaceDir ?? '__global__';
   if (_cachedDefinitions && _cachedWorkspace === cacheKey) {

package/src/core/thinking-os-parser.ts

@@ -45,10 +45,10 @@ export function parseThinkingOsMd(content: string): ThinkingOsDirective[] {
   // Match all <directive ...> ... </directive> blocks
   const directiveRegex = /<directive\s+([^>]*)>([\s\S]*?)<\/directive>/gi;
    
-  let match: RegExpExecArray | null = null;
+  let _match: RegExpExecArray | null = null;
 
-  while ((match = directiveRegex.exec(content)) !== null) {
-    const [, attrs, body] = match;
+  while ((_match = directiveRegex.exec(content)) !== null) {
+    const [, attrs, body] = _match;
 
     const idMatch = /id="([^"]+)"/i.exec(attrs);
     const nameMatch = /name="([^"]+)"/i.exec(attrs);

package/src/core/trajectory.ts

@@ -442,6 +442,7 @@ export class TrajectoryDatabase {
     const now = nowIso();
     // Cast to V2 to access new fields
     const v2Updates = updates;
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
     this.withWrite(() => {
       const setClauses: string[] = ['updated_at = ?'];
       const values: unknown[] = [now];
@@ -554,6 +555,7 @@ export class TrajectoryDatabase {
       LIMIT ? OFFSET ?
     `).all(...values, limit, offset) as Record<string, unknown>[];
 
+    // eslint-disable-next-line complexity -- complexity 13, refactor candidate
     return rows.map((row) => ({
       id: Number(row.id),
       taskId: String(row.task_id),
@@ -625,6 +627,7 @@ export class TrajectoryDatabase {
    * Returns: Analytics data aggregated from trajectory database.
    * Not: Runtime truth or real-time queue state.
    */
+    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
   getEvolutionTaskByTraceId(traceId: string): EvolutionTaskRecord | null {
     const row = this.db.prepare(`
       SELECT id, task_id, trace_id, source, reason, score, status,
@@ -775,6 +778,7 @@ export class TrajectoryDatabase {
       WHERE session_id = ?
       ORDER BY id ASC
     `).all(sessionId) as Record<string, unknown>[];
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
 
     return rows.map((row) => {
       // Extract filePath from params_json if present

package/src/hooks/bash-risk.ts

@@ -66,7 +66,7 @@ export function analyzeBashCommand(
   // - Word joiner (U+2060)
   // - Zero-width invisible separator (U+FEFF)
    
-  const ZERO_WIDTH_CHARS = /[\u200B\u200C\u200D\u2060\uFEFF]/g;
+  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`);
     return 'dangerous'; // Fail-closed: zero-width chars are suspicious

package/src/hooks/gfi-gate.ts

@@ -65,7 +65,7 @@ function block(
     sessionId,
     blockSource: 'gfi-gate',
   }, logger ||  
-  { warn: () => {}, error: () => {} } as const);
+  { warn: () => { /* no-op */ }, error: () => { /* no-op */ } } as const);
 }
 
  

package/src/hooks/lifecycle-routing.ts

@@ -66,6 +66,7 @@ export type LifecycleIntent = 'promote' | 'disable' | 'rollback' | null;
  * Detect implementation lifecycle intent from user message.
  * Returns the detected intent type or null.
  */
+    // eslint-disable-next-line complexity -- complexity 13, refactor candidate
 export function detectLifecycleIntent(message: string): LifecycleIntent {
   // Check promote patterns
   for (const p of PROMOTE_PATTERNS_EN) {

package/src/hooks/pain.ts

@@ -191,7 +191,7 @@ export function handleAfterToolCall(
     // 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;
+    const toolFailureGfi = session?.gfiBySource?.tool_failure || 0;
     
     let resetState: SessionState;
     if (toolFailureGfi > 0) {
@@ -391,6 +391,7 @@ export function handleAfterToolCall(
   });
 }
 
+    // eslint-disable-next-line complexity -- complexity 15, refactor candidate
 function extractErrorType(error: unknown): string {
   if (!error) return 'Unknown';
   const msg = String(error);

package/src/hooks/prompt.ts

@@ -10,6 +10,7 @@ import { extractSummary, getHistoryVersions, parseWorkingMemorySection, workingM
 import { EmpathyObserverWorkflowManager, empathyObserverWorkflowSpec, isExpectedSubagentError } from '../service/subagent-workflow/index.js';
 import { PathResolver } from '../core/path-resolver.js';
 import { isSubagentRuntimeAvailable } from '../utils/subagent-probe.js';
+import { getPendingDiagnosticianTasks } from '../core/diagnostician-task-store.js';
 import {
   matchEmpathyKeywords,
   loadKeywordStore,
@@ -201,6 +202,7 @@ export function loadContextInjectionConfig(workspaceDir: string): ContextInjecti
  * Falls back to main model if no diagnostician model is configured
  * @internal Helper for model configuration resolution
  */
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
 export function getDiagnosticianModel(api: PromptHookApi | null, logger?: PluginLogger): string {
   // Determines logger: prefer api.logger, fallback to provided logger
   // 1. getDiagnosticianModel(api) - uses api.logger
@@ -366,7 +368,7 @@ export async function handleBeforePromptBuild(
   // prependContext: Only short dynamic directives: evolutionDirective + heartbeat
 
    
-  let prependSystemContext = '';
+  let prependSystemContext: string;
   let prependContext = '';
   let appendSystemContext = '';
 
@@ -644,11 +646,44 @@ ACTION: Run self-audit. If stable, reply ONLY with "HEARTBEAT_OK".
         logger?.error(`[PD:Prompt] Failed to read HEARTBEAT: ${String(e)}`);
       }
     }
+
+    // ──── 4b. Inject pending diagnostician tasks ────
+    // FIX (#283): The evolution worker writes pain diagnosis tasks to
+    // diagnostician_tasks.json. The heartbeat prompt hook must read and inject
+    // them so the LLM (acting as diagnostician) can process them.
+    try {
+      const pendingTasks = getPendingDiagnosticianTasks(wctx.stateDir);
+      if (pendingTasks.length > 0) {
+        const taskBlocks = pendingTasks
+          .slice(0, 3)
+          .map(({ id, task }) => `<diagnostician_task id="${id}">\n${task.prompt}\n</diagnostician_task>`)
+          .join('\n\n');
+
+        const pendingCount = pendingTasks.length;
+        const processingNote = pendingCount > 3
+          ? `\n\nNOTE: ${pendingCount - 3} more tasks are queued. Process these 3 first; remaining tasks will be handled on subsequent heartbeats.`
+          : '';
+
+        prependContext += `<diagnostician_tasks pending="${pendingCount}">
+You are acting as a **Pain Diagnostician**. Process the following task(s) by:
+1. Analyzing the pain signal and its context
+2. Identifying the root cause and violated principles
+3. Writing a completion marker file: .evolution_complete_<TASK_ID>
+4. Writing a diagnostic report: .diagnostician_report_<TASK_ID>.json
+
+${taskBlocks}${processingNote}
+</diagnostician_tasks>\n`;
+
+        logger?.info?.(`[PD:Prompt] Injected ${Math.min(pendingCount, 3)}/${pendingCount} pending diagnostician task(s) into heartbeat prompt`);
+      }
+    } catch (e) {
+      logger?.warn?.(`[PD:Prompt] Failed to read diagnostician tasks: ${String(e)}`);
+    }
   }
 
   // ──── 6. Dynamic Attitude Matrix (based on GFI) ────
    
-  let attitudeDirective = '';
+  let attitudeDirective: string;
   const currentGfi = session?.currentGfi || 0;
   
   if (currentGfi >= 70) {

package/src/hooks/subagent.ts

@@ -25,7 +25,7 @@ function createWorkflowManagerForType(
         warn: (m: string) => logger.warn(String(m)),
         error: (m: string) => logger.error(String(m)),
          
-        debug: () => {},
+        debug: () => { /* no-op */ },
     } as unknown as PluginLogger;
 
     switch (workflowType) {

package/src/hooks/trajectory-collector.ts

@@ -212,6 +212,7 @@ export function handleLlmOutput(
  * 消息写入前的处理
  * 记录：用户/助手消息内容
  */
+    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
 export function handleBeforeMessageWrite(
   event: PluginHookBeforeMessageWriteEvent,
   ctx: PluginHookAgentContext & { workspaceDir?: string }

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

@@ -578,6 +578,7 @@ export function createPrinciplesConsoleRoutes(api: OpenClawPluginApi): OpenClawP
     path: ROUTE_PREFIX,
     auth: 'plugin',
     match: 'prefix',
+    // eslint-disable-next-line complexity -- complexity 13, refactor candidate
     async handler(req, res) {
       if (!api.rootDir) { text(res, 500, 'Plugin rootDir not available'); return true; }
       const url = new URL(req.url || ROUTE_PREFIX, 'http://127.0.0.1');
@@ -640,6 +641,7 @@ export function createPrinciplesConsoleRoute(api: OpenClawPluginApi): OpenClawPl
     path: ROUTE_PREFIX,
     auth: 'plugin',
     match: 'prefix',
+    // eslint-disable-next-line complexity -- complexity 15, refactor candidate
     async handler(req, res) {
       if (!api.rootDir) { text(res, 500, 'Plugin rootDir not available'); return true; }
       const url = new URL(req.url || ROUTE_PREFIX, 'http://127.0.0.1');

package/src/index.ts

@@ -118,7 +118,7 @@ function computeRuntimeShadowTaskFingerprint(event: PluginHookSubagentSpawningEv
   return crypto.createHash('sha256').update(JSON.stringify(payload)).digest('hex').slice(0, 16);
 }
 
-function resolveCommandWorkspaceDirStrict(
+function _resolveCommandWorkspaceDirStrict(
   api: OpenClawPluginApi,
   ctx: WorkspaceResolutionContext,
 ): string {

package/src/service/central-database.ts

@@ -200,6 +200,7 @@ export class CentralDatabase {
   /**
    * Sync data from a single workspace into the central database
    */
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
   syncWorkspace(workspaceName: string): number {
     const workspace = this.workspaces.find(w => w.name === workspaceName);
     if (!workspace) {
@@ -713,6 +714,7 @@ export class CentralDatabase {
       syncEnabled: c.sync_enabled === 1,
     }));
   }
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
 
   updateWorkspaceConfig(
     workspaceName: string,

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

@@ -18,6 +18,7 @@ let centralDb: CentralDatabase | null = null;
  */
 const DEFAULT_SYNC_INTERVAL_MS = 5 * 60 * 1000;
 
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
 async function runSyncCycle(): Promise<void> {
   if (!centralDb) {
     logger?.warn?.('[PD:CentralSync] CentralDatabase not initialized, skipping sync');

package/src/service/control-ui-query-service.ts

@@ -255,6 +255,7 @@ export class ControlUiQueryService {
     this.uiDb.dispose();
   }
 
+    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
   getOverview(days = 30): OverviewResponse {
     const stats = this.trajectory.getDataStats();
     const regressionRows = this.uiDb.all<{
@@ -399,6 +400,7 @@ export class ControlUiQueryService {
     };
   }
 
+    // eslint-disable-next-line complexity -- complexity 13, refactor candidate
   listSamples(filters: SampleListFilters = {}): SamplesResponse {
     const page = Math.max(1, Number(filters.page ?? 1));
     const pageSize = clampPageSize(filters.pageSize);

package/src/service/event-log-auditor.ts

@@ -137,6 +137,7 @@ function countAllHooks(filePath: string): Record<string, number> {
  * @param openclawDir - Base OpenClaw directory (e.g., ~/.openclaw)
  * @param expectedToolHooks - Hook names that should appear in the primary workspace
  */
+    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
 export async function auditEventLogs(
   openclawDir: string,
   expectedToolHooks: string[] = ['before_tool_call', 'after_tool_call'],
@@ -209,6 +210,7 @@ export async function auditEventLogs(
 /**
  * Format audit report for display.
  */
+    // eslint-disable-next-line complexity -- complexity 13, refactor candidate
 export function formatAuditReport(report: AuditReport): string {
   const lines: string[] = [];
   

package/src/service/evolution-worker.ts

@@ -382,7 +382,7 @@ function buildFallbackNocturnalSnapshot(
             // #246-fix: Use minToolCalls=0 to avoid filtering out sessions with 0 tool calls.
             // The pain-triggering session may have no tool calls but still be worth tracking.
             const summaries = extractor.listRecentNocturnalCandidateSessions({ limit: 300, minToolCalls: 0 });
-            const match = summaries.find(s => s.sessionId === painContext.mostRecent!.sessionId);
+            const match = summaries.find(s => s.sessionId === painContext.mostRecent?.sessionId);
             if (match) {
                 realStats = {
                     totalAssistantTurns: match.assistantTurnCount,
@@ -1873,6 +1873,7 @@ async function processEvolutionQueue(wctx: WorkspaceContext, logger: PluginLogge
     }
 }
 
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
 async function processDetectionQueue(wctx: WorkspaceContext, api: OpenClawPluginApi, eventLog: EventLog) {
     const {logger} = api;
     try {

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

@@ -337,6 +337,7 @@ export class HealthQueryService {
     }));
   }
 
+    // eslint-disable-next-line complexity -- complexity 13, refactor candidate
   getGateStats(): {
     today: {
       gfiBlocks: number;
@@ -553,8 +554,8 @@ export class HealthQueryService {
     const streamPath = resolvePdPath(this.workspaceDir, 'EVOLUTION_STREAM');
     if (!fs.existsSync(streamPath)) return [];
 
-     
-    let lines: string[] = [];
+    // eslint-disable-next-line @typescript-eslint/init-declarations
+    let lines: string[];
     try {
       const raw = fs.readFileSync(streamPath, 'utf8').trim();
       if (!raw) return [];
@@ -565,8 +566,9 @@ export class HealthQueryService {
 
     const records: RecentPrincipleChange[] = [];
     for (const line of lines) {
-       
-      let event: EvolutionStreamRecord | null = null;
+
+      // eslint-disable-next-line @typescript-eslint/init-declarations
+      let event: EvolutionStreamRecord | null;
       try {
         event = JSON.parse(line) as EvolutionStreamRecord;
       } catch {
@@ -657,6 +659,7 @@ export class HealthQueryService {
     return null;
   }
 
+    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
   private readNocturnalTraining(): {
     queue: { pending: number; inProgress: number; completed: number };
     trinityRecords: { artifactId: string; status: string; createdAt: string }[];
@@ -784,6 +787,8 @@ export class HealthQueryService {
   }
 
    
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
+    // eslint-disable-next-line complexity -- complexity 15, refactor candidate
   private getEventDedupKey(entry: EventLogEntry): string {
     const eventId = typeof entry.data?.eventId === 'string' ? entry.data.eventId : null;
     if (eventId) {
@@ -854,6 +859,8 @@ export class HealthQueryService {
   }
 
    
+    // eslint-disable-next-line complexity -- complexity 13, refactor candidate
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private resolveGateType(row: GateBlockRow): string {
     if (typeof row.gate_type === 'string' && row.gate_type.trim().length > 0) {
       return row.gate_type;
@@ -878,6 +885,7 @@ export class HealthQueryService {
   }
 
    
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private scoreToStatus(score: number): string {
     if (score >= 70) return 'healthy';
     if (score >= 40) return 'warning';
@@ -885,6 +893,7 @@ export class HealthQueryService {
   }
 
    
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private evolutionToStatus(tier: string, points: number): string {
     const lower = tier.toLowerCase();
     if (lower === 'forest' || lower === 'tree') return 'healthy';
@@ -893,6 +902,7 @@ export class HealthQueryService {
   }
 
    
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private safeListFiles(dirPath: string, predicate: (_name: string) => boolean): string[] {
     if (!fs.existsSync(dirPath)) return [];
     try {
@@ -905,6 +915,7 @@ export class HealthQueryService {
   }
 
    
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private readJsonFile<T>(filePath: string, fallback: T): T {
     if (!fs.existsSync(filePath)) return fallback;
     try {
@@ -915,11 +926,13 @@ export class HealthQueryService {
   }
 
    
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private asNumber(value: unknown, fallback: number): number {
     return Number.isFinite(value) ? Number(value) : fallback;
   }
 
    
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private asNullableNumber(value: unknown): number | null {
     if (Number.isFinite(value)) return Number(value);
     if (typeof value === 'string' && value.trim().length > 0) {
@@ -954,7 +967,7 @@ export class HealthQueryService {
         dailyGfiPeak,
         today,
       );
-    } catch (err) {
+    } catch {
       // Non-critical: GFI sync failure should not block queries
     }
   }
@@ -981,6 +994,7 @@ export class HealthQueryService {
    * Read the most recent session JSON file from disk.
    * Used to sync GFI from session-tracker's persistence into SQLite.
    */
+    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
   private readLatestSessionFromFile(): SessionState | null {
     const sessionsDir = path.join(this.stateDir, 'sessions');
     if (!fs.existsSync(sessionsDir)) {
@@ -1015,7 +1029,7 @@ export class HealthQueryService {
       }
 
       return latest;
-    } catch (err) {
+    } catch {
       // Non-critical: failure to read session files should not crash the service
       return null;
     }

package/src/service/nocturnal-runtime.ts

@@ -40,6 +40,7 @@ import { withLockAsync } from '../utils/file-lock.js';
  * Excluded (NOT system sessions):
  * - User sessions like agent:main:feishu:user:xxx — third component is channel type
  */
+    // eslint-disable-next-line complexity -- complexity 15, refactor candidate
 function isSystemSession(state: SessionState): boolean {
     const { sessionId, sessionKey, trigger } = state;
 
@@ -269,6 +270,7 @@ async function writeState(stateDir: string, state: NocturnalRuntimeState): Promi
  * @param trajectoryLastActivityAt - Optional trajectory timestamp as secondary guardrail
  * @returns IdleCheckResult with full diagnostic information
  */
+    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
 export function checkWorkspaceIdle(
     workspaceDir: string,
     options: {
@@ -355,6 +357,7 @@ export function checkWorkspaceIdle(
  * @param options - Cooldown configuration options
  * @returns CooldownCheckResult
  */
+    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
 export function checkCooldown(
     stateDir: string,
     principleId?: string,
@@ -555,6 +558,7 @@ export interface PreflightCheckResult {
  * @param idleCheckOverride - Optional override for idle check result (for testing)
  */
  
+    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
 export function checkPreflight(
     workspaceDir: string,
     stateDir: string,

package/src/service/runtime-summary-service.ts

@@ -362,6 +362,7 @@ export class RuntimeSummaryService {
     return { session: sessions[0], reason: 'latest_active' };
   }
 
+    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
   private static mergeSessionSnapshots(
     persistedSessions: PersistedSessionState[],
     workspaceDir: string
@@ -510,7 +511,9 @@ export class RuntimeSummaryService {
       })
       .slice(-MAX_SOURCE_EVENTS)
       .reverse();
+    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
 
+    // eslint-disable-next-line complexity -- complexity 11
     return filtered.map((entry) => {
       if (entry.type === 'pain_signal') {
         return {
@@ -581,6 +584,7 @@ export class RuntimeSummaryService {
     return [...merged.values()].sort((a, b) => (a.ts || '').localeCompare(b.ts || ''));
   }
 
+    // eslint-disable-next-line complexity -- complexity 15, refactor candidate
   private static getEventDedupKey(entry: EventLogEntry): string {
     const eventId = typeof entry.data?.eventId === 'string' ? entry.data.eventId : null;
     if (eventId) {
@@ -637,6 +641,7 @@ export class RuntimeSummaryService {
     return candidate ? new Date(candidate).getTime() : NaN;
   }
 
+    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
   private static buildDirectiveTaskPreview(item: QueueItem): string {
     const task = typeof item.task === 'string' ? item.task.trim() : '';
     if (task && task.toLowerCase() !== 'undefined') {