Skip to content

Conversation

@roomote
Copy link
Collaborator

@roomote roomote commented Jun 30, 2025

  • Removed context token information (token count and percentage) from environment details sent to AI models
  • Models were inappropriately modifying their behavior based on perceived token constraints
  • Preserved cost information which is still useful for user-facing displays
  • Updated corresponding test to reflect the change

Important

Remove context token information from getEnvironmentDetails() to prevent inappropriate model behavior modifications.

  • Behavior:
    • Removed context token information (token count and percentage) from getEnvironmentDetails() in getEnvironmentDetails.ts.
    • Preserved cost information for user-facing displays.
  • Tests:
    • Updated getEnvironmentDetails.spec.ts to remove checks for context token information.

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

… details

- Removed context token information (token count and percentage) from environment details sent to AI models
- Models were inappropriately modifying their behavior based on perceived token constraints
- Preserved cost information which is still useful for user-facing displays
- Updated corresponding test to reflect the change
@roomote roomote requested review from cte, jr and mrubens as code owners June 30, 2025 08:27
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working labels Jun 30, 2025
@delve-auditor
Copy link

delve-auditor bot commented Jun 30, 2025

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

Security Overview
  • 🔎 Scanned files: 2 changed file(s)
Detected Code Changes

The diff is too large to display a summary of code changes.

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 30, 2025
@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

bug Something isn't working Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants