paperclip-github-plugin 0.5.0 → 0.5.2

package/dist/ui/index.js

@@ -22817,6 +22817,14 @@ var HOST_DESTRUCTIVE_BUTTON_CLASSNAME = [
   "bg-destructive text-white hover:bg-destructive/90",
   "focus-visible:ring-destructive/20 dark:focus-visible:ring-destructive/40"
 ].join(" ");
+var HOST_SUCCESS_BUTTON_CLASSNAME = [
+  HOST_BUTTON_BASE_CLASSNAME,
+  "ghsync__button-tone ghsync__button-tone--success"
+].join(" ");
+var HOST_WARNING_BUTTON_CLASSNAME = [
+  HOST_BUTTON_BASE_CLASSNAME,
+  "ghsync__button-tone ghsync__button-tone--warning"
+].join(" ");
 var HOST_TOOLBAR_BUTTON_SIZE_CLASSNAME = "px-3 has-[>svg]:px-2.5";
 var HOST_ACTION_BUTTON_SIZE_CLASSNAME = "h-9 px-4 py-2 has-[>svg]:px-3";
 var HOST_INLINE_BUTTON_SIZE_CLASSNAME = "h-8 px-3 has-[>svg]:px-2.5";
@@ -22845,7 +22853,7 @@ var PREVIEW_MARKDOWN_REHYPE_PLUGINS = [rehypeRaw, PREVIEW_MARKDOWN_SANITIZE_REHY
 function getPluginActionClassName(options) {
   const variant = options?.variant ?? "secondary";
   const size = options?.size ?? "default";
-  const variantClassName = variant === "primary" ? HOST_DEFAULT_BUTTON_CLASSNAME : variant === "danger" ? HOST_DESTRUCTIVE_BUTTON_CLASSNAME : HOST_OUTLINE_BUTTON_CLASSNAME;
+  const variantClassName = variant === "primary" ? HOST_DEFAULT_BUTTON_CLASSNAME : variant === "success" ? HOST_SUCCESS_BUTTON_CLASSNAME : variant === "warning" ? HOST_WARNING_BUTTON_CLASSNAME : variant === "danger" ? HOST_DESTRUCTIVE_BUTTON_CLASSNAME : HOST_OUTLINE_BUTTON_CLASSNAME;
   const sizeClassName = size === "sm" ? HOST_INLINE_BUTTON_SIZE_CLASSNAME : HOST_ACTION_BUTTON_SIZE_CLASSNAME;
   return ["ghsync__button", variantClassName, sizeClassName, options?.extraClassName].filter(Boolean).join(" ");
 }
@@ -22882,6 +22890,7 @@ function GitHubButtonLabel(props) {
   ] });
 }
 var PROJECT_PULL_REQUESTS_PAGE_ROUTE_PATH = "github-pull-requests";
+var QUICK_REQUEST_CHANGES_REVIEW_SUMMARY = "Requested changes.";
 var LIGHT_PALETTE = {
   text: "#18181b",
   title: "#09090b",
@@ -23356,6 +23365,32 @@ var SHARED_LOADING_STYLES = `
   min-width: 0;
 }
 
+.ghsync__button-tone {
+  border: 1px solid transparent;
+}
+
+.ghsync__button-tone--success {
+  border-color: var(--ghsync-success-border);
+  background: var(--ghsync-success-bg);
+  color: var(--ghsync-success-text);
+}
+
+.ghsync__button-tone--success:hover {
+  background: color-mix(in srgb, var(--ghsync-success-bg) 72%, var(--ghsync-success-border));
+  color: var(--ghsync-success-text);
+}
+
+.ghsync__button-tone--warning {
+  border-color: var(--ghsync-warning-border);
+  background: var(--ghsync-warning-bg);
+  color: var(--ghsync-warning-text);
+}
+
+.ghsync__button-tone--warning:hover {
+  background: color-mix(in srgb, var(--ghsync-warning-bg) 72%, var(--ghsync-warning-border));
+  color: var(--ghsync-warning-text);
+}
+
 .ghsync__loading-inline {
   display: inline-flex;
   align-items: center;
@@ -25063,6 +25098,36 @@ var PROJECT_PULL_REQUESTS_PAGE_STYLES = `
   opacity: 0.42;
 }
 
