linear-sdk-cli 0.1.3 → 0.2.3

package/src/services/project.ts

@@ -8,7 +8,12 @@
 import type { LinearClient } from "@linear/sdk";
 import { withRetry } from "../client.js";
 import { shape } from "../lib/shape.js";
-import { collect, collectRawQuery } from "../lib/pagination.js";
+import {
+  collect,
+  collectRawQuery,
+  inheritPaginationMetadata,
+  pageSize,
+} from "../lib/pagination.js";
 import { usageError, notFound, ambiguous } from "../lib/errors.js";
 import { assertMutation, unwrapMutation } from "../lib/mutation.js";
 import {
@@ -206,7 +211,10 @@ query CliProjectDetail($filter: ProjectFilter!, $includeArchived: Boolean!) {
   }
 }`;
 
-export async function getProjectDetail(client: LinearClient, idArg: string): Promise<ProjectDetail> {
+export async function getProjectDetail(
+  client: LinearClient,
+  idArg: string,
+): Promise<ProjectDetail> {
   const byId = isUuid(idArg);
   const filter = byId ? { id: { eq: idArg } } : { name: { eqIgnoreCase: idArg } };
   const data: any = await withRetry(() =>
@@ -337,7 +345,10 @@ export async function updateProject(client: LinearClient, idArg: string, opts: U
 export async function archiveProject(client: LinearClient, idArg: string) {
   const projectId = await resolveProjectId(client, idArg);
   const project = await withRetry(() => client.project(projectId));
-  await assertMutation(withRetry(() => client.archiveProject(projectId)), "Project archive");
+  await assertMutation(
+    withRetry(() => client.archiveProject(projectId)),
+    "Project archive",
+  );
   return project;
 }
 
@@ -350,7 +361,10 @@ export async function archiveProject(client: LinearClient, idArg: string) {
 export async function deleteProject(client: LinearClient, idArg: string) {
   const projectId = await resolveProjectId(client, idArg);
   const project = await withRetry(() => client.project(projectId));
-  await assertMutation(withRetry(() => client.deleteProject(projectId)), "Project deletion");
+  await assertMutation(
+    withRetry(() => client.deleteProject(projectId)),
+    "Project deletion",
+  );
   return project;
 }
 
@@ -376,16 +390,17 @@ export async function listMilestones(
 ): Promise<MilestoneRow[]> {
   const projectId = await resolveProjectId(client, idArg);
   const project = await withRetry(() => client.project(projectId));
-  const conn = await withRetry(() =>
-    project.projectMilestones({ first: Math.min(limit === Infinity ? 100 : limit, 100) }),
-  );
+  const conn = await withRetry(() => project.projectMilestones({ first: pageSize(limit) }));
   const nodes = await collect(conn as any, limit);
-  return nodes.map((m: any) => ({
-    id: m.id,
-    name: m.name,
-    targetDate: m.targetDate ?? null,
-    progress: m.progress ?? null,
-  }));
+  return inheritPaginationMetadata(
+    nodes.map((m: any) => ({
+      id: m.id,
+      name: m.name,
+      targetDate: m.targetDate ?? null,
+      progress: m.progress ?? null,
+    })),
+    nodes,
+  );
 }
 
 export interface UpdateRow {
@@ -403,21 +418,22 @@ export async function listUpdates(
 ): Promise<UpdateRow[]> {
   const projectId = await resolveProjectId(client, idArg);
   const project = await withRetry(() => client.project(projectId));
-  const conn = await withRetry(() =>
-    project.projectUpdates({ first: Math.min(limit === Infinity ? 100 : limit, 100) }),
-  );
+  const conn = await withRetry(() => project.projectUpdates({ first: pageSize(limit) }));
   const nodes = await collect(conn as any, limit);
-  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,
-      };
-    }),
+  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,
   );
 }
 
@@ -457,17 +473,22 @@ async function resolveTeamIds(
 /** Resolve a project status (by name or type) to a status id. */
 async function resolveStatusId(client: LinearClient, input: string): Promise<string> {
   if (isUuid(input)) return input;
-  const statuses = await withRetry(() => client.projectStatuses({ first: 100 }));
+  const conn = await withRetry(() => client.projectStatuses({ first: 250 }));
+  const statuses = (await collect(conn as any, Infinity)) as Array<{
+    id: string;
+    name: string;
+    type: string;
+  }>;
   const lower = input.toLowerCase();
-  const byName = statuses.nodes.filter((s) => s.name.toLowerCase() === lower);
-  const matches = byName.length
-    ? byName
-    : statuses.nodes.filter((s) => s.type.toLowerCase() === lower);
+  const byName = statuses.filter((s) => s.name.toLowerCase() === lower);
+  const matches = byName.length ? byName : statuses.filter((s) => s.type.toLowerCase() === lower);
   if (matches.length === 0)
     throw notFound(
-      `No project status '${input}'. Available: ${statuses.nodes.map((s) => s.name).join(", ")}`,
+      `No project status '${input}'. Available: ${statuses.map((s) => s.name).join(", ")}`,
     );
   if (matches.length > 1)
-    throw ambiguous(`Multiple project statuses match '${input}': ${matches.map((s) => s.name).join(", ")}`);
+    throw ambiguous(
+      `Multiple project statuses match '${input}': ${matches.map((s) => s.name).join(", ")}`,
+    );
   return matches[0]!.id;
 }

package/src/services/roadmap.ts

@@ -13,7 +13,7 @@
 import type { LinearClient } from "@linear/sdk";
 import { withRetry } from "../client.js";
 import { shape } from "../lib/shape.js";
-import { collect } from "../lib/pagination.js";
+import { collect, inheritPaginationMetadata, pageSize } from "../lib/pagination.js";
 import { usageError, notFound, ambiguous } from "../lib/errors.js";
 import { assertMutation, unwrapMutation } from "../lib/mutation.js";
 import { resolveUserId, isUuid } from "../lib/resolve.js";
@@ -44,11 +44,9 @@ export function toRow(r: any): RoadmapRow {
 }
 
 export async function listRoadmaps(client: LinearClient, limit: number): Promise<RoadmapRow[]> {
-  const conn = await withRetry(() =>
-    client.roadmaps({ first: limit === Infinity ? 100 : Math.min(limit, 100) }),
-  );
+  const conn = await withRetry(() => client.roadmaps({ first: pageSize(limit) }));
   const nodes = await collect(conn as any, limit);
-  return nodes.map(toRow);
+  return inheritPaginationMetadata(nodes.map(toRow), nodes);
 }
 
 export interface RoadmapDetail {
@@ -153,7 +151,10 @@ export async function updateRoadmap(client: LinearClient, idArg: string, opts: U
 
 export async function deleteRoadmap(client: LinearClient, idArg: string) {
   const roadmap = await resolveRoadmap(client, idArg);
-  await assertMutation(withRetry(() => client.deleteRoadmap(roadmap.id)), "Roadmap deletion");
+  await assertMutation(
+    withRetry(() => client.deleteRoadmap(roadmap.id)),
+    "Roadmap deletion",
+  );
   return roadmap;
 }
 

package/src/services/state.ts

@@ -10,7 +10,7 @@
 import type { LinearClient } from "@linear/sdk";
 import { withRetry } from "../client.js";
 import { shape } from "../lib/shape.js";
-import { collect, pageSize } from "../lib/pagination.js";
+import { collect, pageSize, setPaginationMetadata } from "../lib/pagination.js";
 import { usageError } from "../lib/errors.js";
 import { resolveTeam, resolveStateId, isUuid } from "../lib/resolve.js";
 
@@ -41,8 +41,14 @@ export async function listStates(
   const team = await resolveTeam(client, teamInput, defaultTeamKey);
   const teamModel = await withRetry(() => client.team(team.id));
   const conn = await withRetry(() => teamModel.states({ first: pageSize(limit) }));
-  const nodes = await collect(conn as any, limit);
-  return nodes.map(toRow).sort((a, b) => a.position - b.position);
+  // The API's connection order is unrelated to workflow position. Sort the
+  // complete small state set, then apply the caller's limit.
+  const nodes = await collect(conn as any, Infinity);
+  const rows = nodes.map(toRow).sort((a, b) => a.position - b.position);
+  return setPaginationMetadata(
+    limit === Infinity ? rows : rows.slice(0, limit),
+    rows.length > limit,
+  );
 }
 
 export interface StateDetail {

package/src/services/team.ts

@@ -9,7 +9,12 @@
 import type { LinearClient } from "@linear/sdk";
 import { withRetry } from "../client.js";
 import { shape } from "../lib/shape.js";
-import { collect, pageSize } from "../lib/pagination.js";
+import {
+  collect,
+  inheritPaginationMetadata,
+  pageSize,
+  setPaginationMetadata,
+} from "../lib/pagination.js";
 import { usageError } from "../lib/errors.js";
 import { assertMutation, unwrapMutation } from "../lib/mutation.js";
 import { resolveTeam, type ResolvedTeam } from "../lib/resolve.js";
@@ -23,11 +28,23 @@ export interface TeamRow {
 /** The row's shape as `linear commands` advertises it (TES-610); checked against the interface. */
 export const TEAM_ROW_SHAPE = shape<TeamRow>({ id: "string", key: "string", name: "string" });
 
+/** Resolve the selected team to its canonical key, UUID, and name. */
+export async function getTeamIdentity(
+  client: LinearClient,
+  keyArg: string | undefined,
+  defaultTeamKey: string | undefined,
+): Promise<ResolvedTeam> {
+  return resolveTeam(client, keyArg, defaultTeamKey);
+}
+
 /** All teams (key, name, id). */
 export async function listTeams(client: LinearClient, limit: number): Promise<TeamRow[]> {
-  const conn = await withRetry(() => client.teams({ first: limit === Infinity ? 100 : Math.min(limit, 250) }));
+  const conn = await withRetry(() => client.teams({ first: pageSize(limit) }));
   const nodes = await collect(conn as any, limit);
-  return nodes.map((t: any) => ({ id: t.id, key: t.key, name: t.name }));
+  return inheritPaginationMetadata(
+    nodes.map((t: any) => ({ id: t.id, key: t.key, name: t.name })),
+    nodes,
+  );
 }
 
 export interface TeamDetail {
@@ -128,13 +145,16 @@ export async function listMembers(
     (await withRetry(() => team.members({ first: pageSize(limit), includeDisabled }))) as any,
     limit,
   );
-  return nodes.map((u: any) => ({
-    id: u.id,
-    displayName: u.displayName,
-    name: u.name,
-    email: u.email,
-    active: !!u.active,
-  }));
+  return inheritPaginationMetadata(
+    nodes.map((u: any) => ({
+      id: u.id,
+      displayName: u.displayName,
+      name: u.name,
+      email: u.email,
+      active: !!u.active,
+    })),
+    nodes,
+  );
 }
 
 export interface StateRow {
@@ -161,8 +181,8 @@ export async function listStates(
 ): Promise<StateRow[]> {
   const resolved = await resolveTeam(client, keyArg, defaultTeamKey);
   const team = await withRetry(() => client.team(resolved.id));
-  const nodes = await collect((await withRetry(() => team.states())) as any, limit);
-  return nodes
+  const nodes = await collect((await withRetry(() => team.states())) as any, Infinity);
+  const rows = nodes
     .map((s: any) => ({
       id: s.id,
       name: s.name,
@@ -171,6 +191,10 @@ export async function listStates(
       position: s.position,
     }))
     .sort((a: StateRow, b: StateRow) => a.position - b.position);
+  return setPaginationMetadata(
+    limit === Infinity ? rows : rows.slice(0, limit),
+    rows.length > limit,
+  );
 }
 
 export interface LabelRow {
@@ -179,7 +203,11 @@ export interface LabelRow {
   color: string;
 }
 
-export const TEAM_LABEL_ROW_SHAPE = shape<LabelRow>({ id: "string", name: "string", color: "string" });
+export const TEAM_LABEL_ROW_SHAPE = shape<LabelRow>({
+  id: "string",
+  name: "string",
+  color: "string",
+});
 
 export async function listLabels(
   client: LinearClient,
@@ -190,7 +218,10 @@ export async function listLabels(
   const resolved = await resolveTeam(client, keyArg, defaultTeamKey);
   const team = await withRetry(() => client.team(resolved.id));
   const nodes = await collect((await withRetry(() => team.labels())) as any, limit);
-  return nodes.map((l: any) => ({ id: l.id, name: l.name, color: l.color }));
+  return inheritPaginationMetadata(
+    nodes.map((l: any) => ({ id: l.id, name: l.name, color: l.color })),
+    nodes,
+  );
 }
 
 export interface CycleRow {
@@ -217,14 +248,20 @@ export async function listCycles(
 ): Promise<CycleRow[]> {
   const resolved = await resolveTeam(client, keyArg, defaultTeamKey);
   const team = await withRetry(() => client.team(resolved.id));
-  const nodes = await collect((await withRetry(() => team.cycles())) as any, limit);
-  return nodes.map((c: any) => ({
-    id: c.id,
-    number: c.number,
-    name: c.name ?? null,
-    startsAt: c.startsAt?.toISOString?.() ?? (c.startsAt ? String(c.startsAt) : null),
-    endsAt: c.endsAt?.toISOString?.() ?? (c.endsAt ? String(c.endsAt) : null),
-  }));
+  const nodes = await collect((await withRetry(() => team.cycles())) as any, Infinity);
+  const rows = nodes
+    .map((c: any) => ({
+      id: c.id,
+      number: c.number,
+      name: c.name ?? null,
+      startsAt: c.startsAt?.toISOString?.() ?? (c.startsAt ? String(c.startsAt) : null),
+      endsAt: c.endsAt?.toISOString?.() ?? (c.endsAt ? String(c.endsAt) : null),
+    }))
+    .sort((a, b) => b.number - a.number);
+  return setPaginationMetadata(
+    limit === Infinity ? rows : rows.slice(0, limit),
+    rows.length > limit,
+  );
 }
 
 export interface CreateTeamOptions {
@@ -321,7 +358,10 @@ export async function moveTeamIssues(
   to: ResolvedTeam,
 ): Promise<number> {
   const team = await withRetry(() => client.team(from.id));
-  const issues = await collect((await withRetry(() => team.issues({ first: 100 }))) as any, Infinity);
+  const issues = await collect(
+    (await withRetry(() => team.issues({ first: 100 }))) as any,
+    Infinity,
+  );
   const ids = issues.map((i: any) => i.id as string);
   for (let i = 0; i < ids.length; i += MOVE_BATCH) {
     const batch = ids.slice(i, i + MOVE_BATCH);
@@ -339,5 +379,8 @@ export async function moveTeamIssues(
  * user as `feature_not_accessible` like every other plan gate.
  */
 export async function deleteTeam(client: LinearClient, team: ResolvedTeam): Promise<void> {
-  await assertMutation(withRetry(() => client.deleteTeam(team.id)), "Team deletion");
+  await assertMutation(
+    withRetry(() => client.deleteTeam(team.id)),
+    "Team deletion",
+  );
 }

package/src/services/user.ts

@@ -9,7 +9,7 @@
 import type { LinearClient } from "@linear/sdk";
 import { withRetry } from "../client.js";
 import { shape } from "../lib/shape.js";
-import { collect, pageSize } from "../lib/pagination.js";
+import { collect, inheritPaginationMetadata, pageSize } from "../lib/pagination.js";
 import { resolveUserId } from "../lib/resolve.js";
 
 export interface UserRow {
@@ -45,7 +45,7 @@ export async function listUsers(
 ): Promise<UserRow[]> {
   const conn = await withRetry(() => client.users({ first: pageSize(limit), includeDisabled }));
   const nodes = await collect(conn as any, limit);
-  return nodes.map(toRow);
+  return inheritPaginationMetadata(nodes.map(toRow), nodes);
 }
 
 export interface UserDetail {

package/src/services/webhook.ts

@@ -9,8 +9,8 @@
 import { type LinearClient, WebhookResourceType } from "@linear/sdk";
 import { withRetry } from "../client.js";
 import { shape } from "../lib/shape.js";
-import { collect } from "../lib/pagination.js";
-import { usageError, notFound } from "../lib/errors.js";
+import { collect, inheritPaginationMetadata, pageSize } from "../lib/pagination.js";
+import { usageError, notFound, normalizeError } from "../lib/errors.js";
 import { assertMutation, unwrapMutation } from "../lib/mutation.js";
 import { resolveTeam } from "../lib/resolve.js";
 
@@ -56,17 +56,18 @@ export const WEBHOOK_ROW_SHAPE = shape<WebhookRow>({
 
 /** All webhooks in the workspace (relation-light, so no N+1). */
 export async function listWebhooks(client: LinearClient, limit: number): Promise<WebhookRow[]> {
-  const conn = await withRetry(() =>
-    client.webhooks({ first: limit === Infinity ? 100 : Math.min(limit, 100) }),
-  );
+  const conn = await withRetry(() => client.webhooks({ first: pageSize(limit) }));
   const nodes = await collect(conn as any, limit);
-  return nodes.map((w: any) => ({
-    id: w.id,
-    url: w.url ?? null,
-    enabled: w.enabled,
-    resourceTypes: w.resourceTypes ?? [],
-    label: w.label ?? null,
-  }));
+  return inheritPaginationMetadata(
+    nodes.map((w: any) => ({
+      id: w.id,
+      url: w.url ?? null,
+      enabled: w.enabled,
+      resourceTypes: w.resourceTypes ?? [],
+      label: w.label ?? null,
+    })),
+    nodes,
+  );
 }
 
 export interface WebhookDetail {
@@ -174,7 +175,9 @@ export async function updateWebhook(client: LinearClient, id: string, opts: Upda
   if (opts.secret !== undefined) input.secret = opts.secret;
 
   if (Object.keys(input).length === 0)
-    throw usageError("Nothing to update; pass at least one of --url, --enabled/--disabled, --resource.");
+    throw usageError(
+      "Nothing to update; pass at least one of --url, --enabled/--disabled, --resource.",
+    );
 
   return unwrapMutation(
     withRetry(() => client.updateWebhook(id, input as any)),
@@ -184,9 +187,17 @@ export async function updateWebhook(client: LinearClient, id: string, opts: Upda
 }
 
 export async function deleteWebhook(client: LinearClient, id: string): Promise<WebhookDetail> {
-  const webhook = await getWebhookDetail(client, id).catch(() => {
-    throw notFound(`No webhook matching '${id}'.`);
-  });
-  await assertMutation(withRetry(() => client.deleteWebhook(id)), "Webhook deletion");
+  let webhook: WebhookDetail;
+  try {
+    webhook = await getWebhookDetail(client, id);
+  } catch (err) {
+    const normalized = normalizeError(err);
+    if (normalized.code === "not_found") throw notFound(`No webhook matching '${id}'.`);
+    throw normalized;
+  }
+  await assertMutation(
+    withRetry(() => client.deleteWebhook(id)),
+    "Webhook deletion",
+  );
   return webhook;
 }