Skip to content

Ship schemas as package data; schemas.version/path/load_file and an imaging extra (SM-CORE1) - #23

Merged
DatMoshu merged 1 commit into
mainfrom
worker/sm-core1-schemas
Oct 7, 2026
Merged

DatMoshu merged 1 commit into
mainfrom
worker/sm-core1-schemas

Conversation

@DatMoshu

@DatMoshu DatMoshu commented Oct 7, 2026

Copy link
Copy Markdown
Owner

SM-CORE1, accepted by spritemotion-director at b914167.

  • Schemas move into the spritemotion.schemas package data (common/schemas/), with schemas.version, schemas.path and schemas.load_file.
  • Core install has deps=[]; imaging libraries sit behind an imaging extra.
  • Wheel and stdlib-only install test (4/4); pytest 107 pass, 1 skip (needs UO client data); outfit-lab OK; agents check clean.

Merge note: the main checkout's uncommitted WIP edits schemas/equipment-slots.schema.json, which this PR moves to common/schemas/; that WIP re-applies there.

🤖 Generated with Claude Code

…imaging extra

Move schemas/*.schema.json into common/schemas/ (package spritemotion.schemas),
make numpy and Pillow an optional imaging extra, and add an install test that
builds a wheel and imports the core with the standard library only.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-07T02:45:24.732624Z b914167 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@DatMoshu
DatMoshu merged commit dbc8a9a into main Oct 7, 2026
2 checks passed
@DatMoshu
DatMoshu deleted the worker/sm-core1-schemas branch October 7, 2026 02:44
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