Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Comma in terminal ignored #1177

Open
wwwouter opened this issue Feb 25, 2025 · 3 comments
Open

Comma in terminal ignored #1177

wwwouter opened this issue Feb 25, 2025 · 3 comments
Labels
bug Something isn't working

Comments

@wwwouter
Copy link

Which version of the app are you using?

v3.7.2

Which API Provider are you using?

Anthropic

Which Model are you using?

Claude 3.7 Sonnet

What happened?

The terminal outputs Worker.cs(53,27): error CS8600: Converting null literal or possible null value to non-nullable type. But it is being interperted as 5327

We now know the exact line number with the issue: line 5327 in `Worker.cs`. Let's look at that specific line:

Let's look at the specific line 5327 in `Worker.cs`:

Steps to reproduce

  1. Problem with line number in it
  2. Trying to fix it

Relevant API REQUEST output

[execute_command for 'dotnet test ...'] Result:

Command executed.
Output:
....

Worker.cs(5327): error CS8600: Converting null literal or possible null value to non-nullable type.
...

Build failed with 1 error(s) in 5.6s

<environment_details>
# VSCode Visible Files
Worker.cs

# VSCode Open Tabs
 
# Current Time
25/02/2025, 8:51:03 am (Europe/Amsterdam, UTC+1:00)

# Current Context Size (Tokens)
62,143 (31%)

# Current Mode
<slug>code</slug>
<name>Code</name>
<role>You are Roo, a highly skilled software engineer with extensive knowledge in many programming languages, frameworks, design patterns, and best practices.</role>
<custom_instructions>
====

USER'S CUSTOM INSTRUCTIONS

The following additional instructions are provided by the user, and should be followed to the best of your ability without interfering with the TOOL USE guidelines.

Language Preference:
You should always speak and think in the English language.</custom_instructions>
</environment_details>

Additional context

No response

@wwwouter wwwouter added the bug Something isn't working label Feb 25, 2025
@mrubens
Copy link
Collaborator

mrubens commented Feb 25, 2025

This bug drives me nuts! Seems like it must be somewhere in the code to parse the terminal output...

@wwwouter
Copy link
Author

wwwouter commented Feb 25, 2025

It could be a locale issue. # Current Context Size (Tokens) 62,143 (31%)

➜  ~ locale                      
LANG=""
LC_COLLATE="C"
LC_CTYPE="UTF-8"
LC_MESSAGES="C"
LC_MONETARY="C"
LC_NUMERIC="C"
LC_TIME="C"
LC_ALL=
➜  ~ defaults read -g AppleLocale
en_NL

@mrubens
Copy link
Collaborator

mrubens commented Feb 25, 2025

Happens to me in en-us too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants