Skip to content

Conversation

@jviotti
Copy link
Member

@jviotti jviotti commented Jan 8, 2026

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

Signed-off-by: Juan Cruz Viotti <[email protected]>
@jviotti jviotti force-pushed the all-deps-7 branch 4 times, most recently from d725f8f to a7e083f Compare January 9, 2026 13:35
Signed-off-by: Juan Cruz Viotti <[email protected]>
@jviotti jviotti marked this pull request as ready for review January 9, 2026 13:59
@augmentcode
Copy link

augmentcode bot commented Jan 9, 2026

🤖 Augment PR Summary

Summary: This PR upgrades the vendored Sourcemeta dependency set used by Sourcemeta One, and updates One’s code to match the updated upstream APIs.

Changes:

  • Updated the top-level DEPENDENCIES pins for core/blaze/jsonbinpack/hydra/jsonschema to newer revisions.
  • Refreshed vendored third-party libraries pulled in via Hydra (notably curl and CPR), including accompanying CMake find/build glue.
  • Adjusted One’s indexing/generator logic to match updated JSON Schema APIs (e.g., identify()/dialect() returning std::string_view and base dialect being an enum).
  • Updated resolver/index code paths to pass the new “default dialect/default id” parameters where required by the new framing/identification behavior.
  • Minor API hygiene updates: added override to std::exception::what() overrides and added [[nodiscard]] on a few return values.
  • Misc repository housekeeping (e.g., ignore /.cache).

Technical Notes: This is a broad vendoring refresh (hundreds of files changed); the primary functional impact is aligning One with new Sourcemeta core/jsonschema interfaces and refreshed Hydra-provided third-party sources.

🤖 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. No suggestions at this time.

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

@jviotti jviotti merged commit e078ed4 into main Jan 9, 2026
4 checks passed
@jviotti jviotti deleted the all-deps-7 branch January 9, 2026 14:10
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