failed to update thread frames during python debugging

[tom-shan]

Description

When debugging python file, Call Stack will show the message of Load More Stack Frames.
it will show thread frames only after clicking this message.

Reproduction Steps

1 download vscode python plugin
2 start theia
3 create a simple python file and add a breakpoint
4 debug this python file and click Step Over

OS and Theia version:
OS: centos 7
Theia: 6d0abe5
Diagnostics:
a regression caused by d1678ad

[tom-shan]

I think we should put d1678ad#diff-bb75a296f1f7fdc09c2d755ef791d61cR229 in the branch of if (thread.stoppedDetails && thread.stoppedDetails.allThreadsStopped) accoring to it's commit message.

[akosyakov]

@marcdumais-work @marechal-p Could you have a look please? It seems to be caused by the recently merged PR or revert it I would prefer to step back and make a minimal change which does not break it.

[marcdumais-work]

Reproduced on master. Will now confirm if same happens or not before d1678ad

[marcdumais-work]

Will now confirm if same happens or not before d1678ad

Confirmed - testing one commit before d1678ad, the call stack frames are displayed after stepping, without having to click on "load more stack frames".

I will talk with @marechal-p before reverting, but this looks like we will likely proceed doing this later today.

[akosyakov]

@marcdumais-work Can we have a PR to revert d1678ad please? It breaks products which use next version. A proper PR can be reintroduced later. Let me know if you don't have time I will revert it. btw was it a part of last release? We need a bug fix release probably then as well.

[marcdumais-work]

Can we have a PR to revert d1678ad please?

Yes, we'll either have a fix or revert that commit today.

[marcdumais-work]

btw was it a part of last release?

Indeed it was.

We need a bug fix release probably then as well.

+1

[paul-marechal]

From my investigation, it seems that for some reason, the DebugStackFramesSource.getElements function is not called on stop, when replicating the situation mentioned by this issue.

theia/packages/debug/src/browser/view/debug-stack-frames-source.tsx

Line 44 in 357b8e9

*getElements(): IterableIterator<TreeElement> {

I wasn't able to make sense of what doesn't happen/what would need to happen to fix that, and I need to focus on other tasks for now.

[marcdumais-work]

I wasn't able to make sense of what doesn't happen/what would need to happen to fix that, and I need to focus on other tasks for now.

Thanks for the investigation @marechal-p - We'll revert the PR for now and do a bugfix release.

[marcdumais-work]

The original PR is reverted - we'll make sure this issue here is no longer present if we bring it back.