Skip to content

fix(regex): protect internal scalar property boundaries - #1050

Closed
fglock wants to merge 1 commit into
wip/phase36-surrogate-property-rendererfrom
wip/phase36-native-property-marker
Closed

fix(regex): protect internal scalar property boundaries#1050
fglock wants to merge 1 commit into
wip/phase36-surrogate-property-rendererfrom
wip/phase36-native-property-marker

Conversation

@fglock

@fglock fglock commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • prevent generated Unicode property classes from starting inside visible internal scalar marker payloads
  • split generated ranges crossing U+D800..U+DFFF so true surrogate membership remains explicit and atomic
  • add an unchanged standard-Perl-first 10-assertion oracle
  • update the Phase 36 master plan and completed-phase tracker

Validation

  • standard Perl oracle: 10/10
  • PerlOnJava JVM/interpreter oracle: 10/10 on each backend
  • protected HST/quick-check gate: 113/113 on each backend
  • protected surrogate renderer gate: 27/27 on each backend
  • warning-free make: passed in 4m02s
  • direct Joni and packaging verification: passed
  • git diff --check: passed

Stack

This draft is stacked on PR #1049 (wip/phase36-surrogate-property-renderer).
It is intentionally separate from PR #1049's authoritative generated chunks
01-04 map gate.

Generated with Codex

Prevent generated Unicode property classes from inspecting visible internal
scalar marker payloads, while preserving true surrogate membership by splitting
crossing ranges into explicit segments.

Update the Phase 36 plan with the focused and full-build evidence.

Generated with Codex (https://openai.com/codex)

Co-Authored-By: Codex <codex@openai.com>
@fglock

fglock commented Aug 21, 2026

Copy link
Copy Markdown
Owner Author

Superseded by the cumulative Phase 36 integrations merged through PR #1091 and the active successor PR #1089; closing this obsolete stacked review without deleting its branch.

@fglock fglock closed this Aug 21, 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.

1 participant