paperclip-github-plugin 0.9.12 → 0.9.14

package/README.md

@@ -186,7 +186,6 @@ 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 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.12"?.trim() || typeof packageJson.version === "string" && packageJson.version.trim() || process.env.npm_package_version?.trim() || "0.0.0-dev";
+var MANIFEST_VERSION = "0.9.14"?.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

@@ -5421,8 +5421,7 @@ function normalizePaperclipIssueExecutionState(value) {
   const lastDecisionId = normalizeOptionalString2(record.lastDecisionId) ?? null;
   const lastDecisionOutcome = normalizeOptionalString2(record.lastDecisionOutcome) ?? null;
   const completedStageIds = Array.isArray(record.completedStageIds) ? record.completedStageIds.map((stageId) => normalizeOptionalString2(stageId)).filter((stageId) => Boolean(stageId)) : [];
-  const monitor = normalizePaperclipIssueMonitorState(record.monitor);
-  if (!status && currentStageId === null && currentStageIndex === null && currentStageType === null && currentParticipant === null && returnAssignee === null && completedStageIds.length === 0 && lastDecisionId === null && lastDecisionOutcome === null && monitor === null) {
+  if (!status && currentStageId === null && currentStageIndex === null && currentStageType === null && currentParticipant === null && returnAssignee === null && completedStageIds.length === 0 && lastDecisionId === null && lastDecisionOutcome === null) {
     return null;
   }
   return {
@@ -5434,25 +5433,7 @@ function normalizePaperclipIssueExecutionState(value) {
     returnAssignee,
     completedStageIds,
     ...lastDecisionId !== null ? { lastDecisionId } : {},
-    ...lastDecisionOutcome !== null ? { lastDecisionOutcome } : {},
-    ...monitor !== null ? { monitor } : {}
-  };
-}
-function normalizePaperclipIssueMonitorState(value) {
-  if (!value || typeof value !== "object") {
-    return null;
-  }
-  const record = value;
-  const status = normalizeOptionalString2(record.status);
-  const clearedAt = normalizeOptionalString2(record.clearedAt) ?? null;
-  const clearReason = normalizeOptionalString2(record.clearReason) ?? null;
-  if (!status && clearedAt === null && clearReason === null && Object.keys(record).length === 0) {
-    return null;
-  }
-  return {
-    ...status ? { status } : {},
-    clearedAt,
-    clearReason
+    ...lastDecisionOutcome !== null ? { lastDecisionOutcome } : {}
   };
 }
 function getPaperclipIssueAssigneePrincipal(issue) {
@@ -5478,17 +5459,6 @@ function getPaperclipIssueSyncContext(issue) {
     executionState: normalizePaperclipIssueExecutionState(record.executionState)
   };
 }
-function hasScheduledPaperclipIssueMonitor(syncContext) {
-  const monitor = syncContext.executionState?.monitor;
-  if (!monitor) {
-    return false;
-  }
-  if (monitor.clearedAt || monitor.clearReason) {
-    return false;
-  }
-  const normalizedStatus = monitor.status?.trim().toLowerCase();
-  return normalizedStatus === "scheduled";
-}
 function hasUnresolvedPaperclipIssueBlockerSummary(blockers) {
   if (!Array.isArray(blockers)) {
     return false;
@@ -5708,13 +5678,6 @@ function isClearableMaintainerWaitExecutionState(executionState) {
   if (executionState === null) {
     return true;
   }
-  if (hasScheduledPaperclipIssueMonitor({
-    assignee: null,
-    executionPolicy: null,
-    executionState
-  })) {
-    return false;
-  }
   if (executionState.currentParticipant !== null || executionState.returnAssignee !== null || executionState.currentStageId !== null || executionState.currentStageIndex !== null || executionState.currentStageType !== null || executionState.completedStageIds.length > 0 || executionState.lastDecisionId || executionState.lastDecisionOutcome) {
     return false;
   }
@@ -8998,7 +8961,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,18 +9522,6 @@ async function cancelUnmappedTransferredGitHubIssue(ctx, params) {
     };
   }
   const paperclipIssueSyncContext = getPaperclipIssueSyncContext(paperclipIssue);
-  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,
-      currentStatus: paperclipIssue.status,
-      resolvedStatus: "cancelled"
-    });
-    return {
-      updatedStatusesCount: 0
-    };
-  }
   await unlinkPaperclipIssueFromGitHub(ctx, {
     companyId,
     issueId: params.importedIssue.paperclipIssueId
@@ -9899,17 +9850,6 @@ async function synchronizePaperclipIssueStatuses(ctx, octokit, repository, mappi
       importedIssue.lastSeenCommentCount = snapshot.commentCount;
       importedIssue.lastSeenGitHubState = snapshot.state;
       importedIssue.linkedPullRequestCommentCounts = currentLinkedPullRequestCommentCounts;
-      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,
-          githubIssueNumber: githubIssue.number,
-          currentStatus: paperclipIssue.status,
-          resolvedStatus: nextStatus
-        });
-        continue;
-      }
       if (paperclipIssue.status === nextStatus) {
         if (shouldClearTransitionAssignee || shouldClearCompletedExecutionPolicy) {
           updateSyncFailureContext(syncFailureContext, {
@@ -10166,16 +10106,6 @@ 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 (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,
-          currentStatus: paperclipIssue.status,
-          resolvedStatus: nextStatus
-        });
-        continue;
-      }
       if (paperclipIssue.status === nextStatus) {
         if (shouldClearTransitionAssignee || shouldClearCompletedExecutionPolicy) {
           updateSyncFailureContext(syncFailureContext, {

package/package.json

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