Skip to content

[OMEGA-244] Change OpenRouter prompt to the default we use for not so chatty and spammy (it talks too much now)#244

Merged
TossSky merged 2 commits into
asi-alliance:mainfrom
jazzbox35:prompt_fix
Jul 13, 2026
Merged

[OMEGA-244] Change OpenRouter prompt to the default we use for not so chatty and spammy (it talks too much now)#244
TossSky merged 2 commits into
asi-alliance:mainfrom
jazzbox35:prompt_fix

Conversation

@jazzbox35

Copy link
Copy Markdown
Collaborator

Description

The OpenRouter option using GLM overwhelms the user with unnecessary questions. Switch to using the restrained prompt.

How Has This Been Tested?

I tested this with a few questions and it worked.

Checklist

  • [N/A ] The code generated by LLM is reviewed by the PR creator
  • [ X] Self-review completed
  • [ X] Test scenarios above are passed with the version of the code from PR

@jazzbox35
jazzbox35 requested review from TossSky, anseliv and vsbogd July 11, 2026 18:08

@vsbogd vsbogd left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks Mike, looks good to me. Prompt is equal to the prompt_ASICloud.txt but technically the only way to add it is to copy it.

I think we need to test it accurately. So I will raise the task and link it with PR.

@vsbogd vsbogd changed the title Change OpenRouter prompt to the default we use for not so chatty and spammy (it talks too much now) [OMEGA-244] Change OpenRouter prompt to the default we use for not so chatty and spammy (it talks too much now) Jul 11, 2026

@TossSky TossSky left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Tested #244 on live GLM via OpenRouter and the chattiness is gone. Without the prompt GLM answered one request with 5 messages (1 answer plus 4 idle "ready/awaiting"), with the prompt just the answer. I also checked that the anti-spam lines actually reach the model in its prompt, not just went by the reply count.

The prompt is picked by provider name, so it covers provider=OpenRouter, the GLM case in the title. Good to merge for that.

One gap outside this task: when we build with provider=MiniMaxM3, that runs the minimax model through OpenRouter, and it has no prompt file of its own, so it falls back to the default and still spams. The same minimax model over ASICloud is already fine, so covering MiniMaxM3 is just one more copy of that file, better as its own ticket.

@TossSky
TossSky merged commit d889d92 into asi-alliance:main Jul 13, 2026
2 checks passed
@vsbogd

vsbogd commented Jul 13, 2026

Copy link
Copy Markdown
Collaborator

One gap outside this task: when we build with provider=MiniMaxM3, that runs the minimax model through OpenRouter, and it has no prompt file of its own, so it falls back to the default and still spams.

This is fixed in #222 where user is able to specify provider and model separately. MiniMaxM3 provider is removed from the list of providers.

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.

3 participants