Follow-up from PR #4 review (Standards axis).
query-files-cmd (src/eca_cli/state.clj:45-51) blocks on the queryFiles response and degrades to an empty list on timeout/failure:
(let [files (deref p 5000 [])] ...)
A slow or failed chat/queryFiles silently yields "no files" with no signal to the user or log. This matches the existing init-cmd blocking-in-cmd shape, so it is consistent with the repo pattern, but the swallowed 5s timeout is the kind of defensive default Clojure Craft discourages.
Consider surfacing timeout/error (a status-line note or a boundary log) instead of presenting an empty picker as success.
Follow-up from PR #4 review (Standards axis).
query-files-cmd(src/eca_cli/state.clj:45-51) blocks on the queryFiles response and degrades to an empty list on timeout/failure:A slow or failed
chat/queryFilessilently yields "no files" with no signal to the user or log. This matches the existinginit-cmdblocking-in-cmd shape, so it is consistent with the repo pattern, but the swallowed 5s timeout is the kind of defensive default Clojure Craft discourages.Consider surfacing timeout/error (a status-line note or a boundary log) instead of presenting an empty picker as success.