@kud/gh-ink 0.17.0 → 0.19.0

package/dist/index.d.ts

@@ -74,6 +74,15 @@ type GHItem = {
     conversation: number;
     lastActor?: string;
     detail?: GHDetail;
+    /**
+     * Where YOU stand on this row, when the host knows it per row rather than per
+     * tab. Set it and the whose-move band reads it instead of inferring from the
+     * section id — which is what lets one tab hold rows of mixed standing: a
+     * review asked of you and one you have already given differ only in whether
+     * the ball comes back, and that is a fact about the search a row came from,
+     * never about the PR. Left unset, the section id decides as before.
+     */
+    standing?: Standing;
     indent: boolean;
 };
 type TaskRow = {
@@ -163,8 +172,20 @@ type Action = {
     subActions?: Action[];
 };
 type JiraTransition = {
+    /** What the action menu shows. Yours to word; nothing matches on it. */
     label: string;
-    state: string;
+    /**
+     * The TRANSITION name, passed verbatim to `jira issue move`. Not the
+     * destination status, and the two are routinely different strings — jira-cli
+     * matches `t.name.toLowerCase() === wanted`, so a status name here matches no
+     * transition, the move silently does nothing, and the row simply does not
+     * budge. This field was called `state`, which is what invited exactly that:
+     * cockpit had `{ label: "UAT", state: "UAT" }` against an ACC workflow whose
+     * transition is named "Ready for QA" and whose status is "In Testing (QA)" —
+     * no "UAT" anywhere, and no error either.
+     */
+    transition: string;
+    /** Offered as a second step when the transition asks for a resolution. */
     resolutions?: string[];
 };
 declare const relativeTime: (iso: string) => string;
@@ -177,7 +198,8 @@ declare const repoPriority: (repo: string) => number;
 declare const sortItems: (items: GHItem[]) => GHItem[];
 declare const sortByRecency: (items: GHItem[]) => GHItem[];
 declare const insertRepoHeaders: (items: GHItem[]) => AnyItem[];
-declare const whoseMove: (health: Health, sectionId: string) => "you" | "them";
+type Standing = "authored" | "queued" | "spoken";
+declare const whoseMove: (health: Health, sectionId: string, standing?: Standing) => "you" | "them";
 declare const layoutGHItems: (items: GHItem[], sectionId: string) => AnyItem[];
 declare const filterByOrigin: (sections: Section[], keep: "work" | "home", isWorkRepo: (repo: string) => boolean) => Section[];
 declare const filterBySearch: (sections: Section[], query: string) => Section[];
@@ -289,4 +311,4 @@ declare const writeCache: (key: string, data: {
     login: string;
 }) => void;
 
-export { type Action, ActionMenu, type AnyItem, App, COLS, type CiStatus, CiStatusLine, type CiStatusState, CommentsPanel, type CommentsPanelProps, type DetailContext, type ExplainSection, type ExtensionTarget, type GHDetail, type GHItem, HealthPanel, type HealthPanelProps, type InboxExtension, type JiraTransition, type RepoHeader, type Section, type ShowLess, type ShowMore, type SubgroupHeader, type TaskRow, buildActions, buildCheckoutCmd, clipboard, drillCmd, explainItem, filterByOrigin, filterByRepos, filterBySearch, fitCount, healthColor, healthDisplay, healthGlyph, healthLegend, insertRepoHeaders, itermRun, jumpToRepo, jumpToRepoPane, layoutGHItems, maxViewStart, moveCursor, openInTab, readCache, relativeTime, renderMarkdown, repoPriority, reposInSections, resolveRepoPath, runHere, runInPane, runInPaneHorizontal, sameCiStatusState, sortByRecency, sortItems, toCiStatusState, topLevelCount, truncate, useActionMenu, whoseMove, windowCount, withHeaders, withoutItem, writeCache };
+export { type Action, ActionMenu, type AnyItem, App, COLS, type CiStatus, CiStatusLine, type CiStatusState, CommentsPanel, type CommentsPanelProps, type DetailContext, type ExplainSection, type ExtensionTarget, type GHDetail, type GHItem, HealthPanel, type HealthPanelProps, type InboxExtension, type JiraTransition, type RepoHeader, type Section, type ShowLess, type ShowMore, type Standing, type SubgroupHeader, type TaskRow, buildActions, buildCheckoutCmd, clipboard, drillCmd, explainItem, filterByOrigin, filterByRepos, filterBySearch, fitCount, healthColor, healthDisplay, healthGlyph, healthLegend, insertRepoHeaders, itermRun, jumpToRepo, jumpToRepoPane, layoutGHItems, maxViewStart, moveCursor, openInTab, readCache, relativeTime, renderMarkdown, repoPriority, reposInSections, resolveRepoPath, runHere, runInPane, runInPaneHorizontal, sameCiStatusState, sortByRecency, sortItems, toCiStatusState, topLevelCount, truncate, useActionMenu, whoseMove, windowCount, withHeaders, withoutItem, writeCache };

package/dist/index.js

@@ -875,12 +875,26 @@ var insertRepoHeaders = (items) => {
   }
   return result;
 };
