@harness-engineering/core 0.20.0 → 0.21.0

package/dist/index.d.mts

@@ -6478,7 +6478,7 @@ declare class GitHubIssuesSyncAdapter implements TrackerSyncAdapter {
  * Mutates `roadmap` in-place (stores new externalIds).
  * Never throws -- errors collected per-feature.
  */
-declare function syncToExternal(roadmap: Roadmap, adapter: TrackerSyncAdapter, _config: TrackerSyncConfig): Promise<SyncResult>;
+declare function syncToExternal(roadmap: Roadmap, adapter: TrackerSyncAdapter, config: TrackerSyncConfig, prefetchedTickets?: ExternalTicketState[]): Promise<SyncResult>;
 /**
  * Pull execution fields (assignee, status) from external service.
  * - External assignee wins over local assignee
@@ -6487,7 +6487,7 @@ declare function syncToExternal(roadmap: Roadmap, adapter: TrackerSyncAdapter, _
  * Mutates `roadmap` in-place.
  * Never throws -- errors collected per-feature.
  */
-declare function syncFromExternal(roadmap: Roadmap, adapter: TrackerSyncAdapter, config: TrackerSyncConfig, options?: ExternalSyncOptions): Promise<SyncResult>;
+declare function syncFromExternal(roadmap: Roadmap, adapter: TrackerSyncAdapter, config: TrackerSyncConfig, options?: ExternalSyncOptions, prefetchedTickets?: ExternalTicketState[]): Promise<SyncResult>;
 /**
  * Full bidirectional sync: read roadmap, push, pull, write back.
  * Serialized by in-process mutex.

package/dist/index.d.ts

@@ -6478,7 +6478,7 @@ declare class GitHubIssuesSyncAdapter implements TrackerSyncAdapter {
  * Mutates `roadmap` in-place (stores new externalIds).
  * Never throws -- errors collected per-feature.
  */
-declare function syncToExternal(roadmap: Roadmap, adapter: TrackerSyncAdapter, _config: TrackerSyncConfig): Promise<SyncResult>;
+declare function syncToExternal(roadmap: Roadmap, adapter: TrackerSyncAdapter, config: TrackerSyncConfig, prefetchedTickets?: ExternalTicketState[]): Promise<SyncResult>;
 /**
  * Pull execution fields (assignee, status) from external service.
  * - External assignee wins over local assignee
@@ -6487,7 +6487,7 @@ declare function syncToExternal(roadmap: Roadmap, adapter: TrackerSyncAdapter, _
  * Mutates `roadmap` in-place.
  * Never throws -- errors collected per-feature.
  */
-declare function syncFromExternal(roadmap: Roadmap, adapter: TrackerSyncAdapter, config: TrackerSyncConfig, options?: ExternalSyncOptions): Promise<SyncResult>;
+declare function syncFromExternal(roadmap: Roadmap, adapter: TrackerSyncAdapter, config: TrackerSyncConfig, options?: ExternalSyncOptions, prefetchedTickets?: ExternalTicketState[]): Promise<SyncResult>;
 /**
  * Full bidirectional sync: read roadmap, push, pull, write back.
  * Serialized by in-process mutex.

package/dist/index.js

@@ -13767,6 +13767,7 @@ var GitHubIssuesSyncAdapter = class {
       const data = await response.json();
       return (0, import_types25.Ok)({
         externalId,
+        title: data.title,
         status: data.state,
         labels: data.labels.map((l) => l.name),
         assignee: data.assignee ? `@${data.assignee.login}` : null
@@ -13801,6 +13802,7 @@ var GitHubIssuesSyncAdapter = class {
         for (const issue of issues) {
           tickets.push({
             externalId: buildExternalId(this.owner, this.repo, issue.number),
+            title: issue.title,
             status: issue.state,
             labels: issue.labels.map((l) => l.name),
             assignee: issue.assignee ? `@${issue.assignee.login}` : null
@@ -13845,43 +13847,49 @@ var fs22 = __toESM(require("fs"));
 function emptySyncResult() {
   return { created: [], updated: [], assignmentChanges: [], errors: [] };
 }
-async function syncToExternal(roadmap, adapter, _config) {
+async function syncToExternal(roadmap, adapter, config, prefetchedTickets) {
   const result = emptySyncResult();
-  let defaultAssignee = null;
-  const userResult = await adapter.getAuthenticatedUser();
-  if (userResult.ok) {
-    defaultAssignee = userResult.value;
+  const existingByTitle = /* @__PURE__ */ new Map();
+  const configLabels = new Set((config.labels ?? []).map((l) => l.toLowerCase()));
+  if (prefetchedTickets) {
+    for (const ticket of prefetchedTickets) {
+      const hasConfigLabels = configLabels.size === 0 || ticket.labels.some((l) => configLabels.has(l.toLowerCase()));
+      if (!hasConfigLabels) continue;
+      const key = ticket.title.toLowerCase();
+      const prev = existingByTitle.get(key);
+      if (!prev || prev.status === "closed" && ticket.status === "open") {
+        existingByTitle.set(key, ticket);
+      }
+    }
   }
   for (const milestone of roadmap.milestones) {
     for (const feature of milestone.features) {
-      if (!feature.assignee && defaultAssignee) {
-        feature.assignee = defaultAssignee;
-      }
       if (!feature.externalId) {
-        const createResult = await adapter.createTicket(feature, milestone.name);
-        if (createResult.ok) {
-          feature.externalId = createResult.value.externalId;
-          result.created.push(createResult.value);
+        const existing = existingByTitle.get(feature.name.toLowerCase());
+        if (existing) {
+          feature.externalId = existing.externalId;
         } else {
-          result.errors.push({ featureOrId: feature.name, error: createResult.error });
+          const createResult = await adapter.createTicket(feature, milestone.name);
+          if (createResult.ok) {
+            feature.externalId = createResult.value.externalId;
+            result.created.push(createResult.value);
+          } else {
+            result.errors.push({ featureOrId: feature.name, error: createResult.error });
+          }
+          continue;
         }
+      }
+      const updateResult = await adapter.updateTicket(feature.externalId, feature, milestone.name);
+      if (updateResult.ok) {
+        result.updated.push(feature.externalId);
       } else {
-        const updateResult = await adapter.updateTicket(
-          feature.externalId,
-          feature,
-          milestone.name
-        );
-        if (updateResult.ok) {
-          result.updated.push(feature.externalId);
-        } else {
-          result.errors.push({ featureOrId: feature.externalId, error: updateResult.error });
-        }
+        result.errors.push({ featureOrId: feature.externalId, error: updateResult.error });
       }
     }
   }
   return result;
 }
-async function syncFromExternal(roadmap, adapter, config, options) {
+async function syncFromExternal(roadmap, adapter, config, options, prefetchedTickets) {
   const result = emptySyncResult();
   const forceSync = options?.forceSync ?? false;
   const featureByExternalId = /* @__PURE__ */ new Map();
@@ -13893,12 +13901,18 @@ async function syncFromExternal(roadmap, adapter, config, options) {
     }
   }
   if (featureByExternalId.size === 0) return result;
-  const fetchResult = await adapter.fetchAllTickets();
-  if (!fetchResult.ok) {
-    result.errors.push({ featureOrId: "*", error: fetchResult.error });
-    return result;
+  let tickets;
+  if (prefetchedTickets) {
+    tickets = prefetchedTickets;
+  } else {
+    const fetchResult = await adapter.fetchAllTickets();
+    if (!fetchResult.ok) {
+      result.errors.push({ featureOrId: "*", error: fetchResult.error });
+      return result;
+    }
+    tickets = fetchResult.value;
   }
-  for (const ticketState of fetchResult.value) {
+  for (const ticketState of tickets) {
     const feature = featureByExternalId.get(ticketState.externalId);
     if (!feature) continue;
     if (ticketState.assignee !== feature.assignee) {
@@ -13938,8 +13952,10 @@ async function fullSync(roadmapPath, adapter, config, options) {
       };
     }
     const roadmap = parseResult.value;
-    const pushResult = await syncToExternal(roadmap, adapter, config);
-    const pullResult = await syncFromExternal(roadmap, adapter, config, options);
+    const fetchResult = await adapter.fetchAllTickets();
+    const tickets = fetchResult.ok ? fetchResult.value : void 0;
+    const pushResult = await syncToExternal(roadmap, adapter, config, tickets);
+    const pullResult = await syncFromExternal(roadmap, adapter, config, options, tickets);
     fs22.writeFileSync(roadmapPath, serializeRoadmap(roadmap), "utf-8");
     return {
       created: pushResult.created,

package/dist/index.mjs

@@ -11632,6 +11632,7 @@ var GitHubIssuesSyncAdapter = class {
       const data = await response.json();
       return Ok4({
         externalId,
+        title: data.title,
         status: data.state,
         labels: data.labels.map((l) => l.name),
         assignee: data.assignee ? `@${data.assignee.login}` : null
@@ -11666,6 +11667,7 @@ var GitHubIssuesSyncAdapter = class {
         for (const issue of issues) {
           tickets.push({
             externalId: buildExternalId(this.owner, this.repo, issue.number),
+            title: issue.title,
             status: issue.state,
             labels: issue.labels.map((l) => l.name),
             assignee: issue.assignee ? `@${issue.assignee.login}` : null
@@ -11710,43 +11712,49 @@ import * as fs22 from "fs";
 function emptySyncResult() {
   return { created: [], updated: [], assignmentChanges: [], errors: [] };
 }
-async function syncToExternal(roadmap, adapter, _config) {
+async function syncToExternal(roadmap, adapter, config, prefetchedTickets) {
   const result = emptySyncResult();
-  let defaultAssignee = null;
-  const userResult = await adapter.getAuthenticatedUser();
-  if (userResult.ok) {
-    defaultAssignee = userResult.value;
+  const existingByTitle = /* @__PURE__ */ new Map();
+  const configLabels = new Set((config.labels ?? []).map((l) => l.toLowerCase()));
+  if (prefetchedTickets) {
+    for (const ticket of prefetchedTickets) {
+      const hasConfigLabels = configLabels.size === 0 || ticket.labels.some((l) => configLabels.has(l.toLowerCase()));
+      if (!hasConfigLabels) continue;
+      const key = ticket.title.toLowerCase();
+      const prev = existingByTitle.get(key);
+      if (!prev || prev.status === "closed" && ticket.status === "open") {
+        existingByTitle.set(key, ticket);
+      }
+    }
   }
   for (const milestone of roadmap.milestones) {
     for (const feature of milestone.features) {
-      if (!feature.assignee && defaultAssignee) {
-        feature.assignee = defaultAssignee;
-      }
       if (!feature.externalId) {
-        const createResult = await adapter.createTicket(feature, milestone.name);
-        if (createResult.ok) {
-          feature.externalId = createResult.value.externalId;
-          result.created.push(createResult.value);
+        const existing = existingByTitle.get(feature.name.toLowerCase());
+        if (existing) {
+          feature.externalId = existing.externalId;
         } else {
-          result.errors.push({ featureOrId: feature.name, error: createResult.error });
+          const createResult = await adapter.createTicket(feature, milestone.name);
+          if (createResult.ok) {
+            feature.externalId = createResult.value.externalId;
+            result.created.push(createResult.value);
+          } else {
+            result.errors.push({ featureOrId: feature.name, error: createResult.error });
+          }
+          continue;
         }
+      }
+      const updateResult = await adapter.updateTicket(feature.externalId, feature, milestone.name);
+      if (updateResult.ok) {
+        result.updated.push(feature.externalId);
       } else {
-        const updateResult = await adapter.updateTicket(
-          feature.externalId,
-          feature,
-          milestone.name
-        );
-        if (updateResult.ok) {
-          result.updated.push(feature.externalId);
-        } else {
-          result.errors.push({ featureOrId: feature.externalId, error: updateResult.error });
-        }
+        result.errors.push({ featureOrId: feature.externalId, error: updateResult.error });
       }
     }
   }
   return result;
 }
-async function syncFromExternal(roadmap, adapter, config, options) {
+async function syncFromExternal(roadmap, adapter, config, options, prefetchedTickets) {
   const result = emptySyncResult();
   const forceSync = options?.forceSync ?? false;
   const featureByExternalId = /* @__PURE__ */ new Map();
@@ -11758,12 +11766,18 @@ async function syncFromExternal(roadmap, adapter, config, options) {
     }
   }
   if (featureByExternalId.size === 0) return result;
-  const fetchResult = await adapter.fetchAllTickets();
-  if (!fetchResult.ok) {
-    result.errors.push({ featureOrId: "*", error: fetchResult.error });
-    return result;
+  let tickets;
+  if (prefetchedTickets) {
+    tickets = prefetchedTickets;
+  } else {
+    const fetchResult = await adapter.fetchAllTickets();
+    if (!fetchResult.ok) {
+      result.errors.push({ featureOrId: "*", error: fetchResult.error });
+      return result;
+    }
+    tickets = fetchResult.value;
   }
-  for (const ticketState of fetchResult.value) {
+  for (const ticketState of tickets) {
     const feature = featureByExternalId.get(ticketState.externalId);
     if (!feature) continue;
     if (ticketState.assignee !== feature.assignee) {
@@ -11803,8 +11817,10 @@ async function fullSync(roadmapPath, adapter, config, options) {
       };
     }
     const roadmap = parseResult.value;
-    const pushResult = await syncToExternal(roadmap, adapter, config);
-    const pullResult = await syncFromExternal(roadmap, adapter, config, options);
+    const fetchResult = await adapter.fetchAllTickets();
+    const tickets = fetchResult.ok ? fetchResult.value : void 0;
+    const pushResult = await syncToExternal(roadmap, adapter, config, tickets);
+    const pullResult = await syncFromExternal(roadmap, adapter, config, options, tickets);
     fs22.writeFileSync(roadmapPath, serializeRoadmap(roadmap), "utf-8");
     return {
       created: pushResult.created,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@harness-engineering/core",
-  "version": "0.20.0",
+  "version": "0.21.0",
   "description": "Core library for Harness Engineering toolkit",
   "main": "./dist/index.js",
   "module": "./dist/index.mjs",
@@ -50,7 +50,7 @@
     "web-tree-sitter": "^0.24.7",
     "zod": "^3.25.76",
     "@harness-engineering/graph": "0.4.0",
-    "@harness-engineering/types": "0.8.1"
+    "@harness-engineering/types": "0.9.0"
   },
   "devDependencies": {
     "@types/ejs": "^3.1.5",