@prmichaelsen/remember-mcp 3.12.0 → 3.14.0

package/CHANGELOG.md

@@ -5,6 +5,26 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.13.0] - 2026-02-28
+
+### Changed
+
+**Ghost Mode — Server-Side Resolution (M16 hardening)**
+
+- Ghost context now resolved server-side via `AuthContext.ghostMode` instead of LLM-accessible `ghost_context` tool parameter
+- `ServerOptions.ghostMode` added to server factory — agentbase.me passes ghost owner/accessor via mcp-auth extras, trust level resolved from Firestore at server creation
+- `GhostModeContext` type added to `src/types/auth.ts` with `owner_user_id`, `accessor_user_id`, `accessor_trust_level`
+- `AuthContext` extended with optional `ghostMode` field
+
+### Removed
+
+- `ghost_context` parameter from `remember_search_memory` and `remember_query_memory` tool schemas — trust levels are no longer LLM-controllable
+- `GhostContext` interface from `search-memory.ts` — replaced by `GhostModeContext` in `types/auth.ts`
+
+### Security
+
+- Trust level escalation prevention: accessor trust is resolved from GhostConfig (Firestore) at server creation time, not from tool arguments
+
 ## [3.12.0] - 2026-02-27
 
 ### Added

package/agent/milestones/milestone-17-remember-core-migration.md

@@ -0,0 +1,140 @@
+# Milestone 17: remember-core Migration
+
+**Goal**: Migrate remember-mcp tool handlers from inline business logic to remember-core service calls
+**Duration**: 2-3 weeks
+**Dependencies**: M16 (Ghost System complete), remember-core v0.12.0 published
+**Status**: Not Started
+
+---
+
+## Overview
+
+remember-mcp has 21 tool handlers with 50-200 lines of inline business logic each (Weaviate queries, validation, CRUD operations). remember-core (`@prmichaelsen/remember-core` v0.12.0) extracts that logic into 5 DI services (MemoryService, RelationshipService, SpaceService, PreferencesDatabaseService, ConfirmationTokenService) with 120 passing tests.
+
+This milestone migrates 18 of 21 tools to use remember-core as their business logic layer, reducing each tool handler to a thin 10-20 line adapter. 3 tools (search_memory, query_memory, ghost_config) are deferred — they depend on ghost/trust features not yet in remember-core.
+
+Migration guide: `/home/prmichaelsen/.acp/projects/remember-core/docs/migration-guide.md`
+
+---
+
+## Deliverables
+
+### 1. Foundation Layer
+- Install `@prmichaelsen/remember-core` as dependency
+- Create `src/core-services.ts` — service initialization bridge
+- Wire core services into `server-factory.ts` and `server.ts`
+
+### 2. Tool Handler Migration (18 tools)
+- Relationship tools (4): create, update, search, delete → `RelationshipService`
+- Preference tools (2): get, set → `PreferencesDatabaseService`
+- Memory tools (3): create, update, find_similar → `MemoryService`
+- Space confirmation tools (5): publish, retract, revise, confirm, deny → `SpaceService`
+- Space search/moderate tools (3): search_space, query_space, moderate → `SpaceService`
+- Delete memory (1): delete_memory → `MemoryService` + `ConfirmationTokenService`
+
+### 3. Code Cleanup
+- Remove duplicated source files now provided by remember-core
+- Update imports across codebase
+- Verify no broken references
+
+### 4. Deferred (out of scope)
+- `search_memory` — requires ghost/trust support in core
+- `query_memory` — requires ghost/trust support in core
+- `ghost_config` — no core equivalent
+
+---
+
+## Success Criteria
+
+- [ ] 18 of 21 tool handlers delegate to remember-core services
+- [ ] Each migrated handler is ≤30 lines (thin adapter pattern)
+- [ ] All 454+ tests pass after migration
+- [ ] Build succeeds with no TypeScript errors
+- [ ] Response shapes unchanged (backwards compatible)
+- [ ] Duplicated source files removed
+- [ ] 3 deferred tools (search_memory, query_memory, ghost_config) still work unchanged
+
+---
+
+## Key Files to Create
+
+```
+src/
+├── core-services.ts              # Service initialization bridge
+```
+
+## Key Files to Modify
+
+```
+src/
+├── server-factory.ts             # Wire core services into registerHandlers
+├── server.ts                     # Wire core services (standalone mode)
+├── tools/
+│   ├── create-memory.ts          # → MemoryService.create()
+│   ├── update-memory.ts          # → MemoryService.update()
+│   ├── find-similar.ts           # → MemoryService.findSimilar()
+│   ├── delete-memory.ts          # → MemoryService.delete() + tokens
+│   ├── create-relationship.ts    # → RelationshipService.create()
+│   ├── update-relationship.ts    # → RelationshipService.update()
+│   ├── search-relationship.ts    # → RelationshipService.search()
+│   ├── delete-relationship.ts    # → RelationshipService.delete()
+│   ├── get-preferences.ts        # → PreferencesDatabaseService
+│   ├── set-preference.ts         # → PreferencesDatabaseService
+│   ├── publish.ts                # → SpaceService.publish()
+│   ├── retract.ts                # → SpaceService.retract()
+│   ├── revise.ts                 # → SpaceService.revise()
+│   ├── confirm.ts                # → SpaceService.confirm() + MemoryService.delete()
+│   ├── deny.ts                   # → SpaceService.deny()
+│   ├── moderate.ts               # → SpaceService.moderate()
+│   ├── search-space.ts           # → SpaceService.search()
+│   └── query-space.ts            # → SpaceService.query()
+```
+
+## Key Files to Remove (Phase 5)
+
+```
+src/
+├── services/
+│   ├── confirmation-token.service.ts    # → core ConfirmationTokenService
+│   ├── preferences-database.service.ts  # → core PreferencesDatabaseService
+│   ├── space-config.service.ts          # → core getSpaceConfig
+│   └── credentials-provider.ts          # → core credentialsProvider
+├── utils/
+│   └── weaviate-filters.ts              # → core filter utilities
+├── collections/
+│   ├── dot-notation.ts                  # → core getCollectionName
+│   ├── composite-ids.ts                 # → core generateCompositeId
+│   └── tracking-arrays.ts              # → core tracking array utils
+└── constants/
+    └── content-types.ts                 # → core isValidContentType
+```
+
+---
+
+## Tasks
+
+1. [Task 193: Foundation setup](../tasks/milestone-17-remember-core-migration/task-193-foundation-setup.md) - Install core, create service init, wire server-factory (2-3h)
+2. [Task 194: Migrate relationship tools](../tasks/milestone-17-remember-core-migration/task-194-migrate-relationship-tools.md) - 4 tools → RelationshipService (3-4h)
+3. [Task 195: Migrate preference tools](../tasks/milestone-17-remember-core-migration/task-195-migrate-preference-tools.md) - 2 tools → PreferencesDatabaseService (2-3h)
+4. [Task 196: Migrate memory tools](../tasks/milestone-17-remember-core-migration/task-196-migrate-memory-tools.md) - 3 tools → MemoryService (3-4h)
+5. [Task 197: Migrate space confirmation tools](../tasks/milestone-17-remember-core-migration/task-197-migrate-space-confirmation-tools.md) - publish, retract, revise, confirm, deny → SpaceService (6-8h)
+6. [Task 198: Migrate space search/moderate tools](../tasks/milestone-17-remember-core-migration/task-198-migrate-space-search-moderate.md) - search_space, query_space, moderate → SpaceService (4-6h)
+7. [Task 199: Migrate delete_memory tool](../tasks/milestone-17-remember-core-migration/task-199-migrate-delete-memory.md) - delete_memory → MemoryService + ConfirmationTokenService (2-3h)
+8. [Task 200: Code cleanup & verification](../tasks/milestone-17-remember-core-migration/task-200-code-cleanup-verification.md) - Remove duplicated files, full regression (3-4h)
+
+---
+
+## Risks and Mitigation
+
+| Risk | Impact | Probability | Mitigation Strategy |
+|------|--------|-------------|---------------------|
+| Response shape changes break LLM behavior | High | Medium | Every adapter must match current JSON output exactly |
+| Test mock breakage from refactor | Medium | High | Update mocks from inline Weaviate calls to core service methods |
+| Double Firestore initialization | High | Low | Only use remember-mcp's existing initFirestore(), never core's |
+| ensureMemoryCollection ordering | Medium | Medium | Call in createCoreServices() before constructing services |
+| Singleton vs instance pattern mismatch | Medium | Medium | Create ConfirmationTokenService and PreferencesDatabaseService once in core-services.ts |
+
+---
+
+**Next Milestone**: Ghost/trust support in remember-core (enables deferred tools migration)
+**Blockers**: None (remember-core v0.12.0 published and ready)

