@prmichaelsen/remember-mcp 3.15.6 → 3.16.0

package/dist/server.js

@@ -5789,12 +5789,12 @@ var searchMemoryTool = {
   }
 };
 async function handleSearchMemory(args, userId, authContext) {
-  const ghostMode = authContext?.ghostMode;
-  const searchUserId = ghostMode?.owner_user_id ?? userId;
-  const debug = createDebugLogger({ tool: "remember_search_memory", userId: searchUserId, operation: ghostMode ? "ghost search" : "search memory" });
+  const internalContext = authContext?.internalContext;
+  const searchUserId = internalContext?.owner_user_id ?? userId;
+  const debug = createDebugLogger({ tool: "remember_search_memory", userId: searchUserId, operation: internalContext ? "internal search" : "search memory" });
   try {
     debug.info("Tool invoked");
-    debug.trace("Arguments", { args, ghostMode: !!ghostMode });
+    debug.trace("Arguments", { args, internalContext: !!internalContext });
     if (!args.query || args.query.trim() === "") {
       throw new Error("Query cannot be empty");
     }
@@ -5803,18 +5803,21 @@ async function handleSearchMemory(args, userId, authContext) {
       userId: searchUserId,
       query: args.query,
       includeRelationships,
-      ghostMode: !!ghostMode
+      internalContext: !!internalContext
     });
     const collection = getMemoryCollection2(searchUserId);
     const alpha = args.alpha ?? 0.7;
     const limit = args.limit ?? 10;
     const offset = args.offset ?? 0;
     const deletedFilter = buildDeletedFilter(collection, args.deleted_filter || "exclude");
-    const trustFilter = ghostMode ? buildTrustFilter(collection, ghostMode.accessor_trust_level) : null;
+    const trustFilter = internalContext?.accessor_trust_level != null ? buildTrustFilter(collection, internalContext.accessor_trust_level) : null;
     const searchFilters = includeRelationships ? buildCombinedSearchFilters(collection, args.filters) : buildMemoryOnlyFilters(collection, args.filters);
     const hasExplicitTypeFilter = args.filters?.types && args.filters.types.length > 0;
-    const ghostExclusionFilter = !hasExplicitTypeFilter ? collection.filter.byProperty("content_type").notEqual("ghost") : null;
-    const combinedFilters = combineFiltersWithAnd([deletedFilter, trustFilter, ghostExclusionFilter, searchFilters].filter((f) => f !== null));
+    const internalExclusionFilter = !hasExplicitTypeFilter ? combineFiltersWithAnd([
+      collection.filter.byProperty("content_type").notEqual("ghost"),
+      collection.filter.byProperty("content_type").notEqual("agent")
+    ]) : null;
+    const combinedFilters = combineFiltersWithAnd([deletedFilter, trustFilter, internalExclusionFilter, searchFilters].filter((f) => f !== null));
     const searchOptions = {
       alpha,
       limit,
@@ -6160,7 +6163,7 @@ async function handleFindSimilar(args, userId, authContext) {
       deleted_filter: args.deleted_filter
     });
     const filteredMemories = coreResult.similar_memories.filter(
-      (m) => m.content_type !== "ghost"
+      (m) => m.content_type !== "ghost" && m.content_type !== "agent"
     );
     const result = {
       query: {
@@ -6303,27 +6306,30 @@ var queryMemoryTool = {
   }
 };
 async function handleQueryMemory(args, userId, authContext) {
-  const ghostMode = authContext?.ghostMode;
-  const searchUserId = ghostMode?.owner_user_id ?? userId;
-  const debug = createDebugLogger({ tool: "remember_query_memory", userId: searchUserId, operation: ghostMode ? "ghost query" : "query memory" });
+  const internalContext = authContext?.internalContext;
+  const searchUserId = internalContext?.owner_user_id ?? userId;
+  const debug = createDebugLogger({ tool: "remember_query_memory", userId: searchUserId, operation: internalContext ? "internal query" : "query memory" });
   try {
     debug.info("Tool invoked");
-    debug.trace("Arguments", { args, ghostMode: !!ghostMode });
+    debug.trace("Arguments", { args, internalContext: !!internalContext });
     if (!args.query || args.query.trim() === "") {
       throw new Error("Query cannot be empty");
     }
-    logger.info("Querying memories", { userId: searchUserId, query: args.query, ghostMode: !!ghostMode });
+    logger.info("Querying memories", { userId: searchUserId, query: args.query, internalContext: !!internalContext });
     const collection = getMemoryCollection2(searchUserId);
     const limit = args.limit ?? 5;
     const minRelevance = args.min_relevance ?? 0.6;
     const includeContext = args.include_context ?? true;
     const format = args.format ?? "detailed";
     const deletedFilter = buildDeletedFilter(collection, args.deleted_filter || "exclude");
-    const trustFilter = ghostMode ? buildTrustFilter(collection, ghostMode.accessor_trust_level) : null;
+    const trustFilter = internalContext?.accessor_trust_level != null ? buildTrustFilter(collection, internalContext.accessor_trust_level) : null;
     const searchFilters = buildCombinedSearchFilters(collection, args.filters);
     const hasExplicitTypeFilter = args.filters?.types && args.filters.types.length > 0;
-    const ghostExclusionFilter = !hasExplicitTypeFilter ? collection.filter.byProperty("content_type").notEqual("ghost") : null;
-    const combinedFilters = combineFiltersWithAnd([deletedFilter, trustFilter, ghostExclusionFilter, searchFilters].filter((f) => f !== null));
+    const internalExclusionFilter = !hasExplicitTypeFilter ? combineFiltersWithAnd([
+      collection.filter.byProperty("content_type").notEqual("ghost"),
+      collection.filter.byProperty("content_type").notEqual("agent")
+    ]) : null;
+    const combinedFilters = combineFiltersWithAnd([deletedFilter, trustFilter, internalExclusionFilter, searchFilters].filter((f) => f !== null));
     const searchOptions = {
       limit,
       distance: 1 - minRelevance,

package/dist/tools/{create-ghost-memory.d.ts → create-internal-memory.d.ts}

@@ -1,9 +1,10 @@
 /**
- * remember_create_ghost_memory tool
- * Creates a ghost memory with hardcoded content_type and ghost-specific tags
+ * remember_create_internal_memory tool
+ * Creates a ghost or agent memory based on the current session's InternalContext.
+ * Content type and tags are derived from platform HTTP headers, not tool args.
  */
 import type { AuthContext } from '../types/auth.js';
-export declare const createGhostMemoryTool: {
+export declare const createInternalMemoryTool: {
     name: string;
     description: string;
     inputSchema: {
@@ -40,25 +41,22 @@ export declare const createGhostMemoryTool: {
                 type: string;
                 minimum: number;
                 maximum: number;
-                description: string;
             };
             feel_social_weight: {
                 type: string;
                 minimum: number;
                 maximum: number;
-                description: string;
             };
             feel_narrative_importance: {
                 type: string;
                 minimum: number;
                 maximum: number;
-                description: string;
             };
         };
         required: string[];
     };
 };
-export interface CreateGhostMemoryArgs {
+export interface CreateInternalMemoryArgs {
     content: string;
     title?: string;
     tags?: string[];
@@ -66,5 +64,5 @@ export interface CreateGhostMemoryArgs {
     trust?: number;
     [key: string]: any;
 }
-export declare function handleCreateGhostMemory(args: CreateGhostMemoryArgs, userId: string, authContext?: AuthContext): Promise<string>;
-//# sourceMappingURL=create-ghost-memory.d.ts.map
+export declare function handleCreateInternalMemory(args: CreateInternalMemoryArgs, userId: string, authContext?: AuthContext): Promise<string>;
+//# sourceMappingURL=create-internal-memory.d.ts.map

package/dist/tools/internal-tools.spec.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=internal-tools.spec.d.ts.map

package/dist/tools/{query-ghost-memory.d.ts → query-internal-memory.d.ts}

@@ -1,9 +1,9 @@
 /**
- * remember_query_ghost_memory tool
- * Wraps query_memory with hardcoded ghost type filter
+ * remember_query_internal_memory tool
+ * Wraps query_memory with auto-scoped content type and ghost source filters.
  */
 import type { AuthContext } from '../types/auth.js';
-export declare const queryGhostMemoryTool: {
+export declare const queryInternalMemoryTool: {
     name: string;
     description: string;
     inputSchema: {
@@ -25,10 +25,10 @@ export declare const queryGhostMemoryTool: {
         required: string[];
     };
 };
-export interface QueryGhostMemoryArgs {
+export interface QueryInternalMemoryArgs {
     query: string;
     limit?: number;
     min_relevance?: number;
 }
-export declare function handleQueryGhostMemory(args: QueryGhostMemoryArgs, userId: string, authContext?: AuthContext): Promise<string>;
-//# sourceMappingURL=query-ghost-memory.d.ts.map
+export declare function handleQueryInternalMemory(args: QueryInternalMemoryArgs, userId: string, authContext?: AuthContext): Promise<string>;
+//# sourceMappingURL=query-internal-memory.d.ts.map

package/dist/tools/{search-ghost-memory-by.d.ts → search-internal-memory-by.d.ts}

@@ -1,10 +1,10 @@
 /**
- * remember_search_ghost_memory_by tool
- * Wraps search_by with hardcoded ghost type filter
+ * remember_search_internal_memory_by tool
+ * Wraps search_by with auto-scoped content type and ghost source filters.
  */
 import type { AuthContext } from '../types/auth.js';
 import type { SearchByMode } from './search-by.js';
-export declare const searchGhostMemoryByTool: {
+export declare const searchInternalMemoryByTool: {
     name: string;
     description: string;
     inputSchema: {
@@ -41,7 +41,7 @@ export declare const searchGhostMemoryByTool: {
         required: string[];
     };
 };
-export interface SearchGhostMemoryByArgs {
+export interface SearchInternalMemoryByArgs {
     mode: SearchByMode;
     query?: string;
     sort_order?: 'asc' | 'desc';
@@ -50,5 +50,5 @@ export interface SearchGhostMemoryByArgs {
     offset?: number;
     deleted_filter?: 'exclude' | 'include' | 'only';
 }
-export declare function handleSearchGhostMemoryBy(args: SearchGhostMemoryByArgs, userId: string, authContext?: AuthContext): Promise<string>;
-//# sourceMappingURL=search-ghost-memory-by.d.ts.map
+export declare function handleSearchInternalMemoryBy(args: SearchInternalMemoryByArgs, userId: string, authContext?: AuthContext): Promise<string>;
+//# sourceMappingURL=search-internal-memory-by.d.ts.map

package/dist/tools/{search-ghost-memory.d.ts → search-internal-memory.d.ts}

@@ -1,9 +1,9 @@
 /**
- * remember_search_ghost_memory tool
- * Wraps search_memory with hardcoded ghost type filter
+ * remember_search_internal_memory tool
+ * Wraps search_memory with auto-scoped content type and ghost source filters.
  */
 import type { AuthContext } from '../types/auth.js';
-export declare const searchGhostMemoryTool: {
+export declare const searchInternalMemoryTool: {
     name: string;
     description: string;
     inputSchema: {
@@ -41,7 +41,7 @@ export declare const searchGhostMemoryTool: {
         required: string[];
     };
 };
-export interface SearchGhostMemoryArgs {
+export interface SearchInternalMemoryArgs {
     query: string;
     alpha?: number;
     tags?: string[];
@@ -49,5 +49,5 @@ export interface SearchGhostMemoryArgs {
     offset?: number;
     deleted_filter?: 'exclude' | 'include' | 'only';
 }
-export declare function handleSearchGhostMemory(args: SearchGhostMemoryArgs, userId: string, authContext?: AuthContext): Promise<string>;
-//# sourceMappingURL=search-ghost-memory.d.ts.map
+export declare function handleSearchInternalMemory(args: SearchInternalMemoryArgs, userId: string, authContext?: AuthContext): Promise<string>;
+//# sourceMappingURL=search-internal-memory.d.ts.map

package/dist/tools/{update-ghost-memory.d.ts → update-internal-memory.d.ts}

@@ -1,9 +1,9 @@
 /**
- * remember_update_ghost_memory tool
- * Updates a ghost memory after validating content_type
+ * remember_update_internal_memory tool
+ * Updates a ghost or agent memory after validating content_type matches session context.
  */
 import type { AuthContext } from '../types/auth.js';
-export declare const updateGhostMemoryTool: {
+export declare const updateInternalMemoryTool: {
     name: string;
     description: string;
     inputSchema: {
@@ -39,7 +39,7 @@ export declare const updateGhostMemoryTool: {
         required: string[];
     };
 };
-export interface UpdateGhostMemoryArgs {
+export interface UpdateInternalMemoryArgs {
     memory_id: string;
     content?: string;
     title?: string;
@@ -47,5 +47,5 @@ export interface UpdateGhostMemoryArgs {
     weight?: number;
     trust?: number;
 }
-export declare function handleUpdateGhostMemory(args: UpdateGhostMemoryArgs, userId: string, authContext?: AuthContext): Promise<string>;
-//# sourceMappingURL=update-ghost-memory.d.ts.map
+export declare function handleUpdateInternalMemory(args: UpdateInternalMemoryArgs, userId: string, authContext?: AuthContext): Promise<string>;
+//# sourceMappingURL=update-internal-memory.d.ts.map

package/dist/types/auth.d.ts

@@ -24,20 +24,34 @@ export interface UserCredentials {
     user_id: string;
     group_memberships: GroupMembership[];
 }
-/** Ghost conversation context — resolved server-side, never from tool args */
-export interface GhostModeContext {
-    /** The ghost owner's user ID (whose memories are being searched) */
-    owner_user_id: string;
-    /** The accessor's user ID (who is chatting with the ghost) */
+/**
+ * Internal context for ghost/agent sessions — resolved server-side from
+ * platform HTTP headers, never from tool args.
+ *
+ * Replaces the former GhostModeContext. All ghost identity, trust, and
+ * agent context is unified here.
+ */
+export interface InternalContext {
+    /** Whether this is a ghost or agent session */
+    type: 'ghost' | 'agent';
+    /** Ghost sub-type (required when type is 'ghost') */
+    ghost_type?: 'user' | 'space' | 'group';
+    /** Space ID (space ghosts only) */
+    ghost_space?: string;
+    /** Group ID (group ghosts only) */
+    ghost_group?: string;
+    /** The ghost owner's user ID (user ghosts — whose memories are searched) */
+    owner_user_id?: string;
+    /** The accessor's user ID (who is conversing) */
     accessor_user_id: string;
     /** Resolved trust level (looked up from GhostConfig, not user-supplied) */
-    accessor_trust_level: number;
+    accessor_trust_level?: number;
 }
 export interface AuthContext {
     accessToken: string | null;
     credentials: UserCredentials | null;
-    /** Present when the server is running in ghost conversation mode */
-    ghostMode?: GhostModeContext;
+    /** Present when the server is running in ghost or agent mode */
+    internalContext?: InternalContext;
 }
 export type WriteMode = 'owner_only' | 'group_editors' | 'anyone';
 export interface CredentialsProvider {

package/dist/utils/internal-tags.d.ts

@@ -0,0 +1,14 @@
+import type { AuthContext } from '../types/auth.js';
+/**
+ * Build tags for internal (ghost/agent) memories based on the current
+ * session's InternalContext. These tags implement ghost source isolation
+ * so memories from different ghost conversations are distinguishable.
+ *
+ * Tag scheme:
+ *   User ghost (alice):        ['ghost', 'ghost_type:user', 'ghost_owner:user:alice']
+ *   Space ghost (music-lovers): ['ghost', 'ghost_type:space', 'ghost_owner:space:music-lovers']
+ *   Group ghost (band-mates):   ['ghost', 'ghost_type:group', 'ghost_owner:group:band-mates']
+ *   Agent:                      ['agent']
+ */
+export declare function buildInternalTags(authContext: AuthContext): string[];
+//# sourceMappingURL=internal-tags.d.ts.map

package/dist/utils/internal-tags.spec.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=internal-tags.spec.d.ts.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@prmichaelsen/remember-mcp",
-  "version": "3.15.6",
+  "version": "3.16.0",
   "description": "Multi-tenant memory system MCP server with vector search and relationships",
   "main": "dist/server.js",
   "type": "module",
@@ -51,7 +51,6 @@
     "@google-cloud/vision": "^5.3.4",
     "@modelcontextprotocol/sdk": "^1.0.4",
     "@prmichaelsen/firebase-admin-sdk-v8": "^2.2.0",
-    "@prmichaelsen/mcp-auth": "^7.0.4",
     "@prmichaelsen/remember-core": "^0.45.0",
     "dotenv": "^16.4.5",
     "uuid": "^13.0.0",

package/src/e2e-helpers.ts

@@ -24,12 +24,14 @@ export function e2eAuthContext(): AuthContext {
   return { accessToken: null, credentials: null };
 }
 
-/** AuthContext with ghost mode for cross-user tests */
+/** AuthContext with ghost internal context for cross-user tests */
 export function e2eGhostAuthContext(ownerUserId: string, accessorUserId: string, trustLevel = 0.5): AuthContext {
   return {
     accessToken: null,
     credentials: null,
-    ghostMode: {
+    internalContext: {
+      type: 'ghost',
+      ghost_type: 'user',
       owner_user_id: ownerUserId,
       accessor_user_id: accessorUserId,
       accessor_trust_level: trustLevel,

package/src/ghost-persona.e2e.ts

@@ -10,11 +10,11 @@ import {
   e2eEnsureCollection, e2eCleanup, parseResult, waitForIndex,
 } from './e2e-helpers.js';
 import { handleGhostConfig } from './tools/ghost-config.js';
-import { handleCreateGhostMemory } from './tools/create-ghost-memory.js';
-import { handleSearchGhostMemory } from './tools/search-ghost-memory.js';
-import { handleUpdateGhostMemory } from './tools/update-ghost-memory.js';
-import { handleQueryGhostMemory } from './tools/query-ghost-memory.js';
-import { handleSearchGhostMemoryBy } from './tools/search-ghost-memory-by.js';
+import { handleCreateInternalMemory } from './tools/create-internal-memory.js';
+import { handleSearchInternalMemory } from './tools/search-internal-memory.js';
+import { handleUpdateInternalMemory } from './tools/update-internal-memory.js';
+import { handleQueryInternalMemory } from './tools/query-internal-memory.js';
+import { handleSearchInternalMemoryBy } from './tools/search-internal-memory-by.js';
 import { handleGetCore } from './tools/get-core.js';
 
 const ownerId = e2eUserId('ghost_owner');
@@ -79,7 +79,7 @@ describe('E2E: Ghost/Persona', () => {
   // -------------------------------------------------------------------------
 
   it('creates a ghost memory', async () => {
-    const res = parseResult(await handleCreateGhostMemory(
+    const res = parseResult(await handleCreateInternalMemory(
       {
         content: `${accessorId} asked about my favorite hiking trails. They seem genuinely interested in outdoor activities.`,
         weight: 0.6,
@@ -94,7 +94,8 @@ describe('E2E: Ghost/Persona', () => {
     expect(res.memory_id).toBeTruthy();
     expect(res.content_type).toBe('ghost');
     expect(res.tags).toContain('ghost');
-    expect(res.tags).toContain(`ghost:${accessorId}`);
+    expect(res.tags).toContain('ghost_type:user');
+    expect(res.tags).toContain(`ghost_owner:user:${ownerId}`);
     ghostMemoryId = res.memory_id;
     console.log(`  created ghost memory ${ghostMemoryId}`);
   });
@@ -106,10 +107,10 @@ describe('E2E: Ghost/Persona', () => {
   it('searches ghost memories', async () => {
     await waitForIndex(5000);
 
-    // Search ghost memories specifically
-    const res = parseResult(await handleSearchGhostMemory(
+    // Search ghost memories via unified internal tool
+    const res = parseResult(await handleSearchInternalMemory(
       { query: 'hiking trails outdoor activities', limit: 5 },
-      ownerId, ownerAuth,
+      ownerId, ghostAuth,
     ));
 
     expect(res.memories).toBeDefined();
@@ -123,7 +124,7 @@ describe('E2E: Ghost/Persona', () => {
   // -------------------------------------------------------------------------
 
   it('updates the ghost memory', async () => {
-    const res = parseResult(await handleUpdateGhostMemory(
+    const res = parseResult(await handleUpdateInternalMemory(
       {
         memory_id: ghostMemoryId,
         content: `${accessorId} asked about my favorite hiking trails. They seem genuinely interested in outdoor activities. Follow-up: they mentioned they go rock climbing too.`,
@@ -141,10 +142,10 @@ describe('E2E: Ghost/Persona', () => {
   it('queries ghost memories with natural language', async () => {
     await waitForIndex();
 
-    // Use ownerAuth to avoid trust filter — query is already filtered to ghost content_type
-    const res = parseResult(await handleQueryGhostMemory(
+    // Use ghostAuth — query is auto-scoped to ghost content_type via internal context
+    const res = parseResult(await handleQueryInternalMemory(
       { query: 'What are the accessor\'s outdoor interests?', limit: 5, min_relevance: 0.3 },
-      ownerId, ownerAuth,
+      ownerId, ghostAuth,
     ));
 
     expect(res.memories).toBeDefined();
@@ -160,10 +161,10 @@ describe('E2E: Ghost/Persona', () => {
   // This creates an impossible filter: content_type='ghost' AND content_type!='ghost'.
   // Fix: core should skip ghost exclusion when types filter explicitly includes 'ghost'.
   // This test WILL FAIL until the core bug is fixed.
-  it('search_ghost_memory_by byTime returns ghost results', async () => {
-    const res = parseResult(await handleSearchGhostMemoryBy(
+  it('search_internal_memory_by byTime returns ghost results', async () => {
+    const res = parseResult(await handleSearchInternalMemoryBy(
       { mode: 'byTime', limit: 5 },
-      ownerId, ownerAuth,
+      ownerId, ghostAuth,
     ));
 
     expect(res.memories).toBeDefined();