Skip to content

Conversation

@jr
Copy link
Collaborator

@jr jr commented Jul 6, 2025

Use Base64 encoded JSON in the ROO_CODE_CLOUD_ORG_SETTINGS environment variable.


Important

Adds support for static cloud settings in CloudService using StaticSettingsService and environment variable ROO_CODE_CLOUD_ORG_SETTINGS.

  • Behavior:
    • CloudService now supports static cloud settings via ROO_CODE_CLOUD_ORG_SETTINGS environment variable.
    • Chooses StaticSettingsService if ROO_CODE_CLOUD_ORG_SETTINGS is set, otherwise uses CloudSettingsService.
    • StaticSettingsService parses Base64 encoded JSON for settings.
  • Services:
    • Adds StaticSettingsService to handle static settings.
    • Modifies CloudService to initialize StaticSettingsService or CloudSettingsService based on environment variable.
  • Tests:
    • Adds CloudService.integration.test.ts to test service selection logic.
    • Adds StaticSettingsService.test.ts to test parsing and error handling of static settings.
    • Updates CloudService.test.ts to mock CloudSettingsService and test new behavior.

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

Use Base64 encoded JSON in the ROO_CODE_CLOUD_ORG_SETTINGS
environment variable.
@jr jr requested a review from mrubens as a code owner July 6, 2025 15:59
@jr jr requested a review from cte as a code owner July 6, 2025 15:59
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. enhancement New feature or request labels Jul 6, 2025
@delve-auditor
Copy link

delve-auditor bot commented Jul 6, 2025

No security or compliance issues detected. Reviewed everything up to d244171.

Security Overview
  • 🔎 Scanned files: 8 changed file(s)
Detected Code Changes
Change Type Relevant files
Refactor ► CloudService.ts
    Add support for static settings configuration
► CloudSettingsService.ts
    Implement cloud-based settings service
► SettingsService.ts
    Convert to interface definition
► StaticSettingsService.ts
    Implement static settings service
► CloudService.integration.test.ts
    Add integration tests for settings services
► StaticSettingsService.test.ts
    Add unit tests for static settings service
► CloudService.test.ts
    Update tests for new settings architecture

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 Jul 6, 2025
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Jul 7, 2025
@mrubens mrubens merged commit ad201cc into main Jul 7, 2025
14 checks passed
@mrubens mrubens deleted the jr/cloud-static-settings branch July 7, 2025 03:21
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jul 7, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jul 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. lgtm This PR has been approved by a maintainer size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants