Skip to content

append generate_reply instructions as system msg and convert it to user msg if unsupported#5287

Open
longcw wants to merge 4 commits intomainfrom
longc/convert-mid-conversation-instructions
Open

append generate_reply instructions as system msg and convert it to user msg if unsupported#5287
longcw wants to merge 4 commits intomainfrom
longc/convert-mid-conversation-instructions

Conversation

@longcw
Copy link
Copy Markdown
Contributor

@longcw longcw commented Mar 31, 2026

  • generate_reply now appends turn instructions as a system message
  • Add convert_mid_conversation_instructions in provider format utils — converts mid-conversation system messages to user messages with <instructions> tags for Anthropic, Google, and AWS. OpenAI/Mistral unchanged (native support).

@chenghao-mou chenghao-mou requested a review from a team March 31, 2026 09:09
@longcw longcw changed the title fix(llm): append instructions as system msg, convert mid-conversation ones for unsupported LLMs fix(llm): append generate_reply instructions as system message and convert it to user role for unsupported LLM Mar 31, 2026
devin-ai-integration[bot]

This comment was marked as resolved.

@longcw longcw changed the title fix(llm): append generate_reply instructions as system message and convert it to user role for unsupported LLM append generate_reply instructions as system msg and convert it to user msg with <instructions> if unsupported Mar 31, 2026
@longcw longcw changed the title append generate_reply instructions as system msg and convert it to user msg with <instructions> if unsupported append generate_reply instructions as system msg and convert it to user msg if unsupported Mar 31, 2026
devin-ai-integration[bot]

This comment was marked as resolved.

Copy link
Copy Markdown
Member

@chenghao-mou chenghao-mou left a comment

Choose a reason for hiding this comment

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

lgtm

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