@bli-cockpit/mcp 0.1.65 → 0.1.66

package/dist/work-issue-rows.d.ts

@@ -0,0 +1,94 @@
+/**
+ * The row shapes the `work_*` tools read off `/api/work/**`, and the pure
+ * functions that turn them into what a tool answers with: the trimmed list
+ * row, the completeness and outside-project notes, one issue's text, one
+ * history line.
+ *
+ * Split out of `work-tools.ts` (BLI-10445) when that file crossed the
+ * 500-line readability ratchet. The seam: `work-tools.ts` registers the tools
+ * and calls the doors; nothing here calls a door or knows a session, so this
+ * module changes when a door's row shape or a rendered sentence does, and can
+ * be read without the registration code around it.
+ *
+ * Every string moved verbatim.
+ */
+import { type AssigneeIdentity } from "./work-directory.js";
+export interface IssueRow {
+    id: string;
+    identifier: string;
+    title: string;
+    state: string;
+    priority?: number | null;
+    assignee_id?: string | null;
+    project_id?: string | null;
+    parent_id?: string | null;
+    updated_at?: string | null;
+}
+/**
+ * The fields `work_list_issues` promises. The door's row carries the whole
+ * description and a dozen bookkeeping columns; with no limit that was 1,000
+ * issues and 4.2 MB, 3.3 MB of it descriptions the tool said it never
+ * returned (BLI-10432). work_get_issue reads one issue in full.
+ */
+export interface IssueListRow {
+    id: string;
+    identifier: string;
+    title: string;
+    state: string;
+    priority: number | null;
+    assignee_id: string | null;
+    project_id: string | null;
+    parent_id: string | null;
+    updated_at: string | null;
+}
+export declare function issueListRow(row: IssueRow): IssueListRow;
+export interface ListCompleteness {
+    status: "complete" | "partial" | "unknown";
+    total: number | null;
+}
+export declare function listCompleteness(value: unknown): ListCompleteness | null;
+export interface OutsideProject {
+    status: string;
+    issues: {
+        identifier: string;
+        title: string;
+        project_id: string | null;
+        parent_identifier: string | null;
+    }[];
+}
+export declare function outsideProject(value: unknown): OutsideProject | null;
+/** BLI-10447: sub-issues of the listed issues that live in no project or another one. */
+export declare function outsideNote(outside: OutsideProject | null): string;
+export interface OneIssueOk {
+    ref: string;
+    ok: true;
+    issue?: IssueRow & {
+        description?: string | null;
+    };
+    comments: Array<{
+        body_markdown?: string;
+        created_at?: string;
+    }>;
+    commentsUnread?: boolean;
+}
+export interface OneIssueFailure {
+    ref: string;
+    ok: false;
+    text: string;
+}
+export declare function renderOneIssue(result: OneIssueOk, directory: Map<string, AssigneeIdentity>): string;
+export interface HistoryRow {
+    occurred_at?: string;
+    change_type?: string;
+    from_value?: string | null;
+    to_value?: string | null;
+    actor_name?: string | null;
+    source?: string;
+}
+/**
+ * A row the Linear import could not give state names to says which it is.
+ * Printing "(none) -> (none)" is honest and useless; naming the gap keeps
+ * nobody reading a real gap as a broken tool (`commands/issue.ts` says the
+ * same thing in the same words).
+ */
+export declare function historyChange(row: HistoryRow): string;

package/dist/work-issue-rows.js

