Skip to content

Add a new x-format-assertion hint to force-enable format validation#766

Merged
jviotti merged 1 commit into
mainfrom
x-format-assertion
Jun 2, 2026
Merged

Add a new x-format-assertion hint to force-enable format validation#766
jviotti merged 1 commit into
mainfrom
x-format-assertion

Conversation

@jviotti

@jviotti jviotti commented Jun 2, 2026

Copy link
Copy Markdown
Member

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

…idation

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti changed the title Add a new x-format-assertion: true hint for force-enable format validation Add a new x-format-assertion hint to force-enable format validation Jun 2, 2026
@augmentcode

augmentcode Bot commented Jun 2, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR introduces a schema-level opt-in to treat format as an assertion (instead of the default annotation behavior) at specific subschema locations.

Changes:

  • Bumps the vendored blaze dependency to a newer commit.
  • Adds support in the Blaze compiler for a custom sibling keyword x-format-assertion: true to force-enable format validation.
  • Uses the new flag to activate assertion-mode format compilation even when the global format_assertion tweak is not enabled.
  • Extends documentation to explain default format behavior and how to opt into assertion behavior per-location.
  • Adds CLI integration tests covering both Draft 4 and Draft 2020-12 for test and validate commands.

Technical Notes: The new behavior is activated via a boolean-only x-format-assertion sibling and is intended to work across dialects, including older drafts where format is otherwise only an annotation.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

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

Comment thread vendor/blaze/src/compiler/default_compiler_draft3.h

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

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

1 issue found across 8 files

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

Re-trigger cubic

Comment thread docs/validate.markdown
@jviotti jviotti merged commit f97906c into main Jun 2, 2026
15 checks passed
@jviotti jviotti deleted the x-format-assertion branch June 2, 2026 13:34
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