Complete native PostgreSQL and SQLite repositories - #129
Open
arvindh123 wants to merge 4 commits into
Open
arvindh123 wants to merge 4 commits into
arvindh123 wants to merge 4 commits into
Conversation
Move all remaining storage behind typed domain operations with private native adapters. Remove runtime SQL translation and preserve transaction, authorization and outbox behavior. Add sorting, rollback, savepoint and JSON parity regressions. Closes #126
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
atom-docs | 7bb780a | Commit Preview URL Branch Preview URL |
Sep 30 2026, 10:12 PM |
Correct the explicit SQLite audit fixture SQL and update Next.js to 16.3.6 after CI reported GHSA-vcvr-r3jv-pc5j. UI frozen install, lint, 116 tests, build, and high-severity production audit pass.
Remove the remaining PostgreSQL-only wording from the introduction to match the native PostgreSQL/SQLite implementation.
arvindh123
marked this pull request as ready for review
September 30, 2026 22:50
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Add the host DATABASE_URL, PostgreSQL port, SQLite alternatives, pool options, and administrator/encryption guidance. Explain the current Make and Compose backend selection behavior. Both URL examples parse with dotenvy, and Compose configuration validation passes.
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.
Closes #126. Completes the PostgreSQL/SQLite repository migration started in #119 in this single follow-up PR.
All production database operations now use typed domain operations and private native PostgreSQL/SQLite adapters. Business validation, authorization evaluation, transaction orchestration, cache barriers, and audit/outbox behavior stay shared. The runtime SQL translator and generic query compatibility API are removed; test fixtures supply explicit SQL for each backend.
The environment example includes a PostgreSQL host URL and port, SQLite URL alternatives, pool options, and admin/encryption guidance, with the current Make/Compose limitations explained. Both URL examples parse with Atom's dotenvy parser; Docker Compose configuration validation also passes.
The UI CI audit exposed GHSA-vcvr-r3jv-pc5j in the existing Next.js pin. This PR also updates Next.js to 16.3.6; frozen install, lint, all 116 UI tests, production build, and the high-severity production audit pass (one moderate advisory remains).
Validation (completed locally on the submitted tree):
cargo fmt --all --check,cargo clippy --locked -- -D warnings, database boundary/schema parity, frozen v1 contracts and their 12 gate tests, vendored proto check, and generated GraphQL schema comparison.CI covers Rust tests on both database backends, PKI recovery and smoke, UI, documentation, and API/contracts. The Checks tab shows the latest run.
Coverage limits are explicit: the two live AMQP suites (
m27_live_amqp_delivery,m28_amqp_mtls_local_principal) compile but were not executed because their external broker/mTLS setup is not provisioned, matching existing CI exclusions. PostgreSQL lock-observation cases (three object-coordination cases, two entity-authorization cases, and the audit foreign-key lock regression), the process-replacement test, and synthetic PKI serial-collision injection remain PostgreSQL-specific. SQLite is covered by its writer/ownership, concurrent mutation, restart, and shared savepoint regressions; these PostgreSQL-specific skips are not claimed as SQLite passes. SoftHSM backup recovery is verified on PostgreSQL; the provider path runs on both backends.SQLite-to-PostgreSQL data transfer is outside this issue: changing
DATABASE_URLselects a backend and does not move existing data. No applied migrations are changed.