Skip to content

fix: propagate /undo reply to the model - #5881

Open
yushiwuzhengjianyuzi wants to merge 1 commit into
Aider-AI:mainfrom
yushiwuzhengjianyuzi:fix/undo-command-reply
Open

yushiwuzhengjianyuzi wants to merge 1 commit into
Aider-AI:mainfrom
yushiwuzhengjianyuzi:fix/undo-command-reply

Conversation

@yushiwuzhengjianyuzi

Copy link
Copy Markdown

Fixes #5874

Summary

  • Preserve the return value from raw_cmd_undo() so /undo sends the configured acknowledgement to the model.
  • Add a regression test covering successful undo with send_undo_reply=True.

Testing

  • pytest tests/basic/test_commands.py -q -k "cmd_undo"
  • pre-commit run --files aider/commands.py tests/basic/test_commands.py

AI assistance: Codex was used to implement and test this change. Human review is requested.

@CLAassistant

CLAassistant commented Oct 19, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

This branch has not been deployed

No deployments
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.

BUG: /undo never sends undo_command_reply to the model because cmd_undo discards raw_cmd_undo's return value

2 participants