@remnic/core 9.3.656 → 9.3.658

package/dist/semantic-consolidation.js

@@ -7,14 +7,14 @@ import {
   materializeAfterSemanticConsolidation,
   parseConsolidationResponse,
   parseOperatorAwareConsolidationResponse
-} from "./chunk-SWDHVH2P.js";
-import "./chunk-WWMHAMAY.js";
+} from "./chunk-BKRIAXTU.js";
+import "./chunk-BNUAOLDK.js";
 import "./chunk-LN4YGHTM.js";
 import {
   resolveExtensionsRoot
 } from "./chunk-JLNBQWZ2.js";
 import "./chunk-3UXOZBHV.js";
-import "./chunk-NXCK7DO7.js";
+import "./chunk-4PLOQDBB.js";
 import "./chunk-M7XQSUBB.js";
 import "./chunk-5UZXUTVO.js";
 import "./chunk-J6A3CX5N.js";

package/dist/semantic-rule-promotion.js

@@ -1,8 +1,8 @@
 import {
   promoteSemanticRuleFromMemory,
   setSemanticRulePromotionTestHooks
-} from "./chunk-GCYFUTUC.js";
-import "./chunk-NXCK7DO7.js";
+} from "./chunk-FIS5RT6K.js";
+import "./chunk-4PLOQDBB.js";
 import "./chunk-M7XQSUBB.js";
 import "./chunk-5UZXUTVO.js";
 import "./chunk-J6A3CX5N.js";

package/dist/semantic-rule-verifier.js

@@ -1,8 +1,8 @@
 import {
   compareVerifiedSemanticRuleResults,
   searchVerifiedSemanticRules
-} from "./chunk-TFFZUFEP.js";
-import "./chunk-NXCK7DO7.js";
+} from "./chunk-VJYFXDCZ.js";
+import "./chunk-4PLOQDBB.js";
 import "./chunk-M7XQSUBB.js";
 import "./chunk-5UZXUTVO.js";
 import "./chunk-J6A3CX5N.js";

package/dist/storage.d.ts

@@ -27,9 +27,13 @@ interface MemoryLifecycleEventWriteOptions {
  * Strips non-alphanumeric chars, collapses hyphens, removes type prefix duplication.
  * e.g. "My Project" → "my-project"
  *
- * Checks user-defined aliases (from config/aliases.json) first, then built-in aliases.
+ * Checks caller-provided user aliases first, then built-in aliases. Alias
+ * tables are instance state on StorageManager (issue #1534) — use
+ * `storageManager.normalizeEntityName(raw, type)` when normalizing within a
+ * store, or pass `storageManager.entityAliases` explicitly. Without an
+ * aliases argument only the built-in structural aliases apply.
  */
-declare function normalizeEntityName(raw: string, type: string): string;
+declare function normalizeEntityName(raw: string, type: string, aliases?: Readonly<Record<string, string>>): string;
 /**
  * Content-hash dedup index for facts.
  * Normalizes content (lowercase, strip punctuation, collapse whitespace),
@@ -347,11 +351,29 @@ declare class StorageManager {
      */
     private get bufferSurpriseLedgerPath();
     /**
-     * Load user-defined entity aliases from config/aliases.json in the memory store.
-     * File format: { "variant": "canonical", "variant2": "canonical", ... }
-     * Call this once at startup (e.g. from orchestrator.initialize()).
+     * Entity alias table loaded from THIS store's config/aliases.json.
+     * Instance-scoped on purpose (issue #1534): multiple StorageManager
+     * instances in one process (namespaces, hosted profiles, tenants) must
+     * never share alias state — the previous module-level table let whichever
+     * store loaded last rewrite every other store's canonical entity ids.
+     */
+    private userAliases;
+    /** Normalize an entity name using this store's alias table. */
+    normalizeEntityName(raw: string, type: string): string;
+    /**
+     * Read-only view of this store's user alias table, for call sites that
+     * normalize outside the manager (pass it to the free `normalizeEntityName`).
+     */
+    get entityAliases(): Readonly<Record<string, string>>;
+    /**
+     * Reload user-defined entity aliases from config/aliases.json in the memory
+     * store. File format: { "variant": "canonical", ... }. The constructor
+     * already loads aliases, so this is only needed to pick up file changes
+     * (e.g. orchestrator.initialize() re-running on a live instance).
+     * Non-object payloads and non-string or empty alias values are ignored.
      */
     loadAliases(): Promise<void>;
+    private loadAliasesSync;
     ensureDirectories(): Promise<void>;
     writeMemory(category: MemoryCategory, content: string, options?: {
         actor?: string;

package/dist/storage.js

@@ -9,7 +9,7 @@ import {
   parseEntityFile,
   serializeEntityFile,
   stripAttributesSuffix
-} from "./chunk-NXCK7DO7.js";
+} from "./chunk-4PLOQDBB.js";
 import "./chunk-M7XQSUBB.js";
 import "./chunk-5UZXUTVO.js";
 import "./chunk-J6A3CX5N.js";

package/dist/transfer/types.d.ts

@@ -313,13 +313,13 @@ declare const CapsuleBlockSchema: z.ZodObject<{
         peerProfiles: boolean;
     }>;
 }, "strip", z.ZodTypeAny, {
+    schemaVersion: string;
     includes: {
         procedural: boolean;
         taxonomy: boolean;
         identityAnchors: boolean;
         peerProfiles: boolean;
     };
-    schemaVersion: string;
     id: string;
     description: string;
     version: string;
@@ -334,13 +334,13 @@ declare const CapsuleBlockSchema: z.ZodObject<{
         directAnswerEnabled: boolean;
     };
 }, {
+    schemaVersion: string;
     includes: {
         procedural: boolean;
         taxonomy: boolean;
         identityAnchors: boolean;
         peerProfiles: boolean;
     };
-    schemaVersion: string;
     id: string;
     description: string;
     version: string;
@@ -464,13 +464,13 @@ declare const ExportManifestV2Schema: z.ZodObject<{
             peerProfiles: boolean;
         }>;
     }, "strip", z.ZodTypeAny, {
+        schemaVersion: string;
         includes: {
             procedural: boolean;
             taxonomy: boolean;
             identityAnchors: boolean;
             peerProfiles: boolean;
         };
-        schemaVersion: string;
         id: string;
         description: string;
         version: string;
@@ -485,13 +485,13 @@ declare const ExportManifestV2Schema: z.ZodObject<{
             directAnswerEnabled: boolean;
         };
     }, {
+        schemaVersion: string;
         includes: {
             procedural: boolean;
             taxonomy: boolean;
             identityAnchors: boolean;
             peerProfiles: boolean;
         };
-        schemaVersion: string;
         id: string;
         description: string;
         version: string;
@@ -518,13 +518,13 @@ declare const ExportManifestV2Schema: z.ZodObject<{
     pluginVersion: string;
     includesTranscripts: boolean;
     capsule: {
+        schemaVersion: string;
         includes: {
             procedural: boolean;
             taxonomy: boolean;
             identityAnchors: boolean;
             peerProfiles: boolean;
         };
-        schemaVersion: string;
         id: string;
         description: string;
         version: string;
@@ -551,13 +551,13 @@ declare const ExportManifestV2Schema: z.ZodObject<{
     pluginVersion: string;
     includesTranscripts: boolean;
     capsule: {
+        schemaVersion: string;
         includes: {
             procedural: boolean;
             taxonomy: boolean;
             identityAnchors: boolean;
             peerProfiles: boolean;
         };
-        schemaVersion: string;
         id: string;
         description: string;
         version: string;
@@ -683,13 +683,13 @@ declare const ExportBundleV2Schema: z.ZodObject<{
                 peerProfiles: boolean;
             }>;
         }, "strip", z.ZodTypeAny, {
+            schemaVersion: string;
             includes: {
                 procedural: boolean;
                 taxonomy: boolean;
                 identityAnchors: boolean;
                 peerProfiles: boolean;
             };
-            schemaVersion: string;
             id: string;
             description: string;
             version: string;
@@ -704,13 +704,13 @@ declare const ExportBundleV2Schema: z.ZodObject<{
                 directAnswerEnabled: boolean;
             };
         }, {
+            schemaVersion: string;
             includes: {
                 procedural: boolean;
                 taxonomy: boolean;
                 identityAnchors: boolean;
                 peerProfiles: boolean;
             };
-            schemaVersion: string;
             id: string;
             description: string;
             version: string;
@@ -737,13 +737,13 @@ declare const ExportBundleV2Schema: z.ZodObject<{
         pluginVersion: string;
         includesTranscripts: boolean;
         capsule: {
+            schemaVersion: string;
             includes: {
                 procedural: boolean;
                 taxonomy: boolean;
                 identityAnchors: boolean;
                 peerProfiles: boolean;
             };
-            schemaVersion: string;
             id: string;
             description: string;
             version: string;
@@ -770,13 +770,13 @@ declare const ExportBundleV2Schema: z.ZodObject<{
         pluginVersion: string;
         includesTranscripts: boolean;
         capsule: {
+            schemaVersion: string;
             includes: {
                 procedural: boolean;
                 taxonomy: boolean;
                 identityAnchors: boolean;
                 peerProfiles: boolean;
             };
-            schemaVersion: string;
             id: string;
             description: string;
             version: string;
@@ -815,13 +815,13 @@ declare const ExportBundleV2Schema: z.ZodObject<{
         pluginVersion: string;
         includesTranscripts: boolean;
         capsule: {
+            schemaVersion: string;
             includes: {
                 procedural: boolean;
                 taxonomy: boolean;
                 identityAnchors: boolean;
                 peerProfiles: boolean;
             };
-            schemaVersion: string;
             id: string;
             description: string;
             version: string;
@@ -854,13 +854,13 @@ declare const ExportBundleV2Schema: z.ZodObject<{
         pluginVersion: string;
         includesTranscripts: boolean;
         capsule: {
+            schemaVersion: string;
             includes: {
                 procedural: boolean;
                 taxonomy: boolean;
                 identityAnchors: boolean;
                 peerProfiles: boolean;
             };
-            schemaVersion: string;
             id: string;
             description: string;
             version: string;

package/dist/verified-recall.js

@@ -1,9 +1,9 @@
 import {
   compareVerifiedEpisodeResults,
   searchVerifiedEpisodes
-} from "./chunk-JSVFEHLL.js";
+} from "./chunk-46RXRASB.js";
 import "./chunk-HQ6NIBL6.js";
-import "./chunk-NXCK7DO7.js";
+import "./chunk-4PLOQDBB.js";
 import "./chunk-M7XQSUBB.js";
 import "./chunk-5UZXUTVO.js";
 import "./chunk-J6A3CX5N.js";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@remnic/core",
-  "version": "9.3.656",
+  "version": "9.3.658",
   "description": "Framework-agnostic Remnic memory engine — orchestrator, storage, extraction, search, trust zones",
   "type": "module",
   "main": "dist/index.js",

package/src/briefing.ts

@@ -183,7 +183,11 @@ export function parseBriefingFocus(token: string | undefined): BriefingFocus | n
  * an entity only via `frontmatter.entityRef` (no body / tag mention) would
  * silently drop out of an untyped focus filter (codex P2 review on #695).
  */
-export function focusMatchesMemory(memory: MemoryFile, focus: BriefingFocus): boolean {
+export function focusMatchesMemory(
+  memory: MemoryFile,
+  focus: BriefingFocus,
+  entityAliases?: Readonly<Record<string, string>>,
+): boolean {
   const needle = focus.value.toLowerCase();
   const entityRef = (memory.frontmatter.entityRef ?? "").toLowerCase();
 
@@ -216,7 +220,7 @@ export function focusMatchesMemory(memory: MemoryFile, focus: BriefingFocus): bo
   // `Project-Alpha` of type `project` canonicalizes to `project-alpha`,
   // and a focus on `project:Project-Alpha` does the same.
   if (!entityRef) return false;
-  const focusCanonical = normalizeEntityName(focus.value, focus.type);
+  const focusCanonical = normalizeEntityName(focus.value, focus.type, entityAliases);
   // Strip a leading `<type><delimiter>` prefix from a non-canonical
   // entityRef before normalizing — `normalizeEntityName` only strips
   // `<type>-` so other valid verbatim formats (`person:Alice-Test`,
@@ -232,7 +236,7 @@ export function focusMatchesMemory(memory: MemoryFile, focus: BriefingFocus): bo
   if (typeDelimMatch) {
     refForNormalize = refForNormalize.slice(typeDelimMatch[0].length);
   }
-  const refCanonical = normalizeEntityName(refForNormalize, focus.type);
+  const refCanonical = normalizeEntityName(refForNormalize, focus.type, entityAliases);
   return refCanonical === focusCanonical;
 }
 
@@ -834,7 +838,7 @@ export async function buildBriefing(options: BuildBriefingOptions): Promise<Brie
 
   const memoriesInWindow = filterMemoriesByWindow(allMemories, window);
   const focusedMemories = focus
-    ? memoriesInWindow.filter((m) => focusMatchesMemory(m, focus))
+    ? memoriesInWindow.filter((m) => focusMatchesMemory(m, focus, options.storage.entityAliases))
     : memoriesInWindow;
 
   const activeThreads = buildActiveThreads(focusedMemories);

package/src/entity-retrieval.ts

@@ -457,12 +457,17 @@ async function buildEntityMentionIndex(
     Promise.all(storages.map((scopedStorage) => scopedStorage.readAllMemories())),
     readNativeChunks(config, recallNamespaces),
   ]);
-  const entityFiles = entityFileSets.flat();
+  // Pair each entity with the alias table of the store it came from (#1534):
+  // canonical ids must reflect the owning store's aliases, never another
+  // namespace's.
+  const entityRecords = entityFileSets.flatMap((set, index) =>
+    set.map((entity) => ({ entity, aliases: storages[index]!.entityAliases })),
+  );
   const memories = memorySets.flat();
 
   const entities = new Map<string, EntityMentionIndexEntry>();
-  for (const entity of entityFiles) {
-    const canonicalId = normalizeEntityName(entity.name, entity.type);
+  for (const { entity, aliases } of entityRecords) {
+    const canonicalId = normalizeEntityName(entity.name, entity.type, aliases);
     const rawStructuredSections = entity.structuredSections ?? [];
     const rawBeliefLedgerFactKeys = beliefLedgerFactKeys(rawStructuredSections);
     const sanitizedFacts = entity.facts

package/src/orchestrator.ts

@@ -65,7 +65,6 @@ import {
   StorageManager,
   ContentHashIndex,
   fingerprintEntityStructuredFacts,
-  normalizeEntityName,
   normalizeAttributePairs,
   parseEntityFile,
 } from "./storage.js";
@@ -15791,7 +15790,7 @@ export class Orchestrator {
         const type = (entity as any)?.type;
         if (typeof name !== "string" || typeof type !== "string") continue;
         try {
-          const normalized = normalizeEntityName(name, type);
+          const normalized = storage.normalizeEntityName(name, type);
           await storage.addEntityActivity(
             normalized,
             { date: today, note: "Mentioned in conversation" },

package/src/storage.ts

@@ -1,5 +1,5 @@
 import { access, readdir, readFile, stat, writeFile, mkdir, unlink, rename, appendFile, open } from "node:fs/promises";
-import { appendFileSync, createReadStream, mkdirSync, statSync } from "node:fs";
+import { appendFileSync, createReadStream, mkdirSync, readFileSync, statSync } from "node:fs";
 import { createHash } from "node:crypto";
 import path from "node:path";
 import { log } from "./logger.js";
@@ -935,13 +935,6 @@ function inferCurrentStateStatus(
   return inferMemoryStatus(frontmatter, pathRel, fallbackStatus);
 }
 
-/**
- * Entity alias table loaded from the user's local config.
- * Populated by StorageManager.loadAliases() at startup.
- * Falls back to built-in structural aliases (e.g. "open-claw" → "openclaw").
- */
-let userAliases: Record<string, string> = {};
-
 /** Built-in aliases for common structural normalizations (no personal data) */
 const BUILTIN_ALIASES: Record<string, string> = {
   openclaw: "openclaw",
@@ -953,9 +946,17 @@ const BUILTIN_ALIASES: Record<string, string> = {
  * Strips non-alphanumeric chars, collapses hyphens, removes type prefix duplication.
  * e.g. "My Project" → "my-project"
  *
- * Checks user-defined aliases (from config/aliases.json) first, then built-in aliases.
+ * Checks caller-provided user aliases first, then built-in aliases. Alias
+ * tables are instance state on StorageManager (issue #1534) — use
+ * `storageManager.normalizeEntityName(raw, type)` when normalizing within a
+ * store, or pass `storageManager.entityAliases` explicitly. Without an
+ * aliases argument only the built-in structural aliases apply.
  */
-export function normalizeEntityName(raw: string, type: string): string {
+export function normalizeEntityName(
+  raw: string,
+  type: string,
+  aliases?: Readonly<Record<string, string>>,
+): string {
   // Strip type prefix if present (e.g. name="person-jane-doe", type="person")
   const rawStr = typeof raw === "string" ? raw : "";
   const typeStr = typeof type === "string" && type.trim().length > 0 ? type : "entity";
@@ -972,10 +973,14 @@ export function normalizeEntityName(raw: string, type: string): string {
     .replace(/-+/g, "-")
     .replace(/^-|-$/g, "");
 
-  // Check user aliases first, then built-in
-  if (userAliases[normalized]) {
-    normalized = userAliases[normalized];
-  } else if (BUILTIN_ALIASES[normalized]) {
+  // Check caller-provided user aliases first, then built-in. Own-property and
+  // string guards keep inherited object keys (e.g. an entity literally named
+  // "constructor") and malformed alias values from corrupting canonical ids.
+  const userAlias =
+    aliases !== undefined && Object.hasOwn(aliases, normalized) ? aliases[normalized] : undefined;
+  if (typeof userAlias === "string" && userAlias.length > 0) {
+    normalized = userAlias;
+  } else if (Object.hasOwn(BUILTIN_ALIASES, normalized)) {
     normalized = BUILTIN_ALIASES[normalized];
   }
 
@@ -2443,7 +2448,14 @@ export class StorageManager {
   constructor(
     private readonly baseDir: string,
     private readonly entitySchemas?: PluginConfig["entitySchemas"],
-  ) {}
+  ) {
+    // Load this store's alias table at construction (#1534): StorageManager
+    // is created in a dozen places (namespace router, operator toolkit,
+    // compounding engine, cold storage, ...) and most never call
+    // loadAliases() explicitly — every creation path must still get the
+    // store's own aliases, never an empty or foreign table.
+    this.loadAliasesSync();
+  }
 
   /** The root directory of this storage instance. */
   get dir(): string {
@@ -3197,18 +3209,59 @@ export class StorageManager {
   }
 
   /**
-   * Load user-defined entity aliases from config/aliases.json in the memory store.
-   * File format: { "variant": "canonical", "variant2": "canonical", ... }
-   * Call this once at startup (e.g. from orchestrator.initialize()).
+   * Entity alias table loaded from THIS store's config/aliases.json.
+   * Instance-scoped on purpose (issue #1534): multiple StorageManager
+   * instances in one process (namespaces, hosted profiles, tenants) must
+   * never share alias state — the previous module-level table let whichever
+   * store loaded last rewrite every other store's canonical entity ids.
+   */
+  private userAliases: Record<string, string> = {};
+
+  /** Normalize an entity name using this store's alias table. */
+  normalizeEntityName(raw: string, type: string): string {
+    return normalizeEntityName(raw, type, this.userAliases);
+  }
+
+  /**
+   * Read-only view of this store's user alias table, for call sites that
+   * normalize outside the manager (pass it to the free `normalizeEntityName`).
+   */
+  get entityAliases(): Readonly<Record<string, string>> {
+    return this.userAliases;
+  }
+
+  /**
+   * Reload user-defined entity aliases from config/aliases.json in the memory
+   * store. File format: { "variant": "canonical", ... }. The constructor
+   * already loads aliases, so this is only needed to pick up file changes
+   * (e.g. orchestrator.initialize() re-running on a live instance).
+   * Non-object payloads and non-string or empty alias values are ignored.
    */
   async loadAliases(): Promise<void> {
+    this.loadAliasesSync();
+  }
+
+  private loadAliasesSync(): void {
     const aliasPath = path.join(this.baseDir, "config", "aliases.json");
+    // Re-derive from the file on every call: a reload after the file was
+    // fixed, emptied, or removed must never leave a previous table active.
+    this.userAliases = {};
     try {
-      const raw = await readFile(aliasPath, "utf-8");
+      const raw = readFileSync(aliasPath, "utf-8");
       const parsed = JSON.parse(raw);
-      if (typeof parsed === "object" && parsed !== null) {
-        userAliases = parsed as Record<string, string>;
-        log.debug(`loaded ${Object.keys(userAliases).length} entity aliases from ${aliasPath}`);
+      if (typeof parsed === "object" && parsed !== null && !Array.isArray(parsed)) {
+        const cleaned: Record<string, string> = {};
+        for (const [key, value] of Object.entries(parsed)) {
+          if (typeof value === "string" && value.trim().length > 0) {
+            cleaned[key] = value;
+          }
+        }
+        this.userAliases = cleaned;
+        log.debug(`loaded ${Object.keys(cleaned).length} entity aliases from ${aliasPath}`);
+      } else {
+        log.warn(
+          `ignoring ${aliasPath}: payload must be a JSON object mapping variant → canonical strings`,
+        );
       }
     } catch {
       // No aliases file — that's fine, use built-in only
@@ -3679,7 +3732,7 @@ export class StorageManager {
           .filter((fact) => fact.length > 0),
       )]
       : [];
-    let normalized = normalizeEntityName(name, type);
+    let normalized = this.normalizeEntityName(name, type);
 
     // Check for fuzzy match against existing entities before creating a new file
     const match = await this.findMatchingEntity(name, type);
@@ -4648,7 +4701,7 @@ export class StorageManager {
 
     const typePrefix = `${type.toLowerCase()}-`;
     // Extract the name part from the proposed normalized name
-    const proposedFull = normalizeEntityName(proposedName, type);
+    const proposedFull = this.normalizeEntityName(proposedName, type);
     const proposedNamePart = proposedFull.startsWith(typePrefix)
       ? proposedFull.slice(typePrefix.length)
       : proposedFull;
@@ -6104,7 +6157,7 @@ export class StorageManager {
     entity.updated = entityUpdatedAt;
     await this.writeStorageSecureFile(filePath, serializeEntityFile(entity, this.entitySchemas));
     await this.removeEntitySynthesisQueueEntries([
-      ...new Set([name, normalizeEntityName(entity.name, entity.type)]),
+      ...new Set([name, this.normalizeEntityName(entity.name, entity.type)]),
     ]);
     this.invalidateKnowledgeIndexCache();
     this.bumpMemoryStatusVersion(); // invalidate entity cache
@@ -6425,7 +6478,7 @@ export class StorageManager {
         if (dashIdx === -1) continue;
         const type = baseName.slice(0, dashIdx);
         const restOfName = baseName.slice(dashIdx + 1);
-        const canonical = normalizeEntityName(restOfName, type);
+        const canonical = this.normalizeEntityName(restOfName, type);
 
         if (!groups.has(canonical)) groups.set(canonical, []);
         groups.get(canonical)!.push(file);