paperclip-github-plugin 0.8.8 → 0.8.9

package/dist/worker.js

@@ -782,15 +782,15 @@ var PaperclipLabelSyncError = class extends Error {
     const location = params.paperclipApiBaseUrl ? ` at ${params.paperclipApiBaseUrl}` : "";
     let message;
     if (failure?.requiresAuthentication) {
-      message = `Could not map ${labelSubject} because the worker reached an authenticated Paperclip API response${location} instead of JSON. Connect Paperclip board access in plugin settings, set \`PAPERCLIP_API_URL\` to a worker-accessible Paperclip API origin, or expose the local Paperclip API to the worker without browser-session auth.`;
+      message = `Could not map ${labelSubject} because the worker reached an authenticated Paperclip API response${location} instead of JSON. Connect Paperclip board access in plugin settings, set Worker Paperclip API URL to a worker-accessible Paperclip API origin, or expose the local Paperclip API to the worker without browser-session auth.`;
     } else if (failure?.status === 404 || failure?.status === 405) {
-      message = `Could not map ${labelSubject} because the Paperclip label API${location} is not available to the worker. Set \`PAPERCLIP_API_URL\` to a worker-accessible Paperclip API origin, then retry sync.`;
+      message = `Could not map ${labelSubject} because the Paperclip label API${location} is not available to the worker. Set Worker Paperclip API URL to a worker-accessible Paperclip API origin, then retry sync.`;
     } else if (failure?.errorMessage) {
       message = `Could not map ${labelSubject} because the Paperclip label API${location} failed: ${failure.errorMessage}`;
     } else if (params.paperclipApiBaseUrl) {
       message = `Could not map ${labelSubject} because the Paperclip label API at ${params.paperclipApiBaseUrl} is unavailable to the worker.`;
     } else {
-      message = `Could not map ${labelSubject} because no worker-accessible Paperclip label API is configured. Set \`PAPERCLIP_API_URL\` to a worker-accessible Paperclip API origin, then retry sync.`;
+      message = `Could not map ${labelSubject} because no worker-accessible Paperclip label API is configured. Set Worker Paperclip API URL to a worker-accessible Paperclip API origin, then retry sync.`;
     }
     super(message);
     this.status = failure?.status;
@@ -1514,6 +1514,39 @@ function getErrorMessage(error) {
   }
   return String(error);
 }
+function getErrorCause(error) {
+  if (!error || typeof error !== "object" || !("cause" in error)) {
+    return void 0;
+  }
+  return error.cause;
+}
+function getErrorCode(error) {
+  if (!error || typeof error !== "object" || !("code" in error)) {
+    return void 0;
+  }
+  const code = error.code;
+  return typeof code === "string" && code.trim() ? code.trim() : void 0;
+}
+function getErrorDiagnosticMessage(error) {
+  const primaryMessage = getErrorMessage(error).trim();
+  const cause = getErrorCause(error);
+  const causeMessage = cause ? getErrorMessage(cause).trim() : "";
+  const errorCode = getErrorCode(error);
+  const causeCode = cause ? getErrorCode(cause) : void 0;
+  const code = errorCode ?? causeCode;
+  const parts = [primaryMessage || String(error)];
+  if (causeMessage && causeMessage !== primaryMessage) {
+    parts.push(`cause: ${causeMessage}`);
+  }
+  if (code) {
+    parts.push(`code: ${code}`);
+  }
+  return parts.join(" | ");
+}
+function formatPaperclipApiFetchErrorMessage(error, url, init) {
+  const method = typeof init?.method === "string" && init.method.trim() ? init.method.trim().toUpperCase() : "GET";
+  return `Paperclip API fetch failed (${method} ${url}): ${getErrorDiagnosticMessage(error)}`;
+}
 function isPaperclipLabelSyncError(error) {
   return error instanceof PaperclipLabelSyncError;
 }
@@ -2135,12 +2168,12 @@ function getSyncFailureSuggestedAction(error, context) {
   }
   if (isPaperclipLabelSyncError(error)) {
     if (error.requiresAuthentication || error.status === 401 || error.status === 403) {
-      return "The worker could not reuse the board login session for the Paperclip label API. Connect Paperclip board access in settings, or set `PAPERCLIP_API_URL` to a worker-accessible Paperclip API origin, then retry sync.";
+      return "The worker could not reuse the board login session for the Paperclip label API. Connect Paperclip board access in settings, or set Worker Paperclip API URL to a worker-accessible Paperclip API origin, then retry sync.";
     }
     if (error.paperclipApiBaseUrl) {
       return `Confirm that the Paperclip label API at ${error.paperclipApiBaseUrl} is reachable from the plugin worker and returns JSON, then retry sync.`;
     }
-    return "Set `PAPERCLIP_API_URL` to a worker-accessible Paperclip API origin, then retry sync.";
+    return "Set Worker Paperclip API URL to a worker-accessible Paperclip API origin, then retry sync.";
   }
   const rawMessage = getErrorMessage(error).trim().toLowerCase();
   if (rawMessage.includes("could not resolve to a pullrequest")) {
@@ -3955,17 +3988,7 @@ function normalizePaperclipApiBaseUrlByCompanyId(value) {
   }).filter((entry) => entry !== null);
   return entries.length > 0 ? Object.fromEntries(entries) : void 0;
 }
