Skip to content

Implement a new API to get schema dependents #623

Merged
jviotti merged 1 commit intomainfrom
dependents
Jan 27, 2026
Merged

Implement a new API to get schema dependents #623
jviotti merged 1 commit intomainfrom
dependents

Conversation

@jviotti
Copy link
Member

@jviotti jviotti commented Jan 27, 2026

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

@jviotti jviotti force-pushed the dependents branch 5 times, most recently from 7800baf to 78e31d8 Compare January 27, 2026 14:57
@jviotti jviotti changed the title [WIP] Analyze schema dependents Implement a new API to get schema dependents Jan 27, 2026
@jviotti jviotti marked this pull request as ready for review January 27, 2026 14:58
@augmentcode
Copy link

augmentcode bot commented Jan 27, 2026

🤖 Augment PR Summary

Summary: Adds a new API endpoint to expose “schema dependents” (reverse dependencies) for any indexed JSON Schema.

Changes:

  • Introduces a new response schema for the dependents endpoint under the self/v1 collection.
  • Documents GET /self/v1/api/schemas/dependents/{path} in the API docs.
  • Extends the indexer to build a global dependency tree (dependency-tree.metapack) from per-schema dependency traces.
  • Generates per-schema dependents.metapack artifacts by querying that dependency tree.
  • Registers the new route in the precomputed router and wires a new server handler to serve dependents.metapack.
  • Updates CLI output expectations/manifests and adds E2E hurl tests for the new endpoint.

Technical Notes: Dependents are computed as direct+transitive incoming edges; results are served as compressed JSON via the metapack pipeline.

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

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti merged commit f82d38f into main Jan 27, 2026
4 checks passed
@jviotti jviotti deleted the dependents branch January 27, 2026 15:15
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