Code review and improvements

[razeghi71]

Note

Strengthens path handling and stabilizes downloading behavior.

• Refactors BucketWithPrefix to take Sequence[str] and adds from_full_prefix; guards against string misuse and normalizes internal state; updates all call sites (storage, file_list_view, util).
• get_gcs_bucket_and_prefix now builds BucketWithPrefix via from_full_prefix and defaults empty prefix to "".
• GCS.list_prefix/list_all_blobs now construct entries via from_full_prefix.
• Downloader improvements: caches blob listing, sets ProgressBar.total from worker thread via call_from_thread, performs all UI updates thread-safely, uses daemon thread, and preserves directory structure when downloading by computing relative paths from base prefix; cooperative stop on unmount.
• UI/logic tweaks: safer selection handling in FileListView, uses from_full_prefix when navigating, more robust BadRequest parsing, corrected confirmation text; gcp_project typed as Optional[str].

^{Written by Cursor Bugbot for commit c57babe. This will update automatically on new commits. Configure here.}

[cursor]

Cursor Agent can help with this pull request. Just @cursor in comments and I'll start working on changes in this branch.
_{Learn more about Cursor Agents}

[cursor]

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.