+.ghsync-prs-table__icon-button--success {
+  color: var(--ghsync-success-text);
+}
+
+.ghsync-prs-table__icon-button--warning {
+  color: var(--ghsync-warning-text);
+}
+
+.ghsync-prs-table__icon-button--danger {
+  color: var(--ghsync-danger-text);
+}
+
+.ghsync-prs-table__icon-button--success:hover {
+  color: var(--ghsync-success-text);
+  border-color: var(--ghsync-success-border);
+  background: color-mix(in srgb, var(--ghsync-success-bg) 68%, var(--ghsync-surfaceRaised));
+}
+
+.ghsync-prs-table__icon-button--warning:hover {
+  color: var(--ghsync-warning-text);
+  border-color: var(--ghsync-warning-border);
+  background: color-mix(in srgb, var(--ghsync-warning-bg) 68%, var(--ghsync-surfaceRaised));
+}
+
+.ghsync-prs-table__icon-button--danger:hover {
+  color: var(--ghsync-danger-text);
+  border-color: var(--ghsync-danger-border);
+  background: color-mix(in srgb, var(--ghsync-danger-bg) 68%, var(--ghsync-surfaceRaised));
+}
+
 .ghsync-prs-table__status--passed {
   color: var(--ghsync-success-text);
 }
@@ -27560,6 +27625,16 @@ function getPreviewPullRequestCheckToneClass(status) {
       return "ghsync-prs-table__status--pending";
   }
 }
