Two cleanup items from code review with Codex: 1. session.rs: resolve_full_gemini_uuid() should be renamed resolve_short_id_uuid() or similar — the function is now generic (looks for any sibling tool-outputs/session- dir starting with the short prefix), not Gemini-specific. Comments, test names, and test fixture filenames in session.rs should also drop provider language. 2. expand_session_dirs() needs a doc comment stating its contract: patterns are relative to ~/.codex-service; only the /*/ single-level wildcard is supported; missing dirs are silently ignored; do not reuse for discovering session roots before sessions exist.
Two cleanup items from code review with Codex: 1. session.rs: resolve_full_gemini_uuid() should be renamed resolve_short_id_uuid() or similar — the function is now generic (looks for any sibling tool-outputs/session- dir starting with the short prefix), not Gemini-specific. Comments, test names, and test fixture filenames in session.rs should also drop provider language. 2. expand_session_dirs() needs a doc comment stating its contract: patterns are relative to ~/.codex-service; only the /*/ single-level wildcard is supported; missing dirs are silently ignored; do not reuse for discovering session roots before sessions exist.