memory-journal-mcp 7.2.0 → 7.3.0

package/README.md

@@ -10,7 +10,7 @@
 [![MCP Registry](https://img.shields.io/badge/MCP_Registry-Published-green)](https://registry.modelcontextprotocol.io/v0/servers?search=io.github.neverinfamous/memory-journal-mcp)
 [![Security](https://img.shields.io/badge/Security-Enhanced-green.svg)](SECURITY.md)
 [![TypeScript](https://img.shields.io/badge/TypeScript-Strict-blue.svg)](https://github.com/neverinfamous/memory-journal-mcp)
-![Coverage](https://img.shields.io/badge/Coverage-97.14%25-brightgreen.svg)
+![Coverage](https://img.shields.io/badge/Coverage-97.1%25-brightgreen.svg)
 ![Tests](https://img.shields.io/badge/Tests-1782_passed-brightgreen.svg)
 ![E2E Tests](https://img.shields.io/badge/E2E_Tests-391_passed-brightgreen.svg)
 [![CI](https://github.com/neverinfamous/memory-journal-mcp/actions/workflows/gatekeeper.yml/badge.svg)](https://github.com/neverinfamous/memory-journal-mcp/actions/workflows/gatekeeper.yml)
@@ -30,24 +30,24 @@
 
 **67 MCP Tools** · **17 Workflow Prompts** · **34 Resources** · **10 Tool Groups** · **Code Mode** · **GitHub Commander** (Issue Triage, PR Review, Milestone Sprints, Security/Quality/Perf Audits) · **GitHub Integration** (Issues, PRs, Actions, Kanban, Milestones, Insights) · **Team Collaboration** (Shared DB, Vector Search, Cross-Project Insights)
 
-| Feature                       | Description                                                                                                                                                                                                                     |
-| ----------------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
-| **Session Intelligence**      | Agents auto-query project history, create entries at checkpoints, and hand off context between sessions via `/session-summary` and `team-session-summary`                                                                       |
-| **GitHub Integration**        | 18 tools for Issues, PRs, Actions, Kanban, Milestones (%), Copilot Reviews, and 14-day Insights                                                                                                                                 |
-| **Dynamic Project Routing**   | Seamlessly switch contexts and access CI/Issue tracking across multiple repositories using a single server instance via `PROJECT_REGISTRY`                                                                                      |
-| **Knowledge Graphs**          | 8 relationship types linking specs → implementations → tests → PRs with Mermaid visualization                                                                                                                                   |
-| **Hybrid Search**             | Reciprocal Rank Fusion combining FTS5 keywords, semantic vector similarity, auto-heuristics, and date-range filters                                                                                                             |
-| **Code Mode**                 | Execute multi-step operations in a secure sandbox — up to 90% token savings via `mj.*` API                                                                                                                                      |
-| **Configurable Briefing**     | 14 env vars / CLI flags control `memory://briefing` content — entries, team, GitHub detail, skills awareness                                                                                                                    |
-| **Reports & Analytics**       | Standups, retrospectives, PR summaries, digests, period analyses, and milestone tracking                                                                                                                                        |
-| **Team Collaboration**        | 22 tools with full parity — CRUD, vector search, relationship graphs, cross-project insights, author attribution                                                                                                                |
-| **Data Interoperability**     | Bidirectional Markdown roundtripping, unified IO namespace, and schema-safe JSON exports with hard bounds-checked path traversal defenses                                                                                       |
-| **Backup & Restore**          | One-command backup/restore with automated scheduling, retention policies, and safety-net auto-backups                                                                                                                           |
-| **Security & Transport**      | OAuth 2.1 (RFC 9728/8414, JWT/JWKS, scopes), Streamable HTTP + SSE, rate limiting, CORS, SQL injection prevention, non-root Docker                                                                                              |
-| **Structured Error Handling** | Every tool returns `{success, error, code, category, suggestion, recoverable}` — agents get classification, remediation hints, and recoverability signals                                                                       |
-| **Agent Collaboration**       | IDE agents and Copilot share context; review findings become searchable knowledge; agents suggest reusable rules and skills ([setup](docs/copilot-setup.md))                                                                    |
+| Feature                       | Description                                                                                                                                                                                                                                                                                        |
+| ----------------------------- | -------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- |
+| **Session Intelligence**      | Agents auto-query project history, create entries at checkpoints, and hand off context between sessions via `/session-summary` and `team-session-summary`                                                                                                                                          |
+| **GitHub Integration**        | 18 tools for Issues, PRs, Actions, Kanban, Milestones (%), Copilot Reviews, and 14-day Insights                                                                                                                                                                                                    |
+| **Dynamic Project Routing**   | Seamlessly switch contexts and access CI/Issue tracking across multiple repositories using a single server instance via `PROJECT_REGISTRY`                                                                                                                                                         |
+| **Knowledge Graphs**          | 8 relationship types linking specs → implementations → tests → PRs with Mermaid visualization                                                                                                                                                                                                      |
+| **Hybrid Search**             | Reciprocal Rank Fusion combining FTS5 keywords, semantic vector similarity, auto-heuristics, and date-range filters                                                                                                                                                                                |
+| **Code Mode**                 | Execute multi-step operations in a secure sandbox — up to 90% token savings via `mj.*` API                                                                                                                                                                                                         |
+| **Configurable Briefing**     | 14 env vars / CLI flags control `memory://briefing` content — entries, team, GitHub detail, skills awareness                                                                                                                                                                                       |
+| **Reports & Analytics**       | Standups, retrospectives, PR summaries, digests, period analyses, and milestone tracking                                                                                                                                                                                                           |
+| **Team Collaboration**        | 22 tools with full parity — CRUD, vector search, relationship graphs, cross-project insights, author attribution                                                                                                                                                                                   |
+| **Data Interoperability**     | Bidirectional Markdown roundtripping, unified IO namespace, and schema-safe JSON exports with hard bounds-checked path traversal defenses                                                                                                                                                          |
+| **Backup & Restore**          | One-command backup/restore with automated scheduling, retention policies, and safety-net auto-backups                                                                                                                                                                                              |
+| **Security & Transport**      | OAuth 2.1 (RFC 9728/8414, JWT/JWKS, scopes), Streamable HTTP + SSE, rate limiting, CORS, SQL injection prevention, non-root Docker                                                                                                                                                                 |
+| **Structured Error Handling** | Every tool returns `{success, error, code, category, suggestion, recoverable}` — agents get classification, remediation hints, and recoverability signals                                                                                                                                          |
+| **Agent Collaboration**       | IDE agents and Copilot share context; review findings become searchable knowledge; agents suggest reusable rules and skills ([setup](docs/copilot-setup.md))                                                                                                                                       |
 | **Native Agent Skills**       | Bundled foundational coding paradigms (`autonomous-dev`, `bun`, `gitlab`, `golang`, `mysql`, `playwright-standard`, `postgres`, `react-best-practices`, `rust`, `shadcn-ui`, `skill-builder`, `sqlite`, `typescript`, `vitest-standard`) establishing permanent AI behavior and architecture rules |
-| **GitHub Commander**          | Pipeline skills for issue triage, PR reviews, sprint milestones, and security/quality/performance audits with journal trails ([docs](skills/github-commander/SKILL.md))                                                         |
+| **GitHub Commander**          | Pipeline skills for issue triage, PR reviews, sprint milestones, and security/quality/performance audits with journal trails ([docs](skills/github-commander/SKILL.md))                                                                                                                            |
 
 ---
 
@@ -246,7 +246,7 @@ Control which tools are exposed via `MEMORY_JOURNAL_MCP_TOOL_FILTER` (or CLI: `-
 - `memory://help/{group}` - Per-group tool reference with parameters and annotations
 - `memory://briefing/{repo}` - Context targeted to a specific repository
 
-*Note: The `memory://github/status`, `memory://github/insights`, `memory://github/milestones`, and `memory://milestones/{number}` resources also accept an optional `/{repo}` path suffix for cross-repo targeting.*
+_Note: The `memory://github/status`, `memory://github/insights`, `memory://github/milestones`, and `memory://milestones/{number}` resources also accept an optional `/{repo}` path suffix for cross-repo targeting._
 
 ---
 

package/dist/{chunk-GR4T3SRW.js → chunk-CHWIPVQN.js}

@@ -1,5 +1,5 @@
-import { withSessionInit, withPriority, ASSISTANT_FOCUSED, TOOL_GROUPS, HIGH_PRIORITY, LOW_PRIORITY, MEDIUM_PRIORITY, setDefaultSandboxMode, initializeAuditLogger, parseToolFilter, getFilterSummary, getToolFilterFromEnv, getTools, getEnabledGroups, callTool, getGlobalAuditLogger, sendProgress, SUPPORTED_SCOPES, getRequiredScope, hasScope, getAuditResourceDef, execQuery, transformEntryRow, resolveGitHubRepo, isResourceError, milestoneCompletionPct, parseScopes, BASE_SCOPES, getAllToolNames, globalMetrics, DEFAULT_BRIEFING_CONFIG } from './chunk-ORV7ZZOE.js';
-import { logger, GitHubIntegration, ConfigurationError, ResourceNotFoundError, ConnectionError, QueryError, assertNoPathTraversal, ValidationError, MemoryJournalMcpError, validateDateFormatPattern } from './chunk-IWKLHSPU.js';
+import { withSessionInit, withPriority, ASSISTANT_FOCUSED, TOOL_GROUPS, HIGH_PRIORITY, LOW_PRIORITY, MEDIUM_PRIORITY, setDefaultSandboxMode, initializeAuditLogger, parseToolFilter, getFilterSummary, getToolFilterFromEnv, getTools, getEnabledGroups, callTool, getGlobalAuditLogger, sendProgress, SUPPORTED_SCOPES, getRequiredScope, hasScope, getAuditResourceDef, execQuery, transformEntryRow, resolveGitHubRepo, isResourceError, milestoneCompletionPct, parseScopes, BASE_SCOPES, getAllToolNames, globalMetrics, DEFAULT_BRIEFING_CONFIG } from './chunk-ZJJD2F5T.js';
+import { logger, GitHubIntegration, ConfigurationError, ResourceNotFoundError, ConnectionError, QueryError, assertNoPathTraversal, ValidationError, MemoryJournalMcpError, validateDateFormatPattern } from './chunk-WXDEVIFL.js';
 import { createRequire } from 'module';
 import { McpServer, ResourceTemplate } from '@modelcontextprotocol/sdk/server/mcp.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
@@ -1540,7 +1540,7 @@ var VectorSearchManager = class {
                      WHERE embedding MATCH ?
                      ORDER BY distance
                      LIMIT ?`
-      ).all(queryVec, limit * 2);
+      ).all(queryVec, limit);
       const filteredResults = results.map((r) => ({
         entryId: r.entry_id,
         score: 1 / (1 + r.distance)
@@ -1593,7 +1593,7 @@ var VectorSearchManager = class {
                      WHERE embedding MATCH ?
                      ORDER BY distance
                      LIMIT ?`
-      ).all(queryVec, (limit + 1) * 2);
+      ).all(queryVec, limit + 1);
       const filteredResults = results.filter((r) => r.entry_id !== entryId).map((r) => ({
         entryId: r.entry_id,
         score: 1 / (1 + r.distance)
@@ -2476,18 +2476,60 @@ function buildQuickAccess(groups) {
   return table;
 }
 var CODE_MODE_NAMESPACE_ROWS = [
-  { group: "core", label: "Core", namespace: "`mj.core.*`", example: '`mj.core.createEntry("Implemented feature X")`' },
-  { group: "search", label: "Search", namespace: "`mj.search.*`", example: '`mj.search.searchEntries("performance")`' },
-  { group: "analytics", label: "Analytics", namespace: "`mj.analytics.*`", example: "`mj.analytics.getStatistics()`" },
-  { group: "relationships", label: "Relationships", namespace: "`mj.relationships.*`", example: '`mj.relationships.linkEntries(1, 2, "implements")`' },
+  {
+    group: "core",
+    label: "Core",
+    namespace: "`mj.core.*`",
+    example: '`mj.core.createEntry("Implemented feature X")`'
+  },
+  {
+    group: "search",
+    label: "Search",
+    namespace: "`mj.search.*`",
+    example: '`mj.search.searchEntries("performance")`'
+  },
+  {
+    group: "analytics",
+    label: "Analytics",
+    namespace: "`mj.analytics.*`",
+    example: "`mj.analytics.getStatistics()`"
+  },
+  {
+    group: "relationships",
+    label: "Relationships",
+    namespace: "`mj.relationships.*`",
+    example: '`mj.relationships.linkEntries(1, 2, "implements")`'
+  },
   { group: "io", label: "IO", namespace: "`mj.io.*`", example: '`mj.io.exportEntries("json")`' },
-  { group: "admin", label: "Admin", namespace: "`mj.admin.*`", example: "`mj.admin.rebuildVectorIndex()`" },
-  { group: "github", label: "GitHub", namespace: "`mj.github.*`", example: '`mj.github.getGithubIssues({ state: "open" })`' },
-  { group: "backup", label: "Backup", namespace: "`mj.backup.*`", example: "`mj.backup.backupJournal()`" },
-  { group: "team", label: "Team", namespace: "`mj.team.*`", example: '`mj.team.teamCreateEntry("Team update")`' }
+  {
+    group: "admin",
+    label: "Admin",
+    namespace: "`mj.admin.*`",
+    example: "`mj.admin.rebuildVectorIndex()`"
+  },
+  {
+    group: "github",
+    label: "GitHub",
+    namespace: "`mj.github.*`",
+    example: '`mj.github.getGithubIssues({ state: "open" })`'
+  },
+  {
+    group: "backup",
+    label: "Backup",
+    namespace: "`mj.backup.*`",
+    example: "`mj.backup.backupJournal()`"
+  },
+  {
+    group: "team",
+    label: "Team",
+    namespace: "`mj.team.*`",
+    example: '`mj.team.teamCreateEntry("Team update")`'
+  }
 ];
 function buildCodeModeInstructions(groups) {
-  const rows = CODE_MODE_NAMESPACE_ROWS.filter((r) => groups.has(r.group)).map((r) => `| ${r.label.padEnd(13)} | ${r.namespace.padEnd(20)} | ${r.example.padEnd(50)} |`).join("\n");
+  const rows = CODE_MODE_NAMESPACE_ROWS.filter((r) => groups.has(r.group)).map(
+    (r) => `| ${r.label.padEnd(13)} | ${r.namespace.padEnd(20)} | ${r.example.padEnd(50)} |`
+  ).join("\n");
   const fullSection = CODE_MODE_FULL_TEXT;
   const tableStart = fullSection.indexOf("| Group");
   const tableEnd = fullSection.indexOf("\n\n**Features**");
@@ -4173,21 +4215,25 @@ function getGitHubResourceDefinitions() {
             error: kanbanResult.reason
           });
         }
-        let milestoneSummary = null;
-        if (milestoneResult.status === "fulfilled" && milestoneResult.value.length > 0) {
-          milestoneSummary = milestoneResult.value.map((ms) => {
-            const pct = milestoneCompletionPct(ms.openIssues, ms.closedIssues);
-            return {
-              number: ms.number,
-              title: ms.title,
-              state: ms.state,
-              openIssues: ms.openIssues,
-              closedIssues: ms.closedIssues,
-              completionPercentage: pct,
-              dueOn: ms.dueOn
-            };
-          });
+        let milestoneSummary = { openCount: 0, items: [] };
+        if (milestoneResult.status === "fulfilled") {
+          milestoneSummary = {
+            openCount: milestoneResult.value.length,
+            items: milestoneResult.value.map((ms) => {
+              const pct = milestoneCompletionPct(ms.openIssues, ms.closedIssues);
+              return {
+                number: ms.number,
+                title: ms.title,
+                state: ms.state,
+                openIssues: ms.openIssues,
+                closedIssues: ms.closedIssues,
+                completionPercentage: pct,
+                dueOn: ms.dueOn
+              };
+            })
+          };
         } else if (milestoneResult.status === "rejected") {
+          milestoneSummary = null;
           logger.debug("Failed to fetch milestones", {
             module: "RESOURCE",
             operation: "github-status",
@@ -4933,7 +4979,7 @@ function getHelpResourceDefinitions() {
 var toolIndexModule = null;
 async function getAllToolDefinitionsAsync(context) {
   try {
-    toolIndexModule ??= await import('./tools-CXR2FEB2.js');
+    toolIndexModule ??= await import('./tools-MNMGDTQI.js');
     if (toolIndexModule === null) return [];
     const tools = toolIndexModule.getTools(context.db, null);
     return tools.map((t) => ({
@@ -6489,15 +6535,30 @@ function registerPrompts(server, prompts, db, teamDb) {
         ...promptDef.icons ? { icons: promptDef.icons } : {}
       },
       (providedArgs) => {
-        const args = providedArgs;
-        const promptResult = getPrompt(promptDef.name, args, db, teamDb);
-        const result = {
-          messages: promptResult.messages.map((m) => ({
-            role: m.role,
-            content: m.content
-          }))
-        };
-        return Promise.resolve(result);
+        try {
+          const args = providedArgs;
+          const promptResult = getPrompt(promptDef.name, args, db, teamDb);
+          const result = {
+            messages: promptResult.messages.map((m) => ({
+              role: m.role,
+              content: m.content
+            }))
+          };
+          return Promise.resolve(result);
+        } catch (err) {
+          const message = err instanceof Error ? err.message : String(err);
+          return Promise.resolve({
+            messages: [
+              {
+                role: "user",
+                content: {
+                  type: "text",
+                  text: `[Prompt handler error] ${message}`
+                }
+              }
+            ]
+          });
+        }
       }
     );
   }

package/dist/{chunk-IWKLHSPU.js → chunk-WXDEVIFL.js}

@@ -35,7 +35,7 @@ var ERROR_SUGGESTIONS = [
   {
     pattern: /SQLITE_CONSTRAINT|unique constraint/i,
     suggestion: "A uniqueness or integrity constraint was violated. Check input values.",
-    code: "VALIDATION_FAILED"
+    code: "VALIDATION_ERROR"
   },
   // Disk / space patterns
   {
@@ -46,18 +46,18 @@ var ERROR_SUGGESTIONS = [
   {
     pattern: /malformed|invalid json|unexpected token/i,
     suggestion: "The input appears malformed. Check the format and try again.",
-    code: "VALIDATION_FAILED"
+    code: "VALIDATION_ERROR"
   },
   // Schema / types patterns
   {
     pattern: /invalid input syntax for type|requires a.*column/i,
     suggestion: "The provided value is not valid for the assigned type.",
-    code: "VALIDATION_FAILED"
+    code: "VALIDATION_ERROR"
   },
   {
     pattern: /^Missing required parameters:/i,
     suggestion: "Provide all required parameters in your request.",
-    code: "VALIDATION_FAILED"
+    code: "VALIDATION_ERROR"
   },
   // Codemode / Sandbox patterns
   {
@@ -68,7 +68,7 @@ var ERROR_SUGGESTIONS = [
   {
     pattern: /code validation failed/i,
     suggestion: "Check for blocked patterns. Use mj.* API instead.",
-    code: "VALIDATION_FAILED"
+    code: "VALIDATION_ERROR"
   },
   {
     pattern: /sandbox.*not initialized/i,
@@ -148,7 +148,7 @@ var QueryError = class extends MemoryJournalMcpError {
 };
 var ValidationError = class extends MemoryJournalMcpError {
   constructor(message, details) {
-    super(message, "VALIDATION_FAILED", "validation" /* VALIDATION */, {
+    super(message, "VALIDATION_ERROR", "validation" /* VALIDATION */, {
       suggestion: "Check input parameters against the tool schema",
       recoverable: false,
       details

package/dist/{chunk-ORV7ZZOE.js → chunk-ZJJD2F5T.js}

@@ -1,5 +1,5 @@
 import { transformAutoReturn } from './chunk-OKOVZ5QE.js';
-import { GitHubIntegration, resolveAuthor, logger, ValidationError, ResourceNotFoundError, assertSafeDirectoryPath, MemoryJournalMcpError, matchSuggestion, ConfigurationError } from './chunk-IWKLHSPU.js';
+import { GitHubIntegration, resolveAuthor, logger, ValidationError, ResourceNotFoundError, assertSafeDirectoryPath, MemoryJournalMcpError, matchSuggestion, ConfigurationError } from './chunk-WXDEVIFL.js';
 import { z, ZodError } from 'zod';
 import { open, stat, mkdir, rename, appendFile, readdir, readFile } from 'fs/promises';
 import { tmpdir } from 'os';
@@ -59,7 +59,7 @@ async function resolveIssueUrl(context, projectNumber, issueNumber, existingUrl)
       ([_, v]) => v.project_number === projectNumber
     );
     if (entry) {
-      const { GitHubIntegration: GitHubIntegration2 } = await import('./github-integration-2TFMXHIJ.js');
+      const { GitHubIntegration: GitHubIntegration2 } = await import('./github-integration-YODGZH3K.js');
       const targetGithub = new GitHubIntegration2(entry[1].path);
       const repoInfo = await targetGithub.getRepoInfo();
       if (repoInfo.owner && repoInfo.repo) {
@@ -208,7 +208,7 @@ var CreateEntrySchemaMcp = z.object({
   share_with_team: z.boolean().optional().default(false)
 });
 var GetEntryByIdSchema = z.object({
-  entry_id: z.number(),
+  entry_id: z.number().int(),
   include_relationships: z.boolean().optional().default(true)
 });
 var GetEntryByIdSchemaMcp = z.object({
@@ -247,6 +247,7 @@ var EntryByIdOutputSchema = z.object({
   error: z.string().optional()
 }).extend(ErrorFieldsMixin.shape);
 var TestSimpleOutputSchema = z.object({
+  success: z.boolean().optional(),
   message: z.string()
 }).extend(ErrorFieldsMixin.shape);
 var TagsListOutputSchema = z.object({
@@ -364,6 +365,7 @@ function getCoreTools(context) {
           }
           const { score: importance, breakdown: importanceBreakdown } = db.calculateImportance(entry_id);
           const result = {
+            success: true,
             entry,
             importance,
             importanceBreakdown
@@ -389,7 +391,7 @@ function getCoreTools(context) {
         try {
           const { limit, is_personal } = GetRecentEntriesSchema.parse(params);
           const entries = db.getRecentEntries(limit, is_personal);
-          return { entries, count: entries.length };
+          return { success: true, entries, count: entries.length };
         } catch (err) {
           return formatHandlerError(err);
         }
@@ -425,7 +427,7 @@ function getCoreTools(context) {
       handler: (params) => {
         try {
           const { message } = TestSimpleSchema.parse(params);
-          return { message: `Test response: ${message}` };
+          return { success: true, message: `Test response: ${message}` };
         } catch (err) {
           return formatHandlerError(err);
         }
@@ -443,7 +445,7 @@ function getCoreTools(context) {
         try {
           const rawTags = db.listTags();
           const tags = rawTags.map((t) => ({ name: t.name, count: t.usageCount }));
-          return { tags, count: tags.length };
+          return { success: true, tags, count: tags.length };
         } catch (err) {
           return formatHandlerError(err);
         }
@@ -470,6 +472,30 @@ function mergeAndDedup(personal, team, limit) {
   }
   return limit !== void 0 ? merged.slice(0, limit) : merged;
 }
+function passMetadataFilters(entry, options, db) {
+  if (options.isPersonal !== void 0 && entry.isPersonal !== options.isPersonal) return false;
+  if (options.projectNumber !== void 0 && entry.projectNumber !== options.projectNumber)
+    return false;
+  if (options.issueNumber !== void 0 && entry.issueNumber !== options.issueNumber) return false;
+  if (options.prNumber !== void 0 && entry.prNumber !== options.prNumber) return false;
+  if (options.prStatus !== void 0 && entry.prStatus !== options.prStatus) return false;
+  if (options.workflowRunId !== void 0 && entry.workflowRunId !== options.workflowRunId)
+    return false;
+  if (options.entryType && entry.entryType !== options.entryType) return false;
+  if (options.startDate) {
+    const entryDate = entry.timestamp.split("T")[0] ?? "";
+    if (entryDate < options.startDate) return false;
+  }
+  if (options.endDate) {
+    const entryDate = entry.timestamp.split("T")[0] ?? "";
+    if (entryDate > options.endDate) return false;
+  }
+  if (options.tags && options.tags.length > 0) {
+    const entryTags = Array.isArray(entry.tags) ? entry.tags : db.getTagsForEntry(entry.id);
+    if (!options.tags.some((t) => entryTags.includes(t))) return false;
+  }
+  return true;
+}
 
 // src/handlers/tools/search/auto.ts
 var QUESTION_PATTERNS = [
@@ -603,7 +629,7 @@ async function hybridSearch(query, db, vectorManager, options) {
   for (const id of sortedIds) {
     const entry = entriesMap.get(id);
     if (!entry) continue;
-    if (options.isPersonal !== void 0 && entry.isPersonal !== options.isPersonal) continue;
+    if (!passMetadataFilters(entry, options, db)) continue;
     entries.push({ ...entry, source: "personal" });
   }
   return { entries, fusionScores };
@@ -738,7 +764,9 @@ function getSearchTools(context) {
               ...formatHandlerError(
                 new ValidationError(
                   "Search requires either a query string or at least one filter",
-                  { suggestion: "Provide a search query or use get_recent_entries instead" }
+                  {
+                    suggestion: "Provide a search query or use get_recent_entries instead"
+                  }
                 )
               ),
               entries: [],
@@ -766,9 +794,10 @@ function getSearchTools(context) {
                 const result = ftsSearch(input.query, db, teamDb, searchOptions);
                 return { ...result, searchMode: "fts (fallback)" };
               }
+              const internalLimit = hasFilters ? Math.min(Math.max(input.limit * 10, 100), 1e3) : input.limit * 2;
               const semanticResults = await vectorManager.search(
                 query,
-                input.limit,
+                internalLimit,
                 0.25
               );
               const entryIds = semanticResults.map((r) => r.entryId);
@@ -776,11 +805,11 @@ function getSearchTools(context) {
               const entries = semanticResults.map((r) => {
                 const entry = entriesMap.get(r.entryId);
                 if (!entry) return null;
-                if (input.is_personal !== void 0 && entry.isPersonal !== input.is_personal)
-                  return null;
+                if (!passMetadataFilters(entry, searchOptions, db)) return null;
                 return { ...entry, source: "personal" };
               }).filter((e) => e !== null);
               return {
+                success: true,
                 entries,
                 count: entries.length,
                 searchMode: isAuto ? "semantic (auto)" : "semantic"
@@ -798,6 +827,7 @@ function getSearchTools(context) {
                 searchOptions
               );
               return {
+                success: true,
                 entries,
                 count: entries.length,
                 searchMode: isAuto ? "hybrid (auto)" : "hybrid"
@@ -808,6 +838,7 @@ function getSearchTools(context) {
               const result = ftsSearch(input.query, db, teamDb, searchOptions);
               return {
                 ...result,
+                success: true,
                 searchMode: isAuto ? "fts (auto)" : "fts"
               };
             }
@@ -868,9 +899,13 @@ function getSearchTools(context) {
               teamEntries.map((e) => ({ ...e, source: "team" })),
               input.limit
             );
-            return { entries: merged, count: merged.length };
+            return { success: true, entries: merged, count: merged.length };
           }
-          return { entries: personalEntries, count: personalEntries.length };
+          return {
+            success: true,
+            entries: personalEntries,
+            count: personalEntries.length
+          };
         } catch (err) {
           return formatHandlerError(err);
         }
@@ -912,17 +947,19 @@ function getSearchTools(context) {
               count: 0
             };
           }
+          const hasFilters = input.is_personal !== void 0 || input.tags !== void 0 || input.entry_type !== void 0 || input.start_date !== void 0 || input.end_date !== void 0;
+          const internalLimit = hasFilters ? Math.min(Math.max(input.limit * 10, 100), 1e3) : input.limit * 2;
           let results;
           if (input.entry_id !== void 0) {
             results = await vectorManager.searchByEntryId(
               input.entry_id,
-              input.limit ?? 10,
+              internalLimit,
               input.similarity_threshold ?? 0.25
             );
           } else {
             results = await vectorManager.search(
               input.query ?? "",
-              input.limit ?? 10,
+              internalLimit,
               input.similarity_threshold ?? 0.25
             );
           }
@@ -931,29 +968,21 @@ function getSearchTools(context) {
           const entries = results.map((r) => {
             const entry = entriesMap.get(r.entryId);
             if (!entry) return null;
-            if (input.is_personal !== void 0 && entry.isPersonal !== input.is_personal)
-              return null;
-            if (input.tags && input.tags.length > 0) {
-              const entryTags = db.getTagsForEntry(entry.id);
-              if (!input.tags.some((t) => entryTags.includes(t))) return null;
-            }
-            if (input.entry_type && entry.entryType !== input.entry_type)
-              return null;
-            if (input.start_date) {
-              const entryDate = entry.timestamp.split("T")[0] ?? "";
-              if (entryDate < input.start_date) return null;
-            }
-            if (input.end_date) {
-              const entryDate = entry.timestamp.split("T")[0] ?? "";
-              if (entryDate > input.end_date) return null;
-            }
+            const filterOptions = {
+              isPersonal: input.is_personal,
+              tags: input.tags,
+              entryType: input.entry_type,
+              startDate: input.start_date,
+              endDate: input.end_date
+            };
+            if (!passMetadataFilters(entry, filterOptions, db)) return null;
             if (input.entry_id !== void 0 && entry.id === input.entry_id)
               return null;
             return {
               ...entry,
               similarity: Math.round(r.score * 100) / 100
             };
-          }).filter((e) => e !== null);
+          }).filter((e) => e !== null).slice(0, input.limit);
           const stats = vectorManager.getStats();
           const isIndexEmpty = stats.itemCount === 0;
           const includeHint = input.hint_on_empty ?? true;
@@ -962,6 +991,7 @@ function getSearchTools(context) {
           const allNoise = entries.length > 0 && bestSimilarity < QUALITY_FLOOR2;
           const hint = isIndexEmpty && includeHint ? "No entries in vector index. Use rebuild_vector_index to index existing entries." : entries.length === 0 && includeHint ? `No entries matched your query above the similarity threshold (${String(input.similarity_threshold ?? 0.25)}). Try lowering similarity_threshold (e.g., 0.15) for broader matches.` : allNoise ? `Results may be noise \u2014 best similarity (${String(bestSimilarity)}) is below quality floor (${String(QUALITY_FLOOR2)}). Try a more specific query or raise similarity_threshold to filter weak matches.` : void 0;
           return {
+            success: true,
             query: input.query,
             ...input.entry_id !== void 0 ? { entryId: input.entry_id } : {},
             entries,
@@ -1120,7 +1150,7 @@ function getAnalyticsTools(context) {
             end_date,
             project_breakdown
           );
-          return { ...stats, groupBy: group_by };
+          return { success: true, ...stats, groupBy: group_by };
         } catch (err) {
           return formatHandlerError(err);
         }
@@ -1162,6 +1192,7 @@ function getAnalyticsTools(context) {
           );
           if (!projectsResult[0] || projectsResult[0].values.length === 0) {
             return {
+              success: true,
               project_count: 0,
               total_entries: 0,
               projects: [],
@@ -1233,6 +1264,7 @@ function getAnalyticsTools(context) {
             )
           }));
           return {
+            success: true,
             project_count: projects.length,
             total_entries: totalEntries,
             projects: projects.map((p) => ({
@@ -1338,6 +1370,24 @@ function getRelationshipTools(context) {
               success: false
             };
           }
+          const fromEntry = db.getEntryById(input.from_entry_id);
+          if (!fromEntry || fromEntry.deletedAt) {
+            return {
+              ...formatHandlerError(
+                new ResourceNotFoundError("Entry", String(input.from_entry_id))
+              ),
+              success: false
+            };
+          }
+          const toEntry = db.getEntryById(input.to_entry_id);
+          if (!toEntry || toEntry.deletedAt) {
+            return {
+              ...formatHandlerError(
+                new ResourceNotFoundError("Entry", String(input.to_entry_id))
+              ),
+              success: false
+            };
+          }
           const existingRelationships = db.getRelationships(input.from_entry_id);
           const existing = existingRelationships.find(
             (r) => r.fromEntryId === input.from_entry_id && r.toEntryId === input.to_entry_id && r.relationshipType === input.relationship_type
@@ -1400,6 +1450,9 @@ function getRelationshipTools(context) {
             const entry = db.getEntryById(input.entry_id);
             if (!entry) {
               return {
+                ...formatHandlerError(
+                  new ResourceNotFoundError("Entry", String(input.entry_id))
+                ),
                 success: false,
                 entry_count: 0,
                 relationship_count: 0,
@@ -2612,7 +2665,8 @@ var CreateGitHubIssueWithEntryOutputSchema = z.object({
   project: z.object({
     projectNumber: z.number(),
     added: z.boolean(),
-    message: z.string(),
+    message: z.string().optional(),
+    error: z.string().optional(),
     initialStatus: z.object({
       status: z.string(),
       set: z.boolean()
@@ -2711,6 +2765,7 @@ var DeleteMilestoneOutputSchema = z.object({
   instruction: z.string().optional()
 }).extend(ErrorFieldsMixin.shape);
 var RepoInsightsOutputSchema = z.object({
+  success: z.boolean().optional(),
   owner: z.string().optional(),
   repo: z.string().optional(),
   section: z.string().optional(),
@@ -4304,6 +4359,7 @@ function getGitHubInsightsTools(context) {
           const repo = resolved.repo;
           const section = input.sections;
           const result = {
+            success: true,
             owner,
             repo,
             section
@@ -4891,6 +4947,10 @@ var TeamSemanticSearchSchema = z.object({
   entry_id: z.number().optional().describe("Find entries related to this entry ID"),
   limit: z.number().max(500).optional().default(10),
   similarity_threshold: z.number().optional().default(0.25),
+  tags: z.array(z.string()).optional(),
+  entry_type: z.enum(ENTRY_TYPES).optional(),
+  start_date: z.string().regex(DATE_FORMAT_REGEX, DATE_FORMAT_MESSAGE).optional(),
+  end_date: z.string().regex(DATE_FORMAT_REGEX, DATE_FORMAT_MESSAGE).optional(),
   hint_on_empty: z.boolean().optional().default(true).describe("Include hint when no results found (default: true)")
 });
 var TeamSemanticSearchSchemaMcp = z.object({
@@ -4898,6 +4958,10 @@ var TeamSemanticSearchSchemaMcp = z.object({
   entry_id: relaxedNumber().optional().describe("Find entries related to this entry ID"),
   limit: relaxedNumber().optional().default(10),
   similarity_threshold: relaxedNumber().optional().default(0.25),
+  tags: z.array(z.string()).optional(),
+  entry_type: z.string().optional(),
+  start_date: z.string().optional(),
+  end_date: z.string().optional(),
   hint_on_empty: z.boolean().optional().default(true).describe("Include hint when no results found (default: true)")
 });
 var TeamAddToVectorIndexSchema = z.object({
@@ -5148,11 +5212,12 @@ function getTeamSearchTools(context) {
             return { ...TEAM_DB_ERROR_RESPONSE };
           }
           const { query, tags, limit } = TeamSearchSchema.parse(params);
+          const searchLimit = tags && tags.length > 0 ? Math.max(limit * 5, 50) : limit;
           let entries;
           if (query) {
-            entries = teamDb.searchEntries(query, { limit });
+            entries = teamDb.searchEntries(query, { limit: searchLimit });
           } else {
-            entries = teamDb.getRecentEntries(limit);
+            entries = teamDb.getRecentEntries(searchLimit);
           }
           if (tags && tags.length > 0) {
             const entryIds = entries.map((e) => e.id);
@@ -5178,6 +5243,7 @@ function getTeamSearchTools(context) {
               });
             }
           }
+          entries = entries.slice(0, limit);
           const authorMap = batchFetchAuthors(
             teamDb,
             entries.map((e) => e.id)
@@ -5186,7 +5252,7 @@ function getTeamSearchTools(context) {
             ...e,
             author: authorMap.get(e.id) ?? null
           }));
-          return { entries: enriched, count: enriched.length };
+          return { success: true, entries: enriched, count: enriched.length };
         } catch (err) {
           return formatHandlerError(err);
         }
@@ -5229,7 +5295,7 @@ function getTeamSearchTools(context) {
             ...e,
             author: authorMap.get(e.id) ?? null
           }));
-          return { entries: enriched, count: enriched.length };
+          return { success: true, entries: enriched, count: enriched.length };
         } catch (err) {
           return formatHandlerError(err);
         }
@@ -5638,6 +5704,17 @@ function getTeamRelationshipTools(context) {
           const { entry_id, tag, depth } = TeamVisualizeRelationshipsSchema.parse(params);
           let entryIds = [];
           if (entry_id !== void 0) {
+            const fromEntry = teamDb.getEntryById(entry_id);
+            if (!fromEntry) {
+              return {
+                success: false,
+                error: `Team entry ${String(entry_id)} not found`,
+                code: "RESOURCE_NOT_FOUND",
+                category: "resource",
+                suggestion: "Verify the team entry ID and try again",
+                recoverable: true
+              };
+            }
             entryIds = [entry_id];
             const visited = new Set(entryIds);
             let frontier = [...entryIds];
@@ -6083,17 +6160,19 @@ function getTeamVectorTools(context) {
               count: 0
             };
           }
+          const hasFilters = input.tags !== void 0 || input.entry_type !== void 0 || input.start_date !== void 0 || input.end_date !== void 0;
+          const internalLimit = hasFilters ? Math.min(Math.max(input.limit * 10, 100), 1e3) : input.limit * 2;
           let results;
           if (input.entry_id !== void 0) {
             results = await teamVectorManager.searchByEntryId(
               input.entry_id,
-              input.limit ?? 10,
+              internalLimit,
               input.similarity_threshold ?? 0.25
             );
           } else {
             results = await teamVectorManager.search(
               input.query ?? "",
-              input.limit ?? 10,
+              internalLimit,
               input.similarity_threshold ?? 0.25
             );
           }
@@ -6105,12 +6184,24 @@ function getTeamVectorTools(context) {
             if (!entry) return null;
             if (input.entry_id !== void 0 && entry.id === input.entry_id)
               return null;
+            if (!passMetadataFilters(
+              entry,
+              {
+                tags: input.tags,
+                entryType: input.entry_type,
+                startDate: input.start_date,
+                endDate: input.end_date
+              },
+              teamDb
+            )) {
+              return null;
+            }
             return {
               ...entry,
               author: authorMap.get(r.entryId) ?? null,
               similarity: Math.round(r.score * 100) / 100
             };
-          }).filter((e) => e !== null);
+          }).filter((e) => e !== null).slice(0, input.limit);
           const stats = teamVectorManager.getStats();
           const isIndexEmpty = stats.itemCount === 0;
           const includeHint = input.hint_on_empty ?? true;
@@ -6118,6 +6209,7 @@ function getTeamVectorTools(context) {
           const allNoise = entries.length > 0 && bestSimilarity < QUALITY_FLOOR;
           const hint = isIndexEmpty && includeHint ? "No entries in team vector index. Use team_rebuild_vector_index to index existing entries." : entries.length === 0 && includeHint ? `No entries matched your query above the similarity threshold (${String(input.similarity_threshold ?? 0.25)}). Try lowering similarity_threshold (e.g., 0.15) for broader matches.` : allNoise ? `Results may be noise \u2014 best similarity (${String(bestSimilarity)}) is below quality floor (${String(QUALITY_FLOOR)}). Try a more specific query or raise similarity_threshold to filter weak matches.` : void 0;
           return {
+            success: true,
             query: input.query,
             ...input.entry_id !== void 0 ? { entryId: input.entry_id } : {},
             entries,

package/dist/cli.js

@@ -1,7 +1,7 @@
-import { VERSION, createServer } from './chunk-GR4T3SRW.js';
-import { DEFAULT_AUDIT_LOG_MAX_SIZE_BYTES } from './chunk-ORV7ZZOE.js';
+import { VERSION, createServer } from './chunk-CHWIPVQN.js';
+import { DEFAULT_AUDIT_LOG_MAX_SIZE_BYTES } from './chunk-ZJJD2F5T.js';
 import './chunk-OKOVZ5QE.js';
-import { logger } from './chunk-IWKLHSPU.js';
+import { logger } from './chunk-WXDEVIFL.js';
 import { Command } from 'commander';
 import * as fs from 'fs';
 

package/dist/github-integration-YODGZH3K.js

@@ -0,0 +1 @@
+export { GitHubIntegration } from './chunk-WXDEVIFL.js';

package/dist/index.js

@@ -1,7 +1,7 @@
-export { VERSION, createServer } from './chunk-GR4T3SRW.js';
-export { META_GROUPS, TOOL_GROUPS, calculateTokenSavings, filterTools, getAllToolNames, getFilterSummary, getToolFilterFromEnv, getToolGroup, isToolEnabled, parseToolFilter } from './chunk-ORV7ZZOE.js';
+export { VERSION, createServer } from './chunk-CHWIPVQN.js';
+export { META_GROUPS, TOOL_GROUPS, calculateTokenSavings, filterTools, getAllToolNames, getFilterSummary, getToolFilterFromEnv, getToolGroup, isToolEnabled, parseToolFilter } from './chunk-ZJJD2F5T.js';
 import './chunk-OKOVZ5QE.js';
-export { logger } from './chunk-IWKLHSPU.js';
+export { logger } from './chunk-WXDEVIFL.js';
 
 // src/types/index.ts
 var DEFAULT_CONFIG = {

package/dist/{tools-CXR2FEB2.js → tools-MNMGDTQI.js}

@@ -1,3 +1,3 @@
-export { callTool, getGlobalAuditLogger, getTools, initializeAuditLogger } from './chunk-ORV7ZZOE.js';
+export { callTool, getGlobalAuditLogger, getTools, initializeAuditLogger } from './chunk-ZJJD2F5T.js';
 import './chunk-OKOVZ5QE.js';
-import './chunk-IWKLHSPU.js';
+import './chunk-WXDEVIFL.js';

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "memory-journal-mcp",
-    "version": "7.2.0",
+    "version": "7.3.0",
     "description": "Project context management for AI-assisted development - Persistent knowledge graphs and intelligent context recall across fragmented AI threads",
     "type": "module",
     "main": "dist/index.js",
@@ -17,8 +17,8 @@
     "scripts": {
         "build": "tsup",
         "dev": "tsc --watch",
-        "lint": "eslint src/",
-        "lint:fix": "eslint src/ --fix",
+        "lint": "eslint src/ tests/",
+        "lint:fix": "eslint src/ tests/ --fix",
         "typecheck": "tsc --noEmit",
         "check": "npm run lint && npm run typecheck",
         "test": "vitest run",
@@ -76,7 +76,7 @@
         "@playwright/test": "^1.59.1",
         "@types/better-sqlite3": "^7.6.13",
         "@types/express": "^5.0.6",
-        "@types/node": "^25.5.2",
+        "@types/node": "^25.6.0",
         "@vitest/coverage-v8": "^4.1.3",
         "eslint": "^10.2.0",
         "globals": "^17.4.0",

package/skills/package.json

@@ -1,6 +1,6 @@
 {
     "name": "neverinfamous-agent-skills",
-    "version": "1.0.6",
+    "version": "1.0.8",
     "description": "Foundational AI agent metacognitive skills and workflows for the Adamic ecosystem.",
     "type": "module",
     "main": "README.md",

package/dist/github-integration-2TFMXHIJ.js

@@ -1 +0,0 @@
-export { GitHubIntegration } from './chunk-IWKLHSPU.js';