Commit ee55a07
fix: preserve tool approval signatures when approvals are responded (#16589)
## Background
Tool approvals signed via experimental_toolApprovalSecret lost their
signature when moving from approval-requested to approval-responded,
causing resumed requests to fail signature validation with a missing
signature error.
## Summary
Updated manual chat approval responses and streamed tool approval
responses to carry forward an existing approval.signature while
preserving existing approval metadata.
## Testing
Added regression coverage for manual addToolApprovalResponse and
streamed tool-approval-response transitions to assert approval
signatures remain on the responded approval state.
## Related Issues
Fixes #16543
---------
Co-authored-by: dnukumamras <mukund.sarma@vercel.com>1 parent 5f0d553 commit ee55a07
3 files changed
Lines changed: 74 additions & 0 deletions
File tree
- .changeset
- packages/ai/src/ui
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
7698 | 7698 | | |
7699 | 7699 | | |
7700 | 7700 | | |
| 7701 | + | |
| 7702 | + | |
| 7703 | + | |
| 7704 | + | |
| 7705 | + | |
| 7706 | + | |
| 7707 | + | |
| 7708 | + | |
| 7709 | + | |
| 7710 | + | |
| 7711 | + | |
| 7712 | + | |
| 7713 | + | |
| 7714 | + | |
| 7715 | + | |
| 7716 | + | |
| 7717 | + | |
| 7718 | + | |
| 7719 | + | |
| 7720 | + | |
| 7721 | + | |
| 7722 | + | |
| 7723 | + | |
| 7724 | + | |
| 7725 | + | |
| 7726 | + | |
| 7727 | + | |
| 7728 | + | |
| 7729 | + | |
| 7730 | + | |
| 7731 | + | |
| 7732 | + | |
| 7733 | + | |
| 7734 | + | |
| 7735 | + | |
| 7736 | + | |
| 7737 | + | |
| 7738 | + | |
| 7739 | + | |
| 7740 | + | |
| 7741 | + | |
| 7742 | + | |
| 7743 | + | |
| 7744 | + | |
| 7745 | + | |
| 7746 | + | |
| 7747 | + | |
| 7748 | + | |
| 7749 | + | |
| 7750 | + | |
| 7751 | + | |
| 7752 | + | |
| 7753 | + | |
| 7754 | + | |
| 7755 | + | |
| 7756 | + | |
| 7757 | + | |
| 7758 | + | |
| 7759 | + | |
| 7760 | + | |
| 7761 | + | |
| 7762 | + | |
| 7763 | + | |
| 7764 | + | |
| 7765 | + | |
| 7766 | + | |
7701 | 7767 | | |
7702 | 7768 | | |
7703 | 7769 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
731 | 731 | | |
732 | 732 | | |
733 | 733 | | |
| 734 | + | |
| 735 | + | |
| 736 | + | |
734 | 737 | | |
735 | 738 | | |
736 | 739 | | |
| |||
0 commit comments