Feat/0.1 hardening - #29
Merged
Merged
Conversation
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
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.
Summary
Closes out v0.1 hardening with fixes discovered during acceptance testing, a repeatable post-clone setup workflow, and documentation that reflects the shipped v0.1 scope.
/datadirectory on native Linux, while retaining Docker Desktop compatibility../scripts/setup.sh(also available asdeno task setup) to preparedata/devanddata/prod, configure tracked Git hooks, install or build locked dependencies, migrate available development and production databases, and report each workflow as ready or blocked.Title is required.), focus return, andaria-invalid.Root cause and impact
A successful image build did not exercise the production entrypoint against a real host-owned bind mount. On native Linux, a UID/GID mismatch between the image user and
./data/prodcould therefore leave the app unable to create or write SQLite even while CI remained green. The entrypoint now handles that boundary at runtime, and CI exercises the same startup, health, and ownership path used by Compose deployments.For a new clone, users now run one setup command and receive separate Docker and local-development readiness results. Missing Docker does not prevent a valid Deno setup, and missing Deno does not prevent a valid Docker setup. CI separately requires both workflows so partial readiness cannot hide a regression.
Validation
deno task ci— 291 API tests and 94 frontend tests passed, plus lint, type-check, full API QA, and production buildCURRENTon rerun