From da126ad8896e96cc43ef245fe9883c70b962f538 Mon Sep 17 00:00:00 2001 From: CarterPerez-dev Date: Sun, 17 May 2026 15:07:50 -0400 Subject: [PATCH] fix(canary-phase-ui-design): single-source form fields + URL revoke + eventExtra schema MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Parallel audit (superpowers:code-reviewer + general-purpose spec adherence) returned PASS-WITH-NOTES + PASS. All actionable SHOULD-FIX and NIT items addressed in this commit per the fix-in-phase rule. SHOULD-FIX (code review): - Dead ternary at manage/index.tsx:253 (`cursor === undefined ? idx+1 : idx+1`) collapsed to `idx + 1`. EventLogSection prop renamed `cursor` → `cursorIsSet` (boolean) since the value itself was never consumed past the dead branch. - URL.createObjectURL leak in artifact.tsx — both file (base64) and text variants now run inside useEffect with cleanup that calls URL.revokeObjectURL on dependency-change AND on unmount. Extracted into two co-located hooks: useBase64ObjectUrl, useTextObjectUrl. - Text inputs lacked