carloea2 commented on PR #8637:
URL: https://github.com/apache/texera/pull/8637#issuecomment-6179513719

   Could you check the retry path after an Apply failure? At `e7c0d7b`, the 
panel keeps the suggestion visible in `apply_failed` and offers “Apply and 
re-run,” but `applyFix()` returns unless the status is `ready`.
   
   I reproduced this with the actual service code and mocked dependencies: 
analyze a code fix, change the target snippet so Apply fails, restore the 
original snippet, then click Apply again. The second call silently returns 
without writing or rerunning.
   
   Could you either accept `apply_failed` for a retry while rerunning all 
validation, or provide an explicit re-analysis action when the suggestion is 
stale? It would help to add a failure → correction → successful retry test so 
the button matches the behavior offered by the panel.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to