Skip to content

JS: Add support for ES2022's \d flag - #22309

Open
MathiasVP wants to merge 4 commits into
github:mainfrom
MathiasVP:js-add-support-for-d-flag
Open

JS: Add support for ES2022's \d flag#22309
MathiasVP wants to merge 4 commits into
github:mainfrom
MathiasVP:js-add-support-for-d-flag

Conversation

@MathiasVP

Copy link
Copy Markdown
Contributor

Does what it says on the tin.

cc @asgerf

Copilot AI balanced review requested due to automatic review settings August 10, 2026 11:42
@MathiasVP
MathiasVP requested review from a team as code owners August 10, 2026 11:42
@github-actions github-actions Bot added the JS label Aug 10, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds ECMAScript 2022 support for the regular-expression d flag.

Changes:

  • Allows d for ECMAScript edition 13 and later.
  • Adds extraction coverage and expected TRAP output.
Show a summary per file
File Description
Parser.java Extends valid regular-expression flags.
d-flag.js Adds an ES2022 test case.
d-flag.js.trap Records expected extraction output.
options.json Enables experimental syntax for the test.

Review details

Tip

Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 4/4 changed files
  • Comments generated: 1
  • Review effort level: Balanced

Comment thread javascript/extractor/src/com/semmle/jcorn/Parser.java
@MathiasVP MathiasVP changed the title JS: Add support for ES2020's \d flag JS: Add support for ES2022's \d flag Aug 10, 2026

@asgerf asgerf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing! Could you add a change note? Otherwise LGTM

@MathiasVP
MathiasVP force-pushed the js-add-support-for-d-flag branch from 3d22f66 to 224ab74 Compare August 10, 2026 12:22
@MathiasVP

Copy link
Copy Markdown
Contributor Author

Thanks for fixing! Could you add a change note? Otherwise LGTM

Sure! Done in 224ab74

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants