claude-code-memory-explorer 0.13.0 → 0.14.1

package/README.md

@@ -35,7 +35,7 @@ PORT=8080                Custom port (default: 3544, falls back if busy)
 --open                   Open browser on start
 ```
 
-The config dir can also be set via the `CLAUDE_CONFIG_DIR` environment variable. Switch projects at runtime with the picker (Shift+P) or `?project=/path`.
+The config dir can also be set via the `CLAUDE_CONFIG_DIR` environment variable; the analyzer's headless `claude` run inherits it. Switch projects at runtime with the picker (Shift+P) or `?project=/path`.
 
 ## License
 

package/lib/open-editor.js

@@ -221,6 +221,7 @@ function execNoShell(cmd, args, opts = {}) {
     let argv = args;
     const spawnOpts = {
       cwd: opts.cwd || undefined,
+      env: opts.env,
       timeout: opts.timeout,
       windowsHide: true,
       stdio: ['ignore', 'pipe', 'pipe'],
@@ -258,7 +259,9 @@ function execNoShell(cmd, args, opts = {}) {
     child.on('error', reject);
     child.on('close', (code, signal) => {
       if (signal) return reject(new Error(`Command terminated by ${signal}: ${stderr.trim()}`));
-      if (code !== 0) return reject(new Error(stderr.trim() || `Command exited with code ${code}`));
+      // The claude CLI reports its own failures on stdout, so stderr alone is
+      // often empty and the caller would surface only the exit code.
+      if (code !== 0) return reject(new Error(stderr.trim() || stdout.trim() || `Command exited with code ${code}`));
       resolve({ stdout, stderr });
     });
   });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-code-memory-explorer",
-  "version": "0.13.0",
+  "version": "0.14.1",
   "description": "Memory explorer dashboard for Claude Code",
   "main": "server.js",
   "bin": {

package/public/app.js

@@ -4,7 +4,7 @@ let projectData = null;
 let stackData = [];
 let summaryData = null;
 let selectedFileId = null;
-let healthViewOpen = true; // health dashboard is the default view; selecting a file exits it
+let healthViewOpen = false; // opt-in view; the Analyze button toggles it, selecting a file exits it
 
 // #endregion STATE
 
@@ -209,6 +209,8 @@ async function loadProject() {
 // Shared by the project picker, the boot restore, and the hub project shim. Throws with the
 // server's error message so callers can surface it directly.
 async function putProject(dirPath) {
+  // Recents can only be written under the right key once the config dir is known.
+  await projectInfoPromise;
   const res = await fetch('/api/project', {
     method: 'PUT',
     headers: { 'Content-Type': 'application/json' },
@@ -248,10 +250,40 @@ async function submitProjectPicker() {
 }
 
 const RECENT_PROJECTS_MAX = 20;
+const LEGACY_RECENTS_KEY = 'recentProjects';
+
+// Under the hub every config dir's memory app is proxied on one public port, so localStorage is
+// shared between them. Recents are keyed by config dir to keep each one's project separate.
+let claudeConfigDir = '';
+// Retries like the initial load below: in a hub iframe the server may not answer yet.
+const projectInfoPromise = (async () => {
+  for (let attempt = 0; attempt < 5; attempt++) {
+    try {
+      const info = await fetchJSON('/api/project');
+      claudeConfigDir = info.configDir || '';
+      migrateLegacyRecents();
+      return info;
+    } catch {
+      await new Promise((r) => setTimeout(r, 500));
+    }
+  }
+  return {};
+})();
+
+function recentsKey() {
+  return claudeConfigDir ? `${LEGACY_RECENTS_KEY}::${claudeConfigDir}` : LEGACY_RECENTS_KEY;
+}
+
+function migrateLegacyRecents() {
+  const legacy = localStorage.getItem(LEGACY_RECENTS_KEY);
+  if (!legacy || !claudeConfigDir) return;
+  if (!localStorage.getItem(recentsKey())) localStorage.setItem(recentsKey(), legacy);
+  localStorage.removeItem(LEGACY_RECENTS_KEY);
+}
 
 function getRecentProjects() {
   try {
-    return JSON.parse(localStorage.getItem('recentProjects') || '[]');
+    return JSON.parse(localStorage.getItem(recentsKey()) || '[]');
   } catch {
     return [];
   }
@@ -260,7 +292,7 @@ function getRecentProjects() {
 function addRecentProject(p) {
   const recent = getRecentProjects().filter((r) => r !== p);
   recent.unshift(p);
-  localStorage.setItem('recentProjects', JSON.stringify(recent.slice(0, RECENT_PROJECTS_MAX)));
+  localStorage.setItem(recentsKey(), JSON.stringify(recent.slice(0, RECENT_PROJECTS_MAX)));
 }
 
 // The recents list is the switcher; typing a path is the secondary act behind "+ Add path".
@@ -330,7 +362,7 @@ function _removeRecentProject(idx, e) {
   e.stopPropagation();
   const target = pickerRows[idx];
   if (!target) return;
-  localStorage.setItem('recentProjects', JSON.stringify(getRecentProjects().filter((p) => p !== target)));
+  localStorage.setItem(recentsKey(), JSON.stringify(getRecentProjects().filter((p) => p !== target)));
   renderProjectPicker();
 }
 
@@ -356,6 +388,9 @@ function initProjectPicker() {
     e.preventDefault();
     e.stopPropagation();
   });
+  const modal = document.getElementById('projectPickerModal');
+  onEscapeBlur(filter, modal, () => closeModal('projectPickerModal'));
+  onEscapeBlur(document.getElementById('projectPathInput'), modal, () => setPickerMode(false));
 }
 
 // #endregion PROJECT
@@ -366,6 +401,22 @@ function closeModal(id) {
   document.getElementById(id).classList.remove('open');
 }
 
+// Vimium eats Escape inside a text input and only blurs it, so the page never sees the key.
+// A blur that no click in the modal caused, while the window keeps focus and the input is
+// still shown, is that Escape.
+function onEscapeBlur(input, modal, handler) {
+  let pointerDown = false;
+  modal.addEventListener('mousedown', () => {
+    pointerDown = true;
+  });
+  document.addEventListener('mouseup', () => {
+    pointerDown = false;
+  });
+  input.addEventListener('blur', () => {
+    if (!pointerDown && document.hasFocus() && input.checkVisibility()) handler();
+  });
+}
+
 // Each entry pairs a left and a right group onto the same grid rows, so their
 // headings sit level and the shorter group just leaves empty rows.
 // `combo` joins the keys with a plus (a chord) instead of listing them as
@@ -1361,7 +1412,7 @@ function findingPrompt(f) {
   } else if (paths.length > 1) {
     header = 'In the Claude Code instruction files listed below, apply this fix:';
   } else if (f.scope === 'user') {
-    const path = stackData.find((s) => s.id === 'user-claude-md')?.path || '~/.claude/CLAUDE.md';
+    const path = stackData.find((s) => s.id === 'user-claude-md')?.path || summaryData.userClaudeMd;
     header = `In the user-level Claude Code instructions at ${path}, apply this fix:`;
   } else {
     const dir = memoryDirPath();
@@ -2092,20 +2143,24 @@ document.addEventListener('keydown', (e) => {
     .catch(() => ({}));
   if (!cfg.enabled) return;
   window.__HUB__ = cfg;
-  const fwd = (e) => hubPost({ type: 'hub:keydown', key: e.key, ctrl: e.ctrlKey, alt: e.altKey, shift: e.shiftKey });
+  // e.code travels with e.key because macOS composes Option+<key> into a character (Option+P is
+  // 'π'), so the key alone cannot identify the binding. The hub owns the keymap and normalizes;
+  // these tests only decide whether a press is the hub's to handle.
   document.addEventListener('keydown', (e) => {
-    if (e.ctrlKey && e.altKey && (e.key === 'ArrowLeft' || e.key === 'ArrowRight')) {
+    const fwd = () => {
       e.preventDefault();
-      fwd(e);
+      hubPost({ type: 'hub:keydown', key: e.key, code: e.code, ctrl: e.ctrlKey, alt: e.altKey, shift: e.shiftKey });
+    };
+    if (e.ctrlKey && e.altKey && (e.key === 'ArrowLeft' || e.key === 'ArrowRight')) {
+      fwd();
     }
-    // Its own branch: the Alt+digit case below requires !ctrlKey.
-    if (e.ctrlKey && e.altKey && !e.shiftKey && !e.metaKey && e.key.toLowerCase() === 'p') {
-      e.preventDefault();
-      fwd(e);
+    // Own branch: the Alt+digit case below requires !ctrlKey. The hub owns the Ctrl+Alt+letter
+    // keymap and ignores unbound letters.
+    if (e.ctrlKey && e.altKey && !e.shiftKey && !e.metaKey && (/^[a-z]$/i.test(e.key) || /^Key[A-Z]$/.test(e.code))) {
+      fwd();
     }
-    if (e.altKey && !e.ctrlKey && !e.shiftKey && !e.metaKey && /^[1-9]$/.test(e.key)) {
-      e.preventDefault();
-      fwd(e);
+    if (e.altKey && !e.ctrlKey && !e.shiftKey && !e.metaKey && (/^[1-9]$/.test(e.key) || /^Digit[1-9]$/.test(e.code))) {
+      fwd();
     }
   });
 })();
@@ -2247,6 +2302,7 @@ document.addEventListener('DOMContentLoaded', async () => {
   let desiredProject = params.get('project');
   // A hub-pushed project outranks the localStorage recent: initHub() is fire-and-forget at script
   // eval, so hub:project can land before or during this block.
+  await projectInfoPromise;
   if (!desiredProject) desiredProject = hubProjectPath || getRecentProjects()[0] || null;
   if (desiredProject) {
     try {

package/public/index.html

@@ -14,6 +14,11 @@
 <script src="https://cdnjs.cloudflare.com/ajax/libs/highlight.js/11.11.1/highlight.min.js"></script>
 <link rel="manifest" href="/manifest.json">
 <link rel="icon" type="image/svg+xml" href="/icons/icon-svg.svg">
+<!-- Chrome uses the SVG only in the tab strip; app windows, shortcuts and the
+     history list need a raster icon or they fall back to /favicon.ico and then to
+     whatever icon localhost was last seen with, which is a sibling app's. -->
+<link rel="icon" type="image/png" sizes="192x192" href="/icons/icon-192.png">
+<link rel="icon" type="image/png" sizes="512x512" href="/icons/icon-512.png">
 <meta name="theme-color" content="#101114" media="(prefers-color-scheme: dark)">
 <meta name="theme-color" content="#e8e6e3" media="(prefers-color-scheme: light)">
 <meta name="apple-mobile-web-app-capable" content="yes">

package/server.js

@@ -25,7 +25,13 @@ const expandHome = (p) => (typeof p === 'string' ? p : '').replace(/^~/, os.home
 const PORT = getArg('port') || process.env.PORT || 3544;
 const AUTO_OPEN = process.argv.includes('--open');
 const claudeDirArg = getArg('dir') || process.env.CLAUDE_CONFIG_DIR || process.env.CLAUDE_DIR;
-const CLAUDE_DIR = claudeDirArg ? expandHome(claudeDirArg) : path.join(os.homedir(), '.claude');
+const DEFAULT_CLAUDE_DIR = path.join(os.homedir(), '.claude');
+const CLAUDE_DIR = claudeDirArg ? expandHome(claudeDirArg) : DEFAULT_CLAUDE_DIR;
+const USER_CLAUDE_MD = path.join(CLAUDE_DIR, 'CLAUDE.md');
+// Claude Code keeps .claude.json beside ~/.claude by default but inside CLAUDE_CONFIG_DIR
+// when that is set, so the default dir must stay unset for the CLI rather than be spelled out.
+const IS_DEFAULT_CLAUDE_DIR = path.resolve(CLAUDE_DIR) === path.resolve(DEFAULT_CLAUDE_DIR);
+const CLI_ENV = IS_DEFAULT_CLAUDE_DIR ? process.env : { ...process.env, CLAUDE_CONFIG_DIR: CLAUDE_DIR };
 const projectDirArg = getArg('project');
 
 // #endregion CLI_ARGS
@@ -233,8 +239,7 @@ function discoverMemorySources(projectPath) {
   }
 
   // 2. User CLAUDE.md
-  const userClaudeMd = path.join(CLAUDE_DIR, 'CLAUDE.md');
-  const userInfo = fileInfo(userClaudeMd);
+  const userInfo = fileInfo(USER_CLAUDE_MD);
   if (userInfo) {
     sources.push({
       id: 'user-claude-md',
@@ -617,7 +622,7 @@ app.get('/hub-config', (_req, res) => {
 });
 
 app.get('/api/project', (_req, res) => {
-  res.json({ path: currentProjectPath, name: path.basename(currentProjectPath) });
+  res.json({ path: currentProjectPath, name: path.basename(currentProjectPath), configDir: CLAUDE_DIR });
 });
 
 app.put('/api/project', (req, res) => {
@@ -843,7 +848,7 @@ const RUBRIC_CLAUDE_MD = [
 const RUBRIC_CLAUDE_MD_USER = [
   'User-level CLAUDE.md criteria (global instructions, loaded in every session of every project):',
   '- Universality: every rule must hold on any project. Flag rules referencing one specific repo, stack, tool chain, or project path — they belong in that project\'s CLAUDE.md (kind: demote).',
-  '- Machine verification only: check that tools the file names exist on PATH (command -v / which / where) and that files, skills, or agents it references exist under ~/.claude. NEVER verify against the working directory\'s repository — it is an arbitrary project and proves nothing about a global rule.',
+  `- Machine verification only: check that tools the file names exist on PATH (command -v / which / where) and that files, skills, or agents it references exist under ${CLAUDE_DIR}. NEVER verify against the working directory's repository — it is an arbitrary project and proves nothing about a global rule.`,
   '- Self-consistency: flag sections, tags, or mnemonics that contradict or duplicate each other within the file and its imports.',
   '- Executability: flag standards name-dropped without concrete behaviors ("follow style guide X") and rules the model cannot act on — require the 2-3 specific behaviors the user actually wants.',
   '- Flag emphasis inflation (all-caps, NEVER/ALWAYS without stakes) and vague or no-op rules the model cannot act on.',
