@phren/cli 0.0.10 → 0.0.11

package/mcp/dist/cli-govern.js

@@ -144,7 +144,7 @@ export async function handlePruneMemories(args = []) {
                 .filter((e) => e !== null);
         }
         catch (err) {
-            if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+            if ((process.env.PHREN_DEBUG))
                 process.stderr.write(`[phren] cli-govern retrievalLog readParse: ${errorMessage(err)}\n`);
         }
     }
@@ -255,7 +255,7 @@ export async function handleGcMaintain(args = []) {
     const sevenDaysAgo = new Date(Date.now() - 7 * 86400000).toISOString().slice(0, 10);
     let oldCommits = [];
     try {
-        const raw = execSync(`git log --oneline --before="${sevenDaysAgo}" --format="%H %s"`, { cwd: phrenPath, encoding: "utf8" }).trim();
+        const raw = execSync(`git log --before="${sevenDaysAgo}" --format="%H %ci %s"`, { cwd: phrenPath, encoding: "utf8" }).trim();
         if (raw) {
             oldCommits = raw.split("\n").filter((l) => l.includes("auto-save:") || l.includes("[auto]"));
         }
@@ -271,23 +271,21 @@ export async function handleGcMaintain(args = []) {
         report.commitsSquashed = oldCommits.length;
     }
     else {
-        // Group by ISO week (YYYY-Www) based on commit timestamp
+        // Group by ISO week based on commit timestamp (already in the log output)
         const commitsByWeek = new Map();
         for (const line of oldCommits) {
             const hash = line.split(" ")[0];
-            try {
-                const dateStr = execSync(`git log -1 --format="%ci" ${hash}`, { cwd: phrenPath, encoding: "utf8" }).trim();
-                const date = new Date(dateStr);
-                const weekStart = new Date(date);
-                weekStart.setDate(date.getDate() - date.getDay()); // start of week (Sunday)
-                const weekKey = weekStart.toISOString().slice(0, 10);
-                if (!commitsByWeek.has(weekKey))
-                    commitsByWeek.set(weekKey, []);
-                commitsByWeek.get(weekKey).push(hash);
-            }
-            catch {
-                // Skip commits we can't resolve
-            }
+            // Format: "HASH YYYY-MM-DD HH:MM:SS +ZZZZ subject..."
+            const dateMatch = line.match(/^[a-f0-9]+ (\d{4}-\d{2}-\d{2})/);
+            if (!dateMatch)
+                continue;
+            const date = new Date(dateMatch[1]);
+            const weekStart = new Date(date);
+            weekStart.setDate(date.getDate() - date.getDay());
+            const weekKey = weekStart.toISOString().slice(0, 10);
+            if (!commitsByWeek.has(weekKey))
+                commitsByWeek.set(weekKey, []);
+            commitsByWeek.get(weekKey).push(hash);
         }
         // For each week with multiple commits, soft-reset to oldest and amend into a summary
         for (const [weekKey, hashes] of commitsByWeek.entries()) {
@@ -320,64 +318,38 @@ export async function handleGcMaintain(args = []) {
             console.log("Commit squash: all old auto-save weeks have only one commit, nothing to squash.");
         }
     }
-    // 3. Prune stale session markers from ~/.phren/.sessions/ older than 30 days
-    const sessionsDir = path.join(phrenPath, ".sessions");
+    // 3–4. Prune stale files from .sessions/ and .runtime/
     const thirtyDaysAgo = Date.now() - 30 * 86400000;
-    if (fs.existsSync(sessionsDir)) {
-        const entries = fs.readdirSync(sessionsDir);
-        for (const entry of entries) {
-            const fullPath = path.join(sessionsDir, entry);
-            try {
-                const stat = fs.statSync(fullPath);
-                if (stat.mtimeMs < thirtyDaysAgo) {
-                    if (dryRun) {
-                        console.log(`[dry-run] Would remove session marker: .sessions/${entry}`);
-                    }
-                    else {
-                        fs.unlinkSync(fullPath);
-                    }
-                    report.sessionsRemoved++;
-                }
-            }
-            catch {
-                // Skip unreadable entries
-            }
-        }
-    }
-    const sessionsVerb = dryRun ? "Would remove" : "Removed";
-    console.log(`${sessionsVerb} ${report.sessionsRemoved} stale session marker(s) from .sessions/`);
-    // 4. Trim runtime logs from ~/.phren/.runtime/ older than 30 days
-    const runtimeDir = path.join(phrenPath, ".runtime");
-    const logExtensions = new Set([".log", ".jsonl", ".json"]);
-    if (fs.existsSync(runtimeDir)) {
-        const entries = fs.readdirSync(runtimeDir);
-        for (const entry of entries) {
-            const ext = path.extname(entry);
-            if (!logExtensions.has(ext))
-                continue;
-            // Never trim the active audit log or telemetry config
-            if (entry === "audit.log" || entry === "telemetry.json")
+    function pruneStaleFiles(dir, label, filter) {
+        let removed = 0;
+        if (!fs.existsSync(dir))
+            return removed;
+        for (const entry of fs.readdirSync(dir)) {
+            if (filter && !filter(entry))
                 continue;
-            const fullPath = path.join(runtimeDir, entry);
+            const fullPath = path.join(dir, entry);
             try {
-                const stat = fs.statSync(fullPath);
-                if (stat.mtimeMs < thirtyDaysAgo) {
+                if (fs.statSync(fullPath).mtimeMs < thirtyDaysAgo) {
                     if (dryRun) {
-                        console.log(`[dry-run] Would remove runtime log: .runtime/${entry}`);
+                        console.log(`[dry-run] Would remove: ${label}/${entry}`);
                     }
                     else {
                         fs.unlinkSync(fullPath);
                     }
-                    report.runtimeLogsRemoved++;
+                    removed++;
                 }
             }
-            catch {
-                // Skip unreadable entries
-            }
+            catch { /* skip unreadable */ }
         }
+        return removed;
     }
-    const logsVerb = dryRun ? "Would remove" : "Removed";
-    console.log(`${logsVerb} ${report.runtimeLogsRemoved} stale runtime log(s) from .runtime/`);
+    const logExtensions = new Set([".log", ".jsonl", ".json"]);
+    const protectedFiles = new Set(["audit.log", "telemetry.json"]);
+    report.sessionsRemoved = pruneStaleFiles(path.join(phrenPath, ".sessions"), ".sessions");
+    report.runtimeLogsRemoved = pruneStaleFiles(path.join(phrenPath, ".runtime"), ".runtime", (entry) => logExtensions.has(path.extname(entry)) && !protectedFiles.has(entry));
+    const verb = dryRun ? "Would remove" : "Removed";
+    console.log(`${verb} ${report.sessionsRemoved} stale session marker(s) from .sessions/`);
+    console.log(`${verb} ${report.runtimeLogsRemoved} stale runtime log(s) from .runtime/`);
     // 5. Summary
     if (!dryRun) {
         appendAuditLog(phrenPath, "maintain_gc", `gitGc=${report.gitGcRan} squashed=${report.commitsSquashed} sessions=${report.sessionsRemoved} logs=${report.runtimeLogsRemoved}`);
@@ -529,7 +501,7 @@ export async function handleBackgroundMaintenance(projectArg) {
             fs.unlinkSync(markers.lock);
         }
         catch (err) {
-            if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+            if ((process.env.PHREN_DEBUG))
                 process.stderr.write(`[phren] cli-govern backgroundMaintenance unlockFinal: ${errorMessage(err)}\n`);
         }
     }

package/mcp/dist/cli-graph.js

@@ -1,6 +1,7 @@
 import { getPhrenPath } from "./shared.js";
 import { buildIndex, queryRows } from "./shared-index.js";
 import { resolveRuntimeProfile } from "./runtime-profile.js";
+import { errorMessage } from "./utils.js";
 /**
  * CLI: phren graph [--project <name>] [--limit <n>]
  * Displays the fragment knowledge graph as a table.
@@ -115,7 +116,7 @@ export async function handleGraphLink(args) {
         db.run("INSERT OR IGNORE INTO entity_links (source_id, target_id, rel_type, source_doc) VALUES (?, ?, ?, ?)", [sourceId, targetId, "mentions", sourceDoc]);
     }
     catch (err) {
-        console.error(`Failed to link: ${err instanceof Error ? err.message : String(err)}`);
+        console.error(`Failed to link: ${errorMessage(err)}`);
         process.exit(1);
     }
     // Persist to manual-links.json
@@ -144,7 +145,7 @@ export async function handleGraphLink(args) {
         });
     }
     catch (err) {
-        console.error(`Failed to persist manual link: ${err instanceof Error ? err.message : String(err)}`);
+        console.error(`Failed to persist manual link: ${errorMessage(err)}`);
         process.exit(1);
     }
     console.log(`Linked "${fragmentName}" to ${sourceDoc}.`);

package/mcp/dist/cli-hooks-globs.js

@@ -1,6 +1,7 @@
 import * as fs from "fs";
 import * as path from "path";
 import { capCache } from "./shared.js";
+import { errorMessage } from "./utils.js";
 // ── Glob matching and project frontmatter ────────────────────────────────────
 const projectGlobCache = new Map();
 export function clearProjectGlobCache() {
@@ -45,7 +46,7 @@ function parseProjectGlobs(phrenPathLocal, project) {
     }
     catch (err) {
         if (process.env.PHREN_DEBUG)
-            process.stderr.write(`[phren] getProjectGlobs: ${err instanceof Error ? err.message : String(err)}\n`);
+            process.stderr.write(`[phren] getProjectGlobs: ${errorMessage(err)}\n`);
     }
     projectGlobCache.set(project, globs);
     capCache(projectGlobCache);

package/mcp/dist/cli-hooks-output.js

@@ -1,7 +1,7 @@
 import { recordInjection, recordRetrieval, } from "./shared-governance.js";
 import { getDocSourceKey, } from "./shared-index.js";
 import { logImpact, extractFindingIdsFromSnippet, } from "./finding-impact.js";
-import { isFeatureEnabled } from "./utils.js";
+import { isFeatureEnabled, errorMessage } from "./utils.js";
 import { annotateStale } from "./cli-hooks-citations.js";
 import { approximateTokens, fileRelevanceBoost, branchMatchBoost } from "./shared-retrieval.js";
 // ── Progressive disclosure helpers ────────────────────────────────────────────
@@ -56,7 +56,7 @@ export function buildHookOutput(selected, usedTokens, intent, gitCtx, detectedPr
             }
             catch (err) {
                 if (process.env.PHREN_DEBUG)
-                    process.stderr.write(`[phren] injectContext recordRetrieval: ${err instanceof Error ? err.message : String(err)}\n`);
+                    process.stderr.write(`[phren] injectContext recordRetrieval: ${errorMessage(err)}\n`);
             }
         }
     }
@@ -108,7 +108,7 @@ export function buildHookOutput(selected, usedTokens, intent, gitCtx, detectedPr
             }
             catch (err) {
                 if (process.env.PHREN_DEBUG)
-                    process.stderr.write(`[phren] injectContext recordRetrievalOrdered: ${err instanceof Error ? err.message : String(err)}\n`);
+                    process.stderr.write(`[phren] injectContext recordRetrievalOrdered: ${errorMessage(err)}\n`);
             }
             parts.push(`[${getDocSourceKey(doc, phrenPathLocal)}] (${doc.type})`);
             parts.push(annotateStale(snippet));

package/mcp/dist/cli-hooks.js

@@ -5,7 +5,7 @@
 //   cli-hooks-output.ts     — hook output formatting
 //   cli-hooks-globs.ts      — project glob matching
 import { debugLog, sessionMarker, sessionsDir, getPhrenPath, } from "./shared.js";
-import { getRetentionPolicy, getWorkflowPolicy, } from "./shared-governance.js";
+import { mergeConfig, } from "./shared-governance.js";
 import { buildIndex, detectProject, } from "./shared-index.js";
 import { isProjectHookEnabled } from "./project-config.js";
 import { checkConsolidationNeeded, } from "./shared-content.js";
@@ -126,7 +126,7 @@ export async function handleHookPrompt() {
     }
     catch (err) {
         if (process.env.PHREN_DEBUG)
-            process.stderr.write(`[phren] hookPrompt stdinRead: ${err instanceof Error ? err.message : String(err)}\n`);
+            process.stderr.write(`[phren] hookPrompt stdinRead: ${errorMessage(err)}\n`);
         process.exit(0);
     }
     const input = parseHookInput(raw);
@@ -182,6 +182,7 @@ export async function handleHookPrompt() {
         appendAuditLog(getPhrenPath(), "hook_prompt", `status=project_disabled project=${detectedProject}`);
         process.exit(0);
     }
+    const resolvedConfig = mergeConfig(getPhrenPath(), detectedProject ?? undefined);
     const safeQuery = buildRobustFtsQuery(keywords, detectedProject, getPhrenPath());
     if (!safeQuery)
         process.exit(0);
@@ -193,14 +194,17 @@ export async function handleHookPrompt() {
             process.exit(0);
         autoLearnQuerySynonyms(getPhrenPath(), detectedProject, keywordEntries, rows);
         const tTrust0 = Date.now();
-        const policy = getRetentionPolicy(getPhrenPath());
+        const policy = resolvedConfig.retentionPolicy;
         const memoryTtlDays = Number.parseInt(process.env.PHREN_MEMORY_TTL_DAYS || String(policy.ttlDays), 10);
         const trustResult = applyTrustFilter(rows, Number.isNaN(memoryTtlDays) ? policy.ttlDays : memoryTtlDays, policy.minInjectConfidence, policy.decay, getPhrenPath());
         rows = trustResult.rows;
         stage.trustMs = Date.now() - tTrust0;
         if (!rows.length)
             process.exit(0);
-        if (isFeatureEnabled("PHREN_FEATURE_AUTO_EXTRACT", true) && getProactivityLevelForFindings(getPhrenPath()) !== "low" && sessionId && detectedProject && cwd) {
+        const findingsProactivity = resolvedConfig.proactivity.findings
+            ?? resolvedConfig.proactivity.base
+            ?? getProactivityLevelForFindings(getPhrenPath());
+        if (isFeatureEnabled("PHREN_FEATURE_AUTO_EXTRACT", true) && findingsProactivity !== "low" && sessionId && detectedProject && cwd) {
             const marker = sessionMarker(getPhrenPath(), `extracted-${sessionId}-${detectedProject}`);
             if (!fs.existsSync(marker)) {
                 try {
@@ -260,7 +264,9 @@ export async function handleHookPrompt() {
             debugLog(`injection-budget: trimmed ${selected.length} -> ${kept.length} snippets to fit ${maxInjectTokens} token budget`);
         }
         const parts = buildHookOutput(budgetSelected, budgetUsedTokens, intent, gitCtx, detectedProject, stage, safeTokenBudget, getPhrenPath(), sessionId);
-        const taskLevel = getProactivityLevelForTask(getPhrenPath());
+        const taskLevel = resolvedConfig.proactivity.tasks
+            ?? resolvedConfig.proactivity.base
+            ?? getProactivityLevelForTask(getPhrenPath());
         const taskLifecycle = handleTaskPromptLifecycle({
             phrenPath: getPhrenPath(),
             prompt,
@@ -275,8 +281,7 @@ export async function handleHookPrompt() {
         }
         // Inject finding sensitivity agent instruction
         try {
-            const workflowPolicy = getWorkflowPolicy(getPhrenPath());
-            const sensitivity = workflowPolicy.findingSensitivity ?? "balanced";
+            const sensitivity = resolvedConfig.findingSensitivity ?? "balanced";
             const sensitivityConfig = FINDING_SENSITIVITY_CONFIG[sensitivity];
             if (sensitivityConfig) {
                 parts.push("");
@@ -304,36 +309,38 @@ export async function handleHookPrompt() {
         const noticeFile = sessionId ? sessionMarker(getPhrenPath(), `noticed-${sessionId}`) : null;
         const alreadyNoticed = noticeFile ? fs.existsSync(noticeFile) : false;
         if (!alreadyNoticed) {
-            // Clean up stale session markers (>24h old) from .sessions/ dir
-            try {
-                const cutoff = Date.now() - 86400000;
-                const sessDir = sessionsDir(getPhrenPath());
-                if (fs.existsSync(sessDir)) {
-                    for (const f of fs.readdirSync(sessDir)) {
-                        if (!f.startsWith("noticed-") && !f.startsWith("extracted-"))
+            // Defer stale session marker cleanup to after output — it's I/O-heavy and not needed for results
+            setImmediate(() => {
+                try {
+                    const cutoff = Date.now() - 86400000;
+                    const sessDir = sessionsDir(getPhrenPath());
+                    if (fs.existsSync(sessDir)) {
+                        for (const f of fs.readdirSync(sessDir)) {
+                            if (!f.startsWith("noticed-") && !f.startsWith("extracted-"))
+                                continue;
+                            const fp = `${sessDir}/${f}`;
+                            try {
+                                if (fs.statSync(fp).mtimeMs < cutoff)
+                                    fs.unlinkSync(fp);
+                            }
+                            catch { /* ignore per-file errors */ }
+                        }
+                    }
+                    // Also clean stale markers from the phren root
+                    for (const f of fs.readdirSync(getPhrenPath())) {
+                        if (!f.startsWith(".noticed-") && !f.startsWith(".extracted-"))
                             continue;
-                        const fp = `${sessDir}/${f}`;
-                        if (fs.statSync(fp).mtimeMs < cutoff)
+                        const fp = `${getPhrenPath()}/${f}`;
+                        try {
                             fs.unlinkSync(fp);
+                        }
+                        catch { /* ignore */ }
                     }
                 }
-                // Also clean stale markers from the phren root
-                for (const f of fs.readdirSync(getPhrenPath())) {
-                    if (!f.startsWith(".noticed-") && !f.startsWith(".extracted-"))
-                        continue;
-                    const fp = `${getPhrenPath()}/${f}`;
-                    try {
-                        fs.unlinkSync(fp);
-                    }
-                    catch (err) {
-                        if (process.env.PHREN_DEBUG)
-                            process.stderr.write(`[phren] hookPrompt staleNoticeUnlink: ${errorMessage(err)}\n`);
-                    }
+                catch (err) {
+                    debugLog(`stale notice cleanup failed: ${errorMessage(err)}`);
                 }
-            }
-            catch (err) {
-                debugLog(`stale notice cleanup failed: ${errorMessage(err)}`);
-            }
+            });
             const needed = checkConsolidationNeeded(getPhrenPath(), profile);
             if (needed.length > 0) {
                 const notices = needed.map((n) => {

package/mcp/dist/cli-namespaces.js

@@ -5,6 +5,7 @@ import { expandHomePath, findProjectNameCaseInsensitive, getPhrenPath, getProjec
 import { isValidProjectName, errorMessage } from "./utils.js";
 import { readInstallPreferences, writeInstallPreferences } from "./init-preferences.js";
 import { buildSkillManifest, findLocalSkill, findSkill, getAllSkills } from "./skill-registry.js";
+import { detectSkillCollisions } from "./link-skills.js";
 import { setSkillEnabledAndSync, syncSkillLinksForScope } from "./skill-files.js";
 import { findProjectDir } from "./project-locator.js";
 import { TASK_FILE_ALIASES, addTask, completeTask, updateTask, reorderTask, pinTask, removeTask, workNextTask, tidyDoneTasks, linkTaskIssue, promoteTask, resolveTaskItem } from "./data-tasks.js";
@@ -76,7 +77,7 @@ function openInEditor(filePath) {
         execFileSync(editor, [filePath], { stdio: "inherit" });
     }
     catch (err) {
-        if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+        if ((process.env.PHREN_DEBUG))
             process.stderr.write(`[phren] openInEditor: ${errorMessage(err)}\n`);
         console.error(`Editor "${editor}" failed. Set $EDITOR to your preferred editor.`);
         process.exit(1);
@@ -145,7 +146,7 @@ export function handleSkillsNamespace(args, profile) {
             console.log(`Linked skill ${fileName} into ${project}.`);
         }
         catch (err) {
-            if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+            if ((process.env.PHREN_DEBUG))
                 process.stderr.write(`[phren] skill add symlinkFailed: ${errorMessage(err)}\n`);
             fs.copyFileSync(source, dest);
             console.log(`Copied skill ${fileName} into ${project}.`);
@@ -450,7 +451,7 @@ export function handleDetectSkills(args, profile) {
                 continue;
         }
         catch (err) {
-            if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+            if ((process.env.PHREN_DEBUG))
                 process.stderr.write(`[phren] skillList lstat: ${errorMessage(err)}\n`);
         }
         const name = entry.replace(/\.md$/, "");
@@ -548,6 +549,15 @@ function printSkillDoctor(scope, manifest, destDir) {
                 problems.push(`Mirror drift for ${skill.name}: expected ${dest} -> ${skill.root}`);
             }
         }
+        // Check for user-owned files blocking phren skill links
+        const phrenPath = getPhrenPath();
+        const srcDir = scope.toLowerCase() === "global"
+            ? path.join(phrenPath, "global", "skills")
+            : path.join(phrenPath, scope, "skills");
+        const collisions = detectSkillCollisions(srcDir, destDir, phrenPath);
+        for (const collision of collisions) {
+            problems.push(`Skill collision: ${collision.message}`);
+        }
     }
     if (!manifest.problems.length && !problems.length) {
         console.log("\nDoctor: no skill pipeline issues detected.");
@@ -853,7 +863,7 @@ function handleProjectsList(profile) {
             dirFiles = new Set(fs.readdirSync(projectDir));
         }
         catch (err) {
-            if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+            if ((process.env.PHREN_DEBUG))
                 process.stderr.write(`[phren] projects list readdir: ${errorMessage(err)}\n`);
             dirFiles = new Set();
         }
@@ -896,7 +906,7 @@ async function handleProjectsRemove(name, profile) {
         countFiles(projectDir);
     }
     catch (err) {
-        if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+        if ((process.env.PHREN_DEBUG))
             process.stderr.write(`[phren] projects remove countFiles: ${errorMessage(err)}\n`);
     }
     const readline = await import("readline");

package/mcp/dist/cli-search.js

@@ -34,7 +34,7 @@ export function readSearchHistory(phrenPath) {
             .map((line) => JSON.parse(line));
     }
     catch (err) {
-        if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+        if ((process.env.PHREN_DEBUG))
             process.stderr.write(`[phren] readSearchHistory: ${errorMessage(err)}\n`);
         return [];
     }
@@ -253,7 +253,7 @@ export async function runSearch(opts, phrenPath, profile) {
                     logSearchMiss(phrenPath, opts.query, opts.project);
                 }
                 catch (err) {
-                    if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+                    if ((process.env.PHREN_DEBUG))
                         process.stderr.write(`[phren] search logSearchMiss: ${errorMessage(err)}\n`);
                 }
             }

package/mcp/dist/content-archive.js

@@ -107,7 +107,7 @@ function isAlreadyArchived(referenceDir, bullet) {
         }
     }
     catch (err) {
-        if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+        if ((process.env.PHREN_DEBUG))
             process.stderr.write(`[phren] isDuplicateInReference: ${errorMessage(err)}\n`);
     }
     return false;
@@ -271,7 +271,7 @@ export function autoArchiveToReference(phrenPath, project, keepCount) {
             fs.unlinkSync(lockFile);
         }
         catch (err) {
-            if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+            if ((process.env.PHREN_DEBUG))
                 process.stderr.write(`[phren] autoArchiveToReference unlockFile: ${errorMessage(err)}\n`);
         }
     }

package/mcp/dist/content-dedup.js

@@ -2,7 +2,7 @@ import * as fs from "fs";
 import * as path from "path";
 import * as crypto from "crypto";
 import { debugLog, runtimeFile, KNOWN_OBSERVATION_TAGS } from "./shared.js";
-import { isFeatureEnabled, safeProjectPath } from "./utils.js";
+import { isFeatureEnabled, safeProjectPath, errorMessage } from "./utils.js";
 import { UNIVERSAL_TECH_TERMS_RE, EXTRA_ENTITY_PATTERNS } from "./phren-core.js";
 import { isInactiveFindingLine } from "./finding-lifecycle.js";
 // ── LLM provider abstraction ────────────────────────────────────────────────
@@ -50,8 +50,8 @@ async function withCache(cachePath, key, ttlMs, compute) {
         }
     }
     catch (err) {
-        if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
-            process.stderr.write(`[phren] withCache load (${path.basename(cachePath)}): ${err instanceof Error ? err.message : String(err)}\n`);
+        if ((process.env.PHREN_DEBUG))
+            process.stderr.write(`[phren] withCache load (${path.basename(cachePath)}): ${errorMessage(err)}\n`);
     }
     const result = await compute();
     // Persist result
@@ -61,8 +61,8 @@ async function withCache(cachePath, key, ttlMs, compute) {
         persistCache(cachePath, cache);
     }
     catch (err) {
-        if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
-            process.stderr.write(`[phren] withCache persist (${path.basename(cachePath)}): ${err instanceof Error ? err.message : String(err)}\n`);
+        if ((process.env.PHREN_DEBUG))
+            process.stderr.write(`[phren] withCache persist (${path.basename(cachePath)}): ${errorMessage(err)}\n`);
     }
     return result;
 }
