Skip to content

fix(php-transformer): ignore inert live-region scaffolding - #1552

Closed
chubes4 wants to merge 2 commits into
trunkfrom
fix/inert-next-route-announcers
Closed

fix(php-transformer): ignore inert live-region scaffolding#1552
chubes4 wants to merge 2 commits into
trunkfrom
fix/inert-next-route-announcers

Conversation

@chubes4

@chubes4 chubes4 commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Closes #1548

Summary

  • classify empty custom-element live-region scaffolding exclusively from observable DOM semantics
  • require an attribute-free custom wrapper with one empty, bounded, visually clipped ARIA live region
  • omit qualifying runtime scaffolding instead of generating empty HTML fallbacks
  • preserve wrappers with captured text, semantic names, unexpected attributes, runtime metadata, or unrelated custom-element structure
  • keep the source platform wrapper only as regression input proving wrapper-name independence

Verification

  • php tests/unit/inert-capture-scaffolding.php (21 assertions)
  • composer test (292 parity fixtures plus package-install proof)
  • production-shaped 102-page import: removed 204 empty live-region fallbacks while retaining zero content-loss findings
  • production source contains no platform or wrapper-name condition
  • git diff --check origin/trunk...HEAD

AI assistance

OpenAI GPT-5.6 Sol via OpenCode inspected repeated fallback evidence, replaced the initial platform-specific condition with a bounded semantic classifier, added wrapper-independent and fail-closed tests, and ran the focused, full, package, and import verification under Chris Huber's direction.

@chubes4 chubes4 closed this Sep 3, 2026
@chubes4
chubes4 deleted the fix/inert-next-route-announcers branch September 3, 2026 18:43
@chubes4 chubes4 changed the title fix(php-transformer): ignore inert Next.js route announcers fix(php-transformer): ignore inert live-region scaffolding Sep 3, 2026
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.

Ignore inert custom-element live-region scaffolding

1 participant