package/agent/progress.yaml

@@ -5,8 +5,8 @@ project:
   version: 3.12.0
   started: 2026-02-11
   status: in_progress
-  current_milestone: M16
-  last_updated: 2026-02-27
+  current_milestone: M17
+  last_updated: 2026-02-28
 
 milestones:
   - id: M1
@@ -325,6 +325,22 @@ milestones:
     progress: 0%
     estimated_weeks: 1
 
+  - id: M17
+    name: remember-core Migration
+    status: completed
+    progress: 100%
+    estimated_weeks: 2-3
+    tasks_completed: 8
+    tasks_total: 8
+    completed: 2026-02-28
+    notes: |
+      ✅ All 18 tool handlers migrated from inline business logic to remember-core service calls.
+      ✅ Each handler is now a thin 10-20 line adapter calling core services.
+      ✅ 10 duplicated source files removed (services, constants, collections).
+      ✅ Build passing, 390 tests (389 passed, 1 skipped). 64 tests removed (now in core).
+      3 tools deferred (search_memory, query_memory, ghost_config) — require core ghost/trust support.
+      See: agent/milestones/milestone-17-remember-core-migration.md
+
 tasks:
   milestone_1:
     - id: task-1
@@ -879,18 +895,119 @@ tasks:
         ✅ Design doc status updated to Implemented
         ✅ 454 tests passing, typecheck clean, build clean
 
+  milestone_17:
+    - id: task-193
+      name: Foundation Setup
+      status: completed
+      file: agent/tasks/milestone-17-remember-core-migration/task-193-foundation-setup.md
+      estimated_hours: 2-3
+      completed_date: 2026-02-28
+      notes: |
+        ✅ Installed @prmichaelsen/remember-core v0.12.0
+        ✅ Created src/core-services.ts (bridge module)
+        ✅ Build passing, 454 tests (453 passed, 1 skipped)
+
+    - id: task-194
+      name: Migrate Relationship Tools
+      status: completed
+      completed_date: 2026-02-28
+      file: agent/tasks/milestone-17-remember-core-migration/task-194-migrate-relationship-tools.md
+      estimated_hours: 3-4
+      dependencies: [task-193]
+      notes: |
+        4 tools → RelationshipService: create, update, search, delete
+
+    - id: task-195
+      name: Migrate Preference Tools
+      status: completed
+      completed_date: 2026-02-28
+      file: agent/tasks/milestone-17-remember-core-migration/task-195-migrate-preference-tools.md
+      estimated_hours: 2-3
+      dependencies: [task-193]
+      notes: |
+        2 tools → PreferencesDatabaseService: get, set
+
+    - id: task-196
+      name: Migrate Memory Tools
+      status: completed
+      completed_date: 2026-02-28
+      file: agent/tasks/milestone-17-remember-core-migration/task-196-migrate-memory-tools.md
+      estimated_hours: 3-4
+      dependencies: [task-193]
+      notes: |
+        ✅ 3 tools → MemoryService: create, update, find_similar.
+        ✅ Ghost exclusion post-filter in find-similar adapter.
+        search_memory and query_memory deferred (ghost-dependent).
+
+    - id: task-197
+      name: Migrate Space Confirmation Tools
+      status: completed
+      completed_date: 2026-02-28
+      file: agent/tasks/milestone-17-remember-core-migration/task-197-migrate-space-confirmation-tools.md
+      estimated_hours: 6-8
+      dependencies: [task-193]
+      notes: |
+        ✅ 5 tools → SpaceService: publish, retract, revise, confirm, deny.
+        ✅ confirm.ts: 1180→120 lines. delete_memory handled inline, rest delegates to core.
+
+    - id: task-198
+      name: Migrate Space Search/Moderate Tools
+      status: completed
+      completed_date: 2026-02-28
+      file: agent/tasks/milestone-17-remember-core-migration/task-198-migrate-space-search-moderate.md
+      estimated_hours: 4-6
+      dependencies: [task-193]
+      notes: |
+        ✅ 3 tools → SpaceService: search_space, query_space, moderate.
+        ✅ query_space format options stay in adapter.
+        ✅ Deprecated buildBaseFilters/buildModerationFilter kept for test compat.
+
+    - id: task-199
+      name: Migrate delete_memory Tool
+      status: completed
+      completed_date: 2026-02-28
+      file: agent/tasks/milestone-17-remember-core-migration/task-199-migrate-delete-memory.md
+      estimated_hours: 2-3
+      dependencies: [task-193, task-197]
+      notes: |
+        ✅ 1 tool → ConfirmationTokenService.createRequest().
+        ✅ Memory validation deferred to confirm phase.
+
+    - id: task-200
+      name: Code Cleanup & Verification
+      status: completed
+      completed_date: 2026-02-28
+      file: agent/tasks/milestone-17-remember-core-migration/task-200-code-cleanup-verification.md
+      estimated_hours: 3-4
+      dependencies: [task-194, task-195, task-196, task-197, task-198, task-199]
+      notes: |
+        ✅ Removed 10 duplicated files (3 services, 1 constant, 3 collections, 3 specs).
+        ✅ Updated imports in create-memory.ts and v2-smoke.e2e.ts to use @prmichaelsen/remember-core.
+        ✅ Removed empty src/constants/ and src/collections/ directories.
+        ✅ Build passing, 390 tests (389 passed, 1 skipped).
+
 documentation:
   design_documents: 31
-  milestone_documents: 12
+  milestone_documents: 13
   pattern_documents: 7
-  task_documents: 75
+  task_documents: 83
 
 progress:
   planning: 100%
-  implementation: 70%
-  overall: 75%
+  implementation: 80%
+  overall: 85%
 
 recent_work:
+  - date: 2026-02-28
+    description: M17 remember-core Migration COMPLETED
+    items:
+      - ✅ All 8 tasks completed (task-193 through task-200)
+      - ✅ 18 tool handlers migrated to thin adapters calling core services
+      - ✅ 10 duplicated files removed, 2 imports updated to use core
+      - ✅ Build passing, 390 tests (389 passed, 1 skipped)
+      - 📋 3 tools deferred (search_memory, query_memory, ghost_config)
+      - 📋 Phased approach: foundation → relationships → preferences → memory → space → cleanup
+
   - date: 2026-02-27
     description: M16 Ghost System COMPLETE (v3.12.0)
     items:

package/agent/tasks/milestone-17-remember-core-migration/task-193-foundation-setup.md

@@ -0,0 +1,58 @@
+# Task 193: Foundation Setup
+
+**Milestone**: M17 — remember-core Migration
+**Status**: not_started
+**Dependencies**: None
+**Estimated Hours**: 2-3
+
+---
+
+## Objective
+
+Install remember-core, create the service initialization bridge module, and wire core services into both server entry points without changing any tool behavior.
+
+## Deliverables
+
+### 1. Install `@prmichaelsen/remember-core`
+
+Add to `package.json`:
+```json
+"@prmichaelsen/remember-core": "^0.12.0"
+```
+
+### 2. Create `src/core-services.ts`
+
+Service initialization bridge that creates core services scoped to a userId:
+
+```typescript
+import { MemoryService, RelationshipService, SpaceService,
+  PreferencesDatabaseService, ConfirmationTokenService, createLogger
+} from '@prmichaelsen/remember-core';
+
+export function createCoreServices(userId: string) { ... }
+```
+
+Key requirements:
+- Use remember-mcp's existing `getWeaviateClient()` and `getMemoryCollection(userId)`
+- Call `ensureMemoryCollection(userId)` before constructing services
+- Create `ConfirmationTokenService` and `PreferencesDatabaseService` as singletons (shared across users)
+- Do NOT call core's `initFirestore()` — use remember-mcp's existing init
+
+### 3. Wire into `src/server-factory.ts`
+
+- Create core services in `createServer()` after `ensureDatabasesInitialized()`
+- Pass services to `registerHandlers()` as new parameter
+- No tool handler changes yet — services are created but unused
+
+### 4. Wire into `src/server.ts`
+
+- Create core services per-request or use lazy cache by userId
+- Pass services to tool handlers (unused initially)
+
+## Acceptance Criteria
+
+- [ ] `npm install` succeeds with remember-core dependency
+- [ ] `npm run build` passes with no TypeScript errors
+- [ ] `npm test` — all 454 tests still pass (no behavior changes)
+- [ ] `createCoreServices(userId)` returns all 5 service instances
+- [ ] Both server.ts and server-factory.ts create and pass services

package/agent/tasks/milestone-17-remember-core-migration/task-194-migrate-relationship-tools.md

@@ -0,0 +1,47 @@
+# Task 194: Migrate Relationship Tools
+
+**Milestone**: M17 — remember-core Migration
+**Status**: not_started
+**Dependencies**: Task 193 (Foundation setup)
+**Estimated Hours**: 3-4
+
+---
+
+## Objective
+
+Migrate 4 relationship tool handlers from inline Weaviate logic to `RelationshipService` adapters.
+
+## Tools to Migrate
+
+| Tool | File | Core Method |
+|------|------|-------------|
+| `remember_create_relationship` | `src/tools/create-relationship.ts` | `RelationshipService.create()` |
+| `remember_update_relationship` | `src/tools/update-relationship.ts` | `RelationshipService.update()` |
+| `remember_search_relationship` | `src/tools/search-relationship.ts` | `RelationshipService.search()` |
+| `remember_delete_relationship` | `src/tools/delete-relationship.ts` | `RelationshipService.delete()` |
+
+## Adapter Pattern
+
+Each handler becomes ~15-20 lines:
+1. Create debug logger (stays in adapter)
+2. Get `relationship` service from `services` param or `createCoreServices(userId)`
+3. Map MCP args to core input type
+4. Call core service method
+5. Add supplementary response fields (`message`, etc.) to match current JSON shape
+6. Wrap in `handleToolError()` catch
+
+## Key Requirements
+
+- **Response shape preservation**: JSON output must be identical to current output
+- **Debug logging**: `createDebugLogger()` stays in adapter layer
+- **Tool definitions**: `createRelationshipTool` etc. unchanged (MCP schemas stay)
+- **Error handling**: `handleToolError()` wrapper stays in adapter
+
+## Acceptance Criteria
+
+- [ ] All 4 relationship tool handlers delegate to RelationshipService
+- [ ] Each handler is ≤25 lines
+- [ ] Existing relationship tests pass without mock changes (or mocks updated)
+- [ ] `npm run build` passes
+- [ ] `npm test` passes
+- [ ] Response JSON shapes match pre-migration output

package/agent/tasks/milestone-17-remember-core-migration/task-195-migrate-preference-tools.md

@@ -0,0 +1,34 @@
+# Task 195: Migrate Preference Tools
+
+**Milestone**: M17 — remember-core Migration
+**Status**: not_started
+**Dependencies**: Task 193 (Foundation setup)
+**Estimated Hours**: 2-3
+
+---
+
+## Objective
+
+Migrate 2 preference tool handlers from static method calls to `PreferencesDatabaseService` instance methods.
+
+## Tools to Migrate
+
+| Tool | File | Core Method |
+|------|------|-------------|
+| `remember_get_preferences` | `src/tools/get-preferences.ts` | `PreferencesDatabaseService.getPreferences()` |
+| `remember_set_preference` | `src/tools/set-preference.ts` | `PreferencesDatabaseService.updatePreferences()` |
+
+## Key Differences
+
+- **Static → Instance**: remember-mcp uses `PreferencesDatabaseService.getPreferences(userId)` (static). Core uses `preferencesService.getPreferences(userId)` (instance method).
+- **Category filtering**: `get_preferences` does category filtering in the tool handler. This presentation logic stays in the adapter.
+- **formatPreferenceChangeMessage()**: MCP-specific helper stays in adapter.
+
+## Acceptance Criteria
+
+- [ ] Both preference tool handlers delegate to core PreferencesDatabaseService
+- [ ] Category filtering preserved in get_preferences adapter
+- [ ] `formatPreferenceChangeMessage()` still used in set_preference
+- [ ] `npm run build` passes
+- [ ] `npm test` passes
+- [ ] Response JSON shapes match pre-migration output

package/agent/tasks/milestone-17-remember-core-migration/task-196-migrate-memory-tools.md

@@ -0,0 +1,46 @@
+# Task 196: Migrate Memory Tools
+
+**Milestone**: M17 — remember-core Migration
+**Status**: not_started
+**Dependencies**: Task 193 (Foundation setup)
+**Estimated Hours**: 3-4
+
+---
+
+## Objective
+
+Migrate 3 non-ghost memory tool handlers to `MemoryService` adapters.
+
+## Tools to Migrate
+
+| Tool | File | Core Method |
+|------|------|-------------|
+| `remember_create_memory` | `src/tools/create-memory.ts` | `MemoryService.create()` |
+| `remember_update_memory` | `src/tools/update-memory.ts` | `MemoryService.update()` |
+| `remember_find_similar` | `src/tools/find-similar.ts` | `MemoryService.findSimilar()` |
+
+## NOT Migrated (deferred — ghost-dependent)
+
+- `remember_search_memory` — uses `buildTrustFilter()`, ghost exclusion
+- `remember_query_memory` — uses `buildTrustFilter()`, ghost exclusion
+
+## Key Gaps
+
+### create_memory
+- `structured_content` and `skip_template_suggestion` args exist in MCP but NOT in core's `CreateMemoryInput`. Handle in adapter layer (pass through to response, not to core).
+- `context_summary` default differs: MCP uses `'Memory created via MCP'`, core uses `'Memory created'`. Set explicitly in adapter.
+
+### find_similar
+- **Ghost exclusion**: MCP applies `content_type != 'ghost'` filter. Core doesn't exclude ghosts. Post-filter ghost results in adapter.
+
+### update_memory
+- `structured_content` may exist in MCP args but not core input. Handle in adapter.
+
+## Acceptance Criteria
+
+- [ ] All 3 memory tool handlers delegate to MemoryService
+- [ ] Ghost exclusion in find_similar handled via post-filtering
+- [ ] `structured_content` and `skip_template_suggestion` handled in adapter
+- [ ] `npm run build` passes
+- [ ] `npm test` passes
+- [ ] Response JSON shapes match pre-migration output

package/agent/tasks/milestone-17-remember-core-migration/task-197-migrate-space-confirmation-tools.md

@@ -0,0 +1,49 @@
+# Task 197: Migrate Space Confirmation Tools
+
+**Milestone**: M17 — remember-core Migration
+**Status**: not_started
+**Dependencies**: Task 193 (Foundation setup)
+**Estimated Hours**: 6-8
+
+---
+
+## Objective
+
+Migrate 5 space confirmation-flow tools to `SpaceService` adapters. This is the most complex task — `confirm.ts` is 1180 lines with 4 action branches.
+
+## Tools to Migrate
+
+| Tool | File | Core Method |
+|------|------|-------------|
+| `remember_publish` | `src/tools/publish.ts` | `SpaceService.publish()` |
+| `remember_retract` | `src/tools/retract.ts` | `SpaceService.retract()` |
+| `remember_revise` | `src/tools/revise.ts` | `SpaceService.revise()` |
+| `remember_confirm` | `src/tools/confirm.ts` | `SpaceService.confirm()` + `MemoryService.delete()` |
+| `remember_deny` | `src/tools/deny.ts` | `SpaceService.deny()` |
+
+## Key Complexity: confirm.ts
+
+`confirm.ts` dispatches to 4 action executors:
+- `executePublishMemory` → `SpaceService.confirm()` (action: publish_memory)
+- `executeDeleteMemory` → `MemoryService.delete()` (core does direct soft-delete)
+- `executeRetractMemory` → `SpaceService.confirm()` (action: retract_memory)
+- `executeReviseMemory` → `SpaceService.confirm()` (action: revise_memory)
+
+**Critical**: Core's `SpaceService.confirm()` handles publish/retract/revise. For delete_memory, use `MemoryService.delete()` since confirmation token was already validated.
+
+## Error Handling
+
+- MCP tools return verbose JSON error responses (`{ success: false, error: '...', message: '...' }`)
+- Core throws errors
+- Adapters must catch core errors and format into expected JSON responses
+
+## Acceptance Criteria
+
+- [ ] publish, retract, revise generate tokens via SpaceService
+- [ ] confirm delegates to SpaceService.confirm() for publish/retract/revise
+- [ ] confirm delegates to MemoryService.delete() for delete_memory action
+- [ ] deny delegates to SpaceService.deny()
+- [ ] Error responses match current JSON format
+- [ ] `npm run build` passes
+- [ ] `npm test` passes (especially confirm-publish-moderation tests)
+- [ ] Response JSON shapes match pre-migration output

