fix(cli): add missing _log.debug call to session export/import/tell DatabaseError handlers #3184

Merged
freemo merged 1 commits from fix/session-database-error-log-debug into master 2026-04-05 21:13:22 +00:00

1 Commits

Author SHA1 Message Date
freemo fc00834bd3 fix(cli): add missing _log.debug call to session export/import/tell DatabaseError handlers
CI / lint (pull_request) Successful in 27s
CI / typecheck (pull_request) Successful in 1m1s
CI / security (pull_request) Successful in 53s
CI / quality (pull_request) Successful in 42s
CI / build (pull_request) Successful in 27s
CI / helm (pull_request) Successful in 24s
CI / unit_tests (pull_request) Successful in 6m58s
CI / e2e_tests (pull_request) Successful in 17m8s
CI / coverage (pull_request) Successful in 10m45s
CI / integration_tests (pull_request) Successful in 22m52s
CI / docker (pull_request) Successful in 1m23s
CI / status-check (pull_request) Successful in 1s
CI / benchmark-publish (pull_request) Has been skipped
CI / benchmark-regression (pull_request) Successful in 57m12s
Add _log.debug("session export failed", exc_info=True),
_log.debug("session import failed", exc_info=True), and
_log.debug("session tell failed", exc_info=True) to the DatabaseError
handlers in export_session, import_session, and tell commands respectively.

This brings these handlers into consistency with the existing create, list,
show, and delete handlers which already include the _log.debug call for
diagnostic tracebacks when --log-level debug is used.

Also adds three new BDD scenarios and step implementations that verify
_log.debug is called in each of the three handlers, ensuring the pattern
is tested and cannot regress.

ISSUES CLOSED: #2788
2026-04-05 07:28:12 +00:00