pipe-kan 0.20.3 → 0.20.5

package/dist/pipe-kan.js

@@ -261,6 +261,10 @@ function flagsToJql(flags) {
 function emptyList(text) {
   return /no result found/i.test(text);
 }
+function rateLimited(text) {
+  return /\b429\b/.test(text);
+}
+var RETRY_LIMIT = 4;
 var LIST_PAGE = 100;
 function issueKey(issue) {
   if (!issue || typeof issue !== "object" || !("key" in issue))
@@ -319,6 +323,10 @@ function resolveJiraBin(bin = process.env.JIRA_BIN ?? "jira", pathVar = process.
 }
 function createJiraCli(opts = {}) {
   const bin = opts.bin ?? "jira";
+  const retryDelayMs = opts.retryDelayMs ?? 1000;
+  function fmt(args) {
+    return args.filter((arg) => arg !== "--raw").map((arg, i, all) => all[i - 1] === "-q" && arg.length > 48 ? `${arg.slice(0, 48)}…` : arg).join(" ");
+  }
   function run(args) {
     return new Promise((resolveRun, reject) => {
       const env = { ...process.env };
@@ -339,16 +347,36 @@ function createJiraCli(opts = {}) {
       child.on("close", (code) => resolveRun({ code: code ?? 1, stdout, stderr }));
     });
   }
+  async function runRetry(args) {
+    let result = await run(args);
+    for (let attempt = 0;attempt < RETRY_LIMIT; attempt++) {
+      if (result.code === 0 || !rateLimited(result.stderr || result.stdout))
+        return result;
+      const delay = retryDelayMs * 2 ** attempt;
+      console.log(`jira 429 retry ${delay}ms ${fmt(args)}`);
+      if (retryDelayMs) {
+        await new Promise((resolve) => setTimeout(resolve, delay));
+      }
+      result = await run(args);
+    }
+    return result;
+  }
   async function listOnce(args) {
-    const result = await run(args);
+    const started = Date.now();
+    const result = await runRetry(args);
     if (result.code !== 0) {
       const text = result.stderr || result.stdout || "jira issue list failed";
-      if (emptyList(text))
+      if (emptyList(text)) {
+        console.log(`jira ${fmt(args)} -> 0 (${Date.now() - started}ms)`);
         return [];
+      }
+      console.log(`jira ${fmt(args)} fail (${Date.now() - started}ms)`);
       throw new Error(text);
     }
     const parsed = JSON.parse(result.stdout);
-    return Array.isArray(parsed) ? parsed : [];
+    const page = Array.isArray(parsed) ? parsed : [];
+    console.log(`jira ${fmt(args)} -> ${page.length} (${Date.now() - started}ms)`);
+    return page;
   }
   async function listAll(args) {
     if (hasPaginate(args)) {
@@ -377,6 +405,7 @@ function createJiraCli(opts = {}) {
       if (page.length < LIST_PAGE)
         break;
     }
+    console.log(`jira ${fmt(args)} ${issues.length} total`);
     return JSON.stringify(issues);
   }
   return {
@@ -407,7 +436,7 @@ function createJiraCli(opts = {}) {
       ]);
     },
     async move(key, status) {
-      const result = await run(["issue", "move", key, status]);
+      const result = await runRetry(["issue", "move", key, status]);
       if (result.code !== 0) {
         return {
           ok: false,
@@ -417,12 +446,12 @@ function createJiraCli(opts = {}) {
       return { ok: true };
     },
     async open(key) {
-      const result = await run(["open", key, "--no-browser"]);
+      const result = await runRetry(["open", key, "--no-browser"]);
       return result.stdout.trim().split(`
 `).pop() ?? `/browse/${key}`;
     },
     async view(key) {
-      const result = await run(["issue", "view", key, "--raw"]);
+      const result = await runRetry(["issue", "view", key, "--raw"]);
       if (result.code !== 0) {
         throw new Error((result.stderr || result.stdout || "jira issue view failed").trim());
       }
@@ -581,19 +610,6 @@ function stampMissingEpic(board, epic) {
   }
   return board;
 }
-function stampRawParent(raw, epic) {
-  if (!Array.isArray(raw))
-    return [];
-  return raw.map((issue) => {
-    if (!issue || typeof issue !== "object")
-      return issue;
-    const current = issue;
-    const fields = current.fields;
-    if (!fields || typeof fields !== "object" || fields.parent)
-      return issue;
-    return { ...current, fields: { ...fields, parent: { key: epic } } };
-  });
-}
 function cardsOf(raw) {
   if (!Array.isArray(raw))
     return [];
@@ -642,34 +658,46 @@ function createApp(opts) {
     async refresh(next) {
       if (next !== undefined)
         flags = next;
-      const [issues, epics] = await Promise.all([
-        cli.list(flags),
-        cli.listEpics()
-      ]);
-      const nextPayload = JSON.parse(issues);
-      const nextEpics = JSON.parse(epics);
-      const keys = issuesToBoard(nextEpics).epics.map((epic) => epic.key);
-      let nextChildren = [];
-      let nextHasCache = false;
-      let nextError;
+      console.log("Refresh");
       try {
-        nextChildren = JSON.parse(await cli.listChildren(keys));
-        const cards = cardsOf(nextChildren);
-        if (cards.length > 0 && !cards.some((card) => card.epic)) {
-          nextChildren = (await Promise.all(keys.map(async (key) => stampRawParent(JSON.parse(await cli.listEpic(key)), key)))).flat();
+        const issues = await cli.list(flags);
+        const epics = await cli.listEpics();
+        const nextPayload = JSON.parse(issues);
+        const nextEpics = JSON.parse(epics);
+        const keys = issuesToBoard(nextEpics).epics.map((epic) => epic.key);
+        const issueCount = Array.isArray(nextPayload) ? nextPayload.length : 0;
+        console.log(`Refresh listed ${issueCount} issues, ${keys.length} epics`);
+        let nextChildren = [];
+        let nextHasCache = false;
+        let nextError;
+        try {
+          nextChildren = JSON.parse(await cli.listChildren(keys));
+          const cards = cardsOf(nextChildren);
+          if (cards.length > 0 && !cards.some((card) => card.epic)) {
+            console.log(`Refresh children missing Epic keys; skip ${keys.length} per-Epic lists`);
+            nextChildren = [];
+            nextHasCache = false;
+          } else {
+            nextHasCache = true;
+            console.log(`Refresh children ${cards.length}`);
+          }
+        } catch (err) {
+          nextHasCache = false;
+          nextChildren = [];
+          nextError = err instanceof Error ? err.message : "Epic children list failed";
+          console.log("Refresh children failed", nextError);
         }
-        nextHasCache = true;
+        payload = nextPayload;
+        epicsPayload = nextEpics;
+        childrenRaw = nextChildren;
+        hasChildrenCache = nextHasCache;
+        childrenError = nextError;
+        return app.board();
       } catch (err) {
-        nextHasCache = false;
-        nextChildren = [];
-        nextError = err instanceof Error ? err.message : "Epic children list failed";
-      }
-      payload = nextPayload;
-      epicsPayload = nextEpics;
-      childrenRaw = nextChildren;
-      hasChildrenCache = nextHasCache;
-      childrenError = nextError;
-      return app.board();
+        const message = err instanceof Error ? err.message : "Refresh failed";
+        console.error("Refresh failed", message);
+        return { ...app.board(), error: message };
+      }
     },
     async children(epic) {
       if (hasChildrenCache) {
@@ -883,6 +911,14 @@ function readBody(req) {
 function pathOf(req) {
   return new URL(req.url ?? "/", "http://127.0.0.1");
 }
+function reply(req, res, work) {
+  readBody(req).then(work).catch((err) => {
+    const message = err instanceof Error ? err.message : "request failed";
+    console.error(message);
+    if (!res.headersSent)
+      json(res, 500, { error: message });
+  });
+}
 function handleAppApi(req, res, app) {
   const url = pathOf(req);
   const method = (req.method ?? "GET").toUpperCase();
@@ -891,15 +927,14 @@ function handleAppApi(req, res, app) {
     return true;
   }
   if (url.pathname === "/api/refresh" && method === "POST") {
-    readBody(req).then(async (text) => {
+    reply(req, res, async (text) => {
       const body = text ? JSON.parse(text) : {};
-      const board = await app.refresh(body.flags);
-      json(res, 200, board);
+      json(res, 200, await app.refresh(body.flags));
     });
     return true;
   }
   if (url.pathname === "/api/move" && method === "POST") {
-    readBody(req).then(async (text) => {
+    reply(req, res, async (text) => {
       const body = text ? JSON.parse(text) : {};
       const result = await app.move(String(body.key ?? ""), String(body.status ?? ""));
       json(res, result.ok ? 200 : 409, result);
@@ -907,14 +942,14 @@ function handleAppApi(req, res, app) {
     return true;
   }
   if (url.pathname === "/api/epic" && method === "POST") {
-    readBody(req).then(async (text) => {
+    reply(req, res, async (text) => {
       const body = text ? JSON.parse(text) : {};
       json(res, 200, await app.children(String(body.key ?? "")));
     });
     return true;
   }
   if (url.pathname === "/api/open" && method === "POST") {
-    readBody(req).then(async (text) => {
+    reply(req, res, async (text) => {
       const body = text ? JSON.parse(text) : {};
       json(res, 200, await app.open(String(body.key ?? "")));
     });