memory-journal-mcp 7.1.0 → 7.3.0

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

@@ -1,5 +1,5 @@
 import { transformAutoReturn } from './chunk-OKOVZ5QE.js';
-import { GitHubIntegration, resolveAuthor, logger, assertSafeDirectoryPath, MemoryJournalMcpError, matchSuggestion, ConfigurationError } from './chunk-37BQOJDZ.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-FOJ4U6I5.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,6 +114,7 @@ var SIGNIFICANCE_TYPES = [
   "release"
 ];
 var MAX_CONTENT_LENGTH = 5e4;
+var MAX_QUERY_LIMIT = 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}$/;
@@ -207,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({
@@ -215,7 +216,7 @@ var GetEntryByIdSchemaMcp = z.object({
   include_relationships: z.boolean().optional().default(true)
 });
 var GetRecentEntriesSchema = z.object({
-  limit: z.number().max(500).optional().default(5),
+  limit: z.number().min(1).max(MAX_QUERY_LIMIT).optional().default(5),
   is_personal: z.boolean().optional()
 });
 var GetRecentEntriesSchemaMcp = z.object({
@@ -246,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({
@@ -363,6 +365,7 @@ function getCoreTools(context) {
           }
           const { score: importance, breakdown: importanceBreakdown } = db.calculateImportance(entry_id);
           const result = {
+            success: true,
             entry,
             importance,
             importanceBreakdown
@@ -388,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);
         }
@@ -424,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);
         }
@@ -442,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);
         }
@@ -452,7 +455,6 @@ function getCoreTools(context) {
 }
 
 // src/handlers/tools/search/helpers.ts
-var MAX_QUERY_LIMIT = 500;
 var DEDUP_KEY_LENGTH = 200;
 function calcPerDbLimit(limit, hasTeamDb) {
   return hasTeamDb ? Math.min(limit * 2, MAX_QUERY_LIMIT) : limit;
@@ -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 };
@@ -730,11 +756,25 @@ function getSearchTools(context) {
       handler: async (params) => {
         try {
           const input = SearchEntriesSchema.parse(params);
-          const query = input.query || "";
+          const query = input.query?.trim() || "";
           const mode = input.mode;
-          const { resolvedMode, isAuto } = resolveSearchMode(mode, query);
           const hasFilters = input.project_number !== void 0 || input.issue_number !== void 0 || input.pr_number !== void 0 || input.pr_status !== void 0 || input.workflow_run_id !== void 0 || 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 effectiveMode = !query && !hasFilters ? "fts" : resolvedMode;
+          if (!query && !hasFilters) {
+            return {
+              ...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"
+                  }
+                )
+              ),
+              entries: [],
+              count: 0
+            };
+          }
+          const { resolvedMode, isAuto } = resolveSearchMode(mode, query);
+          const effectiveMode = resolvedMode;
           const searchOptions = {
             limit: input.limit,
             isPersonal: input.is_personal,
@@ -754,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);
@@ -764,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"
@@ -786,6 +827,7 @@ function getSearchTools(context) {
                 searchOptions
               );
               return {
+                success: true,
                 entries,
                 count: entries.length,
                 searchMode: isAuto ? "hybrid (auto)" : "hybrid"
@@ -796,6 +838,7 @@ function getSearchTools(context) {
               const result = ftsSearch(input.query, db, teamDb, searchOptions);
               return {
                 ...result,
+                success: true,
                 searchMode: isAuto ? "fts (auto)" : "fts"
               };
             }
@@ -856,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);
         }
@@ -900,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
             );
           }
@@ -919,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;
@@ -950,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,
@@ -1108,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);
         }
