Fix/benalvo/vorpal warning file with no extension

package.json

@@ -780,7 +780,7 @@
       {
         "title": "Checkmarx One",
         "id": "ast-results",
-        "order": 3,
+        "order": 4,
         "properties": {
           "checkmarxOne.apiKey": {
             "type": "string",
@@ -825,7 +825,7 @@
       {
         "title": "Checkmarx AI Security Champion",
         "id": "ask-kics",
-        "order": 2,
+        "order": 3,
         "properties": {
           "CheckmarxSecurityChampion.key": {
             "type": "string",
@@ -849,6 +849,19 @@
             ]
           }
         }
+      },
+      {
+        "title": "Activate Vorpal Auto Scanning",
+        "id": "vorpal",
+        "order": 2,
+        "properties": {
+          "CheckmarxVorpal.ActivateVorpalAutoScanning": {
+            "type": "boolean",
+            "order": 3,
+            "default": false,
+            "markdownDescription": "Scans your file as you code"
+          }
+        }
       }
     ]
   },
@@ -861,7 +874,7 @@
     "copytestproject": "copyfiles -u 2 \"src/resources/**/*\" out/test/ -E",
     "copymedia": "copyfiles \"media/icons/*\" out/ -E",
     "configure-husky": "npx husky install && npx husky add .husky/pre-commit \"npx --no-install lint-staged\"",
