Skip to content

refactor(regex): route conditionals through Joni - #1060

Draft
fglock wants to merge 2 commits into
fix/phase36-joni-composed-fold-policyfrom
integration/phase36-conditional-latest
Draft

refactor(regex): route conditionals through Joni#1060
fglock wants to merge 2 commits into
fix/phase36-joni-composed-fold-policyfrom
integration/phase36-conditional-latest

Conversation

@fglock

@fglock fglock commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • route real Perl conditional constructs through native Joni parsing
  • preserve the temporary branch-reset/subroutine Java guard from the newer parent
  • delete 393 lines of unreachable conditional preprocessing

Stack

This is a two-commit child of fix/phase36-joni-composed-fold-policy (PR #1059).
The deletion commit is exactly range-diff equivalent to POJ3's validated
candidate. The routing commit has one intentional integration difference: it
retains the parent branch-reset hold while adding syntax-aware conditional
routing.

Validation

  • make: BUILD SUCCESSFUL in 3m15s, all 17 tasks executed
  • warning/deprecation/failure scan: zero lines
  • conditional fixture: 22/22 on JVM and interpreter in automatic mode
  • conditional fixture: 22/22 on JVM and interpreter in forced-Joni mode
  • git diff --check: clean

This remains draft until its parent stack lands and CI is green on the rebased
master-based branch.

fglock and others added 2 commits August 18, 2026 20:51
Detect real Perl conditional constructs with the existing syntax-aware
admission scanner so the temporary Java policy delegates their semantics and
diagnostics to Joni. Add policy, direct-engine, and standard-Perl regression
coverage for valid, backtracking, and malformed forms.

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

Co-Authored-By: OpenAI Codex <codex@openai.com>
Ordinary and malformed Perl conditionals are now routed to native Joni
before the Java regex preprocessor runs. Remove the lossy conditional
pre-pass and Java Pattern emulation that can no longer be reached.

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

Co-Authored-By: OpenAI Codex <codex@openai.com>
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