@nkzw/codiff-core 1.11.0 → 1.11.1

package/dist/SharedWalkthroughApp.d.ts

@@ -14,6 +14,7 @@ export type ReviewSurfaceProps = {
         onClosePullRequest?: () => Promise<void> | void;
         onGenerateWalkthrough: () => Promise<void> | void;
         onHome: () => void;
+        onMarkPullRequestReady?: () => Promise<void> | void;
         onMergePullRequest?: (options: PullRequestMergeOptions & {
             autoMerge: boolean;
         }) => Promise<void> | void;

package/dist/app/components/Panels.d.ts

@@ -53,11 +53,12 @@ export declare function CopyCommentsButton({ comments, files, reviewCommentsPref
     showWhitespace: boolean;
 }): import("react").JSX.Element;
 export declare const isPullRequestReviewActionDisabled: (reviewStatus: PullRequestReviewStatus | undefined, event: PullRequestReviewEvent) => boolean;
-export declare function PullRequestReviewButtons({ children, disabled, hasPendingComments, onClosePullRequest, onSubmitReview, reviewStatus, showCommentReview, }: {
+export declare function PullRequestReviewButtons({ children, disabled, hasPendingComments, onClosePullRequest, onMarkPullRequestReady, onSubmitReview, reviewStatus, showCommentReview, }: {
     children?: ReactNode;
     disabled: boolean;
     hasPendingComments: boolean;
     onClosePullRequest?: () => void;
+    onMarkPullRequestReady?: () => void;
     onSubmitReview: (event: PullRequestReviewEvent, body?: string) => Promise<void> | void;
     reviewStatus?: PullRequestReviewStatus;
     showCommentReview?: boolean;

package/dist/react.mjs

@@ -2633,14 +2633,16 @@ function PullRequestReviewAction({ disabled, event, hasPendingComments = false,
 		}) : null]
 	});
 }
-function PullRequestReviewButtons({ children, disabled, hasPendingComments, onClosePullRequest, onSubmitReview, reviewStatus, showCommentReview = false }) {
+function PullRequestReviewButtons({ children, disabled, hasPendingComments, onClosePullRequest, onMarkPullRequestReady, onSubmitReview, reviewStatus, showCommentReview = false }) {
 	const approveBlocked = isPullRequestReviewActionDisabled(reviewStatus, "APPROVE");
 	const commentBlocked = isPullRequestReviewActionDisabled(reviewStatus, "COMMENT");
 	const requestChangesBlocked = isPullRequestReviewActionDisabled(reviewStatus, "REQUEST_CHANGES");
 	const closeStatus = reviewStatus?.close;
 	const closeVisible = onClosePullRequest && closeStatus && closeStatus.disabled !== true;
+	const markReadyStatus = reviewStatus?.markReady;
+	const markReadyVisible = onMarkPullRequestReady && markReadyStatus && markReadyStatus.disabled !== true;
 	const commentVisible = showCommentReview && !commentBlocked;
-	const hasReviewActions = commentVisible || !approveBlocked || !requestChangesBlocked || closeVisible;
+	const hasReviewActions = commentVisible || !approveBlocked || !requestChangesBlocked || markReadyVisible || closeVisible;
 	if (!hasReviewActions && !children) return null;
 	return /* @__PURE__ */ jsxs("div", {
 		"aria-label": hasReviewActions ? "Pull request review actions" : "Pull request status",
@@ -2687,6 +2689,21 @@ function PullRequestReviewButtons({ children, disabled, hasPendingComments, onCl
 				onSubmitReview,
 				title: getPullRequestReviewActionTitle(reviewStatus, "REQUEST_CHANGES", "Request changes")
 			}) : null,
+			markReadyVisible ? /* @__PURE__ */ jsxs(Button, {
+				action: onMarkPullRequestReady,
+				"aria-label": "Mark merge request as ready",
+				className: "review-submit-button ready",
+				disabled,
+				pendingPlaceholder: "Marking ready…",
+				title: markReadyStatus.reason ?? "Mark merge request as ready",
+				type: "button",
+				children: [/* @__PURE__ */ jsx(CheckCircleIcon, {
+					"aria-hidden": true,
+					className: "review-submit-icon ready",
+					size: 15,
+					weight: "bold"
+				}), /* @__PURE__ */ jsx("span", { children: "Mark ready" })]
+			}) : null,
 			closeVisible ? /* @__PURE__ */ jsxs(Button, {
 				action: onClosePullRequest,
 				"aria-label": "Close merge request",
@@ -9654,6 +9671,7 @@ function ReviewSurface({ commenting, externalUrl, gitIdentity = null, initialMod
 	const [generalCommentSubmitting, setGeneralCommentSubmitting] = useState(false);
 	const [pullRequestReviewSubmitting, setPullRequestReviewSubmitting] = useState(null);
 	const [pullRequestCloseSubmitting, setPullRequestCloseSubmitting] = useState(false);
+	const [pullRequestReadySubmitting, setPullRequestReadySubmitting] = useState(false);
 	const [pullRequestMergeSubmitting, setPullRequestMergeSubmitting] = useState(false);
 	const [walkthroughRequestPending, setWalkthroughRequestPending] = useState(false);
 	const walkthroughRequestPendingRef = useRef(false);
@@ -9797,6 +9815,18 @@ function ReviewSurface({ commenting, externalUrl, gitIdentity = null, initialMod
 		pullRequestCloseSubmitting,
 		snapshot.repository.source
 	]);
+	const markPullRequestReady = useCallback(() => {
+		const source = snapshot.repository.source;
+		if (!interactive?.onMarkPullRequestReady || pullRequestReadySubmitting || source.type !== "pull-request" || source.reviewStatus?.markReady?.disabled === true || !source.reviewStatus?.markReady) return;
+		setPullRequestReadySubmitting(true);
+		return Promise.resolve(interactive.onMarkPullRequestReady()).catch((error) => {
+			window.alert(error instanceof Error ? error.message : String(error));
+		}).finally(() => setPullRequestReadySubmitting(false));
+	}, [
+		interactive,
+		pullRequestReadySubmitting,
+		snapshot.repository.source
+	]);
 	const mergePullRequest = useCallback((options) => {
 		if (!interactive?.onMergePullRequest || pullRequestMergeSubmitting) return;
 		setPullRequestMergeSubmitting(true);
@@ -10013,9 +10043,10 @@ function ReviewSurface({ commenting, externalUrl, gitIdentity = null, initialMod
 	const isTerminalMergeState = sourceMergeState ? isTerminalPullRequestMergeState(sourceMergeState) : false;
 	const sourceMergeStatusBadge = sourceMergeState && isTerminalMergeState ? /* @__PURE__ */ jsx(PullRequestMergeStatusBadge, { mergeState: sourceMergeState }) : null;
 	const sourceDescriptionActions = interactive && source.type === "pull-request" ? /* @__PURE__ */ jsx(PullRequestReviewButtons, {
-		disabled: pullRequestReviewSubmitting != null || pullRequestCloseSubmitting,
+		disabled: pullRequestReviewSubmitting != null || pullRequestCloseSubmitting || pullRequestReadySubmitting,
 		hasPendingComments: getPendingPullRequestReviewComments(localReviewComments, activeReviewCommentDraftState).length > 0,
 		onClosePullRequest: closePullRequest,
+		onMarkPullRequestReady: markPullRequestReady,
 		onSubmitReview: submitReview,
 		reviewStatus: source.reviewStatus,
 		showCommentReview: source.provider === "github" || source.host === "github.com",

package/dist/style.css

@@ -5453,7 +5453,6 @@ a.review-top-bar-repository:focus-visible, a.review-top-bar-source:focus-visible
 .review-mode-control button[aria-selected="true"] {
   background: var(--hover-wash);
   color: var(--sidebar-text);
-  box-shadow: 0 1px 2px #00000014;
 }
 
 .review-mode-label {
@@ -8194,7 +8193,7 @@ diffs-container .review-comment-thread {
   position: relative;
 }
 
-.review-submit-button:not(.close) {
+.review-submit-button:is(.approve, .comment, .request-changes) {
   cursor: default;
   gap: 0;
   padding: 0;

package/dist/types.d.ts

@@ -52,6 +52,7 @@ export type PullRequestReviewStatus = {
     approve?: PullRequestReviewActionStatus;
     close?: PullRequestReviewActionStatus;
     comment?: PullRequestReviewActionStatus;
+    markReady?: PullRequestReviewActionStatus;
     requestChanges?: PullRequestReviewActionStatus;
 };
 export type PullRequestMergeCheckStatus = 'failed' | 'neutral' | 'pending' | 'success';
@@ -133,6 +134,7 @@ export type ReviewSource = {
     canEditReviewers?: boolean;
     canEditTitle?: boolean;
     description?: string;
+    draft?: boolean;
     headSha?: string;
     host?: string;
     mergeState?: PullRequestMergeState;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nkzw/codiff-core",
-  "version": "1.11.0",
+  "version": "1.11.1",
   "description": "A fast diff viewer.",
   "license": "MIT",
   "author": {