@@ -0,0 +1,75 @@
+/**
+ * The row shapes the `work_*` tools read off `/api/work/**`, and the pure
+ * functions that turn them into what a tool answers with: the trimmed list
+ * row, the completeness and outside-project notes, one issue's text, one
+ * history line.
+ *
+ * Split out of `work-tools.ts` (BLI-10445) when that file crossed the
+ * 500-line readability ratchet. The seam: `work-tools.ts` registers the tools
+ * and calls the doors; nothing here calls a door or knows a session, so this
+ * module changes when a door's row shape or a rendered sentence does, and can
+ * be read without the registration code around it.
+ *
+ * Every string moved verbatim.
+ */
+import { assigneeLabel } from "./work-directory.js";
+export function issueListRow(row) {
+    return {
+        id: row.id,
+        identifier: row.identifier,
+        title: row.title,
+        state: row.state,
+        priority: row.priority ?? null,
+        assignee_id: row.assignee_id ?? null,
+        project_id: row.project_id ?? null,
+        parent_id: row.parent_id ?? null,
+        updated_at: row.updated_at ?? null,
+    };
+}
+export function listCompleteness(value) {
+    if (!value || typeof value !== "object")
+        return null;
+    const record = value;
+    const status = record["status"];
+    if (status !== "complete" && status !== "partial" && status !== "unknown")
+        return null;
+    return { status, total: typeof record["total"] === "number" ? record["total"] : null };
+}
+export function outsideProject(value) {
+    if (!value || typeof value !== "object")
+        return null;
+    const record = value;
+    if (!Array.isArray(record["issues"]))
+        return null;
+    return { status: String(record["status"] ?? "complete"), issues: record["issues"] };
+}
+/** BLI-10447: sub-issues of the listed issues that live in no project or another one. */
+export function outsideNote(outside) {
+    if (!outside)
+        return "";
+    const named = outside.issues.map((issue) => `${issue.identifier} (under ${issue.parent_identifier ?? "?"}, ${issue.project_id ? `project ${issue.project_id}` : "no project"})`);
+    const found = named.length > 0 ? ` ${named.length} sub-issue(s) of these sit outside the project: ${named.join(", ")}.` : "";
+    const gap = outside.status === "failed" ? " The outside-project check failed (read_failed)."
+        : outside.status === "capped" ? " The outside-project check was capped; more may exist." : "";
+    return found + gap;
+}
+export function renderOneIssue(result, directory) {
+    const { issue, comments } = result;
+    const commentText = comments.map((comment) => `[${comment.created_at ?? ""}] ${comment.body_markdown ?? ""}`).join("\n\n");
+    return `${issue?.identifier ?? result.ref} ${issue?.title ?? ""} (${issue?.state ?? "?"})  ·  assignee ${assigneeLabel(issue?.assignee_id, directory)}`
+        + `\n\n${issue?.description ?? "(no description)"}`
+        + (commentText ? `\n\n--- comments ---\n${commentText}` : "")
+        + (result.commentsUnread ? "\n\n(comments could not be read on this call)" : "");
+}
+/**
+ * A row the Linear import could not give state names to says which it is.
+ * Printing "(none) -> (none)" is honest and useless; naming the gap keeps
+ * nobody reading a real gap as a broken tool (`commands/issue.ts` says the
+ * same thing in the same words).
+ */
+export function historyChange(row) {
+    if (row.from_value === null && row.to_value === null) {
+        return `(no state names recorded — imported from ${row.source ?? "elsewhere"})`;
+    }
+    return `${row.from_value ?? "(none)"} -> ${row.to_value ?? "(none)"}`;
+}

package/dist/work-tools.js

@@ -18,6 +18,7 @@ import { z } from "zod";
 import { callAgentDoor } from "./agent-door.js";
 import { DEFAULT_LIST_LIMIT, doorFailureText, errorResult, moreNote, registrarFor, textResult, withSession, } from "./tool-result.js";
 import { assigneeLabel, resolveAssigneeDirectory, withAssigneeIdentity } from "./work-directory.js";
