@guw thanks for your pr. I have looked into it and here is some feedback:

• I do not see what is the benefit of introducing a global variable this.expandedElements in your code, you can do that all with a local
• The only functionaly difference compared to the original code is that the if (stackFrame) is moved a couple of lines up, which looks great to me
• Please format the code as the indentation is completly broken
• Does this actually fix the issue on your side? If yes I would gladly merge it in once you address my feedback from above.

Thanks again

@isidorn Thanks for the feedback. I think there is an issue with the global variable. I'll revisit the logic.

Re: formatting - Is there a trick? I do get a warning from the pre-commit hook. However, I was expecting VS Code to format on save. Also the format action seems to screw up the whole file.

@guw yes please revisit the logic. In the latest I see that you are still using the global variable.
For formating, I just do right click in editor -> format document. Please try that

@isidorn The global variable is required because we need to persist the list of expanded elements between different executions of the function. It's implementing what you proposed in #16031.

@guw ok, makes sense.
Can you please elaborote on my other questions from above:

• Please format the code as the indentation is completly broken
• Does this actually fix the issue on your side? If yes I would gladly merge it in once you address my feedback from above.
  and
• Is the check on line 82 really necessery? I would assume not

@isidorn Do you have any suggestions on formatting? Whenever I try to format the document using VS Code it makes a ton of changes.

@guw I will format. Can you please ansewer my questions from above.

@isidorn patch rebased, simplified and formatted.

• Confirmed that it fixes the issue
• check on line 82 is gone

@guw looks good to me, thanks a lot for your contribution 🍻