@@ -1150,6 +1192,7 @@ function getAnalyticsTools(context) {
           );
           if (!projectsResult[0] || projectsResult[0].values.length === 0) {
             return {
+              success: true,
               project_count: 0,
               total_entries: 0,
               projects: [],
@@ -1221,6 +1264,7 @@ function getAnalyticsTools(context) {
             )
           }));
           return {
+            success: true,
             project_count: projects.length,
             total_entries: totalEntries,
             projects: projects.map((p) => ({
@@ -1320,10 +1364,28 @@ function getRelationshipTools(context) {
           const input = LinkEntriesSchema.parse(params);
           if (input.from_entry_id === input.to_entry_id) {
             return {
-              success: false,
-              error: "Cannot link an entry to itself",
-              code: "VALIDATION_ERROR",
-              category: "validation"
+              ...formatHandlerError(
+                new ValidationError("Cannot link an entry to itself")
+              ),
+              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);
@@ -1358,12 +1420,13 @@ function getRelationshipTools(context) {
             const isFkError = errMsg.includes("FOREIGN KEY constraint failed");
             if (isFkError) {
               return {
-                success: false,
-                error: `One or both entries not found (from: ${String(input.from_entry_id)}, to: ${String(input.to_entry_id)})`,
-                code: "RESOURCE_NOT_FOUND",
-                category: "resource",
-                suggestion: "Verify both entry IDs exist before linking",
-                recoverable: true
+                ...formatHandlerError(
+                  new ResourceNotFoundError(
+                    "Entry",
+                    `from: ${String(input.from_entry_id)}, to: ${String(input.to_entry_id)}`
+                  )
+                ),
+                success: false
               };
             }
           }
@@ -1387,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,
@@ -2098,11 +2164,7 @@ ${e.content}
             tags: e.tags,
             significance: e.significanceType ?? void 0
           }));
-          const result = await exportEntriesToMarkdown(
-            exportable,
-            input.output_dir,
-            db
-          );
+          const result = await exportEntriesToMarkdown(exportable, input.output_dir, db);
           await sendProgress(progress, 3, 3, "Export complete");
           return result;
         } catch (err) {
@@ -2439,6 +2501,8 @@ var GitHubIssueOutputSchema = z.object({
 }).extend(ErrorFieldsMixin.shape);
 var GitHubIssueDetailsOutputSchema = GitHubIssueOutputSchema.extend({
   body: z.string().nullable(),
+  bodyTruncated: z.boolean().optional(),
+  bodyFullLength: z.number().optional(),
   labels: z.array(z.string()),
   assignees: z.array(z.string()),
   createdAt: z.string(),
@@ -2459,6 +2523,14 @@ var GitHubIssuesListOutputSchema = z.object({
 }).extend(ErrorFieldsMixin.shape);
 var GitHubIssueResultOutputSchema = z.object({
   issue: GitHubIssueDetailsOutputSchema.optional(),
+  comments: z.array(
+    z.object({
+      author: z.string(),
+      body: z.string(),
+      createdAt: z.string()
+    })
+  ).optional(),
+  commentCount: z.number().optional(),
   owner: z.string().optional(),
   repo: z.string().optional(),
   detectedOwner: z.string().nullable().optional(),
@@ -2475,6 +2547,8 @@ var GitHubPullRequestOutputSchema = z.object({
 }).extend(ErrorFieldsMixin.shape);
 var GitHubPRDetailsOutputSchema = GitHubPullRequestOutputSchema.extend({
   body: z.string().nullable(),
+  bodyTruncated: z.boolean().optional(),
+  bodyFullLength: z.number().optional(),
   draft: z.boolean(),
   headBranch: z.string(),
   baseBranch: z.string(),
@@ -2539,7 +2613,9 @@ var StatusOptionOutputSchema = z.object({
 var KanbanColumnOutputSchema = z.object({
   status: z.string(),
   statusOptionId: z.string(),
-  items: z.array(KanbanItemOutputSchema)
+  items: z.array(KanbanItemOutputSchema).optional(),
+  itemCount: z.number().optional(),
+  truncated: z.boolean().optional()
 });
 var KanbanBoardOutputSchema = z.object({
   projectId: z.string().optional(),
@@ -2549,6 +2625,14 @@ var KanbanBoardOutputSchema = z.object({
   statusOptions: z.array(StatusOptionOutputSchema).optional(),
   columns: z.array(KanbanColumnOutputSchema).optional(),
   totalItems: z.number().optional(),
+  itemDirectory: z.array(
+    z.object({
+      id: z.string(),
+      title: z.string(),
+      status: z.string().nullable()
+    })
+  ).optional(),
+  summaryOnly: z.boolean().optional(),
   owner: z.string().optional(),
   detectedOwner: z.string().nullable().optional(),
   detectedRepo: z.string().nullable().optional(),
@@ -2558,16 +2642,19 @@ var KanbanBoardOutputSchema = z.object({
   instruction: z.string().optional()
 }).extend(ErrorFieldsMixin.shape);
 var MoveKanbanItemOutputSchema = z.object({
-  success: z.boolean().optional(),
   itemId: z.string().optional(),
   newStatus: z.string().optional(),
   projectNumber: z.number().optional(),
-  message: z.string().optional(),
-  error: z.string().optional(),
-  requiresUserInput: z.boolean().optional(),
-  hint: z.string().optional(),
   availableStatuses: z.array(z.string()).optional()
 }).extend(ErrorFieldsMixin.shape);
+var AddKanbanItemOutputSchema = z.object({
+  itemId: z.string().optional(),
+  projectNumber: z.number().optional()
+}).extend(ErrorFieldsMixin.shape);
+var DeleteKanbanItemOutputSchema = z.object({
+  itemId: z.string().optional(),
+  projectNumber: z.number().optional()
+}).extend(ErrorFieldsMixin.shape);
 var CreateGitHubIssueWithEntryOutputSchema = z.object({
   success: z.boolean().optional(),
   issue: z.object({
@@ -2578,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()
@@ -2677,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(),
@@ -2858,7 +2947,7 @@ function getGitHubReadTools(context) {
             owner: z.string().optional(),
             repo: z.string().optional(),
             state: z.enum(["open", "closed", "all"]).optional().default("open"),
-            limit: z.number().optional().default(20)
+            limit: z.number().max(MAX_QUERY_LIMIT).optional().default(20)
           }).parse(params);
           const resolved = await resolveOwnerRepo(
             context,
@@ -2908,7 +2997,7 @@ function getGitHubReadTools(context) {
             owner: z.string().optional(),
             repo: z.string().optional(),
             state: z.enum(["open", "closed", "all"]).optional().default("open"),
-            limit: z.number().optional().default(20)
+            limit: z.number().max(MAX_QUERY_LIMIT).optional().default(20)
           }).parse(params);
           const resolved = await resolveOwnerRepo(
             context,
@@ -2943,7 +3032,11 @@ function getGitHubReadTools(context) {
       inputSchema: z.object({
         issue_number: z.number(),
         owner: z.string().optional().describe("LEAVE EMPTY to auto-detect from git"),
-        repo: z.string().optional().describe("LEAVE EMPTY to auto-detect from git")
+        repo: z.string().optional().describe("LEAVE EMPTY to auto-detect from git"),
+        truncate_body: relaxedNumber().optional().default(800).describe(
+          "Max characters for body text (0 = full body, default 800). Reduces token usage."
+        ),
+        include_comments: z.boolean().optional().default(false).describe("Include issue comments (default false). Each comment adds tokens.")
       }),
       outputSchema: GitHubIssueResultOutputSchema,
       annotations: { readOnlyHint: true, idempotentHint: true, openWorldHint: true },
@@ -2952,7 +3045,9 @@ function getGitHubReadTools(context) {
           const input = z.object({
             issue_number: z.number(),
             owner: z.string().optional(),
-            repo: z.string().optional()
+            repo: z.string().optional(),
+            truncate_body: z.number().optional().default(800),
+            include_comments: z.boolean().optional().default(false)
           }).parse(params);
           const resolved = await resolveOwnerRepo(context, input, "is this issue from");
           if ("error" in resolved) return resolved.response;
@@ -2975,8 +3070,30 @@ function getGitHubReadTools(context) {
               detectedRepo: resolved.detectedRepo
             };
           }
+          const truncateBody = input.truncate_body;
+          let bodyTruncated = false;
+          let bodyFullLength;
+          if (truncateBody > 0 && issue.body && issue.body.length > truncateBody) {
+            bodyFullLength = issue.body.length;
+            const remaining = issue.body.length - truncateBody;
+            issue.body = issue.body.slice(0, truncateBody) + `
+[Truncated. Re-run with truncate_body: 0 to view remaining ${String(remaining)} chars]`;
+            bodyTruncated = true;
+          }
+          let comments;
+          if (input.include_comments) {
+            comments = await resolved.github.getIssueComments(
+              resolved.owner,
+              resolved.repo,
+              input.issue_number
+            );
+          }
           return {
-            issue,
+            issue: {
+              ...issue,
+              ...bodyTruncated ? { bodyTruncated: true, bodyFullLength } : {}
+            },
+            ...comments ? { comments, commentCount: comments.length } : {},
             owner: resolved.owner,
             repo: resolved.repo,
             detectedOwner: resolved.detectedOwner,
@@ -2995,7 +3112,10 @@ function getGitHubReadTools(context) {
       inputSchema: z.object({
         pr_number: z.number(),
         owner: z.string().optional().describe("LEAVE EMPTY to auto-detect from git"),
-        repo: z.string().optional().describe("LEAVE EMPTY to auto-detect from git")
+        repo: z.string().optional().describe("LEAVE EMPTY to auto-detect from git"),
+        truncate_body: relaxedNumber().optional().default(800).describe(
+          "Max characters for body text (0 = full body, default 800). Reduces token usage."
+        )
       }),
       outputSchema: GitHubPRResultOutputSchema,
       annotations: { readOnlyHint: true, idempotentHint: true, openWorldHint: true },
@@ -3004,7 +3124,8 @@ function getGitHubReadTools(context) {
           const input = z.object({
             pr_number: z.number(),
             owner: z.string().optional(),
-            repo: z.string().optional()
+            repo: z.string().optional(),
+            truncate_body: z.number().optional().default(800)
           }).parse(params);
           const resolved = await resolveOwnerRepo(context, input, "is this PR from");
           if ("error" in resolved) return resolved.response;
@@ -3027,8 +3148,21 @@ function getGitHubReadTools(context) {
               detectedRepo: resolved.detectedRepo
             };
           }
+          const truncateBody = input.truncate_body;
+          let bodyTruncated = false;
+          let bodyFullLength;
+          if (truncateBody > 0 && pullRequest.body && pullRequest.body.length > truncateBody) {
+            bodyFullLength = pullRequest.body.length;
+            const remaining = pullRequest.body.length - truncateBody;
+            pullRequest.body = pullRequest.body.slice(0, truncateBody) + `
+[Truncated. Re-run with truncate_body: 0 to view remaining ${String(remaining)} chars]`;
+            bodyTruncated = true;
+          }
           return {
-            pullRequest,
+            pullRequest: {
+              ...pullRequest,
+              ...bodyTruncated ? { bodyTruncated: true, bodyFullLength } : {}
+            },
             owner: resolved.owner,
             repo: resolved.repo,
             detectedOwner: resolved.detectedOwner,
@@ -3096,7 +3230,13 @@ function getKanbanTools(context) {
       inputSchema: z.object({
         project_number: z.number().optional().describe("GitHub Project number (optional if repo is registered)"),
         owner: z.string().optional().describe("Project owner - LEAVE EMPTY to auto-detect"),
-        repo: z.string().optional().describe("Repository name - LEAVE EMPTY to auto-detect")
+        repo: z.string().optional().describe("Repository name - LEAVE EMPTY to auto-detect"),
+        summary_only: z.boolean().optional().default(false).describe(
+          "Return column summaries only (name + itemCount), omitting individual items. Saves ~80% tokens."
+        ),
+        item_limit: relaxedNumber().optional().default(25).describe(
+          "Maximum items per column (default 25, max 100). Set to 0 for summary_only behavior."
+        )
       }),
       outputSchema: KanbanBoardOutputSchema,
       annotations: { readOnlyHint: true, idempotentHint: true, openWorldHint: true },
@@ -3105,7 +3245,9 @@ function getKanbanTools(context) {
           const input = z.object({
             project_number: z.number().optional(),
             owner: z.string().optional(),
-            repo: z.string().optional()
+            repo: z.string().optional(),
+            summary_only: z.boolean().optional().default(false),
+            item_limit: z.number().max(100).optional().default(25)
           }).parse(params);
           const resolved = await resolveOwner(context, input.owner);
           if ("error" in resolved) return resolved.response;
@@ -3144,7 +3286,44 @@ function getKanbanTools(context) {
               recoverable: true
             };
           }
-          return board;
+          const summaryOnly = input.summary_only || input.item_limit === 0;
+          if (summaryOnly) {
+            const summaryColumns = board.columns.map((col) => ({
+              status: col.status,
+              statusOptionId: col.statusOptionId,
+              items: [],
+              itemCount: col.items.length
+            }));
+            const itemDirectory = board.columns.flatMap(
+              (col) => col.items.map((item) => ({
+                id: item.id,
+                title: item.title,
+                status: item.status
+              }))
+            );
+            return {
+              ...board,
+              columns: summaryColumns,
+              summaryOnly: true,
+              itemDirectory
+            };
+          }
+          const itemLimit = input.item_limit;
+          const truncatedColumns = board.columns.map((col) => {
+            if (col.items.length <= itemLimit) {
+              return { ...col, itemCount: col.items.length };
+            }
+            return {
+              ...col,
+              items: col.items.slice(0, itemLimit),
+              itemCount: col.items.length,
+              truncated: true
+            };
+          });
+          return {
+            ...board,
+            columns: truncatedColumns
+          };
         } catch (err) {
           return formatHandlerError(err);
         }
@@ -3237,6 +3416,155 @@ function getKanbanTools(context) {
           return formatHandlerError(err);
         }
       }
+    },
+    {
+      name: "add_kanban_item",
+      title: "Add Kanban Item",
+      description: "Add an existing GitHub Issue or Pull Request to a Kanban board (GitHub Project v2). Returns the project item ID on success.",
+      group: "github",
+      inputSchema: z.object({
+        project_number: z.number().optional().describe("GitHub Project number (optional if repo is registered)"),
+        issue_number: z.number().describe("The number of the issue or PR to add"),
+        owner: z.string().optional().describe("Repository owner - LEAVE EMPTY to auto-detect"),
+        repo: z.string().optional().describe("Repository name - LEAVE EMPTY to auto-detect")
+      }),
+      outputSchema: AddKanbanItemOutputSchema,
+      annotations: { readOnlyHint: false, idempotentHint: true, openWorldHint: true },
+      handler: async (params) => {
+        try {
+          const input = z.object({
+            project_number: z.number().optional(),
+            issue_number: z.number(),
+            owner: z.string().optional(),
+            repo: z.string().optional()
+          }).parse(params);
+          const resolved = await resolveOwnerRepo(context, input);
+          if ("error" in resolved) return resolved.response;
+          const projectNum = resolveProjectNumber(
+            context,
+            resolved.repo,
+            input.project_number
+          );
+          if (projectNum === void 0) {
+            return {
+              success: false,
+              error: "project_number is required and could not be resolved from registry. Please supply it explicitly.",
+              code: "VALIDATION_ERROR",
+              category: "validation",
+              recoverable: true
+            };
+          }
+          const issueDetails = await resolved.github.getIssue(
+            resolved.owner,
+            resolved.repo,
+            input.issue_number
+          );
+          if (!issueDetails?.nodeId) {
+            return {
+              success: false,
+              error: `Issue or PR #${String(input.issue_number)} not found or lacks a nodeId`,
+              code: "RESOURCE_NOT_FOUND",
+              category: "resource",
+              recoverable: true
+            };
+          }
+          const board = await resolved.github.getProjectKanban(
+            resolved.owner,
+            projectNum,
+            resolved.repo
+          );
+          if (!board) {
+            return {
+              success: false,
+              error: `Project #${String(projectNum)} not found`,
+              code: "RESOURCE_NOT_FOUND",
+              category: "resource",
+              recoverable: true
+            };
+          }
+          const result = await resolved.github.addProjectItem(
+            board.projectId,
+            issueDetails.nodeId
+          );
+          return {
+            success: result.success,
+            itemId: result.itemId,
+            projectNumber: projectNum,
+            message: result.success ? `Added Issue/PR #${String(input.issue_number)} to Project #${projectNum}` : void 0,
+            error: result.error
+          };
+        } catch (err) {
+          return formatHandlerError(err);
+        }
+      }
+    },
+    {
+      name: "delete_kanban_item",
+      title: "Delete Kanban Item",
+      description: "Remove an item from a GitHub Kanban board cleanly. This untethers it from the project but does NOT close or delete the underlying Issue/PR.",
+      group: "github",
+      inputSchema: z.object({
+        project_number: z.number().optional().describe("GitHub Project number (optional if repo is registered)"),
+        item_id: z.string().describe("The project item ID to remove (from get_kanban_board)"),
+        owner: z.string().optional().describe("Project owner - LEAVE EMPTY to auto-detect"),
+        repo: z.string().optional().describe("Repository name - LEAVE EMPTY to auto-detect")
+      }),
+      outputSchema: DeleteKanbanItemOutputSchema,
+      annotations: { readOnlyHint: false, idempotentHint: false, openWorldHint: true },
+      handler: async (params) => {
+        try {
+          const input = z.object({
+            project_number: z.number().optional(),
+            item_id: z.string(),
+            owner: z.string().optional(),
+            repo: z.string().optional()
+          }).parse(params);
+          const resolved = await resolveOwner(context, input.owner);
+          if ("error" in resolved) return resolved.response;
+          const effectiveRepo = input.repo ?? resolved.repo;
+          const projectNum = resolveProjectNumber(
+            context,
+            effectiveRepo,
+            input.project_number
+          );
+          if (projectNum === void 0) {
+            return {
+              success: false,
+              error: "project_number is required and could not be resolved from registry. Please supply it explicitly.",
+              code: "VALIDATION_ERROR",
+              category: "validation",
+              recoverable: true
+            };
+          }
+          const board = await resolved.github.getProjectKanban(
+            resolved.owner,
+            projectNum,
+            effectiveRepo
+          );
+          if (!board) {
+            return {
+              success: false,
+              error: `Project #${String(projectNum)} not found`,
+              code: "RESOURCE_NOT_FOUND",
+              category: "resource",
+              recoverable: true
+            };
+          }
+          const result = await resolved.github.deleteProjectItem(
+            board.projectId,
+            input.item_id
+          );
+          return {
+            success: result.success,
+            itemId: input.item_id,
+            projectNumber: projectNum,
+            message: result.success ? `Deleted item ${input.item_id} from Project #${projectNum}` : void 0,
+            error: result.error
+          };
+        } catch (err) {
+          return formatHandlerError(err);
+        }
+      }
     }
   ];
 }
@@ -3294,7 +3622,11 @@ function getGitHubIssueTools(context) {
           if (!issue) {
             return {
               success: false,
-              error: "Failed to create GitHub issue. Check GITHUB_TOKEN permissions."
+              error: "Failed to create GitHub issue. Check GITHUB_TOKEN permissions.",
+              code: "API_ERROR",
+              category: "api",
+              suggestion: "Verify GitHub token has write access to issues.",
+              recoverable: true
             };
           }
           const projectNumber = resolveProjectNumber(
@@ -3453,13 +3785,21 @@ Description: ${input.body.slice(0, 200)}${input.body.length > 200 ? "..." : ""}`
           if (!issueDetails) {
             return {
               success: false,
-              error: `Issue #${String(input.issue_number)} not found`
+              error: `Issue #${String(input.issue_number)} not found`,
+              code: "RESOURCE_NOT_FOUND",
+              category: "resource",
+              suggestion: "Verify the issue number exists in this repository.",
+              recoverable: true
             };
           }
           if (issueDetails.state === "CLOSED") {
             return {
-              success: false,
-              error: `Issue #${String(input.issue_number)} is already closed`
+              ...formatHandlerError(
+                new ValidationError(
+                  `Issue #${String(input.issue_number)} is already closed`
+                )
+              ),
+              success: false
             };
           }
           const result = await resolved.github.closeIssue(
@@ -3471,7 +3811,11 @@ Description: ${input.body.slice(0, 200)}${input.body.length > 200 ? "..." : ""}`
           if (!result) {
             return {
               success: false,
-              error: "Failed to close GitHub issue. Check GITHUB_TOKEN permissions."
+              error: "Failed to close GitHub issue. Check GITHUB_TOKEN permissions.",
+              code: "API_ERROR",
+              category: "api",
+              suggestion: "Verify GitHub token has write access to issues.",
+              recoverable: true
             };
           }
           let kanbanResult;
@@ -3630,10 +3974,12 @@ function isResourceError(result) {
 }
 var DEFAULT_BRIEFING_CONFIG = {
   entryCount: 3,
+  summaryCount: 1,
   includeTeam: false,
   issueCount: 0,
   prCount: 0,
   prStatusBreakdown: false,
+  milestoneCount: 3,
   workflowCount: 0,
   workflowStatusBreakdown: false,
   copilotReviews: false
@@ -4013,6 +4359,7 @@ function getGitHubInsightsTools(context) {
           const repo = resolved.repo;
           const section = input.sections;
           const result = {
+            success: true,
             owner,
             repo,
             section
@@ -4373,7 +4720,7 @@ var TeamCreateEntrySchemaMcp = z.object({
   author: z.string().optional()
 });
 var TeamGetRecentSchema = z.object({
-  limit: z.number().max(500).optional().default(10)
+  limit: z.number().min(1).max(500).optional().default(10)
 });
 var TeamGetRecentSchemaMcp = z.object({
   limit: relaxedNumber().optional().default(10)
@@ -4600,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({
@@ -4607,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({
@@ -4857,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);
@@ -4887,6 +5243,7 @@ function getTeamSearchTools(context) {
               });
             }
           }
+          entries = entries.slice(0, limit);
           const authorMap = batchFetchAuthors(
             teamDb,
             entries.map((e) => e.id)
@@ -4895,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);
         }
@@ -4938,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);
         }
@@ -5347,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];
@@ -5792,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
             );
           }
@@ -5814,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;
@@ -5827,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,
@@ -6021,6 +6404,8 @@ var METHOD_ALIASES = {
     context: "getGithubContext",
     kanban: "getKanbanBoard",
     moveKanban: "moveKanbanItem",
+    addKanban: "addKanbanItem",
+    deleteKanban: "deleteKanbanItem",
     milestones: "getGithubMilestones",
     milestone: "getGithubMilestone",
     createMilestone: "createGithubMilestone",
@@ -6095,6 +6480,7 @@ var GROUP_EXAMPLES = {
     'mj.github.getGithubIssues({ state: "open" })',
     'mj.github.getGithubPrs({ state: "open" })',
     "mj.github.getKanbanBoard({ project_number: 1 })",
+    "mj.github.addKanbanItem({ project_number: 1, issue_number: 123 })",
     "mj.github.getGithubMilestones()",
     "mj.github.getRepoInsights()"
   ],
@@ -6143,7 +6529,9 @@ var POSITIONAL_PARAM_MAP = {
   getGithubIssue: "issue_number",
   getGithubPr: "pr_number",
   getKanbanBoard: "project_number",
-  moveKanbanItem: ["item_id", "status"],
+  moveKanbanItem: ["item_id", "target_status"],
+  addKanbanItem: "issue_number",
+  deleteKanbanItem: "item_id",
   getGithubMilestone: "milestone_number",
   createGithubMilestone: "title",
   updateGithubMilestone: "milestone_number",
@@ -6377,8 +6765,8 @@ var DEFAULT_SECURITY_CONFIG = {
   maxCodeLength: 50 * 1024,
   // 50KB
   maxExecutionsPerMinute: 60,
-  maxResultSize: 10 * 1024 * 1024,
-  // 10MB
+  maxResultSize: 100 * 1024,
+  // 100KB — context window protection (configurable via CODE_MODE_MAX_RESULT_SIZE)
   blockedPatterns: [
     /\brequire\s*\(/,
     // No require()
@@ -6493,9 +6881,12 @@ var CodeModeSecurityManager = class {
     const errors = [];
     try {
       const serialized = JSON.stringify(result);
-      if (Buffer.byteLength(serialized, "utf-8") > this.config.maxResultSize) {
+      const actualBytes = Buffer.byteLength(serialized, "utf-8");
+      if (actualBytes > this.config.maxResultSize) {
+        const actualKb = Math.ceil(actualBytes / 1024);
+        const limitKb = Math.ceil(this.config.maxResultSize / 1024);
         errors.push(
-          `Result exceeds maximum size of ${String(this.config.maxResultSize)} bytes`
+          `Result exceeds maximum size of ${String(limitKb)} KB (${String(actualKb)} KB returned). Extract specific fields or aggregate data before returning. Example: instead of \`return await mj.github.getKanbanBoard(5)\`, use \`const b = await mj.github.getKanbanBoard(5); return { columns: b.columns?.length ?? 0, totalItems: b.totalItems }\``
         );
       }
     } catch (error) {
@@ -6871,7 +7262,12 @@ var ExecuteCodeSchemaMcp = z.object({
 var securityManager = null;
 var sandboxPool = null;
 function getSecurityManager() {
-  securityManager ??= new CodeModeSecurityManager();
+  if (!securityManager) {
+    const envMaxSize = process.env["CODE_MODE_MAX_RESULT_SIZE"];
+    const parsedMaxSize = envMaxSize && /^\d+$/.test(envMaxSize) ? parseInt(envMaxSize, 10) : void 0;
+    const overrides = parsedMaxSize !== void 0 && Number.isFinite(parsedMaxSize) && Number.isInteger(parsedMaxSize) && parsedMaxSize > 0 ? { maxResultSize: parsedMaxSize } : void 0;
+    securityManager = new CodeModeSecurityManager(overrides);
+  }
   return securityManager;
 }
 function getSandboxPool() {
@@ -7343,6 +7739,8 @@ var TOOL_GROUPS = {
     "get_github_context",
     "get_kanban_board",
     "move_kanban_item",
+    "add_kanban_item",
+    "delete_kanban_item",
     "create_github_issue_with_entry",
     "close_github_issue_with_entry",
     "get_github_milestones",