gitflic-cli-mcp 0.4.0 → 0.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gitflic-cli-mcp",
-  "version": "0.4.0",
+  "version": "0.4.2",
   "description": "MCP server wrapping the gitflic CLI — exposes the GitFlic REST API as typed tools for AI agents (Claude Code, Cursor, etc.)",
   "type": "module",
   "bin": {
@@ -46,7 +46,7 @@
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.0.0",
-    "gitflic-cli": "^0.3.3",
+    "gitflic-cli": "0.3.5",
     "zod": "^3.25.76"
   }
 }

package/slim.mjs

@@ -79,7 +79,12 @@ const P = {
     status: statusId(i.status),
     labels: labelTitles(i.labels) ?? [],
     assignedUsers: (i.assignedUsers || []).map(userName).filter(Boolean),
-    author: i.userAlias ?? userName(i.createdBy),
+    // NOT i.userAlias: on an issue that is the *project owner's* alias (the
+    // `<userAlias>/<projectAlias>` path), identical on every issue in the repo —
+    // using it would report the owner as the author of everyone's issues.
+    // GitFlic's issue API exposes no author at all, so this stays undefined
+    // (key dropped on serialize) unless the API ever starts sending createdBy.
+    author: userName(i.createdBy),
     ...(i.updatedBy ? { updatedBy: userName(i.updatedBy) } : {}),
     createdAt: i.createdAt, updatedAt: i.updatedAt,
     ...(i.description != null ? { description: i.description } : {}),
@@ -209,7 +214,9 @@ export function applyListFilters(items, args = {}) {
   let out = items;
   if (args.label) out = out.filter((i) => (labelTitles(i.labels) || []).some((t) => ci(t).includes(ci(args.label))));
   if (args.assignee) out = out.filter((i) => (i.assignedUsers || []).some((u) => ci(userName(u)) === ci(args.assignee)));
-  if (args.author) out = out.filter((i) => ci(userName(i.createdBy) || userName(i.user) || i.userAlias || i.authorIdent?.name).includes(ci(args.author)));
+  // Mirrors the author resolution in P.mr / slimCommit. i.userAlias is NOT in
+  // the chain on purpose — it is the project owner's alias, not an author.
+  if (args.author) out = out.filter((i) => ci(userName(i.createdBy) || userName(i.author) || userName(i.user) || i.authorIdent?.name).includes(ci(args.author)));
   if (args.status) out = out.filter((i) => ci(statusId(i.status)) === ci(args.status));
   if (args.targetBranch) out = out.filter((i) => ci(branchName(i.targetBranch)).includes(ci(args.targetBranch)));
   if (args.sourceBranch) out = out.filter((i) => ci(branchName(i.sourceBranch)).includes(ci(args.sourceBranch)));
@@ -223,7 +230,15 @@ export function applyListFilters(items, args = {}) {
   if (args.name) { const re = globToRe(args.name); out = out.filter((i) => re.test(i.name ?? "") || ci(i.name).includes(ci(args.name))); }
   if (args.q) {
     const q = ci(args.q);
-    out = out.filter((i) => `${i.title ?? ""} ${i.description ?? ""} ${i.shortMessage ?? ""} ${i.message ?? ""}`.toLowerCase().includes(q));
+    // Haystack spans every entity's free-text/identity fields so `q` works for
+    // issues/MRs (title+description), commits (messages), users (username/
+    // fullName/name), projects (title/alias) and releases (tagName). Missing
+    // fields collapse to "" — harmless. (Bug fix: user_search used to drop all
+    // server results because the haystack lacked username/fullName.)
+    out = out.filter((i) =>
+      `${i.title ?? ""} ${i.description ?? ""} ${i.shortMessage ?? ""} ${i.message ?? ""} ${i.username ?? ""} ${i.fullName ?? ""} ${i.name ?? ""} ${i.alias ?? ""} ${i.tagName ?? ""}`
+        .toLowerCase()
+        .includes(q));
   }
   if (args.sort) {
     const desc = String(args.sort)[0] === "-";
@@ -279,7 +294,12 @@ export function slimResult(toolName, result, args = {}, env = process.env) {
   // Treat known list tools as lists even when GitFlic omits `_embedded` on a
   // zero-result response, so the {items,total,returned} contract is consistent.
   let lst = listOf(result);
-  if (!lst && shape.list && result.page) lst = { items: [], total: result.page.totalElements };
+  // Zero-result lists omit `_embedded` entirely. Treat any list tool (mapped
+  // OR an unmapped *_list/*_search) carrying a `page` envelope as an empty list
+  // so the {items,total,returned} contract holds even when there are no items.
+  if (!lst && result.page && (shape.list || /_(list|search)$/.test(toolName))) {
+    lst = { items: [], total: result.page.totalElements };
+  }
 
   if (lst) {
     let items = Array.isArray(lst.items) ? lst.items : [];