@harness-engineering/core 0.17.0 → 0.17.1

package/dist/index.d.mts

@@ -5166,6 +5166,10 @@ interface GitHubAdapterOptions {
     fetchFn?: typeof fetch;
     /** Override API base URL (for GitHub Enterprise) */
     apiBase?: string;
+    /** Max retries on rate limit (default: 5) */
+    maxRetries?: number;
+    /** Base delay in ms for exponential backoff (default: 1000) */
+    baseDelayMs?: number;
 }
 declare class GitHubIssuesSyncAdapter implements TrackerSyncAdapter {
     private readonly token;
@@ -5174,8 +5178,14 @@ declare class GitHubIssuesSyncAdapter implements TrackerSyncAdapter {
     private readonly apiBase;
     private readonly owner;
     private readonly repo;
+    private readonly retryOpts;
     constructor(options: GitHubAdapterOptions);
     private headers;
+    /**
+     * Close an issue if the feature status maps to 'closed'.
+     * GitHub Issues API doesn't accept state on POST — requires a follow-up PATCH.
+     */
+    private closeIfDone;
     createTicket(feature: RoadmapFeature, milestone: string): Promise<Result<ExternalTicket>>;
     updateTicket(externalId: string, changes: Partial<RoadmapFeature>): Promise<Result<ExternalTicket>>;
     fetchTicketState(externalId: string): Promise<Result<ExternalTicketState>>;

package/dist/index.d.ts

@@ -5166,6 +5166,10 @@ interface GitHubAdapterOptions {
     fetchFn?: typeof fetch;
     /** Override API base URL (for GitHub Enterprise) */
     apiBase?: string;
+    /** Max retries on rate limit (default: 5) */
+    maxRetries?: number;
+    /** Base delay in ms for exponential backoff (default: 1000) */
+    baseDelayMs?: number;
 }
 declare class GitHubIssuesSyncAdapter implements TrackerSyncAdapter {
     private readonly token;
@@ -5174,8 +5178,14 @@ declare class GitHubIssuesSyncAdapter implements TrackerSyncAdapter {
     private readonly apiBase;
     private readonly owner;
     private readonly repo;
+    private readonly retryOpts;
     constructor(options: GitHubAdapterOptions);
     private headers;
+    /**
+     * Close an issue if the feature status maps to 'closed'.
+     * GitHub Issues API doesn't accept state on POST — requires a follow-up PATCH.
+     */
+    private closeIfDone;
     createTicket(feature: RoadmapFeature, milestone: string): Promise<Result<ExternalTicket>>;
     updateTicket(externalId: string, changes: Partial<RoadmapFeature>): Promise<Result<ExternalTicket>>;
     fetchTicketState(externalId: string): Promise<Result<ExternalTicketState>>;

package/dist/index.js

@@ -12574,6 +12574,29 @@ function labelsForStatus(status, config) {
   }
   return [...base];
 }
+var RETRY_DEFAULTS = { maxRetries: 5, baseDelayMs: 1e3 };
+function sleep(ms) {
+  return new Promise((resolve7) => setTimeout(resolve7, ms));
+}
+async function fetchWithRetry(fetchFn, input, init, opts = RETRY_DEFAULTS) {
+  let lastResponse;
+  for (let attempt = 0; attempt <= opts.maxRetries; attempt++) {
+    const response = await fetchFn(input, init);
+    if (response.status !== 403 && response.status !== 429) return response;
+    lastResponse = response;
+    if (attempt === opts.maxRetries) break;
+    const retryAfter = response.headers.get("Retry-After");
+    let delayMs;
+    if (retryAfter) {
+      const seconds = parseInt(retryAfter, 10);
+      delayMs = isNaN(seconds) ? opts.baseDelayMs : seconds * 1e3;
+    } else {
+      delayMs = opts.baseDelayMs * Math.pow(2, attempt) + Math.random() * 500;
+    }
+    await sleep(delayMs);
+  }
+  return lastResponse;
+}
 var GitHubIssuesSyncAdapter = class {
   token;
   config;
@@ -12581,11 +12604,16 @@ var GitHubIssuesSyncAdapter = class {
   apiBase;
   owner;
   repo;
+  retryOpts;
   constructor(options) {
     this.token = options.token;
     this.config = options.config;
     this.fetchFn = options.fetchFn ?? globalThis.fetch;
     this.apiBase = options.apiBase ?? "https://api.github.com";
+    this.retryOpts = {
+      maxRetries: options.maxRetries ?? RETRY_DEFAULTS.maxRetries,
+      baseDelayMs: options.baseDelayMs ?? RETRY_DEFAULTS.baseDelayMs
+    };
     const repoParts = (options.config.repo ?? "").split("/");
     if (repoParts.length !== 2 || !repoParts[0] || !repoParts[1]) {
       throw new Error(`Invalid repo format: "${options.config.repo}". Expected "owner/repo".`);
@@ -12601,6 +12629,20 @@ var GitHubIssuesSyncAdapter = class {
       "X-GitHub-Api-Version": "2022-11-28"
     };
   }
+  /**
+   * Close an issue if the feature status maps to 'closed'.
+   * GitHub Issues API doesn't accept state on POST — requires a follow-up PATCH.
+   */
+  async closeIfDone(issueNumber, featureStatus) {
+    const externalStatus = this.config.statusMap[featureStatus];
+    if (externalStatus !== "closed") return;
+    await fetchWithRetry(
+      this.fetchFn,
+      `${this.apiBase}/repos/${this.owner}/${this.repo}/issues/${issueNumber}`,
+      { method: "PATCH", headers: this.headers(), body: JSON.stringify({ state: "closed" }) },
+      this.retryOpts
+    );
+  }
   async createTicket(feature, milestone) {
     try {
       const labels = labelsForStatus(feature.status, this.config);
@@ -12610,17 +12652,15 @@ var GitHubIssuesSyncAdapter = class {
         `**Milestone:** ${milestone}`,
         feature.spec ? `**Spec:** ${feature.spec}` : ""
       ].filter(Boolean).join("\n");
-      const response = await this.fetchFn(
+      const response = await fetchWithRetry(
+        this.fetchFn,
         `${this.apiBase}/repos/${this.owner}/${this.repo}/issues`,
         {
           method: "POST",
           headers: this.headers(),
-          body: JSON.stringify({
-            title: feature.name,
-            body,
-            labels
-          })
-        }
+          body: JSON.stringify({ title: feature.name, body, labels })
+        },
+        this.retryOpts
       );
       if (!response.ok) {
         const text = await response.text();
@@ -12628,6 +12668,7 @@ var GitHubIssuesSyncAdapter = class {
       }
       const data = await response.json();
       const externalId = buildExternalId(this.owner, this.repo, data.number);
+      await this.closeIfDone(data.number, feature.status);
       return (0, import_types21.Ok)({ externalId, url: data.html_url });
     } catch (error) {
       return (0, import_types21.Err)(error instanceof Error ? error : new Error(String(error)));
@@ -12648,13 +12689,15 @@ var GitHubIssuesSyncAdapter = class {
         patch.state = externalStatus;
         patch.labels = labelsForStatus(changes.status, this.config);
       }
-      const response = await this.fetchFn(
+      const response = await fetchWithRetry(
+        this.fetchFn,
         `${this.apiBase}/repos/${parsed.owner}/${parsed.repo}/issues/${parsed.number}`,
         {
           method: "PATCH",
           headers: this.headers(),
           body: JSON.stringify(patch)
-        }
+        },
+        this.retryOpts
       );
       if (!response.ok) {
         const text = await response.text();
@@ -12670,12 +12713,14 @@ var GitHubIssuesSyncAdapter = class {
     try {
       const parsed = parseExternalId(externalId);
       if (!parsed) return (0, import_types21.Err)(new Error(`Invalid externalId format: "${externalId}"`));
-      const response = await this.fetchFn(
+      const response = await fetchWithRetry(
+        this.fetchFn,
         `${this.apiBase}/repos/${parsed.owner}/${parsed.repo}/issues/${parsed.number}`,
         {
           method: "GET",
           headers: this.headers()
-        }
+        },
+        this.retryOpts
       );
       if (!response.ok) {
         const text = await response.text();
@@ -12700,12 +12745,14 @@ var GitHubIssuesSyncAdapter = class {
       let page = 1;
       const perPage = 100;
       while (true) {
-        const response = await this.fetchFn(
+        const response = await fetchWithRetry(
+          this.fetchFn,
           `${this.apiBase}/repos/${this.owner}/${this.repo}/issues?state=all&per_page=${perPage}&page=${page}${labelsParam}`,
           {
             method: "GET",
             headers: this.headers()
-          }
+          },
+          this.retryOpts
         );
         if (!response.ok) {
           const text = await response.text();
@@ -12734,13 +12781,15 @@ var GitHubIssuesSyncAdapter = class {
       const parsed = parseExternalId(externalId);
       if (!parsed) return (0, import_types21.Err)(new Error(`Invalid externalId format: "${externalId}"`));
       const login = assignee.startsWith("@") ? assignee.slice(1) : assignee;
-      const response = await this.fetchFn(
+      const response = await fetchWithRetry(
+        this.fetchFn,
         `${this.apiBase}/repos/${parsed.owner}/${parsed.repo}/issues/${parsed.number}/assignees`,
         {
           method: "POST",
           headers: this.headers(),
           body: JSON.stringify({ assignees: [login] })
-        }
+        },
+        this.retryOpts
       );
       if (!response.ok) {
         const text = await response.text();

package/dist/index.mjs

@@ -10465,6 +10465,29 @@ function labelsForStatus(status, config) {
   }
   return [...base];
 }
+var RETRY_DEFAULTS = { maxRetries: 5, baseDelayMs: 1e3 };
+function sleep(ms) {
+  return new Promise((resolve5) => setTimeout(resolve5, ms));
+}
+async function fetchWithRetry(fetchFn, input, init, opts = RETRY_DEFAULTS) {
+  let lastResponse;
+  for (let attempt = 0; attempt <= opts.maxRetries; attempt++) {
+    const response = await fetchFn(input, init);
+    if (response.status !== 403 && response.status !== 429) return response;
+    lastResponse = response;
+    if (attempt === opts.maxRetries) break;
+    const retryAfter = response.headers.get("Retry-After");
+    let delayMs;
+    if (retryAfter) {
+      const seconds = parseInt(retryAfter, 10);
+      delayMs = isNaN(seconds) ? opts.baseDelayMs : seconds * 1e3;
+    } else {
+      delayMs = opts.baseDelayMs * Math.pow(2, attempt) + Math.random() * 500;
+    }
+    await sleep(delayMs);
+  }
+  return lastResponse;
+}
 var GitHubIssuesSyncAdapter = class {
   token;
   config;
@@ -10472,11 +10495,16 @@ var GitHubIssuesSyncAdapter = class {
   apiBase;
   owner;
   repo;
+  retryOpts;
   constructor(options) {
     this.token = options.token;
     this.config = options.config;
     this.fetchFn = options.fetchFn ?? globalThis.fetch;
     this.apiBase = options.apiBase ?? "https://api.github.com";
+    this.retryOpts = {
+      maxRetries: options.maxRetries ?? RETRY_DEFAULTS.maxRetries,
+      baseDelayMs: options.baseDelayMs ?? RETRY_DEFAULTS.baseDelayMs
+    };
     const repoParts = (options.config.repo ?? "").split("/");
     if (repoParts.length !== 2 || !repoParts[0] || !repoParts[1]) {
       throw new Error(`Invalid repo format: "${options.config.repo}". Expected "owner/repo".`);
@@ -10492,6 +10520,20 @@ var GitHubIssuesSyncAdapter = class {
       "X-GitHub-Api-Version": "2022-11-28"
     };
   }
+  /**
+   * Close an issue if the feature status maps to 'closed'.
+   * GitHub Issues API doesn't accept state on POST — requires a follow-up PATCH.
+   */
+  async closeIfDone(issueNumber, featureStatus) {
+    const externalStatus = this.config.statusMap[featureStatus];
+    if (externalStatus !== "closed") return;
+    await fetchWithRetry(
+      this.fetchFn,
+      `${this.apiBase}/repos/${this.owner}/${this.repo}/issues/${issueNumber}`,
+      { method: "PATCH", headers: this.headers(), body: JSON.stringify({ state: "closed" }) },
+      this.retryOpts
+    );
+  }
   async createTicket(feature, milestone) {
     try {
       const labels = labelsForStatus(feature.status, this.config);
@@ -10501,17 +10543,15 @@ var GitHubIssuesSyncAdapter = class {
         `**Milestone:** ${milestone}`,
         feature.spec ? `**Spec:** ${feature.spec}` : ""
       ].filter(Boolean).join("\n");
-      const response = await this.fetchFn(
+      const response = await fetchWithRetry(
+        this.fetchFn,
         `${this.apiBase}/repos/${this.owner}/${this.repo}/issues`,
         {
           method: "POST",
           headers: this.headers(),
-          body: JSON.stringify({
-            title: feature.name,
-            body,
-            labels
-          })
-        }
+          body: JSON.stringify({ title: feature.name, body, labels })
+        },
+        this.retryOpts
       );
       if (!response.ok) {
         const text = await response.text();
@@ -10519,6 +10559,7 @@ var GitHubIssuesSyncAdapter = class {
       }
       const data = await response.json();
       const externalId = buildExternalId(this.owner, this.repo, data.number);
+      await this.closeIfDone(data.number, feature.status);
       return Ok4({ externalId, url: data.html_url });
     } catch (error) {
       return Err3(error instanceof Error ? error : new Error(String(error)));
@@ -10539,13 +10580,15 @@ var GitHubIssuesSyncAdapter = class {
         patch.state = externalStatus;
         patch.labels = labelsForStatus(changes.status, this.config);
       }
-      const response = await this.fetchFn(
+      const response = await fetchWithRetry(
+        this.fetchFn,
         `${this.apiBase}/repos/${parsed.owner}/${parsed.repo}/issues/${parsed.number}`,
         {
           method: "PATCH",
           headers: this.headers(),
           body: JSON.stringify(patch)
-        }
+        },
+        this.retryOpts
       );
       if (!response.ok) {
         const text = await response.text();
@@ -10561,12 +10604,14 @@ var GitHubIssuesSyncAdapter = class {
     try {
       const parsed = parseExternalId(externalId);
       if (!parsed) return Err3(new Error(`Invalid externalId format: "${externalId}"`));
-      const response = await this.fetchFn(
+      const response = await fetchWithRetry(
+        this.fetchFn,
         `${this.apiBase}/repos/${parsed.owner}/${parsed.repo}/issues/${parsed.number}`,
         {
           method: "GET",
           headers: this.headers()
-        }
+        },
+        this.retryOpts
       );
       if (!response.ok) {
         const text = await response.text();
@@ -10591,12 +10636,14 @@ var GitHubIssuesSyncAdapter = class {
       let page = 1;
       const perPage = 100;
       while (true) {
-        const response = await this.fetchFn(
+        const response = await fetchWithRetry(
+          this.fetchFn,
           `${this.apiBase}/repos/${this.owner}/${this.repo}/issues?state=all&per_page=${perPage}&page=${page}${labelsParam}`,
           {
             method: "GET",
             headers: this.headers()
-          }
+          },
+          this.retryOpts
         );
         if (!response.ok) {
           const text = await response.text();
@@ -10625,13 +10672,15 @@ var GitHubIssuesSyncAdapter = class {
       const parsed = parseExternalId(externalId);
       if (!parsed) return Err3(new Error(`Invalid externalId format: "${externalId}"`));
       const login = assignee.startsWith("@") ? assignee.slice(1) : assignee;
-      const response = await this.fetchFn(
+      const response = await fetchWithRetry(
+        this.fetchFn,
         `${this.apiBase}/repos/${parsed.owner}/${parsed.repo}/issues/${parsed.number}/assignees`,
         {
           method: "POST",
           headers: this.headers(),
           body: JSON.stringify({ assignees: [login] })
-        }
+        },
+        this.retryOpts
       );
       if (!response.ok) {
         const text = await response.text();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@harness-engineering/core",
-  "version": "0.17.0",
+  "version": "0.17.1",
   "description": "Core library for Harness Engineering toolkit",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",