@@ -562,8 +562,8 @@ export async function checkSemanticConflicts(phrenPath, project, newFinding, sig
                 return { name: e.name, mtime: fs.statSync(fp).mtimeMs, fp };
             }
             catch (err) {
-                if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
-                    process.stderr.write(`[phren] crossProjectScan stat: ${err instanceof Error ? err.message : String(err)}\n`);
+                if ((process.env.PHREN_DEBUG))
+                    process.stderr.write(`[phren] crossProjectScan stat: ${errorMessage(err)}\n`);
                 return null;
             }
         })
@@ -577,8 +577,8 @@ export async function checkSemanticConflicts(phrenPath, project, newFinding, sig
         }
     }
     catch (err) {
-        if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
-            process.stderr.write(`[phren] crossProjectScan: ${err instanceof Error ? err.message : String(err)}\n`);
+        if ((process.env.PHREN_DEBUG))
+            process.stderr.write(`[phren] crossProjectScan: ${errorMessage(err)}\n`);
     }
     const annotations = [];
     const deadline = Date.now() + CONFLICT_CHECK_TOTAL_TIMEOUT_MS;

package/mcp/dist/embedding.js

@@ -79,7 +79,7 @@ async function openCacheDb(phrenPath) {
             db?.close();
         }
         catch (e2) {
-            if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+            if ((process.env.PHREN_DEBUG))
                 process.stderr.write(`[phren] embedding openCacheDb dbClose: ${e2 instanceof Error ? e2.message : String(e2)}\n`);
         }
         throw err;
