Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Jun 12, 2025

Description

  • Allow openRouterApiKey to be overridden by the uploaded Roo Code settings file.
  • Make experiment keys optional (older builds won't populate new settings, e.g. marketplace and multiFileApplyDiff)

Important

Allow openRouterApiKey override and make experiment keys optional, updating types and UI components accordingly.

  • Behavior:
    • Allow openRouterApiKey to be overridden by run.settings in runTask() in runTask.ts.
    • Make experiment keys optional in experimentsSchema in experiment.ts.
  • Types:
    • Change experiments type from Record<ExperimentId, boolean> to Experiments in ExtensionMessage.ts and experiments.ts.
  • UI:
    • Update ExperimentalSettings component in ExperimentalSettings.tsx to use Experiments type.

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

@cte cte requested review from jr and mrubens as code owners June 12, 2025 17:47
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jun 12, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jun 12, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 12, 2025
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Jun 12, 2025
@cte cte merged commit fe52c19 into main Jun 12, 2025
11 of 12 checks passed
@cte cte deleted the cte/evals-fixes branch June 12, 2025 20:02
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jun 12, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 12, 2025
cte added a commit that referenced this pull request Jun 24, 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. lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants