Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Jul 11, 2025

Related GitHub Issue

Closes: #

Roo Code Task Context (Optional)

Description

Test Procedure

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Testing: New and/or updated tests have been added to cover my changes (if applicable).
  • Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Screenshots / Videos

Documentation Updates

Additional Notes

Get in Touch


Important

Improve local evals by updating logging paths, documentation, and configuration settings.

  • Behavior:
    • Change logging directory to /tmp/evals/runs for non-Docker environments in runEvals.ts and runTask.ts.
    • Remove newTab option from runTask() in runTask.ts.
  • Documentation:
    • Update README.md to replace Docker command with pnpm evals for starting the evals service.
    • Remove alt text from images in README.md.
  • Configuration:
    • Remove commented expose lines from docker-compose.yml for db and redis services.
  • Settings:
    • Update lastShownAnnouncementId in global-settings.ts to "jul-09-2025-3-23-0".

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

@cte cte requested review from jr and mrubens as code owners July 11, 2025 19:08
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Jul 11, 2025
@delve-auditor
Copy link

delve-auditor bot commented Jul 11, 2025

No security or compliance issues detected. Reviewed everything up to 5cacfa3.

Security Overview
  • 🔎 Scanned files: 5 changed file(s)
Detected Code Changes
Change Type Relevant files
Enhancement ► README.md
    Update installation and usage instructions
► docker-compose.yml
    Remove commented out port exposures
► runEvals.ts, runTask.ts
    Add non-containerized log path support
► McpView.tsx, ModesView.tsx
    Update UI layout and remove marketplace buttons
► global-settings.ts
    Update announcement ID
► mistral.ts
    Remove devstral-medium-latest model
► Multiple locale files
    Remove marketplace related 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.

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@cte cte merged commit 99448fc into main Jul 11, 2025
10 checks passed
@cte cte deleted the cte/evals-fixes branch July 11, 2025 19:15
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jul 11, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Jul 11, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jul 11, 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:M This PR changes 30-99 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants