@danielblomma/cortex-mcp 2.0.15 → 2.0.16

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@danielblomma/cortex-mcp",
   "mcpName": "io.github.DanielBlomma/cortex",
-  "version": "2.0.15",
+  "version": "2.0.16",
   "description": "Local, repo-scoped context platform for coding assistants. Semantic search, graph relationships, and architectural rule context.",
   "type": "module",
   "author": "Daniel Blomma",

package/scaffold/mcp/src/cli/telemetry-test.ts

@@ -6,6 +6,7 @@ import {
   loadEnterpriseConfig,
   resolveEnterpriseActivation,
 } from "../core/config.js";
+import { telemetryStatePath } from "../core/telemetry/state-dir.js";
 import { pushMetrics } from "../enterprise/telemetry/sync.js";
 import type { TelemetryMetrics } from "../core/telemetry/collector.js";
 
@@ -25,7 +26,7 @@ import type { TelemetryMetrics } from "../core/telemetry/collector.js";
  */
 
 function readMachineId(contextDir: string): string {
-  const path = join(contextDir, "telemetry", "machine_id");
+  const path = telemetryStatePath(contextDir, "machine_id");
   if (existsSync(path)) {
     try {
       const id = readFileSync(path, "utf8").trim();

package/scaffold/mcp/src/core/license.ts

@@ -1,5 +1,6 @@
 import { readFileSync, writeFileSync, mkdirSync, existsSync, unlinkSync } from "node:fs";
 import { join } from "node:path";
+import { resolveTelemetryStateDir, telemetryStatePath } from "./telemetry/state-dir.js";
 
 export type LicenseVerification =
   | {
@@ -30,7 +31,7 @@ const GRACE_PERIOD_MS = 7 * 24 * 60 * 60 * 1000; // 7d grace if endpoint unreach
 const REQUEST_TIMEOUT_MS = 5000;
 
 function cachePath(contextDir: string): string {
-  return join(contextDir, "telemetry", CACHE_FILE);
+  return telemetryStatePath(contextDir, CACHE_FILE);
 }
 
 function readCache(contextDir: string): CacheEntry | null {
@@ -47,7 +48,7 @@ function readCache(contextDir: string): CacheEntry | null {
 function writeCache(contextDir: string, entry: CacheEntry): void {
   const path = cachePath(contextDir);
   try {
-    mkdirSync(join(contextDir, "telemetry"), { recursive: true });
+    mkdirSync(resolveTelemetryStateDir(contextDir), { recursive: true });
     writeFileSync(path, JSON.stringify(entry, null, 2), "utf8");
   } catch {
     // Cache failures are non-fatal — license check just won't be cached.

package/scaffold/mcp/src/core/telemetry/collector.ts

@@ -2,6 +2,7 @@ import { readFileSync, writeFileSync, mkdirSync, existsSync } from "node:fs";
 import { createHash } from "node:crypto";
 import { hostname, platform, arch } from "node:os";
 import { join } from "node:path";
+import { resolveTelemetryStateDir, telemetryStatePath } from "./state-dir.js";
 
 export type TelemetryMetrics = {
   period_start: string;
@@ -37,7 +38,8 @@ export type TelemetryMetrics = {
 const AVG_TOKENS_PER_RESULT = 400;
 
 function generateInstanceId(contextDir: string): string {
-  const idPath = join(contextDir, "telemetry", "machine_id");
+  const telemetryDir = resolveTelemetryStateDir(contextDir);
+  const idPath = join(telemetryDir, "machine_id");
   if (existsSync(idPath)) {
     try {
       const existing = readFileSync(idPath, "utf8").trim();
@@ -51,7 +53,7 @@ function generateInstanceId(contextDir: string): string {
   const fingerprint = `${hostname()}|${platform()}|${arch()}`;
   const id = createHash("sha256").update(fingerprint).digest("hex").slice(0, 16);
   try {
-    mkdirSync(join(contextDir, "telemetry"), { recursive: true });
+    mkdirSync(telemetryDir, { recursive: true });
     writeFileSync(idPath, id, "utf8");
   } catch (err) {
     process.stderr.write(`[cortex-enterprise] Could not persist instance id: ${err instanceof Error ? err.message : String(err)}\n`);
@@ -142,8 +144,7 @@ export class TelemetryCollector {
   constructor(contextDir: string, clientVersion = "unknown") {
     this.clientVersion = clientVersion;
     this.instanceId = generateInstanceId(contextDir);
-    const telemetryDir = join(contextDir, "telemetry");
-    this.metricsPath = join(telemetryDir, "metrics.json");
+    this.metricsPath = telemetryStatePath(contextDir, "metrics.json");
 
     // Load existing metrics or start fresh
     try {

package/scaffold/mcp/src/core/telemetry/state-dir.ts

@@ -0,0 +1,40 @@
+import { accessSync, constants, mkdirSync } from "node:fs";
+import { join } from "node:path";
+
+const warnedFallbacks = new Set<string>();
+
+function canUseDirectory(dir: string): boolean {
+  try {
+    mkdirSync(dir, { recursive: true });
+    accessSync(dir, constants.R_OK | constants.W_OK | constants.X_OK);
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+export function resolveTelemetryStateDir(contextDir: string): string {
+  const primary = join(contextDir, "telemetry");
+  if (canUseDirectory(primary)) return primary;
+
+  const fallback = join(contextDir, "cache", "telemetry");
+  if (canUseDirectory(fallback)) {
+    const warningKey = `${primary}->${fallback}`;
+    if (!warnedFallbacks.has(warningKey)) {
+      warnedFallbacks.add(warningKey);
+      process.stderr.write(
+        `[cortex-enterprise] telemetry dir not writable at ${primary}; using ${fallback}\n`,
+      );
+    }
+    return fallback;
+  }
+
+  return primary;
+}
+
+export function telemetryStatePath(
+  contextDir: string,
+  ...parts: string[]
+): string {
+  return join(resolveTelemetryStateDir(contextDir), ...parts);
+}

package/scaffold/mcp/src/daemon/main.ts

@@ -13,6 +13,7 @@ import type {
 import { loadEnterpriseConfig, resolveEnterpriseActivation } from "../core/config.js";
 import { pushMetrics } from "../enterprise/telemetry/sync.js";
 import { TelemetryCollector, type TelemetryMetrics } from "../core/telemetry/collector.js";
+import { resolveTelemetryStateDir, telemetryStatePath } from "../core/telemetry/state-dir.js";
 import { AuditWriter, type AuditEntry } from "../core/audit/writer.js";
 import { PolicyStore } from "../core/policy/store.js";
 import {
@@ -84,7 +85,7 @@ async function policyCheck(
 }
 
 function readMetrics(contextDir: string): TelemetryMetrics | null {
-  const path = join(contextDir, "telemetry", "metrics.json");
+  const path = telemetryStatePath(contextDir, "metrics.json");
   if (!existsSync(path)) return null;
   try {
     return JSON.parse(readFileSync(path, "utf8")) as TelemetryMetrics;
@@ -103,7 +104,7 @@ type PendingPush = {
 };
 
 function pendingPushPath(contextDir: string): string {
-  return join(contextDir, "telemetry", "pending-push.json");
+  return telemetryStatePath(contextDir, "pending-push.json");
 }
 
 function readPendingPush(contextDir: string): PendingPush | null {
@@ -118,7 +119,7 @@ function readPendingPush(contextDir: string): PendingPush | null {
 
 function writePendingPush(contextDir: string, pending: PendingPush): void {
   const path = pendingPushPath(contextDir);
-  mkdirSync(join(contextDir, "telemetry"), { recursive: true });
+  mkdirSync(resolveTelemetryStateDir(contextDir), { recursive: true });
   writeFileSync(path, JSON.stringify(pending, null, 2), "utf8");
 }
 

package/scaffold/mcp/tests/telemetry-collector.test.mjs

@@ -1,6 +1,6 @@
 import test from "node:test";
 import assert from "node:assert/strict";
-import { mkdtempSync } from "node:fs";
+import { chmodSync, existsSync, mkdtempSync, mkdirSync } from "node:fs";
 import { tmpdir } from "node:os";
 import path from "node:path";
 
@@ -28,3 +28,33 @@ test("TelemetryCollector counts context.impact as an impact analysis", () => {
   assert.equal(metrics.impact_analyses, 1);
   assert.equal(metrics.tool_metrics["context.impact"].calls, 1);
 });
+
+test("TelemetryCollector falls back when .context/telemetry is not writable", () => {
+  const contextDir = createContextDir("cortex-telemetry-fallback-");
+  const primaryTelemetryDir = path.join(contextDir, "telemetry");
+  const fallbackMetricsPath = path.join(
+    contextDir,
+    "cache",
+    "telemetry",
+    "metrics.json",
+  );
+
+  mkdirSync(primaryTelemetryDir, { recursive: true });
+  chmodSync(primaryTelemetryDir, 0o555);
+
+  try {
+    const collector = new TelemetryCollector(contextDir, "test-version");
+    collector.recordEvent({
+      tool: "context.search",
+      phase: "success",
+      result_count: 1,
+      estimated_tokens_saved: 100,
+      duration_ms: 10,
+    });
+    collector.flush();
+
+    assert.equal(existsSync(fallbackMetricsPath), true);
+  } finally {
+    chmodSync(primaryTelemetryDir, 0o755);
+  }
+});