package/agent/tasks/milestone-17-remember-core-migration/task-198-migrate-space-search-moderate.md

@@ -0,0 +1,46 @@
+# Task 198: Migrate Space Search/Moderate Tools
+
+**Milestone**: M17 — remember-core Migration
+**Status**: not_started
+**Dependencies**: Task 193 (Foundation setup)
+**Estimated Hours**: 4-6
+
+---
+
+## Objective
+
+Migrate 3 space search and moderation tools to `SpaceService` adapters.
+
+## Tools to Migrate
+
+| Tool | File | Core Method |
+|------|------|-------------|
+| `remember_search_space` | `src/tools/search-space.ts` | `SpaceService.search()` |
+| `remember_query_space` | `src/tools/query-space.ts` | `SpaceService.query()` |
+| `remember_moderate` | `src/tools/moderate.ts` | `SpaceService.moderate()` |
+
+## Key Details
+
+### search_space
+- Maps directly to `SpaceService.search(input, authContext)`
+- No ghost dependency (ghost filtering only on personal collections)
+- Moderator permission checks handled by core service
+
+### query_space
+- **Format gap**: MCP has `format: 'detailed'|'compact'` and `include_context` options not in core's `QuerySpaceInput`
+- Core returns raw results; adapter applies formatting (compact text summary vs detailed objects)
+- Formatting logic stays in adapter
+
+### moderate
+- Maps to `SpaceService.moderate(input, authContext)`
+- Core's AuthContext (no ghostMode) is fine — moderate doesn't use ghostMode
+- Permission checking (`canModerate`, `canModerateAny`) handled by core
+
+## Acceptance Criteria
+
+- [ ] All 3 tools delegate to SpaceService
+- [ ] query_space format/include_context options handled in adapter
+- [ ] Moderator permission checks work correctly via core
+- [ ] `npm run build` passes
+- [ ] `npm test` passes
+- [ ] Response JSON shapes match pre-migration output

package/agent/tasks/milestone-17-remember-core-migration/task-199-migrate-delete-memory.md

@@ -0,0 +1,43 @@
+# Task 199: Migrate delete_memory Tool
+
+**Milestone**: M17 — remember-core Migration
+**Status**: not_started
+**Dependencies**: Task 193 (Foundation setup), Task 197 (confirm.ts migration)
+**Estimated Hours**: 2-3
+
+---
+
+## Objective
+
+Migrate the delete_memory tool to use core services while preserving the two-phase confirmation flow.
+
+## Tool to Migrate
+
+| Tool | File | Core Methods |
+|------|------|-------------|
+| `remember_delete_memory` | `src/tools/delete-memory.ts` | `MemoryService` (fetch/validate) + `ConfirmationTokenService` (token) |
+
+## Two-Phase Pattern
+
+**Phase 1** (`handleDeleteMemory`):
+1. Use `MemoryService` to fetch and validate the memory (ownership, doc_type, not-deleted checks)
+2. Detect orphaned relationships (relationships that would be orphaned by deletion)
+3. Use `ConfirmationTokenService` from core to create confirmation token
+4. Return token preview to user
+
+**Phase 2** (in `confirm.ts`, already migrated in Task 197):
+- Use `MemoryService.delete()` for soft-delete when token is confirmed
+
+## Key Requirement
+
+- The confirmation token flow MUST be preserved — no direct deletion
+- Orphan detection logic may need to stay in adapter if core doesn't provide it
+
+## Acceptance Criteria
+
+- [ ] Phase 1 uses core services for fetch/validate and token creation
+- [ ] Phase 2 (in confirm.ts) uses MemoryService.delete()
+- [ ] Orphan detection still works
+- [ ] `npm run build` passes
+- [ ] `npm test` passes
+- [ ] Response JSON shapes match pre-migration output