Fix summarization progress message breaking list rendering

[roblourens]

Fix #249148

[roblourens]

@jrieken from #249105. Thanks for cleaning up this code assigning handles for the progress task. Real culprit is this line: : revive(progress) as IChatProgress; which is a bit weird because IChatProgressDto is not actually compatible with IChatProgress. Not sure what the proper fix is, progress task is an oddball.

We've had many issues with serializing/deserializing all these different types in different scenarios and really I need to find the right pattern for this and eliminate any special cases.

[jrieken]

@roblourens Thanks for the fix and sorry for the breakage. I remember thinking really hard about the code and still came to the wrong conclusion :-)

Yeah, the handles are now assigned on the EH side so that chat progress can be send in bulks. That was needed for the new/better editing UX