@gmickel/gno 1.30.7 → 1.32.0

package/src/serve/public/pages/DocView.tsx

@@ -13,6 +13,7 @@ import {
   LinkIcon,
   Loader2Icon,
   PencilIcon,
+  QuoteIcon,
   Share2Icon,
   SquareArrowOutUpRightIcon,
   TextIcon,
@@ -30,6 +31,10 @@ import {
 
 import type { PdfFallbackReason } from "../lib/pdf";
 
+import {
+  FILE_REFACTOR_APPLY_CONFIRMATION,
+  type FileRefactorPreviewPlan,
+} from "../../../core/file-refactor-contract";
 import { extractSections } from "../../../core/sections";
 import {
   CodeBlock,
@@ -46,6 +51,7 @@ import {
   OutgoingLinksPanel,
   type OutgoingLink,
 } from "../components/OutgoingLinksPanel";
+import { RefactorImpactPreview } from "../components/RefactorImpactPreview";
 import { RelatedNotesSidebar } from "../components/RelatedNotesSidebar";
 import { TagInput } from "../components/TagInput";
 import { Badge } from "../components/ui/badge";
@@ -83,6 +89,15 @@ import {
   downloadPublishArtifactFile,
   type PublishExportResponse,
 } from "../lib/publish-export";
+import {
+  buildReadableSectionUrl,
+  createCitationSectionUrl,
+  readSectionTargetLinkParam,
+  resolveSectionLinkNavigation,
+  SECTION_LINK_NOTICE_COPY,
+  stripSectionTargetLinkParam,
+  type SectionLinkNoticeKind,
+} from "../lib/section-links";
 import { subscribeWorkspaceActionRequest } from "../lib/workspace-events";
 
 /** Lazy so pdfjs is never pulled for non-PDF documents. */
@@ -189,6 +204,9 @@ interface RenameDocResponse {
   uri: string;
   path: string;
   relPath: string;
+  planDigest?: string;
+  status?: string;
+  warning?: string;
   refactorWarnings?: {
     warnings: string[];
   };
@@ -199,6 +217,9 @@ interface MoveDocResponse {
   uri: string;
   path: string;
   relPath: string;
+  planDigest?: string;
+  status?: string;
+  warning?: string;
   refactorWarnings?: {
     warnings: string[];
   };
@@ -349,13 +370,23 @@ export default function DocView({ navigate }: PageProps) {
   const [renaming, setRenaming] = useState(false);
   const [renameError, setRenameError] = useState<string | null>(null);
   const [renameValue, setRenameValue] = useState("");
-  const [renameWarnings, setRenameWarnings] = useState<string[]>([]);
+  const [renamePlan, setRenamePlan] = useState<FileRefactorPreviewPlan | null>(
+    null
+  );
+  const [renamePlanLoading, setRenamePlanLoading] = useState(false);
+  const [renameConfirmed, setRenameConfirmed] = useState(false);
+  const [renameOutcome, setRenameOutcome] = useState<string | null>(null);
   const [moveDialogOpen, setMoveDialogOpen] = useState(false);
   const [moving, setMoving] = useState(false);
   const [moveError, setMoveError] = useState<string | null>(null);
   const [moveFolderPath, setMoveFolderPath] = useState("");
   const [moveName, setMoveName] = useState("");
-  const [moveWarnings, setMoveWarnings] = useState<string[]>([]);
+  const [movePlan, setMovePlan] = useState<FileRefactorPreviewPlan | null>(
+    null
+  );
+  const [movePlanLoading, setMovePlanLoading] = useState(false);
+  const [moveConfirmed, setMoveConfirmed] = useState(false);
+  const [moveOutcome, setMoveOutcome] = useState<string | null>(null);
   const [duplicateDialogOpen, setDuplicateDialogOpen] = useState(false);
   const [duplicating, setDuplicating] = useState(false);
   const [duplicateError, setDuplicateError] = useState<string | null>(null);
@@ -389,9 +420,13 @@ export default function DocView({ navigate }: PageProps) {
   const [activeSectionAnchor, setActiveSectionAnchor] = useState<string | null>(
     null
   );
+  const [sectionLinkNotice, setSectionLinkNotice] =
+    useState<SectionLinkNoticeKind | null>(null);
+  const [blockHashNavigation, setBlockHashNavigation] = useState(false);
 
   // Request sequencing - ignore stale responses on rapid navigation
   const requestIdRef = useRef(0);
+  const sectionResolveRequestRef = useRef(0);
   const latestDocEvent = useDocEvents();
 
   // App remounts page on route/query changes, so URI is stable per render.
@@ -404,6 +439,10 @@ export default function DocView({ navigate }: PageProps) {
     () => window.location.hash.replace(/^#/u, ""),
     []
   );
+  const encodedSectionTarget = useMemo(
+    () => readSectionTargetLinkParam(window.location.search),
+    []
+  );
   const highlightedLines = useMemo(() => {
     if (!currentTarget.lineStart) return [];
     const end = currentTarget.lineEnd ?? currentTarget.lineStart;
@@ -564,7 +603,78 @@ export default function DocView({ navigate }: PageProps) {
   }, []);
 
   useEffect(() => {
-    if (!currentHash || showRawView || loading) {
+    if (!sectionLinkNotice) {
+      return;
+    }
+    const timer = window.setTimeout(() => setSectionLinkNotice(null), 3200);
+    return () => {
+      window.clearTimeout(timer);
+    };
+  }, [sectionLinkNotice]);
+
+  useEffect(() => {
+    if (!doc?.content || loading || !encodedSectionTarget) {
+      return;
+    }
+
+    const requestId = ++sectionResolveRequestRef.current;
+    const content = doc.content;
+    void resolveSectionLinkNavigation({
+      content,
+      uri: doc.uri,
+      encodedTarget: encodedSectionTarget,
+      hashAnchor: currentHash,
+    }).then((result) => {
+      if (requestId !== sectionResolveRequestRef.current) {
+        return;
+      }
+      setBlockHashNavigation(result.blockHashNavigation);
+      if (result.notice) {
+        setSectionLinkNotice(result.notice);
+      }
+      if (result.cleanCitationParam) {
+        const cleanedSearch = stripSectionTargetLinkParam(
+          window.location.search
+        );
+        const nextHash = result.navigateAnchor
+          ? `#${result.navigateAnchor}`
+          : window.location.hash;
+        window.history.replaceState(
+          {},
+          "",
+          `${window.location.pathname}${cleanedSearch}${nextHash}`
+        );
+      }
+      if (result.blockHashNavigation || !result.navigateAnchor) {
+        return;
+      }
+      if (showRawView) {
+        return;
+      }
+      requestAnimationFrame(() => {
+        document
+          .getElementById(result.navigateAnchor ?? "")
+          ?.scrollIntoView({ behavior: "smooth", block: "start" });
+        setActiveSectionAnchor(result.navigateAnchor);
+      });
+    });
+  }, [
+    currentHash,
+    doc?.content,
+    doc?.uri,
+    encodedSectionTarget,
+    loading,
+    showRawView,
+  ]);
+
+  useEffect(() => {
+    if (
+      blockHashNavigation ||
+      encodedSectionTarget ||
+      !currentHash ||
+      showRawView ||
+      loading
+    ) {
       return;
     }
 
@@ -574,7 +684,13 @@ export default function DocView({ navigate }: PageProps) {
         ?.scrollIntoView({ behavior: "smooth", block: "start" });
       setActiveSectionAnchor(currentHash);
     });
-  }, [currentHash, loading, showRawView]);
+  }, [
+    blockHashNavigation,
+    currentHash,
+    encodedSectionTarget,
+    loading,
+    showRawView,
+  ]);
 
   const breadcrumbs = doc ? parseBreadcrumbs(doc.collection, doc.relPath) : [];
   const sections = useMemo(
@@ -582,6 +698,79 @@ export default function DocView({ navigate }: PageProps) {
     [parsedContent.body]
   );
 
+  const copyReadableSectionLink = useCallback(
+    (anchor: string) => {
+      if (!doc) {
+        return;
+      }
+      void navigator.clipboard
+        .writeText(
+          buildReadableSectionUrl(window.location.origin, {
+            uri: doc.uri,
+            view: "rendered",
+            anchor,
+          })
+        )
+        .then(() => {
+          setSectionLinkNotice("copied_link");
+        })
+        .catch(() => {
+          setSectionLinkNotice("clipboard_unavailable");
+        });
+    },
+    [doc]
+  );
+
+  const copyCitationSectionLink = useCallback(
+    async (anchor: string) => {
+      const content = doc?.content;
+      if (!doc || !content) {
+        return;
+      }
+      const citationUrl = await createCitationSectionUrl({
+        origin: window.location.origin,
+        uri: doc.uri,
+        content,
+        anchor,
+        view: "rendered",
+      });
+      if (!citationUrl) {
+        setSectionLinkNotice("citation_unavailable");
+        return;
+      }
+      try {
+        await navigator.clipboard.writeText(citationUrl);
+        setSectionLinkNotice("copied_citation");
+      } catch {
+        setSectionLinkNotice("clipboard_unavailable");
+      }
+    },
+    [doc]
+  );
+
+  const jumpToSection = useCallback(
+    (anchor: string) => {
+      setShowRawView(false);
+      setBlockHashNavigation(false);
+      requestAnimationFrame(() => {
+        document.getElementById(anchor)?.scrollIntoView({
+          behavior: "smooth",
+          block: "start",
+        });
+        window.history.replaceState(
+          {},
+          "",
+          `${buildDocDeepLink({
+            uri: doc?.uri ?? "",
+            view: "rendered",
+          })}#${anchor}`
+        );
+        setActiveSectionAnchor(anchor);
+      });
+    },
+    [doc?.uri]
+  );
+
   useEffect(() => {
     if (showRawView || sections.length === 0) {
       setActiveSectionAnchor(sections[0]?.anchor ?? null);
@@ -707,7 +896,9 @@ export default function DocView({ navigate }: PageProps) {
     const filename = doc.relPath.split("/").pop() ?? doc.relPath;
     setRenameValue(filename);
     setRenameError(null);
-    setRenameWarnings([]);
+    setRenamePlan(null);
+    setRenameConfirmed(false);
+    setRenameOutcome(null);
     setRenameDialogOpen(true);
   }, [doc]);
 
@@ -715,7 +906,11 @@ export default function DocView({ navigate }: PageProps) {
     if (!renameDialogOpen || !doc || !renameValue.trim()) {
       return;
     }
-    void apiFetch<{ refactorWarnings?: { warnings: string[] } }>(
+    let cancelled = false;
+    setRenamePlanLoading(true);
+    setRenamePlan(null);
+    setRenameConfirmed(false);
+    void apiFetch<FileRefactorPreviewPlan>(
       `/api/docs/${encodeURIComponent(doc.docid)}/refactor-plan`,
       {
         method: "POST",
@@ -725,22 +920,42 @@ export default function DocView({ navigate }: PageProps) {
           uri: doc.uri,
         }),
       }
-    ).then(({ data }) => {
-      setRenameWarnings(data?.refactorWarnings?.warnings ?? []);
+    ).then(({ data, error: err }) => {
+      if (cancelled) {
+        return;
+      }
+      setRenamePlanLoading(false);
+      if (err) {
+        setRenameError(err);
+        setRenamePlan(null);
+        return;
+      }
+      setRenameError(null);
+      setRenamePlan(data);
     });
+    return () => {
+      cancelled = true;
+    };
   }, [doc, renameDialogOpen, renameValue]);
 
   const handleRename = useCallback(async () => {
-    if (!doc) {
+    if (!doc || !renamePlan?.canApply || !renameConfirmed) {
       return;
     }
     setRenaming(true);
     setRenameError(null);
+    setRenameOutcome(null);
     const { data, error: err } = await apiFetch<RenameDocResponse>(
       `/api/docs/${encodeURIComponent(doc.docid)}/rename`,
       {
         method: "POST",
-        body: JSON.stringify({ name: renameValue, uri: doc.uri }),
+        body: JSON.stringify({
+          name: renameValue,
+          uri: doc.uri,
+          planDigest: renamePlan.planDigest,
+          confirmation: FILE_REFACTOR_APPLY_CONFIRMATION,
+          schemaVersion: renamePlan.schemaVersion,
+        }),
       }
     );
     setRenaming(false);
@@ -750,11 +965,19 @@ export default function DocView({ navigate }: PageProps) {
       return;
     }
 
+    if (data?.status === "applied_with_sync_pending" || data?.warning) {
+      setRenameOutcome(
+        data.warning ??
+          "Files renamed, but index sync is still pending. Run Update All to finish."
+      );
+      return;
+    }
+
     setRenameDialogOpen(false);
     if (data?.uri) {
       navigate(`/doc?uri=${encodeURIComponent(data.uri)}`);
     }
-  }, [doc, navigate, renameValue]);
+  }, [doc, navigate, renameConfirmed, renamePlan, renameValue]);
 
   const handleStartMove = useCallback(() => {
     if (!doc) {
@@ -763,7 +986,9 @@ export default function DocView({ navigate }: PageProps) {
     setMoveFolderPath(getParentPath(doc.relPath));
     setMoveName(doc.relPath.split("/").pop() ?? doc.relPath);
     setMoveError(null);
-    setMoveWarnings([]);
+    setMovePlan(null);
+    setMoveConfirmed(false);
+    setMoveOutcome(null);
     setMoveDialogOpen(true);
   }, [doc]);
 
@@ -771,7 +996,11 @@ export default function DocView({ navigate }: PageProps) {
     if (!moveDialogOpen || !doc || !moveFolderPath.trim()) {
       return;
     }
-    void apiFetch<{ refactorWarnings?: { warnings: string[] } }>(
+    let cancelled = false;
+    setMovePlanLoading(true);
+    setMovePlan(null);
+    setMoveConfirmed(false);
+    void apiFetch<FileRefactorPreviewPlan>(
       `/api/docs/${encodeURIComponent(doc.docid)}/refactor-plan`,
       {
         method: "POST",
@@ -782,17 +1011,31 @@ export default function DocView({ navigate }: PageProps) {
           uri: doc.uri,
         }),
       }
-    ).then(({ data }) => {
-      setMoveWarnings(data?.refactorWarnings?.warnings ?? []);
+    ).then(({ data, error: err }) => {
+      if (cancelled) {
+        return;
+      }
+      setMovePlanLoading(false);
+      if (err) {
+        setMoveError(err);
+        setMovePlan(null);
+        return;
+      }
+      setMoveError(null);
+      setMovePlan(data);
     });
+    return () => {
+      cancelled = true;
+    };
   }, [doc, moveDialogOpen, moveFolderPath, moveName]);
 
   const handleMove = useCallback(async () => {
-    if (!doc) {
+    if (!doc || !movePlan?.canApply || !moveConfirmed) {
       return;
     }
     setMoving(true);
     setMoveError(null);
+    setMoveOutcome(null);
     const { data, error: err } = await apiFetch<MoveDocResponse>(
       `/api/docs/${encodeURIComponent(doc.docid)}/move`,
       {
@@ -801,6 +1044,9 @@ export default function DocView({ navigate }: PageProps) {
           folderPath: moveFolderPath,
           name: moveName,
           uri: doc.uri,
+          planDigest: movePlan.planDigest,
+          confirmation: FILE_REFACTOR_APPLY_CONFIRMATION,
+          schemaVersion: movePlan.schemaVersion,
         }),
       }
     );
@@ -811,11 +1057,19 @@ export default function DocView({ navigate }: PageProps) {
       return;
     }
 
+    if (data?.status === "applied_with_sync_pending" || data?.warning) {
+      setMoveOutcome(
+        data.warning ??
+          "Files moved, but index sync is still pending. Run Update All to finish."
+      );
+      return;
+    }
+
     setMoveDialogOpen(false);
     if (data?.uri) {
       navigate(`/doc?uri=${encodeURIComponent(data.uri)}`);
     }
-  }, [doc, moveFolderPath, moveName, navigate]);
+  }, [doc, moveConfirmed, moveFolderPath, moveName, movePlan, navigate]);
 
   const handleStartDuplicate = useCallback(() => {
     if (!doc) {
@@ -1170,8 +1424,19 @@ export default function DocView({ navigate }: PageProps) {
         <>
           <div className="mx-3 border-border/20 border-t" />
           <div className="px-3 py-3">
-            <div className="mb-2 font-mono text-[10px] text-muted-foreground/50 uppercase tracking-[0.15em]">
-              Outline
+            <div className="mb-2 flex items-center justify-between gap-2">
+              <div className="font-mono text-[10px] text-muted-foreground/50 uppercase tracking-[0.15em]">
+                Outline
+              </div>
+              {sectionLinkNotice && (
+                <div
+                  aria-live="polite"
+                  className="min-w-0 truncate font-mono text-[10px] text-muted-foreground/70"
+                  role="status"
+                >
+                  {SECTION_LINK_NOTICE_COPY[sectionLinkNotice]}
+                </div>
+              )}
             </div>
             <div className="w-full min-w-0 max-w-full space-y-0.5 overflow-x-hidden">
               {sections.map((section) => (
@@ -1185,25 +1450,9 @@ export default function DocView({ navigate }: PageProps) {
                   style={{ paddingLeft: `${section.level * 7}px` }}
                 >
                   <button
-                    className="flex w-full min-w-0 max-w-full cursor-pointer items-start gap-2 overflow-hidden rounded px-1 py-0.5 pr-7 text-left text-xs transition-colors hover:bg-muted/20 hover:text-foreground"
+                    className="flex w-full min-w-0 max-w-full cursor-pointer items-start gap-2 overflow-hidden rounded px-1 py-0.5 pr-12 text-left text-xs transition-colors hover:bg-muted/20 hover:text-foreground"
                     onClick={() => {
-                      setShowRawView(false);
-                      requestAnimationFrame(() => {
-                        document
-                          .getElementById(section.anchor)
-                          ?.scrollIntoView({
-                            behavior: "smooth",
-                            block: "start",
-                          });
-                        window.history.replaceState(
-                          {},
-                          "",
-                          `${buildDocDeepLink({
-                            uri: doc?.uri ?? "",
-                            view: "rendered",
-                          })}#${section.anchor}`
-                        );
-                      });
+                      jumpToSection(section.anchor);
                     }}
                     type="button"
                   >
@@ -1221,20 +1470,40 @@ export default function DocView({ navigate }: PageProps) {
                       </TooltipContent>
                     </Tooltip>
                   </button>
-                  <button
-                    className="absolute top-1 right-1 cursor-pointer rounded p-1 opacity-0 transition-all hover:bg-muted/20 hover:text-foreground focus-visible:opacity-100 group-hover:opacity-100"
-                    onClick={() => {
-                      void navigator.clipboard.writeText(
-                        `${window.location.origin}${buildDocDeepLink({
-                          uri: doc?.uri ?? "",
-                          view: "rendered",
-                        })}#${section.anchor}`
-                      );
-                    }}
-                    type="button"
-                  >
-                    <CopyIcon className="size-3" />
-                  </button>
+                  <div className="absolute top-1 right-1 flex items-center gap-0.5 opacity-0 transition-all focus-within:opacity-100 group-hover:opacity-100">
+                    <Tooltip>
+                      <TooltipTrigger asChild>
+                        <button
+                          aria-label={`Copy link to ${section.title}`}
+                          className="cursor-pointer rounded p-1 hover:bg-muted/20 hover:text-foreground"
+                          onClick={() => {
+                            copyReadableSectionLink(section.anchor);
+                          }}
+                          type="button"
+                        >
+                          <CopyIcon className="size-3" />
+                        </button>
+                      </TooltipTrigger>
+                      <TooltipContent side="left">Copy link</TooltipContent>
+                    </Tooltip>
+                    <Tooltip>
+                      <TooltipTrigger asChild>
+                        <button
+                          aria-label={`Copy local citation link to ${section.title}`}
+                          className="cursor-pointer rounded p-1 hover:bg-muted/20 hover:text-foreground"
+                          onClick={() => {
+                            void copyCitationSectionLink(section.anchor);
+                          }}
+                          type="button"
+                        >
+                          <QuoteIcon className="size-3" />
+                        </button>
+                      </TooltipTrigger>
+                      <TooltipContent side="left">
+                        Copy local citation link
+                      </TooltipContent>
+                    </Tooltip>
+                  </div>
                 </div>
               ))}
             </div>
@@ -2024,39 +2293,61 @@ export default function DocView({ navigate }: PageProps) {
       </Dialog>
 
       <Dialog onOpenChange={setRenameDialogOpen} open={renameDialogOpen}>
-        <DialogContent>
+        <DialogContent className="sm:max-w-lg">
           <DialogHeader>
             <DialogTitle>Rename document</DialogTitle>
             <DialogDescription>
-              Rename the file on disk inside its current folder. This does not
-              move it to another collection yet.
+              Rename the file on disk inside its current folder. Reference
+              rewrites follow the exact plan digest below.
             </DialogDescription>
           </DialogHeader>
           <input
+            aria-label="New file name"
             className="w-full rounded-md border border-input bg-transparent px-3 py-2 text-sm"
             onChange={(event) => setRenameValue(event.target.value)}
             value={renameValue}
           />
+          <RefactorImpactPreview
+            confirmed={renameConfirmed}
+            loading={renamePlanLoading}
+            onConfirmedChange={setRenameConfirmed}
+            outcomeMessage={renameOutcome}
+            outcomeTone="warning"
+            plan={renamePlan}
+          />
           {renameError && (
-            <div className="rounded-lg bg-destructive/10 p-3 text-destructive text-sm">
+            <div
+              className="rounded-lg bg-destructive/10 p-3 text-destructive text-sm"
+              role="alert"
+            >
               {renameError}
             </div>
           )}
-          {renameWarnings.length > 0 && (
-            <div className="rounded-lg border border-amber-500/30 bg-amber-500/10 p-3 text-amber-500 text-sm">
-              {renameWarnings.map((warning) => (
-                <div key={warning}>{warning}</div>
-              ))}
-            </div>
-          )}
           <DialogFooter className="gap-2 sm:gap-0">
             <Button
-              onClick={() => setRenameDialogOpen(false)}
+              onClick={() => {
+                if (renameOutcome && renamePlan) {
+                  setRenameDialogOpen(false);
+                  navigate(
+                    `/doc?uri=${encodeURIComponent(renamePlan.target.uri)}`
+                  );
+                  return;
+                }
+                setRenameDialogOpen(false);
+              }}
               variant="outline"
             >
-              Cancel
+              {renameOutcome ? "Open note" : "Cancel"}
             </Button>
-            <Button disabled={renaming} onClick={() => void handleRename()}>
+            <Button
+              disabled={
+                renaming ||
+                !renamePlan?.canApply ||
+                !renameConfirmed ||
+                Boolean(renameOutcome)
+              }
+              onClick={() => void handleRename()}
+            >
               {renaming && (
                 <Loader2Icon className="mr-1.5 size-4 animate-spin" />
               )}
@@ -2067,42 +2358,69 @@ export default function DocView({ navigate }: PageProps) {
       </Dialog>
 
       <Dialog onOpenChange={setMoveDialogOpen} open={moveDialogOpen}>
-        <DialogContent>
+        <DialogContent className="sm:max-w-lg">
           <DialogHeader>
             <DialogTitle>Move document</DialogTitle>
             <DialogDescription>
               Move the current note to another folder inside this collection.
+              Reference rewrites follow the exact plan digest below.
             </DialogDescription>
           </DialogHeader>
           <div className="space-y-3">
             <Input
+              aria-label="Destination folder path"
               onChange={(event) => setMoveFolderPath(event.target.value)}
               placeholder="projects/research"
               value={moveFolderPath}
             />
             <Input
+              aria-label="File name"
               onChange={(event) => setMoveName(event.target.value)}
               placeholder="note.md"
               value={moveName}
             />
+            <RefactorImpactPreview
+              confirmed={moveConfirmed}
+              loading={movePlanLoading}
+              onConfirmedChange={setMoveConfirmed}
+              outcomeMessage={moveOutcome}
+              outcomeTone="warning"
+              plan={movePlan}
+            />
             {moveError && (
-              <div className="rounded-lg bg-destructive/10 p-3 text-destructive text-sm">
+              <div
+                className="rounded-lg bg-destructive/10 p-3 text-destructive text-sm"
+                role="alert"
+              >
                 {moveError}
               </div>
             )}
-            {moveWarnings.length > 0 && (
-              <div className="rounded-lg border border-amber-500/30 bg-amber-500/10 p-3 text-amber-500 text-sm">
-                {moveWarnings.map((warning) => (
-                  <div key={warning}>{warning}</div>
-                ))}
-              </div>
-            )}
           </div>
           <DialogFooter className="gap-2 sm:gap-0">
-            <Button onClick={() => setMoveDialogOpen(false)} variant="outline">
-              Cancel
+            <Button
+              onClick={() => {
+                if (moveOutcome && movePlan) {
+                  setMoveDialogOpen(false);
+                  navigate(
+                    `/doc?uri=${encodeURIComponent(movePlan.target.uri)}`
+                  );
+                  return;
+                }
+                setMoveDialogOpen(false);
+              }}
+              variant="outline"
+            >
+              {moveOutcome ? "Open note" : "Cancel"}
             </Button>
-            <Button disabled={moving} onClick={() => void handleMove()}>
+            <Button
+              disabled={
+                moving ||
+                !movePlan?.canApply ||
+                !moveConfirmed ||
+                Boolean(moveOutcome)
+              }
+              onClick={() => void handleMove()}
+            >
               {moving && <Loader2Icon className="mr-1.5 size-4 animate-spin" />}
               Move
             </Button>