Funky event order (active before start)

[jrieken]

testing #31475

Have an extension with listener as shown below. Debug a node.js app that just interval's forever and notice the following order of events. Weird is that CUSTOM EVENT comes first (tho not always) and that CHANGE ACTIVE comes before START

CUSTOM EVENT a07ab282-ca16-437c-be61-8637ecc0fb6d scriptLoaded
CUSTOM EVENT a07ab282-ca16-437c-be61-8637ecc0fb6d scriptLoaded
CHANGE ACTIVE a07ab282-ca16-437c-be61-8637ecc0fb6d
START a07ab282-ca16-437c-be61-8637ecc0fb6d
CUSTOM EVENT a07ab282-ca16-437c-be61-8637ecc0fb6d scriptLoaded
CUSTOM EVENT a07ab282-ca16-437c-be61-8637ecc0fb6d scriptLoaded
CUSTOM EVENT a07ab282-ca16-437c-be61-8637ecc0fb6d scriptLoaded
STOP a07ab282-ca16-437c-be61-8637ecc0fb6d

    vscode.debug.onDidStartDebugSession(e => {
        console.log('START', e.id);
    })

    vscode.debug.onDidTerminateDebugSession(e => {
        console.log('STOP', e.id);
    })

    vscode.debug.onDidChangeActiveDebugSession(e => {
        console.log('CHANGE ACTIVE', e.id);
    })

    vscode.debug.onDidReceiveDebugSessionCustomEvent(e => {
        console.log('CUSTOM EVENT', e.session.id, e.event);
    })

[weinand]

• Before we know that the debug session is up-and-running, some scripts have already been loaded. So it will be difficult to have custom events always come after START.
• but CHANGE ACTIVE should not come before START, that's something we can fix.

[jrieken]

@weinand This is what I am seeing now. The order seems correct, but the start and active event seem to be send twice...

START 4909af9c-79ae-4367-96cb-eacb5fde16d6
CHANGE ACTIVE 4909af9c-79ae-4367-96cb-eacb5fde16d6
START 4909af9c-79ae-4367-96cb-eacb5fde16d6
CHANGE ACTIVE 4909af9c-79ae-4367-96cb-eacb5fde16d6
CUSTOM EVENT 4909af9c-79ae-4367-96cb-eacb5fde16d6 script
CUSTOM EVENT 4909af9c-79ae-4367-96cb-eacb5fde16d6 scriptLoaded

[isidorn]

@jrieken I can not repro this.
I am using mac, here's my extension.ts. Let me know what you are doing differently

'use strict';
// The module 'vscode' contains the VS Code extensibility API
// Import the module and reference it with the alias vscode in your code below
import * as vscode from 'vscode';

// this method is called when your extension is activated
// your extension is activated the very first time the command is executed
export function activate(context: vscode.ExtensionContext) {

    // Use the console to output diagnostic information (console.log) and errors (console.error)
    // This line of code will only be executed once when your extension is activated
    console.log('Congratulations, your extension "tsext-first" is now active!');

    // The command has been defined in the package.json file
    // Now provide the implementation of the command with  registerCommand
    // The commandId parameter must match the command field in package.json
    vscode.debug.onDidStartDebugSession(e => {
        console.log('START', e.id);
    })

    vscode.debug.onDidTerminateDebugSession(e => {
        console.log('STOP', e.id);
    })

    vscode.debug.onDidChangeActiveDebugSession(e => {
        console.log('CHANGE ACTIVE', e.id);
    })

    vscode.debug.onDidReceiveDebugSessionCustomEvent(e => {
        console.log('CUSTOM EVENT', e.session.id, e.event);
    })



    let disposable = vscode.commands.registerCommand('extension.sayHello', () => {
        // The code you place here will be executed every time your command is executed
        const configuration = vscode.workspace.getConfiguration('testExtension', vscode.window.activeTextEditor.document.uri);
        configuration.get('resourceConfiguration');
        console.log(configuration.inspect('resourceConfiguration'));
        console.log(configuration.inspect('resourceConfiguration'));
        // Display a message box to the user
        vscode.window.showInformationMessage('Hello World!');

    });

    context.subscriptions.push(disposable);
}

// this method is called when your extension is deactivated
export function deactivate() {
}

Here's a js program that I am debugging

function fib(i) {
    if (i < 2) {
        for(var t = 0; t < 10; t++) {
            console.log(t);
        }
        return 1;
    }

    return fib(i - 1) + fib(i - 2);
}

console.log(fib(20));

[vscodebot]

This issue has been closed automatically because it needs more information and has not had recent activity. Please refer to our guidelines for filing issues. Thank you for your contributions.

[roblourens]

Should this have been closed?

[vscodebot]

This issue has been closed automatically because it needs more information and has not had recent activity. Please refer to our guidelines for filing issues. Thank you for your contributions.