pipe-kan 0.23.1 → 0.23.2

package/dist/pipe-kan.js

@@ -300,6 +300,134 @@ function flagsToJql(flags) {
   return clauses.join(" AND ");
 }
 
+// src/store.ts
+var ME = {
+  displayName: "Person A",
+  emailAddress: "user@test.com",
+  name: "user@test.com"
+};
+var TRANSITIONS = [
+  { id: "11", name: "To Do" },
+  { id: "21", name: "In Progress" },
+  { id: "31", name: "Done" }
+];
+var ALLOWED = {
+  "To Do": ["In Progress"],
+  "In Progress": ["To Do", "Done"],
+  Done: ["In Progress"]
+};
+function validIssueKey(key) {
+  return /^[A-Z][A-Z0-9]*-\d+$/i.test(key.trim());
+}
+function fieldValue(issue, field) {
+  switch (field) {
+    case "project":
+      return issue.key.split("-")[0] ?? "";
+    case "assignee":
+      return [
+        issue.fields.assignee?.emailAddress,
+        issue.fields.assignee?.displayName
+      ].filter(Boolean).join(`
+`);
+    case "status":
+      return issue.fields.status.name;
+    case "parent":
+      return issue.fields.parent?.key ?? "";
+    case "type": {
+      const named = issue.fields.issuetype ?? issue.fields.issueType;
+      if (named && typeof named === "object" && "name" in named && typeof named.name === "string") {
+        return named.name;
+      }
+      return "";
+    }
+    case "key":
+      return issue.key;
+    default:
+      return "";
+  }
+}
+function matchesClause(issue, clause) {
+  const match = clause.trim().match(/^(\w+)\s*(!=|=)\s*"?([^"]+)"?$/);
+  if (!match)
+    return true;
+  const [, field, op, raw] = match;
+  const expected = raw.trim();
+  const actual = fieldValue(issue, field);
+  const hit = field === "assignee" ? actual.split(`
+`).some((value) => value.toLowerCase() === expected.toLowerCase()) : actual.toLowerCase() === expected.toLowerCase();
+  return op === "!=" ? !hit : hit;
+}
+function matchJql(issue, jql) {
+  const stripped = jql.replace(/\s+ORDER BY\s+.+$/i, "").trim();
+  if (!stripped)
+    return true;
+  return stripped.split(/\s+AND\s+/i).every((clause) => matchesClause(issue, clause));
+}
+
+class IssueStore {
+  issues;
+  constructor(issues) {
+    this.issues = issues;
+  }
+  static fromRaw(raw) {
+    if (!Array.isArray(raw)) {
+      throw new Error("Fixture must be a JSON array");
+    }
+    return new IssueStore(structuredClone(raw));
+  }
+  all() {
+    return this.issues;
+  }
+  rawJson() {
+    return JSON.stringify(this.issues, null, 2);
+  }
+  get(key) {
+    return this.issues.find((issue) => issue.key.toUpperCase() === key.toUpperCase());
+  }
+  list(jql = "") {
+    return this.issues.filter((issue) => matchJql(issue, jql));
+  }
+  childrenOf(keys) {
+    const listed = new Set(keys.filter(validIssueKey));
+    return this.issues.filter((issue) => {
+      if (fieldValue(issue, "type").toLowerCase() === "epic")
+        return false;
+      return listed.has(fieldValue(issue, "parent"));
+    });
+  }
+  transitions(key) {
+    const issue = this.get(key);
+    if (!issue)
+      return [];
+    const allowed = ALLOWED[issue.fields.status.name] ?? [];
+    return TRANSITIONS.map((transition) => ({
+      ...transition,
+      isAvailable: allowed.includes(transition.name)
+    }));
+  }
+  move(key, status) {
+    const issue = this.get(key);
+    if (!issue)
+      return { ok: false, error: `Issue ${key} not found` };
+    const transition = this.transitions(key).find((item) => item.name.toLowerCase() === status.toLowerCase() || item.id === status);
+    if (!transition || !transition.isAvailable) {
+      const available = this.transitions(key).filter((item) => item.isAvailable).map((item) => item.name).join(", ");
+      return {
+        ok: false,
+        error: `invalid transition state "${status}"
+Available states for issue ${key}: ${available}`
+      };
+    }
+    issue.fields.status.name = transition.name;
+    if (transition.name === "Done") {
+      issue.fields.resolution = { name: "Done" };
+    } else {
+      delete issue.fields.resolution;
+    }
+    return { ok: true };
+  }
+}
+
 // src/cli.ts
 function emptyList(text) {
   return /no result found/i.test(text);
@@ -332,7 +460,7 @@ function createStoreCli(store) {
       return JSON.stringify(issues, null, 2);
     },
     async listChildren(keys) {
-      return JSON.stringify(store.childrenOf(keys), null, 2);
+      return JSON.stringify(store.childrenOf(keys.filter(validIssueKey)), null, 2);
     },
     async move(key, status) {
       return store.move(key, status);
@@ -468,9 +596,12 @@ function createJiraCli(opts = {}) {
       ]);
     },
     async listChildren(keys) {
-      if (!keys.length)
+      const validKeys = keys.filter(validIssueKey);
+      if (!validKeys.length) {
+        console.log("Refresh children skipped; no valid parent keys");
         return "[]";
-      const list = keys.map((key) => `"${key}"`).join(", ");
+      }
+      const list = validKeys.map((key) => `"${key}"`).join(", ");
       return listAll([
         "issue",
         "list",
@@ -741,10 +872,10 @@ function createApp(opts) {
             console.log(`Refresh children ${cards.length}`);
           }
         } catch (err) {
-          nextHasCache = false;
-          nextChildren = [];
+          nextChildren = childrenRaw;
+          nextHasCache = hasChildrenCache;
           nextError = err instanceof Error ? err.message : "Epic children list failed";
-          console.log("Refresh children failed", nextError);
+          console.log("Refresh children failed; keeping existing children", nextError);
         }
         payload = nextPayload;
         epicsPayload = nextEpics;
@@ -805,131 +936,6 @@ function createApp(opts) {
   return app;
 }
 
-// src/store.ts
-var ME = {
-  displayName: "Person A",
-  emailAddress: "user@test.com",
-  name: "user@test.com"
-};
-var TRANSITIONS = [
-  { id: "11", name: "To Do" },
-  { id: "21", name: "In Progress" },
-  { id: "31", name: "Done" }
-];
-var ALLOWED = {
-  "To Do": ["In Progress"],
-  "In Progress": ["To Do", "Done"],
-  Done: ["In Progress"]
-};
-function fieldValue(issue, field) {
-  switch (field) {
-    case "project":
-      return issue.key.split("-")[0] ?? "";
-    case "assignee":
-      return [
-        issue.fields.assignee?.emailAddress,
-        issue.fields.assignee?.displayName
-      ].filter(Boolean).join(`
-`);
-    case "status":
-      return issue.fields.status.name;
-    case "parent":
-      return issue.fields.parent?.key ?? "";
-    case "type": {
-      const named = issue.fields.issuetype ?? issue.fields.issueType;
-      if (named && typeof named === "object" && "name" in named && typeof named.name === "string") {
-        return named.name;
-      }
-      return "";
-    }
-    case "key":
-      return issue.key;
-    default:
-      return "";
-  }
-}
-function matchesClause(issue, clause) {
-  const match = clause.trim().match(/^(\w+)\s*(!=|=)\s*"?([^"]+)"?$/);
-  if (!match)
-    return true;
-  const [, field, op, raw] = match;
-  const expected = raw.trim();
-  const actual = fieldValue(issue, field);
-  const hit = field === "assignee" ? actual.split(`
-`).some((value) => value.toLowerCase() === expected.toLowerCase()) : actual.toLowerCase() === expected.toLowerCase();
-  return op === "!=" ? !hit : hit;
-}
-function matchJql(issue, jql) {
-  const stripped = jql.replace(/\s+ORDER BY\s+.+$/i, "").trim();
-  if (!stripped)
-    return true;
-  return stripped.split(/\s+AND\s+/i).every((clause) => matchesClause(issue, clause));
-}
-
-class IssueStore {
-  issues;
-  constructor(issues) {
-    this.issues = issues;
-  }
-  static fromRaw(raw) {
-    if (!Array.isArray(raw)) {
-      throw new Error("Fixture must be a JSON array");
-    }
-    return new IssueStore(structuredClone(raw));
-  }
-  all() {
-    return this.issues;
-  }
-  rawJson() {
-    return JSON.stringify(this.issues, null, 2);
-  }
-  get(key) {
-    return this.issues.find((issue) => issue.key.toUpperCase() === key.toUpperCase());
-  }
-  list(jql = "") {
-    return this.issues.filter((issue) => matchJql(issue, jql));
-  }
-  childrenOf(keys) {
-    const listed = new Set(keys);
-    return this.issues.filter((issue) => {
-      if (fieldValue(issue, "type").toLowerCase() === "epic")
-        return false;
-      return listed.has(fieldValue(issue, "parent"));
-    });
-  }
-  transitions(key) {
-    const issue = this.get(key);
-    if (!issue)
-      return [];
-    const allowed = ALLOWED[issue.fields.status.name] ?? [];
-    return TRANSITIONS.map((transition) => ({
-      ...transition,
-      isAvailable: allowed.includes(transition.name)
-    }));
-  }
-  move(key, status) {
-    const issue = this.get(key);
-    if (!issue)
-      return { ok: false, error: `Issue ${key} not found` };
-    const transition = this.transitions(key).find((item) => item.name.toLowerCase() === status.toLowerCase() || item.id === status);
-    if (!transition || !transition.isAvailable) {
-      const available = this.transitions(key).filter((item) => item.isAvailable).map((item) => item.name).join(", ");
-      return {
-        ok: false,
-        error: `invalid transition state "${status}"
-Available states for issue ${key}: ${available}`
-      };
-    }
-    issue.fields.status.name = transition.name;
-    if (transition.name === "Done") {
-      issue.fields.resolution = { name: "Done" };
-    } else {
-      delete issue.fields.resolution;
-    }
-    return { ok: true };
-  }
-}
-
 // src/boot.ts
 async function createBoardApp(opts) {
   const env = opts.env ?? process.env;
@@ -11754,6 +11760,21 @@ function readBody3(req) {
 function pathOf3(req) {
   return new URL(req.url ?? "/", "http://127.0.0.1");
 }
+function childrenJqlError(jql) {
+  const trimmed = jql.trim();
+  if (/\bin\s*\(\s*\)/i.test(trimmed)) {
+    return "The value '' does not exist for the field 'parent'.";
+  }
+  const childrenMatch = trimmed.match(/(?:parent|"Epic Link")\s+in\s+\(([^)]+)\)/i);
+  if (childrenMatch) {
+    const values = childrenMatch[1].split(",").map((value) => value.trim().replace(/^["']|["']$/g, ""));
+    const invalid = values.filter((value) => !validIssueKey(value));
+    if (invalid.length) {
+      return `No issues have a parent epic with key or name '${invalid[0]}'.`;
+    }
+  }
+  return;
+}
 function handleFakeJira(req, res, store) {
   const url = pathOf3(req);
   const path = url.pathname;
@@ -11763,7 +11784,13 @@ function handleFakeJira(req, res, store) {
     return true;
   }
   if ((path === "/rest/api/3/search/jql" || path === "/rest/api/2/search" || path === "/rest/api/3/search") && method === "GET") {
-    const issues = store.list(url.searchParams.get("jql") ?? "");
+    const jql = url.searchParams.get("jql") ?? "";
+    const error = childrenJqlError(jql);
+    if (error) {
+      json3(res, 400, { errorMessages: [error] });
+      return true;
+    }
+    const issues = store.list(jql);
     json3(res, 200, { expand: "schema,names", isLast: true, issues });
     return true;
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pipe-kan",
-  "version": "0.23.1",
+  "version": "0.23.2",
   "description": "Local Kanban for jira-cli",
   "license": "MIT",
   "type": "module",