memtrace-skills 1.1.7 → 1.1.9

package/dist/commands/doctor.d.ts

@@ -9,8 +9,19 @@ export interface DoctorReport {
     binaryOk: boolean;
     binaryPath?: string;
     binaryVersion?: string;
+    /**
+     * Windows only: whether the Microsoft Visual C++ 2015-2022 runtime the
+     * embedding runtime (onnxruntime.dll) links against is installed.
+     * `undefined` on other platforms. MEMBIZ-64: on a fresh Windows machine
+     * `LoadLibraryExW failed` with the DLL present means this runtime is
+     * missing, and nothing else in the install flow said so.
+     */
+    vcRuntimeOk?: boolean;
+    vcRuntimeDir?: string;
     agents: AgentReport[];
 }
+export declare const VC_REDIST_URL = "https://aka.ms/vs/17/release/vc_redist.x64.exe";
+export declare function checkWindowsRuntime(platform?: NodeJS.Platform, systemRoot?: string | undefined): Pick<DoctorReport, 'vcRuntimeOk' | 'vcRuntimeDir'>;
 export declare function runDoctorChecks(): Promise<DoctorReport>;
 export declare function formatReport(r: DoctorReport): string;
 export declare function runDoctor(): Promise<number>;

package/dist/commands/doctor.js

@@ -4,6 +4,16 @@ import path from 'path';
 import { commandExists, execCommand } from '../utils.js';
 import { safeReadJson } from '../fs-safe.js';
 import { ALL_TRANSFORMERS } from '../transformers/index.js';
+export const VC_REDIST_URL = 'https://aka.ms/vs/17/release/vc_redist.x64.exe';
+/** The DLLs onnxruntime.dll needs from the VC++ 2015-2022 x64 runtime. */
+const VC_RUNTIME_DLLS = ['vcruntime140.dll', 'vcruntime140_1.dll', 'msvcp140.dll'];
+export function checkWindowsRuntime(platform = process.platform, systemRoot = process.env.SystemRoot) {
+    if (platform !== 'win32')
+        return {};
+    const vcRuntimeDir = path.join(systemRoot ?? 'C:\\Windows', 'System32');
+    const vcRuntimeOk = VC_RUNTIME_DLLS.every(dll => fs.existsSync(path.join(vcRuntimeDir, dll)));
+    return { vcRuntimeOk, vcRuntimeDir };
+}
 async function checkBinary() {
     if (!(await commandExists('memtrace')))
         return { binaryOk: false };
@@ -95,12 +105,19 @@ function vscodeUserMcpPath() {
 function checkAgent(agent) {
     if (agent === 'claude') {
         const skillsDir = path.join(os.homedir(), '.claude', 'skills');
-        const mcpConfigPath = path.join(os.homedir(), '.claude', 'settings.json');
+        // MEMBIZ-66: Claude Code reads user-scope MCP servers from ~/.claude.json.
+        // An entry that exists only in ~/.claude/settings.json is ignored by the
+        // CLI, so reporting it as registered was a false positive — check the
+        // file the CLI reads first and only fall back to settings.json.
+        const userConfigPath = path.join(os.homedir(), '.claude.json');
+        const settingsPath = path.join(os.homedir(), '.claude', 'settings.json');
+        const inUserConfig = mcpHasMemtrace(userConfigPath);
+        const mcpConfigPath = inUserConfig ? userConfigPath : settingsPath;
         return {
             agent,
             skillsFound: countMemtraceSkills(skillsDir),
             skillsDir,
-            mcpRegistered: mcpHasMemtrace(mcpConfigPath),
+            mcpRegistered: inUserConfig,
             mcpConfigPath,
         };
     }
@@ -206,6 +223,7 @@ export async function runDoctorChecks() {
     const binary = await checkBinary();
     return {
         ...binary,
+        ...checkWindowsRuntime(),
         agents: ALL_TRANSFORMERS.map(t => checkAgent(t.name)),
     };
 }
@@ -217,6 +235,13 @@ export function formatReport(r) {
     lines.push(`${check(r.binaryOk)} memtrace binary${r.binaryPath ? '   ' + r.binaryPath : ''}`);
     if (r.binaryVersion)
         lines.push(`  version: ${r.binaryVersion}`);
+    if (r.vcRuntimeOk !== undefined) {
+        lines.push(`${check(r.vcRuntimeOk)} Visual C++ 2015-2022 runtime   ${r.vcRuntimeDir ?? ''}`);
+        if (!r.vcRuntimeOk) {
+            lines.push(`  memtrace's embedding runtime (onnxruntime.dll) cannot load without it.`);
+            lines.push(`  install: ${VC_REDIST_URL}`);
+        }
+    }
     for (const a of r.agents) {
         lines.push(`${check(a.skillsFound > 0)} ${a.agent} skills installed   ${a.skillsFound} in ${a.skillsDir}`);
         const integrationLabel = a.agent === 'pi' ? 'Pi package registered' : 'MCP registered';
@@ -228,6 +253,7 @@ export async function runDoctor() {
     const report = await runDoctorChecks();
     console.log(formatReport(report));
     const healthy = report.binaryOk
+        && report.vcRuntimeOk !== false
         && report.agents.every(a => a.skillsFound > 0 && a.mcpRegistered);
     return healthy ? 0 : 1;
 }

package/dist/transformers/claude.d.ts

@@ -75,6 +75,8 @@ export interface SettingsCleanupResult {
  * installer versions and Claude marketplace schema changes.
  */
 export declare function removeClaudeSettingsEntriesAt(settingsPath: string): SettingsCleanupResult;
+/** `~/.claude.json` — where Claude Code stores user-scope `mcpServers`. */
+export declare function claudeUserConfigFile(): string;
 /**
  * Full Claude Code plugin installation.
  *

package/dist/transformers/claude.js

@@ -199,10 +199,15 @@ export function registerMcpInSettingsAt(settingsPath, memtraceBinary) {
         ? existing.env
         : {};
     const mergedEnv = { ...existingEnv, ...MEMTRACE_MCP_ENV };
+    // Hosts defer MCP tool definitions behind ToolSearch by default, so an
+    // unmarked server reaches the model as bare tool names while built-in
+    // Grep/Glob/Read arrive with full schemas — a routing bias no instruction
+    // text can overcome. alwaysLoad opts the memtrace tools back into turn one.
     settings.mcpServers['memtrace'] = {
         command: memtraceBinary,
         args: ['mcp'],
         env: mergedEnv,
+        alwaysLoad: true,
     };
     writeJsonAtomic(settingsPath, settings);
     return { registered: true };
@@ -499,9 +504,22 @@ async function registerMcpServer(memtraceBinaryPath) {
     const viaCli = await tryMcpAddJson(memtraceBinaryPath);
     if (viaCli)
         return;
-    // Strategy 2 (fallback): direct settings.json merge (safe + atomic)
+    // Strategy 2 (fallback): direct JSON merge (safe + atomic).
+    //
+    // MEMBIZ-66: Claude Code reads user-scope MCP servers from `~/.claude.json`,
+    // not from `~/.claude/settings.json`. When the `claude` CLI was missing or
+    // slow (a 5 s timeout on Windows is easy to hit), this fallback wrote only
+    // settings.json — skills loaded, every `mcp__memtrace__*` tool was absent,
+    // and `memtrace doctor` still reported the registration as healthy. Write
+    // the file the CLI actually reads, and keep settings.json for older
+    // tooling that still looks there.
+    registerMcpInSettingsAt(claudeUserConfigFile(), memtraceBinaryPath);
     registerMcpInSettingsAt(settingsFile, memtraceBinaryPath);
 }
+/** `~/.claude.json` — where Claude Code stores user-scope `mcpServers`. */
+export function claudeUserConfigFile() {
+    return path.join(os.homedir(), '.claude.json');
+}
 /**
  * Full Claude Code plugin installation.
  *

package/dist/transformers/shared.js

@@ -89,6 +89,14 @@ export function registerVsCodeMcpAt(filePath, binary) {
         type: 'stdio',
         command: binary,
         args: ['mcp'],
+        // MEMBIZ-50 D2: `cwd` is the ONE anchor that drives both store
+        // resolution and repository discovery in `memtrace mcp`. Without it a
+        // user-scope entry spawns the server in the editor's own working
+        // directory (often the user profile), where it resolves a stale
+        // user-level `.memdb` instead of the project's — every project then
+        // shares one wrong, empty store. VS Code substitutes the variable per
+        // window, so one global entry serves every project correctly.
+        cwd: '${workspaceFolder}',
         env: MEMTRACE_MCP_ENV,
     };
     writeJsonAtomic(filePath, cfg);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "memtrace-skills",
-  "version": "1.1.7",
+  "version": "1.1.9",
   "description": "Memtrace skills for AI coding agents — codebase exploration, temporal evolution, impact analysis, and more.",
   "type": "module",
   "bin": {