Phase 0 & 0.5: harden the template before Copier conversion#25
Merged
Conversation
Bump the Python floor to 3.13 (regenerated uv.lock), purge mypy-era leftovers, fix the docker-compose first-run (auto-create schema, reload, named volume), harden the Dockerfile (multi-stage, non-root, EXPOSE, UV_COMPILE_BYTECODE), refresh .dockerignore, and tidy the Makefile and lint config. Closes #7, #8, #10, #12 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Drop the paths filters, run push on main + pull_request on all branches, bump checkout/setup-uv, drop setup-python and the pip upgrade, fix the docker image tag, and enforce the ADR registry via make adr-check. Closes #14 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Delete the unused BasePydanticField/PydanticModelField/PydanticModelListField/Base.model_dump helpers and BaseEnum (plus their tests), and the empty utils package. Closes #11 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…tection Add an explicit write-back path (identity map + merge on commit) with a DeactivateUser demo and regression test, raise UnhandledCommand for unregistered commands, guard event collection on un-entered units of work, and detect in-memory SQLite via make_url. Records the persistence decision as ADR 0016. Closes #15, #16, #17 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
17 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Phase 0 & 0.5 — harden the template before the Copier conversion
Implements the Phase 0 and Phase 0.5 work items from the conversion epic #24. All changes were engineered, then validated together on Python 3.13.5:
make adr-check) · Docker build succeeds (non-root)Highlights
docker compose up(schema auto-create, live reload, named volume) and a hardened multi-stage, non-root Dockerfile.FASTAPI_BACKEND_CORS_ORIGINS(credentials auto-disabled on wildcard),DEBUG=False.503+Erroron failure).mergeon commit) with aDeactivateUserdemo and regression test — closes a silent data-loss trap. Recorded as ADR 0016.Commits
Six conventional commits grouped by concern (build/CI/refactor/security/service/entrypoint).
Closes #7
Closes #8
Closes #9
Closes #10
Closes #11
Closes #12
Closes #13
Closes #14
Closes #15
Closes #16
Closes #17
Closes #18
Closes #19
🤖 Generated with Claude Code