TUI: show full patch diff in approval modal

[vmotta8]

Description

• Addresses illegible diff in apply_patch approval flow, relates to apply_patch command generates requests with illegible diff #2005
• Onboarding option “No, ask me to approve” maps to UnlessTrusted + ReadOnly (ensures AskUser)

Screenshots:

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[vmotta8]

I have read the CLA Document and I hereby sign the CLA

[overthetop]

@vmotta8 is there a way to show the diff even when there is auto approve? I just want to follow the edits and stop the model in case it goes sideways.

[nestedlogicsol]

Possibly related? #2049

[vmotta8]

@nestedlogicsol oh, you're right, and this pr does almost the same thing: #2050
@overthetop this pr will address your point as well
it will probably be merged soon

[nestedlogicsol]

I don't see any evidence of the approval dialogue. How did you get it to work in your environment?

[vmotta8]

@nestedlogicsol You can build a release binary using cargo build --release --bin codex inside codex-rs and then run the executable from any directory
The binary will be generated in target/release/codex

[nestedlogicsol]

@vmotta8 sorry I was unclear - I meant that I didn't see evidence of fixing the missing "proposed patch" functionality in #2049 & #2050 that seems to be present in your PR.

They've merged 2049 and 2050 into main, I bet you can use their new diff implementation combined with your "proposed patch" fix (maybe your changes to trust_directory.rs?) to cover all the issues we've talked about so far.

[vmotta8]

@nestedlogicsol Sure! I’ve updated it to keep the new diff implementation from main, and now it only contains the fix for the missing “proposed patch” functionality.