@kb-labs/rest-api-app 2.89.0 → 2.94.0

package/dist/index.js

@@ -868,7 +868,7 @@ async function registerEventRoutes(server, basePath, registry, readiness, eventH
         const ready = isReady(readiness);
         const reason = resolveReadinessReason(readiness);
         const pluginSnapshot = metricsCollector.getLastPluginMountSnapshot();
-        const redisStatus = registry.getRedisStatus?.();
+        const redisStatus = "getRedisStatus" in registry && typeof registry.getRedisStatus === "function" ? registry.getRedisStatus() : void 0;
         send({
           type: "health",
           status: health.status,
@@ -886,7 +886,11 @@ async function registerEventRoutes(server, basePath, registry, readiness, eventH
           pluginRoutesLastDurationMs: readiness.pluginRoutesLastDurationMs ?? null,
           redisEnabled: redisStatus?.enabled ?? false,
           redisHealthy: redisStatus?.healthy ?? true,
-          redisStates: redisStatus?.roles
+          redisStates: redisStatus ? {
+            publisher: redisStatus.roles.publisher ?? null,
+            subscriber: redisStatus.roles.subscriber ?? null,
+            cache: redisStatus.roles.cache ?? null
+          } : void 0
         });
       }).catch((error) => {
         if (request.kbLogger) {
@@ -2261,12 +2265,15 @@ async function registerPluginSnapshotRoutes(server, config, registry) {
         },
         registryErrors: {
           total: registryErrors.length,
-          items: registryErrors.map((err) => ({
-            pluginPath: err.pluginPath,
-            pluginId: err.pluginId,
-            error: err.error,
-            code: err.code
-          }))
+          items: registryErrors.map((err) => {
+            const e = err;
+            return {
+              pluginPath: e.pluginPath,
+              pluginId: e.pluginId,
+              error: e.error,
+              code: e.code
+            };
+          })
         },
         diagnostics: {
           total: discoveryDiagnostics.length,
@@ -2982,8 +2989,9 @@ async function registerObservabilityRoutes(fastify, config, repoRoot, historical
         fastify.log.debug("Fetching system metrics from all instances");
         const allMetrics = [];
         try {
-          if ("scan" in platform17.cache && typeof platform17.cache.scan === "function") {
-            const keys = await platform17.cache.scan("system-metrics:*");
+          const cacheWithScan = platform17.cache;
+          if ("scan" in platform17.cache && typeof cacheWithScan.scan === "function") {
+            const keys = await cacheWithScan.scan("system-metrics:*");
             for (const key of keys) {
               const metrics = await platform17.cache.get(key);
               if (metrics) {
@@ -3639,14 +3647,15 @@ function getEventData(event) {
   if (event.schema === "kb.v1" && event.payload) {
     return event.payload;
   }
-  return event.properties || {};
+  return event.properties ?? {};
 }
 function extractDateRange(query) {
-  if (!query) {
+  if (!query || typeof query !== "object") {
     return { from: void 0, to: void 0 };
   }
-  const from = query.from;
-  const to = query.to;
+  const q = query;
+  const from = q.from;
+  const to = q.to;
   if (from && !isValidISODate(from)) {
     throw new Error('Invalid "from" date format. Expected ISO 8601 datetime (e.g., 2026-01-01T00:00:00Z)');
   }
@@ -3656,22 +3665,27 @@ function extractDateRange(query) {
   return { from, to };
 }
 function extractModelFilter(query) {
-  if (!query || !query.models) {
+  if (!query || typeof query !== "object") {
     return void 0;
   }
-  const modelsParam = query.models;
+  const q = query;
+  if (!q.models) {
+    return void 0;
+  }
+  const modelsParam = q.models;
   const models = modelsParam.split(",").map((m) => m.trim()).filter((m) => m.length > 0);
   return models.length > 0 ? models : void 0;
 }
 function extractStatsOptions(query) {
-  if (!query) {
+  if (!query || typeof query !== "object") {
     return {};
   }
-  const groupBy = query.groupBy;
+  const q = query;
+  const groupBy = q.groupBy;
   const validGroupBy = ["hour", "day", "week", "month"];
   const resolvedGroupBy = groupBy && validGroupBy.includes(groupBy) ? groupBy : void 0;
-  const breakdownBy = query.breakdownBy;
-  const metricsParam = query.metrics;
+  const breakdownBy = q.breakdownBy;
+  const metricsParam = q.metrics;
   const metrics = metricsParam ? metricsParam.split(",").map((m) => m.trim()).filter((m) => m.length > 0) : void 0;
   return {
     groupBy: resolvedGroupBy,
@@ -4618,8 +4632,9 @@ Log Entries (${Math.min(relevantLogs.length, 100)} most recent):
 `;
       }
     }
-    if (includeContext.stackTraces && log.err?.stack) {
-      prompt += `  stack: ${log.err.stack.split("\n").slice(0, 5).join("\n  ")}
+    const logErr = log.err;
+    if (includeContext.stackTraces && logErr?.stack) {
+      prompt += `  stack: ${logErr.stack.split("\n").slice(0, 5).join("\n  ")}
 `;
     }
   });
@@ -4778,7 +4793,7 @@ async function registerLogRoutes(server, config, eventHub) {
       return reply.code(503).send({
         ok: false,
         error: "Log query failed",
-        message: error?.message ?? "Unknown error"
+        message: error instanceof Error ? error.message : "Unknown error"
       });
     }
   });
@@ -4811,7 +4826,7 @@ async function registerLogRoutes(server, config, eventHub) {
         return reply.code(500).send({
           ok: false,
           error: "Failed to fetch log",
-          message: error?.message ?? "Unknown error"
+          message: error instanceof Error ? error.message : "Unknown error"
         });
       }
     }
@@ -4842,7 +4857,7 @@ async function registerLogRoutes(server, config, eventHub) {
         return reply.code(500).send({
           ok: false,
           error: "Failed to fetch related logs",
-          message: error?.message ?? "Unknown error"
+          message: error instanceof Error ? error.message : "Unknown error"
         });
       }
     }
@@ -4931,7 +4946,7 @@ async function registerLogRoutes(server, config, eventHub) {
       return reply.code(500).send({
         ok: false,
         error: "Failed to fetch stats",
-        message: error?.message ?? "Unknown error"
+        message: error instanceof Error ? error.message : "Unknown error"
       });
     }
   });
@@ -5039,7 +5054,7 @@ async function registerLogRoutes(server, config, eventHub) {
       return reply.code(500).send({
         ok: false,
         error: "Log summarization failed",
-        message: error?.message ?? "Unknown error"
+        message: error instanceof Error ? error.message : "Unknown error"
       });
     }
   });
@@ -5081,10 +5096,11 @@ async function loadPlatformConfig(repoRoot) {
       };
     }
     const config = configResult.data;
+    const configPlatform = typeof config.platform === "object" && config.platform !== null ? config.platform : {};
     return {
-      adapters: config.platform?.adapters ?? {},
-      adapterOptions: config.platform?.adapterOptions ?? {},
-      execution: config.platform?.execution ?? { mode: "in-process" }
+      adapters: configPlatform.adapters ?? {},
+      adapterOptions: configPlatform.adapterOptions ?? {},
+      execution: configPlatform.execution ?? { mode: "in-process" }
     };
   } catch (error) {
     return {
@@ -5657,11 +5673,11 @@ function normalizeBasePath2(basePath) {
 async function registerRoutes(server, config, repoRoot, registry) {
   registerRouteCollector(server);
   const initialSnapshot = registry.snapshot();
-  const initialRedisStatus = registry.getRedisStatus?.();
-  const initialRedisStates = initialRedisStatus?.roles ?? {
-    publisher: null,
-    subscriber: null,
-    cache: null
+  const initialRedisStatus = "getRedisStatus" in registry && typeof registry.getRedisStatus === "function" ? registry.getRedisStatus() : void 0;
+  const initialRedisStates = {
+    publisher: initialRedisStatus?.roles.publisher ?? null,
+    subscriber: initialRedisStatus?.roles.subscriber ?? null,
+    cache: initialRedisStatus?.roles.cache ?? null
   };
   const registryLoaded = initialSnapshot.plugins.length > 0 && !initialSnapshot.partial && !initialSnapshot.stale;
   const readiness = {
@@ -5706,7 +5722,7 @@ async function registerRoutes(server, config, repoRoot, registry) {
   server.kbEventHub = eventHub;
   const broadcastState = async () => {
     const pluginsSnapshot = metricsCollector.getLastPluginMountSnapshot();
-    const redisStatus = registry.getRedisStatus?.();
+    const redisStatus = "getRedisStatus" in registry && typeof registry.getRedisStatus === "function" ? registry.getRedisStatus() : void 0;
     if (redisStatus) {
       handleRedisUpdate(redisStatus);
       readiness.redisEnabled = redisStatus.enabled;
@@ -6088,15 +6104,17 @@ function registerErrorGuard(server) {
       return;
     }
     const requestId = request.id || "unknown";
+    const err = error instanceof Error ? error : new Error(String(error));
+    const errWithStatus = error;
     const envelope = {
       status: "error",
-      http: error.statusCode || 500,
+      http: errWithStatus.statusCode || 500,
       code: ErrorCode.INTERNAL,
-      message: error.message || "Internal server error",
+      message: err.message || "Internal server error",
       details: {
-        error: error.message || String(error)
+        error: err.message || String(error)
       },
-      trace: error.stack,
+      trace: err.stack,
       meta: {
         requestId,
         pluginId: "system",
@@ -6320,17 +6338,19 @@ async function createServer(config, repoRoot, registry) {
     const customLogger = {
       [kDisableRequestLogging]: true,
       // Disable Fastify's built-in request logging
-      debug: (obj, msg) => {
+      debug: (_obj, _msg) => {
+      },
+      info: (_obj, _msg) => {
       },
-      info: (obj, msg) => {
+      warn: (_obj, _msg) => {
       },
-      warn: (obj, msg) => {
+      error: (_obj, _msg) => {
       },
-      error: (obj, msg) => {
+      fatal: (_obj, _msg) => {
       },
-      fatal: (obj, msg) => {
+      trace: (_obj, _msg) => {
       },
-      trace: (obj, msg) => {
+      silent: (_obj, _msg) => {
       },
       child: () => customLogger,
       level: "silent"
@@ -6478,7 +6498,7 @@ async function bootstrap(cwd = process.cwd()) {
   await metricsCollector2.start(1e4, 6e4);
   const address = await server.listen({
     port: config.port,
-    host: "0.0.0.0"
+    host: process.env.REST_API_HOST ?? "0.0.0.0"
   });
   bootstrapLogger.info("REST API server listening", { address });
   const shutdown = async (signal) => {