@pcircle/memesh 4.8.3 → 4.8.5

package/dist/core/doctor.js

@@ -28,6 +28,7 @@ import { autoCaptureDecision } from './capture-flag.js';
 import { guardFromMetadata } from './guards.js';
 import { getAgentMessageStorageReport } from './agent-message-storage.js';
 import { readHostConfigFile } from '../host-runtime/config.js';
+import { summariseTelemetry } from './llm-telemetry.js';
 const EMBEDDING_PROBE_TIMEOUT_MS = 15000;
 const EXPECTED_HOOK_TYPES = ['PreToolUse', 'SessionStart', 'PostToolUse', 'Stop', 'PreCompact'];
 const AGENT_MESSAGE_STORAGE_QUOTA_ENV = 'MEMESH_AGENT_MESSAGE_STORAGE_QUOTA_BYTES';
@@ -216,28 +217,49 @@ function inspectConfigFile(existsSyncImpl, readFileSyncImpl, getConfigPathImpl,
         return createCheck('config', 'Config', 'fail', `${configPath} could not be read or parsed (${msg}). Every setting in it — LLM provider, fallbacks, embedder — is being silently ignored right now.`, `Fix the JSON or remove the file to fall back to defaults: mv ${configPath} ${configPath}.bak`, { code: 'config-parse.unreadable', params: { path: configPath, detail: msg } });
     }
 }
-function inspectMcpConfig(packageRoot, existsSyncImpl, readFileSyncImpl) {
-    const mcpPath = path.join(packageRoot, '.mcp.json');
+const MCP_PLACEHOLDER = '${CLAUDE_PLUGIN_ROOT}';
+function declaredMcpManifest(packageRoot, readFileSyncImpl) {
+    const parsed = parseJsonFile(path.join(packageRoot, '.claude-plugin', 'plugin.json'), readFileSyncImpl);
+    if (!parsed.ok)
+        return null;
+    const declared = parsed.value.mcpServers;
+    if (typeof declared !== 'string' || !declared.startsWith('./'))
+        return null;
+    return declared.slice(2);
+}
+function inspectMcpConfig(packageRoot, installChannel, existsSyncImpl, readFileSyncImpl, env) {
+    const relativeManifest = declaredMcpManifest(packageRoot, readFileSyncImpl);
+    if (relativeManifest === null) {
+        return createCheck('mcp-config', 'MCP config', 'fail', '.claude-plugin/plugin.json declares no `mcpServers` path, so Claude Code has no MeMesh MCP server to start.', 'Reinstall MeMesh so the plugin manifest and the MCP manifest it names are both restored.', { code: 'mcp-config.missing' });
+    }
+    const label = relativeManifest;
+    const mcpPath = path.join(packageRoot, relativeManifest);
     if (!existsSyncImpl(mcpPath)) {
-        return createCheck('mcp-config', 'MCP config', 'fail', '.mcp.json is missing.', 'Restore `.mcp.json` from the package or reinstall MeMesh.', { code: 'mcp-config.missing' });
+        return createCheck('mcp-config', 'MCP config', 'fail', `${label} is missing.`, 'Reinstall MeMesh so the plugin manifest and the MCP manifest it names are both restored.', { code: 'mcp-config.missing' });
     }
     const parsed = parseJsonFile(mcpPath, readFileSyncImpl);
     if (!parsed.ok) {
-        return createCheck('mcp-config', 'MCP config', 'fail', '.mcp.json is not valid JSON.', `Fix ${mcpPath} so Claude Code can read the MCP server definition.`, { code: 'mcp-config.invalid-json', params: { path: mcpPath } });
+        return createCheck('mcp-config', 'MCP config', 'fail', `${label} is not valid JSON.`, `Fix ${mcpPath} so Claude Code can read the MCP server definition.`, { code: 'mcp-config.invalid-json', params: { path: mcpPath } });
     }
     const server = parsed.value.mcpServers?.memesh;
     if (!server || typeof server.command !== 'string') {
-        return createCheck('mcp-config', 'MCP config', 'fail', '.mcp.json does not define a usable `memesh` MCP server entry.', 'Reinstall MeMesh or restore the `mcpServers.memesh` entry in `.mcp.json`.', { code: 'mcp-config.no-entry' });
+        return createCheck('mcp-config', 'MCP config', 'fail', `${label} does not define a usable \`memesh\` MCP server entry.`, `Reinstall MeMesh or restore the \`mcpServers.memesh\` entry in \`${label}\`.`, { code: 'mcp-config.no-entry' });
     }
     const args = Array.isArray(server.args) ? server.args : [];
     const entry = typeof args[0] === 'string' ? args[0] : null;
     if (entry) {
-        const resolved = path.resolve(entry.replaceAll('${CLAUDE_PLUGIN_ROOT}', packageRoot));
+        const pluginRoot = installChannel === 'plugin-marketplace' ? packageRoot : (env.CLAUDE_PLUGIN_ROOT || null);
+        if (entry.includes(MCP_PLACEHOLDER) && pluginRoot === null) {
+            return createCheck('mcp-config', 'MCP config', 'warn', `${label} starts \`${entry}\`. NOT VERIFIED: \`${MCP_PLACEHOLDER}\` is substituted by the Claude Code plugin runtime, and this is a ${installChannel} install with CLAUDE_PLUGIN_ROOT unset — so the file it names was not checked.`, `Verify it the way the plugin runtime would: CLAUDE_PLUGIN_ROOT=${packageRoot} memesh doctor`, { code: 'mcp-config.placeholder-unresolved', params: { entry, channel: installChannel } });
+        }
+        const resolved = pluginRoot === null
+            ? path.resolve(packageRoot, entry)
+            : path.resolve(entry.replaceAll(MCP_PLACEHOLDER, pluginRoot));
         if (!existsSyncImpl(resolved)) {
-            return createCheck('mcp-config', 'MCP config', 'fail', `.mcp.json starts \`${entry}\`, and that file is not in this install — so every memesh MCP tool fails to start.`, 'Reinstall MeMesh; if you edited `.mcp.json` by hand, point it back at `${CLAUDE_PLUGIN_ROOT}/dist/mcp/server.js`.', { code: 'mcp-config.entry-missing', params: { entry, resolved } });
+            return createCheck('mcp-config', 'MCP config', 'fail', `${label} starts \`${entry}\`, and that file is not in this install — so every memesh MCP tool fails to start.`, `Reinstall MeMesh; if you edited \`${label}\` by hand, point it back at \`${MCP_PLACEHOLDER}/dist/mcp/server.js\`.`, { code: 'mcp-config.entry-missing', params: { entry, resolved } });
         }
     }
-    return createCheck('mcp-config', 'MCP config', 'pass', '.mcp.json is present, defines the memesh MCP server, and the script it starts exists.');
+    return createCheck('mcp-config', 'MCP config', 'pass', `${label} is present, defines the memesh MCP server, and the script it starts exists.`);
 }
 function extractHookScriptPaths(hooksConfig, packageRoot) {
     const hooks = hooksConfig.hooks;
@@ -619,7 +641,32 @@ function inspectNativeBinding(packageRoot, _existsSyncImpl, probeImpl = defaultN
     }
     return createCheck('native-binding', 'SQLite and vector search', 'warn', `sqlite-vec could not be loaded: ${result.message}. Memories are still saved and found by keyword; only search by meaning is off.`, `Run: cd "${packageRoot}" && npm install --omit=dev`, { code: 'native-binding.load-failed', params: { detail: result.message, root: packageRoot } });
 }
-function inspectShellCli(installChannel, packageRoot, resolveShellMemeshImpl) {
+function readVersionFromInstalledBinary(binaryPath, existsSyncImpl, readFileSyncImpl, realpathSyncImpl = fs.realpathSync) {
+    let resolved;
+    try {
+        resolved = realpathSyncImpl(binaryPath);
+    }
+    catch {
+        resolved = binaryPath;
+    }
+    let dir = path.dirname(resolved);
+    for (let depth = 0; depth < 8; depth += 1) {
+        const pkgPath = path.join(dir, 'package.json');
+        if (existsSyncImpl(pkgPath)) {
+            const parsed = parseJsonFile(pkgPath, readFileSyncImpl);
+            if (!parsed.ok)
+                return null;
+            const { name, version } = parsed.value;
+            return name === '@pcircle/memesh' && typeof version === 'string' ? version : null;
+        }
+        const parent = path.dirname(dir);
+        if (parent === dir)
+            return null;
+        dir = parent;
+    }
+    return null;
+}
+function inspectShellCli(installChannel, packageRoot, packageVersion, resolveShellMemeshImpl, existsSyncImpl, readFileSyncImpl) {
     const shellPath = resolveShellMemeshImpl();
     const isSameAsCurrent = shellPath ? path.resolve(shellPath).startsWith(path.resolve(packageRoot)) : false;
     const hasDistinctShellCli = !!shellPath && !isSameAsCurrent;
@@ -629,7 +676,25 @@ function inspectShellCli(installChannel, packageRoot, resolveShellMemeshImpl) {
             : 'Running from npm-global install — shell access available in this terminal.');
     }
     if (hasDistinctShellCli) {
-        return createCheck('shell-cli', 'Shell CLI on PATH', 'pass', `\`memesh\` resolves to ${shellPath} (separate from this install at ${packageRoot}). Both paths coexist and share the same DB.`);
+        const shellVersion = readVersionFromInstalledBinary(shellPath, existsSyncImpl, readFileSyncImpl);
+        const shellIsBehind = shellVersion ? classifyBump(shellVersion, packageVersion) : null;
+        const thisIsBehind = shellVersion ? classifyBump(packageVersion, shellVersion) : null;
+        if (shellIsBehind) {
+            return createCheck('shell-cli', 'Shell CLI on PATH', 'warn', `\`memesh\` resolves to ${shellPath} (separate from this install at ${packageRoot}), and it is running ${shellVersion} — behind this install's ${packageVersion}. Both share the same DB, but an agent using this install and a human typing \`memesh\` in a terminal are running different code.`, 'Run `npm install -g @pcircle/memesh@latest` to bring the shell CLI up to date — a separate global install is never updated automatically by the plugin marketplace.');
+        }
+        if (thisIsBehind) {
+            const pluginHost = installChannel === 'plugin-marketplace' ? detectPluginHost(packageRoot) : null;
+            const fix = installChannel !== 'plugin-marketplace'
+                ? `Update this install (a ${installChannel}) to ${shellVersion} or newer via its own channel — see \`memesh status\`.`
+                : pluginHost
+                    ? `Run \`${PLUGIN_REFRESH_COMMANDS[pluginHost]}\` to bring this plugin copy to ${shellVersion} (or newer).`
+                    : `Bring this plugin copy to ${shellVersion} (or newer) with your host's refresh command — `
+                        + `Claude Code: \`${PLUGIN_REFRESH_COMMANDS['claude-code']}\`; `
+                        + `Codex: \`${PLUGIN_REFRESH_COMMANDS.codex}\`.`;
+            return createCheck('shell-cli', 'Shell CLI on PATH', 'warn', `\`memesh\` resolves to ${shellPath} (separate from this install at ${packageRoot}), and it is running ${shellVersion} — ahead of this install's ${packageVersion}.`, fix);
+        }
+        return createCheck('shell-cli', 'Shell CLI on PATH', 'pass', `\`memesh\` resolves to ${shellPath} (separate from this install at ${packageRoot}). Both paths coexist and share the same DB`
+            + (shellVersion ? `, both on ${packageVersion}.` : ' — could not read the shell copy\'s own version to compare.'));
     }
     if (installChannel === 'plugin-marketplace') {
         const host = detectPluginHost(packageRoot) === 'codex' ? 'Codex CLI' : 'Claude Code';
@@ -827,6 +892,33 @@ function inspectPluginCacheCurrency(installChannel, pluginHost, packageRoot, ins
     }
     return createCheck('plugin-cache', `Plugin cache source record is current (${hostLabel})`, 'warn', `The plugin cache records commit ${installedSha.slice(0, 8)}, but the marketplace has moved to ${marketplaceSha.slice(0, 8)} under the same version — ${hostLabel} does not normally refresh a cache whose version did not change, so refresh the cache before relying on the newer marketplace code.`, `Run \`${command}\` to refresh the cache in place, then restart ${hostLabel}.`, { code: 'plugin-cache.stale', params: { installed: installedSha.slice(0, 8), marketplace: marketplaceSha.slice(0, 8), host: hostLabel, command } });
 }
+function annotateNpmGlobalPluginCacheVersion(check, discoveredPackageRoot, hostLabel, runningVersion) {
+    const cacheRoot = path.dirname(discoveredPackageRoot);
+    const discoveredVersion = path.basename(discoveredPackageRoot);
+    let amended = check;
+    if (discoveredVersion !== runningVersion && classifyBump(runningVersion, discoveredVersion)) {
+        const skewNote = `This npm-global install is on ${runningVersion}; the ${hostLabel} plugin cache is on ${discoveredVersion}. `
+            + 'The plugin marketplace\'s own auto-updater only ever refreshes its plugin copy — it cannot and will not update this separate npm-global install.';
+        const skewFix = `Run \`memesh update\` to bring this npm-global install to ${discoveredVersion} (or newer) — it does not update itself automatically.`;
+        amended = {
+            ...amended,
+            status: amended.status === 'pass' ? 'warn' : amended.status,
+            summary: `${amended.summary} ${skewNote}`,
+            fix: amended.fix ? `${amended.fix} Also: ${skewFix}` : skewFix,
+            code: undefined,
+            params: undefined,
+        };
+    }
+    const cachedVersions = versionedPluginCacheRoots(cacheRoot);
+    if (cachedVersions.length > 2) {
+        amended = {
+            ...amended,
+            summary: `${amended.summary} ${cachedVersions.length} versioned copies of the ${hostLabel} plugin are cached under ${cacheRoot}; old ones are never removed automatically. `
+                + `Delete ones you no longer need once no ${hostLabel} process is using them, e.g. \`rm -rf "${cachedVersions[0]}"\`.`,
+        };
+    }
+    return amended;
+}
 function inspectClaudeChannelRegistration(existsSyncImpl, readFileSyncImpl) {
     const configPath = path.join(homeDir(), '.claude.json');
     let parsed = null;
@@ -977,7 +1069,13 @@ async function inspectUpdateStatus(packageVersion, getUpdateCheckImpl, installSu
             return createCheck('update-status', 'Update status', 'pass', `Running pre-release version (${packageVersion}), npm latest is ${update.latestVersion}`);
         }
     }
-    return createCheck('update-status', 'Update status', update.freshness === 'stale' ? 'warn' : 'pass', `Version ${packageVersion} is current${update.freshness === 'stale' ? ', but cached update data is stale.' : '.'}`, update.freshness === 'stale'
+    const checkedHoursAgo = update.lastSuccessfulCheckAt === null
+        ? null
+        : (Date.now() - Date.parse(update.lastSuccessfulCheckAt)) / 3600_000;
+    const checkedAgo = formatHoursAgo(checkedHoursAgo);
+    return createCheck('update-status', 'Update status', update.freshness === 'stale' ? 'warn' : 'pass', update.freshness === 'stale'
+        ? `As of the last check (${checkedAgo}), ${packageVersion} was the latest version — that check is more than 24h old, so a newer release may exist since. Doctor never makes a live registry call itself.`
+        : `As of the last check (${checkedAgo}), ${packageVersion} was the latest version.`, update.freshness === 'stale'
         ? 'Run `memesh status` while online to refresh cached update metadata.'
         : undefined, update.freshness === 'stale'
         ? { code: 'update-status.stale', params: { version: packageVersion } }
@@ -1175,6 +1273,33 @@ async function inspectEmbeddingProbe(capabilities, probeCapabilities, embedTextI
         clearTimeout(timer);
     }
 }
+const LLM_TELEMETRY_HEALTH_WINDOW_DAYS = 7;
+const LLM_TELEMETRY_HEALTH_MIN_CALLS = 3;
+function inspectLlmTelemetryHealth(db, windowDays = LLM_TELEMETRY_HEALTH_WINDOW_DAYS, minCalls = LLM_TELEMETRY_HEALTH_MIN_CALLS) {
+    let summaries;
+    try {
+        summaries = summariseTelemetry(windowDays, db);
+    }
+    catch {
+        return undefined;
+    }
+    if (summaries.length === 0)
+        return undefined;
+    const broken = summaries
+        .filter((s) => s.total_calls >= minCalls && s.successes === 0)
+        .sort((a, b) => b.total_calls - a.total_calls);
+    if (broken.length > 0) {
+        const detail = broken.map((s) => `${s.flow} (${s.total_calls} call${s.total_calls === 1 ? '' : 's'}, 0 succeeded)`).join(', ');
+        return createCheck('llm_telemetry_health', 'AI feature health', 'warn', `${broken.length} AI-backed feature${broken.length === 1 ? '' : 's'} failed every call in the last `
+            + `${windowDays} days: ${detail}. Those features are silently doing nothing.`, 'Run `memesh telemetry` for the full per-flow detail, then check the model/provider configured for '
+            + 'the failing flow. `memesh doctor --probe` confirms whether it answers a live call.', { code: 'llm-telemetry.silent-failure', params: { count: broken.length, detail, windowDays } });
+    }
+    const totalCalls = summaries.reduce((n, s) => n + s.total_calls, 0);
+    const totalSuccesses = summaries.reduce((n, s) => n + s.successes, 0);
+    const rate = totalCalls > 0 ? Math.round((totalSuccesses / totalCalls) * 100) : 100;
+    return createInfo('llm_telemetry_health', 'AI feature health', `${summaries.length} AI-backed flow${summaries.length === 1 ? '' : 's'} made ${totalCalls} call(s) in the `
+        + `last ${windowDays} days; ${rate}% succeeded.`);
+}
 async function inspectLlmProbe(capabilities, probeCapabilities, probeProviderImpl) {
     const llm = capabilities.llm;
     if (!llm) {
@@ -1204,7 +1329,7 @@ function summarizeOverallStatus(checks) {
     return 'PASS';
 }
 export async function runDoctor(options) {
-    const { packageRoot, packageVersion, probeHttp = false, probeCapabilities = false, embedTextImpl = embedText, probeProviderImpl = probeProvider, httpBaseUrl = 'http://127.0.0.1:3737', platform = process.platform, openDatabaseImpl = openDatabase, closeDatabaseImpl = closeDatabase, isDatabaseOpenImpl = isDatabaseOpen, detectCapabilitiesImpl = detectCapabilities, getConfigPathImpl = getConfigPath, getUpdateCheckImpl = getUpdateCheck, getCurrentInstallChannelImpl = getCurrentInstallChannel, installedPluginsPathImpl, marketplaceHeadShaImpl = defaultMarketplaceHeadSha, pluginCacheDiscoveryImpl, getInstallChannelSupportImpl = getInstallChannelSupport, existsSyncImpl = fs.existsSync, readFileSyncImpl = fs.readFileSync, statSyncImpl = fs.statSync, fetchImpl = fetch, agentMessageStoragePolicy, nativeBindingProbeImpl, resolveShellMemeshImpl = defaultResolveShellMemesh, probeMessageCapability = process.env.MEMESH_DOCTOR_PROBE_MESSAGE_CAPABILITY === '1', messageCapabilityProbeImpl = probeInstalledMessageCapability, probeMessageRouterStatus = process.env.MEMESH_DOCTOR_PROBE_MESSAGE_ROUTER === '1', messageRouterStatusProbeImpl = defaultMessageRouterStatusProbe, } = options;
+    const { packageRoot, packageVersion, probeHttp = false, probeCapabilities = false, embedTextImpl = embedText, probeProviderImpl = probeProvider, httpBaseUrl = 'http://127.0.0.1:3737', platform = process.platform, envImpl = process.env, openDatabaseImpl = openDatabase, closeDatabaseImpl = closeDatabase, isDatabaseOpenImpl = isDatabaseOpen, detectCapabilitiesImpl = detectCapabilities, getConfigPathImpl = getConfigPath, getUpdateCheckImpl = getUpdateCheck, getCurrentInstallChannelImpl = getCurrentInstallChannel, installedPluginsPathImpl, marketplaceHeadShaImpl = defaultMarketplaceHeadSha, pluginCacheDiscoveryImpl, getInstallChannelSupportImpl = getInstallChannelSupport, existsSyncImpl = fs.existsSync, readFileSyncImpl = fs.readFileSync, statSyncImpl = fs.statSync, fetchImpl = fetch, agentMessageStoragePolicy, nativeBindingProbeImpl, resolveShellMemeshImpl = defaultResolveShellMemesh, probeMessageCapability = process.env.MEMESH_DOCTOR_PROBE_MESSAGE_CAPABILITY === '1', messageCapabilityProbeImpl = probeInstalledMessageCapability, probeMessageRouterStatus = process.env.MEMESH_DOCTOR_PROBE_MESSAGE_ROUTER === '1', messageRouterStatusProbeImpl = defaultMessageRouterStatusProbe, } = options;
     const wasDbOpenBeforeUs = isDatabaseOpenImpl();
     const safeCloseDatabaseImpl = wasDbOpenBeforeUs
         ? () => undefined
@@ -1259,7 +1384,14 @@ export async function runDoctor(options) {
                     ? 'The vector index could not be read, so how much of your memory semantic recall can see is unknown'
                     : `${missingVectors} memor${missingVectors === 1 ? 'y has' : 'ies have'} no search vector, `
                         + 'so semantic recall cannot find them (keyword search still works)';
-            dbChecks.push(createCheck('vector_index', 'Vector Index', 'warn', owed, `Run 'memesh reindex' to fix. This will restore full search functionality.`, { code: 'vector-index.stale', params: { missing: missingVectors ?? -1 } }));
+            const noEmbedderConfigured = detectCapabilitiesImpl().embeddings === 'tfidf';
+            const vectorIndexFix = noEmbedderConfigured
+                ? `No embedder is configured, so reindex has nothing to embed with — run 'memesh config set embedder.provider ollama' (or 'openai') first, then 'memesh reindex'.`
+                : `Run 'memesh reindex' to fix. This will restore full search functionality.`;
+            dbChecks.push(createCheck('vector_index', 'Vector Index', 'warn', owed, vectorIndexFix, {
+                code: noEmbedderConfigured ? 'vector-index.stale-no-embedder' : 'vector-index.stale',
+                params: { missing: missingVectors ?? -1 },
+            }));
         }
         const generation = readVectorGeneration();
         if (generation.state !== 'none') {
@@ -1340,6 +1472,9 @@ export async function runDoctor(options) {
                 dbChecks.push(createInfo('citation_compliance', 'Memory citation rate', `${cited} of ${citationTotal} session(s) with injected memories cited at least one (${rate}%).`));
             }
         }
+        const llmTelemetryHealth = inspectLlmTelemetryHealth(db);
+        if (llmTelemetryHealth)
+            dbChecks.push(llmTelemetryHealth);
     }
     catch (err) {
         const message = err instanceof Error ? err.message : 'unknown database error';
@@ -1407,7 +1542,7 @@ export async function runDoctor(options) {
         }
     }
     checks.push(inspectConfigFile(existsSyncImpl, readFileSyncImpl, getConfigPathImpl, detectCapabilitiesImpl().llm));
-    checks.push(inspectMcpConfig(packageRoot, existsSyncImpl, readFileSyncImpl));
+    checks.push(inspectMcpConfig(packageRoot, install, existsSyncImpl, readFileSyncImpl, envImpl));
     checks.push(...inspectHooksConfig(packageRoot, platform, existsSyncImpl, readFileSyncImpl, statSyncImpl));
     const pluginHost = detectPluginHost(packageRoot);
     let codexPluginCacheDetected = pluginHost === 'codex';
@@ -1426,9 +1561,12 @@ export async function runDoctor(options) {
                 codexPluginCacheDetected = true;
             if (discovered.host === 'claude-code')
                 claudePluginCacheDetected = true;
-            const check = discovered.unverifiableReason
+            let check = discovered.unverifiableReason
                 ? pluginCacheUnverifiable(discovered.host, discovered.unverifiableReason)
                 : inspectPluginCacheCurrency('plugin-marketplace', discovered.host, discovered.packageRoot, discovered.installedPluginsPath, readFileSyncImpl, existsSyncImpl, marketplaceHeadShaImpl);
+            if (check && !discovered.unverifiableReason) {
+                check = annotateNpmGlobalPluginCacheVersion(check, discovered.packageRoot, discovered.host === 'codex' ? 'Codex' : 'Claude Code', packageVersion);
+            }
             if (check) {
                 const hostName = discovered.host;
                 const index = (discoveredCounts.get(discovered.host) ?? 0) + 1;
@@ -1450,7 +1588,7 @@ export async function runDoctor(options) {
     checks.push(inspectDashboardArtifact(packageRoot, existsSyncImpl));
     checks.push(inspectNodeRuntime(packageRoot, existsSyncImpl, readFileSyncImpl));
     checks.push(inspectNativeBinding(packageRoot, existsSyncImpl, nativeBindingProbeImpl));
-    checks.push(inspectShellCli(install, packageRoot, resolveShellMemeshImpl));
+    checks.push(inspectShellCli(install, packageRoot, packageVersion, resolveShellMemeshImpl, existsSyncImpl, readFileSyncImpl));
     checks.push(verifySkillsManifest(packageRoot, existsSyncImpl, readFileSyncImpl, installSupport));
     checks.push(inspectMessageCapability(packageRoot, probeMessageCapability, messageCapabilityProbeImpl));
     checks.push(await inspectMessageRouterStatus(probeMessageRouterStatus, messageRouterStatusProbeImpl));