-var AUTHORED_SECTIONS = /* @__PURE__ */ new Set(["open", "draft", "assigned", "issues"]);
-var whoseMove = (health, sectionId) => {
-  if (health === "threads" || health === "approved") return "you";
-  const yours = AUTHORED_SECTIONS.has(sectionId) ? ["ci-fail", "conflict", "changes-req"] : ["waiting", "pending"];
-  return yours.includes(health) ? "you" : "them";
-};
+var STANDING = {
+  mine: "authored",
+  // `open` and `draft` predate `mine` and are the same standing: a host that
+  // still splits its own PRs by draft-ness keeps working, and one that folds
+  // them into a single tab (as cockpit does — the band already sinks a draft,
+  // so the split said it twice) gets the same reading.
+  open: "authored",
+  draft: "authored",
+  assigned: "authored",
+  issues: "authored",
+  review: "queued",
+  incoming: "queued",
+  reviewed: "spoken"
+};
+var YOURS = {
+  authored: ["ci-fail", "conflict", "changes-req", "threads", "approved"],
+  queued: ["waiting", "pending", "threads", "approved"],
+  spoken: ["threads"]
+};
+var whoseMove = (health, sectionId, standing) => YOURS[standing ?? STANDING[sectionId] ?? "queued"].includes(health) ? "you" : "them";
 var BAND_LABEL = {
   you: "Your move",
   them: "Their move"
@@ -890,7 +904,9 @@ var layoutGHItems = (items, sectionId) => {
   const sorted = sortItems(items);
   const bands = ["you", "them"].map((side) => ({
     side,
-    rows: sorted.filter((i) => whoseMove(i.health, sectionId) === side)
+    rows: sorted.filter(
+      (i) => whoseMove(i.health, sectionId, i.standing) === side
+    )
   }));
   const filled = bands.filter((b) => b.rows.length > 0);
   if (filled.length < 2) return insertRepoHeaders(sorted);
@@ -1293,7 +1309,7 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
         run: () => {
         },
         subActions: jiraTransitions.map(
-          ({ label, state, resolutions }) => resolutions && resolutions.length > 0 ? {
+          ({ label, transition, resolutions }) => resolutions && resolutions.length > 0 ? {
             label,
             hint: "",
             run: () => {
@@ -1303,7 +1319,7 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
               hint: "",
               run: () => {
                 showFlash(`\u22EF ${label} \xB7 ${resolution}\u2026`);
-                void quietly`jira issue move ${item.ticket} ${state} --resolution ${resolution}`.then(() => {
+                void quietly`jira issue move ${item.ticket} ${transition} --resolution ${resolution}`.then(() => {
                   showFlash(`\u2713 ${label} \xB7 ${resolution}`);
                   ext?.onActed?.();
                 }).catch(() => showFlash(`\u2717 Move to ${label} failed`));
@@ -1314,7 +1330,7 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
             hint: "",
             run: () => {
               showFlash(`\u22EF Moving to ${label}\u2026`);
-              void quietly`jira issue move ${item.ticket} ${state}`.then(() => {
+              void quietly`jira issue move ${item.ticket} ${transition}`.then(() => {
                 showFlash(`\u2713 Moved to ${label}`);
                 ext?.onActed?.();
               }).catch(() => showFlash(`\u2717 Move to ${label} failed`));
@@ -2411,7 +2427,7 @@ var BrowseScreen = ({
     if (input === "t" && activeItem && activeItem.kind === "task" && activeItem.ticket && jiraTransitions && jiraTransitions.length > 0) {
       const jiraKey = activeItem.ticket;
       const actions = jiraTransitions.map(
-        ({ label, state, resolutions }) => resolutions && resolutions.length > 0 ? {
+        ({ label, transition, resolutions }) => resolutions && resolutions.length > 0 ? {
           label,
           hint: "",
           run: () => {
@@ -2422,7 +2438,7 @@ var BrowseScreen = ({
             run: () => {
               menu.close();
               showFlash(`\u22EF ${label} \xB7 ${resolution}\u2026`);
-              quietly`jira issue move ${jiraKey} ${state} --resolution ${resolution}`.then(() => {
+              quietly`jira issue move ${jiraKey} ${transition} --resolution ${resolution}`.then(() => {
                 showFlash(`\u2713 ${label} \xB7 ${resolution}`);
                 onActed?.();
               }).catch(() => showFlash(`\u2717 Move to ${label} failed`));
@@ -2434,7 +2450,7 @@ var BrowseScreen = ({
           run: () => {
             menu.close();
             showFlash(`\u22EF Moving to ${label}\u2026`);
-            quietly`jira issue move ${jiraKey} ${state}`.then(() => {
+            quietly`jira issue move ${jiraKey} ${transition}`.then(() => {
               showFlash(`\u2713 Moved to ${label}`);
               onActed?.();
             }).catch(() => showFlash(`\u2717 Move to ${label} failed`));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kud/gh-ink",
-  "version": "0.17.0",
+  "version": "0.19.0",
   "description": "Ink components for rendering GitHub PR review comments and health — controlled, presentation-only, built on @kud/ink-ui and fed by @kud/gh.",
   "type": "module",
   "main": "dist/index.js",