paperclip-github-plugin 0.9.11 → 0.9.13

package/README.md

@@ -186,7 +186,7 @@ Additional behavior:
 - Sync-driven handoffs to agent assignees best-effort enqueue an explicit wakeup so the next reviewer, approver, or executor can pick the issue up even when their agent is not running heartbeats.
 - Open imported issues that are already in `backlog` stay in `backlog` until someone changes them in Paperclip.
 - If an imported issue is `done` or `cancelled` and GitHub shows it open again with no linked pull request, sync moves it to `todo` so agents can pick it up again.
-- When an issue has an active Paperclip issue monitor, GitHub Sync lets that monitor own the wait. It still refreshes GitHub metadata, but it does not change the issue's Paperclip status, assignee, execution state, transition comments, or wakeups until the monitor is no longer active.
+- When an issue has a scheduled Paperclip issue monitor, GitHub Sync lets that monitor own the wait. It still refreshes GitHub metadata, but it does not change the issue's Paperclip status, assignee, execution state, transition comments, or wakeups until the monitor fires or is cleared.
 - Trusted new GitHub comments from the original issue author or a verified maintainer/admin can move an open imported issue back into active work, whether the new comment lands on the source issue, in a linked pull request's top-level comment stream, or in a linked pull request review thread; GitHub Sync uses `in_progress` when it can route the issue to an executor and otherwise `todo`.
 - When the sync changes a Paperclip issue status, it adds a Paperclip comment explaining what changed and why.
 

package/dist/manifest.js

@@ -642,7 +642,7 @@ var PULL_REQUEST_ASSET_API_ROUTE_URL_PATH = `/api/plugins/${GITHUB_SYNC_PLUGIN_I
 var require2 = createRequire(import.meta.url);
 var packageJson = require2("../package.json");
 var SCHEDULE_TICK_CRON = "* * * * *";
-var MANIFEST_VERSION = "0.9.11"?.trim() || typeof packageJson.version === "string" && packageJson.version.trim() || process.env.npm_package_version?.trim() || "0.0.0-dev";
+var MANIFEST_VERSION = "0.9.13"?.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

@@ -5478,7 +5478,7 @@ function getPaperclipIssueSyncContext(issue) {
     executionState: normalizePaperclipIssueExecutionState(record.executionState)
   };
 }
-function hasActivePaperclipIssueMonitor(syncContext) {
+function hasScheduledPaperclipIssueMonitor(syncContext) {
   const monitor = syncContext.executionState?.monitor;
   if (!monitor) {
     return false;
@@ -5487,7 +5487,7 @@ function hasActivePaperclipIssueMonitor(syncContext) {
     return false;
   }
   const normalizedStatus = monitor.status?.trim().toLowerCase();
-  return normalizedStatus !== "cleared" && normalizedStatus !== "completed" && normalizedStatus !== "cancelled" && normalizedStatus !== "canceled";
+  return normalizedStatus === "scheduled";
 }
 function hasUnresolvedPaperclipIssueBlockerSummary(blockers) {
   if (!Array.isArray(blockers)) {
@@ -5708,7 +5708,7 @@ function isClearableMaintainerWaitExecutionState(executionState) {
   if (executionState === null) {
     return true;
   }
-  if (hasActivePaperclipIssueMonitor({
+  if (hasScheduledPaperclipIssueMonitor({
     assignee: null,
     executionPolicy: null,
     executionState
@@ -8998,7 +8998,7 @@ async function updatePaperclipIssueState(ctx, params) {
   });
   const issuePatch = {
     status: nextStatus,
-    ...syncExecutionStatePatch === null ? { executionState: null } : {},
+    ...syncExecutionStatePatch !== void 0 ? { executionState: syncExecutionStatePatch } : {},
     ...clearExecutionPolicy ? { executionPolicy: null, executionState: null } : {}
   };
   if (nextAssignee) {
@@ -9559,8 +9559,8 @@ async function cancelUnmappedTransferredGitHubIssue(ctx, params) {
     };
   }
   const paperclipIssueSyncContext = getPaperclipIssueSyncContext(paperclipIssue);
-  if (hasActivePaperclipIssueMonitor(paperclipIssueSyncContext)) {
-    ctx.logger.info("GitHub sync skipped transferred issue cancellation because an issue monitor is active.", {
+  if (hasScheduledPaperclipIssueMonitor(paperclipIssueSyncContext)) {
+    ctx.logger.info("GitHub sync skipped transferred issue cancellation because an issue monitor is scheduled.", {
       companyId,
       issueId: params.importedIssue.paperclipIssueId,
       transferredRepositoryUrl: params.transferredRepository.url,
@@ -9899,8 +9899,8 @@ async function synchronizePaperclipIssueStatuses(ctx, octokit, repository, mappi
       importedIssue.lastSeenCommentCount = snapshot.commentCount;
       importedIssue.lastSeenGitHubState = snapshot.state;
       importedIssue.linkedPullRequestCommentCounts = currentLinkedPullRequestCommentCounts;
-      if (hasActivePaperclipIssueMonitor(paperclipIssueSyncContext)) {
-        ctx.logger.info("GitHub sync skipped Paperclip issue state changes because an issue monitor is active.", {
+      if (hasScheduledPaperclipIssueMonitor(paperclipIssueSyncContext)) {
+        ctx.logger.info("GitHub sync skipped Paperclip issue state changes because an issue monitor is scheduled.", {
           companyId: mapping.companyId,
           issueId: importedIssue.paperclipIssueId,
           repositoryUrl: repository.url,
@@ -10166,8 +10166,8 @@ async function synchronizePaperclipPullRequestIssueStatuses(ctx, octokit, mappin
       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 (hasActivePaperclipIssueMonitor(paperclipIssueSyncContext)) {
-        ctx.logger.info("GitHub sync skipped Paperclip pull request issue state changes because an issue monitor is active.", {
+      if (hasScheduledPaperclipIssueMonitor(paperclipIssueSyncContext)) {
+        ctx.logger.info("GitHub sync skipped Paperclip pull request issue state changes because an issue monitor is scheduled.", {
           companyId: mapping.companyId,
           issueId: paperclipIssueId,
           repositoryUrl: primaryRepository?.url,

package/package.json

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