paperclip-github-plugin 0.7.0 → 0.7.1

package/dist/manifest.js

@@ -516,7 +516,7 @@ var COMPANY_METRIC_API_ROUTE_URL_PATH = `/api/plugins/${GITHUB_SYNC_PLUGIN_ID}/a
 var require2 = createRequire(import.meta.url);
 var packageJson = require2("../package.json");
 var SCHEDULE_TICK_CRON = "* * * * *";
-var MANIFEST_VERSION = "0.7.0"?.trim() || typeof packageJson.version === "string" && packageJson.version.trim() || process.env.npm_package_version?.trim() || "0.0.0-dev";
+var MANIFEST_VERSION = "0.7.1"?.trim() || typeof packageJson.version === "string" && packageJson.version.trim() || process.env.npm_package_version?.trim() || "0.0.0-dev";
 var manifest = {
   id: GITHUB_SYNC_PLUGIN_ID,
   apiVersion: 1,

package/dist/worker.js

@@ -5026,7 +5026,10 @@ function resolvePaperclipIssueExecutorAssignee(syncContext, advancedSettings) {
   return null;
 }
 function resolveSyncTransitionAssignee(params) {
-  const { nextStatus, syncContext, advancedSettings } = params;
+  const { currentStatus, nextStatus, syncContext, advancedSettings } = params;
+  if (isHealthyMaintainerWaitTransition({ currentStatus, nextStatus, syncContext })) {
+    return null;
+  }
   if (nextStatus === "in_review") {
     return resolvePaperclipIssueReviewAssignee(syncContext, advancedSettings);
   }
@@ -5035,6 +5038,10 @@ function resolveSyncTransitionAssignee(params) {
   }
   return null;
 }
+function isHealthyMaintainerWaitTransition(params) {
+  const { currentStatus, nextStatus, syncContext } = params;
+  return nextStatus === "in_review" && (currentStatus === "done" || currentStatus === "in_review") && syncContext.executionState === null && syncContext.executionPolicy !== null;
+}
 function doesPaperclipIssueAssigneeMatch(currentAssignee, nextAssignee) {
   return isSamePaperclipIssueAssigneePrincipal(currentAssignee, nextAssignee);
 }
@@ -7599,6 +7606,7 @@ async function updatePaperclipIssueState(ctx, params) {
     nextStatus,
     nextAssignee,
     clearAssignee,
+    clearExecutionPolicy,
     transitionComment,
     transitionCommentAnnotation,
     paperclipApiBaseUrl
@@ -7613,7 +7621,8 @@ async function updatePaperclipIssueState(ctx, params) {
   });
   const issuePatch = {
     status: nextStatus,
-    ...syncExecutionStatePatch === null ? { executionState: null } : {}
+    ...syncExecutionStatePatch === null ? { executionState: null } : {},
+    ...clearExecutionPolicy ? { executionPolicy: null, executionState: null } : {}
   };
   if (nextAssignee) {
     if (nextAssignee.kind === "agent") {
@@ -8125,12 +8134,18 @@ async function synchronizePaperclipIssueStatuses(ctx, octokit, repository, mappi
         maintainerAuthoredImportedIssue,
         hasExecutorHandoffTarget: Boolean(executorTransitionAssignee)
       });
+      const shouldPreserveMaintainerWaitRouting = isHealthyMaintainerWaitTransition({
+        currentStatus: paperclipIssue.status,
+        nextStatus,
+        syncContext: paperclipIssueSyncContext
+      });
       const nextTransitionAssignee = resolveSyncTransitionAssignee({
+        currentStatus: paperclipIssue.status,
         nextStatus,
         syncContext: paperclipIssueSyncContext,
         advancedSettings
       });
-      const shouldClearTransitionAssignee = nextStatus === "in_review" && nextTransitionAssignee === null && paperclipIssueSyncContext.assignee !== null;
+      const shouldClearTransitionAssignee = nextStatus === "in_review" && (nextTransitionAssignee === null || shouldPreserveMaintainerWaitRouting) && paperclipIssueSyncContext.assignee !== null;
       const nextAssigneeChanged = nextTransitionAssignee ? !doesPaperclipIssueAssigneeMatch(paperclipIssueSyncContext.assignee, nextTransitionAssignee.principal) : false;
       const shouldWakeImportedAssignee = wasImportedThisRun && paperclipIssue.status === nextStatus && nextStatus === "todo" && paperclipIssueSyncContext.assignee?.kind === "agent";
       const shouldWakeTransitionAssignee = paperclipIssue.status !== nextStatus && nextTransitionAssignee?.principal.kind === "agent" && isActionablePaperclipIssueStatus(nextStatus) && (nextAssigneeChanged || paperclipIssue.status !== nextStatus);
@@ -8152,6 +8167,7 @@ async function synchronizePaperclipIssueStatuses(ctx, octokit, repository, mappi
             syncContext: paperclipIssueSyncContext,
             nextStatus,
             clearAssignee: true,
+            ...shouldPreserveMaintainerWaitRouting ? { clearExecutionPolicy: true } : {},
             transitionComment: "",
             paperclipApiBaseUrl
           });
@@ -8187,6 +8203,7 @@ async function synchronizePaperclipIssueStatuses(ctx, octokit, repository, mappi
         nextStatus,
         ...nextTransitionAssignee ? { nextAssignee: nextTransitionAssignee.principal } : {},
         ...shouldClearTransitionAssignee ? { clearAssignee: true } : {},
+        ...shouldPreserveMaintainerWaitRouting ? { clearExecutionPolicy: true } : {},
         transitionComment: transitionComment.body,
         transitionCommentAnnotation: transitionComment.annotation,
         paperclipApiBaseUrl
@@ -8310,12 +8327,18 @@ async function synchronizePaperclipPullRequestIssueStatuses(ctx, octokit, mappin
         pullRequest,
         hasExecutorHandoffTarget: Boolean(executorTransitionAssignee)
       });
+      const shouldPreserveMaintainerWaitRouting = isHealthyMaintainerWaitTransition({
+        currentStatus: paperclipIssue.status,
+        nextStatus,
+        syncContext: paperclipIssueSyncContext
+      });
       const nextTransitionAssignee = resolveSyncTransitionAssignee({
+        currentStatus: paperclipIssue.status,
         nextStatus,
         syncContext: paperclipIssueSyncContext,
         advancedSettings
       });
-      const shouldClearTransitionAssignee = nextStatus === "in_review" && nextTransitionAssignee === null && paperclipIssueSyncContext.assignee !== null;
+      const shouldClearTransitionAssignee = nextStatus === "in_review" && (nextTransitionAssignee === null || shouldPreserveMaintainerWaitRouting) && paperclipIssueSyncContext.assignee !== null;
       const nextAssigneeChanged = nextTransitionAssignee ? !doesPaperclipIssueAssigneeMatch(paperclipIssueSyncContext.assignee, nextTransitionAssignee.principal) : false;
       const shouldWakeTransitionAssignee = paperclipIssue.status !== nextStatus && nextTransitionAssignee?.principal.kind === "agent" && isActionablePaperclipIssueStatus(nextStatus) && (nextAssigneeChanged || paperclipIssue.status !== nextStatus);
       if (paperclipIssue.status === nextStatus) {
@@ -8332,6 +8355,7 @@ async function synchronizePaperclipPullRequestIssueStatuses(ctx, octokit, mappin
             syncContext: paperclipIssueSyncContext,
             nextStatus,
             clearAssignee: true,
+            ...shouldPreserveMaintainerWaitRouting ? { clearExecutionPolicy: true } : {},
             transitionComment: "",
             paperclipApiBaseUrl
           });
@@ -8356,6 +8380,7 @@ async function synchronizePaperclipPullRequestIssueStatuses(ctx, octokit, mappin
         nextStatus,
         ...nextTransitionAssignee ? { nextAssignee: nextTransitionAssignee.principal } : {},
         ...shouldClearTransitionAssignee ? { clearAssignee: true } : {},
+        ...shouldPreserveMaintainerWaitRouting ? { clearExecutionPolicy: true } : {},
         transitionComment,
         paperclipApiBaseUrl
       });
@@ -13235,6 +13260,11 @@ function shouldStartWorkerHost(moduleUrl, entry = process.argv[1]) {
     return resolve(entry) === resolve(modulePath);
   }
 }
+var __testing = {
+  buildSyncFallbackExecutionStatePatch,
+  isHealthyMaintainerWaitTransition,
+  resolveSyncTransitionAssignee
+};
 var plugin = definePlugin({
   async setup(ctx) {
     pluginRuntimeContext = ctx;
@@ -13617,6 +13647,7 @@ if (shouldStartWorkerHost(import.meta.url)) {
   startWorkerRpcHost({ plugin });
 }
 export {
+  __testing,
   worker_default as default,
   shouldStartWorkerHost
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "paperclip-github-plugin",
-  "version": "0.7.0",
+  "version": "0.7.1",
   "description": "Paperclip plugin for synchronizing GitHub issues into Paperclip projects.",
   "license": "Apache-2.0",
   "type": "module",