Merge pull request #61 from mathworks/dklilley/release/1.3.3

MATLAB language server - v1.3.3

Author: dklilley

.github/ISSUE_TEMPLATE/bug_report.md

@@ -24,6 +24,7 @@ A clear and concise description of what you expected to happen.
 If applicable, add screenshots to help explain your problem.
 
 **Useful Information**
+ - MATLAB Version: 
  - OS Version: 
  - Language Server Client: [e.g. MATLAB extension for Visual Studio Code]
  - Client Version: 

README.md

@@ -26,6 +26,15 @@ MATLAB language server supports these editors by installing the corresponding ex
 
 ### Unreleased
 
+### 1.3.3
+Release date: 2025-05-15
+
+Added:
+* Support for debugging P-coded files when the corresponding source file is available
+
+Fixed:
+* Resolves potential crashes when using code completion in files without a .m file extension
+
 ### 1.3.2
 Release date: 2025-03-06
 

matlab/+matlabls/+handlers/+completions/getCompletions.m

@@ -2,6 +2,11 @@
     % GETCOMPLETIONS Retrieves the data for the possible completions at the cursor position in the given code.
 
     % Copyright 2025 The MathWorks, Inc.
+    [~, ~, ext] = fileparts(fileName);
+    if ~isempty(fileName) && ~strcmpi(ext, '.m')
+        % Expected .m file extension
+        error('MATLAB:vscode:invalidFileExtension', 'The provided file must have a .m extension to process completions.');
+    end
 
     completionResultsStr = matlabls.internal.getCompletionsData(code, fileName, cursorPosition);
     completionsData = filterCompletionResults(completionResultsStr);

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "matlab-language-server",
-  "version": "1.3.2",
+  "version": "1.3.3",
   "description": "Language Server for MATLAB code",
   "main": "./src/index.ts",
   "bin": "./out/index.js",

package.json

@@ -5,6 +5,7 @@ import { DebugProtocol } from '@vscode/debugprotocol';
 import { DebugServices, BreakpointInfo } from './DebugServices'
 import { ResolvablePromise, createResolvablePromise } from '../utils/PromiseUtils'
 import { IMVM, MVMError, MatlabState } from '../mvm/impl/MVM';
