@harness-engineering/core 0.17.0 → 0.18.0

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,18 @@ var GitHubIssuesSyncAdapter = class {
   apiBase;
   owner;
   repo;
+  retryOpts;
+  /** Cached GitHub milestone name -> ID mapping */
+  milestoneCache = null;
   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".`);
@@ -12593,6 +12623,43 @@ var GitHubIssuesSyncAdapter = class {
     this.owner = repoParts[0];
     this.repo = repoParts[1];
   }
+  /**
+   * Fetch all GitHub milestones and build the name -> ID cache.
+   */
+  async loadMilestones() {
+    if (this.milestoneCache) return this.milestoneCache;
+    this.milestoneCache = /* @__PURE__ */ new Map();
+    const response = await fetchWithRetry(
+      this.fetchFn,
+      `${this.apiBase}/repos/${this.owner}/${this.repo}/milestones?state=all&per_page=100`,
+      { method: "GET", headers: this.headers() },
+      this.retryOpts
+    );
+    if (response.ok) {
+      const data = await response.json();
+      for (const m of data) {
+        this.milestoneCache.set(m.title, m.number);
+      }
+    }
+    return this.milestoneCache;
+  }
+  /**
+   * Get or create a GitHub milestone by name. Returns the milestone number.
+   */
+  async ensureMilestone(name) {
+    const cache = await this.loadMilestones();
+    if (cache.has(name)) return cache.get(name);
+    const response = await fetchWithRetry(
+      this.fetchFn,
+      `${this.apiBase}/repos/${this.owner}/${this.repo}/milestones`,
+      { method: "POST", headers: this.headers(), body: JSON.stringify({ title: name }) },
+      this.retryOpts
+    );
+    if (!response.ok) return null;
+    const data = await response.json();
+    cache.set(name, data.number);
+    return data.number;
+  }
   headers() {
     return {
       Authorization: `Bearer ${this.token}`,
@@ -12601,26 +12668,32 @@ 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);
-      const body = [
-        feature.summary,
-        "",
-        `**Milestone:** ${milestone}`,
-        feature.spec ? `**Spec:** ${feature.spec}` : ""
-      ].filter(Boolean).join("\n");
-      const response = await this.fetchFn(
+      const labels = [...labelsForStatus(feature.status, this.config), "feature"];
+      const body = [feature.summary, "", feature.spec ? `**Spec:** ${feature.spec}` : ""].filter(Boolean).join("\n");
+      const milestoneId = await this.ensureMilestone(milestone);
+      const issuePayload = { title: feature.name, body, labels };
+      if (milestoneId) issuePayload.milestone = milestoneId;
+      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
-          })
-        }
+        { method: "POST", headers: this.headers(), body: JSON.stringify(issuePayload) },
+        this.retryOpts
       );
       if (!response.ok) {
         const text = await response.text();
@@ -12628,12 +12701,13 @@ 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)));
     }
   }
