@reftrixmcp/mcp-server 0.1.8 → 0.2.0

package/src/services/background-search.service.ts

@@ -11,7 +11,6 @@
  */
 
 import { logger } from "../utils/logger";
-import { isDevelopmentEnvironment } from "./production-guard";
 import {
   executeHybridSearch,
   buildFulltextConditions,
@@ -93,6 +92,10 @@ function mapRowToResult(row: BackgroundSearchRow): BackgroundDesignSearchResult
 interface BackgroundSearchFilters {
   designType?: string;
   webPageId?: string;
+  webPageUrl?: string;
+  tags?: string[];
+  industry?: string;
+  audience?: string;
 }
 
 interface BuildWhereResult {
@@ -121,6 +124,37 @@ function buildWhereClause(
     paramIndex++;
   }
 
+  if (filters?.webPageUrl) {
+    conditions.push(
+      `EXISTS (SELECT 1 FROM web_pages wp WHERE wp.id = bd.web_page_id AND wp.url = $${paramIndex})`
+    );
+    params.push(filters.webPageUrl);
+    paramIndex++;
+  }
+
+  if (filters?.tags && filters.tags.length > 0) {
+    conditions.push(`bd.tags @> $${paramIndex}::text[]`);
+    params.push(filters.tags);
+    paramIndex++;
+  }
+
+  // industry/audience: filter via quality_benchmarks EXISTS subquery
+  if (filters?.industry) {
+    conditions.push(
+      `EXISTS (SELECT 1 FROM quality_benchmarks qb WHERE qb.web_page_id = bd.web_page_id AND qb.industry = $${paramIndex})`
+    );
+    params.push(filters.industry);
+    paramIndex++;
+  }
+
+  if (filters?.audience) {
+    conditions.push(
+      `EXISTS (SELECT 1 FROM quality_benchmarks qb WHERE qb.web_page_id = bd.web_page_id AND qb.audience = $${paramIndex})`
+    );
+    params.push(filters.audience);
+    paramIndex++;
+  }
+
   return {
     whereClause: conditions.length > 0 ? `WHERE ${conditions.join(" AND ")}` : "",
     params,
@@ -158,11 +192,9 @@ export function createBackgroundSearchService(config: BackgroundSearchServiceCon
     try {
       return await embeddingService.generateEmbedding(query, "query");
     } catch (error) {
-      if (isDevelopmentEnvironment()) {
-        logger.warn("[BackgroundSearchService] Embedding generation failed", {
-          error: error instanceof Error ? error.message : "Unknown error",
-        });
-      }
+      logger.warn("[BackgroundSearchService] Embedding generation failed", {
+        error: error instanceof Error ? error.message : "Unknown error",
+      });
       return null;
     }
   }
@@ -307,11 +339,9 @@ export function createBackgroundSearchService(config: BackgroundSearchServiceCon
 
           return toRankedItems(rows);
         } catch (ftError) {
-          if (isDevelopmentEnvironment()) {
-            logger.warn("[BackgroundSearchService] Full-text search failed, using vector only", {
-              error: ftError instanceof Error ? ftError.message : "Unknown error",
-            });
-          }
+          logger.warn("[BackgroundSearchService] Full-text search failed, using vector only", {
+            error: ftError instanceof Error ? ftError.message : "Unknown error",
+          });
           return [];
         }
       };
@@ -338,11 +368,9 @@ export function createBackgroundSearchService(config: BackgroundSearchServiceCon
 
       return { results, total: hybridResults.length };
     } catch (error) {
-      if (isDevelopmentEnvironment()) {
-        logger.error("[BackgroundSearchService] Hybrid search error, falling back to vector", {
-          error: error instanceof Error ? error.message : "Unknown error",
-        });
-      }
+      logger.warn("[BackgroundSearchService] Hybrid search error, falling back to vector", {
+        error: error instanceof Error ? error.message : "Unknown error",
+      });
       // フォールバック: ベクトル検索のみ
       return searchBackgroundDesigns(embedding, options);
     }

package/src/services/browser-process-manager.ts

@@ -38,6 +38,18 @@ import { type Browser } from "playwright";
 import { execSync } from "child_process";
 import { logger, isDevelopment } from "../utils/logger";
 
+/**
+ * Playwright Browser の内部API（_browserType._launchedProcess）にアクセスするための型拡張。
+ * 公開型定義には含まれないが、実行時に存在する。
+ */
+interface BrowserInternals {
+  _browserType?: {
+    _launchedProcess?: {
+      pid: number;
+    };
+  };
+}
+
 // =============================================
 // 型定義
 // =============================================
@@ -110,9 +122,8 @@ export class BrowserProcessManager {
   private getBrowserPid(): number | null {
     try {
       // Playwrightの内部APIを使用
-      // eslint-disable-next-line @typescript-eslint/no-explicit-any
-      const browserAny = this.browser as any;
-      const launchedProcess = browserAny._browserType?._launchedProcess;
+      const browserInternals = this.browser as unknown as BrowserInternals;
+      const launchedProcess = browserInternals._browserType?._launchedProcess;
       return launchedProcess?.pid ?? null;
     } catch {
       return null;
@@ -136,13 +147,11 @@ export class BrowserProcessManager {
     } catch (error) {
       const errorMessage = error instanceof Error ? error.message : String(error);
 
-      if (isDevelopment()) {
-        logger.warn("[BrowserProcessManager] browser.close() failed", {
-          error: errorMessage,
-          forceKillOnTimeout: this.forceKillOnTimeout,
-          pid: this.browserPid,
-        });
-      }
+      logger.warn("[BrowserProcessManager] browser.close() failed", {
+        error: errorMessage,
+        forceKillOnTimeout: this.forceKillOnTimeout,
+        pid: this.browserPid,
+      });
 
       if (this.forceKillOnTimeout && this.browserPid) {
         await this.forceKill();
@@ -255,14 +264,12 @@ export class BrowserProcessManager {
       // ESRCH: プロセスが存在しない（既に終了している）
       const errnoError = error as { code?: string };
       if (errnoError.code !== "ESRCH") {
-        if (isDevelopment()) {
-          logger.error("[BrowserProcessManager] forceKill error", {
-            error: error instanceof Error ? error.message : String(error),
-            pid: this.browserPid,
-          });
-        }
-      } else if (isDevelopment()) {
-        logger.debug(
+        logger.warn("[BrowserProcessManager] forceKill error", {
+          error: error instanceof Error ? error.message : String(error),
+          pid: this.browserPid,
+        });
+      } else {
+        logger.warn(
           `[BrowserProcessManager] Process ${this.browserPid} already terminated (ESRCH)`
         );
       }
@@ -323,12 +330,10 @@ export class BrowserProcessManager {
         }
       }
     } catch (error) {
-      if (isDevelopment()) {
-        logger.error("[BrowserProcessManager] killAllChildren error", {
-          error: error instanceof Error ? error.message : String(error),
-          pid: this.browserPid,
-        });
-      }
+      logger.warn("[BrowserProcessManager] killAllChildren error", {
+        error: error instanceof Error ? error.message : String(error),
+        pid: this.browserPid,
+      });
     }
   }
 }

package/src/services/layout-embedding.service.ts

@@ -830,11 +830,9 @@ export class LayoutEmbeddingService {
         }
       } catch (cacheError) {
         // キャッシュエラーは無視して続行
-        if (isDevelopment()) {
-          logger.warn("[LayoutEmbedding] Cache read error, proceeding without cache", {
-            error: cacheError instanceof Error ? cacheError.message : "Unknown error",
-          });
-        }
+        logger.warn("[LayoutEmbedding] Cache read error, proceeding without cache", {
+          error: cacheError instanceof Error ? cacheError.message : "Unknown error",
+        });
       }
     }
 
@@ -860,11 +858,9 @@ export class LayoutEmbeddingService {
       };
 
       embeddingCache.set(cacheKey, cacheEntry).catch((error) => {
-        if (isDevelopment()) {
-          logger.warn("[LayoutEmbedding] Cache write error", {
-            error: error instanceof Error ? error.message : "Unknown error",
-          });
-        }
+        logger.warn("[LayoutEmbedding] Cache write error", {
+          error: error instanceof Error ? error.message : "Unknown error",
+        });
       });
     }
 
@@ -988,14 +984,12 @@ export class LayoutEmbeddingService {
       }
     } catch (batchError) {
       // バッチ処理が失敗した場合、個別に処理
-      if (isDevelopment()) {
-        logger.warn(
-          "[LayoutEmbedding] Batch processing with Vision failed, falling back to individual",
-          {
-            error: batchError instanceof Error ? batchError.message : "Unknown error",
-          }
-        );
-      }
+      logger.warn(
+        "[LayoutEmbedding] Batch processing with Vision failed, falling back to individual",
+        {
+          error: batchError instanceof Error ? batchError.message : "Unknown error",
+        }
+      );
 
       for (let i = 0; i < sections.length; i++) {
         const section = sections[i];
@@ -1090,11 +1084,9 @@ export class LayoutEmbeddingService {
       }
     } catch (batchError) {
       // バッチ処理が失敗した場合、個別に処理
-      if (isDevelopment()) {
-        logger.warn("[LayoutEmbedding] Batch processing failed, falling back to individual", {
-          error: batchError instanceof Error ? batchError.message : "Unknown error",
-        });
-      }
+      logger.warn("[LayoutEmbedding] Batch processing failed, falling back to individual", {
+        error: batchError instanceof Error ? batchError.message : "Unknown error",
+      });
 
       for (let i = 0; i < sections.length; i++) {
         const section = sections[i];
@@ -1347,11 +1339,9 @@ export class LayoutEmbeddingService {
           logger.info("[LayoutEmbedding] ONNX pipeline disposed for memory recovery");
         }
       } catch (disposeError) {
-        if (isDevelopment()) {
-          logger.warn("[LayoutEmbedding] Pipeline dispose warning", {
-            error: disposeError instanceof Error ? disposeError.message : "Unknown error",
-          });
-        }
+        logger.warn("[LayoutEmbedding] Pipeline dispose warning", {
+          error: disposeError instanceof Error ? disposeError.message : "Unknown error",
+        });
       }
     }
   }

package/src/services/layout-search.service.ts

@@ -180,6 +180,41 @@ function buildWhereClause(filters?: LayoutSearchFilters): { clause: string; para
     paramIndex++;
   }
 
+  if (filters?.webPageId) {
+    conditions.push(`wp.id = $${paramIndex}`);
+    params.push(filters.webPageId);
+    paramIndex++;
+  }
+
+  if (filters?.webPageUrl) {
+    conditions.push(`wp.url = $${paramIndex}`);
+    params.push(filters.webPageUrl);
+    paramIndex++;
+  }
+
+  // Common search filters (industry/audience/tags)
+  if (filters?.tags && filters.tags.length > 0) {
+    conditions.push(`sp.tags @> $${paramIndex}::text[]`);
+    params.push(filters.tags);
+    paramIndex++;
+  }
+
+  if (filters?.industry) {
+    conditions.push(
+      `EXISTS (SELECT 1 FROM quality_benchmarks qb WHERE qb.web_page_id = wp.id AND qb.industry = $${paramIndex})`
+    );
+    params.push(filters.industry);
+    paramIndex++;
+  }
+
+  if (filters?.audience) {
+    conditions.push(
+      `EXISTS (SELECT 1 FROM quality_benchmarks qb WHERE qb.web_page_id = wp.id AND qb.audience = $${paramIndex})`
+    );
+    params.push(filters.audience);
+    paramIndex++;
+  }
+
   return {
     clause: conditions.length > 0 ? conditions.join(" AND ") : "",
     params,
@@ -536,11 +571,9 @@ export class LayoutSearchService implements ILayoutSearchService {
       const embeddingService = this.getEmbeddingService();
       return await embeddingService.generateEmbedding(query, "query");
     } catch (error) {
-      if (isDevelopment()) {
-        logger.warn("[LayoutSearchService] Embedding generation failed, returning null", {
-          error: error instanceof Error ? error.message : "Unknown error",
-        });
-      }
+      logger.warn("[LayoutSearchService] Embedding generation failed, returning null", {
+        error: error instanceof Error ? error.message : "Unknown error",
+      });
       // エラー時もnullを返し、空の結果を返す
       return null;
     }
@@ -570,9 +603,7 @@ export class LayoutSearchService implements ILayoutSearchService {
       prisma = this.getPrismaClient();
     } catch {
       // PrismaClientが利用できない場合はnullを返す
-      if (isDevelopment()) {
-        logger.warn("[LayoutSearchService] PrismaClient not available, returning null");
-      }
+      logger.warn("[LayoutSearchService] PrismaClient not available, returning null");
       return null;
     }
 
@@ -649,11 +680,9 @@ export class LayoutSearchService implements ILayoutSearchService {
         );
         total = Number(countResult[0]?.total || 0);
       } catch (dbError) {
-        if (isDevelopment()) {
-          logger.warn("[LayoutSearchService] Vector search failed, returning empty results", {
-            error: dbError instanceof Error ? dbError.message : "Unknown error",
-          });
-        }
+        logger.warn("[LayoutSearchService] Vector search failed, returning empty results", {
+          error: dbError instanceof Error ? dbError.message : "Unknown error",
+        });
         // データベースエラー時は空の結果を返す
         return {
           results: [],
@@ -695,11 +724,9 @@ export class LayoutSearchService implements ILayoutSearchService {
         total,
       };
     } catch (error) {
-      if (isDevelopment()) {
-        logger.error("[LayoutSearchService] Search error", {
-          error: error instanceof Error ? error.message : "Unknown error",
-        });
-      }
+      logger.warn("[LayoutSearchService] Search error", {
+        error: error instanceof Error ? error.message : "Unknown error",
+      });
       throw error;
     }
   }
@@ -735,9 +762,7 @@ export class LayoutSearchService implements ILayoutSearchService {
     try {
       prisma = this.getPrismaClient();
     } catch {
-      if (isDevelopment()) {
-        logger.warn("[LayoutSearchService] PrismaClient not available, returning null");
-      }
+      logger.warn("[LayoutSearchService] PrismaClient not available, returning null");
       return null;
     }
 
@@ -839,11 +864,9 @@ export class LayoutSearchService implements ILayoutSearchService {
           return toRankedItems(results);
         } catch (ftError) {
           // 全文検索の失敗はハイブリッド検索全体をブロックしない
-          if (isDevelopment()) {
-            logger.warn("[LayoutSearchService] Full-text search failed, using vector only", {
-              error: ftError instanceof Error ? ftError.message : "Unknown error",
-            });
-          }
+          logger.warn("[LayoutSearchService] Full-text search failed, using vector only", {
+            error: ftError instanceof Error ? ftError.message : "Unknown error",
+          });
           return [];
         }
       };
@@ -889,11 +912,9 @@ export class LayoutSearchService implements ILayoutSearchService {
         total: hybridResults.length,
       };
     } catch (error) {
-      if (isDevelopment()) {
-        logger.error("[LayoutSearchService] Hybrid search error, falling back to vector only", {
-          error: error instanceof Error ? error.message : "Unknown error",
-        });
-      }
+      logger.warn("[LayoutSearchService] Hybrid search error, falling back to vector only", {
+        error: error instanceof Error ? error.message : "Unknown error",
+      });
       // フォールバック: ベクトル検索のみ
       return this.searchSectionPatterns(embedding, options);
     }

