Bug: insert_content passes isWriteProtected as the partial flag, causing ‘Current ask promise was ignored’

[hannesrudolph]

Deeper investigation reveals the root cause is in , not .

Summary

• The error “Current ask promise was ignored (Add options to always approve write and execute operations #1)” is thrown by when it receives a partial/streaming update for an ask that wasn't initiated as partial. See .
• incorrectly calls . The third argument to is the flag.
• When is (as it is for paths under ), the call is treated as a partial update, which is not expected, causing the promise to be ignored and the error to be thrown.

Problematic Code
The issue is here:

Suggested Fix

• The wrapper in should be used, as it correctly handles the flag as the 5th argument.
• The calls in and should be changed to use the function, similar to how does it.
• Example correct implementation from :

This will ensure the boolean is not misinterpreted as the flag.

[roomote]

I'm exploring a potential fix for this issue where insert_content and search_and_replace incorrectly pass isWriteProtected as the partial flag to task.ask(). Will open a PR to share progress.

[roomote]

I've opened PR #7649 with a fix for this issue: #7649

The fix replaces the direct task.ask() calls in both insertContentTool.ts and searchAndReplaceTool.ts with the askApproval wrapper function, which correctly handles the parameter ordering. This ensures that isWriteProtected is passed as the 4th parameter rather than being misinterpreted as the partial flag.

All tests are passing locally and CI checks are currently running. Feedback welcome!