[antosubash]

Summary

• db: make_include_object drops NEW foreign keys from autogenerate #812: make_include_object skips only reflected foreign keys that the models lack. A newly modeled FK on an existing column is now emitted. An FK with the same columns and target but different options (ondelete, deferrable) is replaced with a drop plus a create. FKs that exist only in migrations stay preserved.
• db: StrEnum migration rendering uses values while the runtime type uses names, and drops enum options #813 (behaviour change): StrEnum migration rendering uses the column's configured labels and keeps native_enum, create_constraint and length. A plain StrEnum field now renders member-name labels, which is what the runtime type binds. Value labels need values_callable on the model. Existing revisions are not rewritten.
• db: build_module_metadata swallows ModuleNotFoundError raised inside an existing models.py #814: build_module_metadata re-raises a ModuleNotFoundError raised inside an existing models.py. Before, that module's tables silently dropped out of autogenerate.
• db: make doctor runs check_data_compatibility on an unfiltered session; boot uses the filtered one #815 (behaviour change): make doctor runs check_data_compatibility on the framework's filtered sessions: soft-deleted rows are hidden and tenant queries are scoped. Doctor always enforces strict tenancy, which is stricter than a single-tenant boot, so install-wide checks must opt into all_tenants(). Doctor no longer switches a SQLite file to WAL.
• Docs (docs/database/migrations.md, per-module-base.md, docs/framework/lifecycle.md, docs/module-authoring.md), the shipped migrations skill and the CHANGELOG are updated.

Verification

• Local CI: make lint passed, along with these suites:
  • make test-py: 7680 passed
  • make test-js: 891 passed
  • make doctor: clean
  • e2e: 53 passed against a live API + Vite
• New regression tests are in framework/db/tests/test_db_migration_review_regressions.py and test_db_doctor_session_filters.py.
• Not browser-tested: no UI changes.
• Verification report: https://claude.ai/artifact/2q55vibfG1kTnDzVTkQcKL

Report links are Claude Artifacts and start private; the author can share them from claude.ai/code/artifacts if reviewers need access.

Closes #812
Closes #813
Closes #814
Closes #815

Test plan

• CI is green, including the Postgres workflow and migrations-roundtrip-pg

https://claude.ai/code/session_01FXHA3ZicrtZ4C5QPEgxfgq

[chatgpt-codex-connector]

Codex Review Summary

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

Review	Status	Commit	Review trigger
🔒 Security Review	✅ Completed 2026-11-04T14:48:18.746208Z	0efc66f	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.

[cloudflare-workers-and-pages]

Deploying simple-module-python with    Cloudflare Pages

Latest commit:	0efc66f
Status:	✅  Deploy successful!
Preview URL:	https://7a39a5df.simple-module-python.pages.dev
Branch Preview URL:	https://fix-db-review-migrations.simple-module-python.pages.dev

View logs