Extract single shared Database pool, eliminate 4 duplicated pool creations

Fork: 0

root / navi-1

Browse code Extract single shared Database pool, eliminate 4 duplicated pool creations - Create navi/db.py::Database managing one asyncpg pool - KvStore, PgSessionStore, MemoryStore, RecallScheduler now accept pool in constructor - AppContainer holds Database, shutdown closes one pool instead of 4 - create_container creates one pool and passes it to all stores - All tests updated to set _initialized=True on fakes to skip DDL 392 passed, 1 skipped Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com> feature/navi-code master vmkdemo
1 parent 4c5f683 commit f002ea16f058b09201ff66821cadc4bd43008d7c Eugene Sukhodolskiy authored on 18 May

Browse code

- Create navi/db.py::Database managing one asyncpg pool
- KvStore, PgSessionStore, MemoryStore, RecallScheduler now accept pool in constructor
- AppContainer holds Database, shutdown closes one pool instead of 4
- create_container creates one pool and passes it to all stores
- All tests updated to set _initialized=True on fakes to skip DDL

392 passed, 1 skipped

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

feature/navi-code master vmkdemo

1 parent 4c5f683 commit f002ea16f058b09201ff66821cadc4bd43008d7c

Eugene Sukhodolskiy authored on 18 May

Patch

Unified Split

Showing 10 changed files

Ignore Space Show notes View navi/core/container.py

Ignore Space Show notes View navi/core/pg_session_store.py

Ignore Space Show notes View navi/core/scheduler.py

Ignore Space Show notes View navi/db.py 0 → 100644

Ignore Space Show notes View navi/main.py

Ignore Space Show notes View navi/memory/store.py

Ignore Space Show notes View navi/store/__init__.py

Ignore Space Show notes View tests/conftest_factory.py

Ignore Space Show notes View tests/integration/conftest.py

Ignore Space Show notes View tests/unit/store/test_kv_store.py

Show line notes below