@bli-cockpit/cli 0.2.145 → 0.2.146

package/dist/commands/issue-directory.js

@@ -11,29 +11,32 @@ import { askAgentDoor } from "./agent-door.js";
 import { writeLine } from "./cli-io.js";
 const TEAM_DIRECTORY_TIMEOUT_MS = 30_000;
 /**
- * One `/api/team/members` call however many issues are in hand. Best-effort:
- * a directory that cannot be read still lets the issue(s) print their bare
- * `assignee_id`, the pre-BLI-10450 behaviour, rather than failing the whole
- * read over an enrichment nobody asked to gate on.
+ * The `/api/team/members` call by itself. The response is the whole org
+ * roster either way — it never actually depended on which issues came back —
+ * so a caller with its own read already in flight (BLI-10579: `issue list`'s
+ * `/api/work/issues`) can start this one at the same time instead of after.
  */
-export async function resolveAssigneeDirectory(door, assigneeIds) {
-    const wanted = new Set(assigneeIds.filter((id) => Boolean(id)));
-    const directory = new Map();
-    if (wanted.size === 0)
-        return directory;
+async function fetchTeamDirectory(door) {
     const answer = await askAgentDoor(door, {
         path: "/api/team/members",
         method: "GET",
         label: "issue assignee directory",
         timeoutMs: TEAM_DIRECTORY_TIMEOUT_MS,
     });
-    if (!answer.ok) {
-        writeLine(door.io.stderr, `[issue cli] assignee directory unread ${JSON.stringify({ reason: answer.reason, wanted: wanted.size })}`);
+    if (!answer.ok)
+        return { ok: false, people: [], reason: answer.reason };
+    return { ok: true, people: answer.body.people ?? [] };
+}
+function buildDirectory(door, fetched, wanted) {
+    const directory = new Map();
+    if (wanted.size === 0)
+        return directory;
+    if (!fetched.ok) {
+        writeLine(door.io.stderr, `[issue cli] assignee directory unread ${JSON.stringify({ reason: fetched.reason, wanted: wanted.size })}`);
         return directory;
     }
-    const people = answer.body.people ?? [];
     let resolved = 0;
-    for (const person of people) {
+    for (const person of fetched.people) {
         const userId = person.account?.userId;
         if (!userId || !wanted.has(userId))
             continue;
@@ -43,6 +46,36 @@ export async function resolveAssigneeDirectory(door, assigneeIds) {
     writeLine(door.io.stderr, `[issue cli] assignee directory resolved ${JSON.stringify({ wanted: wanted.size, resolved })}`);
     return directory;
 }
+/**
+ * One `/api/team/members` call however many issues are in hand. Best-effort:
+ * a directory that cannot be read still lets the issue(s) print their bare
+ * `assignee_id`, the pre-BLI-10450 behaviour, rather than failing the whole
+ * read over an enrichment nobody asked to gate on. Skips the call entirely
+ * when nobody in the batch is assigned.
+ */
+export async function resolveAssigneeDirectory(door, assigneeIds) {
+    const wanted = new Set(assigneeIds.filter((id) => Boolean(id)));
+    if (wanted.size === 0)
+        return new Map();
+    return buildDirectory(door, await fetchTeamDirectory(door), wanted);
+}
+/**
+ * BLI-10579: start the directory read now, before the assignee ids that will
+ * eventually filter it are even known — its network call never needed them.
+ * Pair with `finishAssigneeDirectory` once the caller's own request answers.
+ */
+export function beginAssigneeDirectoryFetch(door) {
+    return fetchTeamDirectory(door);
+}
+/**
+ * Always awaits the in-flight fetch, even when `wanted` turns out empty —
+ * it started eagerly, before that was knowable, so there is no unwaited
+ * background call left dangling past the command's exit.
+ */
+export async function finishAssigneeDirectory(door, fetchInFlight, assigneeIds) {
+    const wanted = new Set(assigneeIds.filter((id) => Boolean(id)));
+    return buildDirectory(door, await fetchInFlight, wanted);
+}
 /** `assignee_name`/`assignee_email` alongside the bare id, for JSON output. */
 export function withAssigneeIdentity(row, directory) {
     const identity = row.assignee_id ? directory.get(row.assignee_id) : undefined;

package/dist/commands/issue.js

@@ -27,7 +27,7 @@
  */
 import { askAgentDoor, emitAgentDoor, failAgentDoor, openAgentDoor, } from "./agent-door.js";
 import { writeLine } from "./cli-io.js";
-import { assigneeLabel, resolveAssigneeDirectory, withAssigneeIdentity } from "./issue-directory.js";
+import { assigneeLabel, beginAssigneeDirectoryFetch, finishAssigneeDirectory, withAssigneeIdentity } from "./issue-directory.js";
 import { showIssue } from "./issue-show.js";
 import { READ_DEADLINE_MS, TAG, resolveProjectId, sizeForEstimate, withSizeLabel, } from "./issue-contracts.js";
 import { commentIssue, createIssue, moveIssue, updateIssue } from "./issue-write.js";
@@ -105,14 +105,24 @@ async function listIssues(command, door) {
         query.set("project_id", resolved.id);
     }
     const suffix = query.toString();
+    // BLI-10579: the assignee directory (`/api/team/members`) is the whole org
+    // roster regardless of which issues come back — it never actually depended
+    // on the issue list — so start it now, alongside the issues request,
+    // instead of waiting for that answer first (measured ~580ms of the ~580ms
+    // `work list` p50 was exactly this serial second call).
+    const directoryFetch = beginAssigneeDirectoryFetch(door);
     const answer = await askAgentDoor(door, {
         path: `/api/work/issues${suffix ? `?${suffix}` : ""}`,
         method: "GET",
         label: "issue list",
         timeoutMs: READ_DEADLINE_MS,
     });
-    if (!answer.ok)
+    if (!answer.ok) {
+        // Not awaited for its own sake — `finishAssigneeDirectory` always awaits
+        // the in-flight fetch so it never leaks past this command's exit.
+        await finishAssigneeDirectory(door, directoryFetch, []);
         return failAgentDoor(door, TAG, answer.reason, answer.detail);
+    }
     const body = answer.body;
     const issues = body.issues ?? [];
     // BLI-10447: the gaps go to stderr in both modes, so `--json` stdout stays one object.
@@ -121,7 +131,7 @@ async function listIssues(command, door) {
     // BLI-10450: one extra call for the whole list, never one per row — the
     // uuid `assignee_id` alone forced a second `cockpit team members` call and
     // a join done by hand to answer "who owns what".
-    const directory = await resolveAssigneeDirectory(door, issues.map((issue) => issue.assignee_id));
+    const directory = await finishAssigneeDirectory(door, directoryFetch, issues.map((issue) => issue.assignee_id));
     if (door.json) {
         return emitAgentDoor(door, {
             ok: true,

package/dist/commands/public-root.js

@@ -21,7 +21,7 @@ export async function runCockpitCli(argv, io) {
   }
 
   if (command === "--version" || command === "-V" || command === "version") {
-    writeLine(io?.stdout ?? process.stdout, "0.2.145");
+    writeLine(io?.stdout ?? process.stdout, "0.2.146");
     return 0;
   }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/cli",
-  "version": "0.2.145",
+  "version": "0.2.146",
   "private": false,
   "type": "module",
   "bin": {