@livekit/changesets-changelog-github 0.0.3 → 0.1.0

package/dist/index.js

@@ -25,6 +25,43 @@ module.exports = __toCommonJS(src_exports);
 var import_dotenv = require("dotenv");
 var import_get_github_info = require("@changesets/get-github-info");
 (0, import_dotenv.config)();
+var MAX_RETRIES = 3;
+var RETRY_BASE_DELAY_MS = 200;
+var TRANSIENT_ERROR_CODES = /* @__PURE__ */ new Set([
+  "ERR_STREAM_PREMATURE_CLOSE",
+  "ECONNRESET",
+  "ETIMEDOUT",
+  "ECONNREFUSED",
+  "EPIPE",
+  "ENOTFOUND",
+  "EAI_AGAIN"
+]);
+function isTransientError(error) {
+  if (!error || typeof error !== "object")
+    return false;
+  const code = error.code;
+  if (code && TRANSIENT_ERROR_CODES.has(code))
+    return true;
+  const message = error.message;
+  return typeof message === "string" && /premature close/i.test(message);
+}
+var delay = (ms) => new Promise((resolve) => setTimeout(resolve, ms));
+async function withRetry(fn) {
+  let lastError;
+  for (let attempt = 0; attempt <= MAX_RETRIES; attempt++) {
+    try {
+      return await fn();
+    } catch (error) {
+      lastError = error;
+      if (attempt === MAX_RETRIES || !isTransientError(error))
+        throw error;
+      await delay(RETRY_BASE_DELAY_MS * Math.pow(2, attempt));
+    }
+  }
+  throw lastError;
+}
+var getInfoWithRetry = (request) => withRetry(() => (0, import_get_github_info.getInfo)(request));
+var getInfoFromPullRequestWithRetry = (request) => withRetry(() => (0, import_get_github_info.getInfoFromPullRequest)(request));
 var changelogFunctions = {
   getDependencyReleaseLine: async (changesets, dependenciesUpdated, options) => {
     if (!options.repo) {
@@ -37,7 +74,7 @@ var changelogFunctions = {
     const changesetLink = `- Updated dependencies [${(await Promise.all(
       changesets.map(async (cs) => {
         if (cs.commit) {
-          let { links } = await (0, import_get_github_info.getInfo)({
+          let { links } = await getInfoWithRetry({
             repo: options.repo,
             commit: cs.commit
           });
@@ -74,7 +111,7 @@ var changelogFunctions = {
     const [firstLine, ...futureLines] = replacedChangelog.split("\n").map((l) => l.trimRight());
     const links = await (async () => {
       if (prFromSummary !== void 0) {
-        let { links: links2 } = await (0, import_get_github_info.getInfoFromPullRequest)({
+        let { links: links2 } = await getInfoFromPullRequestWithRetry({
           repo: options.repo,
           pull: prFromSummary
         });
@@ -88,7 +125,7 @@ var changelogFunctions = {
       }
       const commitToFetchFrom = commitFromSummary || changeset.commit;
       if (commitToFetchFrom) {
-        let { links: links2 } = await (0, import_get_github_info.getInfo)({
+        let { links: links2 } = await getInfoWithRetry({
           repo: options.repo,
           commit: commitToFetchFrom
         });

package/dist/index.mjs

@@ -2,6 +2,43 @@
 import { config } from "dotenv";
 import { getInfo, getInfoFromPullRequest } from "@changesets/get-github-info";
 config();
+var MAX_RETRIES = 3;
+var RETRY_BASE_DELAY_MS = 200;
+var TRANSIENT_ERROR_CODES = /* @__PURE__ */ new Set([
+  "ERR_STREAM_PREMATURE_CLOSE",
+  "ECONNRESET",
+  "ETIMEDOUT",
+  "ECONNREFUSED",
+  "EPIPE",
+  "ENOTFOUND",
+  "EAI_AGAIN"
+]);
+function isTransientError(error) {
+  if (!error || typeof error !== "object")
+    return false;
+  const code = error.code;
+  if (code && TRANSIENT_ERROR_CODES.has(code))
+    return true;
+  const message = error.message;
+  return typeof message === "string" && /premature close/i.test(message);
+}
+var delay = (ms) => new Promise((resolve) => setTimeout(resolve, ms));
+async function withRetry(fn) {
+  let lastError;
+  for (let attempt = 0; attempt <= MAX_RETRIES; attempt++) {
+    try {
+      return await fn();
+    } catch (error) {
+      lastError = error;
+      if (attempt === MAX_RETRIES || !isTransientError(error))
+        throw error;
+      await delay(RETRY_BASE_DELAY_MS * Math.pow(2, attempt));
+    }
+  }
+  throw lastError;
+}
+var getInfoWithRetry = (request) => withRetry(() => getInfo(request));
+var getInfoFromPullRequestWithRetry = (request) => withRetry(() => getInfoFromPullRequest(request));
 var changelogFunctions = {
   getDependencyReleaseLine: async (changesets, dependenciesUpdated, options) => {
     if (!options.repo) {
@@ -14,7 +51,7 @@ var changelogFunctions = {
     const changesetLink = `- Updated dependencies [${(await Promise.all(
       changesets.map(async (cs) => {
         if (cs.commit) {
-          let { links } = await getInfo({
+          let { links } = await getInfoWithRetry({
             repo: options.repo,
             commit: cs.commit
           });
@@ -51,7 +88,7 @@ var changelogFunctions = {
     const [firstLine, ...futureLines] = replacedChangelog.split("\n").map((l) => l.trimRight());
     const links = await (async () => {
       if (prFromSummary !== void 0) {
-        let { links: links2 } = await getInfoFromPullRequest({
+        let { links: links2 } = await getInfoFromPullRequestWithRetry({
           repo: options.repo,
           pull: prFromSummary
         });
@@ -65,7 +102,7 @@ var changelogFunctions = {
       }
       const commitToFetchFrom = commitFromSummary || changeset.commit;
       if (commitToFetchFrom) {
-        let { links: links2 } = await getInfo({
+        let { links: links2 } = await getInfoWithRetry({
           repo: options.repo,
           commit: commitToFetchFrom
         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@livekit/changesets-changelog-github",
-  "version": "0.0.3",
+  "version": "0.1.0",
   "description": "A compact changelog entry generator based on the official changesets changelog generator",
   "main": "dist/index.js",
   "module": "dist/index.mjs",
@@ -8,15 +8,16 @@
   "repository": "https://github.com/livekit/changesets-changelog-github",
   "scripts": {
     "build": "tsup-node",
+    "publish": "tsup-node --onSuccess \"npm publish\"",
     "test": "jest"
   },
   "dependencies": {
-    "@changesets/get-github-info": "^0.5.1",
-    "@changesets/types": "^5.2.0",
+    "@changesets/get-github-info": "^0.8.0",
+    "@changesets/types": "^6.1.0",
     "dotenv": "^8.1.0"
   },
   "devDependencies": {
-    "@changesets/parse": "*",
+    "@changesets/parse": "0.3.16",
     "@types/jest": "29.1.1",
     "changesets": "^1.0.2",
     "changesets-cli": "^0.0.0",

package/src/index.ts

@@ -5,6 +5,55 @@ import { getInfo, getInfoFromPullRequest } from "@changesets/get-github-info";
 
 config();
 
+// `@changesets/get-github-info` makes a single `node-fetch@2` GraphQL call to
+// GitHub and rethrows on failure. On GitHub-hosted runners (Node 24+), transient
+// GitHub connection resets surface through node-fetch's gzip stream handling as a
+// hard `ERR_STREAM_PREMATURE_CLOSE` ("Premature close") error, which aborts the
+// whole `changeset version` step. These failures are environmental and transient,
+// so we retry the fetch a few times with exponential backoff before giving up.
+const MAX_RETRIES = 3;
+const RETRY_BASE_DELAY_MS = 200;
+
+const TRANSIENT_ERROR_CODES = new Set([
+  "ERR_STREAM_PREMATURE_CLOSE",
+  "ECONNRESET",
+  "ETIMEDOUT",
+  "ECONNREFUSED",
+  "EPIPE",
+  "ENOTFOUND",
+  "EAI_AGAIN",
+]);
+
+function isTransientError(error: unknown): boolean {
+  if (!error || typeof error !== "object") return false;
+  const code = (error as { code?: string }).code;
+  if (code && TRANSIENT_ERROR_CODES.has(code)) return true;
+  const message = (error as { message?: string }).message;
+  return typeof message === "string" && /premature close/i.test(message);
+}
+
+const delay = (ms: number) => new Promise((resolve) => setTimeout(resolve, ms));
+
+async function withRetry<T>(fn: () => Promise<T>): Promise<T> {
+  let lastError: unknown;
+  for (let attempt = 0; attempt <= MAX_RETRIES; attempt++) {
+    try {
+      return await fn();
+    } catch (error) {
+      lastError = error;
+      if (attempt === MAX_RETRIES || !isTransientError(error)) throw error;
+      await delay(RETRY_BASE_DELAY_MS * Math.pow(2, attempt));
+    }
+  }
+  throw lastError;
+}
+
+const getInfoWithRetry: typeof getInfo = (request) =>
+  withRetry(() => getInfo(request));
+const getInfoFromPullRequestWithRetry: typeof getInfoFromPullRequest = (
+  request
+) => withRetry(() => getInfoFromPullRequest(request));
+
 const changelogFunctions: ChangelogFunctions = {
   getDependencyReleaseLine: async (
     changesets,
@@ -22,7 +71,7 @@ const changelogFunctions: ChangelogFunctions = {
       await Promise.all(
         changesets.map(async (cs) => {
           if (cs.commit) {
-            let { links } = await getInfo({
+            let { links } = await getInfoWithRetry({
               repo: options.repo,
               commit: cs.commit,
             });
@@ -73,7 +122,7 @@ const changelogFunctions: ChangelogFunctions = {
 
     const links = await (async () => {
       if (prFromSummary !== undefined) {
-        let { links } = await getInfoFromPullRequest({
+        let { links } = await getInfoFromPullRequestWithRetry({
           repo: options.repo,
           pull: prFromSummary,
         });
@@ -87,7 +136,7 @@ const changelogFunctions: ChangelogFunctions = {
       }
       const commitToFetchFrom = commitFromSummary || changeset.commit;
       if (commitToFetchFrom) {
-        let { links } = await getInfo({
+        let { links } = await getInfoWithRetry({
           repo: options.repo,
           commit: commitToFetchFrom,
         });

package/src/retry.test.ts

@@ -0,0 +1,99 @@
+import changelogFunctions from "./index";
+import parse from "@changesets/parse";
+
+// Auto-mock so we can drive `getInfo` / `getInfoFromPullRequest` per-test,
+// simulating the transient `ERR_STREAM_PREMATURE_CLOSE` GitHub fetch failures.
+jest.mock("@changesets/get-github-info");
+
+import { getInfo, getInfoFromPullRequest } from "@changesets/get-github-info";
+
+const mockedGetInfo = getInfo as jest.MockedFunction<typeof getInfo>;
+const mockedGetInfoFromPullRequest =
+  getInfoFromPullRequest as jest.MockedFunction<typeof getInfoFromPullRequest>;
+
+const getReleaseLine = changelogFunctions.getReleaseLine;
+
+const data = {
+  commit: "a085003",
+  user: "Andarist",
+  pull: 1613,
+  repo: "emotion-js/emotion",
+};
+
+const links = {
+  user: `@${data.user}`,
+  pull: `[#${data.pull}](https://github.com/${data.repo}/pull/${data.pull})`,
+  commit: `[\`${data.commit}\`](https://github.com/${data.repo}/commit/${data.commit})`,
+};
+
+const expectedLine = `\n\n- something - [#1613](https://github.com/emotion-js/emotion/pull/1613) (@Andarist)\n`;
+
+const prematureClose = () =>
+  Object.assign(new Error("Premature close"), {
+    code: "ERR_STREAM_PREMATURE_CLOSE",
+  });
+
+const getChangeset = (content: string, commit: string | undefined) => {
+  return [
+    {
+      ...parse(
+        `---
+  pkg: "minor"
+  ---
+  something
+  ${content}
+  `
+      ),
+      id: "some-id",
+      commit,
+    },
+    "minor",
+    { repo: data.repo },
+  ] as const;
+};
+
+it("retries getInfo when the GitHub fetch fails with a premature close", async () => {
+  mockedGetInfo
+    .mockRejectedValueOnce(prematureClose())
+    .mockRejectedValueOnce(prematureClose())
+    .mockResolvedValue({ pull: data.pull, user: data.user, links });
+
+  const result = await getReleaseLine(...getChangeset("", data.commit));
+
+  expect(mockedGetInfo).toHaveBeenCalledTimes(3);
+  expect(result).toEqual(expectedLine);
+});
+
+it("retries getInfoFromPullRequest when the GitHub fetch fails with a premature close", async () => {
+  mockedGetInfoFromPullRequest
+    .mockRejectedValueOnce(prematureClose())
+    .mockResolvedValue({ commit: data.commit, user: data.user, links });
+
+  const result = await getReleaseLine(
+    ...getChangeset(`pr: #${data.pull}`, data.commit)
+  );
+
+  expect(mockedGetInfoFromPullRequest).toHaveBeenCalledTimes(2);
+  expect(result).toEqual(expectedLine);
+});
+
+it("gives up after exhausting retries on a persistent transient failure", async () => {
+  mockedGetInfo.mockRejectedValue(prematureClose());
+
+  await expect(
+    getReleaseLine(...getChangeset("", data.commit))
+  ).rejects.toMatchObject({ code: "ERR_STREAM_PREMATURE_CLOSE" });
+
+  // initial attempt + MAX_RETRIES (3)
+  expect(mockedGetInfo).toHaveBeenCalledTimes(4);
+});
+
+it("does not retry non-transient errors", async () => {
+  mockedGetInfo.mockRejectedValue(new Error("Bad credentials"));
+
+  await expect(
+    getReleaseLine(...getChangeset("", data.commit))
+  ).rejects.toThrow("Bad credentials");
+
+  expect(mockedGetInfo).toHaveBeenCalledTimes(1);
+});