+import { historyChange, issueListRow, listCompleteness, outsideNote, outsideProject, renderOneIssue, } from "./work-issue-rows.js";
 /** The closed state vocabulary, verbatim from `lib/work/api-doors.ts` `WORK_ISSUE_STATES`. */
 export const WORK_ISSUE_STATES = [
     "backlog",
@@ -29,46 +30,6 @@ export const WORK_ISSUE_STATES = [
 ];
 /** `/api/work/issues`' own ceiling, and the most a caller may ask for here. */
 const ISSUE_LIST_MAX = 1000;
-function issueListRow(row) {
-    return {
-        id: row.id,
-        identifier: row.identifier,
-        title: row.title,
-        state: row.state,
-        priority: row.priority ?? null,
-        assignee_id: row.assignee_id ?? null,
-        project_id: row.project_id ?? null,
-        parent_id: row.parent_id ?? null,
-        updated_at: row.updated_at ?? null,
-    };
-}
-function listCompleteness(value) {
-    if (!value || typeof value !== "object")
-        return null;
-    const record = value;
-    const status = record["status"];
-    if (status !== "complete" && status !== "partial" && status !== "unknown")
-        return null;
-    return { status, total: typeof record["total"] === "number" ? record["total"] : null };
-}
-function outsideProject(value) {
-    if (!value || typeof value !== "object")
-        return null;
-    const record = value;
-    if (!Array.isArray(record["issues"]))
-        return null;
-    return { status: String(record["status"] ?? "complete"), issues: record["issues"] };
-}
-/** BLI-10447: sub-issues of the listed issues that live in no project or another one. */
-function outsideNote(outside) {
-    if (!outside)
-        return "";
-    const named = outside.issues.map((issue) => `${issue.identifier} (under ${issue.parent_identifier ?? "?"}, ${issue.project_id ? `project ${issue.project_id}` : "no project"})`);
-    const found = named.length > 0 ? ` ${named.length} sub-issue(s) of these sit outside the project: ${named.join(", ")}.` : "";
-    const gap = outside.status === "failed" ? " The outside-project check failed (read_failed)."
-        : outside.status === "capped" ? " The outside-project check was capped; more may exist." : "";
-    return found + gap;
-}
 /** One issue plus its comments — a comment read that fails does not lose the issue already in hand. */
 async function fetchOneIssue(deps, session, ref) {
     const response = await callAgentDoor(session, deps.fetchImpl, "GET", `/api/work/issues/${encodeURIComponent(ref)}`);
@@ -87,26 +48,6 @@ async function fetchOneIssue(deps, session, ref) {
         ...(commentsResponse.ok ? {} : { commentsUnread: true }),
     };
 }
-function renderOneIssue(result, directory) {
-    const { issue, comments } = result;
-    const commentText = comments.map((comment) => `[${comment.created_at ?? ""}] ${comment.body_markdown ?? ""}`).join("\n\n");
-    return `${issue?.identifier ?? result.ref} ${issue?.title ?? ""} (${issue?.state ?? "?"})  ·  assignee ${assigneeLabel(issue?.assignee_id, directory)}`
-        + `\n\n${issue?.description ?? "(no description)"}`
-        + (commentText ? `\n\n--- comments ---\n${commentText}` : "")
-        + (result.commentsUnread ? "\n\n(comments could not be read on this call)" : "");
-}
-/**
- * A row the Linear import could not give state names to says which it is.
- * Printing "(none) -> (none)" is honest and useless; naming the gap keeps
- * nobody reading a real gap as a broken tool (`commands/issue.ts` says the
- * same thing in the same words).
- */
-function historyChange(row) {
-    if (row.from_value === null && row.to_value === null) {
-        return `(no state names recorded — imported from ${row.source ?? "elsewhere"})`;
-    }
-    return `${row.from_value ?? "(none)"} -> ${row.to_value ?? "(none)"}`;
-}
 /** A project id from a project NAME or a uuid — exact after case-folding, never fuzzy. */
 async function resolveProjectId(deps, session, ref) {
     const response = await callAgentDoor(session, deps.fetchImpl, "GET", "/api/work/projects?include_archived=true");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bli-cockpit/mcp",
-  "version": "0.1.65",
+  "version": "0.1.66",
   "private": false,
   "description": "bli-tower: an MCP server over BLI Cockpit's agent doors: JARVIS (jarvis_*), documents (docs_*), channels (msg_*), issues (work_*), the daily page (brief_*), meeting notes (notes_*), the ops board (ops_status/slack_*), settings/team/model, Scout and the workbook, plus the legacy event-stream tools (emit_event, get_ticket_timeline, get_active_tickets).",
   "type": "module",