Skip to content

Extract JSON ingestion into a dedicated gem#1121

Open
jwils wants to merge 1 commit intomainfrom
joshuaw/pr-1079-json-ingestion-pr1
Open

Extract JSON ingestion into a dedicated gem#1121
jwils wants to merge 1 commit intomainfrom
joshuaw/pr-1079-json-ingestion-pr1

Conversation

@jwils
Copy link
Copy Markdown
Collaborator

@jwils jwils commented Apr 7, 2026

Why

PR #1079 mixed extraction, new config surface area, and test movement into one large change. This first step isolates the JSON-ingestion extraction while preserving the existing schema_definition behavior and test surface as much as possible.

What

  • Add the new elasticgraph-json_ingestion gem with the extracted JSON schema ingestion implementation and tests
  • Keep elasticgraph-schema_definition auto-wired for JSON ingestion and preserve old require paths via compatibility wrappers
  • Drop the new serializer-extension config surface from this first PR and keep the old API shape

Risk Assessment

Medium. This moves schema-definition internals and artifact generation code, but preserves the existing public behavior and is covered by the extracted gem specs plus the preserved schema_definition unit and integration suites.

References

Generated with Codex

@jwils jwils force-pushed the joshuaw/pr-1079-json-ingestion-pr1 branch 5 times, most recently from 7411c18 to dc21bab Compare April 7, 2026 23:21
@jwils jwils force-pushed the joshuaw/pr-1079-json-ingestion-pr1 branch from dc21bab to 233c97e Compare April 7, 2026 23:36
@jwils jwils marked this pull request as ready for review April 8, 2026 02:25
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