@reftrixmcp/mcp-server 0.1.7 → 0.2.0

package/src/tools/system-health.ts

@@ -14,6 +14,7 @@
  */
 
 import { logger, isDevelopment } from "../utils/logger";
+import { sanitizeErrorMessage } from "../utils/sanitize-error";
 import { getToolMetricsStats } from "../router";
 import type { MetricsStats } from "../services/metrics-collector";
 import { getEmbeddingCacheStats } from "../services/layout-embedding.service";
@@ -453,12 +454,10 @@ export async function systemHealthHandler(input?: unknown): Promise<SystemHealth
           };
         }
       } catch (cacheError) {
-        if (isDevelopment()) {
-          logger.warn("[MCP Tool] Failed to get embedding cache stats", {
-            error: cacheError instanceof Error ? cacheError.message : "Unknown error",
-            requestId,
-          });
-        }
+        logger.warn("[MCP Tool] Failed to get embedding cache stats", {
+          error: cacheError instanceof Error ? cacheError.message : "Unknown error",
+          requestId,
+        });
       }
     }
 
@@ -489,15 +488,13 @@ export async function systemHealthHandler(input?: unknown): Promise<SystemHealth
           disk_usage_bytes: cssStats.diskUsageBytes,
         };
       } catch (cssAnalysisCacheError) {
-        if (isDevelopment()) {
-          logger.warn("[MCP Tool] Failed to get CSS analysis cache stats", {
-            error:
-              cssAnalysisCacheError instanceof Error
-                ? cssAnalysisCacheError.message
-                : "Unknown error",
-            requestId,
-          });
-        }
+        logger.warn("[MCP Tool] Failed to get CSS analysis cache stats", {
+          error:
+            cssAnalysisCacheError instanceof Error
+              ? cssAnalysisCacheError.message
+              : "Unknown error",
+          requestId,
+        });
       }
     }
 
@@ -604,13 +601,11 @@ export async function systemHealthHandler(input?: unknown): Promise<SystemHealth
           });
         }
       } catch (visionHardwareError) {
-        if (isDevelopment()) {
-          logger.warn("[MCP Tool] Failed to get vision hardware status", {
-            error:
-              visionHardwareError instanceof Error ? visionHardwareError.message : "Unknown error",
-            requestId,
-          });
-        }
+        logger.warn("[MCP Tool] Failed to get vision hardware status", {
+          error:
+            visionHardwareError instanceof Error ? visionHardwareError.message : "Unknown error",
+          requestId,
+        });
         // エラー時は環境変数のみを報告
         healthData.vision_hardware = {
           force_cpu_mode: process.env["VISION_FORCE_CPU_MODE"]?.toLowerCase() === "true",
@@ -618,8 +613,7 @@ export async function systemHealthHandler(input?: unknown): Promise<SystemHealth
           detected_type: "CPU", // 安全側
           vram_bytes: 0,
           is_gpu_available: false,
-          detection_error:
-            visionHardwareError instanceof Error ? visionHardwareError.message : "Unknown error",
+          detection_error: sanitizeErrorMessage(visionHardwareError),
           ollama_url: "http://localhost:11434",
         };
       }
@@ -648,19 +642,16 @@ export async function systemHealthHandler(input?: unknown): Promise<SystemHealth
   } catch (error) {
     const processingTime = Date.now() - startTime;
 
-    if (isDevelopment()) {
-      logger.error("[MCP Tool] system.health error", {
-        error: error instanceof Error ? error.message : "Unknown error",
-        stack: error instanceof Error ? error.stack : undefined,
-        requestId,
-      });
-    }
+    logger.error("[MCP Tool] system.health error", {
+      error: error instanceof Error ? error.message : "Unknown error",
+      requestId,
+    });
 
     return createErrorResponseWithRequestId(
       "HEALTH_CHECK_ERROR",
-      `ヘルスチェック中にエラーが発生しました: ${error instanceof Error ? error.message : "Unknown error"}`,
+      sanitizeErrorMessage(error),
       requestId,
-      error instanceof Error ? { stack: error.stack } : undefined,
+      undefined,
       { processing_time_ms: processingTime }
     );
   }

package/src/utils/di-factory.ts