+import fs from 'node:fs';
 
 enum BreakpointChangeType {
     ADD,
@@ -233,6 +234,8 @@ export default class MatlabDebugAdaptor {
 
     private _setupListeners (): void {
         this._debugServices.on(DebugServices.Events.BreakpointAdded, async (breakpoint: BreakpointInfo) => {
+            breakpoint.filePath = this._mapToMFile(breakpoint.filePath, false);
+
             this._matlabBreakpoints.push(breakpoint);
 
             this._breakpointChangeListeners.forEach((listener) => {
@@ -241,6 +244,8 @@ export default class MatlabDebugAdaptor {
         });
 
         this._debugServices.on(DebugServices.Events.BreakpointRemoved, async (breakpoint: BreakpointInfo) => {
+            breakpoint.filePath = this._mapToMFile(breakpoint.filePath, false);
+
             this._matlabBreakpoints = this._matlabBreakpoints.filter((existingBreakpoint) => {
                 return !existingBreakpoint.equals(breakpoint, true);
             });
@@ -422,6 +427,7 @@ export default class MatlabDebugAdaptor {
         }
 
         const canonicalizedPath = await this._getCanonicalPath(source.path);
+        const pathToSetOrClear = this._mapToPFile(canonicalizedPath, true);
 
         const newBreakpoints: BreakpointInfo[] = (args.breakpoints != null)
             ? args.breakpoints.map((breakpoint) => {
@@ -458,7 +464,7 @@ export default class MatlabDebugAdaptor {
         // Remove all breakpoints that are now gone.
         const breakpointsRemovalPromises: Array<Promise<void>> = [];
         breakpointsToRemove.forEach((breakpoint: BreakpointInfo) => {
-            breakpointsRemovalPromises.push(this._mvm.clearBreakpoint(breakpoint.filePath, breakpoint.lineNumber));
+            breakpointsRemovalPromises.push(this._mvm.clearBreakpoint(pathToSetOrClear, breakpoint.lineNumber));
         })
         await Promise.all(breakpointsRemovalPromises);
 
@@ -476,12 +482,12 @@ export default class MatlabDebugAdaptor {
 
             let matlabBreakpointInfos: BreakpointInfo[] = [];
             const listener = this._registerBreakpointChangeListener((changeType, bpInfo) => {
-                if (changeType === BreakpointChangeType.ADD && bpInfo.filePath === canonicalizedPath) {
+                if (changeType === BreakpointChangeType.ADD && bpInfo.filePath === pathToSetOrClear) {
                     matlabBreakpointInfos.push(bpInfo);
                 }
             });
 
-            await this._mvm.setBreakpoint(canonicalizedPath, newBreakpoint.info.lineNumber, newBreakpoint.info.condition);
+            await this._mvm.setBreakpoint(pathToSetOrClear, newBreakpoint.info.lineNumber, newBreakpoint.info.condition);
 
             listener.remove();
 
@@ -501,6 +507,36 @@ export default class MatlabDebugAdaptor {
         this._clearPendingBreakpointsRequest();
     }
 
+    _mapToPFile (filePath: string, checkIfExists: boolean): string {
+        // If this is an m-file then convert to p-file and check existence
+        if (filePath.endsWith('.m')) {
+            const pFile = filePath.substring(0, filePath.length - 1) + 'p';
+            if (!checkIfExists || fs.existsSync(pFile)) {
+                return pFile;
+            } else {
+                return filePath;
+            }
+        }
+
+        // Not an m file so p-code not supported
+        return filePath;
+    }
+
+    _mapToMFile (filePath: string, checkIfExists: boolean): string {
+        // If this is an p-file then convert to m-file and check existence
+        if (filePath.endsWith('.p')) {
+            const mFile = filePath.substring(0, filePath.length - 1) + 'm';
+            if (!checkIfExists || fs.existsSync(mFile)) {
+                return mFile;
+            } else {
+                return filePath;
+            }
+        }
+
+        // Not an m file so p-code not supported
+        return filePath;
+    }
+
     async continueRequest (response: DebugProtocol.ContinueResponse, args: DebugProtocol.ContinueArguments, request?: DebugProtocol.Request): Promise<void> {
         try {
             await this._mvm.eval("if system_dependent('IsDebugMode')==1, dbcont; end");
@@ -558,15 +594,18 @@ export default class MatlabDebugAdaptor {
             if (stack[0]?.mwtype !== undefined) {
                 stack = stack[0]
                 const size = stack.mwsize[0];
-                const newStack = [];
+                const transformedStack = [];
                 for (let i = 0; i < size; i++) {
-                    newStack.push(new debug.StackFrame(size - i + 1, stack.mwdata.name[i], new debug.Source(stack.mwdata.name[i], stack.mwdata.file[i]), Math.abs(stack.mwdata.line[i]), 1))
+                    transformedStack.push({ name: stack.mwdata.name[i], file: stack.mwdata.file[i], line: stack.mwdata.line[i] });
                 }
-                return newStack;
-            } else {
-                const numberOfStackFrames: number = stack.length;
-                return stack.map((stackFrame: MatlabData, i: number) => new debug.StackFrame(numberOfStackFrames - i + 1, stackFrame.name, new debug.Source(stackFrame.name as string, stackFrame.file as string), Math.abs(stackFrame.line), 1));
+                stack = transformedStack;
             }
+
+            const numberOfStackFrames: number = stack.length;
+            return stack.map((stackFrame: MatlabData, i: number) => {
+                const fileName: string = this._mapToMFile(stackFrame.file, true);
+                return new debug.StackFrame(numberOfStackFrames - i + 1, stackFrame.name, new debug.Source(stackFrame.name as string, fileName), Math.abs(stackFrame.line), 1)
+            });
         };
 
         const stack = transformStack(stackResponse.result);

src/debug/MatlabDebugAdaptor.ts

@@ -10,6 +10,7 @@ import ConfigurationManager from '../lifecycle/ConfigurationManager'
 import MVM from '../mvm/impl/MVM'
 import Logger from '../logging/Logger'
 import parse from '../mvm/MdaParser'
+import * as FileNameUtils from '../utils/FileNameUtils'
 
 interface WorkspaceFileIndexedResponse {
     isDone: boolean
@@ -115,7 +116,8 @@ export default class Indexer {
             return
         }
 
-        const fileContentBuffer = await fs.readFile(URI.parse(uri).fsPath)
+        const filePath = FileNameUtils.getFilePathFromUri(uri)
+        const fileContentBuffer = await fs.readFile(filePath)
         const code = fileContentBuffer.toString()
         const rawCodeData = await this.getCodeData(code, uri)
 
@@ -136,7 +138,7 @@ export default class Indexer {
      * @returns The raw data extracted from the document
      */
     private async getCodeData (code: string, uri: string): Promise<RawCodeData | null> {
-        const filePath = URI.parse(uri).fsPath
+        const filePath = FileNameUtils.getFilePathFromUri(uri)
         const analysisLimit = (await ConfigurationManager.getConfiguration()).maxFileSizeForAnalysis
 
         try {

src/indexing/Indexer.ts

@@ -8,8 +8,8 @@ import Expression from '../utils/ExpressionUtils'
 import { getTextOnLine } from '../utils/TextDocumentUtils'
 import PathResolver from '../providers/navigation/PathResolver'
 import * as fs from 'fs/promises'
-import { URI } from 'vscode-uri'
 import Indexer from './Indexer'
+import * as FileNameUtils from '../utils/FileNameUtils'
 
 export enum RequestType {
     Definition,
@@ -321,7 +321,7 @@ class SymbolSearchService {
         }
 
         // Ensure URI is not a directory. This can occur with some packages.
-        const fileStats = await fs.stat(URI.parse(resolvedUri).fsPath)
+        const fileStats = await fs.stat(FileNameUtils.getFilePathFromUri(resolvedUri))
         if (fileStats.isDirectory()) {
             return null
         }

src/indexing/SymbolSearchService.ts

@@ -6,9 +6,9 @@ import Logger from '../logging/Logger'
 import MatlabLifecycleManager from './MatlabLifecycleManager'
 import * as os from 'os'
 import path from 'path'
-import { URI } from 'vscode-uri'
 import MVM, { IMVM, MatlabState } from '../mvm/impl/MVM'
 import parse from '../mvm/MdaParser'
+import * as FileNameUtils from '../utils/FileNameUtils'
 
 export default class PathSynchronizer {
     constructor (private readonly matlabLifecycleManager: MatlabLifecycleManager, private readonly mvm: MVM) {}
@@ -161,9 +161,7 @@ export default class PathSynchronizer {
 
     private convertWorkspaceFoldersToFilePaths (workspaceFolders: WorkspaceFolder[]): string[] {
         return workspaceFolders.map(folder => {
-            const uri = URI.parse(folder.uri)
-
-            return path.normalize(uri.fsPath)
+            return path.normalize(FileNameUtils.getFilePathFromUri(folder.uri))
         });
     }