memory-journal-mcp 7.2.0 → 7.4.0

package/dist/{chunk-ORV7ZZOE.js → chunk-P5V2VY6N.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) {
@@ -114,7 +114,7 @@ var SIGNIFICANCE_TYPES = [
   "release"
 ];
 var MAX_CONTENT_LENGTH = 5e4;
-var MAX_QUERY_LIMIT = 500;
+var MAX_QUERY_LIMIT = Number("500");
 var DATE_MIN_SENTINEL = "1970-01-01";
 var DATE_MAX_SENTINEL = "2999-12-31";
 var DATE_FORMAT_REGEX = /^\d{4}-\d{2}-\d{2}$/;
@@ -140,7 +140,8 @@ var EntryOutputSchema = z.object({
   workflowRunId: z.number().nullable().optional(),
   workflowName: z.string().nullable().optional(),
   workflowStatus: z.string().nullable().optional(),
-  source: z.enum(["personal", "team"]).optional()
+  source: z.enum(["personal", "team"]).optional(),
+  importanceScore: z.number().optional().describe("Importance score (0.0-1.0), present when sort_by=importance")
 }).extend(ErrorFieldsMixin.shape);
 var EntriesListOutputSchema = z.object({
   entries: z.array(EntryOutputSchema).optional(),
@@ -208,7 +209,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({
@@ -217,11 +218,13 @@ var GetEntryByIdSchemaMcp = z.object({
 });
 var GetRecentEntriesSchema = z.object({
   limit: z.number().min(1).max(MAX_QUERY_LIMIT).optional().default(5),
-  is_personal: z.boolean().optional()
+  is_personal: z.boolean().optional(),
+  sort_by: z.enum(["timestamp", "importance"]).optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var GetRecentEntriesSchemaMcp = z.object({
   limit: relaxedNumber().optional().default(5),
-  is_personal: z.boolean().optional()
+  is_personal: z.boolean().optional(),
+  sort_by: z.string().optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var CreateEntryMinimalSchema = z.object({
   content: z.string().min(1).max(MAX_CONTENT_LENGTH)
@@ -247,6 +250,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 +368,7 @@ function getCoreTools(context) {
           }
           const { score: importance, breakdown: importanceBreakdown } = db.calculateImportance(entry_id);
           const result = {
+            success: true,
             entry,
             importance,
             importanceBreakdown
@@ -387,9 +392,9 @@ function getCoreTools(context) {
       annotations: { readOnlyHint: true, idempotentHint: true, openWorldHint: false },
       handler: (params) => {
         try {
-          const { limit, is_personal } = GetRecentEntriesSchema.parse(params);
-          const entries = db.getRecentEntries(limit, is_personal);
-          return { entries, count: entries.length };
+          const { limit, is_personal, sort_by } = GetRecentEntriesSchema.parse(params);
+          const entries = db.getRecentEntries(limit, is_personal, sort_by);
+          return { success: true, entries, count: entries.length };
         } catch (err) {
           return formatHandlerError(err);
         }
@@ -425,7 +430,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 +448,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);
         }
@@ -457,10 +462,19 @@ var DEDUP_KEY_LENGTH = 200;
 function calcPerDbLimit(limit, hasTeamDb) {
   return hasTeamDb ? Math.min(limit * 2, MAX_QUERY_LIMIT) : limit;
 }
-function mergeAndDedup(personal, team, limit) {
+function mergeAndDedup(personal, team, limit, sortBy = "timestamp") {
   const seen = /* @__PURE__ */ new Set();
   const merged = [];
-  const all = [...personal, ...team].sort((a, b) => b.timestamp.localeCompare(a.timestamp));
+  const all = [...personal, ...team].sort((a, b) => {
+    if (sortBy === "importance") {
+      const scoreA = Number(a["importanceScore"]) || 0;
+      const scoreB = Number(b["importanceScore"]) || 0;
+      if (scoreA !== scoreB) {
+        return scoreB - scoreA;
+      }
+    }
+    return b.timestamp.localeCompare(a.timestamp);
+  });
   for (const entry of all) {
     const key = entry.content.slice(0, DEDUP_KEY_LENGTH);
     if (!seen.has(key)) {
@@ -470,6 +484,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 = [
@@ -510,7 +548,7 @@ function ftsSearch(query, db, teamDb, options) {
   const perDbLimit = calcPerDbLimit(options.limit, !!teamDb);
   let personalEntries;
   if (!query && !hasFilters) {
-    personalEntries = db.getRecentEntries(perDbLimit, options.isPersonal);
+    personalEntries = db.getRecentEntries(perDbLimit, options.isPersonal, options.sortBy);
   } else {
     personalEntries = db.searchEntries(query || "", {
       limit: perDbLimit,
@@ -523,13 +561,14 @@ function ftsSearch(query, db, teamDb, options) {
       tags: options.tags,
       entryType: options.entryType,
       startDate: options.startDate,
-      endDate: options.endDate
+      endDate: options.endDate,
+      sortBy: options.sortBy
     });
   }
   if (teamDb && options.isPersonal !== true) {
     let teamEntries;
     if (!query && !hasFilters) {
-      teamEntries = teamDb.getRecentEntries(perDbLimit);
+      teamEntries = teamDb.getRecentEntries(perDbLimit, void 0, options.sortBy);
     } else {
       teamEntries = teamDb.searchEntries(query || "", {
         limit: perDbLimit,
@@ -541,13 +580,15 @@ function ftsSearch(query, db, teamDb, options) {
         tags: options.tags,
         entryType: options.entryType,
         startDate: options.startDate,
-        endDate: options.endDate
+        endDate: options.endDate,
+        sortBy: options.sortBy
       });
     }
     const merged = mergeAndDedup(
       personalEntries.map((e) => ({ ...e, source: "personal" })),
       teamEntries.map((e) => ({ ...e, source: "team" })),
-      options.limit
+      options.limit,
+      options.sortBy
     );
     return { entries: merged, count: merged.length };
   }
@@ -603,7 +644,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 };
@@ -625,7 +666,8 @@ var SearchEntriesSchema = z.object({
   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()
+  end_date: z.string().regex(DATE_FORMAT_REGEX, DATE_FORMAT_MESSAGE).optional(),
+  sort_by: z.enum(["timestamp", "importance"]).optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var SearchEntriesSchemaMcp = z.object({
   query: z.string().optional(),
@@ -642,7 +684,8 @@ var SearchEntriesSchemaMcp = z.object({
   tags: z.array(z.string()).optional(),
   entry_type: z.string().optional(),
   start_date: z.string().optional(),
-  end_date: z.string().optional()
+  end_date: z.string().optional(),
+  sort_by: z.string().optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var SearchByDateRangeSchema = z.object({
   start_date: z.string().regex(DATE_FORMAT_REGEX, DATE_FORMAT_MESSAGE),
@@ -654,7 +697,8 @@ var SearchByDateRangeSchema = z.object({
   issue_number: z.number().optional(),
   pr_number: z.number().optional(),
   workflow_run_id: z.number().optional(),
-  limit: z.number().max(MAX_QUERY_LIMIT).optional().default(500)
+  limit: z.number().max(MAX_QUERY_LIMIT).optional().default(500),
+  sort_by: z.enum(["timestamp", "importance"]).optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var SearchByDateRangeSchemaMcp = z.object({
   start_date: z.string(),
@@ -666,7 +710,8 @@ var SearchByDateRangeSchemaMcp = z.object({
   issue_number: relaxedNumber().optional(),
   pr_number: relaxedNumber().optional(),
   workflow_run_id: relaxedNumber().optional(),
-  limit: relaxedNumber().optional().default(500)
+  limit: relaxedNumber().optional().default(500),
+  sort_by: z.string().optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var SemanticSearchSchema = z.object({
   query: z.string().optional(),
@@ -738,7 +783,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: [],
@@ -758,7 +805,8 @@ function getSearchTools(context) {
             tags: input.tags,
             entryType: input.entry_type,
             startDate: input.start_date,
-            endDate: input.end_date
+            endDate: input.end_date,
+            sortBy: input.sort_by
           };
           switch (effectiveMode) {
             case "semantic": {
@@ -766,9 +814,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 +825,31 @@ 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)
+                if (!passMetadataFilters(
+                  entry,
+                  searchOptions,
+                  db
+                ))
                   return null;
                 return { ...entry, source: "personal" };
-              }).filter((e) => e !== null);
+              }).filter((e) => e !== null).slice(0, input.limit);
+              if (input.sort_by === "importance") {
+                const scored = entries.map((e) => {
+                  const { score } = db.calculateImportance(e.id);
+                  return { ...e, importanceScore: Math.round(score * 100) / 100 };
+                });
+                scored.sort(
+                  (a, b) => (b.importanceScore ?? 0) - (a.importanceScore ?? 0)
+                );
+                return {
+                  success: true,
+                  entries: scored,
+                  count: scored.length,
+                  searchMode: isAuto ? "semantic (auto)" : "semantic"
+                };
+              }
               return {
+                success: true,
                 entries,
                 count: entries.length,
                 searchMode: isAuto ? "semantic (auto)" : "semantic"
@@ -797,7 +866,24 @@ function getSearchTools(context) {
                 vectorManager,
                 searchOptions
               );
+              if (input.sort_by === "importance") {
+                const scored = entries.map((e) => {
+                  const entryId = e["id"];
+                  const { score } = db.calculateImportance(entryId);
+                  return { ...e, importanceScore: Math.round(score * 100) / 100 };
+                });
+                scored.sort(
+                  (a, b) => (b.importanceScore ?? 0) - (a.importanceScore ?? 0)
+                );
+                return {
+                  success: true,
+                  entries: scored,
+                  count: scored.length,
+                  searchMode: isAuto ? "hybrid (auto)" : "hybrid"
+                };
+              }
               return {
+                success: true,
                 entries,
                 count: entries.length,
                 searchMode: isAuto ? "hybrid (auto)" : "hybrid"
@@ -808,6 +894,7 @@ function getSearchTools(context) {
               const result = ftsSearch(input.query, db, teamDb, searchOptions);
               return {
                 ...result,
+                success: true,
                 searchMode: isAuto ? "fts (auto)" : "fts"
               };
             }
@@ -847,7 +934,8 @@ function getSearchTools(context) {
             issueNumber: input.issue_number,
             prNumber: input.pr_number,
             workflowRunId: input.workflow_run_id,
-            limit: perDbLimit
+            limit: perDbLimit,
+            sortBy: input.sort_by
           });
           if (teamDb && input.is_personal !== true) {
             const teamEntries = teamDb.searchByDateRange(
@@ -860,17 +948,23 @@ function getSearchTools(context) {
                 issueNumber: input.issue_number,
                 prNumber: input.pr_number,
                 workflowRunId: input.workflow_run_id,
-                limit: perDbLimit
+                limit: perDbLimit,
+                sortBy: input.sort_by
               }
             );
             const merged = mergeAndDedup(
               personalEntries.map((e) => ({ ...e, source: "personal" })),
               teamEntries.map((e) => ({ ...e, source: "team" })),
-              input.limit
+              input.limit,
+              input.sort_by
             );
-            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 +1006,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 +1027,22 @@ 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)
+            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.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;
-            }
             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 +1051,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 +1210,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 +1252,7 @@ function getAnalyticsTools(context) {
           );
           if (!projectsResult[0] || projectsResult[0].values.length === 0) {
             return {
+              success: true,
               project_count: 0,
               total_entries: 0,
               projects: [],
@@ -1233,6 +1324,7 @@ function getAnalyticsTools(context) {
             )
           }));
           return {
+            success: true,
             project_count: projects.length,
             total_entries: totalEntries,
             projects: projects.map((p) => ({
@@ -1338,6 +1430,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 +1510,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 +2725,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()
@@ -2643,7 +2757,8 @@ var CloseGitHubIssueWithEntryOutputSchema = z.object({
   }).optional(),
   kanban: z.object({
     moved: z.boolean(),
-    projectNumber: z.number(),
+    projectNumber: z.number().optional(),
+    error: z.string().optional(),
     message: z.string().optional()
   }).optional(),
   message: z.string().optional(),
@@ -2711,6 +2826,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 +4420,7 @@ function getGitHubInsightsTools(context) {
           const repo = resolved.repo;
           const section = input.sections;
           const result = {
+            success: true,
             owner,
             repo,
             section
@@ -4664,20 +4781,24 @@ var TeamCreateEntrySchemaMcp = z.object({
   author: z.string().optional()
 });
 var TeamGetRecentSchema = z.object({
-  limit: z.number().min(1).max(500).optional().default(10)
+  limit: z.number().min(1).max(500).optional().default(10),
+  sort_by: z.enum(["timestamp", "importance"]).optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var TeamGetRecentSchemaMcp = z.object({
-  limit: relaxedNumber().optional().default(10)
+  limit: relaxedNumber().optional().default(10),
+  sort_by: z.string().optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var TeamSearchSchema = z.object({
   query: z.string().optional(),
   tags: z.array(z.string()).optional(),
-  limit: z.number().max(500).optional().default(10)
+  limit: z.number().max(500).optional().default(10),
+  sort_by: z.enum(["timestamp", "importance"]).optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var TeamSearchSchemaMcp = z.object({
   query: z.string().optional(),
   tags: z.array(z.string()).optional(),
-  limit: relaxedNumber().optional().default(10)
+  limit: relaxedNumber().optional().default(10),
+  sort_by: z.string().optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var TeamGetEntryByIdSchema = z.object({
   entry_id: z.number(),
@@ -4692,14 +4813,16 @@ var TeamSearchByDateRangeSchema = z.object({
   end_date: z.string().regex(DATE_FORMAT_REGEX, DATE_FORMAT_MESSAGE),
   entry_type: z.enum(ENTRY_TYPES).optional(),
   tags: z.array(z.string()).optional(),
-  limit: z.number().max(500).optional().default(50)
+  limit: z.number().max(500).optional().default(50),
+  sort_by: z.enum(["timestamp", "importance"]).optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var TeamSearchByDateRangeSchemaMcp = z.object({
   start_date: z.string().optional().describe("Start date (YYYY-MM-DD)"),
   end_date: z.string().optional().describe("End date (YYYY-MM-DD)"),
   entry_type: z.string().optional(),
   tags: z.array(z.string()).optional(),
-  limit: relaxedNumber().optional().default(50)
+  limit: relaxedNumber().optional().default(50),
+  sort_by: z.string().optional().default("timestamp").describe("Sort results by timestamp (default) or importance score")
 });
 var TeamUpdateEntrySchema = z.object({
   entry_id: z.number(),
@@ -4891,6 +5014,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 +5025,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({
@@ -4978,6 +5109,40 @@ var TeamCrossProjectInsightsOutputSchema = z.object({
   success: z.boolean().optional(),
   error: z.string().optional()
 }).extend(ErrorFieldsMixin.shape);
+var TeamCollaborationMatrixSchema = z.object({
+  period: z.enum(["week", "month", "quarter"]).optional().default("month").describe("Time granularity for the activity heatmap"),
+  limit: z.number().max(100).optional().default(20).describe("Max authors to include")
+});
+var TeamCollaborationMatrixSchemaMcp = z.object({
+  period: z.string().optional().default("month"),
+  limit: relaxedNumber().optional().default(20)
+});
+var TeamCollaborationMatrixOutputSchema = z.object({
+  success: z.boolean().optional(),
+  totalAuthors: z.number().optional(),
+  totalEntries: z.number().optional(),
+  authorActivity: z.array(
+    z.object({
+      author: z.string(),
+      period: z.string(),
+      entryCount: z.number()
+    })
+  ).optional(),
+  crossAuthorLinks: z.array(
+    z.object({
+      fromAuthor: z.string(),
+      toAuthor: z.string(),
+      linkCount: z.number()
+    })
+  ).optional(),
+  impactFactor: z.array(
+    z.object({
+      author: z.string(),
+      inboundLinks: z.number()
+    })
+  ).optional(),
+  error: z.string().optional()
+}).extend(ErrorFieldsMixin.shape);
 
 // src/handlers/tools/team/core-tools.ts
 function getTeamCoreTools(context) {
@@ -5090,8 +5255,8 @@ function getTeamCoreTools(context) {
           if (!teamDb) {
             return { ...TEAM_DB_ERROR_RESPONSE };
           }
-          const { limit } = TeamGetRecentSchema.parse(params);
-          const entries = teamDb.getRecentEntries(limit);
+          const { limit, sort_by } = TeamGetRecentSchema.parse(params);
+          const entries = teamDb.getRecentEntries(limit, void 0, sort_by);
           const authorMap = batchFetchAuthors(
             teamDb,
             entries.map((e) => e.id)
@@ -5147,12 +5312,16 @@ function getTeamSearchTools(context) {
           if (!teamDb) {
             return { ...TEAM_DB_ERROR_RESPONSE };
           }
-          const { query, tags, limit } = TeamSearchSchema.parse(params);
+          const { query, tags, limit, sort_by } = TeamSearchSchema.parse(params);
+          const searchLimit = tags && tags.length > 0 ? Math.min(Math.max(limit * 5, 50), 1e3) : limit;
           let entries;
           if (query) {
-            entries = teamDb.searchEntries(query, { limit });
+            entries = teamDb.searchEntries(query, {
+              limit: searchLimit,
+              sortBy: sort_by
+            });
           } else {
-            entries = teamDb.getRecentEntries(limit);
+            entries = teamDb.getRecentEntries(searchLimit, void 0, sort_by);
           }
           if (tags && tags.length > 0) {
             const entryIds = entries.map((e) => e.id);
@@ -5178,6 +5347,7 @@ function getTeamSearchTools(context) {
               });
             }
           }
+          entries = entries.slice(0, limit);
           const authorMap = batchFetchAuthors(
             teamDb,
             entries.map((e) => e.id)
@@ -5186,7 +5356,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);
         }
@@ -5205,7 +5375,7 @@ function getTeamSearchTools(context) {
           if (!teamDb) {
             return { ...TEAM_DB_ERROR_RESPONSE };
           }
-          const { start_date, end_date, entry_type, tags, limit } = TeamSearchByDateRangeSchema.parse(params);
+          const { start_date, end_date, entry_type, tags, limit, sort_by } = TeamSearchByDateRangeSchema.parse(params);
           if (start_date > end_date) {
             return {
               success: false,
@@ -5219,7 +5389,8 @@ function getTeamSearchTools(context) {
           const entries = teamDb.searchByDateRange(start_date, end_date, {
             entryType: entry_type,
             tags,
-            limit
+            limit,
+            sortBy: sort_by
           });
           const authorMap = batchFetchAuthors(
             teamDb,
@@ -5229,7 +5400,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);
         }
@@ -5450,6 +5621,7 @@ function getTeamAnalyticsTools(context) {
           );
           if (!projectsResult[0] || projectsResult[0].values.length === 0) {
             return {
+              success: true,
               project_count: 0,
               total_entries: 0,
               projects: [],
@@ -5526,6 +5698,7 @@ function getTeamAnalyticsTools(context) {
             )
           }));
           return {
+            success: true,
             project_count: projects.length,
             total_entries: totalEntries,
             projects: projects.map((p) => ({
@@ -5540,6 +5713,97 @@ function getTeamAnalyticsTools(context) {
           return formatHandlerError(err);
         }
       }
+    },
+    {
+      name: "team_get_collaboration_matrix",
+      title: "Team Collaboration Matrix",
+      description: "Analyze cross-author collaboration: activity heatmap per period, cross-linking patterns between authors, and impact factor (inbound links). Requires TEAM_DB_PATH.",
+      group: "team",
+      inputSchema: TeamCollaborationMatrixSchemaMcp,
+      outputSchema: TeamCollaborationMatrixOutputSchema,
+      annotations: { readOnlyHint: true, idempotentHint: true, openWorldHint: false },
+      handler: (params) => {
+        try {
+          if (!teamDb) {
+            return { ...TEAM_DB_ERROR_RESPONSE };
+          }
+          const { period, limit } = TeamCollaborationMatrixSchema.parse(params);
+          const dateFormat = period === "week" ? "%Y-W%W" : period === "quarter" ? "%Y-Q" : "%Y-%m";
+          const activityResult = teamDb.executeRawQuery(
+            `SELECT
+                            COALESCE(author, 'unknown') AS author,
+                            strftime('${dateFormat}', timestamp) AS period,
+                            COUNT(*) AS entry_count
+                        FROM memory_journal
+                        WHERE deleted_at IS NULL
+                        GROUP BY author, period
+                        ORDER BY period DESC, entry_count DESC
+                        LIMIT ?`,
+            [limit * 10]
+            // Up to 10 periods per author
+          );
+          const authorActivity = activityResult[0]?.values.map((row) => ({
+            author: row[0],
+            period: row[1],
+            entryCount: row[2]
+          })) ?? [];
+          const crossLinkResult = teamDb.executeRawQuery(
+            `SELECT
+                            COALESCE(m1.author, 'unknown') AS from_author,
+                            COALESCE(m2.author, 'unknown') AS to_author,
+                            COUNT(*) AS link_count
+                        FROM relationships r
+                        JOIN memory_journal m1 ON r.from_entry_id = m1.id
+                        JOIN memory_journal m2 ON r.to_entry_id = m2.id
+                        WHERE m1.deleted_at IS NULL AND m2.deleted_at IS NULL
+                            AND COALESCE(m1.author, 'unknown') != COALESCE(m2.author, 'unknown')
+                        GROUP BY from_author, to_author
+                        ORDER BY link_count DESC
+                        LIMIT ?`,
+            [limit]
+          );
+          const crossAuthorLinks = crossLinkResult[0]?.values.map((row) => ({
+            fromAuthor: row[0],
+            toAuthor: row[1],
+            linkCount: row[2]
+          })) ?? [];
+          const impactResult = teamDb.executeRawQuery(
+            `SELECT
+                            COALESCE(m2.author, 'unknown') AS author,
+                            COUNT(*) AS inbound_links
+                        FROM relationships r
+                        JOIN memory_journal m2 ON r.to_entry_id = m2.id
+                        WHERE m2.deleted_at IS NULL
+                        GROUP BY author
+                        ORDER BY inbound_links DESC
+                        LIMIT ?`,
+            [limit]
+          );
+          const impactFactor = impactResult[0]?.values.map((row) => ({
+            author: row[0],
+            inboundLinks: row[1]
+          })) ?? [];
+          const totalsResult = teamDb.executeRawQuery(
+            `SELECT
+                            COUNT(DISTINCT COALESCE(author, 'unknown')) AS total_authors,
+                            COUNT(*) AS total_entries
+                        FROM memory_journal
+                        WHERE deleted_at IS NULL`
+          );
+          const totalAuthors = totalsResult[0]?.values[0]?.[0] ?? 0;
+          const totalEntries = totalsResult[0]?.values[0]?.[1] ?? 0;
+          return {
+            success: true,
+            totalAuthors,
+            totalEntries,
+            authorActivity,
+            crossAuthorLinks,
+            impactFactor
+          };
+        } catch (err) {
+          return formatHandlerError(err);
+        }
+      }
     }
   ];
 }
@@ -5638,6 +5902,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 +6358,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 +6382,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 +6407,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,
@@ -7682,7 +7972,8 @@ var TOOL_GROUPS = {
     "team_get_vector_index_stats",
     "team_rebuild_vector_index",
     "team_add_to_vector_index",
-    "team_get_cross_project_insights"
+    "team_get_cross_project_insights",
+    "team_get_collaboration_matrix"
   ],
   codemode: ["mj_execute_code"]
 };