pipe-kan 0.20.4 → 0.21.0

package/dist/pipe-kan.js

@@ -324,6 +324,9 @@ 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 };
@@ -349,23 +352,31 @@ function createJiraCli(opts = {}) {
     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, retryDelayMs * 2 ** attempt));
+        await new Promise((resolve) => setTimeout(resolve, delay));
       }
       result = await run(args);
     }
     return result;
   }
   async function listOnce(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)) {
@@ -394,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 {
@@ -598,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 [];
@@ -633,6 +632,18 @@ function createApp(opts) {
     hasChildrenCache = true;
     childrenError = undefined;
   }
+  function currentStatus(key) {
+    const fromPayload = payload.find((issue) => issue.key === key)?.fields?.status?.name;
+    return fromPayload ?? issuesToBoard(epicsPayload).epics.find((epic) => epic.key === key)?.status ?? app.board().epics.find((epic) => epic.key === key)?.status;
+  }
+  async function tryMove(key, status) {
+    const current = currentStatus(key);
+    if (current === status) {
+      return { ok: true, noop: true, error: undefined };
+    }
+    const result = await cli.move(key, status);
+    return result.ok ? { ok: true, noop: false, error: undefined } : { ok: false, noop: false, error: result.error };
+  }
   const app = {
     get flags() {
       return flags;
@@ -659,38 +670,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)) {
-          const stamped = [];
-          for (const key of keys) {
-            stamped.push(...stampRawParent(JSON.parse(await cli.listEpic(key)), key));
+        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}`);
           }
-          nextChildren = stamped;
+        } 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) {
@@ -708,18 +727,20 @@ function createApp(opts) {
       return stampMissingEpic(issuesToBoard(JSON.parse(await cli.listEpic(epic))), epic);
     },
     async move(key, status) {
-      const fromPayload = payload.find((issue) => issue.key === key)?.fields?.status?.name;
-      const current = fromPayload ?? issuesToBoard(epicsPayload).epics.find((epic) => epic.key === key)?.status ?? app.board().epics.find((epic) => epic.key === key)?.status;
-      if (current === status) {
-        return { ok: true, noop: true, board: app.board() };
-      }
-      const result = await cli.move(key, status);
+      const result = await tryMove(key, status);
       if (!result.ok) {
         return { ok: false, error: result.error, board: app.board() };
       }
+      if (result.noop) {
+        return { ok: true, noop: true, board: app.board() };
+      }
       await app.refresh();
       return { ok: true, board: app.board() };
     },
+    async moveRaw(key, status) {
+      const result = await tryMove(key, status);
+      return result.ok ? { ok: true, noop: result.noop } : { ok: false, error: result.error };
+    },
     async open(key) {
       const urlP = cli.open(key);
       try {
@@ -904,6 +925,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();
@@ -912,30 +941,36 @@ 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 ?? ""));
+      const key = String(body.key ?? "");
+      const status = String(body.status ?? "");
+      if (body.raw) {
+        const result = await app.moveRaw(key, status);
+        json(res, result.ok ? 200 : 409, result);
+        return;
+      }
+      const result = await app.move(key, status);
       json(res, result.ok ? 200 : 409, result);
     });
     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 ?? "")));
     });