fix(providers): promote Oracle Cloud and Nextcloud to stable, show Blomp in the production grid - #577
Open
axpnet wants to merge 3 commits into
Open
fix(providers): promote Oracle Cloud and Nextcloud to stable, show Blomp in the production grid#577axpnet wants to merge 3 commits into
axpnet wants to merge 3 commits into
Conversation
Oracle Cloud shipped with the v1.5.0 provider batch and kept `stable: false` ever since, which renders the red "Experimental provider" banner in Quick Connect and tells the user the service may be unavailable. The flag means "not fully tested" (`ProviderConfig.stable`), and nobody had tested it: when Alibaba OSS and Tencent COS were promoted in 5272f99, Oracle was left behind rather than found broken. Verified against a live OCI account (eu-milan-1, S3 compatibility endpoint): connect 398ms, list, mkdir, stat, 64 KB upload with a matching remote sha256, byte-identical download roundtrip, rename, server-side copy, 32 MB multipart upload with a matching sha256, recursive size scan, delete and recursive delete. The bucket was left exactly as found. The one unsupported call is `df` (`storage_info`), which is normal for S3 backends with no quota API and is what the manual "Total storage" field exists for. It is not a reason to warn the user about the service. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Blomp was promoted to production on 2026-07-28 (798811b): the registry carries `stable: true` and a comment recording the live verification, that the account listing 403 is by design, and that swift.rs falls back to the per-account container. The protocol grid never followed. It still wrapped the Blomp entry in `import.meta.env.DEV`, under a comment claiming Blomp is "hidden in production until storage proxy 403 is resolved (stable: false in registry)", which the registry has contradicted since that release. The result was a provider shipped as production in one surface and dev-only in another. Removes the DEV gate and replaces the stale comment with what the registry actually records. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
✅ Snyk checks have passed. No issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
📝 WalkthroughWalkthroughBlomp is now visible in production protocol selection. Oracle Cloud is now marked as stable in the provider registry. ChangesProvider availability
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
Nextcloud carried `stable: false, // Not tested yet` since the entry was written, so Quick Connect warned users that Nextcloud "is not stable yet and the service may be unavailable" while ROADMAP.md lists it among the supported presets. The two statements could not both be true. Verified against a self-hosted Nextcloud behind nginx: connect in 934ms with the DAV endpoint auto-detected from the bare host, quota reported through the OCS API (unmetered total), list, mkdir, stat, 64 KB upload with a matching remote sha256, byte-identical download roundtrip, rename, server-side copy, 32 MB upload with a matching sha256, public share link creation, and recursive delete. The share link returned 404 after the delete, so nothing was left behind. Listing was cross-checked against a second, independent Nextcloud (Tab.digital) to separate server quirks from client behaviour. DigitalOcean Spaces and S3Drive keep the flag: they are untested, not known broken, and promoting them without a live run would repeat the mistake this series fixes. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Provider visibility flags had drifted from reality, and each drift was telling users something untrue.
Oracle Cloud showed the red "Experimental provider" banner, which says the service may be unavailable and that connections can fail through no fault of the user. The flag behind it,
ProviderConfig.stable, means "not fully tested". Oracle arrived with the v1.5.0 provider batch and was never tested: when Alibaba OSS and Tencent COS were promoted in 5272f99, Oracle was left behind rather than found broken.Nextcloud carried
stable: false, // Not tested yetsince the entry was written, so the GUI warned that Nextcloud "is not stable yet" whileROADMAP.mdlists it among the supported presets. Both could not be true.Blomp was promoted to production on 2026-07-28 in 798811b, with the registry recording
stable: trueand the live verification. The protocol grid never followed: it still wrapped Blomp inimport.meta.env.DEVunder a comment claiming it is "hidden in production until storage proxy 403 is resolved (stable: false in registry)", which the registry has contradicted since that release.Verification
Both promotions were tested against live accounts before the flag was flipped. Same battery, run end to end, with every test artifact removed afterwards.
df)s3-list-recursiveOracle's
dfreturnsOperation not supported: storage_info, which is normal for S3 backends with no quota API and is exactly what the manual "Total storage" field exists for. It is not a reason to warn a user about the service.Nextcloud listing was cross-checked against a second, independent Nextcloud (Tab.digital) to separate server quirks from client behaviour. The self-hosted box exposes a phantom
remote.phpcollection at every path depth, which is an nginx rewrite on that server: the second Nextcloud lists clean.Local gate on the branch:
tsc --noEmitclean,vitest run80 files and 718 tests passing, re-run after each commit.Still flagged
DigitalOcean Spaces and S3Drive keep
stable: false. They are untested, not known broken, and promoting them without a live run would repeat the mistake this series fixes.Not in this PR
The banner copy itself deserves a rewrite. It reads "{provider} is not stable yet and the service may be unavailable", which blames the third party, while the flag only records that we have not verified it. That is a copy change across every locale and belongs in its own PR.