@@ -970,7 +975,7 @@ const REVIEWER_COMMON = [
 // directory is an arbitrary project and must not influence a global-file audit.
 const REVIEWER_COMMON_USER = [
   'You review the user-level (global) CLAUDE.md that loads in every session of every project; the parent passes its name and full content.',
-  'Verify before judging, against the machine only: check tools the file names with `command -v` / `which` / `where` (the only Bash commands you may run), and check files, skills, or agents it references under ~/.claude with your read tools. NEVER read or judge against the repository in your working directory — it is one arbitrary project and proves nothing about a global rule.',
+  `Verify before judging, against the machine only: check tools the file names with \`command -v\` / \`which\` / \`where\` (the only Bash commands you may run), and check files, skills, or agents it references under ${CLAUDE_DIR} with your read tools. NEVER read or judge against the repository in your working directory — it is one arbitrary project and proves nothing about a global rule.`,
   'Also judge whether the content is still needed at all: if the tool it references is gone from the machine or the file it points to no longer exists, say so explicitly.',
   'A check you could not run is not evidence of absence. If a probe is denied by permissions or errors out, report that claim as UNCHECKED and say why — never turn a failed check into a stale or invalidate finding.',
 ];
@@ -1019,7 +1024,7 @@ const ANALYSIS_AGENTS = {
     [RUBRIC_CLAUDE_MD, RUBRIC_IMPORTS],
   ),
   'claude-md-user-reviewer': reviewerAgent(
-    'Reviews and verifies the user-level (global) CLAUDE.md against the user rubric, PATH, and ~/.claude — never against the working repository.',
+    `Reviews and verifies the user-level (global) CLAUDE.md against the user rubric, PATH, and ${CLAUDE_DIR} — never against the working repository.`,
     [RUBRIC_CLAUDE_MD_USER, RUBRIC_IMPORTS],
     REVIEWER_COMMON_USER,
   ),
@@ -1039,7 +1044,7 @@ function runClaudeAnalysis(prompt, model, cwd) {
     try {
       // cwd = the project root so the agent can verify claims against the actual
       // files with its read-only tools (Read/Glob/Grep need no permission grants).
-      child = spawn('claude', args, { cwd: cwd || os.tmpdir(), windowsHide: true });
+      child = spawn('claude', args, { cwd: cwd || os.tmpdir(), env: CLI_ENV, windowsHide: true });
     } catch (e) {
       return resolve({ ok: false, error: `Failed to spawn claude: ${e.message}` });
     }
@@ -1313,7 +1318,9 @@ app.get('/api/summary', (_req, res) => {
   for (const f of sources) scopeChars[f.scope] = (scopeChars[f.scope] || 0) + (f.chars || 0);
   const totalChars = sources.reduce((s, f) => s + (f.chars || 0), 0) + skillDesc.chars;
   // ids: which sources make up the footprint, so the client can highlight them
-  res.json({ totalFiles, totalLines, totalBytes, totalChars, scopeChars, skillDesc, alwaysLoaded, conditional, onDemand, ids: sources.map(s => s.id) });
+  // userClaudeMd: where the user-level file lives even when it does not exist yet, so the
+  // client can name it in fix prompts without assuming ~/.claude
+  res.json({ totalFiles, totalLines, totalBytes, totalChars, scopeChars, skillDesc, alwaysLoaded, conditional, onDemand, ids: sources.map(s => s.id), userClaudeMd: USER_CLAUDE_MD });
 });
 
 app.get('/api/stack', (_req, res) => {