@ijfw/memory-server 1.5.3 → 1.5.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ijfw/memory-server",
-  "version": "1.5.3",
+  "version": "1.5.4",
   "description": "Cross-platform persistent memory server for IJFW. 14 MCP tools (memory + admin/update + brain). Works with 15 platforms: 14 via MCP (Claude Code, Codex, Gemini CLI, Cursor, Windsurf, Copilot, Hermes, Wayland, OpenCode, QwenCode, Cline, KimiCode, OpenClaw, Antigravity) plus Aider via the rules-only tier.",
   "author": "Sean Donahoe",
   "license": "MIT",

package/src/brain/dream-pipeline.js

@@ -129,7 +129,7 @@ function buildExtractionPrompt(chunk) {
   ].join('\n');
 }
 
-export async function defaultExtractFacts({ file, text, chunks, env, guard, callTieredFn }) {
+export async function defaultExtractFacts({ file: _file, text, chunks, env, guard, callTieredFn }) {
   // No LLM configured -> graceful no-op (preserves prior behavior).
   if (!llmReachable(env)) return [];
   if (!text || typeof text !== 'string') return [];

package/src/brain/export.js

@@ -18,7 +18,12 @@ import { resolveBrainPaths } from './paths.js';
 import { validateSafeRepoPath } from './path-guard.js';
 
 const WIKI_TYPES = ['concepts', 'entities', 'decisions', 'milestones'];
-const WIKILINK_RE = /\[\[([^\]\n|]+)(?:\|[^\]\n]*)?\]\]/g;
+// Match the entire bracket-pair content as a single negated character class.
+// The slug (left of optional `|`) is extracted in JS after the regex match
+// instead of via a nested optional group -- safe-regex flagged the original
+// pattern (`[^\]\n|]+(?:\|[^\]\n]*)?`) as potentially unsafe even though it
+// can't backtrack catastrophically. This single-class form is provably linear.
+const WIKILINK_RE = /\[\[([^\]\n]+)\]\]/g;
 
 function findPage(wikiDir, slug) {
   for (const t of WIKI_TYPES) {
@@ -32,7 +37,11 @@ function parseWikilinks(md) {
   if (!md) return [];
   const out = new Set();
   for (const m of md.matchAll(WIKILINK_RE)) {
-    const target = m[1].trim();
+    // The capture group now includes the optional `|alias` suffix; the slug
+    // is the substring before the first `|` (matches the original semantics).
+    const inner = m[1];
+    const pipeIdx = inner.indexOf('|');
+    const target = (pipeIdx === -1 ? inner : inner.slice(0, pipeIdx)).trim();
     if (target) out.add(target);
   }
   return [...out];

package/src/handlers/brain-handler.js

@@ -297,7 +297,7 @@ function verbConflictResolve(db, repoRoot, args) {
   return { ok: true, resolved: true, winnerId: args.winnerId, supersededIds, validTo: chosenValidTo };
 }
 
-export async function handleIjfwBrain({ verb, args = {}, db, repoRoot, env, opts = {} } = {}) {
+export async function handleIjfwBrain({ verb, args = {}, db, repoRoot, env: _env, opts = {} } = {}) {
   if (!verb || typeof verb !== 'string') return { ok: false, error: 'missing-verb' };
   switch (verb) {
     case 'think':              return verbThink(db, repoRoot, args, opts);

package/src/orchestrator/discipline-selector.js

@@ -149,23 +149,6 @@ function extractTypeFromFrontmatter(raw) {
   return null;
 }
 
-/**
- * Return true if any direct child entry name of `dir` satisfies `predicate`.
- * Best-effort: returns false on any readdir error.
- *
- * @param {string} dir
- * @param {(name: string) => boolean} predicate
- * @returns {boolean}
- */
-function hasDirEntryMatching(dir, predicate) {
-  try {
-    const entries = readdirSync(dir);
-    return entries.some(predicate);
-  } catch {
-    return false;
-  }
-}
-
 /**
  * Return true if `name` is a direct child of `repoRoot` that is a directory
  * AND contains at least one `.md` file. Avoids false-positive on plain files

package/src/server.js

@@ -28,7 +28,7 @@ import { migrateFactsInternalOnce } from './brain/migrate-facts-internal-once.js
 // importing server.js.
 import { runLayoutMigrations } from './memory/layout-migrations/index.js';
 import { homedir } from 'os';
-import { fileURLToPath, pathToFileURL } from 'url';
+import { fileURLToPath } from 'url';
 import { createHash, randomBytes } from 'crypto';
 
 // Read version dynamically from package.json so bumps don't require a code change.
@@ -351,7 +351,7 @@ const __isServerEntryPoint = (() => {
 })();
 // Back-compat values for the line-2349 re-export. New code MUST use paths().memoryDir / paths().sessionsDir.
 const MEMORY_DIR = join(IJFW_DIR, 'memory');
-const SESSIONS_DIR = join(IJFW_DIR, 'sessions');
+// SESSIONS_DIR removed -- replaced by paths().sessionsDir via brainPaths() helper.
 const GLOBAL_DIR = join(homedir(), '.ijfw', 'memory');
 // Legacy single-file location (pre-Phase 2). Still read for backward compat
 // but new writes go to the faceted structure.