diff --git a/docs/extension-settings.md b/docs/extension-settings.md index 187e6aee..e1f0ca45 100644 --- a/docs/extension-settings.md +++ b/docs/extension-settings.md @@ -73,7 +73,11 @@ pkg:/source/main.brs(10:20) ...ce/main.brs(10:20) ``` ### `brightscript.debug.enableVariablesPanel` +_(deprecated) Use `deferScopeLoading` instead._ + Defaults to `true`. Enables automatic population of the debug variable panel on a breakpoint or runtime errors +### `brightscript.debug.deferScopeLoading` +Defaults to `false`. Will defer the population of the `Local` scope variables until the user expands it in the variables panel. ### `brightscript.debug.autoResolveVirtualVariables` (Experimental) Defaults to `false`. Enables automatic population of the virtual variables. ### `brightscript.debug.enhanceREPLCompletions` diff --git a/package-lock.json b/package-lock.json index 5c6cc53a..57af1a6f 100644 --- a/package-lock.json +++ b/package-lock.json @@ -37,7 +37,7 @@ "postman-request": "^2.88.1-postman.32", "pretty-bytes": "^5.6.0", "resolve": "^1.22.8", - "roku-debug": "^0.21.22", + "roku-debug": "^0.21.23", "roku-deploy": "^3.12.4", "roku-test-automation": "^2.0.10", "semver": "^7.1.3", @@ -9162,9 +9162,9 @@ } }, "node_modules/roku-debug": { - "version": "0.21.22", - "resolved": "https://registry.npmjs.org/roku-debug/-/roku-debug-0.21.22.tgz", - "integrity": "sha512-1HITKLIPwThNIIjb/Yebp6rSgEvEnDKGjUKX/MJKawnvZr1h6DT2smtdgYZze3BoYV22GVeZj6aL8AilfUBS3A==", + "version": "0.21.23", + "resolved": "https://registry.npmjs.org/roku-debug/-/roku-debug-0.21.23.tgz", + "integrity": "sha512-rrnxAOsYINdor0sVRLmD2XQa8/o8BrtvxnCmLje+1WkoFSAm+4EvJwUT3tHN9UfS0vx6gUk00QflQJyqv6ECTw==", "dependencies": { "@rokucommunity/logger": "^0.3.9", "@types/request": "^2.48.8", diff --git a/package.json b/package.json index dde32c24..8da23d02 100644 --- a/package.json +++ b/package.json @@ -79,7 +79,7 @@ "postman-request": "^2.88.1-postman.32", "pretty-bytes": "^5.6.0", "resolve": "^1.22.8", - "roku-debug": "^0.21.22", + "roku-debug": "^0.21.23", "roku-deploy": "^3.12.4", "roku-test-automation": "^2.0.10", "semver": "^7.1.3", @@ -790,8 +790,14 @@ "enableVariablesPanel": { "type": "boolean", "description": "Enables automatic population of the debug variable panel on a breakpoint or runtime errors.", + "deprecationMessage": "Use 'deferScopeLoading' instead.", "default": true }, + "deferScopeLoading": { + "type": "boolean", + "description": "Will defer the population of the 'Local' scope variables until the user expands it in the variables panel.", + "default": false + }, "autoResolveVirtualVariables": { "type": "boolean", "description": "(Experimental) Enables automatic population of the virtual variables.", @@ -2241,9 +2247,16 @@ "brightscript.debug.enableVariablesPanel": { "type": "boolean", "description": "Enables automatic population of the debug variable panel on a breakpoint or runtime errors.", + "deprecationMessage": "Use 'deferScopeLoading' instead.", "default": true, "scope": "resource" }, + "brightscript.debug.deferScopeLoading": { + "type": "boolean", + "description": "Will defer the population of the 'Local' scope variables until the user expands it in the variables panel.", + "default": false, + "scope": "resource" + }, "brightscript.debug.autoResolveVirtualVariables": { "type": "boolean", "description": "(Experimental) Enables automatic population of the virtual variables.", diff --git a/src/DebugConfigurationProvider.ts b/src/DebugConfigurationProvider.ts index 5d14d466..f7af4a3d 100644 --- a/src/DebugConfigurationProvider.ts +++ b/src/DebugConfigurationProvider.ts @@ -55,6 +55,7 @@ export class BrightScriptDebugConfigurationProvider implements DebugConfiguratio disableScreenSaver: true, retainStagingFolder: false, enableVariablesPanel: true, + deferScopeLoading: false, autoResolveVirtualVariables: false, enhanceREPLCompletions: false, showHiddenVariables: false, diff --git a/src/managers/TelemetryManager.ts b/src/managers/TelemetryManager.ts index 948fd27e..fbb96e26 100644 --- a/src/managers/TelemetryManager.ts +++ b/src/managers/TelemetryManager.ts @@ -45,6 +45,12 @@ export class TelemetryManager implements Disposable { this.reporter.sendTelemetryEvent('startDebugSession', { enableDebugProtocol: boolToString(initialConfig.enableDebugProtocol), + enableVariablesPanel: boolToString(initialConfig.enableVariablesPanel), + deferScopeLoading: boolToString(initialConfig.deferScopeLoading), + autoResolveVirtualVariables: boolToString(initialConfig.autoResolveVirtualVariables), + enhanceREPLCompletions: boolToString(initialConfig.enhanceREPLCompletions), + rewriteDevicePathsInLogs: boolToString(initialConfig.rewriteDevicePathsInLogs), + showHiddenVariables: boolToString(initialConfig.showHiddenVariables), debugConnectionType: debugConnectionType?.toString(), retainDeploymentArchive: boolToString(initialConfig.retainDeploymentArchive), retainStagingFolder: boolToString(initialConfig.retainStagingFolder),