-function getRuntimePaperclipApiBaseUrl() {
-  if (typeof process === "undefined" || !process?.env) {
-    return void 0;
-  }
-  return normalizePaperclipApiBaseUrl(process.env.PAPERCLIP_API_URL);
-}
 function resolvePaperclipApiBaseUrl(...values) {
-  const runtimePaperclipApiBaseUrl = getRuntimePaperclipApiBaseUrl();
-  if (runtimePaperclipApiBaseUrl) {
-    return runtimePaperclipApiBaseUrl;
-  }
   for (const value of values) {
     const normalizedValue = normalizePaperclipApiBaseUrl(value);
     if (normalizedValue) {
@@ -3983,10 +4006,6 @@ function getConfiguredPaperclipApiBaseUrl(settings, config, companyId) {
   ) : resolvePaperclipApiBaseUrl(config?.paperclipApiBaseUrl, settings?.paperclipApiBaseUrl);
 }
 function resolveTrustedPaperclipApiBaseUrlInput(value, settings, config, companyId) {
-  const runtimePaperclipApiBaseUrl = getRuntimePaperclipApiBaseUrl();
-  if (runtimePaperclipApiBaseUrl) {
-    return runtimePaperclipApiBaseUrl;
-  }
   const requestedPaperclipApiBaseUrl = normalizePaperclipApiBaseUrl(value);
   const configuredPaperclipApiBaseUrl = normalizePaperclipApiBaseUrl(config?.paperclipApiBaseUrl);
   const normalizedCompanyId = normalizeCompanyId(companyId);
@@ -5637,9 +5656,6 @@ function resolvePaperclipIssueStatus(params) {
 }
 function resolvePaperclipPullRequestIssueStatus(params) {
   const { currentStatus, pullRequest, hasExecutorHandoffTarget } = params;
-  if (currentStatus === "done" || currentStatus === "cancelled") {
-    return currentStatus;
-  }
   if (shouldPreserveBlockedExternalPullRequestWait({
     currentStatus,
     linkedPullRequests: [pullRequest]
@@ -5648,7 +5664,7 @@ function resolvePaperclipPullRequestIssueStatus(params) {
   }
   return resolvePaperclipStatusFromLinkedPullRequests([pullRequest], {
     preferInProgress: hasExecutorHandoffTarget,
-    preserveTransientUnknownMergeabilityWait: currentStatus === "in_review"
+    preserveTransientUnknownMergeabilityWait: currentStatus === "done" || currentStatus === "in_review"
   });
 }
 async function listLinkedPullRequestsForIssue(octokit, repository, issueNumber) {
@@ -7828,7 +7844,14 @@ function applyPaperclipApiAuthentication(init, companyId) {
   };
 }
 async function fetchPaperclipApi(url, init, options) {
-  return fetch(url, applyPaperclipApiAuthentication(init, options?.companyId));
+  const authenticatedInit = applyPaperclipApiAuthentication(init, options?.companyId);
+  try {
+    return await fetch(url, authenticatedInit);
+  } catch (error) {
+    throw new Error(formatPaperclipApiFetchErrorMessage(error, url, authenticatedInit), {
+      cause: error
+    });
+  }
 }
 async function detectPaperclipBoardAccessRequirement(paperclipApiBaseUrl) {
   if (!paperclipApiBaseUrl) {
@@ -15113,6 +15136,7 @@ function shouldStartWorkerHost(moduleUrl, entry = process.argv[1]) {
 var __testing = {
   buildSyncFallbackExecutionStatePatch,
   createGitHubToolOctokit,
+  formatPaperclipApiFetchErrorMessage,
   hasUnresolvedPaperclipIssueBlocker,
   isHealthyMaintainerWaitTransition,
   resolveSyncTransitionAssignee
@@ -15148,6 +15172,7 @@ var plugin = definePlugin({
         ...getPublicSettingsForScope(settingsForResponse, requestedCompanyId),
         ...includeAssignees ? { availableAssignees } : {},
         totalSyncedIssuesCount: countImportedIssuesForMappings(importRegistry, scopedMappings),
+        paperclipApiBaseUrlConfigured: Boolean(normalizePaperclipApiBaseUrl(config.paperclipApiBaseUrl)),
         githubTokenConfigured,
         paperclipBoardAccessConfigured: requestedCompanyId ? hasConfiguredPaperclipBoardAccess(settingsForResponse, config, requestedCompanyId) : hasConfiguredPaperclipBoardAccessForMappings(settingsForResponse, config, scopedMappings),
         ...savedBoardTokenRef ? { paperclipBoardAccessConfigSyncRef: savedBoardTokenRef } : {},

package/package.json

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