@clipboard-health/groundcrew 4.2.0 → 4.2.1

package/dist/lib/{boardSource.js → adapters/linear/fetch.js}

@@ -1,19 +1,16 @@
 /**
- * Linear adapter — turns the viewer's GraphQL state into a `BoardState`
- * snapshot. Owns the GraphQL queries and shape parsing so callers consume a
- * typed `BoardState` instead of raw nodes.
+ * Linear adapter — GraphQL fetch helpers for board/issue data.
  *
- * There is no project / view / status configuration: the only filter is
- * "assigned to the API key's viewer AND carries an `agent-*` label."
- * State classification is driven by Linear's workflow `state.type`
+ * There is no project / view / status configuration: the only server-side
+ * filter is "assigned to the API key's viewer AND carries an `agent-*`
+ * label." State classification is driven by Linear's workflow `state.type`
  * (`unstarted` | `started` | `completed` | `canceled` | `duplicate`) —
- * never by status name — so workspaces with renamed columns (Todo → To Do,
- * Done → Shipped, etc.) Just Work.
+ * never by status name — so workspaces with renamed columns (Todo -> To Do,
+ * Done -> Shipped, etc.) Just Work without per-team config.
  */
-import { AGENT_ANY_MODEL, isShippedDefaultDisabled } from "./config.js";
-import { RepositoryResolutionError } from "./ticketSource.js";
-import { log } from "./util.js";
-export const AGENT_LABEL_PREFIX = "agent-";
+import { RepositoryResolutionError } from "../../ticketSource.js";
+import { log } from "../../util.js";
+import { AGENT_LABEL_PREFIX, resolveModelFor, resolveRepositoryFor, } from "./parsing.js";
 export const ISSUES_PAGE_SIZE = 250;
 // `state.type` values surfaced by `fetch()`. `backlog` / `triage` are dropped
 // at the GraphQL filter; everything else is post-classified by these names.
@@ -24,14 +21,6 @@ const ACTIONABLE_STATE_TYPES = [
     "canceled",
     "duplicate",
 ];
