Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Jun 29, 2025

Per Google's request for a more standard implementation


Important

Remove Gemini CLI provider and related configurations, components, and translations from the codebase.

  • Provider Removal:
    • Remove gemini-cli from provider-settings.ts, index.ts, and api/index.ts.
    • Delete gemini-cli.ts and its test gemini-cli.spec.ts.
  • Schema and Configurations:
    • Remove geminiCliSchema from provider-settings.ts.
    • Remove GeminiCliHandler from api/index.ts and providers/index.ts.
    • Update checkExistApiConfig.ts to remove gemini-cli special case.
  • UI Components:
    • Remove GeminiCli component and its test from ApiOptions.tsx and providers/index.ts.
    • Update constants.ts and useSelectedModel.ts to remove gemini-cli references.
  • Translations:
    • Remove geminiCli related translations from all locale files.
  • Validation:
    • Update validate.ts to remove gemini-cli validation logic.

This description was created by Ellipsis for 3541b71. You can customize this summary. It will automatically update as commits are pushed.

@mrubens mrubens requested review from cte and jr as code owners June 29, 2025 03:37
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Jun 29, 2025
@delve-auditor
Copy link

delve-auditor bot commented Jun 29, 2025

No security or compliance issues detected. Reviewed everything up to 3541b71.

Security Overview
  • 🔎 Scanned files: 51 changed file(s)
Detected Code Changes
Change Type Relevant files
Refactor ► provider-settings.ts
    Remove Gemini CLI Provider
► providers/gemini-cli.ts
    Delete Gemini CLI Provider Implementation
► providers/gemini-cli.spec.ts
    Delete Gemini CLI Provider Tests
► ApiOptions.tsx
    Remove Gemini CLI Options
► i18n locales
    Remove Gemini CLI Translations

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 29, 2025
@mrubens mrubens merged commit d64e677 into main Jun 29, 2025
20 checks passed
@mrubens mrubens deleted the remove_gemini_cli branch June 29, 2025 03:48
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 29, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 29, 2025
baobinhday pushed a commit to baobinhday/Roo-Code that referenced this pull request Jul 2, 2025
hannesrudolph pushed a commit that referenced this pull request Jul 3, 2025
utarn pushed a commit to modelharbor/ModelHarbor-Agent that referenced this pull request Jul 4, 2025
snoyiatk pushed a commit to snoyiatk/Roo-Code that referenced this pull request Jul 9, 2025
MichaelYochpaz added a commit to MichaelYochpaz/Roo-Code that referenced this pull request Jul 12, 2025
This reverts commit 35b0f55.

# Conflicts:
#	src/i18n/locales/ca/common.json
#	src/i18n/locales/zh-TW/common.json
#	webview-ui/src/i18n/locales/ca/settings.json
#	webview-ui/src/i18n/locales/de/settings.json
#	webview-ui/src/i18n/locales/en/settings.json
#	webview-ui/src/i18n/locales/es/settings.json
#	webview-ui/src/i18n/locales/fr/settings.json
#	webview-ui/src/i18n/locales/hi/settings.json
#	webview-ui/src/i18n/locales/id/settings.json
#	webview-ui/src/i18n/locales/it/settings.json
#	webview-ui/src/i18n/locales/ja/settings.json
#	webview-ui/src/i18n/locales/ko/settings.json
#	webview-ui/src/i18n/locales/nl/settings.json
#	webview-ui/src/i18n/locales/pl/settings.json
#	webview-ui/src/i18n/locales/pt-BR/settings.json
#	webview-ui/src/i18n/locales/ru/settings.json
#	webview-ui/src/i18n/locales/tr/settings.json
#	webview-ui/src/i18n/locales/vi/settings.json
#	webview-ui/src/i18n/locales/zh-CN/settings.json
#	webview-ui/src/i18n/locales/zh-TW/settings.json
p30arena added a commit to p30arena/Roo-Code that referenced this pull request Aug 1, 2025
MichaelYochpaz added a commit to MichaelYochpaz/Roo-Code that referenced this pull request Aug 7, 2025
Yandrik added a commit to Yandrik/Roo-Code that referenced this pull request Aug 7, 2025
MichaelYochpaz added a commit to MichaelYochpaz/Roo-Code that referenced this pull request Aug 8, 2025
Yandrik added a commit to Yandrik/Roo-Code that referenced this pull request Sep 18, 2025
MichaelYochpaz added a commit to MichaelYochpaz/Roo-Code that referenced this pull request Oct 3, 2025
youngzyl pushed a commit to youngzyl/Roo-Code that referenced this pull request Oct 25, 2025
baobinhday pushed a commit to baobinhday/Roo-Code that referenced this pull request Nov 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants