Skip to content

Conversation

@jviotti
Copy link
Member

@jviotti jviotti commented Dec 21, 2025

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

@augmentcode
Copy link

augmentcode bot commented Dec 21, 2025

🤖 Augment PR Summary

Summary: Updates the vendored sourcemeta/core dependency to commit 42bd02b4…, aligning jsonbinpack with the latest Core APIs and schema tooling.

Changes:

  • Bump Core revision in DEPENDENCIES and adjust compiler canonicalization to use AlterSchemaMode::Canonicalize.
  • Add string whitespace helpers in Core JSON (e.g. JSON::is_trimmed()) and minor hashing/trim refactors.
  • Extend JSON Pointer support (stringify PointerTemplate, hashing for pointer templates, convenience constructors).
  • Improve JSON Schema framing/instance-location propagation, especially around definitions and reference chains.
  • Refactor schema transformer internals (rule storage order, richer apply() return with a health percentage) and update type parsing to accept string-or-array type.
  • Enhance YAML/JSON reading behavior by falling back based on parse errors when the extension is unknown.
  • Restructure/expand alterschema rules, introducing new canonicalizer/common/linter rules and renaming modes to Linter/Canonicalize.

Technical Notes: This is largely an upstream vendored change; downstream callers should account for the updated alterschema mode names and the updated schema-transformer apply()/check() return types.

🤖 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.

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti force-pushed the upgrade-new-alterschema branch from 630daba to dedf86c Compare December 21, 2025 01:15
@jviotti jviotti changed the title Upgrade Core to 42bd02b438576a175eb6ccddca2285fdb6033c0f Upgrade Core to 31d3868e3f4217be3ed5de3b0ea31d521f1b873c Dec 21, 2025
@jviotti jviotti merged commit 6cef045 into main Dec 21, 2025
13 checks passed
@jviotti jviotti deleted the upgrade-new-alterschema branch December 21, 2025 01:16
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