package/src/services/layout-to-code.service.ts

@@ -678,9 +678,7 @@ function normalizeCssUrls(css: string, baseUrl?: string): string {
   try {
     parsedBaseUrl = new URL(baseUrl);
   } catch {
-    if (isDevelopment()) {
-      logger.warn("[normalizeCssUrls] Invalid base URL", { baseUrl });
-    }
+    logger.warn("[normalizeCssUrls] Invalid base URL", { baseUrl });
     return result;
   }
 
@@ -732,12 +730,10 @@ function normalizeCssUrls(css: string, baseUrl?: string): string {
         return `url("${absoluteUrl}")`;
       } catch {
         // URL解決に失敗した場合はそのまま
-        if (isDevelopment()) {
-          logger.warn("[normalizeCssUrls] Failed to resolve URL", {
-            url: trimmedUrl,
-            baseUrl,
-          });
-        }
+        logger.warn("[normalizeCssUrls] Failed to resolve URL", {
+          url: trimmedUrl,
+          baseUrl,
+        });
         return match;
       }
     }
@@ -1362,12 +1358,10 @@ export class LayoutToCodeService implements ILayoutToCodeService {
 
       return recordToSectionPattern(record);
     } catch (error) {
-      if (isDevelopment()) {
-        logger.warn("[LayoutToCodeService] Error getting section pattern, returning null", {
-          id,
-          error: error instanceof Error ? error.message : "Unknown error",
-        });
-      }
+      logger.warn("[LayoutToCodeService] Error getting section pattern, returning null", {
+        id,
+        error: error instanceof Error ? error.message : "Unknown error",
+      });
       // エラー時もnullを返す
       return null;
     }
