Skip to content

fix: handle Copilot session-state without SQLite DB - #29

Open
erSpell wants to merge 1 commit into
dezgit2025:mainfrom
erSpell:fix/copilot-session-state-health-fallback
Open

erSpell wants to merge 1 commit into
dezgit2025:mainfrom
erSpell:fix/copilot-session-state-health-fallback

Conversation

@erSpell

@erSpell erSpell commented Sep 8, 2026

Copy link
Copy Markdown

Summary

Preserve POSIX/WSL-style Copilot CLI workspace labels on Windows instead of rewriting leading slashes.
Add regression coverage for health discovery when Copilot CLI session-state events exist without the legacy SQLite session-store.db.
Relevant regression/provider tests and lint successful. The local suite excluding symlink-creation tests passed. The excluded symlink tests require Windows symlink privileges in this environment and are unrelated to the session-state fallback fix.

Tests

. .venv/Scripts/activate && python -m pytest src/session_recall/tests/test_provider_backends.py src/session_recall/tests/test_health_schema_multistorage.py -q
. .venv/Scripts/activate && python -m compileall -q src/session_recall && python -m ruff check src
. .venv/Scripts/activate && python -m pytest src/session_recall/tests -q -k 'not symlink'
. .Full Windows local testing excludes symlink tests
. .Closes [Bug] database not found. Copilot changed storage format from session-store.db to session-state folder #19

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.

1 participant