-  async updateTicket(externalId, changes) {
+  async updateTicket(externalId, changes, milestone) {
     try {
       const parsed = parseExternalId(externalId);
       if (!parsed) return (0, import_types21.Err)(new Error(`Invalid externalId format: "${externalId}"`));
@@ -12646,15 +12720,21 @@ var GitHubIssuesSyncAdapter = class {
       if (changes.status !== void 0) {
         const externalStatus = this.config.statusMap[changes.status];
         patch.state = externalStatus;
-        patch.labels = labelsForStatus(changes.status, this.config);
+        patch.labels = [...labelsForStatus(changes.status, this.config), "feature"];
+      }
+      if (milestone) {
+        const milestoneId = await this.ensureMilestone(milestone);
+        if (milestoneId) patch.milestone = milestoneId;
       }
-      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 +12750,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 +12782,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 +12818,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();
@@ -12771,7 +12857,11 @@ async function syncToExternal(roadmap, adapter, _config) {
           result.errors.push({ featureOrId: feature.name, error: createResult.error });
         }
       } else {
-        const updateResult = await adapter.updateTicket(feature.externalId, feature);
+        const updateResult = await adapter.updateTicket(
+          feature.externalId,
+          feature,
+          milestone.name
+        );
         if (updateResult.ok) {
           result.updated.push(feature.externalId);
         } else {

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,18 @@ var GitHubIssuesSyncAdapter = class {
   apiBase;
   owner;
   repo;
+  retryOpts;
+  /** Cached GitHub milestone name -> ID mapping */
+  milestoneCache = null;
   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".`);
@@ -10484,6 +10514,43 @@ var GitHubIssuesSyncAdapter = class {
     this.owner = repoParts[0];
     this.repo = repoParts[1];
   }
+  /**
+   * Fetch all GitHub milestones and build the name -> ID cache.
+   */
+  async loadMilestones() {
+    if (this.milestoneCache) return this.milestoneCache;
+    this.milestoneCache = /* @__PURE__ */ new Map();
+    const response = await fetchWithRetry(
+      this.fetchFn,
+      `${this.apiBase}/repos/${this.owner}/${this.repo}/milestones?state=all&per_page=100`,
+      { method: "GET", headers: this.headers() },
+      this.retryOpts
+    );
+    if (response.ok) {
+      const data = await response.json();
+      for (const m of data) {
+        this.milestoneCache.set(m.title, m.number);
+      }
+    }
+    return this.milestoneCache;
+  }
+  /**
+   * Get or create a GitHub milestone by name. Returns the milestone number.
+   */
+  async ensureMilestone(name) {
+    const cache = await this.loadMilestones();
+    if (cache.has(name)) return cache.get(name);
+    const response = await fetchWithRetry(
+      this.fetchFn,
+      `${this.apiBase}/repos/${this.owner}/${this.repo}/milestones`,
+      { method: "POST", headers: this.headers(), body: JSON.stringify({ title: name }) },
+      this.retryOpts
+    );
+    if (!response.ok) return null;
+    const data = await response.json();
+    cache.set(name, data.number);
+    return data.number;
+  }
   headers() {
     return {
       Authorization: `Bearer ${this.token}`,
@@ -10492,26 +10559,32 @@ 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);
-      const body = [
-        feature.summary,
-        "",
-        `**Milestone:** ${milestone}`,
-        feature.spec ? `**Spec:** ${feature.spec}` : ""
-      ].filter(Boolean).join("\n");
-      const response = await this.fetchFn(
+      const labels = [...labelsForStatus(feature.status, this.config), "feature"];
+      const body = [feature.summary, "", feature.spec ? `**Spec:** ${feature.spec}` : ""].filter(Boolean).join("\n");
+      const milestoneId = await this.ensureMilestone(milestone);
+      const issuePayload = { title: feature.name, body, labels };
+      if (milestoneId) issuePayload.milestone = milestoneId;
+      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
-          })
-        }
+        { method: "POST", headers: this.headers(), body: JSON.stringify(issuePayload) },
+        this.retryOpts
       );
       if (!response.ok) {
         const text = await response.text();
@@ -10519,12 +10592,13 @@ 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)));
     }
   }
-  async updateTicket(externalId, changes) {
+  async updateTicket(externalId, changes, milestone) {
     try {
       const parsed = parseExternalId(externalId);
       if (!parsed) return Err3(new Error(`Invalid externalId format: "${externalId}"`));
@@ -10537,15 +10611,21 @@ var GitHubIssuesSyncAdapter = class {
       if (changes.status !== void 0) {
         const externalStatus = this.config.statusMap[changes.status];
         patch.state = externalStatus;
-        patch.labels = labelsForStatus(changes.status, this.config);
+        patch.labels = [...labelsForStatus(changes.status, this.config), "feature"];
+      }
+      if (milestone) {
+        const milestoneId = await this.ensureMilestone(milestone);
+        if (milestoneId) patch.milestone = milestoneId;
       }
-      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 +10641,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 +10673,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 +10709,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();
@@ -10662,7 +10748,11 @@ async function syncToExternal(roadmap, adapter, _config) {
           result.errors.push({ featureOrId: feature.name, error: createResult.error });
         }
       } else {
-        const updateResult = await adapter.updateTicket(feature.externalId, feature);
+        const updateResult = await adapter.updateTicket(
+          feature.externalId,
+          feature,
+          milestone.name
+        );
         if (updateResult.ok) {
           result.updated.push(feature.externalId);
         } else {