linear-sdk-cli 0.1.0 → 0.1.2

package/src/output/table.ts

@@ -148,14 +148,15 @@ export function selectPairs(
 export function renderTable<T>(
   rows: T[],
   columns: Column<T>[],
-  opts: { color?: boolean; empty?: string } = {},
+  opts: { color?: boolean } = {},
 ): string {
   const color = opts.color ?? false;
-  const colors = pc.createColors(color);
-  if (rows.length === 0) return colors.dim(sanitizeForTerminal(opts.empty ?? "(no results)"));
+  if (rows.length === 0) return color ? pc.dim("(no results)") : "(no results)";
 
   const headers = columns.map((c) => c.header ?? c.key);
-  const body = rows.map((row) => columns.map((c) => truncate(cell(c.value(row)), c.max ?? 60)));
+  const body = rows.map((row) =>
+    columns.map((c) => truncate(cell(c.value(row)), c.max ?? 60)),
+  );
 
   const widths = columns.map((_, i) => {
     const header = headers[i] ?? "";
@@ -166,7 +167,7 @@ export function renderTable<T>(
   const pad = (s: string, w: number) => s + " ".repeat(Math.max(0, w - displayWidth(s)));
 
   const headerLine = headers
-    .map((h, i) => pad(colors.bold(h), widths[i] ?? 0))
+    .map((h, i) => pad(color ? pc.bold(h) : h, widths[i] ?? 0))
     .join("  ")
     .trimEnd();
 
@@ -186,13 +187,12 @@ export function renderDetail(
   opts: { color?: boolean } = {},
 ): string {
   const color = opts.color ?? false;
-  const colors = pc.createColors(color);
   const visible = pairs.filter(([, v]) => v !== undefined && v !== null);
   const keyWidth = visible.reduce((m, [k]) => Math.max(m, k.length), 0);
   return visible
     .map(([k, v]) => {
       const label = (k + ":").padEnd(keyWidth + 1);
-      return `${colors.dim(label)} ${cell(v)}`;
+      return `${color ? pc.dim(label) : label} ${cell(v)}`;
     })
     .join("\n");
 }

package/src/services/agent-session.ts

@@ -14,7 +14,7 @@
 import type { LinearClient } from "@linear/sdk";
 import { withRetry } from "../client.js";
 import { shape } from "../lib/shape.js";
-import { collectRawQuery, hasMoreResults, setPaginationMetadata } from "../lib/pagination.js";
+import { collectRawQuery } from "../lib/pagination.js";
 import { notFound } from "../lib/errors.js";
 
 /** The statuses Linear reports for a session (AgentSessionStatus). */
@@ -35,11 +35,7 @@ export interface SessionUser {
   displayName: string;
 }
 
-const SESSION_USER_SHAPE = shape<SessionUser>({
-  id: "string",
-  name: "string",
-  displayName: "string",
-});
+const SESSION_USER_SHAPE = shape<SessionUser>({ id: "string", name: "string", displayName: "string" });
 
 export interface AgentSessionRow {
   id: string;
@@ -158,11 +154,7 @@ function finish(
   const filtered = opts.status ? present.filter((r) => r.status === opts.status) : present;
   // Newest first, whichever query found them (an issue's comments come oldest first).
   filtered.sort((a, b) => (a.createdAt < b.createdAt ? 1 : a.createdAt > b.createdAt ? -1 : 0));
-  if (limit === Infinity) return setPaginationMetadata(filtered, false);
-  return setPaginationMetadata(
-    filtered.slice(0, limit),
-    filtered.length > limit || (!opts.status && filtered.length >= limit && hasMoreResults(rows)),
-  );
+  return limit === Infinity ? filtered : filtered.slice(0, limit);
 }
 
 function toRow(s: any): AgentSessionRow {

package/src/services/attachment.ts

@@ -11,7 +11,7 @@
 import type { LinearClient } from "@linear/sdk";
 import { withRetry } from "../client.js";
 import { shape } from "../lib/shape.js";
-import { collect, inheritPaginationMetadata, pageSize } from "../lib/pagination.js";
+import { collect, pageSize } from "../lib/pagination.js";
 import { notFound, usageError } from "../lib/errors.js";
 import { assertMutation, unwrapMutation } from "../lib/mutation.js";
 import { resolveIssue } from "../lib/resolve.js";
@@ -65,7 +65,7 @@ export async function listAttachments(
   const issue = await resolveIssue(client, issueArg);
   const conn = await withRetry(() => issue.attachments({ first: pageSize(limit) }));
   const nodes = await collect(conn as any, limit);
-  return inheritPaginationMetadata(nodes.map(toRow), nodes);
+  return nodes.map(toRow);
 }
 
 export interface CreateOptions {
@@ -128,15 +128,9 @@ export async function attachFiles(
   issueArg: string,
   paths: string[],
   opts: AttachFilesOptions,
-): Promise<{
-  issue: { id: string; identifier: string };
-  attachments: AttachedFile[];
-  comment?: { id: string; url: string };
-}> {
+): Promise<{ issue: { id: string; identifier: string }; attachments: AttachedFile[]; comment?: { id: string; url: string } }> {
   if (opts.title !== undefined && paths.length > 1) {
-    throw usageError(
-      "--title names a single attachment; with several files, drop it to use each file's name.",
-    );
+    throw usageError("--title names a single attachment; with several files, drop it to use each file's name.");
   }
   validateUploads(paths, { public: opts.public });
   const issue = await resolveIssue(client, issueArg);
@@ -184,10 +178,7 @@ export async function attachFiles(
 export async function deleteAttachment(client: LinearClient, id: string) {
   const attachment = await withRetry(() => client.attachment(id));
   if (!attachment) throw notFound(`No attachment ${id}.`);
-  await assertMutation(
-    withRetry(() => client.deleteAttachment(id)),
-    "Attachment deletion",
-  );
+  await assertMutation(withRetry(() => client.deleteAttachment(id)), "Attachment deletion");
   return { id: attachment.id, title: attachment.title };
 }
 

package/src/services/comment.ts

@@ -210,12 +210,9 @@ export async function updateComment(
   current?: string,
 ) {
   if (body.trim() === "") {
-    throw usageError(
-      "Refusing to blank the comment body. To remove a comment, use 'comment delete'.",
-    );
+    throw usageError("Refusing to blank the comment body. To remove a comment, use 'comment delete'.");
   }
-  if (current !== undefined && body === current)
-    throw usageError("Comment body unchanged; nothing to update.");
+  if (current !== undefined && body === current) throw usageError("Comment body unchanged; nothing to update.");
   return unwrapMutation(
     withRetry(() => client.updateComment(commentId, { body })),
     "comment",
@@ -224,19 +221,14 @@ export async function updateComment(
 }
 
 export async function deleteComment(client: LinearClient, commentId: string) {
-  await assertMutation(
-    withRetry(() => client.deleteComment(commentId)),
-    "Comment deletion",
-  );
+  await assertMutation(withRetry(() => client.deleteComment(commentId)), "Comment deletion");
   return { id: commentId };
 }
 
 /** Resolve (or unresolve) a comment thread. */
 export async function setResolved(client: LinearClient, commentId: string, resolved: boolean) {
   return unwrapMutation(
-    withRetry(() =>
-      resolved ? client.commentResolve(commentId) : client.commentUnresolve(commentId),
-    ),
+    withRetry(() => (resolved ? client.commentResolve(commentId) : client.commentUnresolve(commentId))),
     "comment",
     `Comment ${resolved ? "resolve" : "unresolve"}`,
   );

package/src/services/cycle.ts

@@ -8,7 +8,7 @@
 import type { LinearClient } from "@linear/sdk";
 import { withRetry } from "../client.js";
 import { shape } from "../lib/shape.js";
-import { collect, pageSize, setPaginationMetadata } from "../lib/pagination.js";
+import { collect, pageSize } from "../lib/pagination.js";
 import { usageError, notFound } from "../lib/errors.js";
 import { unwrapMutation } from "../lib/mutation.js";
 import { resolveTeam, resolveCycleId, isUuid } from "../lib/resolve.js";
@@ -44,16 +44,11 @@ export async function listCycles(
   const team = await resolveTeam(client, teamInput, defaultTeamKey);
   const teamModel = await withRetry(() => client.team(team.id));
   const conn = await withRetry(() => teamModel.cycles({ first: pageSize(limit) }));
-  // Cycle connection order is not its user-facing number order. Sort first so
-  // `--limit` means the newest cycles, not an arbitrary API prefix.
-  const nodes = await collect(conn as any, Infinity);
+  const nodes = await collect(conn as any, limit);
   const rows = nodes.map(toRow);
   // Surface most-recent (highest number) cycle first.
   rows.sort((a, b) => b.number - a.number);
-  return setPaginationMetadata(
-    limit === Infinity ? rows : rows.slice(0, limit),
-    rows.length > limit,
-  );
+  return rows;
 }
 
 /** The team's currently active cycle, if any. */
@@ -179,9 +174,7 @@ async function resolveCycleArg(
   if (isUuid(idArg)) return idArg;
   const teamKey = teamInput ?? defaultTeamKey;
   if (!teamKey)
-    throw usageError(
-      "Resolving a cycle by number or 'current' requires --team (or pass a cycle id).",
-    );
+    throw usageError("Resolving a cycle by number or 'current' requires --team (or pass a cycle id).");
   const team = await resolveTeam(client, teamKey, undefined);
   return resolveCycleId(client, team.id, idArg);
 }

package/src/services/document.ts

@@ -19,7 +19,7 @@
 import type { LinearClient } from "@linear/sdk";
 import { withRetry } from "../client.js";
 import { collectRawQuery } from "../lib/pagination.js";
-import { usageError, notFound, normalizeError } from "../lib/errors.js";
+import { usageError, notFound } from "../lib/errors.js";
 import { assertMutation, unwrapMutation } from "../lib/mutation.js";
 import {
   resolveProjectId,
@@ -100,9 +100,7 @@ export async function resolveTarget(
     case "cycle": {
       const teamKey = sel.team ?? defaultTeamKey;
       if (!teamKey)
-        throw usageError(
-          "--cycle needs a team to look the cycle up in: pass --team <KEY> or set a default team.",
-        );
+        throw usageError("--cycle needs a team to look the cycle up in: pass --team <KEY> or set a default team.");
       const team = await resolveTeam(client, teamKey, undefined);
       return { kind: "cycle", id: await resolveCycleId(client, team.id, sel.value) };
     }
@@ -171,9 +169,7 @@ function targets(n: any): DocumentTargets {
     initiative: n.initiative ?? null,
     team: n.team ?? null,
     cycle: n.cycle ? { id: n.cycle.id, number: n.cycle.number, name: n.cycle.name ?? null } : null,
-    release: n.release
-      ? { id: n.release.id, name: n.release.name, version: n.release.version ?? null }
-      : null,
+    release: n.release ? { id: n.release.id, name: n.release.name, version: n.release.version ?? null } : null,
   };
 }
 
@@ -188,8 +184,7 @@ export function describeTarget(t: DocumentTargets): string | null {
   if (t.initiative) return `Initiative: ${t.initiative.name}`;
   if (t.team) return `Team: ${t.team.key} ${t.team.name}`;
   if (t.cycle) return `Cycle: #${t.cycle.number}${t.cycle.name ? ` ${t.cycle.name}` : ""}`;
-  if (t.release)
-    return `Release: ${t.release.name}${t.release.version ? ` (${t.release.version})` : ""}`;
+  if (t.release) return `Release: ${t.release.name}${t.release.version ? ` (${t.release.version})` : ""}`;
   return null;
 }
 
@@ -238,15 +233,10 @@ query CliDocumentDetail($id: String!) {
   }
 }`;
 
-export async function getDocumentDetail(
-  client: LinearClient,
-  idArg: string,
-): Promise<DocumentDetail> {
+export async function getDocumentDetail(client: LinearClient, idArg: string): Promise<DocumentDetail> {
   // An unknown id is an API error ("Could not find referenced Document"), which
   // the error boundary already reads as not-found (exit 3); a null is guarded too.
-  const data: any = await withRetry(() =>
-    (client as any).client.rawRequest(DETAIL_QUERY, { id: idArg }),
-  );
+  const data: any = await withRetry(() => (client as any).client.rawRequest(DETAIL_QUERY, { id: idArg }));
   const n = data?.data?.document;
   if (!n) throw notFound(`No document matching '${idArg}'.`);
   return {
@@ -277,11 +267,7 @@ export interface CreateOptions extends DocumentTargetOptions {
  * NOT a fallback here: the command decides that, so the service never turns
  * "no target" into "the team from config" behind the caller's back.
  */
-export async function createDocument(
-  client: LinearClient,
-  opts: CreateOptions,
-  defaultTeamKey?: string,
-) {
+export async function createDocument(client: LinearClient, opts: CreateOptions, defaultTeamKey?: string) {
   const sel = selectTarget(opts);
   if (!sel) throw usageError(`A document needs a target: pass one of ${TARGET_FLAGS}.`);
   const input: Record<string, any> = { title: opts.title };
@@ -332,10 +318,7 @@ export async function updateDocument(
 
 export async function deleteDocument(client: LinearClient, idArg: string) {
   const document = await resolveDocument(client, idArg);
-  await assertMutation(
-    withRetry(() => client.deleteDocument(document.id)),
-    "Document deletion",
-  );
+  await assertMutation(withRetry(() => client.deleteDocument(document.id)), "Document deletion");
   return document;
 }
 
@@ -349,9 +332,7 @@ async function resolveDocument(client: LinearClient, idArg: string) {
   // The SDK's document() also resolves a slugId; fall back to it directly.
   try {
     return await withRetry(() => client.document(idArg));
-  } catch (err) {
-    const normalized = normalizeError(err);
-    if (normalized.code === "not_found") throw notFound(`No document matching '${idArg}'.`);
-    throw normalized;
+  } catch {
+    throw notFound(`No document matching '${idArg}'.`);
   }
 }

package/src/services/favorite.ts

@@ -13,7 +13,7 @@
 import type { LinearClient } from "@linear/sdk";
 import { withRetry } from "../client.js";
 import { shape } from "../lib/shape.js";
-import { collect, inheritPaginationMetadata, pageSize } from "../lib/pagination.js";
+import { collect, pageSize } from "../lib/pagination.js";
 import { usageError } from "../lib/errors.js";
 import { assertMutation, unwrapMutation } from "../lib/mutation.js";
 import { resolveIssue, resolveProjectId, isUuid } from "../lib/resolve.js";
@@ -69,16 +69,13 @@ export function entityLabel(entity: any): string {
 export async function listFavorites(client: LinearClient, limit: number): Promise<FavoriteRow[]> {
   const conn = await withRetry(() => client.favorites({ first: pageSize(limit) }));
   const nodes = await collect(conn as any, limit);
-  return inheritPaginationMetadata(
-    await Promise.all(
-      nodes.map(async (f: any) => ({
-        id: f.id,
-        type: f.type,
-        name: await favoriteName(f),
-        url: f.url ?? null,
-      })),
-    ),
-    nodes,
+  return Promise.all(
+    nodes.map(async (f: any) => ({
+      id: f.id,
+      type: f.type,
+      name: await favoriteName(f),
+      url: f.url ?? null,
+    })),
   );
 }
 
@@ -139,9 +136,6 @@ export async function addFavorite(client: LinearClient, opts: AddOptions) {
 
 /** Delete a favorite by id. */
 export async function removeFavorite(client: LinearClient, id: string) {
-  await assertMutation(
-    withRetry(() => client.deleteFavorite(id)),
-    "Favorite removal",
-  );
+  await assertMutation(withRetry(() => client.deleteFavorite(id)), "Favorite removal");
   return { id };
 }

package/src/services/initiative-update.ts

@@ -7,7 +7,7 @@
 
 import type { LinearClient } from "@linear/sdk";
 import { withRetry } from "../client.js";
-import { collect, inheritPaginationMetadata, pageSize } from "../lib/pagination.js";
+import { collect } from "../lib/pagination.js";
 import { normalizeUpdatePayload, type Health, type UpdateRow } from "../lib/status-update.js";
 import { resolveInitiative } from "./initiative.js";
 
@@ -20,22 +20,21 @@ export async function listInitiativeUpdates(
   limit: number,
 ): Promise<UpdateRow[]> {
   const initiative = await resolveInitiative(client, idArg);
-  const conn = await withRetry(() => initiative.initiativeUpdates({ first: pageSize(limit) }));
+  const conn = await withRetry(() =>
+    initiative.initiativeUpdates({ first: Math.min(limit === Infinity ? 100 : limit, 100) }),
+  );
   const nodes = await collect(conn as any, limit);
-  return inheritPaginationMetadata(
-    await Promise.all(
-      nodes.map(async (u: any) => {
-        const user = await u.user;
-        return {
-          id: u.id,
-          createdAt: u.createdAt?.toISOString?.() ?? String(u.createdAt),
-          user: user?.displayName ?? "unknown",
-          body: u.body ?? "",
-          health: u.health ?? null,
-        };
-      }),
-    ),
-    nodes,
+  return Promise.all(
+    nodes.map(async (u: any) => {
+      const user = await u.user;
+      return {
+        id: u.id,
+        createdAt: u.createdAt?.toISOString?.() ?? String(u.createdAt),
+        user: user?.displayName ?? "unknown",
+        body: u.body ?? "",
+        health: u.health ?? null,
+      };
+    }),
   );
 }
 

package/src/services/initiative.ts

@@ -164,9 +164,7 @@ export const INITIATIVE_DETAIL_SHAPE = shape<InitiativeDetail>({
   archivedAt: "string|null",
   owner: "string|null",
   creator: "string|null",
-  projects: [
-    { id: "string", name: "string", status: { nullable: { name: "string", type: "string" } } },
-  ],
+  projects: [{ id: "string", name: "string", status: { nullable: { name: "string", type: "string" } } }],
 });
 
 /** How many linked projects `view` reads; more than this is unusual and shows a truncation note. */
@@ -189,11 +187,7 @@ export async function getInitiativeDetail(
   const projectRows = await Promise.all(
     (projects?.nodes ?? []).map(async (p: any) => {
       const status = await p.status;
-      return {
-        id: p.id,
-        name: p.name,
-        status: status ? { name: status.name, type: status.type } : null,
-      };
+      return { id: p.id, name: p.name, status: status ? { name: status.name, type: status.type } : null };
     }),
   );
   return {
@@ -263,7 +257,11 @@ export interface UpdateOptions {
   color?: string;
 }
 
-export async function updateInitiative(client: LinearClient, idArg: string, opts: UpdateOptions) {
+export async function updateInitiative(
+  client: LinearClient,
+  idArg: string,
+  opts: UpdateOptions,
+) {
   const initiative = await resolveInitiative(client, idArg);
   const input: Record<string, any> = {};
   if (opts.name !== undefined) input.name = opts.name;
@@ -422,11 +420,7 @@ export async function resolveInitiative(
   const matches: any[] = [];
   for (;;) {
     const conn = await withRetry(() =>
-      client.initiatives({
-        first: 100,
-        after,
-        includeArchived: opts.includeArchived === true,
-      } as any),
+      client.initiatives({ first: 100, after, includeArchived: opts.includeArchived === true } as any),
     );
     for (const n of conn.nodes) {
       if (n.name.toLowerCase() === lower) matches.push(n);
@@ -444,6 +438,7 @@ export async function resolveInitiative(
 export function resolveStatus(input: string): string {
   const lower = input.toLowerCase();
   const match = STATUSES.find((s) => s.toLowerCase() === lower);
-  if (!match) throw usageError(`Invalid status '${input}'. Valid: ${STATUSES.join(", ")}.`);
+  if (!match)
+    throw usageError(`Invalid status '${input}'. Valid: ${STATUSES.join(", ")}.`);
   return match;
 }