@harness-engineering/core 0.17.1 → 0.18.0

package/dist/index.js

@@ -12605,6 +12605,8 @@ var GitHubIssuesSyncAdapter = class {
   owner;
   repo;
   retryOpts;
+  /** Cached GitHub milestone name -> ID mapping */
+  milestoneCache = null;
   constructor(options) {
     this.token = options.token;
     this.config = options.config;
@@ -12621,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}`,
@@ -12645,21 +12684,15 @@ var GitHubIssuesSyncAdapter = class {
   }
   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 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) {
@@ -12674,7 +12707,7 @@ var GitHubIssuesSyncAdapter = class {
       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}"`));
@@ -12687,7 +12720,11 @@ 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 fetchWithRetry(
         this.fetchFn,
@@ -12820,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

@@ -10496,6 +10496,8 @@ var GitHubIssuesSyncAdapter = class {
   owner;
   repo;
   retryOpts;
+  /** Cached GitHub milestone name -> ID mapping */
+  milestoneCache = null;
   constructor(options) {
     this.token = options.token;
     this.config = options.config;
@@ -10512,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}`,
@@ -10536,21 +10575,15 @@ var GitHubIssuesSyncAdapter = class {
   }
   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 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) {
@@ -10565,7 +10598,7 @@ var GitHubIssuesSyncAdapter = class {
       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}"`));
