fix: instruct fix agent to write summaries in Simplified Chinese - #169
fix: instruct fix agent to write summaries in Simplified Chinese#169cwbcheng wants to merge 1 commit into
Conversation
61b363a to
0185074
Compare
|
Rebased onto Flagging a scope question before this merges. The branch previously sat on top of Applied to Mechanically it is ready. Whether it should merge upstream is a product call: if the goal is a fork customization it probably belongs in the fork, and if the goal is upstream language support it likely wants a config setting rather than a hardcoded prompt line. Verified locally: |
Problem
When PatchDeck resolves a review thread, the follow-up comment posted to GitHub mixes languages: the title line is Chinese (
已在提交 \abc1234` 中处理。) but the agent summary is English, because the fix-agent prompt (buildAgentFixPrompt`) never told the agent which language to use. Only the code-owner fallback prompt contained the Chinese-language instruction.Example posted on PR #434:
Fix
Add a language instruction to
buildAgentFixPromptso every summary block the fix agent emits (posted as follow-up comments on the PR) is written in Simplified Chinese, matching the existing Chinese templates and the code-owner fallback prompt.Tests
write every summary block in Simplified Chinese.node --import tsx --test server/babysitter.test.ts(110 tests pass),npm run checkpasses.