test(db): align PostgreSQL smoke fixtures with current schema - #1468
test(db): align PostgreSQL smoke fixtures with current schema#1468seonghobae wants to merge 6 commits into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
PR governance metadata gate is not ready for
|
|
The current-head |
Maintainer exact-head validation
|
… into codex/pr1468-restack
…b.com/ContextualWisdomLab/naruon into codex/pr1468-restack # Conflicts: # CHANGELOG.md
|
현재 head |
Merge #1503 normally as the prerequisite to #1468, preserving the unique bootstrap regression assertion and changelog. The owner already carries the is_read fixture delta. Verified merged tree 00e170c with fresh Alembic upgrade and 75 clean-lock strict PostgreSQL tests, Ruff, and diff checks. No delta is discarded or protected merge claimed. Co-Authored-By: Codex <noreply@openai.com> Signed-off-by: Seongho Bae <me@seonghobae.me>
Non-force merge of updated #1468, retaining the PDF upload contract and Proposed ADR. Merged tree e0717d3 passes fresh Alembic upgrade, 76 clean-lock strict PostgreSQL/dependency tests, Ruff, and diff checks. Owner service release and hosted review/Checks remain separate prerequisites; no 64 MiB sidecar runtime or protected merge is claimed. Co-Authored-By: Codex <noreply@openai.com> Signed-off-by: Seongho Bae <me@seonghobae.me>
Preserve the unique no-unconditional-legacy-index assertion and both owner histories. Correct the historical-only canonical-index release claim; conditional legacy handling remains inherited from #1503. Exact dependency sync, fresh and repeat migration 0020, and 131 strict PostgreSQL tests pass. Search and migration performance remain Draft acceptance gates in #1572; no protected completion is claimed. Co-authored-by: Codex <noreply@openai.com> Signed-off-by: Seongho Bae <me@seonghobae.me>
Normally merge #1468 while preserving the PDF admission, signed-session, and deferred-worker delta. Exact synchronization, fresh and repeat migration 0020, and 132 strict PostgreSQL/search tests pass. Renumber the colliding unpublished ADR proposal to 0021 and retain its former identity and owner-release gate. Correct provider maturity claims; 64MiB NewsDOM runtime and search performance remain unverified. Co-authored-by: Codex <noreply@openai.com> Signed-off-by: Seongho Bae <me@seonghobae.me>
Current search-owner propagation receipt (2026-09-05)
Head
53ce38ed6683d01a9d113069f5ac5a8f17e133a2, tree723d9f4b4d503cf19b67d473a8d5398b7b0dd114, normally merges prior head037b58adeda53e6c847f8949494b9b518a94dac9with #1572cd8ff413d4ed8a5f2855c47a21a31db5661cd487. The PR is retargeted to the search owner, preserving both histories and the unique bootstrap assertion. Its only parent-relative files are CHANGELOG and the existing bootstrap test; the release note no longer claims that conditional legacy index handling was removed.Exact dependency sync, fresh and repeat migration to
0020_search_trigram_storage, and 131 tests with-W errorpass, 0 failed/skipped, in 44.26 seconds. Ruff and diff checks pass. Local artifacts:/private/tmp/naruon-search-index-rca.VRmUrq/prop1468_tests.xmlandprop1468_migration.log; isolated resources were removed by the exact-project exit trap. These are local receipts, not hosted/protected/released evidence.Keep Draft: #1572 retains the representative search p95 and migration-cost acceptance gates, and exact-head required Checks/independent review remain necessary. CodeRabbit/Devin metadata alone is not qualifying approval. Earlier receipts below retain historical scope only; they do not describe this new head.
What changed
emailsindex DDL.The inherited fix: create workspace_entities/workspace_documents registry and provision Workspace rows #1503 owner creates a structured legacy index only if that table exists;
email_recordsremains the canonical model.email_records.is_readstate explicitly, matching the current migration/model contract.
This is a small prerequisite for the current PDF DOM upload follow-up (#1427):
it restores the full backend smoke gate without mixing unrelated product code
into that PR.
Historical verification (before the current owner restack)
ruff check backend/scripts/bootstrap_db.py backend/tests/test_bootstrap_db.py backend/tests/test_data_api.pygit diff --checkPYTHONWARNINGS=error python -m pytest backend/tests -q(
1833 passed, 2 skipped)2 passed)Risk and rollback
The current unique delta is a four-line bootstrap guard and a four-line changelog
entry. Production migration repair is inherited from #1503 and must follow its
retained-data rollback policy; do not revert inherited prerequisites as if they
were disposable fixture changes.
Current owner-restack receipt — 2026-09-05
037b58adeda53e6c847f8949494b9b518a94dac9; direct base:fix/workspace-document-registry-migration@19d5860bc27e860acba940390f5792721cd99e5e.603914596cb379a5be25c33f0990404d08054ffcand the full direct-parent delta; no force push or predecessor closure.uv sync --locked, a fresh isolated PostgreSQL 16.15 database migrated through0019_email_read_state_repairusinguv run --frozen python scripts/migrate_db.py.backend/:uv run --frozen python -m pytest -q -W error -ra --tb=short tests/test_alembic_migrations.py tests/test_bootstrap_db.py tests/test_data_api.py tests/test_email_read_state_migration_postgres.py tests/test_legacy_document_scope_postgres.py tests/test_workspace_document_migration.py tests/test_container_dependency_pin_contract.py— 75 passed, 0 failed, 0 skipped. These are local migrated-PostgreSQL/source tests, not protected-merge, deployed, or external-owner release evidence.git diff --checkpassed in that integration run.