-export function isGroundcrewIssue(issue) {
-    return issue.model !== undefined && issue.repository !== undefined;
-}
-// Canonical RepositoryResolutionError lives in ./ticketSource.ts (imported at
-// the top of this file). Re-exported here so existing consumers of
-// boardSource.ts keep compiling until a follow-up PR completes the consumer
-// refactor and deletes this file.
-export { RepositoryResolutionError };
 export function createBoardSource(deps) {
     const { config, client } = deps;
     return {
@@ -75,18 +64,6 @@ export function isTerminalStatusForBlocker(blocker) {
 async function fetchBoard(client, config) {
     const nodes = [];
     let after = null;
-    // Three server-side filters narrow the response to tickets the orchestrator
-    // can actually act on:
-    //   1. Assignee: the API key's own viewer. groundcrew is a single-user
-    //      orchestrator — every ticket it dispatches is "this user's work."
-    //   2. Label: at least one `agent-*` label — i.e. the user opted the
-    //      ticket in to groundcrew. Without this, every human-owned ticket
-    //      would round-trip back just to be filtered out client-side.
-    //   3. State type: scoped to actionable values (`unstarted`, `started`,
-    //      `completed`, `canceled`, `duplicate`) so backlog/triage tickets never
-    //      make it into the page.
-    // The client-side `isGroundcrewIssue` guard in dispatcher.ts is
-    // belt-and-suspenders against query drift, not the load-bearing filter.
     const stateTypes = [...ACTIONABLE_STATE_TYPES];
     for (;;) {
         // oxlint-disable-next-line no-await-in-loop -- pagination cursor depends on the previous response
@@ -163,14 +140,14 @@ export function modelForResolution(resolution) {
     if (resolution.kind === "disabled-fallback") {
         return resolution.fallbackModel;
     }
-    return AGENT_ANY_MODEL;
+    return "any";
 }
-export function resolveTodoAgentMetadata(arguments_) {
+function resolveTodoAgentMetadata(arguments_) {
     const { ticket, description, modelResolution, config, isTodo } = arguments_;
     let repository;
     let model;
     if (modelResolution.kind !== "no-label" && isTodo) {
-        const resolution = resolveRepositoryFor({ description, config, ticket });
+        const resolution = resolveRepositoryFor({ description, config });
         if (resolution.kind === "ok") {
             ({ repository } = resolution);
             model = modelForResolution(modelResolution);
@@ -186,6 +163,7 @@ function buildLinearIssue(input) {
         id: input.identifier.toLowerCase(),
         uuid: input.uuid,
         title: input.title,
+        description: input.description,
         status: input.status,
         statusId: input.statusId,
         stateType: input.stateType,
@@ -202,11 +180,6 @@ function buildLinearIssue(input) {
 function issueFromNode(node, config) {
     const modelResolution = resolveModelFor({ labels: node.labels.nodes, config });
     warnIfDisabledFallback(node.identifier, modelResolution, config);
-    // Only the dispatcher reads `Issue.repository` / `Issue.model`, and only on
-    // tickets in the Todo column it's about to pick up. Resolving them for In
-    // Progress (already running) or Done (cleaner only needs the id) would just
-    // invite tick-spam warnings on already-finished tickets — e.g. when a
-    // description was edited or knownRepositories changed after dispatch.
     const { repository, model } = resolveTodoAgentMetadata({
         ticket: node.identifier,
         /* v8 ignore next @preserve -- BoardIssues query selects description; the ?? guard normalises a null vs undefined edge */
@@ -219,6 +192,8 @@ function issueFromNode(node, config) {
         identifier: node.identifier,
         uuid: node.id,
         title: node.title,
+        /* v8 ignore next @preserve -- BoardIssues query always selects description; this `?? ""` is a defensive null vs undefined edge */
+        description: node.description ?? "",
         /* v8 ignore next @preserve -- BoardIssues query always returns state */
         status: node.state?.name ?? "Unknown",
         /* v8 ignore next @preserve -- BoardIssues query always returns state */
@@ -233,23 +208,6 @@ function issueFromNode(node, config) {
         inverseRelations: node.inverseRelations,
     });
 }
-function escapeRegex(value) {
-    return value.replaceAll(/[$()*+.?[\\\]^{|}]/g, String.raw `\$&`);
-}
-// Sort by descending length so longer names match first — `api-admin`
-// must beat `api` when both are configured. `\b` treats `-` as a word
-// boundary, so without this ordering `api` would win on `api-admin`.
-function buildRepositoryRegex(config) {
-    const candidates = config.workspace.knownRepositories.flatMap((repo) => {
-        const slashIndex = repo.indexOf("/");
-        return slashIndex === -1 ? [repo] : [repo, repo.slice(slashIndex + 1)];
-    });
-    const alternation = candidates
-        .toSorted((a, b) => b.length - a.length)
-        .map(escapeRegex)
-        .join("|");
-    return new RegExp(String.raw `\b(${alternation})\b`);
-}
 const ISSUE_LABEL_PAGE_SIZE = 50;
 const ISSUE_RELATION_PAGE_SIZE = 50;
 export async function fetchBlockersForTicket(arguments_) {
@@ -294,7 +252,7 @@ export async function fetchRawLinearIssue(arguments_) {
         title
         description
         team { id }
-        state { name type }
+        state { id name type }
         children { nodes { id } }
         labels(first: ${ISSUE_LABEL_PAGE_SIZE}) {
           nodes { name }
@@ -328,6 +286,8 @@ export async function fetchRawLinearIssue(arguments_) {
         stateName: issue.state?.name ?? "",
         /* v8 ignore next @preserve -- ResolveIssue query selects state; null only if Linear genuinely returns a stateless ticket */
         stateType: issue.state?.type ?? "",
+        /* v8 ignore next @preserve -- ResolveIssue query selects state; null only if Linear genuinely returns a stateless ticket */
+        stateId: issue.state?.id ?? "",
         blockers: blockersFromRelations(issue.inverseRelations?.nodes ?? []),
         hasMoreBlockers: issue.inverseRelations?.pageInfo.hasNextPage ?? false,
         hasChildren: (issue.children?.nodes.length ?? 0) > 0,
@@ -374,53 +334,6 @@ export async function fetchInProgressIssueCount(arguments_) {
         after = page.pageInfo.endCursor;
     }
 }
-export function resolveRepositoryFor(arguments_) {
-    const { description, config } = arguments_;
-    if (description === undefined || description.length === 0) {
-        return { kind: "missing" };
-    }
-    const match = buildRepositoryRegex(config).exec(description)?.[1];
-    if (match === undefined) {
-        return { kind: "missing" };
-    }
-    // `buildRepositoryRegex` matches both the full `owner/repo` entry and its bare
-    // suffix, so the captured value can be either form. Downstream code composes
-    // the resolved value with `workspace.projectDir` and needs the exact
-    // `knownRepositories` entry, so resolve back to that form here.
-    const candidates = config.workspace.knownRepositories.filter((entry) => entry === match || entry.endsWith(`/${match}`));
-    if (candidates.length !== 1) {
-        return { kind: "missing" };
-    }
-    const [canonical] = candidates;
-    /* v8 ignore next 3 @preserve -- candidates.length === 1 guarantees [0] is defined */
-    if (canonical === undefined) {
-        return { kind: "missing" };
-    }
-    return { kind: "ok", repository: canonical };
-}
-export function resolveModelFor(arguments_) {
-    const { labels, config } = arguments_;
-    const parsed = parseAgentLabels(labels, config);
-    if (parsed === undefined) {
-        return { kind: "no-label" };
-    }
-    if (parsed.model === AGENT_ANY_MODEL) {
-        return { kind: "agent-any" };
-    }
-    if (parsed.disabledFallback !== undefined) {
-        return {
-            kind: "disabled-fallback",
-            requestedModel: parsed.disabledFallback,
-            fallbackModel: parsed.model,
-        };
-    }
-    return { kind: "matched", model: parsed.model };
-}
-/**
- * `agent-any` collapses to `models.default` here — manual setup doesn't run
- * the usage-gated `any` resolver, so the caller gets a concrete model name
- * instead of a sentinel that downstream code can't interpret.
- */
 export async function fetchResolvedIssue(arguments_) {
     const { client, config, ticket } = arguments_;
     const upper = ticket.toUpperCase();
@@ -428,7 +341,6 @@ export async function fetchResolvedIssue(arguments_) {
     const repositoryResolution = resolveRepositoryFor({
         description: raw.description,
         config,
-        ticket: upper,
     });
     if (repositoryResolution.kind === "missing") {
         throw new RepositoryResolutionError({
@@ -452,35 +364,11 @@ export async function fetchResolvedIssue(arguments_) {
         repository: repositoryResolution.repository,
         model,
         teamId: raw.teamId,
+        stateType: raw.stateType,
+        status: raw.stateName,
+        statusId: raw.stateId,
     };
 }
-function parseAgentLabels(labels, config) {
-    const agentLabels = labels.filter((label) => label.name.startsWith(AGENT_LABEL_PREFIX));
-    if (agentLabels.length === 0) {
-        return undefined;
-    }
-    let disabledFallback;
-    for (const label of agentLabels) {
-        const name = label.name.slice(AGENT_LABEL_PREFIX.length);
-        if (name === AGENT_ANY_MODEL) {
-            return { model: AGENT_ANY_MODEL };
-        }
-        // Own-property check, not `in`: a label like `agent-toString` or
-        // `agent-__proto__` would otherwise resolve through the prototype chain
-        // instead of falling back to `models.default`.
-        if (Object.hasOwn(config.models.definitions, name)) {
-            return { model: name };
-        }
-        if (disabledFallback === undefined && isShippedDefaultDisabled(config, name)) {
-            disabledFallback = name;
-        }
-    }
-    const fallback = { model: config.models.default };
-    if (disabledFallback !== undefined) {
-        fallback.disabledFallback = disabledFallback;
-    }
-    return fallback;
-}
 export function warnIfDisabledFallback(ticket, modelResolution, config) {
     if (modelResolution.kind !== "disabled-fallback") {
         return;

package/dist/lib/adapters/linear/index.d.ts

@@ -2,4 +2,5 @@ import type { AdapterDefinition } from "../../adapterDefinition.ts";
 import { linearAdapterConfigSchema } from "./schema.ts";
 declare const definition: AdapterDefinition<typeof linearAdapterConfigSchema>;
 export default definition;
+export type { LinearSourceRef } from "./factory.ts";
 //# sourceMappingURL=index.d.ts.map

package/dist/lib/adapters/linear/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/lib/adapters/linear/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,4BAA4B,CAAC;AAGpE,OAAO,EAAE,yBAAyB,EAAE,MAAM,aAAa,CAAC;AAExD,QAAA,MAAM,UAAU,EAAE,iBAAiB,CAAC,OAAO,yBAAyB,CAInE,CAAC;eAEa,UAAU"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../../src/lib/adapters/linear/index.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,4BAA4B,CAAC;AAGpE,OAAO,EAAE,yBAAyB,EAAE,MAAM,aAAa,CAAC;AAExD,QAAA,MAAM,UAAU,EAAE,iBAAiB,CAAC,OAAO,yBAAyB,CAInE,CAAC;eAEa,UAAU;AAEzB,YAAY,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC"}

package/dist/lib/adapters/linear/parsing.d.ts

@@ -0,0 +1,44 @@
+/**
+ * Linear adapter — parsing helpers for model/repository resolution from
+ * issue labels and descriptions. Extracted from boardSource.ts (Task 10).
+ */
+import { type ResolvedConfig } from "../../config.ts";
+export declare const AGENT_LABEL_PREFIX = "agent-";
+export type RepositoryResolution = {
+    kind: "ok";
+    repository: string;
+} | {
+    kind: "missing";
+};
+export type ModelResolution = {
+    kind: "matched";
+    model: string;
+} | {
+    kind: "no-label";
+} | {
+    kind: "agent-any";
+} | {
+    kind: "disabled-fallback";
+    requestedModel: string;
+    fallbackModel: string;
+};
+export declare function buildRepositoryRegex(config: ResolvedConfig): RegExp;
+export declare function resolveRepositoryFor(arguments_: {
+    description: string | undefined;
+    config: ResolvedConfig;
+}): RepositoryResolution;
+interface ParseRepositoryArguments {
+    description: string | undefined;
+    config: ResolvedConfig;
+    repositoryRegex: RegExp;
+    ticket: string;
+}
+export declare function parseRepository(arguments_: ParseRepositoryArguments): string;
+export declare function resolveModelFor(arguments_: {
+    labels: {
+        name: string;
+    }[];
+    config: ResolvedConfig;
+}): ModelResolution;
+export {};
+//# sourceMappingURL=parsing.d.ts.map

package/dist/lib/adapters/linear/parsing.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"parsing.d.ts","sourceRoot":"","sources":["../../../../src/lib/adapters/linear/parsing.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAA6C,KAAK,cAAc,EAAE,MAAM,iBAAiB,CAAC;AAGjG,eAAO,MAAM,kBAAkB,WAAW,CAAC;AAE3C,MAAM,MAAM,oBAAoB,GAAG;IAAE,IAAI,EAAE,IAAI,CAAC;IAAC,UAAU,EAAE,MAAM,CAAA;CAAE,GAAG;IAAE,IAAI,EAAE,SAAS,CAAA;CAAE,CAAC;AAE5F,MAAM,MAAM,eAAe,GACvB;IAAE,IAAI,EAAE,SAAS,CAAC;IAAC,KAAK,EAAE,MAAM,CAAA;CAAE,GAClC;IAAE,IAAI,EAAE,UAAU,CAAA;CAAE,GACpB;IAAE,IAAI,EAAE,WAAW,CAAA;CAAE,GACrB;IAAE,IAAI,EAAE,mBAAmB,CAAC;IAAC,cAAc,EAAE,MAAM,CAAC;IAAC,aAAa,EAAE,MAAM,CAAA;CAAE,CAAC;AASjF,wBAAgB,oBAAoB,CAAC,MAAM,EAAE,cAAc,GAAG,MAAM,CAUnE;AAiDD,wBAAgB,oBAAoB,CAAC,UAAU,EAAE;IAC/C,WAAW,EAAE,MAAM,GAAG,SAAS,CAAC;IAChC,MAAM,EAAE,cAAc,CAAC;CACxB,GAAG,oBAAoB,CAuBvB;AAED,UAAU,wBAAwB;IAChC,WAAW,EAAE,MAAM,GAAG,SAAS,CAAC;IAChC,MAAM,EAAE,cAAc,CAAC;IACvB,eAAe,EAAE,MAAM,CAAC;IACxB,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,wBAAgB,eAAe,CAAC,UAAU,EAAE,wBAAwB,GAAG,MAAM,CA2B5E;AAkDD,wBAAgB,eAAe,CAAC,UAAU,EAAE;IAC1C,MAAM,EAAE;QAAE,IAAI,EAAE,MAAM,CAAA;KAAE,EAAE,CAAC;IAC3B,MAAM,EAAE,cAAc,CAAC;CACxB,GAAG,eAAe,CAiBlB"}

package/dist/lib/adapters/linear/parsing.js

@@ -0,0 +1,144 @@
+/**
+ * Linear adapter — parsing helpers for model/repository resolution from
+ * issue labels and descriptions. Extracted from boardSource.ts (Task 10).
+ */
+import { AGENT_ANY_MODEL, isShippedDefaultDisabled } from "../../config.js";
+import { RepositoryResolutionError } from "../../ticketSource.js";
+export const AGENT_LABEL_PREFIX = "agent-";
+function escapeRegex(value) {
+    return value.replaceAll(/[$()*+.?[\\\]^{|}]/g, String.raw `\$&`);
+}
+// Sort by descending length so longer names match first — `api-admin`
+// must beat `api` when both are configured. `\b` treats `-` as a word
+// boundary, so without this ordering `api` would win on `api-admin`.
+export function buildRepositoryRegex(config) {
+    const candidates = config.workspace.knownRepositories.flatMap((repo) => {
+        const slashIndex = repo.indexOf("/");
+        return slashIndex === -1 ? [repo] : [repo, repo.slice(slashIndex + 1)];
+    });
+    const alternation = candidates
+        .toSorted((a, b) => b.length - a.length)
+        .map(escapeRegex)
+        .join("|");
+    return new RegExp(String.raw `\b(${alternation})\b`);
+}
+function canonicalizeRepositoryMatch(description, config, repositoryRegex) {
+    if (description === undefined || description.length === 0) {
+        return { kind: "missing" };
+    }
+    // Guard against an empty knownRepositories config: buildRepositoryRegex
+    // would produce /\b()\b/, which matches the empty string at any word
+    // boundary and returns a bogus "" match. Treat that as "no repo could
+    // be resolved" so neither the dispatch path nor the doctor path emits
+    // a spurious empty-string repository.
+    if (config.workspace.knownRepositories.length === 0) {
+        return { kind: "missing" };
+    }
+    const matched = repositoryRegex.exec(description)?.[1];
+    if (matched === undefined) {
+        return { kind: "missing" };
+    }
+    const candidates = config.workspace.knownRepositories.filter((r) => r === matched || r.endsWith(`/${matched}`));
+    if (candidates.length > 1) {
+        return { kind: "ambiguous" };
+    }
+    if (candidates.length === 1) {
+        /* v8 ignore next @preserve -- length-1 guarantees [0] defined */
+        // oxlint-disable-next-line typescript/no-non-null-assertion -- length-1 guarantees [0] is defined
+        return { kind: "canonical", repository: candidates[0] };
+    }
+    return { kind: "unknown", repository: matched };
+}
+export function resolveRepositoryFor(arguments_) {
+    const { description, config } = arguments_;
+    const match = canonicalizeRepositoryMatch(description, config, buildRepositoryRegex(config));
+    switch (match.kind) {
+        case "missing":
+        case "ambiguous": {
+            // Ambiguous matches surface as "missing" so fetchResolvedIssue throws
+            // RepositoryResolutionError — same conflation parseRepository uses,
+            // and the right call for single-ticket flows: the launcher can't
+            // disambiguate "matched N known repos" any more than the dispatcher can.
+            return { kind: "missing" };
+        }
+        case "canonical":
+        case "unknown": {
+            return { kind: "ok", repository: match.repository };
+        }
+        /* v8 ignore next 5 @preserve -- exhaustive over CanonicalizedRepositoryMatch.kind */
+        default: {
+            throw new Error(`resolveRepositoryFor: unexpected match kind ${match.kind}`);
+        }
+    }
+}
+export function parseRepository(arguments_) {
+    const { description, config, repositoryRegex, ticket } = arguments_;
+    const match = canonicalizeRepositoryMatch(description, config, repositoryRegex);
+    switch (match.kind) {
+        case "missing":
+        case "ambiguous": {
+            throw new RepositoryResolutionError({
+                ticket,
+                repositories: config.workspace.knownRepositories,
+            });
+        }
+        case "canonical": {
+            return match.repository;
+        }
+        case "unknown": {
+            // No match in knownRepositories — return the asserted name as-is. The
+            // dispatcher's dispatchableRepository helper WARN-logs and skips at
+            // the host layer, uniformly across all sources.
+            return match.repository;
+        }
+        /* v8 ignore next 5 @preserve -- exhaustive over CanonicalizedRepositoryMatch.kind */
+        default: {
+            throw new Error(`parseRepository: unexpected match kind ${match.kind}`);
+        }
+    }
+}
+function parseAgentLabels(labels, config) {
+    const agentLabels = labels.filter((label) => label.name.startsWith(AGENT_LABEL_PREFIX));
+    if (agentLabels.length === 0) {
+        return undefined;
+    }
+    let disabledFallback;
+    for (const label of agentLabels) {
+        const name = label.name.slice(AGENT_LABEL_PREFIX.length);
+        if (name === AGENT_ANY_MODEL) {
+            return { model: AGENT_ANY_MODEL };
+        }
+        // Own-property check, not `in`: a label like `agent-toString` or
+        // `agent-__proto__` would otherwise resolve through the prototype chain
+        // instead of falling back to `models.default`.
+        if (Object.hasOwn(config.models.definitions, name)) {
+            return { model: name };
+        }
+        if (disabledFallback === undefined && isShippedDefaultDisabled(config, name)) {
+            disabledFallback = name;
+        }
+    }
+    const fallback = { model: config.models.default };
+    if (disabledFallback !== undefined) {
+        fallback.disabledFallback = disabledFallback;
+    }
+    return fallback;
+}
+export function resolveModelFor(arguments_) {
+    const { labels, config } = arguments_;
+    const parsed = parseAgentLabels(labels, config);
+    if (parsed === undefined) {
+        return { kind: "no-label" };
+    }
+    if (parsed.model === AGENT_ANY_MODEL) {
+        return { kind: "agent-any" };
+    }
+    if (parsed.disabledFallback !== undefined) {
+        return {
+            kind: "disabled-fallback",
+            requestedModel: parsed.disabledFallback,
+            fallbackModel: parsed.model,
+        };
+    }
+    return { kind: "matched", model: parsed.model };
+}

package/dist/lib/{linearIssueStatus.d.ts → adapters/linear/writeback.d.ts}

@@ -6,10 +6,9 @@ interface LinearIssueReference {
 }
 interface LinearIssueStatusUpdater {
     markInProgress(issue: LinearIssueReference): Promise<void>;
-    resetMissingInProgressCache(): void;
 }
 export declare function createLinearIssueStatusUpdater(arguments_: {
     client: LinearClient;
 }): LinearIssueStatusUpdater;
 export {};
-//# sourceMappingURL=linearIssueStatus.d.ts.map
+//# sourceMappingURL=writeback.d.ts.map

package/dist/lib/adapters/linear/writeback.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"writeback.d.ts","sourceRoot":"","sources":["../../../../src/lib/adapters/linear/writeback.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,aAAa,CAAC;AAIhD,UAAU,oBAAoB;IAC5B,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;CAChB;AAED,UAAU,wBAAwB;IAChC,cAAc,CAAC,KAAK,EAAE,oBAAoB,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC;CAC5D;AAED,wBAAgB,8BAA8B,CAAC,UAAU,EAAE;IACzD,MAAM,EAAE,YAAY,CAAC;CACtB,GAAG,wBAAwB,CAgD3B"}

package/dist/lib/{linearIssueStatus.js → adapters/linear/writeback.js}

@@ -1,12 +1,18 @@
-import { log } from "./util.js";
+import { log } from "../../util.js";
 export function createLinearIssueStatusUpdater(arguments_) {
     const { client } = arguments_;
-    // The first matching workflow state per team is cached by teamId so the
-    // status writeback doesn't re-query Linear on every dispatch. Teams that
-    // genuinely lack an `started` workflow state stay in the negative cache
-    // until the dispatcher resets it at the top of each tick.
+    // Positive cache only. Keyed by teamId because the workflow `state.type ===
+    // "started"` lookup yields a single stateId per team — independent of which
+    // project the ticket belongs to. State ids don't change for misconfig
+    // reasons, so caching successful resolutions is safe across the process.
+    //
+    // No negative cache: a missing "started" workflow state is a Linear-side
+    // config issue the operator can correct mid-session, and a negative cache
+    // would mask that recovery until process restart. Slot count caps
+    // markInProgress calls per tick at 1-5, so re-fetching team states on
+    // every failing attempt costs at most a handful of extra Linear API calls
+    // per tick.
     const inProgressStateByTeam = new Map();
-    let teamsMissingInProgress = new Set();
     async function getInProgressStateId(teamId) {
         if (teamId.length === 0) {
             return undefined;
@@ -15,17 +21,13 @@ export function createLinearIssueStatusUpdater(arguments_) {
         if (cached !== undefined) {
             return cached;
         }
-        if (teamsMissingInProgress.has(teamId)) {
-            return undefined;
-        }
         const team = await client.team(teamId);
         const states = await team.states();
-        // Use the workflow state's *type* — Linear standardises on
-        // `started` for in-progress columns regardless of how the user renames
-        // them, so this works without any per-team status-name configuration.
+        // Use the workflow state's `type` — Linear standardises on `started` for
+        // in-progress columns regardless of how the user renames them, so this
+        // works without any per-team status-name configuration.
         const inProgress = states.nodes.find((state) => state.type === "started");
         if (inProgress?.id === undefined) {
-            teamsMissingInProgress.add(teamId);
             return undefined;
         }
         inProgressStateByTeam.set(teamId, inProgress.id);
@@ -39,8 +41,5 @@ export function createLinearIssueStatusUpdater(arguments_) {
         await client.updateIssue(issue.uuid, { stateId });
         log(`Marked ${issue.id} as in progress`);
     }
-    function resetMissingInProgressCache() {
-        teamsMissingInProgress = new Set();
-    }
-    return { markInProgress, resetMissingInProgressCache };
+    return { markInProgress };
 }

package/dist/lib/adapters/shell/factory.d.ts

@@ -14,7 +14,7 @@
  *  - `fetch` is required by the Zod schema.
  */
 import type { AdapterContext } from "../../adapterDefinition.ts";
-import type { Issue as CanonicalIssue, TicketSource } from "../../ticketSource.ts";
+import { type Issue as CanonicalIssue, type TicketSource } from "../../ticketSource.ts";
 import { type ShellAdapterConfig, type ShellIssue } from "./schema.ts";
 export declare function toCanonicalIssue(shellIssue: ShellIssue, sourceName: string): CanonicalIssue;
 export declare function createShellTicketSource(config: ShellAdapterConfig, _context: AdapterContext): TicketSource;

package/dist/lib/adapters/shell/factory.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"factory.d.ts","sourceRoot":"","sources":["../../../../src/lib/adapters/shell/factory.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,4BAA4B,CAAC;AACjE,OAAO,KAAK,EAEV,KAAK,IAAI,cAAc,EACvB,YAAY,EACb,MAAM,uBAAuB,CAAC;AAG/B,OAAO,EACL,KAAK,kBAAkB,EAEvB,KAAK,UAAU,EAEhB,MAAM,aAAa,CAAC;AAyBrB,wBAAgB,gBAAgB,CAAC,UAAU,EAAE,UAAU,EAAE,UAAU,EAAE,MAAM,GAAG,cAAc,CAoB3F;AAED,wBAAgB,uBAAuB,CACrC,MAAM,EAAE,kBAAkB,EAC1B,QAAQ,EAAE,cAAc,GACvB,YAAY,CA+Ed"}
+{"version":3,"file":"factory.d.ts","sourceRoot":"","sources":["../../../../src/lib/adapters/shell/factory.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,4BAA4B,CAAC;AACjE,OAAO,EAGL,KAAK,KAAK,IAAI,cAAc,EAC5B,KAAK,YAAY,EAClB,MAAM,uBAAuB,CAAC;AAG/B,OAAO,EACL,KAAK,kBAAkB,EAEvB,KAAK,UAAU,EAEhB,MAAM,aAAa,CAAC;AAyBrB,wBAAgB,gBAAgB,CAAC,UAAU,EAAE,UAAU,EAAE,UAAU,EAAE,MAAM,GAAG,cAAc,CAsB3F;AAED,wBAAgB,uBAAuB,CACrC,MAAM,EAAE,kBAAkB,EAC1B,QAAQ,EAAE,cAAc,GACvB,YAAY,CAgFd"}

package/dist/lib/adapters/shell/factory.js

@@ -13,6 +13,7 @@
  *  - `markInProgress` absent → silent no-op.
  *  - `fetch` is required by the Zod schema.
  */
+import { toCanonicalId, } from "../../ticketSource.js";
 import { invokeShellCommand } from "./invoke.js";
 import { shellFetchOutputSchema, shellIssueSchema, } from "./schema.js";
 const DEFAULT_TIMEOUTS = {
@@ -31,12 +32,14 @@ function mergeTimeouts(overrides) {
 }
 export function toCanonicalIssue(shellIssue, sourceName) {
     const blockers = shellIssue.blockers.map((b) => ({
-        id: `${sourceName}:${b.id}`,
+        id: toCanonicalId(sourceName, b.id),
         title: b.title,
         status: b.status,
+        ...(b.statusReason !== undefined && { statusReason: b.statusReason }),
+        ...(b.nativeStatus !== undefined && { nativeStatus: b.nativeStatus }),
     }));
     return {
-        id: `${sourceName}:${shellIssue.id}`,
+        id: toCanonicalId(sourceName, shellIssue.id),
         source: sourceName,
         title: shellIssue.title,
         description: shellIssue.description,
@@ -81,10 +84,11 @@ export function createShellTicketSource(config, _context) {
         },
         fetch: runFetch,
         async resolveOne(naturalId) {
+            const canonicalId = toCanonicalId(sourceName, naturalId);
             const resolveCommand = config.commands.resolveOne;
             if (resolveCommand === undefined) {
                 const all = await runFetch();
-                return all.find((i) => i.id === `${sourceName}:${naturalId}`);
+                return all.find((i) => i.id === canonicalId);
             }
             const result = await invokeShellCommand({
                 command: resolveCommand,
@@ -93,7 +97,7 @@ export function createShellTicketSource(config, _context) {
                 env: config.env,
                 substitutions: {
                     id: naturalId,
-                    canonicalId: `${sourceName}:${naturalId}`,
+                    canonicalId,
                     name: sourceName,
                 },
                 sourceName,

package/dist/lib/adapters/shell/invoke.d.ts

@@ -11,19 +11,12 @@
  * Exit code 0 = success; exit code 3 = "not found" (caller decides how to
  * interpret); any other nonzero exit throws.
  */
-export declare const SHELL_COMMAND_MAX_BUFFER_BYTES: number;
 export declare class ShellAdapterTimeoutError extends Error {
     constructor(arguments_: {
         command: string;
         timeoutMs: number;
     });
 }
-export declare class ShellAdapterOutputLimitError extends Error {
-    constructor(arguments_: {
-        command: string;
-        maxBytes: number;
-    });
-}
 interface InvokeArgs {
     command: string;
     timeoutMs: number;
@@ -33,11 +26,15 @@ interface InvokeArgs {
     substitutions?: Record<string, string> | undefined;
     /** Source name for log prefixing. */
     sourceName: string;
+    /** Override the default per-stream stdout/stderr cap (10 MB). Used by tests. */
+    maxOutputBytes?: number;
 }
 interface InvokeResult {
     stdout: string;
     stderr: string;
     exitCode: number;
+    /** True if either stream hit the byte cap and the rest was discarded. */
+    truncated: boolean;
 }
 export declare function applySubstitutions(command: string, subs: Record<string, string>): string;
 export declare function invokeShellCommand(args: InvokeArgs): Promise<InvokeResult>;

package/dist/lib/adapters/shell/invoke.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"invoke.d.ts","sourceRoot":"","sources":["../../../../src/lib/adapters/shell/invoke.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAMH,eAAO,MAAM,8BAA8B,QAAmB,CAAC;AAG/D,qBAAa,wBAAyB,SAAQ,KAAK;IACjD,YAAmB,UAAU,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,SAAS,EAAE,MAAM,CAAA;KAAE,EAGpE;CACF;AAED,qBAAa,4BAA6B,SAAQ,KAAK;IACrD,YAAmB,UAAU,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,QAAQ,EAAE,MAAM,CAAA;KAAE,EAKnE;CACF;AAED,UAAU,UAAU;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3B,GAAG,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,SAAS,CAAC;IACzC,aAAa,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,SAAS,CAAC;IACnD,qCAAqC;IACrC,UAAU,EAAE,MAAM,CAAC;CACpB;AAED,UAAU,YAAY;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,EAAE,MAAM,CAAC;CAClB;AAmBD,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,MAAM,CAMxF;AAED,wBAAsB,kBAAkB,CAAC,IAAI,EAAE,UAAU,GAAG,OAAO,CAAC,YAAY,CAAC,CA8HhF"}
+{"version":3,"file":"invoke.d.ts","sourceRoot":"","sources":["../../../../src/lib/adapters/shell/invoke.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AAcH,qBAAa,wBAAyB,SAAQ,KAAK;IACjD,YAAmB,UAAU,EAAE;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,SAAS,EAAE,MAAM,CAAA;KAAE,EAGpE;CACF;AAED,UAAU,UAAU;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,KAAK,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3B,GAAG,CAAC,EAAE,MAAM,GAAG,SAAS,CAAC;IACzB,GAAG,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,SAAS,CAAC;IACzC,aAAa,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,SAAS,CAAC;IACnD,qCAAqC;IACrC,UAAU,EAAE,MAAM,CAAC;IACnB,gFAAgF;IAChF,cAAc,CAAC,EAAE,MAAM,CAAC;CACzB;AAED,UAAU,YAAY;IACpB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,EAAE,MAAM,CAAC;IACjB,yEAAyE;IACzE,SAAS,EAAE,OAAO,CAAC;CACpB;AAMD,wBAAgB,kBAAkB,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,MAAM,CAMxF;AAED,wBAAsB,kBAAkB,CAAC,IAAI,EAAE,UAAU,GAAG,OAAO,CAAC,YAAY,CAAC,CAkGhF"}