fix: preserve URL query parameters in storage flash for signed URLs

[ambient-code]

Summary

• Fix operator_for_path() to preserve URL query parameters (e.g. ?Expires=...&Signature=...&Key-Pair-Id=...) when creating OpenDAL HTTP operators, so signed URLs are fetched correctly
• Fix bearer token path in flash() to similarly preserve query params
• Update _filename(), _get_decompression_command(), and _transfer_bg_thread() to properly strip query parameters when extracting filenames for storage and compression detection

Fixes #271

Test plan

• Added unit tests for operator_for_path with query params
• Added unit tests for _filename with query params
• Added unit tests for _get_decompression_command with query params
• Added integration test for signed URL flash flow
• All existing tests pass (44 flashers tests, 31 opendal tests)

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8f2ef871-c132-4adb-b41d-61c8e1741151

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/271-complex-url-handling

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[netlify]

✅ Deploy Preview for jumpstarter-docs ready!

Name	Link
🔨 Latest commit	0e5b701
🔍 Latest deploy log	https://app.netlify.com/projects/jumpstarter-docs/deploys/69d6848081369f00081ceb9f
😎 Deploy Preview	https://deploy-preview-435--jumpstarter-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[mangelajo]

@ambient-code CI is failing here, can you check?

[mangelajo]

@ambient-code handle comments

[ambient-code]

CI Fix Pushed

Root cause: The test_operator_for_path_preserves_query_params test in driver_test.py was failing on macOS because /tmp is a symlink to /private/tmp. The test asserted str(path) == "/tmp/image.raw.xz" but Path.resolve() (called inside operator_for_path) returns /private/tmp/image.raw.xz on macOS.

Fix (0e5b701): Changed the filesystem path assertion to compare against Path("/tmp/image.raw.xz").resolve() instead of the hardcoded string, so the test passes on both Linux and macOS.

Other failing jobs:

• pytest job: Not related to this PR — it's a gate job that unconditionally runs exit 1
• Other matrix jobs (ubuntu 3.11, ubuntu 3.12, macos 3.11, macos 3.12): Were cancelled after the macOS 3.13 failure, not independent failures
• ubuntu 3.13: Passed successfully

No unaddressed review comments — only bot comments from CodeRabbit and Netlify were present.