@@ -1491,12 +1485,10 @@ export class LayoutToCodeService implements ILayoutToCodeService {
 
       return result;
     } catch (error) {
-      if (isDevelopment()) {
-        logger.error("[LayoutToCodeService] Code generation error", {
-          patternId: pattern.id,
-          error: error instanceof Error ? error.message : "Unknown error",
-        });
-      }
+      logger.warn("[LayoutToCodeService] Code generation error", {
+        patternId: pattern.id,
+        error: error instanceof Error ? error.message : "Unknown error",
+      });
       throw error;
     }
   }

package/src/services/ml/mood-brandtone-embedding.service.ts

@@ -784,14 +784,9 @@ export class MoodBrandToneEmbeddingService {
       }
     } catch (batchError) {
       // バッチ処理が失敗した場合、個別に処理
-      if (isDevelopment()) {
-        logger.warn(
-          "[MoodBrandToneEmbedding] Batch processing failed, falling back to individual",
-          {
-            error: batchError instanceof Error ? batchError.message : "Unknown error",
-          }
-        );
-      }
+      logger.warn("[MoodBrandToneEmbedding] Batch processing failed, falling back to individual", {
+        error: batchError instanceof Error ? batchError.message : "Unknown error",
+      });
 
       await this.processBatchFallback(
         validItems,
@@ -924,8 +919,7 @@ function buildUpsertData(
  * pgvector形式でEmbeddingを更新するヘルパー関数
  */
 async function updateVectorEmbeddings(
-  // eslint-disable-next-line @typescript-eslint/no-explicit-any
-  tx: any,
+  tx: Pick<IPrismaClient, "$executeRawUnsafe">,
   sectionEmbeddingId: string,
   data: { mood?: MoodBrandToneEmbeddingResult; brandTone?: MoodBrandToneEmbeddingResult }
 ): Promise<void> {

package/src/services/motion/analyzers/webgl-motion-analyzer.ts

@@ -233,9 +233,7 @@ export class WebGLMotionAnalyzer {
       return result;
     } catch (error) {
       const message = error instanceof Error ? error.message : "Unknown error";
-      if (isDevelopment()) {
-        logger.error("[WebGLMotionAnalyzer] Analysis failed", { error: message });
-      }
+      logger.warn("[WebGLMotionAnalyzer] Analysis failed", { error: message });
       return this.createErrorResult("ANALYSIS_FAILED", message);
     }
   }

package/src/services/motion/existing-animation-detector.service.ts

@@ -160,12 +160,10 @@ export class ExistingAnimationDetectorService {
         });
       }
     } catch (error) {
-      if (isDevelopment()) {
-        logger.warn("[ExistingAnimationDetector] Failed to scan CSS file", {
-          filePath,
-          error: error instanceof Error ? error.message : "Unknown error",
-        });
-      }
+      logger.warn("[ExistingAnimationDetector] Failed to scan CSS file", {
+        filePath,
+        error: error instanceof Error ? error.message : "Unknown error",
+      });
       // ファイル読み取りエラーは空配列を返す
     }
 
