Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a mechanism to prevent the resolution of component references when they appear within an action context in the A2uiMessageProcessor, including a new unit test for verification. Feedback suggests extending the isInsideAction guard to component array references to fully address potential data corruption, ensuring the flag propagates correctly for nested actions, and using double quotes for string literals to maintain style consistency.
zeroasterisk
left a comment
There was a problem hiding this comment.
Please do the isComponentArrayReference recommendation too
All done! This was taken care of in commit 0438e34. The |
Strings in action context that matched component IDs were incorrectly resolved to component nodes, causing data corruption (e.g.,
[object Object]in payloads).Fix
Added an
isInsideActionflag to skip component resolution when processing action contexts inmodel-processor.ts.Verification
model-processor.test.ts.Fixes #1155
Pre-launch Checklist