paperclip-github-plugin 0.7.0 → 0.7.2

package/README.md

@@ -89,7 +89,7 @@ Paperclip agents can search GitHub for duplicates, read and update issues, post
 ## Requirements
 
 - Node.js 20+
-- a Paperclip host on `2026.427.0` or newer with plugin installation enabled
+- a Paperclip host with plugin installation enabled. GitHub Sync is built and tested against Paperclip `2026.427.0`; the manifest relies on explicit capabilities instead of a strict host-version gate because current latest/development hosts can report `0.0.0` during plugin upgrade.
 - a GitHub token with API access to the repositories you want to sync
 
 ## Install from npm
@@ -248,6 +248,7 @@ Because the KPI attribution endpoint is a native plugin JSON route rather than a
 
 ## Troubleshooting
 
+- If an older GitHub Sync build fails upgrade with `requires host version 2026.427.0 or newer, but this server is running 0.0.0`, upgrade to a build that removes the strict manifest host-version gate. The host is reporting a development-version sentinel, so the plugin now relies on declared capabilities and runtime fallbacks instead.
 - If setup is reported as incomplete, confirm that a GitHub token has been saved or that `${PAPERCLIP_HOME:-~/.paperclip}/plugins/github-sync/config.json` contains `githubToken`, and make sure at least one mapping has a created Paperclip project.
 - If Paperclip says board access is required, open plugin settings inside the affected company and complete the Paperclip board access flow before retrying sync.
 - If GitHub Sync agent tools fail with `403 {"error":"Board access required"}` on `/api/plugins/tools` or `/api/plugins/tools/execute`, the current Paperclip host rejected the request before the plugin worker ran. Re-run from a board-authenticated session or agent run that has board access to the target company.

package/dist/manifest.js

@@ -516,12 +516,11 @@ 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.2"?.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,
   version: MANIFEST_VERSION,
-  minimumHostVersion: "2026.427.0",
   displayName: "GitHub Sync",
   description: "Synchronize GitHub issues into Paperclip projects.",
   author: "\xC1lvaro S\xE1nchez-Mariscal",

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.2",
   "description": "Paperclip plugin for synchronizing GitHub issues into Paperclip projects.",
   "license": "Apache-2.0",
   "type": "module",