Skip to content

Verify selected graphics at their delivered placements - #20

Merged
ajhcs merged 1 commit into
mainfrom
codex/cse-visual-placement
Nov 10, 2026
Merged

ajhcs merged 1 commit into
mainfrom
codex/cse-visual-placement

Conversation

@ajhcs

@ajhcs ajhcs commented Nov 10, 2026 •

Copy link
Copy Markdown
Owner

Fresh installed-session testing exposed two errors in the visual-guidance evaluator. A valid delivery placed the selected component SVG beside a separate documentary flow, while the evaluator required that component inside the standalone flow. The missing-font check could also count an unrelated CLI usage error as a resource failure.

Check each identified SVG occurrence across delivered HTML/SVG, retaining the entire native master in every occurrence and matching the renderer inventory. Bind missing-font evidence to the exact selected profile and independently replayed current resource failure. Generic usage errors, wrong/stale profiles and incomplete or changed graphics fail.

The profile-selection guidance, renderer, PDF checker, font/graphic assets and eleven skill entrypoints are unchanged. Version 0.2.0-alpha.8 distinguishes the corrected package and preserves existing installed version paths.

Validation:

  • 24 behavioral regression tests pass, including separate placement and changed/partial/split/unidentified geometry controls, plus unrelated/stale/wrong-profile failure controls.
  • Four retained fresh native-session deliveries pass the corrected checker without changing any artifact: indirect profile 46 checks, client override 47, missing font 14 and text-only negative 9.
  • All four targeted suites pass (50 tests; the opt-in browser test skipped in the plain interpreter); the renderer's existing real-browser qualification remains byte-applicable.
  • Independent review found no additional P1/P2 issue in this bounded correction.

Full Python 3.10/3.12 branch and pull-request CI passed on the exact final commit: 77 suites / 1,233 tests per version, with only the opt-in browser test skipped. Installation and a final fresh native-session check follow the merged source tree.

@ajhcs
ajhcs merged commit 2b0ba0a into main Nov 10, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant