feat: remove Roo Code Cloud integration (Closes: #31) - #71
Merged
Conversation
The upstream Roo Code Cloud backend (app.roocode.com) has been permanently shut down (HTTP 410 Gone). This commit removes the entire packages/cloud/ module and all its dependencies across the codebase. - Delete packages/cloud/ directory (35 files, ~400 lines) - Remove CloudService initialization from extension activation - Remove all CloudService.instance.* calls from ClineProvider - Remove cloud auth message handlers (sign-in/sign-out/manual URL) - Remove cloud telemetry capture from Task.ts - Simplify MdmService compliance checks - Remove cloud fields from ExtensionState interface - Clean i18n locales of roo.signInUnavailable key - Update 12 test files, delete cloudAuth spec - Create ADR documenting the decision Co-authored-by: hanneke-de-vries <dhanneke204@gmail.com>
- Remove remaining sharingEnabled/publicSharingEnabled from ExtensionStateContextType - Remove organizationSettingsVersion from MarketplaceView - Prune stale eslint suppressions for deleted cloud code
- extension.ts: Replace cloudLogger (removed with cloud service) with inline lambda using outputChannel.appendLine for MdmService logger - extension.spec.ts: Remove obsolete globalThis.cloudLogger shim
…View - Remove regressed /auth/clerk/callback handler from handleUri.ts - Remove unused ExtensionStateContext import from MarketplaceView.tsx
…egression, and lint - Add missing 'sharingEnabled', 'publicSharingEnabled', 'cloudAuthSkipModel' properties to ExtensionState type (packages/types) - Add '/auth/clerk/callback' handler in handleUri to show deprecation message for legacy cloud auth (src/activate) - Update zoo-hero-dark.png screenshot snapshot to match current rendering (webview-ui) - Remove unused extensionState variable and related imports from MarketplaceView.tsx Co-authored-by: hanneke-de-vries <dhanneke204@gmail.com>
Remove suppression entry for __tests__/extension.spec.ts that no longer has any @typescript-eslint/no-explicit-any violations. Co-authored-by: hanneke-de-vries <dhanneke204@gmail.com>
- Fix MarketplaceView tests to match actual component behavior (context-based re-fetching was already removed) - Fix unused 'result' variable in BulkInstallModal.tsx (_result prefix) - Prune stale ESLint suppressions for extension.spec.ts Co-authored-by: hanneke-de-vries <dhanneke204@gmail.com>
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
|
|
||
| // Listen for bulk install results | ||
| useEffect(() => { | ||
| const handleMessage = (event: MessageEvent) => { |
Add English fallback translations for new bulkInstall marketplace keys to all 17 backend and 12 webview non-English locale files. Co-authored-by: hanneke-de-vries <dhanneke204@gmail.com>
- checkDiskSpace test now accounts for Windows skipping the df check - validateInstallPath test uses path.resolve/path.join for platform-agnostic paths Co-authored-by: hanneke-de-vries <dhanneke204@gmail.com>
Co-authored-by: hanneke-de-vries <dhanneke204@gmail.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
3 participants
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.
Summary
Removes the entire
packages/cloud/module and all its dependencies. The upstream Roo Code Cloud backend (app.roocode.com) has been permanently shut down (HTTP 410 Gone).Changes
src/docs/adr-cloud-removal.mdVerification
@roo-code/cloudimports remain in source codeCloudServicereferences remain in source codeCloses: #31