Skip to content

Fix crash on unevaluatedProperties and a default identifier#758

Merged
jviotti merged 1 commit into
mainfrom
fix-unevaluated
May 15, 2026
Merged

Fix crash on unevaluatedProperties and a default identifier#758
jviotti merged 1 commit into
mainfrom
fix-unevaluated

Conversation

@jviotti
Copy link
Copy Markdown
Member

@jviotti jviotti commented May 15, 2026

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

Fixes: #756
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
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.

No issues found across 16 files

Re-trigger cubic

@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented May 15, 2026

🤖 Augment PR Summary

Summary: This PR updates vendored dependencies and fixes a crash related to unevaluatedProperties by improving how Blaze registers unevaluated entries across alternate schema identifiers/bases.

Changes:

  • Blaze: registers `unevaluatedProperties` / `unevaluatedItems` under all reachable base URIs for the same schema pointer to avoid missing lookups and crashes.
  • Core: adds a new email component with RFC 5321 mailbox validation (is_email).
  • Core I/O: introduces non-atomic write_file overloads for strings, byte spans, and stream-writer callbacks.
  • URI Template: adds URITemplate::is_uritemplate for validation without constructing the parsed token representation.
  • URI Template Router: extends supported template syntax with optional greedy capture ({/var*}) and adds an optional base_url metadata field.
  • Router serialization: bumps the binary router format version and persists/restores base_url and optional-expansion semantics.
  • Build system: wires the new Core email component into CMake options/config.

Technical Notes: The router format is versioned; this PR increments the version to reflect header/layout and matching behavior changes.

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

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

@jviotti jviotti merged commit bb02cdd into main May 15, 2026
14 checks passed
@jviotti jviotti deleted the fix-unevaluated branch May 15, 2026 14:26
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request May 18, 2026
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [sourcemeta/jsonschema](https://github.com/sourcemeta/jsonschema) | patch | `v15.6.1` → `v15.6.2` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>sourcemeta/jsonschema (sourcemeta/jsonschema)</summary>

### [`v15.6.2`](https://github.com/sourcemeta/jsonschema/releases/tag/v15.6.2)

[Compare Source](sourcemeta/jsonschema@v15.6.1...v15.6.2)

#### What's Changed

- Fix crash on `unevaluatedProperties` and a default identifier by [@&#8203;jviotti](https://github.com/jviotti) in [#&#8203;758](sourcemeta/jsonschema#758)

**Full Changelog**: <sourcemeta/jsonschema@v15.6.1...v15.6.2>

</details>

---

### Configuration

📅 **Schedule**: (UTC)

- Branch creation
  - At any time (no schedule defined)
- Automerge
  - At any time (no schedule defined)

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever MR is behind base branch, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Mend Renovate](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0My4xODAuMCIsInVwZGF0ZWRJblZlciI6IjQzLjE4MC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiIsImxhYmVscyI6WyJSZW5vdmF0ZSBCb3QiLCJhdXRvbWF0aW9uOmJvdC1hdXRob3JlZCIsImRlcGVuZGVuY3ktdHlwZTo6cGF0Y2giXX0=-->
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.

Unexpected error when validating schema with $dynamicAnchor and unevaluatedProperties

1 participant