From 68d746df2a0ef87cf465847c06018dc8b371e80e Mon Sep 17 00:00:00 2001 From: Teknium <127238744+teknium1@users.noreply.github.com> Date: Wed, 2 Sep 2026 23:02:46 -0700 Subject: [PATCH] refactor(hermes_cli): sessions_cmd list/recover minor tightening --- hermes_cli/sessions_cmd.py | 12 +++++------- 1 file changed, 5 insertions(+), 7 deletions(-) diff --git a/hermes_cli/sessions_cmd.py b/hermes_cli/sessions_cmd.py index ec436bd5af..e1937d5751 100644 --- a/hermes_cli/sessions_cmd.py +++ b/hermes_cli/sessions_cmd.py @@ -169,8 +169,7 @@ def _cmd_recover(args): ) progress.finish() except (SessionRecoveryError, OSError, sqlite3.DatabaseError) as exc: - print(f"Error: session recovery failed: {exc}") - print("The supplied source database was not replaced or deleted.") + print(f"Error: session recovery failed: {exc}\nThe supplied source database was not replaced or deleted.") return 1 if report_path is not None: try: @@ -261,11 +260,10 @@ def _cmd_list(db, args): _ws_filter = (getattr(args, "workspace", None) or "").strip() if _ws_filter: _needle = _ws_filter.lower() - - def _in_workspace(s): - key = (_ws_key(s) or "").lower() - return bool(key) and (_needle in key or _needle == os.path.basename(key.rstrip("/\\"))) - sessions = [s for s in sessions if _in_workspace(s)] + keyed = ((s, (_ws_key(s) or "").lower()) for s in sessions) + sessions = [ + s for s, key in keyed if key and (_needle in key or _needle == os.path.basename(key.rstrip("/\\"))) + ] if not sessions: print("No sessions found.") return