nesquena-hermes c312dd36ca fix: state_sync.py -- correct class name, constructor type, title API, connection leak ...

Three bugs found during review:
1. Class is SessionDB not HermesState -- would silently no-op on every install
2. SessionDB.__init__ takes Path not str -- would crash with AttributeError
3. _execute_write() takes a callable not SQL+params -- wrong signature.
   Replaced with public set_session_title() API.
4. Each call opened a persistent SQLite connection and never closed it.
   Added try/finally db.close() to prevent WAL leak under sustained load.

Co-authored-by: Nathan Esquenazi <nesquena@gmail.com>

2026-04-04 20:08:20 -07:00

3.0 KiB

Raw Blame History

View Raw