-    "test": "export TEST=true && npm run compile && extest setup-and-run './out/test/**/*test.js' -c 1.71.0  -i -r .",
+    "test": "export TEST=true && npm run compile && extest setup-and-run './out/test/**/*test.js' -c 1.87.2  -i -r .",
     "win-test": "set TEST=true&& npm run compile && extest setup-and-run './out/test/**/*test.js' -c 1.87.2  -i -r ."
   },
   "devDependencies": {
@@ -883,7 +896,7 @@
     "webpack-cli": "^5.1.4"
   },
   "dependencies": {
-    "@checkmarxdev/ast-cli-javascript-wrapper": "0.0.108",
+    "@checkmarxdev/ast-cli-javascript-wrapper": "0.0.110",
     "copyfiles": "2.4.1",
     "eslint-config-prettier": "^9.1.0",
     "eslint-plugin-node": "^11.1.0",

src/commands/vorpalCommand.ts

@@ -0,0 +1,76 @@
+import * as vscode from 'vscode';
+import { Logs } from "../models/logs";
+import { clearVorpalProblems, installVorpal, scanVorpal } from "../vorpal/scanVorpal";
+import { constants } from '../utils/common/constants';
+
+let timeout = null;
+export class VorpalCommand {
+  context: vscode.ExtensionContext;
+  logs: Logs;
+  onDidChangeTextDocument: vscode.Disposable;
+  constructor(context: vscode.ExtensionContext, logs: Logs) {
+    this.context = context;
+    this.logs = logs;
+  }
+  public async registerVorpal() {
+    try {
+      const vorpalActive = vscode.workspace
+        .getConfiguration("CheckmarxVorpal")
+        .get("ActivateVorpalAutoScanning") as boolean;
+      if (vorpalActive) {
+        await this.installVorpal();
+        await this.registerVorpalScanOnChangeText();
+        this.logs.info(constants.vorpalStart);
+      } else {
+        await this.disposeVorpalScanOnChangeText();
+        await clearVorpalProblems();
+        this.logs.info(constants.vorpalDisabled);
+      }
+    } catch(error) {
+      console.error(error);
+   }
+  }
+  public installVorpal() {
+    installVorpal(this.logs);
+    this.onDidChangeTextDocument = vscode.workspace.onDidChangeTextDocument(
+      // Must be no less than 2000ms. Otherwise, the temporary file can be deleted before the vorpal scan is finished.
+      this.debounce(this.onTextChange, 2000) 
+    );
+    }
+
+  public onTextChange(event) {
+    try {
+      scanVorpal(event.document, this.logs);
+    } catch (error) {
+      console.error(error);
+      this.logs.warn("fail to scan vorpal");
+    }
+  }
+  // Debounce function
+  public debounce(func, wait) {
+    const context = this;
+    console.log("onDidChangeTextDocument");
+    return function (...args) {
+      try {
+        const later = () => {
+          clearTimeout(timeout);
+          func.apply(context, args);
+        };
+        clearTimeout(timeout);
+        timeout = setTimeout(later, wait);
+      } catch (error) {
+        console.error(error);
+      }
+    };
+  }
+
+  public registerVorpalScanOnChangeText() {
+    this.context.subscriptions.push(this.onDidChangeTextDocument);
+  }
+  public disposeVorpalScanOnChangeText() {
+    if (this.onDidChangeTextDocument) {
+      this.onDidChangeTextDocument.dispose();
+      this.context.subscriptions.push(this.onDidChangeTextDocument);
+    }
+  }
+}

src/cx/cx.ts

@@ -5,9 +5,7 @@ import CxScan from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/scan/CxSc
 import CxProject from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/project/CxProject";
 import CxCodeBashing from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/codebashing/CxCodeBashing";
 import { CxConfig } from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/wrapper/CxConfig";
-import {
-	constants
-} from "../utils/common/constants";
+import { constants } from "../utils/common/constants";
 import { getFilePath, getResultsFilePath } from "../utils/utils";
 import { SastNode } from "../models/sastNode";
 import AstError from "../exceptions/AstError";
@@ -17,6 +15,7 @@ import { CxPlatform } from "./cxPlatform";
 import { CxCommandOutput } from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/wrapper/CxCommandOutput";
 import { ChildProcessWithoutNullStreams } from "child_process";
 import CxLearnMoreDescriptions from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/learnmore/CxLearnMoreDescriptions";
+import CxVorpal from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/vorpal/CxVorpal";
 import { messages } from "../utils/common/messages";
 export class Cx implements CxPlatform {
 	async scaScanCreate(sourcePath: string): Promise<CxScaRealtime | undefined> {
@@ -425,4 +424,39 @@ export class Cx implements CxPlatform {
 		statusBarItem.text = text;
 		show ? statusBarItem.show() : statusBarItem.hide();
 	}
-}
+	async installVorpal(): Promise<CxVorpal> {
+		let config = this.getAstConfiguration();
+		if (!config) {
+		  config = new CxConfig();
+		}
+		const cx = new CxWrapper(config);
+		const scans = await cx.scanVorpal(null, true, constants.vsCodeAgent);
+		if (scans.payload && scans.exitCode === 0) {
+		  return scans.payload[0];
+		} else {
+		  return this.getVorpalError(scans.status, "Failed to run vorpal engine");
+		}
+	  }
+
+	private getVorpalError(scanStatus: string, errorMessage: string) {
+		console.error(errorMessage);
+		const errorRes = new CxVorpal();
+		errorRes.error = scanStatus;
+		return errorRes;
+	}
+
+	  async scanVorpal(sourcePath: string): Promise<CxVorpal> {
+		let config = this.getAstConfiguration();
+		if (!config) {
+		  config = new CxConfig();
+		}
+		const cx = new CxWrapper(config);
+		const scans = await cx.scanVorpal(sourcePath, false, constants.vsCodeAgent);
+		if (scans.payload && scans.exitCode === 0) {
+		  return scans.payload[0];
+		} else {
+		  return this.getVorpalError(scans.status, "Fail to call vorpal scan");
+		}
+	  }
+	}
+

src/cx/cxMock.ts

@@ -7,9 +7,12 @@ import { CxConfig } from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/wra
 import { getFilePath } from "../utils/utils";
 import { writeFileSync } from "fs";
 import { CxPlatform } from "./cxPlatform";
+import CxVorpal from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/vorpal/CxVorpal";
 import { EMPTY_RESULTS_SCAN_ID } from "../test/utils/envs";
 
 export class CxMock implements CxPlatform {
+
+
 	// eslint-disable-next-line @typescript-eslint/no-explicit-any
 	async scaScanCreate(): Promise<CxScaRealtime[] | any> {
 		return [
@@ -552,7 +555,7 @@ export class CxMock implements CxPlatform {
 		await this.sleep(1000);
 		return [{ conversationId: '0', response: ["Mock message response from gpt"] }];
 	}
-
+	
 	sleep(ms) {
 		return new Promise(resolve => setTimeout(resolve, ms));
 	}
@@ -561,5 +564,13 @@ export class CxMock implements CxPlatform {
 		statusBarItem.text = text;
 		show ? statusBarItem.show() : statusBarItem.hide();
 	}
+
+	installVorpal(): Promise<CxVorpal> {
+		return null;
+	}
+
+	async scanVorpal(sourcePath: string): Promise<CxVorpal> {
+		return new CxVorpal();
+	}
 }
 

src/cx/cxPlatform.ts

@@ -8,6 +8,7 @@ import { Logs } from "../models/logs";
 import { ChildProcessWithoutNullStreams } from "child_process";
 import { CxCommandOutput } from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/wrapper/CxCommandOutput";
 import CxLearnMoreDescriptions from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/learnmore/CxLearnMoreDescriptions";
+import CxVorpal from "@checkmarxdev/ast-cli-javascript-wrapper/dist/main/vorpal/CxVorpal";
 
 export interface CxPlatform {
 	/**
@@ -176,5 +177,16 @@ export interface CxPlatform {
 		 * @param statusBarItem The {@link vscode.StatusBarItem} associated with the results.
 	*/
 	updateStatusBarItem(text: string, show: boolean, statusBarItem: vscode.StatusBarItem);
+
+  /**
+   * install the Vorpal engine
+   */
+  installVorpal(): Promise<CxVorpal>;
+
+  /**
+   * Scan the edited file in the vorpal engine and show the results in the problem section
+   * @param sourcePath the edited file sent to the vorpal engine
+   */
+  scanVorpal(sourcePath: string): Promise<CxVorpal>;
 }
 

src/extension.ts

@@ -23,11 +23,13 @@ import { WorkspaceListener } from "./utils/listener/workspaceListener";
 import { DocAndFeedbackView } from "./views/docsAndFeedbackView/docAndFeedbackView";
 import { messages } from "./utils/common/messages";
 import { commands } from "./utils/common/commands";
+import { VorpalCommand } from "./commands/vorpalCommand";
 
 export async function activate(context: vscode.ExtensionContext) {
   // Create logs channel and make it visible
   const output = vscode.window.createOutputChannel(constants.extensionFullName);
   const logs = new Logs(output);
+  logs.show();
   logs.info(messages.pluginRunning);
 
   // Status bars creation
@@ -175,6 +177,8 @@ export async function activate(context: vscode.ExtensionContext) {
       }
     }
   });
+  const vorpalCommand = new VorpalCommand(context, logs);
+  vorpalCommand.registerVorpal();
   // Register Settings
   const commonCommand = new CommonCommand(context, logs);
   commonCommand.registerSettings();
@@ -186,7 +190,7 @@ export async function activate(context: vscode.ExtensionContext) {
   // SCA auto scanning enablement
   await commonCommand.executeCheckScaScanEnabled();
   // execute command to listen to settings change
-  await executeCheckSettingsChange(kicsStatusBarItem, logs);
+  await executeCheckSettingsChange(kicsStatusBarItem, logs, vorpalCommand);
 
   const treeCommand = new TreeCommand(
     context,

src/test/0.welcome.test.ts

@@ -7,7 +7,7 @@ describe("Welcome view test", () => {
   let bench: Workbench;
 
   before(async function () {
-    this.timeout(8000);
+    this.timeout(100000);
     bench = new Workbench();
   });
 

src/test/1.settings.test.ts

@@ -60,4 +60,25 @@ describe("Extension settings tests", () => {
     const enablement = await setting.getValue();
     expect(enablement).to.equal(true);
   });
+
+  it("verify vorpal checkbox exists in the settings", async function () {
+    settingsEditor = await bench.openSettings();
+    const vorpalCheckbox = await settingsEditor.findSetting(
+      "ActivateVorpalAutoScanning",
+      "Checkmarx Vorpal"
+    );
+    let vorpalCheckboxValue = await vorpalCheckbox.getValue();
+    expect(vorpalCheckboxValue).to.not.be.undefined;
+  });
+
+  it("vorpal starts when the Vorpal checkbox is True in settings", async function () {
+    settingsEditor = await bench.openSettings();
+    const vorpalCheckbox = await settingsEditor.findSetting(
+      "ActivateVorpalAutoScanning",
+      "Checkmarx Vorpal"
+    );
+    await vorpalCheckbox.setValue(true);
+    let vorpalCheckboxValue = await vorpalCheckbox.getValue();
+    expect(vorpalCheckboxValue).to.be.true;
+  });
 });