Skip to content

Add linter rules that check for ECMA-262 regex compliance and support 2020-12 format assertion#764

Merged
jviotti merged 2 commits into
mainfrom
new-rules
Jun 1, 2026
Merged

Add linter rules that check for ECMA-262 regex compliance and support 2020-12 format assertion#764
jviotti merged 2 commits into
mainfrom
new-rules

Conversation

@jviotti
Copy link
Copy Markdown
Member

@jviotti jviotti commented Jun 1, 2026

No description provided.

jviotti added 2 commits June 1, 2026 16:07
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
… 2020-12 format assertion

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti changed the title new rules Add linter rules that check for ECMA-262 regex compliance and support 2020-12 format assertion Jun 1, 2026
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 46 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread test/validate/pass_2020_12_format_assertion_optional.sh
@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented Jun 1, 2026

🤖 Augment PR Summary

Summary: Updates vendored Core/Blaze to add new linting and validation capabilities, notably ECMA-262 regex compliance checks and JSON Schema 2020-12 format-assertion support.

Changes:

  • Bump vendored core and blaze revisions (and refresh the JSON-Schema-Test-Suite rev).
  • Add Blaze linter rules to flag non‑ECMA‑262 regular expressions in pattern and patternProperties.
  • Enable compilation/evaluation of the 2020‑12 format-assertion vocabulary and extend string format assertions (IRI/partial-time/CSS2 color).
  • Introduce Core utilities for CSS2 colors, RFC3339 partial-time parsing, IRI syntax checking, and JSON-RPC batch envelope detection; adjust MCP error helper signature.
  • Update/rename several tests from “fail” to “pass” to reflect newly supported vocabularies and behavior.

Technical notes: Expands Core components consumed by Blaze (e.g., unicode/idna/punycode/css) and tightens some compiler hardening flags.

🤖 Was this summary useful? React with 👍 or 👎

Copy link
Copy Markdown

@augmentcode augmentcode Bot left a comment

Choose a reason for hiding this comment

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

Review completed. 3 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread vendor/core/src/core/uritemplate/uritemplate_router.cc
Comment thread vendor/core/src/lang/text/include/sourcemeta/core/text.h
Comment thread vendor/core/src/core/jsonrpc/jsonrpc.cc
@jviotti jviotti merged commit 6538a3e into main Jun 1, 2026
15 checks passed
@jviotti jviotti deleted the new-rules branch June 1, 2026 20:29
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