Skip to content

Describe ingesting schemas with $id in the getting started guide#827

Merged
jviotti merged 2 commits intomainfrom
external-schemas-docs
Apr 16, 2026
Merged

Describe ingesting schemas with $id in the getting started guide#827
jviotti merged 2 commits intomainfrom
external-schemas-docs

Conversation

@jviotti
Copy link
Copy Markdown
Member

@jviotti jviotti commented Apr 15, 2026

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

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 1 file

@augmentcode
Copy link
Copy Markdown

augmentcode bot commented Apr 16, 2026

🤖 Augment PR Summary

Summary: This PR expands the Getting Started guide to cover ingesting third‑party JSON Schemas that already declare a $id.

Changes:

  • Adds a new “Ingesting Schemas with $id” section explaining why schema identifiers can conflict with a registry’s own URL
  • Documents using the baseUri collection option to rebase external schema identifiers onto the registry URL
  • Provides a worked example with an external schema and a corresponding one.json configuration snippet
  • Explains the resulting registry path and what happens when baseUri is omitted
  • Links to the public configuration used by schemas.sourcemeta.com as a real-world reference

🤖 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. 1 suggestion posted.

Fix All in Augment

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

Comment thread docs/getting-started.md
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti merged commit 4f378bb into main Apr 16, 2026
6 checks passed
@jviotti jviotti deleted the external-schemas-docs branch April 16, 2026 00:30
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Benchmark Index (community)

Details
Benchmark suite Current: 8ed6e08 Previous: 2b5a294 Ratio
Add one schema (0 existing) 18 ms 19 ms 0.95
Add one schema (100 existing) 23 ms 24 ms 0.96
Add one schema (1000 existing) 73 ms 73 ms 1
Add one schema (10000 existing) 698 ms 630 ms 1.11
Update one schema (1 existing) 16 ms 16 ms 1
Update one schema (101 existing) 23 ms 23 ms 1
Update one schema (1001 existing) 72 ms 75 ms 0.96
Update one schema (10001 existing) 801 ms 635 ms 1.26
Cached rebuild (1 existing) 8 ms 9 ms 0.89
Cached rebuild (101 existing) 10 ms 11 ms 0.91
Cached rebuild (1001 existing) 30 ms 31 ms 0.97
Cached rebuild (10001 existing) 248 ms 243 ms 1.02
Index 100 schemas 116 ms 105 ms 1.10
Index 1000 schemas 1028 ms 989 ms 1.04
Index 10000 schemas 13051 ms 13133 ms 0.99

This comment was automatically generated by workflow using github-action-benchmark.

Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Benchmark Index (enterprise)

Details
Benchmark suite Current: 8ed6e08 Previous: 2b5a294 Ratio
Add one schema (0 existing) 21 ms 20 ms 1.05
Add one schema (100 existing) 24 ms 24 ms 1
Add one schema (1000 existing) 72 ms 75 ms 0.96
Add one schema (10000 existing) 649 ms 855 ms 0.76
Update one schema (1 existing) 18 ms 19 ms 0.95
Update one schema (101 existing) 24 ms 24 ms 1
Update one schema (1001 existing) 72 ms 75 ms 0.96
Update one schema (10001 existing) 599 ms 809 ms 0.74
Cached rebuild (1 existing) 9 ms 10 ms 0.90
Cached rebuild (101 existing) 12 ms 12 ms 1
Cached rebuild (1001 existing) 31 ms 33 ms 0.94
Cached rebuild (10001 existing) 244 ms 296 ms 0.82
Index 100 schemas 110 ms 114 ms 0.96
Index 1000 schemas 964 ms 1107 ms 0.87
Index 10000 schemas 12578 ms 13140 ms 0.96

This comment was automatically generated by workflow using github-action-benchmark.

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