paperclip-github-plugin 0.5.0 → 0.5.1

package/README.md

@@ -65,7 +65,7 @@ The plugin does more than mirror issue text. It looks at linked pull requests, C
 
 ### Project pull request command center
 
-Each mapped project can expose a **Pull Requests** entry in the sidebar that opens a live GitHub queue page for that repository. The sidebar badge uses a lightweight total-count read, while the queue keeps the default view fast by loading only the current 10-row page, uses a repo-wide metrics read for the summary cards, reuses that cached metrics scan to keep filtered views fast by fetching only the visible filtered rows, keeps repo-scoped count, metrics, and per-PR review/check insight caches warm for repeat visits, lets operators explicitly bust those caches with Refresh when they want a live reread, shows total, mergeable, reviewable, and failing cards that filter the table, includes an **Up to date** column that distinguishes current branches, clean update candidates, conflict cases, and unknown freshness when GitHub cannot confirm the comparison, shows the PR target branch with a highlighted default-branch badge, keeps the list sorted by most recently updated first, paginates larger repositories, keeps a compact bottom detail pane with markdown-and-HTML-rendered conversation plus an inline comment composer, supports deterministic **Update branch** actions for clean behind-base pull requests, adds Copilot quick actions that post `@copilot` requests for **Fix CI**, **Rebase**, and **Address review feedback**, requests Copilot through GitHub’s native reviewer flow for **Review**, keeps the existing comment, review, re-run CI, merge, and close actions, hides any pull request action whose required GitHub permission is not verified for the saved token, and opens linked Paperclip issues in a plugin-provided right drawer so operators can stay on the queue page.
+Each mapped project can expose a **Pull Requests** entry in the sidebar that opens a live GitHub queue page for that repository. The sidebar badge uses a lightweight total-count read, while the queue keeps the default view fast by loading only the current 10-row page, uses a repo-wide metrics read for the summary cards, reuses that cached metrics scan to keep filtered views fast by fetching only the visible filtered rows, keeps repo-scoped count, metrics, and per-PR review/check insight caches warm for repeat visits, lets operators explicitly bust those caches with Refresh when they want a live reread, shows total, mergeable, reviewable, and failing cards that filter the table, includes an **Up to date** column that distinguishes current branches, clean update candidates, conflict cases, and unknown freshness when GitHub cannot confirm the comparison, shows the PR target branch with a highlighted default-branch badge, keeps the list sorted by most recently updated first, paginates larger repositories, keeps a compact bottom detail pane with markdown-and-HTML-rendered conversation plus an inline comment composer, supports deterministic **Update branch** actions for clean behind-base pull requests, adds Copilot quick actions that post `@copilot` requests for **Fix CI**, **Rebase**, and **Address review feedback**, requests Copilot through GitHub’s native reviewer flow for **Review**, keeps comment, review, quick approve/request-changes, re-run CI, merge, and close actions available, lets the review modal submit comment-only, approve, or request-changes reviews, hides any pull request action whose required GitHub permission is not verified for the saved token, and opens linked Paperclip issues in a plugin-provided right drawer so operators can stay on the queue page.
 
 Paperclip issue linkage on the queue prefers the GitHub issue that the pull request closes, so imported GitHub issues and delivery work stay connected in the same project view. If a pull request has no closing-issue-backed link yet, the queue falls back to the Paperclip issue created directly from that pull request and updates the table immediately when that create action returns.
 

package/dist/manifest.js

@@ -511,7 +511,7 @@ var require2 = createRequire(import.meta.url);
 var packageJson = require2("../package.json");
 var DASHBOARD_WIDGET_CAPABILITY = "ui.dashboardWidget.register";
 var SCHEDULE_TICK_CRON = "* * * * *";
-var MANIFEST_VERSION = "0.5.0"?.trim() || typeof packageJson.version === "string" && packageJson.version.trim() || process.env.npm_package_version?.trim() || "0.0.0-dev";
+var MANIFEST_VERSION = "0.5.1"?.trim() || typeof packageJson.version === "string" && packageJson.version.trim() || process.env.npm_package_version?.trim() || "0.0.0-dev";
 var manifest = {
   id: "paperclip-github-plugin",
   apiVersion: 1,

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":
@@ -29353,6 +29428,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 +29591,10 @@ function GitHubSyncProjectPullRequestsPage() {
     setCommentModalPullRequestId(pullRequestId);
     setCommentModalDraft("");
   }
+  function openReviewModal(pullRequestId) {
+    setReviewModalPullRequestId(pullRequestId);
+    setReviewModalDraft("");
+  }
   function openClosePullRequestModal(pullRequest) {
     setCloseModalPullRequest(pullRequest);
   }
@@ -29801,23 +29881,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 = {
@@ -29832,13 +29931,16 @@ function GitHubSyncProjectPullRequestsPage() {
           mergeable: resolvePreviewPullRequestMergeable(nextRecord)
         };
       });
-      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 +29953,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;
@@ -29908,7 +30023,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 +30245,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 +30335,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 +30391,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 +30530,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 +30550,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 +30674,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 +30691,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 +30729,7 @@ function GitHubSyncProjectPullRequestsPage() {
               "button",
               {
                 type: "button",
-                className: getPluginActionClassName({ variant: "primary", size: "sm" }),
+                className: getPluginActionClassName({ variant: "success", size: "sm" }),
                 onClick: () => {
                   void handleMergePullRequest(selectedPullRequest);
                 },
@@ -31200,7 +31358,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 +31373,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 +31400,7 @@ function GitHubSyncProjectPullRequestsPage() {
                         onClick: () => {
                           void handleReviewPullRequest("request_changes");
                         },
-                        disabled: reviewModalPending,
+                        disabled: !reviewModalDraft.trim() || reviewModalPending,
                         children: /* @__PURE__ */ jsx2(
                           LoadingButtonContent,
                           {
@@ -31238,7 +31415,7 @@ function GitHubSyncProjectPullRequestsPage() {
                       "button",
                       {
                         type: "button",
-                        className: getPluginActionClassName({ variant: "primary" }),
+                        className: getPluginActionClassName({ variant: "success" }),
                         onClick: () => {
                           void handleReviewPullRequest("approve");
                         },
@@ -31295,7 +31472,7 @@ function GitHubSyncProjectPullRequestsPage() {
                     "button",
                     {
                       type: "button",
-                      className: getPluginActionClassName({ variant: "primary" }),
+                      className: getPluginActionClassName({ variant: "warning" }),
                       onClick: () => {
                         void handleRerunCi();
                       },