@telefonica/mistica 16.66.0 → 16.67.0

package/doc/agents/figma-verifier.md

@@ -0,0 +1,49 @@
+# Figma verifier
+
+Audit a just-completed Figma implementation. Compare the running app against the Figma design visually and
+structurally, then report drift. Re-read [../figma-mcp.md](../figma-mcp.md) for the prime directive and skill
+rules before starting.
+
+## Tasks
+
+1. **Fetch the Figma screenshot** via `mcp__plugin_figma_figma__get_screenshot` for the entry node.
+   `get_metadata` gives the frame's viewport dimensions.
+
+2. **Render the implementation**:
+
+   - Start the dev server in the background, piping output to a log file.
+   - Wait for it to start.
+   - Screenshot via Playwright CLI:
+
+     ```bash
+     npx -y playwright@1.60.0 screenshot \
+       --viewport-size=<width>,<height> \
+       --full-page \
+       <url> \
+       /tmp/verifier-actual.png
+     ```
+
+     Use the Figma frame's dimensions. For responsive designs, repeat per breakpoint.
+
+3. **Compare visually**: open `/tmp/verifier-actual.png` and compare against the Figma screenshot. Catalog
+   drift in: layout, spacing, typography (weight/size/line-height — compare letter forms), colors, assets
+   (missing / wrong), and component primitive choice.
+
+4. **Structural / rule check**:
+
+   - Re-drill every composite component with `get_design_context({ disableCodeConnect: true })` and compare
+     props against the drilled-in DOM.
+   - Read each implementation file. Flag: hex literals or raw `var(--...)`, manual `font-size` /
+     `font-family`, composite props that don't match the non-CodeConnect DOM, spacing values absent from the
+     DOM, mismatched semantic structure (`Stack` vs `Inline`, flat vs nested), native HTML where a Mistica
+     primitive exists, wrong/missing skin/font/body-background for the file's brand.
+
+5. **Report**. One summary line (`clean` or `N issues found, M blocking`), then a bullet list ordered by
+   severity. Each issue:
+
+   - `[VISUAL]` or `[RULE]` prefix
+   - Figma node id (or screenshot region) + what the design specifies
+   - File:line (for `[RULE]`) or screenshot region (for `[VISUAL]`)
+   - What's wrong + proposed fix
+
+   Only flag drift and rule violations — no architectural rewrites or "improvements".

package/doc/figma-mcp.md

@@ -4,6 +4,23 @@ Mandatory reading whenever you are implementing UI from Figma through the Figma
 a `figma.com/...` URL. This file only covers the translation layer — the rest of the Mistica docs
 (`patterns.md`, `components.md`, `layout.md`, `design-tokens.md`) still apply.
 
+## Step 0: ask the user whether to run the verifier
+
+Before fetching any Figma node or writing any code, ask the user this question exactly once (use a
+structured-question tool if your harness has one):
+
+- **Question**:
+  `Run the verifier loop after implementing? It compares the rendered app against Figma both visually (Playwright screenshot) and structurally (DOM + rule audit), then iterates fixes.`
+- **Options**:
+  - **Yes, run the verifier** — recommended when fidelity matters. Catches drift the implementer can't see. It
+    will take more time and cost more tokens.
+  - **No, skip the verifier** — faster path. The agent still self-checks and reviews its work, but no
+    independent comparison runs.
+
+Remember the answer for the rest of this task. **One answer covers the whole implementation — do not re-ask
+per section.** If the user opted in, run the [verifier loop](#verifier-loop-run-before-declaring-done). If
+they opted out, do not spawn the verifier; the self-checks are the only gate.
+
 ## Prime directive: read the DOM verbatim
 
 The Figma MCP response gives you two things:
@@ -134,3 +151,21 @@ Before closing out a section always:
 
 If you can't check an item off against the DOM, re-read the DOM (with `disableCodeConnect: true` if the node
 is CodeConnect-mapped) before committing the value.
+
+## Verifier loop (run before declaring done)
+
+Skip if the user opted out in [Step 0](#step-0-ask-the-user-whether-to-run-the-verifier).
+
+Spawn a **new** subagent (via your harness's subagent mechanism — e.g. Claude's `Agent` tool with
+`subagent_type: general-purpose`, Codex's equivalent) pointed at
+[agents/figma-verifier.md](./agents/figma-verifier.md). Never reuse a subagent across iterations. Pass in the
+prompt:
+
+- Figma `fileKey` and entry `nodeId`
+- Implementation file paths you wrote or edited
+- Local assets directory
+- Dev-server command and URL (check `package.json`)
+
+If the verifier reports clean, done. Otherwise fix the cited issues — each fix justified by Figma DOM or
+screenshot — and re-run with a fresh subagent. Stop after clean, 3 iterations, or out-of-scope issues (missing
+tokens/assets, designer clarification — surface these instead of fudging).

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@telefonica/mistica",
-    "version": "16.66.0",
+    "version": "16.67.0",
     "license": "MIT",
     "repository": {
         "type": "git",

package/src/package-version.tsx

@@ -1,2 +1,2 @@
 // DO NOT EDIT THIS FILE. It's autogenerated by set-version.js
-export const PACKAGE_VERSION = '16.66.0' as string;
+export const PACKAGE_VERSION = '16.67.0' as string;

package/src/video.tsx

@@ -127,6 +127,8 @@ export interface VideoElement extends HTMLDivElement {
     /** Stops the video and shows the poster image (if available) */
     stop: () => void;
     setCurrentTime: (time: number) => void;
+    getCurrentTime: () => number;
+    getDuration: () => number;
     /**
      * Sets the display mode of a track by its index.
      * - 'showing': track is visible
@@ -348,6 +350,8 @@ const Video = React.forwardRef<VideoElement, VideoProps>(
                                     videoRef.current.currentTime = time;
                                 }
                             };
+                            containerElement.getCurrentTime = () => videoRef.current?.currentTime ?? 0;
+                            containerElement.getDuration = () => videoRef.current?.duration ?? NaN;
                             containerElement.stop = () => {
                                 if (videoRef.current) {
                                     videoRef.current.pause();