@klhapp/skillmux 1.9.2 → 1.10.0

package/src/router-core.ts

@@ -4,6 +4,8 @@ import { join } from "node:path";
 import { buildAuditRow } from "./audit";
 import { embeddingDimension, embeddingFingerprint, expandHome, loadConfig } from "./config";
 import { RemoteInferenceError } from "./clients";
+import { log } from "./logger";
+import { warn } from "./output";
 import {
   deleteSkill,
   findExactMatch,
@@ -274,7 +276,7 @@ export async function syncVaultIfNeeded(): Promise<void> {
     const invalidIds: string[] = [];
     const skills = await scanVaults(vaultPath, localVaultPaths, (skillId, error) => {
       invalidIds.push(skillId);
-      console.error(`warning: keeping previous index entry for ${skillId}: ${error}`);
+      warn(`keeping previous index entry for ${skillId}: ${error}`);
     });
     const rows = skills.map(toSkillRow);
     // See rebuildIndex: a skill_id invalid in one root can still be valid in
@@ -361,7 +363,7 @@ async function reindexOneSkill(db: Database, vaultPath: string, skillId: string)
     upsertSkill(db, await readSkill(vaultPath, skillId));
     backfillEmbeddings().catch(() => {});
   } catch (error) {
-    console.error(`warning: keeping previous index entry for ${skillId}: ${error}`);
+    warn(`keeping previous index entry for ${skillId}: ${error}`);
   }
 }
 
@@ -395,7 +397,7 @@ export async function startVaultWatcher(): Promise<() => void> {
   // A watcher error (e.g. the vault root disappearing) must degrade the index,
   // not crash the server — an unhandled 'error' event would throw.
   watcher.on("error", (error) => {
-    console.error(`warning: vault watcher error, live updates paused: ${error}`);
+    warn(`vault watcher error, live updates paused: ${error}`);
   });
 
   return () => {
@@ -554,14 +556,7 @@ export async function retrieveAndRerank(
       retrieval = "lexical";
       degraded_from = clients.rerank ? "reranked" : "hybrid";
       degradation_reason = classifyInferenceError("embedding", embedRes.error);
-      console.error(
-        JSON.stringify({
-          level: "warn",
-          stage: "embedding",
-          degraded_from,
-          reason: degradation_reason,
-        }),
-      );
+      log.warn("embedding", { degraded_from, reason: degradation_reason });
     } else {
       try {
         const queryVec = (embedRes as Float32Array[])[0];
@@ -574,14 +569,7 @@ export async function retrieveAndRerank(
         retrieval = "lexical";
         degraded_from = clients.rerank ? "reranked" : "hybrid";
         degradation_reason = classifyInferenceError("embedding", embedError);
-        console.error(
-          JSON.stringify({
-            level: "warn",
-            stage: "embedding",
-            degraded_from,
-            reason: degradation_reason,
-          }),
-        );
+        log.warn("embedding", { degraded_from, reason: degradation_reason });
       }
     }
   }
@@ -601,14 +589,7 @@ export async function retrieveAndRerank(
       scores = null;
       degraded_from = "reranked";
       degradation_reason = classifyInferenceError("reranker", rerankError);
-      console.error(
-        JSON.stringify({
-          level: "warn",
-          stage: "reranker",
-          degraded_from,
-          reason: degradation_reason,
-        }),
-      );
+      log.warn("reranker", { degraded_from, reason: degradation_reason });
     }
   }