Skip to content

fix(harness): align store with database config#3168

Open
yu-xin-c wants to merge 1 commit into
bytedance:mainfrom
yu-xin-c:fix/store-database-config-2810
Open

fix(harness): align store with database config#3168
yu-xin-c wants to merge 1 commit into
bytedance:mainfrom
yu-xin-c:fix/store-database-config-2810

Conversation

@yu-xin-c
Copy link
Copy Markdown

Summary

  • make async and sync Store factories follow the same persistence priority as the checkpointer: legacy checkpointer, unified database, then memory fallback
  • add unified database-backed sqlite/postgres store construction paths
  • add regression coverage for async make_store, sync store_context, and sync get_store using the unified database config

Fixes #2810.

Verification

  • uv run ruff format packages/harness/deerflow/runtime/store/async_provider.py packages/harness/deerflow/runtime/store/provider.py tests/test_store_database_config.py
  • uv run ruff check packages/harness/deerflow/runtime/store/async_provider.py packages/harness/deerflow/runtime/store/provider.py tests/test_store_database_config.py
  • uv run pytest tests/test_store_database_config.py tests/test_checkpointer.py -q

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(harness): align store persistence with unified database config

1 participant