@designfever/web-review-kit 0.8.2 → 0.8.4

package/README.md

@@ -27,10 +27,10 @@ This package does not own internal operator tools, private admin keys, or produc
 - [Testing](docs/testing.md): Vitest adapter contract tests and local verification commands.
 - [Custom adapter sample](docs/adaptor.sample.ts): starting point for host-owned remote adapters.
 - [DB setup](docs/db-setup.md): optional Supabase `review_items` setup, RLS, presence notes, and validation.
-- [Architecture and runtime logic](docs/architecture.md): core runtime, React shell, coordinate, anchor, and extension boundaries.
-- [Figma overlay](docs/figma-overlay.md): how the shell toggles a host Figma overlay.
+- [Architecture and runtime logic](docs/architecture.md): core runtime, React shell, coordinate, anchor, sitemap, and feature ownership boundaries.
+- [Figma overlay](docs/figma-overlay.md): host helper behavior and package-managed image overlay state.
 - [Grid overlay](docs/grid-overlay.md): how the shell toggles a host grid/helper overlay.
-- [Release notes 0.8.2](docs/release-notes-0.8.2.md): Figma image layer edit/delete tooltip removal.
+- [Release notes 0.8.4](docs/release-notes-0.8.4.md): reliable viewport capture for zero-size gradient elements.
 
 ## Quick Start
 

package/dist/{chunk-ZWJNUOYV.js → chunk-FWN7RQCW.js}

@@ -931,6 +931,15 @@ function isMeaningfulClass(value) {
   ) && !normalized.startsWith("mq-");
 }
 
+// src/core/error.ts
+function getErrorMessage(error, fallback) {
+  if (error instanceof Error) return error.message;
+  if (error && typeof error === "object" && "message" in error && typeof error.message === "string") {
+    return error.message;
+  }
+  return fallback;
+}
+
 // src/core/id.ts
 function createId() {
   if ("randomUUID" in crypto) return crypto.randomUUID();
@@ -966,6 +975,13 @@ var HOTKEY_KEY_ALIASES = {
   n: ["\u315C"],
   m: ["\u3161"]
 };
+function isEditableEventTarget(event) {
+  const path = event.composedPath?.() ?? [];
+  const element = path[0] ?? event.target;
+  if (!element || typeof element.tagName !== "string") return false;
+  const tag = element.tagName;
+  return tag === "INPUT" || tag === "TEXTAREA" || tag === "SELECT" || element.isContentEditable === true;
+}
 function isHotkey(event, hotkey) {
   const parts = hotkey.split("+").map((part) => part.trim().toLowerCase()).filter(Boolean);
   const key = parts.find(
@@ -4443,13 +4459,6 @@ var WebReviewKitView = class {
 // src/core/web.review.kit.app.ts
 var ROOT_ID = "df-web-review-kit-root";
 var VIEWPORT_SCROLL_OPTIONS = { capture: true, passive: true };
-function isEditableEventTarget(event) {
-  const path = event.composedPath?.() ?? [];
-  const element = path[0] ?? event.target;
-  if (!element || typeof element.tagName !== "string") return false;
-  const tag = element.tagName;
-  return tag === "INPUT" || tag === "TEXTAREA" || tag === "SELECT" || element.isContentEditable === true;
-}
 function createWebReviewKit(options) {
   if (typeof window === "undefined" || typeof document === "undefined") {
     return createNoopController();
@@ -5004,46 +5013,28 @@ var WebReviewKitApp = class {
       this.root.style.display = previousDisplay;
     }
   }
-  async captureDomDraft(input) {
-    if (this.isCapturingViewport) return;
-    const environment = this.getEnvironment();
-    const draft = this.domDraft;
-    if (!draft) return;
-    if (!environment?.captureViewport) {
-      this.draftError = "Viewport capture helper is not available.";
-      this.render();
-      return;
-    }
-    const captureInput = this.createViewportCaptureInput(
-      environment,
+  captureDomDraft(input) {
+    return this.captureDraft(
       input,
-      input.selection?.viewport
+      () => this.domDraft,
+      (draft) => {
+        this.domDraft = draft;
+      }
+    );
+  }
+  captureAreaDraft(input) {
+    return this.captureDraft(
+      input,
+      () => this.areaDraft,
+      (draft) => {
+        this.areaDraft = draft;
+      }
     );
-    this.draftError = "";
-    this.isCapturingViewport = true;
-    this.render();
-    try {
-      const result = await environment.captureViewport(captureInput);
-      const attachment = this.createCaptureDraftAttachment(result, captureInput);
-      const currentDraft = this.domDraft ?? draft;
-      this.domDraft = {
-        ...currentDraft,
-        attachments: [...currentDraft.attachments ?? [], attachment]
-      };
-    } catch (error) {
-      this.draftError = this.getErrorMessage(
-        error,
-        "Failed to capture viewport."
-      );
-    } finally {
-      this.isCapturingViewport = false;
-      this.render();
-    }
   }
-  async captureAreaDraft(input) {
+  async captureDraft(input, getDraft, setDraft) {
     if (this.isCapturingViewport) return;
     const environment = this.getEnvironment();
-    const draft = this.areaDraft;
+    const draft = getDraft();
     if (!draft) return;
     if (!environment?.captureViewport) {
       this.draftError = "Viewport capture helper is not available.";
@@ -5061,16 +5052,13 @@ var WebReviewKitApp = class {
     try {
       const result = await environment.captureViewport(captureInput);
       const attachment = this.createCaptureDraftAttachment(result, captureInput);
-      const currentDraft = this.areaDraft ?? draft;
-      this.areaDraft = {
+      const currentDraft = getDraft() ?? draft;
+      setDraft({
         ...currentDraft,
         attachments: [...currentDraft.attachments ?? [], attachment]
-      };
+      });
     } catch (error) {
-      this.draftError = this.getErrorMessage(
-        error,
-        "Failed to capture viewport."
-      );
+      this.draftError = getErrorMessage(error, "Failed to capture viewport.");
     } finally {
       this.isCapturingViewport = false;
       this.render();
@@ -5209,17 +5197,10 @@ var WebReviewKitApp = class {
     );
   }
   getCreateItemErrorMessage(error, wasUploadingAttachments) {
-    const message = this.getErrorMessage(error, "Failed to save QA.");
+    const message = getErrorMessage(error, "Failed to save QA.");
     const reason = error && typeof error === "object" && "reason" in error && typeof error.reason === "string" ? ` (${error.reason})` : "";
     return wasUploadingAttachments && reason ? `Attachment upload failed${reason}: ${message}` : message;
   }
-  getErrorMessage(error, fallback) {
-    if (error instanceof Error) return error.message;
-    if (error && typeof error === "object" && "message" in error && typeof error.message === "string") {
-      return error.message;
-    }
-    return fallback;
-  }
   async restoreItem(item) {
     this.setModeState("idle");
     this.clearDrafts();
@@ -5283,6 +5264,8 @@ export {
   isPointInViewport,
   toHostPoint,
   clamp,
+  getErrorMessage,
+  isEditableEventTarget,
   isHotkey,
   getHotkeyActionKey,
   DEFAULT_REVIEW_VIEWPORTS,
@@ -5299,4 +5282,4 @@ export {
   reviewTypographyTokens,
   createWebReviewKit
 };
-//# sourceMappingURL=chunk-ZWJNUOYV.js.map
+//# sourceMappingURL=chunk-FWN7RQCW.js.map