030

Author: DavidQ

docs/dev/codex_commands.md

@@ -1,30 +1,29 @@
-# Codex Commands - PR_26126_029-preview-generator-v2-fullscreen-native-size
+# Codex Commands - PR_26126_030-preview-generator-v2-status-header-and-fullscreen-css-fix
 
 ```bash
-codex run "Create PR_26126_029-preview-generator-v2-fullscreen-native-size. Fix Preview Generator V2 fullscreen capture wording and behavior only. Preserve existing generation behavior. Full Screen capture must not require 1600x900; use the actual available fullscreen/browser viewport size. Update Capture mode label from \"Full Screen (1600x900 HTML Page)\" to \"Full Screen\". Remove hardcoded 1600x900 wording from UI, logs, errors, and tests. If fullscreen capture fails, report \"Full Screen capture failed\" with the underlying error, not \"failed at 1600x900\". Preserve Canvas Only behavior. Do not modify samples. Do not add schema. Produce review artifacts."
+codex run "Create PR_26126_030-preview-generator-v2-status-header-and-fullscreen-css-fix. Fix Preview Generator V2 UI/capture only. Preserve existing generation behavior. Status must not render accordion affordances; remove the \"+\" from the Status header. Keep Clear on the same line as Status, but render it as a normal button, not accordion UI. Fix Full Screen capture failure caused by unsupported CSS color(...) values by sanitizing/normalizing unsupported CSS color functions only for the cloned capture DOM before html2canvas runs. Do not change live page styling. Do not fall back silently. Full Screen should succeed when the only blocker is unsupported CSS color(...). Preserve Canvas Only behavior. Do not modify samples. Do not add schema. Produce review artifacts."
 ```
 
 ## Validation Commands
 
 ```bash
-rg -n "1600|900|1600x900|fullscreen1600|Full Screen \\(|failed at" tools/preview-generator-v2/index.html
-rg -n "Full Screen capture failed|Full Screen|Canvas Only|fullscreen|getAvailableFullScreenCaptureSize|extractFullPageSvg" tools/preview-generator-v2/index.html
+rg -n "statusAccordion|Status|accordion-v2__icon|preview-generator-v2__status-header-row|preview-generator-v2__status-content|replaceUnsupportedColorFunctions|sanitizeComputedColorStyles|sanitizeClonedToolDocument|onclone" tools/preview-generator-v2/index.html
+node -e "const fs=require('fs');const html=fs.readFileSync('tools/preview-generator-v2/index.html','utf8');const status=html.match(/<section id=\"statusAccordion\"[\\s\\S]*?<\\/section>/)?.[0]||'';if(!status)throw new Error('Missing status section');if(status.includes('accordion-v2__icon')||status.includes('accordion-v2__header'))throw new Error('Status still has accordion affordance markup');console.log('status section has no accordion affordance');"
 node -e "const fs=require('fs');const html=fs.readFileSync('tools/preview-generator-v2/index.html','utf8');const scripts=[...html.matchAll(/<script(?![^>]*\\bsrc=)[^>]*>([\\s\\S]*?)<\\/script>/gi)].map(m=>m[1]);scripts.forEach((script,i)=>{new Function(script);console.log('inline script '+(i+1)+' syntax ok');});"
 git diff --check
-git diff --name-only -- tools/preview-generator-v2/index.html docs/dev/codex_commands.md docs/dev/commit_comment.txt docs/dev/reports/codex_changed_files.txt docs/dev/reports/codex_review.diff
 git diff --name-only -- '*.json' tools/shared tools/schemas start_of_day
 npm run test:workspace-v2
 npm test
 ```
 
 ## Playwright
 
-No Playwright test was added. This PR changes Preview Generator V2 fullscreen capture wording and capture sizing only. `npm run test:workspace-v2` is attempted as the standard command if available.
+No Playwright test was added. This PR changes Preview Generator V2 status header markup and cloned-DOM fullscreen capture sanitization only. `npm run test:workspace-v2` is attempted as the standard command if available.
 
 ## Test Notes
 
 `npm run test:workspace-v2` is not defined in the current `package.json`. `npm test` was attempted and still fails in the existing shared extraction guard baseline, outside this PR scope.
 
 ## Manual Test
 
-Use Preview Generator V2 with Canvas Only to confirm existing canvas capture still works. Use Full Screen on a target with unsupported fullscreen capture CSS and confirm it logs `Full Screen capture failed: <underlying error>` without fallback or fixed-size wording.
+Open Preview Generator V2 and confirm Status has no accordion plus/minus affordance, with Clear on the same line. Run Full Screen capture against a page whose only blocker is CSS `color(...)` and confirm it no longer fails for that parser error. Recheck Canvas Only capture still works.

docs/dev/commit_comment.txt

@@ -1 +1 @@
-Use native viewport sizing for Preview Generator V2 fullscreen capture - PR_26126_029-preview-generator-v2-fullscreen-native-size
+Remove status accordion affordance and sanitize fullscreen capture CSS colors - PR_26126_030-preview-generator-v2-status-header-and-fullscreen-css-fix

docs/dev/reports/codex_changed_files.txt

@@ -1,19 +1,22 @@
-# PR_26126_029-preview-generator-v2-fullscreen-native-size
+# PR_26126_030-preview-generator-v2-status-header-and-fullscreen-css-fix
 
-PR_029 files:
+PR_030 files:
 - docs/dev/codex_commands.md
 - docs/dev/commit_comment.txt
 - docs/dev/reports/codex_changed_files.txt
 - docs/dev/reports/codex_review.diff
 - tools/preview-generator-v2/index.html
 
 Validation notes:
-- Preview Generator V2 no longer contains `1600`, `900`, `1600x900`, `fullscreen1600`, `Full Screen (` or `failed at` in the active tool file.
+- Status section no longer contains `accordion-v2__header` or `accordion-v2__icon` markup.
+- Clear remains in the Status header row as a normal button.
+- Full Screen capture calls `sanitizeClonedToolDocument` from both fullscreen `html2canvas` paths.
+- The cloned capture DOM normalizes unsupported CSS `color(...)` values in style elements, CSS rules, inline styles, and computed color-related properties.
 - Inline scripts in `tools/preview-generator-v2/index.html` parse successfully with `new Function`.
-- `git diff --check` passed for the PR_029 target files.
+- `git diff --check` passed for the PR_030 target files.
 - `npm run test:workspace-v2` is unavailable because the script is not defined in the current `package.json`.
 - `npm test` still fails in the existing shared extraction guard baseline outside this PR scope.
 
-Existing workspace changes excluded from this PR_029 ZIP:
+Existing workspace changes excluded from this PR_030 ZIP:
 - Staged PR_028 sample-local README changes.
 - Unstaged `samples/phase-12/*/assets/images/preview.svg` changes.