@@ -10578,7 +10611,11 @@ 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 fetchWithRetry(
         this.fetchFn,
@@ -10711,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 {

package/dist/{matchers-D20x48U9.d.mts → matchers-Dj1t5vpg.d.mts}

@@ -9,17 +9,17 @@ declare const ViolationSchema: z.ZodObject<{
     detail: z.ZodString;
     severity: z.ZodEnum<["error", "warning"]>;
 }, "strip", z.ZodTypeAny, {
-    file: string;
     id: string;
-    severity: "error" | "warning";
+    file: string;
     detail: string;
-    category?: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
+    severity: "error" | "warning";
+    category?: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
 }, {
-    file: string;
     id: string;
-    severity: "error" | "warning";
+    file: string;
     detail: string;
-    category?: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
+    severity: "error" | "warning";
+    category?: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
 }>;
 type Violation = z.infer<typeof ViolationSchema>;
 declare const MetricResultSchema: z.ZodObject<{
@@ -33,41 +33,41 @@ declare const MetricResultSchema: z.ZodObject<{
         detail: z.ZodString;
         severity: z.ZodEnum<["error", "warning"]>;
     }, "strip", z.ZodTypeAny, {
-        file: string;
         id: string;
-        severity: "error" | "warning";
+        file: string;
         detail: string;
-        category?: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
+        severity: "error" | "warning";
+        category?: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
     }, {
-        file: string;
         id: string;
-        severity: "error" | "warning";
+        file: string;
         detail: string;
-        category?: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
+        severity: "error" | "warning";
+        category?: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
     }>, "many">;
     metadata: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodUnknown>>;
 }, "strip", z.ZodTypeAny, {
     value: number;
+    category: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth";
     scope: string;
-    category: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth";
     violations: {
-        file: string;
         id: string;
-        severity: "error" | "warning";
+        file: string;
         detail: string;
-        category?: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
+        severity: "error" | "warning";
+        category?: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
     }[];
     metadata?: Record<string, unknown> | undefined;
 }, {
     value: number;
+    category: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth";
     scope: string;
-    category: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth";
     violations: {
-        file: string;
         id: string;
-        severity: "error" | "warning";
+        file: string;
         detail: string;
-        category?: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
+        severity: "error" | "warning";
+        category?: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
     }[];
     metadata?: Record<string, unknown> | undefined;
 }>;
@@ -101,7 +101,7 @@ declare const ArchBaselineSchema: z.ZodObject<{
     version: 1;
     updatedAt: string;
     updatedFrom: string;
-    metrics: Partial<Record<"complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth", {
+    metrics: Partial<Record<"circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth", {
         value: number;
         violationIds: string[];
     }>>;
@@ -109,7 +109,7 @@ declare const ArchBaselineSchema: z.ZodObject<{
     version: 1;
     updatedAt: string;
     updatedFrom: string;
-    metrics: Partial<Record<"complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth", {
+    metrics: Partial<Record<"circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth", {
         value: number;
         violationIds: string[];
     }>>;
@@ -121,12 +121,12 @@ declare const CategoryRegressionSchema: z.ZodObject<{
     currentValue: z.ZodNumber;
     delta: z.ZodNumber;
 }, "strip", z.ZodTypeAny, {
-    category: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth";
+    category: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth";
     baselineValue: number;
     currentValue: number;
     delta: number;
 }, {
-    category: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth";
+    category: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth";
     baselineValue: number;
     currentValue: number;
     delta: number;
@@ -141,17 +141,17 @@ declare const ArchDiffResultSchema: z.ZodObject<{
         detail: z.ZodString;
         severity: z.ZodEnum<["error", "warning"]>;
     }, "strip", z.ZodTypeAny, {
-        file: string;
         id: string;
-        severity: "error" | "warning";
+        file: string;
         detail: string;
-        category?: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
+        severity: "error" | "warning";
+        category?: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
     }, {
-        file: string;
         id: string;
-        severity: "error" | "warning";
+        file: string;
         detail: string;
-        category?: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
+        severity: "error" | "warning";
+        category?: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
     }>, "many">;
     resolvedViolations: z.ZodArray<z.ZodString, "many">;
     preExisting: z.ZodArray<z.ZodString, "many">;
@@ -161,12 +161,12 @@ declare const ArchDiffResultSchema: z.ZodObject<{
         currentValue: z.ZodNumber;
         delta: z.ZodNumber;
     }, "strip", z.ZodTypeAny, {
-        category: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth";
+        category: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth";
         baselineValue: number;
         currentValue: number;
         delta: number;
     }, {
-        category: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth";
+        category: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth";
         baselineValue: number;
         currentValue: number;
         delta: number;
@@ -174,16 +174,16 @@ declare const ArchDiffResultSchema: z.ZodObject<{
 }, "strip", z.ZodTypeAny, {
     passed: boolean;
     newViolations: {
-        file: string;
         id: string;
-        severity: "error" | "warning";
+        file: string;
         detail: string;
-        category?: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
+        severity: "error" | "warning";
+        category?: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
     }[];
     resolvedViolations: string[];
     preExisting: string[];
     regressions: {
-        category: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth";
+        category: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth";
         baselineValue: number;
         currentValue: number;
         delta: number;
@@ -191,16 +191,16 @@ declare const ArchDiffResultSchema: z.ZodObject<{
 }, {
     passed: boolean;
     newViolations: {
-        file: string;
         id: string;
-        severity: "error" | "warning";
+        file: string;
         detail: string;
-        category?: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
+        severity: "error" | "warning";
+        category?: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth" | undefined;
     }[];
     resolvedViolations: string[];
     preExisting: string[];
     regressions: {
-        category: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth";
+        category: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth";
         baselineValue: number;
         currentValue: number;
         delta: number;
@@ -215,15 +215,15 @@ declare const ArchConfigSchema: z.ZodObject<{
     thresholds: z.ZodDefault<z.ZodRecord<z.ZodEnum<["circular-deps", "layer-violations", "complexity", "coupling", "forbidden-imports", "module-size", "dependency-depth"]>, z.ZodUnion<[z.ZodNumber, z.ZodRecord<z.ZodString, z.ZodNumber>]>>>;
     modules: z.ZodDefault<z.ZodRecord<z.ZodString, z.ZodRecord<z.ZodEnum<["circular-deps", "layer-violations", "complexity", "coupling", "forbidden-imports", "module-size", "dependency-depth"]>, z.ZodUnion<[z.ZodNumber, z.ZodRecord<z.ZodString, z.ZodNumber>]>>>>;
 }, "strip", z.ZodTypeAny, {
-    thresholds: Partial<Record<"complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth", number | Record<string, number>>>;
-    modules: Record<string, Partial<Record<"complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth", number | Record<string, number>>>>;
     enabled: boolean;
     baselinePath: string;
+    thresholds: Partial<Record<"circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth", number | Record<string, number>>>;
+    modules: Record<string, Partial<Record<"circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth", number | Record<string, number>>>>;
 }, {
-    thresholds?: Partial<Record<"complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth", number | Record<string, number>>> | undefined;
-    modules?: Record<string, Partial<Record<"complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth", number | Record<string, number>>>> | undefined;
     enabled?: boolean | undefined;
     baselinePath?: string | undefined;
+    thresholds?: Partial<Record<"circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth", number | Record<string, number>>> | undefined;
+    modules?: Record<string, Partial<Record<"circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth", number | Record<string, number>>>> | undefined;
 }>;
 type ArchConfig = z.infer<typeof ArchConfigSchema>;
 declare const ConstraintRuleSchema: z.ZodObject<{
@@ -233,16 +233,16 @@ declare const ConstraintRuleSchema: z.ZodObject<{
     scope: z.ZodString;
     targets: z.ZodOptional<z.ZodArray<z.ZodString, "many">>;
 }, "strip", z.ZodTypeAny, {
+    category: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth";
     scope: string;
     id: string;
     description: string;
-    category: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth";
     targets?: string[] | undefined;
 }, {
+    category: "circular-deps" | "layer-violations" | "complexity" | "coupling" | "forbidden-imports" | "module-size" | "dependency-depth";
     scope: string;
     id: string;
     description: string;
-    category: "complexity" | "coupling" | "circular-deps" | "layer-violations" | "forbidden-imports" | "module-size" | "dependency-depth";
     targets?: string[] | undefined;
 }>;
 type ConstraintRule = z.infer<typeof ConstraintRuleSchema>;