Merge pull request #628 from isc-bsaviano/fix-594

Add configuration option to turn off post-save compile

Author: isc-bsaviano

package.json

@@ -913,6 +913,11 @@
           "description": "Automatically show terminal when connected to docker-compose.",
           "type": "boolean",
           "default": false
+        },
+        "objectscript.compileOnSave": {
+          "description": "Automatically compile an InterSystems file when saved in the editor.",
+          "type": "boolean",
+          "default": true
         }
       }
     },

src/commands/compile.ts

@@ -240,7 +240,11 @@ async function compile(docs: CurrentFile[], flags?: string): Promise<any> {
     .then(loadChanges);
 }
 
-export async function importAndCompile(askFlags = false, document?: vscode.TextDocument): Promise<any> {
+export async function importAndCompile(
+  askFlags = false,
+  document?: vscode.TextDocument,
+  compileFile = true
+): Promise<any> {
   const file = currentFile(document);
   if (!file) {
     return;
@@ -260,7 +264,15 @@ export async function importAndCompile(askFlags = false, document?: vscode.TextD
     })
     .then(() => {
       if (!file.fileName.startsWith("\\.vscode\\")) {
-        compile([file], flags);
+        if (compileFile) {
+          compile([file], flags);
+        } else {
+          if (file.uri.scheme === FILESYSTEM_SCHEMA || file.uri.scheme === FILESYSTEM_READONLY_SCHEMA) {
+            // Fire the file changed event to avoid VSCode alerting the user on the next save that
+            // "The content of the file is newer."
+            fileSystemProvider.fireFileChanged(file.uri);
+          }
+        }
       }
     });
 }

src/extension.ts

@@ -534,7 +534,7 @@ export async function activate(context: vscode.ExtensionContext): Promise<any> {
   workspace.onDidSaveTextDocument((file) => {
     if (schemas.includes(file.uri.scheme) || languages.includes(file.languageId)) {
       if (documentBeingProcessed !== file) {
-        return importAndCompile(false, file);
+        return importAndCompile(false, file, config("compileOnSave"));
       }
     }
   });