Add some breakpoint logging

src/adapters/DebugProtocolAdapter.ts

@@ -746,6 +746,7 @@ export class DebugProtocolAdapter {
 
     private syncBreakpointsPromise = Promise.resolve();
     public async syncBreakpoints() {
+        this.logger.log('syncBreakpoints()');
         //wait for the previous sync to finish
         this.syncBreakpointsPromise = this.syncBreakpointsPromise
             //ignore any errors
@@ -766,6 +767,7 @@ export class DebugProtocolAdapter {
 
         //compute breakpoint changes since last sync
         const diff = await this.breakpointManager.getDiff(this.projectManager.getAllProjects());
+        this.logger.log('Syncing breakpoints', diff);
 
         // REMOVE breakpoints (delete these breakpoints from the device)
         if (diff.removed.length > 0) {

src/debugProtocol/client/DebugProtocolClient.ts

@@ -755,7 +755,7 @@ export class DebugProtocolClient {
 
         this.activeRequests.set(request.data.requestId, request);
 
-        return new Promise<T>((resolve) => {
+        return new Promise<T>((resolve, reject) => {
             let unsubscribe = this.on('response', (response) => {
                 if (response.data.requestId === request.data.requestId) {
                     unsubscribe();
@@ -774,7 +774,9 @@ export class DebugProtocolClient {
                     request: request
                 });
             } else {
-                throw new Error(`Control socket was closed - Command: ${Command[request.data.command]}`);
+                reject(
+                    new Error(`Control socket was closed - Command: ${Command[request.data.command]}`)
+                );
             }
         });
     }

src/debugSession/BrightScriptDebugSession.ts

@@ -398,7 +398,7 @@ export class BrightScriptDebugSession extends BaseDebugSession {
             }
 
             //send any compile errors to the client
-            await this.rokuAdapter.sendErrors();
+            await this.rokuAdapter?.sendErrors();
             this.logger.error('Error. Shutting down.', e);
             return this.shutdown();
         }
@@ -720,6 +720,7 @@ export class BrightScriptDebugSession extends BaseDebugSession {
      * Called every time a breakpoint is created, modified, or deleted, for each file. This receives the entire list of breakpoints every time.
      */
     public async setBreakPointsRequest(response: DebugProtocol.SetBreakpointsResponse, args: DebugProtocol.SetBreakpointsArguments) {
+        this.logger.log('setBreakpointsRequest');
         let sanitizedBreakpoints = this.breakpointManager.replaceBreakpoints(args.source.path, args.breakpoints);
         //sort the breakpoints
         let sortedAndFilteredBreakpoints = orderBy(sanitizedBreakpoints, [x => x.line, x => x.column]);
@@ -729,6 +730,7 @@ export class BrightScriptDebugSession extends BaseDebugSession {
         };
         this.sendResponse(response);
 
+        this.logger.debug('[setBreakpointsRequest] syncBreakpoints()', args);
         await this.rokuAdapter?.syncBreakpoints();
     }
 

src/managers/ProjectManager.ts

@@ -39,6 +39,8 @@ export class ProjectManager {
         enableDebugProtocol?: boolean;
     };
 
+    public logger = logger.createLogger('[ProjectManager]');
+
     public mainProject: Project;
     public componentLibraryProjects = [] as ComponentLibraryProject[];
 
@@ -158,6 +160,7 @@ export class ProjectManager {
         //convert entry point staging location to source location
         let sourceLocation = await this.getSourceLocation(entryPoint.relativePath, entryPoint.lineNumber);
 
+        this.logger.info(`Registering entry breakpoint at ${sourceLocation.filePath}:${sourceLocation.lineNumber} (${entryPoint.pathAbsolute}:${entryPoint.lineNumber})`);
         //register the entry breakpoint
         this.breakpointManager.setBreakpoint(sourceLocation.filePath, {
             //+1 to select the first line of the function