standup-mr 0.3.0 → 0.3.1

package/CHANGELOG.md

@@ -4,6 +4,31 @@ All notable changes to standup-mr are recorded here. The format follows
 [Keep a Changelog](https://keepachangelog.com/en/1.1.0/) and the project uses
 [semantic versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.3.1] - 2026-09-01
+
+### Fixed
+
+- One flaky sub-request no longer costs the whole note. A run fans out to
+  several requests per merge request — pull detail, check runs, reviews, and
+  for a red pipeline the run, the job and its log — and any one of them
+  returning a 502 aborted the entire standup. Server errors (5xx) and dropped
+  connections are now retried twice with a short backoff before giving up. When
+  the retries run out the error is raised exactly as before: nothing is
+  swallowed. A rejected token (401), a refused resource (403), a rate limit
+  (429) and a missing resource (404) are not retried, because retrying them is
+  never the right answer.
+- A blocker whose diagnosis could not be fetched is now reported as a blocker
+  with `job: "unknown"` and an error line reading
+  `diagnosis unavailable: <reason>`, instead of failing the run. The merge
+  request is genuinely blocked either way, and the log fetch already degraded
+  this way; the metadata calls around it now match. **A 401 is never degraded**
+  — a revoked or invalid token still fails loudly, which is the whole reason
+  these calls throw in the first place.
+
+### Added
+
+- `mcpName` in `package.json`, for publication to the official MCP Registry.
+
 ## [0.3.0] - 2026-09-01
 
 ### Added

package/README.md

@@ -188,6 +188,10 @@ that the package is missing.
 - **On GitHub, CI that reports only through the legacy commit-statuses API**
   — still how some vendors integrate — shows up as `pipelineMissing`. Check
   state is read from check-runs only.
+- **A blocker whose diagnosis could not be fetched is still reported**, as
+  `job: "unknown"` with a `diagnosis unavailable: …` error line. The merge
+  request is blocked either way; only the explanation is missing. Server
+  errors are retried twice first, and a rejected token still fails the run.
 
 ## Upgrading from 0.1.x
 

package/dist/{chunk-DL2P3XR6.js → chunk-2X2XEEUY.js}

@@ -218,9 +218,11 @@ function buildUrl(api, path, params) {
   return `${base}?${query.toString()}`;
 }
 var ApiError = class extends Error {
-  constructor(message) {
+  status;
+  constructor(message, status) {
     super(message);
     this.name = "ApiError";
+    this.status = status;
   }
 };
 function remaining(response) {
@@ -244,27 +246,67 @@ function assertUsable(response, host) {
   if (response.ok || response.status === 404) return;
   if (response.status === 403 || response.status === 429) {
     if (remaining(response) === "0") {
-      throw new ApiError(`${host} rate limit reached${resetAt(response)}.`);
+      throw new ApiError(`${host} rate limit reached${resetAt(response)}.`, response.status);
     }
     const retry = retryAfter(response);
-    if (retry) throw new ApiError(`${host} rate limit reached${retry}.`);
+    if (retry) throw new ApiError(`${host} rate limit reached${retry}.`, response.status);
   }
   if (response.status === 401) {
     throw new ApiError(
-      `${host} rejected the token (401). Check the token and its scopes.`
+      `${host} rejected the token (401). Check the token and its scopes.`,
+      401
     );
   }
   if (response.status === 403) {
     throw new ApiError(
-      `${host} refused the request (403). The token has no access to that resource.`
+      `${host} refused the request (403). The token has no access to that resource.`,
+      403
     );
   }
-  throw new ApiError(`${host} returned ${response.status}.`);
+  throw new ApiError(`${host} returned ${response.status}.`, response.status);
 }
 function unreachable(host, cause) {
   const detail = cause instanceof Error ? cause.message : String(cause);
   return new ApiError(`Could not reach ${host}: ${detail}`);
 }
+var RETRY_ATTEMPTS = 3;
+var RETRY_BACKOFF_MS = [250, 750];
+var wait = (ms) => new Promise((resolve) => setTimeout(resolve, ms));
+async function sendWithRetry(fetchImpl, url, init, host, sleep = wait) {
+  let last;
+  for (let attempt = 0; attempt < RETRY_ATTEMPTS; attempt += 1) {
+    if (attempt > 0) await sleep(RETRY_BACKOFF_MS[attempt - 1]);
+    const final = attempt === RETRY_ATTEMPTS - 1;
+    try {
+      const response = await fetchImpl(url, init);
+      if (response.status >= 500 && !final) continue;
+      return response;
+    } catch (cause) {
+      last = cause;
+      if (final) throw unreachable(host, cause);
+    }
+  }
+  throw unreachable(host, last);
+}
+
+// src/providers/base/diagnosis.ts
+var DIAGNOSIS_UNAVAILABLE = "diagnosis unavailable";
+var UNKNOWN = "unknown";
+function degradable(cause) {
+  return cause instanceof ApiError && cause.status !== 401;
+}
+function undiagnosed(mr, cause) {
+  return {
+    provider: mr.provider,
+    project: mr.project,
+    mr: mr.iid,
+    title: mr.title,
+    job: UNKNOWN,
+    stage: UNKNOWN,
+    url: mr.url,
+    errors: [`${DIAGNOSIS_UNAVAILABLE}: ${cause.message}`]
+  };
+}
 
 // src/trace/trace.constants.ts
 var ANSI = /\x1b\[[0-9;]*[a-zA-Z]/g;
@@ -414,11 +456,12 @@ var GitHubProvider = class {
     };
   }
   async send(url, init) {
-    try {
-      return await this.fetchImpl(url, { headers: this.headers(), ...init });
-    } catch (cause) {
-      throw unreachable(this.host, cause);
-    }
+    return await sendWithRetry(
+      this.fetchImpl,
+      url,
+      { headers: this.headers(), ...init },
+      this.host
+    );
   }
   async getJson(path, params) {
     const response = await this.send(buildUrl(this.api, path, params));
@@ -648,9 +691,17 @@ var GitHubProvider = class {
   }
   async getBlockers(mrs) {
     const red = mrs.filter((mr) => mr.pipeline === "failed");
-    const diagnosed = await Promise.all(red.map((mr) => this.diagnose(mr)));
+    const diagnosed = await Promise.all(red.map((mr) => this.tryDiagnose(mr)));
     return diagnosed.filter((row) => row !== null);
   }
+  async tryDiagnose(mr) {
+    try {
+      return await this.diagnose(mr);
+    } catch (cause) {
+      if (degradable(cause)) return undiagnosed(mr, cause);
+      throw cause;
+    }
+  }
 };
 
 // src/providers/gitlab/gitlab.constants.ts
@@ -671,14 +722,12 @@ var GitLabProvider = class {
     this.fetchImpl = fetchImpl;
   }
   async send(url) {
-    let response;
-    try {
-      response = await this.fetchImpl(url, {
-        headers: { "PRIVATE-TOKEN": this.token }
-      });
-    } catch (cause) {
-      throw unreachable(this.host, cause);
-    }
+    const response = await sendWithRetry(
+      this.fetchImpl,
+      url,
+      { headers: { "PRIVATE-TOKEN": this.token } },
+      this.host
+    );
     assertUsable(response, this.host);
     return response;
   }
@@ -830,31 +879,36 @@ var GitLabProvider = class {
     );
     return rows.sort((a, b) => b.updated.localeCompare(a.updated));
   }
+  async diagnose(mr) {
+    const jobs = await this.getJson(
+      `projects/${mr.projectId}/pipelines/${mr.pipelineId}/jobs`,
+      { per_page: PAGE_SIZE2 }
+    ) ?? [];
+    const job = jobs.find((j) => j.status === "failed");
+    if (!job) return null;
+    const trace = await this.getText(`projects/${mr.projectId}/jobs/${job.id}/trace`);
+    return {
+      provider: "gitlab",
+      project: mr.project,
+      mr: mr.iid,
+      title: mr.title,
+      job: job.name,
+      stage: job.stage,
+      url: mr.url,
+      errors: extractErrors(trace)
+    };
+  }
+  async tryDiagnose(mr) {
+    try {
+      return await this.diagnose(mr);
+    } catch (cause) {
+      if (degradable(cause)) return undiagnosed(mr, cause);
+      throw cause;
+    }
+  }
   async getBlockers(mrs) {
     const red = mrs.filter((mr) => mr.pipeline === "failed");
-    const diagnosed = await Promise.all(
-      red.map(async (mr) => {
-        const jobs = await this.getJson(
-          `projects/${mr.projectId}/pipelines/${mr.pipelineId}/jobs`,
-          { per_page: PAGE_SIZE2 }
-        ) ?? [];
-        const job = jobs.find((j) => j.status === "failed");
-        if (!job) return null;
-        const trace = await this.getText(
-          `projects/${mr.projectId}/jobs/${job.id}/trace`
-        );
-        return {
-          provider: "gitlab",
-          project: mr.project,
-          mr: mr.iid,
-          title: mr.title,
-          job: job.name,
-          stage: job.stage,
-          url: mr.url,
-          errors: extractErrors(trace)
-        };
-      })
-    );
+    const diagnosed = await Promise.all(red.map((mr) => this.tryDiagnose(mr)));
     return diagnosed.filter((row) => row !== null);
   }
 };
@@ -1103,6 +1157,12 @@ export {
   ApiError,
   assertUsable,
   unreachable,
+  RETRY_ATTEMPTS,
+  RETRY_BACKOFF_MS,
+  sendWithRetry,
+  DIAGNOSIS_UNAVAILABLE,
+  degradable,
+  undiagnosed,
   extractErrors,
   repoFromUrl,
   mapEvent,

package/dist/cli.js

@@ -5,7 +5,7 @@ import {
   connect,
   postWebhook,
   toMarkdown
-} from "./chunk-DL2P3XR6.js";
+} from "./chunk-2X2XEEUY.js";
 
 // src/cli/cli.ts
 import { existsSync, readFileSync, realpathSync } from "fs";

package/dist/index.d.ts

@@ -142,10 +142,19 @@ interface Provider {
 
 declare function buildUrl(api: string, path: string, params?: Record<string, string | number>): string;
 declare class ApiError extends Error {
-    constructor(message: string);
+    readonly status?: number;
+    constructor(message: string, status?: number);
 }
 declare function assertUsable(response: Response, host: string): void;
 declare function unreachable(host: string, cause: unknown): ApiError;
+declare const RETRY_ATTEMPTS = 3;
+declare const RETRY_BACKOFF_MS: number[];
+type Sleep = (ms: number) => Promise<void>;
+declare function sendWithRetry(fetchImpl: FetchLike, url: string, init: RequestInit | undefined, host: string, sleep?: Sleep): Promise<Response>;
+
+declare const DIAGNOSIS_UNAVAILABLE = "diagnosis unavailable";
+declare function degradable(cause: unknown): cause is ApiError;
+declare function undiagnosed(mr: MergeRequest, cause: ApiError): Blocker;
 
 type Params$1 = Record<string, string | number>;
 declare class GitHubProvider implements Provider {
@@ -171,6 +180,7 @@ declare class GitHubProvider implements Provider {
     private checkRunBlocker;
     private diagnose;
     getBlockers(mrs: MergeRequest[]): Promise<Blocker[]>;
+    private tryDiagnose;
 }
 
 interface RawCommit {
@@ -247,6 +257,8 @@ declare class GitLabProvider implements Provider {
     getMyMrs(today: Date): Promise<MergeRequest[]>;
     private approvedByMe;
     getReviews(identity: Identity, today: Date): Promise<Review[]>;
+    private diagnose;
+    private tryDiagnose;
     getBlockers(mrs: MergeRequest[]): Promise<Blocker[]>;
 }
 
@@ -269,4 +281,4 @@ declare function buildReport(provider: Provider, today: Date, lang?: string, loo
 
 declare function extractErrors(rawTrace: string, limit?: number): string[];
 
-export { type ActiveDay, type ActivityEvent, ApiError, type Blocker, type Bucket, ConfigError, type FetchLike, GITHUB_LABELS, GITLAB_LABELS, GitHubProvider, GitLabProvider, type Identity, type MergeRequest, type Provider, type ProviderKind, type ProviderLabels, type Review, STALE_DAYS, type SelectOptions, type StandupReport, approvedBy, assertUsable, buildReport, buildUrl, chooseKind, classify, connect, countChangesRequested, extractErrors, ghHosts, ghToken, glabHosts, glabToken, isoDay, label, latestStateByReviewer, localAt, mapEvent, markMissingPipelines, normalizeChecks, parseGlabHosts, parseLoggedInHosts, postWebhook, previousActiveDays, repoFromUrl, resolveHost, resolveToken, toMarkdown, unreachable };
+export { type ActiveDay, type ActivityEvent, ApiError, type Blocker, type Bucket, ConfigError, DIAGNOSIS_UNAVAILABLE, type FetchLike, GITHUB_LABELS, GITLAB_LABELS, GitHubProvider, GitLabProvider, type Identity, type MergeRequest, type Provider, type ProviderKind, type ProviderLabels, RETRY_ATTEMPTS, RETRY_BACKOFF_MS, type Review, STALE_DAYS, type SelectOptions, type StandupReport, approvedBy, assertUsable, buildReport, buildUrl, chooseKind, classify, connect, countChangesRequested, degradable, extractErrors, ghHosts, ghToken, glabHosts, glabToken, isoDay, label, latestStateByReviewer, localAt, mapEvent, markMissingPipelines, normalizeChecks, parseGlabHosts, parseLoggedInHosts, postWebhook, previousActiveDays, repoFromUrl, resolveHost, resolveToken, sendWithRetry, toMarkdown, undiagnosed, unreachable };

package/dist/index.js

@@ -2,10 +2,13 @@
 import {
   ApiError,
   ConfigError,
+  DIAGNOSIS_UNAVAILABLE,
   GITHUB_LABELS,
   GITLAB_LABELS,
   GitHubProvider,
   GitLabProvider,
+  RETRY_ATTEMPTS,
+  RETRY_BACKOFF_MS,
   STALE_DAYS,
   approvedBy,
   assertUsable,
@@ -15,6 +18,7 @@ import {
   classify,
   connect,
   countChangesRequested,
+  degradable,
   extractErrors,
   ghHosts,
   ghToken,
@@ -34,16 +38,21 @@ import {
   repoFromUrl,
   resolveHost,
   resolveToken,
+  sendWithRetry,
   toMarkdown,
+  undiagnosed,
   unreachable
-} from "./chunk-DL2P3XR6.js";
+} from "./chunk-2X2XEEUY.js";
 export {
   ApiError,
   ConfigError,
+  DIAGNOSIS_UNAVAILABLE,
   GITHUB_LABELS,
   GITLAB_LABELS,
   GitHubProvider,
   GitLabProvider,
+  RETRY_ATTEMPTS,
+  RETRY_BACKOFF_MS,
   STALE_DAYS,
   approvedBy,
   assertUsable,
@@ -53,6 +62,7 @@ export {
   classify,
   connect,
   countChangesRequested,
+  degradable,
   extractErrors,
   ghHosts,
   ghToken,
@@ -72,6 +82,8 @@ export {
   repoFromUrl,
   resolveHost,
   resolveToken,
+  sendWithRetry,
   toMarkdown,
+  undiagnosed,
   unreachable
 };

package/dist/mcp/server.js

@@ -243,9 +243,11 @@ function buildUrl(api, path, params) {
   return `${base}?${query.toString()}`;
 }
 var ApiError = class extends Error {
-  constructor(message) {
+  status;
+  constructor(message, status) {
     super(message);
     this.name = "ApiError";
+    this.status = status;
   }
 };
 function remaining(response) {
@@ -269,27 +271,67 @@ function assertUsable(response, host) {
   if (response.ok || response.status === 404) return;
   if (response.status === 403 || response.status === 429) {
     if (remaining(response) === "0") {
-      throw new ApiError(`${host} rate limit reached${resetAt(response)}.`);
+      throw new ApiError(`${host} rate limit reached${resetAt(response)}.`, response.status);
     }
     const retry = retryAfter(response);
-    if (retry) throw new ApiError(`${host} rate limit reached${retry}.`);
+    if (retry) throw new ApiError(`${host} rate limit reached${retry}.`, response.status);
   }
   if (response.status === 401) {
     throw new ApiError(
-      `${host} rejected the token (401). Check the token and its scopes.`
+      `${host} rejected the token (401). Check the token and its scopes.`,
+      401
     );
   }
   if (response.status === 403) {
     throw new ApiError(
-      `${host} refused the request (403). The token has no access to that resource.`
+      `${host} refused the request (403). The token has no access to that resource.`,
+      403
     );
   }
-  throw new ApiError(`${host} returned ${response.status}.`);
+  throw new ApiError(`${host} returned ${response.status}.`, response.status);
 }
 function unreachable(host, cause) {
   const detail = cause instanceof Error ? cause.message : String(cause);
   return new ApiError(`Could not reach ${host}: ${detail}`);
 }
+var RETRY_ATTEMPTS = 3;
+var RETRY_BACKOFF_MS = [250, 750];
+var wait = (ms) => new Promise((resolve) => setTimeout(resolve, ms));
+async function sendWithRetry(fetchImpl, url, init, host, sleep = wait) {
+  let last;
+  for (let attempt = 0; attempt < RETRY_ATTEMPTS; attempt += 1) {
+    if (attempt > 0) await sleep(RETRY_BACKOFF_MS[attempt - 1]);
+    const final = attempt === RETRY_ATTEMPTS - 1;
+    try {
+      const response = await fetchImpl(url, init);
+      if (response.status >= 500 && !final) continue;
+      return response;
+    } catch (cause) {
+      last = cause;
+      if (final) throw unreachable(host, cause);
+    }
+  }
+  throw unreachable(host, last);
+}
+
+// src/providers/base/diagnosis.ts
+var DIAGNOSIS_UNAVAILABLE = "diagnosis unavailable";
+var UNKNOWN = "unknown";
+function degradable(cause) {
+  return cause instanceof ApiError && cause.status !== 401;
+}
+function undiagnosed(mr, cause) {
+  return {
+    provider: mr.provider,
+    project: mr.project,
+    mr: mr.iid,
+    title: mr.title,
+    job: UNKNOWN,
+    stage: UNKNOWN,
+    url: mr.url,
+    errors: [`${DIAGNOSIS_UNAVAILABLE}: ${cause.message}`]
+  };
+}
 
 // src/providers/github/github.constants.ts
 var PAGE_SIZE = 100;
@@ -398,11 +440,12 @@ var GitHubProvider = class {
     };
   }
   async send(url, init) {
-    try {
-      return await this.fetchImpl(url, { headers: this.headers(), ...init });
-    } catch (cause) {
-      throw unreachable(this.host, cause);
-    }
+    return await sendWithRetry(
+      this.fetchImpl,
+      url,
+      { headers: this.headers(), ...init },
+      this.host
+    );
   }
   async getJson(path, params) {
     const response = await this.send(buildUrl(this.api, path, params));
@@ -632,9 +675,17 @@ var GitHubProvider = class {
   }
   async getBlockers(mrs) {
     const red = mrs.filter((mr) => mr.pipeline === "failed");
-    const diagnosed = await Promise.all(red.map((mr) => this.diagnose(mr)));
+    const diagnosed = await Promise.all(red.map((mr) => this.tryDiagnose(mr)));
     return diagnosed.filter((row) => row !== null);
   }
+  async tryDiagnose(mr) {
+    try {
+      return await this.diagnose(mr);
+    } catch (cause) {
+      if (degradable(cause)) return undiagnosed(mr, cause);
+      throw cause;
+    }
+  }
 };
 
 // src/providers/gitlab/gitlab.constants.ts
@@ -655,14 +706,12 @@ var GitLabProvider = class {
     this.fetchImpl = fetchImpl;
   }
   async send(url) {
-    let response;
-    try {
-      response = await this.fetchImpl(url, {
-        headers: { "PRIVATE-TOKEN": this.token }
-      });
-    } catch (cause) {
-      throw unreachable(this.host, cause);
-    }
+    const response = await sendWithRetry(
+      this.fetchImpl,
+      url,
+      { headers: { "PRIVATE-TOKEN": this.token } },
+      this.host
+    );
     assertUsable(response, this.host);
     return response;
   }
@@ -814,31 +863,36 @@ var GitLabProvider = class {
     );
     return rows.sort((a, b) => b.updated.localeCompare(a.updated));
   }
+  async diagnose(mr) {
+    const jobs = await this.getJson(
+      `projects/${mr.projectId}/pipelines/${mr.pipelineId}/jobs`,
+      { per_page: PAGE_SIZE2 }
+    ) ?? [];
+    const job = jobs.find((j) => j.status === "failed");
+    if (!job) return null;
+    const trace = await this.getText(`projects/${mr.projectId}/jobs/${job.id}/trace`);
+    return {
+      provider: "gitlab",
+      project: mr.project,
+      mr: mr.iid,
+      title: mr.title,
+      job: job.name,
+      stage: job.stage,
+      url: mr.url,
+      errors: extractErrors(trace)
+    };
+  }
+  async tryDiagnose(mr) {
+    try {
+      return await this.diagnose(mr);
+    } catch (cause) {
+      if (degradable(cause)) return undiagnosed(mr, cause);
+      throw cause;
+    }
+  }
   async getBlockers(mrs) {
     const red = mrs.filter((mr) => mr.pipeline === "failed");
-    const diagnosed = await Promise.all(
-      red.map(async (mr) => {
-        const jobs = await this.getJson(
-          `projects/${mr.projectId}/pipelines/${mr.pipelineId}/jobs`,
-          { per_page: PAGE_SIZE2 }
-        ) ?? [];
-        const job = jobs.find((j) => j.status === "failed");
-        if (!job) return null;
-        const trace = await this.getText(
-          `projects/${mr.projectId}/jobs/${job.id}/trace`
-        );
-        return {
-          provider: "gitlab",
-          project: mr.project,
-          mr: mr.iid,
-          title: mr.title,
-          job: job.name,
-          stage: job.stage,
-          url: mr.url,
-          errors: extractErrors(trace)
-        };
-      })
-    );
+    const diagnosed = await Promise.all(red.map((mr) => this.tryDiagnose(mr)));
     return diagnosed.filter((row) => row !== null);
   }
 };

package/package.json

@@ -1,6 +1,7 @@
 {
     "name": "standup-mr",
-    "version": "0.3.0",
+    "version": "0.3.1",
+    "mcpName": "io.github.Jubstaaa/standup-mr",
     "description": "Standup notes from merge request state, not commit logs.",
     "keywords": [
         "standup",