Editors whose tab was not active on first render never landed in edit… - #603
Editors whose tab was not active on first render never landed in edit…#603GabrielMalava wants to merge 10 commits into
Conversation
…orsRef, so Run fell back to executing the entire script instead of the statement under the cursor.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (1 file)
Reviewed by step-3.7-flash · Input: 90.2K · Output: 27.4K · Cached: 1.1M |
|
I reviewed this locally and the fix looks correct. The root cause checks out: Local verification passed:
My only non-blocking note is that the PR does not add a regression test for the exact flow it fixes: a tab that is inactive on initial render, then becomes active, followed by Run executing the statement under the cursor. Adding coverage for that interaction would make the fix safer against future regressions. Looks good to merge from my side. |
…orsRef, so Run fell back to executing the entire script instead of the statement under the cursor.