Skip to content

test(regexp): retire nested-alternation Prism skips - #308

Merged
pmatos merged 2 commits into
mainfrom
sym/jsse/271-regexp-nested-alternations-choose-the-wrong-greedy-match
Jul 20, 2026
Merged

test(regexp): retire nested-alternation Prism skips#308
pmatos merged 2 commits into
mainfrom
sym/jsse/271-regexp-nested-alternations-choose-the-wrong-greedy-match

Conversation

@pmatos

@pmatos pmatos commented Jul 17, 2026

Copy link
Copy Markdown
Owner

Summary

  • add spec-derived regression coverage for the reported parenthesis and Bison brace match-priority patterns
  • remove the three JSSE-only PrismJS fixture skips so JSSE and Node execute the same 2,563 fixtures
  • update the library-harness documentation to reflect full PrismJS coverage
  • preserve the current RegExp backend because the current main branch already matches the required ECMAScript ordering for both reproductions

Validation

  • uv run python scripts/run-custom-tests.py test262-extra/RegExp-nested-alternation-priority.js (1/1)
  • uv run python scripts/run-custom-tests.py (7/7)
  • uv run python scripts/run-test262.py test262/test/built-ins/RegExp/ (3,756/3,756)
  • ./scripts/run-library-tests.sh prismjs --clean (JSSE and Node: 2,563/2,563)
  • cargo fmt --check
  • cargo clippy
  • ./scripts/lint.sh
  • cargo build --release
  • cargo test --release (294 unit tests plus smoke oracle)
  • TZ=UTC uv run python scripts/run-test262.py (99,568/99,568; zero regressions)

The ambient Europe/Berlin full-suite run exposed 22 unrelated default-time-zone Intl failures; the repository baseline was last verified under UTC, and the affected Intl subset passes 328/328 under TZ=UTC.

Closes #271

@pmatos
pmatos merged commit 068ffd6 into main Jul 20, 2026
8 checks passed
@pmatos
pmatos deleted the sym/jsse/271-regexp-nested-alternations-choose-the-wrong-greedy-match branch July 20, 2026 05:56
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 0.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RegExp nested alternations choose the wrong greedy match

1 participant