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

Bug fix "Bots are entirely incapable of giving themselves items." #415

Closed
wants to merge 3 commits into from

Conversation

BF5258
Copy link

@BF5258 BF5258 commented Jan 21, 2025

The code that translates messages prevents the LLM from running commands.
https://discord.com/channels/1303399789995626667/1320514453632061510/1320514453632061510

This change runs any commands in the message before it is translated.

Copy link
Collaborator

@MaxRobinsonTheGreat MaxRobinsonTheGreat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this supposed to let it run many console commands in one message? bit weird, since you can't do that normally. instead can you just check if the message starts with /. that would be more readable

@BF5258
Copy link
Author

BF5258 commented Jan 22, 2025

That was my intention; otherwise, it must give itself tools and armor one at time.

With multiple commands per message:

image

Without:

image
Would it be better to send the LLM a response message instead, similar to the ! actions? That way it can continue writing while also receiving feedback about if and why the command failed.

@BF5258 BF5258 closed this Jan 31, 2025
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