@@ -214,11 +212,9 @@ export class ExistingAnimationDetectorService {
         });
       } catch {
         // パースエラーはスキップして続行
-        if (isDevelopment()) {
-          logger.debug("[ExistingAnimationDetector] Failed to parse keyframe", {
-            name: block.name,
-          });
-        }
+        logger.warn("[ExistingAnimationDetector] Failed to parse keyframe", {
+          name: block.name,
+        });
       }
     }
 
@@ -678,12 +674,10 @@ export class ExistingAnimationDetectorService {
         }
       }
     } catch (error) {
-      if (isDevelopment()) {
-        logger.warn("[ExistingAnimationDetector] Failed to scan directory", {
-          dirPath,
-          error: error instanceof Error ? error.message : "Unknown error",
-        });
-      }
+      logger.warn("[ExistingAnimationDetector] Failed to scan directory", {
+        dirPath,
+        error: error instanceof Error ? error.message : "Unknown error",
+      });
     }
 
     return animations;

package/src/services/motion/frame-analysis-save.helper.ts

@@ -319,14 +319,9 @@ export async function saveFrameAnalysisToDb(
   } catch (error) {
     const errorMessage = error instanceof Error ? error.message : "Unknown error";
 
-    if (isDevelopment()) {
-      logger.warn(
-        "[FrameAnalysisSaveHelper] Frame analysis DB save failed (graceful degradation)",
-        {
-          error: errorMessage,
-        }
-      );
-    }
+    logger.warn("[FrameAnalysisSaveHelper] Frame analysis DB save failed (graceful degradation)", {
+      error: errorMessage,
+    });
 
     return { saved: false, error: errorMessage };
   }

package/src/services/motion/frame-capture.service.ts

@@ -24,6 +24,7 @@ import type { Page, PageScreenshotOptions } from "playwright";
 import * as path from "path";
 import * as os from "os";
 import * as fs from "fs/promises";
+import { logger } from "../../utils/logger";
 
 // ============================================================================
 // 型定義
@@ -562,11 +563,10 @@ export class FrameCaptureService {
         );
       }
     } catch {
-      // ディレクトリ読み取りエラーは無視（空のディレクトリなど）
-      if (this.isDevelopment) {
-        // eslint-disable-next-line no-console -- Intentional debug log in development
-        console.log("[FrameCaptureService] Could not clear directory (may be empty):", dir);
-      }
+      // ディレクトリ読み取りエラーは軽微な問題のため警告レベルで記録
+      logger.warn("[FrameCaptureService] Could not clear directory (may be empty)", {
+        directory: dir,
+      });
     }
   }