Skip to content

chore(schema): re-sync vendored ingest.v1.json from data-ingestors master#79

Open
LukasWodka wants to merge 1 commit into
developfrom
chore/sync-ingest-schema
Open

chore(schema): re-sync vendored ingest.v1.json from data-ingestors master#79
LukasWodka wants to merge 1 commit into
developfrom
chore/sync-ingest-schema

Conversation

@LukasWodka

Copy link
Copy Markdown
Contributor

Re-syncs the CLI's embedded internal/schema/ingest.v1.json with tracebloc/data-ingestors master so the Schema drift check CI job passes again (it currently fails on every cli PR, including #78).

Ran scripts/sync-schema.sh; verified --check is clean, go build, and go test ./... all pass.

Notable upstream content: target_size description [height, width][width, height] (matches PIL.Image.size / ImageResolutionValidator) + current category-conditional blocks.

Distinct from client-runtime#106 (which synced client-runtime's vendored copy); this is the CLI's separate embedded copy. Once merged, #78 picks it up via a develop merge.

🤖 Generated with Claude Code

…ster

The embedded CLI copy had drifted from tracebloc/data-ingestors master, so the
`Schema drift check` CI job (scripts/sync-schema.sh --check) fails on every cli
PR. Re-synced via `scripts/sync-schema.sh`.

Brings in the current upstream schema, including a target_size description
change ("[height, width]" -> "[width, height]", noting it matches PIL.Image.size
+ ImageResolutionValidator) and the latest category-conditional blocks.

Unblocks the drift check for open cli PRs (e.g. #78). Separate from data-ingestors
itself and from client-runtime#106 (which synced client-runtime's own vendored copy).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@LukasWodka

Copy link
Copy Markdown
Contributor Author

👋 Heads-up — Code review queue is at 11 / 8

Above the WIP limit. The team convention is to review existing PRs before opening new work.

Open PRs currently in Code review (oldest first):

Pull from review before opening new work. (This is a nudge from the kanban WIP check, not a block.)

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.

3 participants