gitflic-cli-mcp 0.4.0 → 0.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gitflic-cli-mcp",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "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": {

package/slim.mjs

@@ -223,7 +223,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 +287,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 : [];