Skip to content

Conversation

@yoava7
Copy link

@yoava7 yoava7 commented Jan 7, 2026

With the addition of the name validation in src/agent/connection_handler.js, the andy-4 and andy-4-reasoning profiles now have an invalid name due to using a hyphen.

I made the minimal change to change their name field to use underscores instead, like what is used in claude_thinker, this lets the bots connect to a server and function.
I wasn't sure whether it would be better to also change the filename of the profile, as the model itself has a hyphen-- I'll leave that up to you :D

Note: andy-4-reasoning does not currently have a commented out profile in the profiles list in settings.js, that might be something to consider adding. The filename for it is also andy-4-reasoning, but the name is andy-4-thinking.

@yoava7 yoava7 marked this pull request as ready for review January 7, 2026 20:35
@Sweaterdog
Copy link
Contributor

No? None of the things you mentioned are broken. The profiles work as-is, I tested them when I made them to ensure Andy-4-reasoning.json actually made the model think.

@yoava7
Copy link
Author

yoava7 commented Jan 8, 2026

Are you sure the profiles still work currently? Maybe I misconfigured something but whenever I enable one of those profiles I get the message [LoginGuard] Invalid name '${name}'. Must be 3-16 alphanumeric/underscore characters. (with '${name}' being the name of the profile).

Looking at the git history the profiles were made 5 months ago, and the connection handler was created last month.
The profiles probably worked until the addition of the name format check inside the connection handler, though maybe they still work and my setup is messed up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants