Preserve host alignment and native state; add reproducible problem gallery - #167
Draft
CodeinScrubs wants to merge 4 commits into
Draft
CodeinScrubs wants to merge 4 commits into
CodeinScrubs wants to merge 4 commits into
Conversation
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This is a stacked follow-up to #162, based on
fix/audit-regressions-20260926. It adds concrete source-preserving repairs and an evidence gallery, not a claim that every language/device/renderer is globally production-certified.Reproducible visual evidence at
967201dNine-pair gallery · Engineering packet · Readiness and remaining gaps
The three new 390 px mobile cases are curated logical reconstructions inspired by five October 2 captures, not recovered ChatGPT source, edited screenshots or a modified ChatGPT client. Originals remain private. The
Retrieval:case explicitly declares Persian paragraph intent because its default character majority is LTR. English formulas retain independent LTR bases. Host physical-left alignment stays unchanged.Fresh local verification of the mobile delta
fast-urialert chore(deps): bump gradle/actions/wrapper-validation from 0b6dd653ba04f4f93bf581ec31e66cbd7dcb644d to ed408507eac070d1f99cc633dbcf757c94c7933a #18; this branch uses patched 3.1.8. The alert was not dismissed.Follow-up boundary hardening at
c82907dAdded direct first/last-line geometry assertions for the terminal period/Persian question mark, without relaxing assertions or timeouts. Chromium and WebKit passed; one Firefox context-teardown run reported a compositor failure and is retained as a failure. The unchanged three-test Firefox rerun passed.
The new supplementary-plane negatives reproduced attached-query false positives in .NET regex and JVM lookbehind. Both now check the full neighboring scalar/code point for only the two new ASCII query forms. Prefix/suffix cases include
𝒜and𐒠in all five core suites. The Kotlin regression failed before the fix and all 40 Android core tests passed after it. The 19 JavaScript/Markdown tests passed again. Current-head native/compiler/security gate results are reported below.Current native integration boundary: a whole-control Android TextView or UIKit direction is not an independent-base Markdown renderer. Keep differing prose/formula/code blocks separate, and preserve authored physical alignment and original source.
The final source delta passed 20/20 CI jobs in run 36986883745, including the supplementary-plane negatives on all five core implementations and the punctuation geometry checks on all three browser engines. CodeQL run 36986880207 also completed successfully for all five languages: C#, JavaScript/TypeScript, Java/Kotlin, Swift and Rust. Both are on exact head
c82907d725cf9c86cb28b8914c08649e00744d15. Use those conclusions for the current head; old green results are not substituted for new ones. Current browser tests and readiness snapshot include the added boundary assertions and remaining limits.Hosted gates for
967201dCI run 36982395852 passed 20/20 jobs and CodeQL run 36982391670 passed all five languages on
967201d. A documentation-only follow-up atbff2a52also passed all 20 CI jobs in run 36983360624. These results do not validate the newerc82907ddelta. CodeQL is explicitly dispatched because its PR trigger is restricted to amainbase while this draft is stacked.Earlier verification of
b999435CI run 36936425858 passed 20/20 jobs, including Android emulator, Apple package/iOS simulator, .NET8/WPF/sample, Rust on three OSes, browser/packed-consumer/quality, SBOM and performance jobs. CodeQL run 36936426301 passed five languages. These results do not validate the newer mobile changes.
Earlier local .NET 10 source-linked core/WPF validation passed 5,853 assertions and 941 corpus cases, but is not a substitute for the fresh .NET 8 job. Earlier Firefox repeat-run compositor failures were retained, not counted as passes; the new complete one-worker browser run above passed.
Release/rollout boundaries
This is unreleased
0.4.0source, not a new registry artifact or a ChatGPT/Codex integration. No merge or registry publication was performed. Installing published npm0.4.0or Maven0.1.2does not install these source-branch repairs.Apple independent paragraph restoration, native extended-grapheme parity, hostile tiny-chunk streams, native-speaker review, physical device/IME/accessibility tests and downstream pilots remain acceptance work. Native HTML/ICU/BidiFormatter and other tools can be preferable at their respective layers. Neither universal superiority nor zero defects is claimed.