principles-disciple 1.28.2 → 1.29.0

package/src/core/nocturnal-arbiter.ts

@@ -159,7 +159,7 @@ export interface TrinityStageValidationResult {
  * Validate a Dreamer output contract.
  * Ensures the output is well-formed before passing to Philosopher.
  */
-    // eslint-disable-next-line complexity -- complexity 11, slightly over threshold
+     
 export function validateDreamerOutput(output: unknown): TrinityStageValidationResult {
   const failures: string[] = [];
 
@@ -238,7 +238,7 @@ export function validateDreamerOutput(output: unknown): TrinityStageValidationRe
  * Validate a Philosopher output contract.
  * Ensures the output is well-formed before passing to Scribe.
  */
-    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
+     
 export function validatePhilosopherOutput(output: unknown): TrinityStageValidationResult {
   const failures: string[] = [];
 
@@ -262,7 +262,7 @@ export function validatePhilosopherOutput(output: unknown): TrinityStageValidati
     failures.push('Philosopher output must have a judgments array');
   } else {
     // Validate each judgment
-    // eslint-disable-next-line complexity -- complexity 13, refactor candidate
+     
     obj.judgments.forEach((judgment: unknown, idx: number) => {
       if (judgment === null || judgment === undefined || typeof judgment !== 'object') {
         failures.push(`Philosopher judgment at index ${idx} is not an object`);
@@ -694,6 +694,7 @@ export function parseAndValidateArtifact(
 ): ArbiterResult {
   // Step 1: Parse JSON
    
+  // eslint-disable-next-line @typescript-eslint/init-declarations
   let parsed: unknown;
   try {
     parsed = JSON.parse(jsonString);

package/src/core/nocturnal-candidate-scoring.ts

@@ -293,6 +293,7 @@ export function validateCandidateDiversity(
 
   for (let i = 0; i < candidates.length; i++) {
     for (let j = i + 1; j < candidates.length; j++) {
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       const overlap = computeKeywordOverlap(
         candidates[i].betterDecision ?? '',
         candidates[j].betterDecision ?? '',
@@ -334,7 +335,9 @@ export function validateCandidateDiversity(
  * Returns value between 0 and 1.
  */
 function computeKeywordOverlap(textA: string, textB: string): number {
+  // eslint-disable-next-line @typescript-eslint/no-use-before-define
   const wordsA = extractKeywords(textA);
+  // eslint-disable-next-line @typescript-eslint/no-use-before-define
   const wordsB = extractKeywords(textB);
 
   if (wordsA.length === 0 && wordsB.length === 0) return 0;
@@ -373,6 +376,7 @@ function extractKeywords(text: string): string[] {
  * @returns All scored and ranked candidates
  */
  
+// eslint-disable-next-line @typescript-eslint/max-params
 export function rankCandidates(
   candidates: DreamerCandidate[],
   judgments: PhilosopherJudgment[],
@@ -460,6 +464,7 @@ export function rankCandidates(
  * @returns Tournament result with winner
  */
  
+// eslint-disable-next-line @typescript-eslint/max-params
 export function runTournament(
   candidates: DreamerCandidate[],
   judgments: PhilosopherJudgment[],

package/src/core/nocturnal-compliance.ts

@@ -242,22 +242,31 @@ export function detectOpportunity(principleId: string, session: SessionEvents):
    
   switch (principleId) {
     case 'T-01':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT01Opportunity(session);
     case 'T-02':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT02Opportunity(session);
     case 'T-03':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT03Opportunity(session);
     case 'T-04':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT04Opportunity(session);
     case 'T-05':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT05Opportunity(session);
     case 'T-06':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT06Opportunity(session);
     case 'T-07':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT07Opportunity(session);
     case 'T-08':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT08Opportunity(session);
     case 'T-09':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT09Opportunity(session);
     default:
       return { applicable: false, reason: `Unknown principle: ${principleId}` };
@@ -513,7 +522,7 @@ function detectT09Opportunity(session: SessionEvents): OpportunityMatch {
  * trigger pattern. Since P_* principles don't have T-xx specific detectors,
  * we use the presence of negative signals as violation evidence.
  */
-    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
+     
 export function detectViolation(principleId: string, session: SessionEvents): ViolationMatch {
   // #216: P_* principles (pain-derived) — generic violation detection
   if (principleId.startsWith('P_')) {
@@ -541,22 +550,31 @@ export function detectViolation(principleId: string, session: SessionEvents): Vi
    
   switch (principleId) {
     case 'T-01':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT01Violation(session);
     case 'T-02':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT02Violation(session);
     case 'T-03':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT03Violation(session);
     case 'T-04':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT04Violation(session);
     case 'T-05':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT05Violation(session);
     case 'T-06':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT06Violation(session);
     case 'T-07':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT07Violation(session);
     case 'T-08':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT08Violation(session);
     case 'T-09':
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       return detectT09Violation(session);
     default:
       console.warn(`[PD:Compliance] Unknown principle ID: ${principleId} — treating as no violation. Check for typos (P-001 vs P_001).`);
@@ -915,9 +933,11 @@ export function computeCompliance(
       : 0;
 
   // Compute violationTrend using windows
+  // eslint-disable-next-line @typescript-eslint/no-use-before-define
   const violationTrend = computeViolationTrend(applicableSessions, windowSize);
 
   // Build explanation
+  // eslint-disable-next-line @typescript-eslint/no-use-before-define
   const explanation = buildExplanation(
     principleId,
     applicableOpportunityCount,
@@ -989,6 +1009,7 @@ function computeViolationTrend(
  * Builds a human-readable explanation for the compliance result.
  */
  
+// eslint-disable-next-line @typescript-eslint/max-params
 function buildExplanation(
   principleId: string,
   applicableOpportunityCount: number,

package/src/core/nocturnal-dataset.ts

@@ -284,6 +284,7 @@ function withRegistryLock<T>(workspaceDir: string, fn: (_records: NocturnalDatas
  * @returns RegisterSampleResult
  */
  
+// eslint-disable-next-line @typescript-eslint/max-params
 export function registerSample(
   workspaceDir: string,
   artifact: NocturnalArtifact,
@@ -426,6 +427,7 @@ const VALID_TRANSITIONS: Record<NocturnalReviewStatus, NocturnalReviewStatus[]>
  * @throws Error if transition is invalid
  */
  
+// eslint-disable-next-line @typescript-eslint/max-params
 export function updateReviewStatus(
   workspaceDir: string,
   sampleFingerprint: string,
@@ -645,7 +647,7 @@ export function getDatasetStats(
  * @param targetModelFamily - Default target family for migrated samples
  * @returns Number of newly registered samples
  */
-    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
+     
 export function migrateSampleArtifacts(
   workspaceDir: string,
   targetModelFamily: string | null = null

package/src/core/nocturnal-export.ts

@@ -158,6 +158,7 @@ function computeDatasetFingerprint(sampleFingerprints: string[]): string {
  * Caller guarantees record.targetModelFamily is non-null.
  */
  
+// eslint-disable-next-line @typescript-eslint/max-params
 function serializeORPOSample(
   record: NocturnalDatasetRecord,
   artifact: ReturnType<typeof readDatasetArtifact>,
@@ -166,6 +167,7 @@ function serializeORPOSample(
   datasetFingerprint: string
 ): ORPOSample {
   const now = new Date().toISOString();
+  // eslint-disable-next-line @typescript-eslint/no-use-before-define
   const rejected = buildEvidenceBoundedRejected(artifact, evidenceSummary);
 
   return {
@@ -178,6 +180,7 @@ function serializeORPOSample(
     prompt: rejected,
     chosen: artifact.betterDecision,
     rejected,
+    // eslint-disable-next-line @typescript-eslint/no-use-before-define
     rationale: buildEvidenceBoundedRationale(evidenceSummary),
     datasetMetadata: {
       sampleFingerprint: record.sampleFingerprint,
@@ -292,6 +295,7 @@ export function exportORPOSamples(
   });
 
    
+  // eslint-disable-next-line @typescript-eslint/init-declarations
   let eligibleRecords: typeof allApprovedRecords;
 
   if (targetModelFamily !== undefined && targetModelFamily !== null) {
@@ -341,6 +345,7 @@ export function exportORPOSamples(
 
     // Read artifact (throws on error — distinguishes read failure from missing artifact)
      
+    // eslint-disable-next-line @typescript-eslint/init-declarations
     let artifact;
     try {
       artifact = readDatasetArtifact(workspaceDir, record.sampleFingerprint);

package/src/core/nocturnal-reasoning-deriver.ts

@@ -11,6 +11,7 @@
  * - deriveContextualFactors: Compute contextual factors from snapshot (Plan 02)
  */
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
 import type { NocturnalAssistantTurn, NocturnalToolCall, NocturnalUserTurn, NocturnalSessionSnapshot } from './nocturnal-trajectory-extractor.js';
 import { detectThinkingModelMatches, listThinkingModels } from './thinking-models.js';
 
@@ -143,6 +144,7 @@ export function deriveReasoningChain(assistantTurns: NocturnalAssistantTurn[]):
     // Without thinking tags we cannot extract a genuine reasoning trace, so
     // we fall back to 'low' rather than misleading the downstream pipeline
     // with activation derived from non-thinking patterns in the response text.
+    // eslint-disable-next-line @typescript-eslint/init-declarations
     let confidenceSignal: "high" | "medium" | "low";
     if (thinkingContent.length === 0) {
       confidenceSignal = 'low';
@@ -211,6 +213,7 @@ export function deriveDecisionPoints(
 
   // Binary search: find rightmost assistant turn with createdAt < tcTime
   const findBeforeTurn = (tcTime: number): NocturnalAssistantTurn | undefined => {
+    // eslint-disable-next-line @typescript-eslint/init-declarations
     let lo = 0, hi = sortedTurns.length - 1, result: NocturnalAssistantTurn | undefined;
     while (lo <= hi) {
       const mid = (lo + hi) >>> 1;
@@ -233,7 +236,9 @@ export function deriveDecisionPoints(
       : '';
 
     // On failure, find next assistant turn after tool call
+    // eslint-disable-next-line @typescript-eslint/init-declarations
     let afterReflection: string | undefined;
+    // eslint-disable-next-line @typescript-eslint/init-declarations
     let confidenceDelta: number | undefined;
 
     if (tc.outcome === 'failure') {
@@ -280,7 +285,7 @@ export function deriveDecisionPoints(
  *
  * Empty/missing data returns all-false defaults. Never throws.
  */
-    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
+     
 export function deriveContextualFactors(
   snapshot: NocturnalSessionSnapshot,
 ): DerivedContextualFactors {

package/src/core/nocturnal-snapshot-contract.ts

@@ -53,6 +53,7 @@ export function validateNocturnalSnapshotIngress(
     }
   }
 
+  // eslint-disable-next-line @typescript-eslint/prefer-destructuring
   const stats = value.stats;
   if (!isObjectRecord(stats)) {
     reasons.push('snapshot.stats must be an object');

package/src/core/nocturnal-trinity.ts

@@ -584,6 +584,7 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
     this.cleanupStaleTempDirs();
   }
 
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   isRuntimeAvailable(): boolean {
     return true;
   }
@@ -642,7 +643,7 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
    * runEmbeddedPiAgent does NOT read config.agents.defaults.model —
    * it requires explicit params.provider and params.model.
    */
-    // eslint-disable-next-line complexity -- complexity 12, refactor candidate
+     
   private resolveModel(): { provider: string; model: string } {
     const config = this.loadFullConfig();
     const agents = config?.agents as Record<string, unknown> | undefined;
@@ -681,6 +682,7 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
   /**
    * Extract text from runEmbeddedPiAgent result.
    */
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private extractPayloadText(result: { payloads?: { isError?: boolean; text?: string }[] }): string {
     return (result.payloads ?? [])
       .filter(p => !p.isError)
@@ -690,11 +692,13 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
   }
 
   /** Clamp a value to [0, 1] range — used for LLM-produced scores that may be out of range */
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private clamp01(val: unknown, fallback = 0): number {
     if (typeof val !== 'number' || !Number.isFinite(val)) return fallback;
     return Math.min(1, Math.max(0, val));
   }
 
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private classifyRuntimeError(error: unknown): TrinityRuntimeFailureCode {
     const detail = error instanceof Error ? error.message : String(error);
     return /timeout/i.test(detail) ? 'runtime_timeout' : 'runtime_run_failed';
@@ -742,6 +746,7 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
     } catch (err) {
       return this.buildRuntimeFailureDreamerOutput(this.classifyRuntimeError(err), err);
     } finally {
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
       try { fs.unlinkSync(sessionFile); } catch (err) { this.api.logger?.warn?.(`[Trinity] Failed to delete session file: ${sessionFile}`); }
     }
   }
@@ -786,11 +791,13 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
     } catch (err) {
       return this.buildRuntimeFailurePhilosopherOutput(this.classifyRuntimeError(err), err);
     } finally {
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
       try { fs.unlinkSync(sessionFile); } catch (err) { this.api.logger?.warn?.(`[Trinity] Failed to delete session file: ${sessionFile}`); }
     }
   }
 
 
+  // eslint-disable-next-line @typescript-eslint/max-params
   async invokeScribe(
     dreamerOutput: DreamerOutput,
     philosopherOutput: PhilosopherOutput,
@@ -834,6 +841,7 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
       this.recordFailure(this.classifyRuntimeError(err), err);
       return null;
     } finally {
+      // eslint-disable-next-line @typescript-eslint/no-unused-vars
       try { fs.unlinkSync(sessionFile); } catch (err) { this.api.logger?.warn?.(`[Trinity] Failed to delete session file: ${sessionFile}`); }
     }
   }
@@ -857,6 +865,7 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
   // ---------------------------------------------------------------------------
 
    
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private buildDreamerPrompt(
     snapshot: NocturnalSessionSnapshot,
     principleId: string,
@@ -953,6 +962,7 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
   }
 
    
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private buildPhilosopherPrompt(
     dreamerOutput: DreamerOutput,
     principleId: string,
@@ -1035,8 +1045,10 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
 
     return sections.join('\n');
   }
+ 
 
    
+  // eslint-disable-next-line @typescript-eslint/max-params, @typescript-eslint/class-methods-use-this
   private buildScribePrompt(
     dreamerOutput: DreamerOutput,
     philosopherOutput: PhilosopherOutput,
@@ -1245,10 +1257,12 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
               falsePositiveEstimate?: number;
               implementationComplexity: string;
               breakingChangeRisk: boolean;
+             
             } = {
               implementationComplexity: (risks.implementationComplexity as string) ?? 'medium',
               breakingChangeRisk: Boolean(risks.breakingChangeRisk),
             };
+            // eslint-disable-next-line @typescript-eslint/no-unnecessary-type-assertion
             if (hasFp) risksObj.falsePositiveEstimate = this.clamp01(fp as number);
             return { risks: risksObj };
           })() : {}),
@@ -1292,6 +1306,7 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
   }
 
    
+  // eslint-disable-next-line @typescript-eslint/max-params
   private parseScribeOutput(
     text: string,
     snapshot: NocturnalSessionSnapshot,
@@ -1360,6 +1375,7 @@ export class OpenClawTrinityRuntimeAdapter implements TrinityRuntimeAdapter {
    * Extract JSON object from text that may contain markdown code blocks.
    */
    
+  // eslint-disable-next-line @typescript-eslint/class-methods-use-this
   private extractJson(text: string): string | null {
     // Try direct parse first
     try {
@@ -1699,7 +1715,7 @@ export interface TrinityResult {
  * In production, this would call the actual Dreamer subagent.
  * The stub generates plausible candidates based on snapshot signals.
  */
-    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
+     
 export function invokeStubDreamer(
   snapshot: NocturnalSessionSnapshot,
   principleId: string,
@@ -1896,7 +1912,9 @@ export function invokeStubPhilosopher(
 
     // Deterministic 6D scores based on strategic perspective (Phase 35 D-07 mapping)
     const perspective = candidate.strategicPerspective;
+    // eslint-disable-next-line @typescript-eslint/init-declarations
     let sixDScores: Philosopher6DScores;
+    // eslint-disable-next-line @typescript-eslint/init-declarations
     let riskAssessment: PhilosopherRiskAssessment;
 
     if (perspective === 'conservative_fix') {
@@ -1996,6 +2014,7 @@ export function invokeStubPhilosopher(
  * The stub uses tournament selection (scoring + thresholds) to pick the winner.
  */
  
+// eslint-disable-next-line @typescript-eslint/max-params
 export function invokeStubScribe(
   dreamerOutput: DreamerOutput,
   philosopherOutput: PhilosopherOutput,
@@ -2074,6 +2093,7 @@ export function runTrinity(options: RunTrinityOptions): TrinityResult {
   // Stub path: use synchronous stub implementations
   if (config.useStubs) {
      
+    // eslint-disable-next-line @typescript-eslint/no-use-before-define
     return runTrinityWithStubs(snapshot, principleId, config);
   }
 
@@ -2113,6 +2133,7 @@ export async function runTrinityAsync(options: RunTrinityOptions): Promise<Trini
   if (config.useStubs) {
     // Stub path: use synchronous stubs
      
+    // eslint-disable-next-line @typescript-eslint/no-use-before-define
     return runTrinityWithStubs(snapshot, principleId, config);
   }
 
@@ -2250,7 +2271,7 @@ export async function runTrinityAsync(options: RunTrinityOptions): Promise<Trini
 
 /**
  * Internal: Run Trinity chain with stub implementations (synchronous).
-    // eslint-disable-next-line complexity -- complexity 14, refactor candidate
+    // eslint-disable-next-line complexity, @typescript-eslint/class-methods-use-this -- complexity 14, refactor candidate
  */
 function runTrinityWithStubs(
   snapshot: NocturnalSessionSnapshot,

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

@@ -54,6 +54,7 @@ async function safeTail(filePath: string): Promise<string[]> {
   try {
     // Check existence and stats asynchronously
      
+    // eslint-disable-next-line @typescript-eslint/init-declarations
     let stat: fs.Stats;
     try {
       stat = await fsPromises.stat(filePath);
@@ -203,7 +204,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',
@@ -236,6 +237,7 @@ export async function extractRecentConversation(
 /**
  * Extracts failed tool call context with argument correlation.
  */
+// eslint-disable-next-line @typescript-eslint/max-params
 export async function extractFailedToolContext(
   sessionId: string,
   agentId: string,

package/src/core/pain.ts

@@ -98,6 +98,7 @@ export function validatePainFlag(data: Record<string, string>): string[] {
   return missing;
 }
 
+// eslint-disable-next-line @typescript-eslint/max-params
 export function computePainScore(rc: number, isSpiral: boolean, missingTestCommand: boolean, softScore: number, projectDir?: string): number {
   let score = Math.max(0, softScore || 0);
   
@@ -211,6 +212,7 @@ export function readPainFlagData(projectDir: string): Record<string, string> {
 
     // Detect JSON format (wrong — should be KV)
     if (content.startsWith('{')) {
+      // eslint-disable-next-line @typescript-eslint/init-declarations
       let json: Record<string, unknown>;
       try {
         json = JSON.parse(content);
@@ -279,7 +281,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
+    // eslint-disable-next-line @typescript-eslint/max-params -- complexity 12, refactor candidate
 export function trackPrincipleValue(
   workspaceDir: string,
   painData: { reason?: string; source?: string; score?: string },

package/src/core/path-resolver.ts

@@ -420,18 +420,24 @@ export function resolveWorkspaceDirFromApi(
     if (!api) return undefined;
 
     // 1. Official API: api.runtime.agent.resolveAgentWorkspaceDir
-     
+
     const officialAgent = (api.runtime as { agent?: { resolveAgentWorkspaceDir?: (cfg: unknown, id: string) => string } }).agent;
-     
+
     if (officialAgent?.resolveAgentWorkspaceDir) {
         try {
             return officialAgent.resolveAgentWorkspaceDir(api.config, agentId ?? 'main');
         } catch {
-            // Fall through to PathResolver
+            // Fall through to config check
         }
     }
 
-    // 2. Fallback: PathResolver (PD_WORKSPACE_DIR env, config file, default)
+    // 2. Direct config workspaceDir (for tests and programmatic usage)
+    const cfgWorkspaceDir = (api.config as { workspaceDir?: string })?.workspaceDir;
+    if (cfgWorkspaceDir && cfgWorkspaceDir.trim()) {
+        return cfgWorkspaceDir.trim();
+    }
+
+    // 3. Fallback: PathResolver (PD_WORKSPACE_DIR env, config file, default)
     try {
         const pr = new PathResolver();
         return pr.getWorkspaceDir();

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

@@ -103,7 +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>();
@@ -160,6 +160,7 @@ function buildCronJob(
     payload: {
       kind: 'agentTurn',
        
+      // eslint-disable-next-line @typescript-eslint/no-use-before-define
       message: buildTaskPrompt(task, logger),
       lightContext: task.execution.lightContext ?? true,
       timeoutSeconds: task.execution.timeoutSeconds ?? 120,
@@ -294,6 +295,7 @@ export async function reconcilePDTasks(
 
   const cronStore = await readCronStore(logger);
    
+  // eslint-disable-next-line @typescript-eslint/no-use-before-define
   const healthUpdated = healthCheck(declared, cronStore, logger);
   const actions = diff(healthUpdated, cronStore.jobs);
 

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

@@ -56,6 +56,7 @@ export function initTaskMeta(task: PDTaskSpec): PDTaskSpec {
 }
 
  
+// eslint-disable-next-line @typescript-eslint/max-params
 export function updateSyncMeta(
   task: PDTaskSpec,
   status: 'ok' | 'error',

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

@@ -27,7 +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>,
@@ -65,6 +65,7 @@ export function assessDeprecatedReadiness(
   );
 
    
+  // eslint-disable-next-line @typescript-eslint/init-declarations
   let status: DeprecatedReadinessStatus;
   if (blockingReasons.length === 0 && stableCoverageRatio === 1) {
     status = 'ready';

package/src/core/principle-training-state.ts

@@ -63,6 +63,7 @@ export function createDefaultPrincipleState(principleId: string): PrincipleTrain
 
 export function loadStore(stateDir: string): PrincipleTrainingStore {
    
+  // eslint-disable-next-line @typescript-eslint/no-use-before-define
   return ledgerTrainingStore(stateDir);
 }
 
@@ -76,6 +77,7 @@ export function saveStore(stateDir: string, store: PrincipleTrainingStore): void
 
 export async function loadStoreAsync(stateDir: string): Promise<PrincipleTrainingStore> {
    
+  // eslint-disable-next-line @typescript-eslint/no-use-before-define
   return ledgerTrainingStore(stateDir);
 }
 

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

@@ -78,6 +78,7 @@ function isRecord(value: unknown): value is Record<string, unknown> {
 }
 
  
+// eslint-disable-next-line @typescript-eslint/max-params
 function clampFloat(value: unknown, min: number, max: number, fallback: number): number {
   if (typeof value !== 'number' || !Number.isFinite(value)) {
     return fallback;
@@ -86,6 +87,7 @@ function clampFloat(value: unknown, min: number, max: number, fallback: number):
 }
 
  
+// eslint-disable-next-line @typescript-eslint/max-params
 function clampInt(value: unknown, min: number, max: number, fallback: number): number {
   if (typeof value !== 'number' || !Number.isFinite(value)) {
     return fallback;
@@ -318,6 +320,7 @@ function writeLedgerUnlocked(filePath: string, store: HybridLedgerStore): void {
  
 function mutateLedger<T>(stateDir: string, mutate: (store: HybridLedgerStore) => T): T {
    
+  // eslint-disable-next-line @typescript-eslint/no-use-before-define
   const filePath = getLedgerFilePath(stateDir);
   return withLock(filePath, () => {
     const store = readLedgerFromFile(filePath);
@@ -330,6 +333,7 @@ function mutateLedger<T>(stateDir: string, mutate: (store: HybridLedgerStore) =>
  
 async function mutateLedgerAsync<T>(stateDir: string, mutate: (store: HybridLedgerStore) => Promise<T>): Promise<T> {
    
+  // eslint-disable-next-line @typescript-eslint/no-use-before-define
   const filePath = getLedgerFilePath(stateDir);
   return withLockAsync(filePath, async () => {
     const store = readLedgerFromFile(filePath);

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

@@ -50,6 +50,7 @@ function trainingStateToTreePrinciple(
     text: `Principle ${principleId}`, // Minimal text, will be enriched from PRINCIPLES.md if available
     triggerPattern: '', // Unknown from legacy data
     action: '', // Unknown from legacy data
+    // eslint-disable-next-line @typescript-eslint/no-use-before-define
     status: mapInternalizationStatusToPrincipleStatus(state.internalizationStatus),
     priority: 'P1', // Default priority
     scope: 'general',
@@ -92,7 +93,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