@@ -0,0 +1,64 @@
+// SPDX-FileCopyrightText: 2026 TKMD and Reftrix Contributors
+// SPDX-License-Identifier: AGPL-3.0-only
+
+/**
+ * 汎用DIファクトリーユーティリティ / Generic DI Factory Utility
+ *
+ * MCPツールハンドラーで使用されるDIファクトリーボイラープレートを共通化する。
+ * Centralizes the DI factory boilerplate used across MCP tool handlers.
+ *
+ * @example
+ * ```typescript
+ * // Before (9 lines per factory):
+ * let serviceFactory: (() => IMyService) | null = null;
+ * export function setMyServiceFactory(factory: () => IMyService): void {
+ *   serviceFactory = factory;
+ * }
+ * export function resetMyServiceFactory(): void {
+ *   serviceFactory = null;
+ * }
+ *
+ * // After (1 line per factory):
+ * const myServiceDI = createDIFactory<IMyService>("MyService");
+ * export const setMyServiceFactory = myServiceDI.set;
+ * export const resetMyServiceFactory = myServiceDI.reset;
+ * ```
+ */
+
+/**
+ * DIファクトリーのインターフェース / DI Factory interface
+ *
+ * @template T - ファクトリーが生成するサービスの型 / Type of service produced by factory
+ */
+export interface DIFactory<T> {
+  /** 現在のファクトリー関数を取得（未設定時はnull） / Get current factory (null if unset) */
+  get: () => (() => T) | null;
+  /** ファクトリー関数を設定 / Set factory function */
+  set: (factory: () => T) => void;
+  /** ファクトリーをnullにリセット / Reset factory to null */
+  reset: () => void;
+}
+
+/**
+ * 汎用DIファクトリーを生成する / Create a generic DI factory
+ *
+ * 標準パターン（let factory = null / set / reset）のボイラープレートを1行に削減する。
+ * Reduces the standard pattern (let factory = null / set / reset) to a single line.
+ *
+ * @template T - ファクトリーが生成するサービスの型 / Type of service produced by factory
+ * @param _name - デバッグ用のファクトリー名（将来のログ拡張用） / Factory name for debugging (reserved for future logging)
+ * @returns DIFactory<T> - get / set / reset メソッドを持つオブジェクト / Object with get / set / reset methods
+ */
+export function createDIFactory<T>(_name: string): DIFactory<T> {
+  let factory: (() => T) | null = null;
+
+  return {
+    get: (): (() => T) | null => factory,
+    set: (f: () => T): void => {
+      factory = f;
+    },
+    reset: (): void => {
+      factory = null;
+    },
+  };
+}

package/src/utils/prisma-wrapper-factory.ts

@@ -65,22 +65,23 @@ export interface WrappedTable {
 
 /**
  * テーブルの最小インターフェース
- * 注: anyを使用しているのは、実際のPrismaClientのcreate引数が
- * 非常に複雑で、ジェネリクスやSelectSubset等の型を含むため
+ * Prisma Client の実引数は SelectSubset<T, Args> 等の複雑なジェネリクスを含み、
+ * unknown[] では PrismaClient からの代入互換性が壊れるため、引数に any[] を使用する。
+ * 戻り値は最低限の構造を保証する。
  */
 interface MinimalTable {
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  create: (...args: any[]) => Promise<{ id: string } | any>;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  createMany?: (...args: any[]) => Promise<{ count: number } | any>;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  upsert?: (...args: any[]) => Promise<{ id: string } | any>;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  deleteMany?: (...args: any[]) => Promise<{ count: number } | any>;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  findUnique?: (...args: any[]) => Promise<any | null>;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  findMany?: (...args: any[]) => Promise<any[]>;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Prisma SelectSubset<T> generics require any for assignability from PrismaClient
+  create: (...args: any[]) => Promise<{ id: string }>;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Prisma SelectSubset<T> generics require any for assignability from PrismaClient
+  createMany?: (...args: any[]) => Promise<{ count: number }>;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Prisma SelectSubset<T> generics require any for assignability from PrismaClient
+  upsert?: (...args: any[]) => Promise<{ id: string }>;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Prisma SelectSubset<T> generics require any for assignability from PrismaClient
+  deleteMany?: (...args: any[]) => Promise<{ count: number }>;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Prisma SelectSubset<T> generics require any for assignability from PrismaClient
+  findUnique?: (...args: any[]) => Promise<unknown | null>;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Prisma SelectSubset<T> generics require any for assignability from PrismaClient
+  findMany?: (...args: any[]) => Promise<unknown[]>;
 }
 
 /**
@@ -111,11 +112,11 @@ interface MinimalPrismaClient {
   jSAnimationPattern?: MinimalTable;
   jSAnimationEmbedding?: MinimalTable;
   backgroundDesignEmbedding?: MinimalTable;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Prisma $executeRawUnsafe accepts heterogeneous value types
   $executeRawUnsafe: (...args: any[]) => Promise<unknown>;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Prisma $queryRawUnsafe accepts heterogeneous value types
   $queryRawUnsafe: (...args: any[]) => Promise<unknown>;
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Prisma transaction callback receives internal client variant
   $transaction?: <T>(fn: (tx: any) => Promise<T>, options?: TransactionOptions) => Promise<T>;
 }
 

package/src/utils/sanitize-error.ts

@@ -0,0 +1,132 @@
+// SPDX-FileCopyrightText: 2026 TKMD and Reftrix Contributors
+// SPDX-License-Identifier: AGPL-3.0-only
+
+/**
+ * Unified error message sanitization utility
+ * 統一エラーメッセージサニタイズユーティリティ
+ *
+ * Prevents internal structure leakage (CWE-209) by mapping
+ * raw error objects to safe, user-facing messages.
+ * 内部構造の漏洩防止（CWE-209）のため、生のエラーオブジェクトを
+ * 安全なユーザー向けメッセージにマッピング。
+ *
+ * @module utils/sanitize-error
+ */
+
+// =====================================================
+// Prisma Error Code Mapping / Prismaエラーコードマッピング
+// =====================================================
+
+const PRISMA_ERROR_MESSAGES: Record<string, string> = {
+  // Common Request Errors / 一般的なリクエストエラー
+  P2000: "Value too long for the column",
+  P2001: "Record not found",
+  P2002: "A record with this value already exists",
+  P2003: "Foreign key constraint failed",
+  P2025: "Record not found",
+
+  // Connection Errors / 接続エラー
+  P1001: "Database server is unreachable",
+  P1002: "Database server connection timed out",
+  P1003: "Database does not exist",
+  P1008: "Operation timed out",
+  P1017: "Database server closed the connection",
+};
+
+// =====================================================
+// Generic Error Category Messages / 汎用エラーカテゴリメッセージ
+// =====================================================
+
+const CATEGORY_MESSAGES = {
+  database: "Database operation failed",
+  network: "Network request failed",
+  timeout: "Operation timed out",
+  validation: "Input validation failed",
+  not_found: "Resource not found",
+  service: "Service is temporarily unavailable",
+  internal: "An internal error occurred",
+} as const;
+
+// =====================================================
+// sanitizeErrorMessage / エラーサニタイズ関数
+// =====================================================
+
+/**
+ * Sanitize error for client-facing response.
+ * クライアント向けレスポンスのエラーをサニタイズ。
+ *
+ * - Prisma errors: mapped by error code (P2002, P2025, etc.)
+ * - Network/timeout errors: detected by message keywords
+ * - All others: generic "An internal error occurred"
+ *
+ * Server-side logging should use the original error object.
+ * サーバーサイドのログには元のエラーオブジェクトを使用すること。
+ *
+ * @param error - Raw error (unknown type from catch blocks)
+ * @returns Safe, user-facing error message
+ *
+ * @example
+ * ```typescript
+ * catch (error) {
+ *   logger.error("Operation failed", { error });
+ *   return createErrorResponse({ message: sanitizeErrorMessage(error) });
+ * }
+ * ```
+ */
+export function sanitizeErrorMessage(error: unknown): string {
+  if (error instanceof Error) {
+    // Prisma known request errors (P2xxx, P1xxx)
+    const prismaError = error as { code?: string };
+    if (prismaError.code && prismaError.code in PRISMA_ERROR_MESSAGES) {
+      return PRISMA_ERROR_MESSAGES[prismaError.code]!;
+    }
+
+    // Prisma-like error codes (Pxxxx pattern) — generic DB message
+    if (prismaError.code && /^P\d{4}$/.test(prismaError.code)) {
+      return CATEGORY_MESSAGES.database;
+    }
+
+    // Detect category from error message (keyword-based)
+    const lowerMessage = error.message.toLowerCase();
+
+    if (lowerMessage.includes("timeout") || lowerMessage.includes("timed out")) {
+      return CATEGORY_MESSAGES.timeout;
+    }
+    if (
+      lowerMessage.includes("econnrefused") ||
+      lowerMessage.includes("etimedout") ||
+      lowerMessage.includes("enotfound") ||
+      lowerMessage.includes("fetch failed")
+    ) {
+      return CATEGORY_MESSAGES.network;
+    }
+    if (lowerMessage.includes("not found") || lowerMessage.includes("does not exist")) {
+      return CATEGORY_MESSAGES.not_found;
+    }
+  }
+
+  // Error code string (from domain-specific error code enums)
+  if (typeof error === "string") {
+    return CATEGORY_MESSAGES.internal;
+  }
+
+  return CATEGORY_MESSAGES.internal;
+}
+
+/**
+ * Sanitize error by MCP error code string.
+ * MCPエラーコード文字列からサニタイズ。
+ *
+ * Used when the caller already has a domain-specific error code
+ * (e.g., PREFERENCE_MCP_ERROR_CODES.PROFILE_NOT_FOUND).
+ *
+ * @param errorCode - Domain-specific error code string
+ * @param codeToMessageMap - Mapping of error codes to user-facing messages
+ * @returns Safe, user-facing error message
+ */
+export function sanitizeErrorCode(
+  errorCode: string,
+  codeToMessageMap: Record<string, string>
+): string {
+  return codeToMessageMap[errorCode] ?? (CATEGORY_MESSAGES.internal as string);
+}

package/src/utils/url-validator.ts

@@ -492,9 +492,7 @@ export function validateExternalUrl(url: string): UrlValidationResult {
   try {
     urlObj = new URL(trimmedUrl);
   } catch {
-    if (isDevelopment()) {
-      logger.debug("[url-validator] Failed to parse URL", { url: trimmedUrl });
-    }
+    logger.warn("[url-validator] Failed to parse URL", { url: trimmedUrl });
     return {
       valid: false,
       error: "Invalid URL format: unable to parse URL",

package/src/workers/batch-quality-worker.ts

@@ -197,13 +197,11 @@ async function processBatchQualityJob(
       } catch (error) {
         const errorMessage = error instanceof Error ? error.message : String(error);
 
-        if (isDevelopment()) {
-          logger.warn("[BatchQualityWorker] Item evaluation failed", {
-            batchJobId: jobId,
-            itemIndex: item.index,
-            error: errorMessage,
-          });
-        }
+        logger.warn("[BatchQualityWorker] Item evaluation failed", {
+          batchJobId: jobId,
+          itemIndex: item.index,
+          error: errorMessage,
+        });
 
         return {
           index: item.index,

package/src/workers/page-analyze-worker.ts

@@ -21,12 +21,12 @@
  * 2. extendJobLock: explicit lock extension at async phase boundaries
  * Together they provide dual-layer stall prevention for long-running jobs (30+ minutes).
  *
- * Architecture (v0.1.10+, TDA-C1 refactoring):
+ * Architecture (v0.2.0+, Phase 1/3 parallelization):
  * This file is a thin orchestrator. Phase logic lives in ./phases/:
  *   phase-0-ingest.ts  — HTML ingest + WebPage DB save
- *   phase-1-layout.ts  — Layout Analysis + Part Extraction
+ *   phase-1-layout.ts  — Layout Analysis + Part Extraction        ┐ Promise.all
+ *   phase-3-quality.ts — Quality Evaluation                       ┘ (parallel)
  *   phase-2-motion.ts  — Scroll Vision + Motion Detection + Scroll Vision Analysis
- *   phase-3-quality.ts — Quality Evaluation
  *   phase-4-narrative.ts — Narrative Analysis + Responsive Analysis
  *   phase-5-embedding.ts — Embedding Generation (text + visual)
  * Shared types/constants/helpers are in ./phases/types.ts.
@@ -346,17 +346,42 @@ async function processPageAnalyzeJob(
     });
 
     // =====================================================
-    // Phase 1: Layout Analysis + Phase 1.1: Part Extraction
+    // Phase 1 + Phase 3 並列実行 / Parallel Phase 1 + Phase 3
     // =====================================================
-    await processLayoutPhase(state, ctx, {
-      defaultAnalyzeLayout: defaultAnalyzeLayout as never,
-      saveBackgroundDesigns,
-      saveSectionPatterns,
-      postProcessSections,
-      extractPartsFromSection: extractPartsFromSection as never,
-      saveExtractedParts: saveExtractedParts as never,
-      prisma,
-    });
+    // Phase 3 (Quality) は state.html のみに依存し、Phase 1 (Layout) の
+    // 出力（layoutResultForNarrative, sectionSaveResult 等）を参照しない。
+    // 両フェーズは PipelineState 内の異なるフィールドに書き込むため、
+    // 同一オブジェクトへの同時書き込み競合は発生しない:
+    //   Phase 1: layoutResultForNarrative, sectionSaveResult, bgSaveResult, results.layout
+    //   Phase 3: memoryAborted, narrativePreDisabled, visionPreDisabled, results.quality
+    // completedPhases/failedPhases への Array.push() は Node.js
+    // シングルスレッドイベントループにおいて同期的に完了するため安全。
+    // statusTracker は phase ごとに独立スロットを使用するため並列呼び出し安全。
+    //
+    // Phase 3 (Quality) depends only on state.html (set by Phase 0).
+    // It does NOT read Phase 1 outputs (layoutResultForNarrative, sectionSaveResult, etc.).
+    // Both phases write to distinct PipelineState fields, preventing concurrent
+    // write conflicts. Array.push() on completedPhases/failedPhases is safe
+    // in Node.js single-threaded event loop (synchronous completion).
+    // statusTracker uses independent slots per phase, safe for parallel calls.
+    await Promise.all([
+      processLayoutPhase(state, ctx, {
+        defaultAnalyzeLayout: defaultAnalyzeLayout as never,
+        saveBackgroundDesigns,
+        saveSectionPatterns,
+        postProcessSections,
+        extractPartsFromSection: extractPartsFromSection as never,
+        saveExtractedParts: saveExtractedParts as never,
+        prisma,
+      }),
+      processQualityPhase(state, ctx, {
+        defaultEvaluateQuality: defaultEvaluateQuality as never,
+        saveQualityEvaluation,
+        saveQualityBenchmarks,
+        buildQualityBenchmarkInputs,
+        prisma,
+      }),
+    ]);
 
     // =====================================================
     // Ollama Vision Unload (1st point): After Phase 1 / before Phase 1.5
@@ -384,17 +409,6 @@ async function processPageAnalyzeJob(
       prisma,
     });
 
-    // =====================================================
-    // Phase 3: Quality Evaluation
-    // =====================================================
-    await processQualityPhase(state, ctx, {
-      defaultEvaluateQuality: defaultEvaluateQuality as never,
-      saveQualityEvaluation,
-      saveQualityBenchmarks,
-      buildQualityBenchmarkInputs,
-      prisma,
-    });
-
     // =====================================================
     // Phase 4 + 4.5: Narrative + Responsive (includes Ollama unload 3rd point)
     // =====================================================

package/src/workers/phases/phase-0-ingest.ts

@@ -46,7 +46,7 @@ export interface IngestPhaseDeps {
   };
   prisma: {
     webPage: {
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any -- Prisma upsert args use SelectSubset<T> generics incompatible with structural types
       upsert: (args: any) => Promise<{ id: string }>;
     };
   };

package/src/workers/phases/phase-1-layout.ts

@@ -91,17 +91,19 @@ export interface LayoutPhaseDeps {
 
   /** Save extracted parts to DB */
   saveExtractedParts: (
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any -- PrismaClient variant, cast at call site
-    prisma: any,
+    prisma: unknown,
     webPageId: string,
     sectionPatternId: string,
     parts: Array<{ [key: string]: unknown }>,
     sourceUrl: string | null
   ) => Promise<PartSaveResult>;
 
-  /** Prisma client instance (cast at call sites) */
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- multiple Prisma client subtypes used
-  prisma: any;
+  /**
+   * Prisma client instance.
+   * 複数の Prisma サブタイプ（BackgroundDesignPrismaClient, SectionPatternPrismaClient,
+   * PrismaClient）として使用されるため、呼び出し側で `as unknown as SpecificType` でキャストする。
+   */
+  prisma: unknown;
 }
 
 // ============================================================================
@@ -317,8 +319,9 @@ export async function processLayoutPhase(
             state.layoutResultForNarrative &&
             postProcessedSections.length !== sectionsWithCss.length
           ) {
-            // eslint-disable-next-line @typescript-eslint/no-explicit-any -- LayoutSection.visionFeatures(unknown) vs SectionVisionFeatures型の互換性のためキャスト
-            state.layoutResultForNarrative.sections = postProcessedSections as any;
+            // LayoutSection.visionFeatures(unknown) vs SectionVisionFeatures型の互換性のためキャスト
+            state.layoutResultForNarrative.sections =
+              postProcessedSections as unknown as NonNullable<LayoutServiceResult["sections"]>;
             state.layoutResultForNarrative.sectionCount = postProcessedSections.length;
           }
 

package/src/workers/phases/phase-2-motion.ts

@@ -52,9 +52,12 @@ import type {
 import type {
   MotionServiceResult,
   MotionDetectionContext,
+  MotionDetectionExtendedContext,
   IPageAnalyzePrismaClient,
   JSAnimationFullResult,
 } from "../../tools/page/handlers/types";
+import type { PageAnalyzeInput } from "../../tools/page/schemas";
+import type { MotionPatternInput } from "../../services/worker-db-save.service";
 
 // GPU Resource Manager type
 import type { GpuResourceManager } from "../../services/gpu-resource-manager";
@@ -99,11 +102,9 @@ export interface MotionPhaseDeps {
   defaultDetectMotion: (
     html: string,
     url: string,
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    options?: any,
+    options?: PageAnalyzeInput["motionOptions"],
     dbContext?: MotionDetectionContext,
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    extendedContext?: any,
+    extendedContext?: MotionDetectionExtendedContext,
     preExtractedCssUrls?: string[],
     sharedBrowser?: Browser
   ) => Promise<MotionServiceResult>;
@@ -118,8 +119,7 @@ export interface MotionPhaseDeps {
   saveMotionPatterns: (
     prisma: MotionPatternPrismaClient,
     webPageId: string,
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    patterns: any[],
+    patterns: MotionPatternInput[],
     sourceUrl: string
   ) => Promise<SaveResult>;
 
@@ -148,12 +148,13 @@ export interface MotionPhaseDeps {
     close: () => Promise<void>;
   };
 
-  /** Prisma client for DB operations */
-  prisma: {
-    // Used as MotionPatternPrismaClient, JsAnimationPatternPrismaClient, ScrollVisionPrismaClient
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    [key: string]: any;
-  };
+  /**
+   * Prisma client for DB operations.
+   * 複数の Prisma サブタイプ（MotionPatternPrismaClient, JsAnimationPatternPrismaClient,
+   * ScrollVisionPrismaClient, IPageAnalyzePrismaClient）として使用されるため、
+   * 呼び出し側で `as unknown as SpecificType` でキャストする。
+   */
+  prisma: unknown;
 }
 
 // ============================================================================
@@ -572,17 +573,15 @@ export async function processMotionPhase(
       const probeResult = await readinessProbe.check();
 
       if (!probeResult.ready) {
-        if (isDevelopment()) {
-          logger.warn(
-            "[PageAnalyzeWorker] Ollama readiness probe failed, skipping scroll vision analysis",
-            {
-              reason: probeResult.reason,
-              vram: probeResult.vram,
-              waitRetries: probeResult.waitRetries,
-              totalWaitMs: probeResult.totalWaitMs,
-            }
-          );
-        }
+        logger.warn(
+          "[PageAnalyzeWorker] Ollama readiness probe failed, skipping scroll vision analysis",
+          {
+            reason: probeResult.reason,
+            vram: probeResult.vram,
+            waitRetries: probeResult.waitRetries,
+            totalWaitMs: probeResult.totalWaitMs,
+          }
+        );
         // Graceful Degradation: VRAM不足時はVision分析をスキップ（ジョブは継続）
         state.scrollVisionCapturesForDeferred = null;
       }

package/src/workers/phases/phase-4-narrative.ts

@@ -286,23 +286,19 @@ export async function processNarrativePhase(
       // SSRF対策: URLを検証
       const urlValidation = deps.validateExternalUrl(url);
       if (!urlValidation.valid) {
-        if (isDevelopment()) {
-          logger.warn("[PageAnalyzeWorker] Responsive SSRF blocked", {
-            url,
-            error: urlValidation.error,
-          });
-        }
+        logger.warn("[PageAnalyzeWorker] Responsive SSRF blocked", {
+          url,
+          error: urlValidation.error,
+        });
         statusTracker.skipPhase("responsive", `SSRF blocked: ${urlValidation.error}`);
       } else {
         // robots.txt チェック（respect_robots_txt パラメータを伝搬）
         const robotsResult = await deps.isUrlAllowedByRobotsTxt(url, options.respectRobotsTxt);
         if (!robotsResult.allowed) {
-          if (isDevelopment()) {
-            logger.warn("[PageAnalyzeWorker] Responsive blocked by robots.txt", {
-              url,
-              reason: robotsResult.reason,
-            });
-          }
+          logger.warn("[PageAnalyzeWorker] Responsive blocked by robots.txt", {
+            url,
+            reason: robotsResult.reason,
+          });
           statusTracker.skipPhase(
             "responsive",
             `Robots.txt blocked: ${robotsResult.reason}. ` +