@@ -126,13 +126,13 @@ function persistDb(phrenPath, db) {
                 }
             }
             catch (err) {
-                if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
-                    process.stderr.write(`[phren] embedding persistDb onDiskLoad: ${err instanceof Error ? err.message : String(err)}\n`);
+                if ((process.env.PHREN_DEBUG))
+                    process.stderr.write(`[phren] embedding persistDb onDiskLoad: ${errorMessage(err)}\n`);
                 try {
                     onDisk?.close();
                 }
                 catch (e2) {
-                    if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+                    if ((process.env.PHREN_DEBUG))
                         process.stderr.write(`[phren] embedding persistDb onDiskClose: ${e2 instanceof Error ? e2.message : String(e2)}\n`);
                 }
                 onDisk = null;
@@ -149,7 +149,7 @@ function persistDb(phrenPath, db) {
                         onDisk.close();
                     }
                     catch (e2) {
-                        if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+                        if ((process.env.PHREN_DEBUG))
                             process.stderr.write(`[phren] embedding persistDb onDiskCloseFinally: ${e2 instanceof Error ? e2.message : String(e2)}\n`);
                     }
             }
@@ -269,7 +269,7 @@ export async function getCachedEmbedding(phrenPath, text, apiKey, model) {
             db?.close();
         }
         catch (e2) {
-            if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+            if ((process.env.PHREN_DEBUG))
                 process.stderr.write(`[phren] embedding getCachedEmbedding dbClose: ${e2 instanceof Error ? e2.message : String(e2)}\n`);
         }
     }
@@ -320,7 +320,7 @@ export async function getCachedEmbeddings(phrenPath, texts, apiKey, model) {
             db?.close();
         }
         catch (e2) {
-            if ((process.env.PHREN_DEBUG || process.env.PHREN_DEBUG))
+            if ((process.env.PHREN_DEBUG))
                 process.stderr.write(`[phren] embedding getCachedEmbeddings dbClose: ${e2 instanceof Error ? e2.message : String(e2)}\n`);
         }
     }