feat(debugger): Update filewatch to look for .map and .js and send /r…

…eload to MC

package.json

@@ -1,7 +1,7 @@
 {
     "name": "minecraft-debugger",
-    "displayName": "Minecraft Bedrock Edition Debugger",
-    "description": "Debug your JavaScript code running in Minecraft Bedrock Edition.",
+    "displayName": "Minecraft Bedrock Debugger",
+    "description": "Debug your JavaScript code running in Minecraft Edition.",
     "version": "0.0.0-semantically-released",
     "publisher": "mojang-studios",
     "author": {
@@ -177,6 +177,21 @@
         ],
         "configuration": {
             "properties": {
+                "minecraft-debugger.reloadOnSourceChanges.enabled": {
+                    "type": "boolean",
+                    "default": false,
+                    "description": "Reload Minecraft on source changes. Watch localRoot or sourceMapRoot."
+                },
+                "minecraft-debugger.reloadOnSourceChanges.delay": {
+                    "type": "number",
+                    "default": 500,
+                    "description": "Delay in milliseconds before reloading Minecraft."
+                },
+                "minecraft-debugger.reloadOnSourceChanges.globPattern": {
+                    "type": "string",
+                    "default": "",
+                    "description": "Override the default locations and monitor all workspace files with the specified glob pattern."
+                },
                 "minecraft-debugger.showDiagnosticViewOnConnect": {
                     "type": "boolean",
                     "default": false,

src/Session.ts

@@ -20,6 +20,7 @@ import {
     InputBoxOptions,
     QuickPickItem,
     QuickPickOptions,
+    RelativePattern,
     Uri,
     workspace,
     window,
@@ -128,7 +129,7 @@ export class Session extends DebugSession {
     private _threads = new Set<number>();
     private _requests = new Map<number, PendingResponse>();
     private _sourceMaps: SourceMaps = new SourceMaps('');
-    private _fileWatcher?: FileSystemWatcher;
+    private _sourceFileWatcher?: FileSystemWatcher;
     private _activeThreadId: number = 0; // the one being debugged
     private _localRoot: string = '';
     private _sourceMapRoot?: string;
@@ -167,6 +168,11 @@ export class Session extends DebugSession {
         this._eventEmitter.removeAllListeners('start-profiler');
         this._eventEmitter.removeAllListeners('stop-profiler');
         this._eventEmitter.removeAllListeners('request-debugger-status');
+
+        if (this._sourceFileWatcher) {
+            this._sourceFileWatcher.dispose();
+            this._sourceFileWatcher = undefined;
+        }
     }
 
     // ------------------------------------------------------------------------
@@ -661,7 +667,7 @@ export class Session extends DebugSession {
         );
 
         // watch for source map changes
-        this.createSourceMapFileWatcher(this._sourceMapRoot);
+        this.createSourceMapFileWatcher(this._localRoot, this._sourceMapRoot);
 
         // Now that a connection is established, and capabilities have been delivered, send this event to
         // tell VSCode to ask Minecraft/debugee for config data (breakpoints etc).
@@ -689,11 +695,6 @@ export class Session extends DebugSession {
             this._connectionSocket.destroy();
         }
         this._connectionSocket = undefined;
-
-        if (this._fileWatcher) {
-            this._fileWatcher.dispose();
-            this._fileWatcher = undefined;
-        }
     }
 
     // close and terminate session (could be from debugee request)
@@ -1015,22 +1016,59 @@ export class Session extends DebugSession {
         }
     }
 
-    private createSourceMapFileWatcher(sourceMapRoot?: string) {
-        if (this._fileWatcher) {
-            this._fileWatcher.dispose();
-            this._fileWatcher = undefined;
+    private createSourceMapFileWatcher(localRoot: string, sourceMapRoot?: string) {
+        if (this._sourceFileWatcher) {
+            this._sourceFileWatcher.dispose();
+            this._sourceFileWatcher = undefined;
         }
-        if (sourceMapRoot) {
-            this._fileWatcher = workspace.createFileSystemWatcher('**/*.{map}', false, false, false);
-            this._fileWatcher.onDidChange(uri => {
-                this._sourceMaps.reset();
-            });
-            this._fileWatcher.onDidCreate(uri => {
-                this._sourceMaps.reset();
-            });
-            this._fileWatcher.onDidDelete(uri => {
-                this._sourceMaps.reset();
-            });
+
+        const minReloadDelay = 500;
+        const config = workspace.getConfiguration('minecraft-debugger');
+        const reloadOnSourceChangesEnabled = config.get<boolean>('reloadOnSourceChanges.enabled');
+        const reloadOnSourceChangesDelay = Math.max(config.get<number>('reloadOnSourceChanges.delay') || minReloadDelay, minReloadDelay);
+        const reloadOnSourceChangesGlobPattern = config.get<string>('reloadOnSourceChanges.globPattern');
+        if (!reloadOnSourceChangesEnabled) {
+            return;
+        }
+
+        // Either monitor the build output (TS->JS) by looking at .map and .js files in sourceMapRoot,
+        // or monitor .js files directly if not using TS or source maps by looking at localRoot,
+        // or monitor a specific glob pattern for all files within the workspace.
+        let globPattern = undefined;
+        if (reloadOnSourceChangesGlobPattern) {
+            globPattern = new RelativePattern(workspace.workspaceFolders?.[0].uri.fsPath || '', reloadOnSourceChangesGlobPattern);
+        }
+        else if (sourceMapRoot) {
+            globPattern = new RelativePattern(sourceMapRoot, '**/*.{map,js}');
+        }
+        else if (localRoot) {
+            globPattern = new RelativePattern(localRoot, '**/*.js');
+        }
+
+        if (globPattern) {
+            this._sourceFileWatcher = workspace.createFileSystemWatcher(globPattern, false, false, false);
+        }
+
+        const doReload = (): void => {
+            this._sourceMaps.clearCache();
+            this.onRunMinecraftCommand('say §aPerforming Auto-Reload');
+            this.onRunMinecraftCommand('reload');
+        };
+
+        const debounce = (func: () => void, wait: number): (() => void) => {
+            let timeout: NodeJS.Timeout;
+            return () => {
+                clearTimeout(timeout);
+                timeout = setTimeout(func, wait);
+            };
+        };
+
+        const debouncedReload = debounce(doReload, reloadOnSourceChangesDelay);
+
+        if (this._sourceFileWatcher) {
+            this._sourceFileWatcher.onDidChange(debouncedReload);
+            this._sourceFileWatcher.onDidCreate(debouncedReload);
+            this._sourceFileWatcher.onDidDelete(debouncedReload);
         }
     }
 

src/SourceMaps.ts

@@ -221,7 +221,7 @@ export class SourceMaps {
                 : SourceMapConsumer.LEAST_UPPER_BOUND;
     }
 
-    public reset() {
+    public clearCache() {
         this._sourceMapCache.reset();
     }
 

src/panels/HomeViewProvider.ts

@@ -60,6 +60,10 @@ export class HomeViewProvider implements vscode.WebviewViewProvider {
                     vscode.commands.executeCommand('minecraft-debugger.showMinecraftDiagnostics');
                     break;
                 }
+                case 'show-settings': {
+                    vscode.commands.executeCommand('workbench.action.openSettings', '@ext:mojang-studios.minecraft-debugger');
+                    break;
+                }
                 case 'run-minecraft-command': {
                     if (!message.command || message.command.trim() === '') {
                         vscode.window.showErrorMessage('Minecraft Command Shortcut can not be empty.');

webview-ui/src/home_panel/App.tsx

@@ -1,10 +1,10 @@
 
 // Copyright (C) Microsoft Corporation.  All rights reserved.
 
-import { useEffect, useRef, useState } from 'react';
+import { useEffect, useState } from 'react';
 import CommandSection from './controls/CommandSection'
 import { CommandButton, CommandHandlers, getCommandHandlers } from './handlers/CommandHandlers';
-import DiagnosticSection from './controls/DiagnosticsSection';
+import GeneralSection from './controls/GeneralSection';
 import ProfilerSection from './controls/ProfilerSection';
 import { CaptureItem, ProfilerHandlers, getProfilerHandlers } from './handlers/ProfilerHandlers';
 import StatusSection from './controls/StatusSection';
@@ -22,6 +22,10 @@ const onShowDiagnosticsPanel = () => {
     vscode.postMessage({ type: 'show-diagnostics' });
 };
 
+const onShowSettings = () => {
+    vscode.postMessage({ type: 'show-settings' });
+};
+
 const onRunCommand = (command: string) => {
     vscode.postMessage({ type: 'run-minecraft-command', command: command });
 };
@@ -121,8 +125,9 @@ const App = () => {
             <StatusSection
                 debuggerConnected={debuggerConnected}
             />
-            <DiagnosticSection
+            <GeneralSection
                 onShowDiagnosticsPanel={onShowDiagnosticsPanel}
+                onShowSettings={onShowSettings}
             />
             <CommandSection
                 debuggerConnected={debuggerConnected}

webview-ui/src/home_panel/controls/DiagnosticsSection.tsx → webview-ui/src/home_panel/controls/GeneralSection.tsx

@@ -4,19 +4,23 @@
 import React from 'react';
 import { VSCodeButton } from '@vscode/webview-ui-toolkit/react';
 
-interface DiagnosticsSectionProps {
+interface GeneralSectionProps {
     onShowDiagnosticsPanel: () => void;
+    onShowSettings(): void;
 }
 
-const DiagnosticsSection: React.FC<DiagnosticsSectionProps> = ({ onShowDiagnosticsPanel }) => {
+const GeneralSection: React.FC<GeneralSectionProps> = ({ onShowDiagnosticsPanel, onShowSettings }) => {
     return (
         <div className="section">
-            <h3 className="title">Diagnostics</h3>
+            <h3 className="title">Actions</h3>
             <VSCodeButton className="standard-button" onClick={onShowDiagnosticsPanel}>
                 Show Diagnostics
             </VSCodeButton>
+            <VSCodeButton className="standard-button" onClick={onShowSettings}>
+                Show Settings
+            </VSCodeButton>
         </div>
     );
 };
 
-export default DiagnosticsSection;
+export default GeneralSection;