Skip to content

Conversation

@jviotti
Copy link
Member

@jviotti jviotti commented Jan 21, 2026

Signed-off-by: Juan Cruz Viotti [email protected]

@jviotti jviotti changed the title Upgrade Blaze to dff0468526a694121539850ab53a41fa0bb42ceb Speed up schema compilation by orders of magnitude Jan 21, 2026
@augmentcode
Copy link

augmentcode bot commented Jan 21, 2026

🤖 Augment PR Summary

Summary: This PR upgrades the vendored sourcemeta/blaze (and sourcemeta/core) to newer upstream commits.

Changes:

  • Reworked Blaze compilation to build a set of precompiled jump targets (instead of emitting a single flat instruction stream plus label/mark instructions).
  • Added a compiler post-processing pass (postprocess.h) to inline small jump targets and apply additional FastValidation-specific structural optimizations.
  • Updated the evaluator to execute schema.targets[0] as the entrypoint, and to resolve ControlJump and ControlDynamicAnchorJump against the new targets/labels layout.
  • Changed template JSON serialization/deserialization to include targets and a labels table, adjusting CLI test expectations accordingly.
  • Removed legacy JSON Pointer mangle support from the vendored Core library.

Technical Notes: Dynamic-anchor lookup now uses a precomputed (label → target index) mapping, and several compilation-time knobs were simplified/renamed (e.g., target inlining threshold).

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@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. 2 suggestions posted.

Fix All in Augment

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

@jviotti jviotti merged commit f5b4df2 into main Jan 21, 2026
4 checks passed
@jviotti jviotti deleted the blaze-fast branch January 21, 2026 18:55
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.

2 participants