+function getPreviewPullRequestInlineActionToneClass(tone) {
+  switch (tone) {
+    case "success":
+      return "ghsync-prs-table__icon-button--success";
+    case "warning":
+      return "ghsync-prs-table__icon-button--warning";
+    default:
+      return "ghsync-prs-table__icon-button--danger";
+  }
+}
 function getPreviewPullRequestUpToDateMeta(status) {
   switch (status) {
     case "up_to_date":
@@ -27714,8 +27789,13 @@ function resolvePreviewPullRequestReviewable(record) {
   const unresolvedCopilotThreads = typeof record.copilotUnresolvedReviewThreads === "number" ? record.copilotUnresolvedReviewThreads : record.unresolvedReviewThreads;
   return record.githubMergeable === true && record.checksStatus === "passed" && unresolvedCopilotThreads === 0;
 }
-function resolvePreviewPullRequestMergeable(record) {
-  return record.githubMergeable === true && record.checksStatus === "passed" && record.reviewApprovals > 0 && record.unresolvedReviewThreads === 0;
+function resolvePreviewPullRequestTargetsDefaultBranch(record, options) {
+  const baseBranch = record.baseBranch.trim();
+  const defaultBranchName = options?.defaultBranchName?.trim();
+  return Boolean(baseBranch && defaultBranchName && baseBranch === defaultBranchName);
+}
+function resolvePreviewPullRequestMergeable(record, options) {
+  return record.githubMergeable === true && record.checksStatus === "passed" && record.reviewApprovals > 0 && record.reviewChangesRequested === 0 && record.unresolvedReviewThreads === 0 && resolvePreviewPullRequestTargetsDefaultBranch(record, options);
 }
 function matchesPreviewPullRequestFilter(record, filter) {
   switch (filter) {
@@ -28770,6 +28850,26 @@ function formatSyncFailureRepository(repositoryUrl) {
   }
   return repositoryUrl.trim();
 }
+function getLinkedPullRequestsForIssueDetails(issueDetails) {
+  if (issueDetails.linkedPullRequests && issueDetails.linkedPullRequests.length > 0) {
+    return issueDetails.linkedPullRequests;
+  }
+  return issueDetails.linkedPullRequestNumbers.map((pullRequestNumber) => ({
+    number: pullRequestNumber,
+    repositoryUrl: issueDetails.repositoryUrl
+  }));
+}
+function formatIssueDetailLinkedPullRequestLabel(pullRequest, issueRepositoryUrl) {
+  const pullRequestRepository = parseRepositoryReference(pullRequest.repositoryUrl);
+  if (!pullRequestRepository) {
+    return `PR #${pullRequest.number}`;
+  }
+  const issueRepository = parseRepositoryReference(issueRepositoryUrl);
+  if (issueRepository && issueRepository.owner.toLowerCase() === pullRequestRepository.owner.toLowerCase() && issueRepository.repo.toLowerCase() === pullRequestRepository.repo.toLowerCase()) {
+    return `PR #${pullRequest.number}`;
+  }
+  return `${pullRequestRepository.owner}/${pullRequestRepository.repo}#${pullRequest.number}`;
+}
 function getSyncFailureLogEntries(syncState) {
   if (syncState.recentFailures?.length) {
     return syncState.recentFailures.filter((entry) => typeof entry.message === "string" && entry.message.trim());
@@ -29353,6 +29453,7 @@ function GitHubSyncProjectPullRequestsPage() {
   const issueModalPending = issueModalPullRequest ? pendingActionKey === `create-issue:${issueModalPullRequest.id}` : false;
   const commentModalPending = commentModalPullRequest ? pendingActionKey === `comment-modal:${commentModalPullRequest.id}` : false;
   const reviewModalApprovePending = reviewModalPullRequest ? pendingActionKey === `review:${reviewModalPullRequest.id}:approve` : false;
+  const reviewModalCommentPending = reviewModalPullRequest ? pendingActionKey === `review:${reviewModalPullRequest.id}:comment` : false;
   const reviewModalRequestChangesPending = reviewModalPullRequest ? pendingActionKey === `review:${reviewModalPullRequest.id}:request_changes` : false;
   const reviewModalPending = reviewModalPullRequest ? Boolean(pendingActionKey?.startsWith(`review:${reviewModalPullRequest.id}:`)) : false;
   const rerunCiModalPending = rerunCiModalPullRequest ? pendingActionKey === `rerun-ci:${rerunCiModalPullRequest.id}` : false;
@@ -29515,6 +29616,10 @@ function GitHubSyncProjectPullRequestsPage() {
     setCommentModalPullRequestId(pullRequestId);
     setCommentModalDraft("");
   }
+  function openReviewModal(pullRequestId) {
+    setReviewModalPullRequestId(pullRequestId);
+    setReviewModalDraft("");
+  }
   function openClosePullRequestModal(pullRequest) {
     setCloseModalPullRequest(pullRequest);
   }
@@ -29801,23 +29906,42 @@ function GitHubSyncProjectPullRequestsPage() {
       setPendingActionKey((current) => current === actionKey ? null : current);
     }
   }
-  async function handleReviewPullRequest(review) {
-    if (!reviewModalPullRequest || !hostContext.companyId || !projectId) {
+  function getSubmittedReviewToast(review, pullRequestNumber) {
+    switch (review) {
+      case "approved":
+        return {
+          title: `Approved #${pullRequestNumber}`,
+          body: "GitHub review submitted."
+        };
+      case "changes_requested":
+        return {
+          title: `Requested changes on #${pullRequestNumber}`,
+          body: "GitHub change request submitted."
+        };
+      default:
+        return {
+          title: `Commented on #${pullRequestNumber}`,
+          body: "GitHub review comment submitted."
+        };
+    }
+  }
+  async function submitPullRequestReview(pullRequest, review, body, options) {
+    if (!hostContext.companyId || !projectId) {
       return;
     }
-    const actionKey = `review:${reviewModalPullRequest.id}:${review}`;
+    const actionKey = `review:${pullRequest.id}:${review}`;
     setPendingActionKey(actionKey);
     try {
       const result = await reviewPullRequest({
         companyId: hostContext.companyId,
         projectId,
         repositoryUrl: pageData.repositoryUrl,
-        pullRequestNumber: reviewModalPullRequest.number,
+        pullRequestNumber: pullRequest.number,
         review,
-        body: reviewModalDraft.trim()
+        body
       });
       const updatedAt = (/* @__PURE__ */ new Date()).toISOString();
-      patchPullRequestRow(reviewModalPullRequest.id, (current) => {
+      patchPullRequestRow(pullRequest.id, (current) => {
         const nextReviewApprovals = current.reviewApprovals + (result.review === "approved" ? 1 : 0);
         const nextReviewChangesRequested = current.reviewChangesRequested + (result.review === "changes_requested" ? 1 : 0);
         const nextRecord = {
@@ -29829,16 +29953,21 @@ function GitHubSyncProjectPullRequestsPage() {
         return {
           ...nextRecord,
           reviewable: resolvePreviewPullRequestReviewable(nextRecord),
-          mergeable: resolvePreviewPullRequestMergeable(nextRecord)
+          mergeable: resolvePreviewPullRequestMergeable(nextRecord, {
+            defaultBranchName: pageData.defaultBranchName
+          })
         };
       });
-      closeReviewModal();
-      refreshSelectedPullRequestDetails(reviewModalPullRequest.id);
+      if (options?.closeModal) {
+        closeReviewModal();
+      }
+      refreshSelectedPullRequestDetails(pullRequest.id);
       refreshPullRequestMetricsPanel();
       notifyGitHubSyncPullRequestsChanged();
+      const submittedReviewToast = getSubmittedReviewToast(result.review, pullRequest.number);
       toast({
-        title: result.review === "approved" ? `Approved #${reviewModalPullRequest.number}` : `Requested changes on #${reviewModalPullRequest.number}`,
-        body: result.review === "approved" ? "GitHub review submitted." : "GitHub change request submitted.",
+        title: submittedReviewToast.title,
+        body: submittedReviewToast.body,
         tone: "success"
       });
     } catch (error) {
@@ -29851,6 +29980,19 @@ function GitHubSyncProjectPullRequestsPage() {
       setPendingActionKey((current) => current === actionKey ? null : current);
     }
   }
+  async function handleReviewPullRequest(review) {
+    if (!reviewModalPullRequest) {
+      return;
+    }
+    await submitPullRequestReview(reviewModalPullRequest, review, reviewModalDraft.trim(), { closeModal: true });
+  }
+  async function handleQuickPullRequestReview(pullRequest, review) {
+    await submitPullRequestReview(
+      pullRequest,
+      review,
+      review === "request_changes" ? QUICK_REQUEST_CHANGES_REVIEW_SUMMARY : ""
+    );
+  }
   async function handleRerunCi() {
     if (!rerunCiModalPullRequest || !hostContext.companyId || !projectId) {
       return;
@@ -29874,7 +30016,9 @@ function GitHubSyncProjectPullRequestsPage() {
         return {
           ...nextRecord,
           reviewable: resolvePreviewPullRequestReviewable(nextRecord),
-          mergeable: resolvePreviewPullRequestMergeable(nextRecord)
+          mergeable: resolvePreviewPullRequestMergeable(nextRecord, {
+            defaultBranchName: pageData.defaultBranchName
+          })
         };
       });
       closeRerunCiModal();
@@ -29908,7 +30052,7 @@ function GitHubSyncProjectPullRequestsPage() {
     /* @__PURE__ */ jsx2("th", { scope: "col", className: "ghsync-prs-table__cell--center", children: "Checks" }),
     /* @__PURE__ */ jsx2("th", { scope: "col", className: "ghsync-prs-table__cell--center", children: "Up to date" }),
     /* @__PURE__ */ jsx2("th", { scope: "col", className: "ghsync-prs-table__cell--center", children: "Target branch" }),
-    /* @__PURE__ */ jsx2("th", { scope: "col", className: "ghsync-prs-table__cell--center", children: "Reviews" }),
+    /* @__PURE__ */ jsx2("th", { scope: "col", className: "ghsync-prs-table__cell--center", children: "Approvals" }),
     /* @__PURE__ */ jsx2("th", { scope: "col", className: "ghsync-prs-table__cell--center", children: "Review threads" }),
     /* @__PURE__ */ jsx2("th", { scope: "col", className: "ghsync-prs-table__cell--center", children: "Comments" }),
     /* @__PURE__ */ jsx2("th", { scope: "col", className: "ghsync-prs-table__cell--center", children: "Last updated" }),
@@ -30130,12 +30274,16 @@ function GitHubSyncProjectPullRequestsPage() {
               const mergeActionKey = `merge:${pullRequest.id}`;
               const commentModalActionKey = `comment-modal:${pullRequest.id}`;
               const reviewActionPrefix = `review:${pullRequest.id}:`;
+              const reviewApproveActionKey = `review:${pullRequest.id}:approve`;
+              const reviewRequestChangesActionKey = `review:${pullRequest.id}:request_changes`;
               const rerunCiActionKey = `rerun-ci:${pullRequest.id}`;
               const updateBranchActionKey = `update-branch:${pullRequest.id}`;
               const closeActionKey = `close:${pullRequest.id}`;
               const copilotPending = Boolean(pendingActionKey?.startsWith(copilotActionPrefix));
               const commentModalPending2 = pendingActionKey === commentModalActionKey;
               const reviewPending = Boolean(pendingActionKey?.startsWith(reviewActionPrefix));
+              const reviewApprovePending = pendingActionKey === reviewApproveActionKey;
+              const reviewRequestChangesPending = pendingActionKey === reviewRequestChangesActionKey;
               const rerunCiPending = pendingActionKey === rerunCiActionKey;
               const updateBranchPending = pendingActionKey === updateBranchActionKey;
               const mergePending = pendingActionKey === mergeActionKey;
@@ -30216,7 +30364,7 @@ function GitHubSyncProjectPullRequestsPage() {
                         "button",
                         {
                           type: "button",
-                          className: "ghsync-prs-table__icon-button",
+                          className: `ghsync-prs-table__icon-button ${getPreviewPullRequestInlineActionToneClass("warning")}`,
                           title: `Re-run CI for #${pullRequest.number}`,
                           onClick: () => setRerunCiPullRequestId(pullRequest.id),
                           disabled: rerunCiPending,
@@ -30272,40 +30420,82 @@ function GitHubSyncProjectPullRequestsPage() {
                     /* @__PURE__ */ jsx2("td", { className: "ghsync-prs-table__cell--center", children: /* @__PURE__ */ jsxs2("div", { className: "ghsync-prs-table__metric-group", children: [
                       hasReviewSummary ? /* @__PURE__ */ jsxs2("a", { className: "ghsync-prs-table__metric-link", href: pullRequest.reviewsUrl, target: "_blank", rel: "noreferrer", children: [
                         pullRequest.reviewApprovals > 0 ? /* @__PURE__ */ jsxs2(Fragment2, { children: [
-                          /* @__PURE__ */ jsx2(CheckPassedIcon, { className: "ghsync-prs-icon" }),
+                          /* @__PURE__ */ jsx2(CheckPassedIcon, { className: "ghsync-prs-icon ghsync-prs-table__status--passed" }),
                           /* @__PURE__ */ jsx2("span", { children: pullRequest.reviewApprovals })
                         ] }) : null,
                         pullRequest.reviewChangesRequested > 0 ? /* @__PURE__ */ jsxs2(Fragment2, { children: [
-                          /* @__PURE__ */ jsx2(CheckFailedIcon, { className: "ghsync-prs-icon" }),
+                          /* @__PURE__ */ jsx2(CheckFailedIcon, { className: "ghsync-prs-icon ghsync-prs-table__status--failed" }),
                           /* @__PURE__ */ jsx2("span", { children: pullRequest.reviewChangesRequested })
                         ] }) : null
                       ] }) : null,
-                      pullRequest.reviewable && canReviewPullRequests ? /* @__PURE__ */ jsx2(
-                        "button",
-                        {
-                          type: "button",
-                          className: "ghsync-prs-table__icon-button",
-                          title: `Review #${pullRequest.number}`,
-                          onClick: () => {
-                            setReviewModalPullRequestId(pullRequest.id);
-                            setReviewModalDraft("");
-                          },
-                          disabled: reviewPending,
-                          children: /* @__PURE__ */ jsx2(
-                            LoadingIconButtonContent,
-                            {
-                              busy: reviewPending,
-                              busyLabel: `Reviewing #${pullRequest.number}`,
-                              icon: /* @__PURE__ */ jsx2(ReviewIcon, { className: "ghsync-prs-icon" })
-                            }
-                          )
-                        }
-                      ) : null
+                      pullRequest.reviewable && canReviewPullRequests ? /* @__PURE__ */ jsxs2(Fragment2, { children: [
+                        /* @__PURE__ */ jsx2(
+                          "button",
+                          {
+                            type: "button",
+                            className: "ghsync-prs-table__icon-button",
+                            title: `Review #${pullRequest.number}`,
+                            "aria-label": `Review #${pullRequest.number}`,
+                            onClick: () => openReviewModal(pullRequest.id),
+                            disabled: reviewPending,
+                            children: /* @__PURE__ */ jsx2(
+                              LoadingIconButtonContent,
+                              {
+                                busy: false,
+                                busyLabel: `Reviewing #${pullRequest.number}`,
+                                icon: /* @__PURE__ */ jsx2(ReviewIcon, { className: "ghsync-prs-icon" })
+                              }
+                            )
+                          }
+                        ),
+                        /* @__PURE__ */ jsx2(
+                          "button",
+                          {
+                            type: "button",
+                            className: `ghsync-prs-table__icon-button ${getPreviewPullRequestInlineActionToneClass("success")}`,
+                            title: `Approve #${pullRequest.number}`,
+                            "aria-label": `Approve #${pullRequest.number}`,
+                            onClick: () => {
+                              void handleQuickPullRequestReview(pullRequest, "approve");
+                            },
+                            disabled: reviewPending,
+                            children: /* @__PURE__ */ jsx2(
+                              LoadingIconButtonContent,
+                              {
+                                busy: reviewApprovePending,
+                                busyLabel: `Approving #${pullRequest.number}`,
+                                icon: /* @__PURE__ */ jsx2(CheckPassedIcon, { className: "ghsync-prs-icon" })
+                              }
+                            )
+                          }
+                        ),
+                        /* @__PURE__ */ jsx2(
+                          "button",
+                          {
+                            type: "button",
+                            className: `ghsync-prs-table__icon-button ${getPreviewPullRequestInlineActionToneClass("danger")}`,
+                            title: `Request changes on #${pullRequest.number}`,
+                            "aria-label": `Request changes on #${pullRequest.number}`,
+                            onClick: () => {
+                              void handleQuickPullRequestReview(pullRequest, "request_changes");
+                            },
+                            disabled: reviewPending,
+                            children: /* @__PURE__ */ jsx2(
+                              LoadingIconButtonContent,
+                              {
+                                busy: reviewRequestChangesPending,
+                                busyLabel: `Requesting changes on #${pullRequest.number}`,
+                                icon: /* @__PURE__ */ jsx2(CheckFailedIcon, { className: "ghsync-prs-icon" })
+                              }
+                            )
+                          }
+                        )
+                      ] }) : null
                     ] }) }),
                     /* @__PURE__ */ jsx2("td", { className: "ghsync-prs-table__cell--center", children: pullRequest.unresolvedReviewThreads > 0 ? /* @__PURE__ */ jsxs2("a", { className: "ghsync-prs-table__metric-link", href: pullRequest.reviewThreadsUrl, target: "_blank", rel: "noreferrer", children: [
                       /* @__PURE__ */ jsx2(CommentIcon, { className: "ghsync-prs-icon" }),
                       /* @__PURE__ */ jsx2("span", { children: pullRequest.unresolvedReviewThreads })
-                    ] }) : hasResolvedReviewThreads ? /* @__PURE__ */ jsx2("a", { className: "ghsync-prs-table__metric-link", href: pullRequest.reviewThreadsUrl, target: "_blank", rel: "noreferrer", children: /* @__PURE__ */ jsx2(CheckPassedIcon, { className: "ghsync-prs-icon" }) }) : null }),
+                    ] }) : hasResolvedReviewThreads ? /* @__PURE__ */ jsx2("a", { className: "ghsync-prs-table__metric-link", href: pullRequest.reviewThreadsUrl, target: "_blank", rel: "noreferrer", children: /* @__PURE__ */ jsx2(CheckPassedIcon, { className: "ghsync-prs-icon ghsync-prs-table__status--passed" }) }) : null }),
                     /* @__PURE__ */ jsx2("td", { className: "ghsync-prs-table__cell--center", children: /* @__PURE__ */ jsxs2("div", { className: "ghsync-prs-table__metric-group", children: [
                       /* @__PURE__ */ jsxs2("a", { className: "ghsync-prs-table__metric-link", href: pullRequest.commentsUrl, target: "_blank", rel: "noreferrer", children: [
                         /* @__PURE__ */ jsx2(CommentIcon, { className: "ghsync-prs-icon" }),
@@ -30369,7 +30559,7 @@ function GitHubSyncProjectPullRequestsPage() {
                         "button",
                         {
                           type: "button",
-                          className: "ghsync-prs-table__icon-button",
+                          className: `ghsync-prs-table__icon-button ${getPreviewPullRequestInlineActionToneClass("success")}`,
                           title: `Merge #${pullRequest.number}`,
                           onClick: () => {
                             void handleMergePullRequest(pullRequest);
@@ -30389,7 +30579,7 @@ function GitHubSyncProjectPullRequestsPage() {
                         "button",
                         {
                           type: "button",
-                          className: "ghsync-prs-table__icon-button",
+                          className: `ghsync-prs-table__icon-button ${getPreviewPullRequestInlineActionToneClass("danger")}`,
                           title: `Close PR #${pullRequest.number}`,
                           "aria-label": `Close PR #${pullRequest.number}`,
                           onClick: () => {
@@ -30513,10 +30703,7 @@ function GitHubSyncProjectPullRequestsPage() {
               {
                 type: "button",
                 className: getPluginActionClassName({ variant: "secondary", size: "sm" }),
-                onClick: () => {
-                  setReviewModalPullRequestId(selectedPullRequest.id);
-                  setReviewModalDraft("");
-                },
+                onClick: () => openReviewModal(selectedPullRequest.id),
                 disabled: selectedPullRequestReviewPending,
                 children: /* @__PURE__ */ jsx2(
                   LoadingButtonContent,
@@ -30533,7 +30720,7 @@ function GitHubSyncProjectPullRequestsPage() {
               "button",
               {
                 type: "button",
-                className: getPluginActionClassName({ variant: "secondary", size: "sm" }),
+                className: getPluginActionClassName({ variant: "warning", size: "sm" }),
                 onClick: () => setRerunCiPullRequestId(selectedPullRequest.id),
                 disabled: selectedPullRequestRerunCiPending,
                 children: /* @__PURE__ */ jsx2(
@@ -30571,7 +30758,7 @@ function GitHubSyncProjectPullRequestsPage() {
               "button",
               {
                 type: "button",
-                className: getPluginActionClassName({ variant: "primary", size: "sm" }),
+                className: getPluginActionClassName({ variant: "success", size: "sm" }),
                 onClick: () => {
                   void handleMergePullRequest(selectedPullRequest);
                 },
@@ -30758,7 +30945,7 @@ function GitHubSyncProjectPullRequestsPage() {
                 ] }),
                 /* @__PURE__ */ jsxs2("div", { className: "ghsync-prs-meta__row", children: [
                   /* @__PURE__ */ jsx2("span", { className: "ghsync-prs-meta__label", children: "Mergeability" }),
-                  /* @__PURE__ */ jsx2("div", { className: "ghsync-prs-meta__value", children: selectedPullRequest.mergeable ? "Mergeable now" : "Blocked until checks and review feedback settle" })
+                  /* @__PURE__ */ jsx2("div", { className: "ghsync-prs-meta__value", children: selectedPullRequest.mergeable ? "Mergeable now" : "Blocked by checks, review feedback, or target branch rules" })
                 ] })
               ] })
             ] }),
@@ -31200,7 +31387,7 @@ function GitHubSyncProjectPullRequestsPage() {
                       className: "ghsync-prs-modal__textarea",
                       value: reviewModalDraft,
                       onChange: (event) => setReviewModalDraft(event.currentTarget.value),
-                      placeholder: "Optional comment"
+                      placeholder: "Required for Comment and Request changes. Optional for Approve."
                     }
                   )
                 ] }),
@@ -31215,6 +31402,25 @@ function GitHubSyncProjectPullRequestsPage() {
                     }
                   ),
                   /* @__PURE__ */ jsxs2("div", { className: "ghsync-prs-modal__split-actions", children: [
+                    /* @__PURE__ */ jsx2(
+                      "button",
+                      {
+                        type: "button",
+                        className: getPluginActionClassName({ variant: "secondary" }),
+                        onClick: () => {
+                          void handleReviewPullRequest("comment");
+                        },
+                        disabled: !reviewModalDraft.trim() || reviewModalPending,
+                        children: /* @__PURE__ */ jsx2(
+                          LoadingButtonContent,
+                          {
+                            busy: reviewModalCommentPending,
+                            label: "Comment",
+                            busyLabel: "Commenting\u2026"
+                          }
+                        )
+                      }
+                    ),
                     /* @__PURE__ */ jsx2(
                       "button",
                       {
@@ -31223,7 +31429,7 @@ function GitHubSyncProjectPullRequestsPage() {
                         onClick: () => {
                           void handleReviewPullRequest("request_changes");
                         },
-                        disabled: reviewModalPending,
+                        disabled: !reviewModalDraft.trim() || reviewModalPending,
                         children: /* @__PURE__ */ jsx2(
                           LoadingButtonContent,
                           {
@@ -31238,7 +31444,7 @@ function GitHubSyncProjectPullRequestsPage() {
                       "button",
                       {
                         type: "button",
-                        className: getPluginActionClassName({ variant: "primary" }),
+                        className: getPluginActionClassName({ variant: "success" }),
                         onClick: () => {
                           void handleReviewPullRequest("approve");
                         },
@@ -31295,7 +31501,7 @@ function GitHubSyncProjectPullRequestsPage() {
                     "button",
                     {
                       type: "button",
-                      className: getPluginActionClassName({ variant: "primary" }),
+                      className: getPluginActionClassName({ variant: "warning" }),
                       onClick: () => {
                         void handleRerunCi();
                       },
@@ -32647,7 +32853,7 @@ function GitHubSyncSettingsPage() {
                     }
                   }
                 ),
-                /* @__PURE__ */ jsx2("p", { className: "ghsync__hint", children: "The assignee that resumes work when GitHub Sync sends an issue back to active execution, such as failing CI, unresolved review threads, or a trusted new GitHub comment." })
+                /* @__PURE__ */ jsx2("p", { className: "ghsync__hint", children: "The assignee that resumes work when GitHub Sync sends an issue back to active execution, such as failing CI, non-mergeable linked pull requests, unresolved review threads, or a trusted new GitHub comment." })
               ] }),
               /* @__PURE__ */ jsxs2("div", { className: "ghsync__field", children: [
                 /* @__PURE__ */ jsx2("label", { htmlFor: "advanced-reviewer-assignee", children: "Reviewer handoff" }),
@@ -33802,6 +34008,7 @@ function GitHubSyncIssueDetailTabContent(props) {
   if (detailTabState === "hidden") {
     return null;
   }
+  const linkedPullRequests = issueDetails ? getLinkedPullRequestsForIssueDetails(issueDetails) : [];
   return /* @__PURE__ */ jsxs2("section", { className: "ghsync-issue-detail", style: props.themeVars, children: [
     /* @__PURE__ */ jsx2("style", { children: EXTENSION_SURFACE_STYLES }),
     detailTabState === "loading" ? /* @__PURE__ */ jsx2("p", { className: "ghsync-extension-empty", children: "Loading GitHub sync details\u2026" }) : null,
@@ -33882,19 +34089,19 @@ function GitHubSyncIssueDetailTabContent(props) {
         ] }),
         /* @__PURE__ */ jsxs2("div", { className: "ghsync-extension-metric", children: [
           /* @__PURE__ */ jsx2("span", { children: "Linked PRs" }),
-          /* @__PURE__ */ jsx2("strong", { children: issueDetails.linkedPullRequestNumbers.length })
+          /* @__PURE__ */ jsx2("strong", { children: linkedPullRequests.length })
         ] }),
         /* @__PURE__ */ jsxs2("div", { className: "ghsync-extension-metric", children: [
           /* @__PURE__ */ jsx2("span", { children: "Last synced" }),
           /* @__PURE__ */ jsx2("strong", { children: issueDetails.syncedAt ? formatDate(issueDetails.syncedAt, "Unknown") : "Pending refresh" })
         ] })
       ] }),
-      issueDetails.linkedPullRequestNumbers.length > 0 ? /* @__PURE__ */ jsxs2("div", { className: "ghsync-issue-detail__section", children: [
+      linkedPullRequests.length > 0 ? /* @__PURE__ */ jsxs2("div", { className: "ghsync-issue-detail__section", children: [
         /* @__PURE__ */ jsx2("div", { className: "ghsync-issue-detail__section-heading", children: "Linked pull requests" }),
-        /* @__PURE__ */ jsx2("div", { className: "ghsync-extension-links", children: issueDetails.linkedPullRequestNumbers.map((pullRequestNumber) => /* @__PURE__ */ jsx2(
+        /* @__PURE__ */ jsx2("div", { className: "ghsync-extension-links", children: linkedPullRequests.map((pullRequest) => /* @__PURE__ */ jsx2(
           "a",
           {
-            href: `${issueDetails.repositoryUrl}/pull/${pullRequestNumber}`,
+            href: `${pullRequest.repositoryUrl}/pull/${pullRequest.number}`,
             target: "_blank",
             rel: "noreferrer",
             className: getPluginActionClassName({
@@ -33902,9 +34109,14 @@ function GitHubSyncIssueDetailTabContent(props) {
               size: "sm",
               extraClassName: "ghsync-extension-link"
             }),
-            children: /* @__PURE__ */ jsx2(GitHubButtonLabel, { label: `PR #${pullRequestNumber}` })
+            children: /* @__PURE__ */ jsx2(
+              GitHubButtonLabel,
+              {
+                label: formatIssueDetailLinkedPullRequestLabel(pullRequest, issueDetails.repositoryUrl)
+              }
+            )
           },
-          pullRequestNumber
+          `${pullRequest.repositoryUrl}:${pullRequest.number}`
         )) })
       ] }) : null,
       issueDetails.labels && issueDetails.labels.length > 0 ? /* @__PURE__ */ jsxs2("div", { className: "ghsync-issue-detail__section", children: [