screengraft 0.41.0 → 0.41.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "screengraft",
-  "version": "0.41.0",
+  "version": "0.41.1",
   "description": "Put a UI screenshot or screen recording onto a photographed device screen with the perspective exactly right \u2014 a homography you confirm by hand, not a generative guess.",
   "keywords": [
     "mockup",

package/scripts/ui.py

@@ -702,6 +702,10 @@ def _adopt(role, path):
     return {"path": real, "size": [im.shape[1], im.shape[0]], **meta}
 
 
+class RenderBusy(Exception):
+    """A request that must not run while a render is in flight."""
+
+
 def _clear(role):
     """Un-choose a source. The mirror of _adopt(), and the only way to start
     over: picking a different file was the one exit, and a reload restores the
@@ -719,6 +723,13 @@ def _clear(role):
     """
     if role not in ROLES:
         raise ValueError(f"role must be one of {', '.join(ROLES)}")
+    # A render in flight would publish its output and sidecar AFTER this
+    # cleared them, from a source that is no longer loaded -- the exact stale
+    # artefact this exists to prevent. Refused, the way the render route
+    # refuses a second render; the page shows the message.
+    with RENDER_LOCK:
+        if RENDER["state"] == "running":
+            raise RenderBusy("a render is running — wait for it, then remove the source")
     if role == "photo":
         SESSION.update(photo=None, corners=None, output=None)
     else:
@@ -897,7 +908,10 @@ class Handler(BaseHTTPRequestHandler):
                 return self._json(_adopt(b["role"], b["path"]))
 
             if u.path == "/api/clear":
-                return self._json(_clear(b.get("role")))
+                try:
+                    return self._json(_clear(b.get("role")))
+                except RenderBusy as exc:
+                    return self._json({"error": str(exc)}, 409)
 
             if u.path == "/api/figma":
                 return self._json(SESSION.enqueue({

package/ui/index.html

@@ -1490,6 +1490,11 @@ async function clearSource(role){
     // Nothing to draw on: the canvas goes back to its empty state rather
     // than showing the last photograph with no way to act on it.
     const c = $('#c'); c.getContext('2d').clearRect(0, 0, c.width, c.height);
+    // ... and the edge view, which would otherwise keep showing a strip of a
+    // photograph that is no longer here.
+    for (const id of ['strip', 'stripF']) {
+      const k = $('#' + id); if (k) k.getContext('2d').clearRect(0, 0, k.width, k.height);
+    }
     $('#cvbar').hidden = true;
     $('#detSt').className = 'status';
     $('#detSt').textContent = 'Photo removed. A fit you saved for it comes back when you pick it again.';
@@ -1501,6 +1506,10 @@ async function clearSource(role){
       : 'Screenshot removed.';
   }
   markStale();
+  // The control that had focus has just hidden itself; without this, focus
+  // drops to <body> and a keyboard user is nowhere. The empty chip is the
+  // natural next action.
+  chip.focus();
 }
 document.querySelectorAll('.chipclear').forEach(b => { b.onclick = () => clearSource(b.dataset.role); });