Addressed review comments

Achal1607 · Achal1607 · commit 38129b880a55 · 2024-10-25T16:14:10.000+05:30
diff --git a/vscode/l10n/bundle.l10n.en.json b/vscode/l10n/bundle.l10n.en.json
@@ -45,7 +45,7 @@
   "jdk.extension.fileSelector.label.testFilesOrSourceFiles": "Test files or source files associated to each other",
   "jdk.extension.fileSelector.label.noFileSelected": "No file selected",
   "jdk.extension.fileSelector.label.noTestFound": "No Test or Tested class found",
-  "jdk.extension.cache.message.confirmToDeleteCache": "Are you sure you want to delete cache for this workspace?",
+  "jdk.extension.cache.message.confirmToDeleteCache": "Are you sure you want to delete the cache for this workspace and reload the window?",
   "jdk.extension.cache.label.confirmation.yes":"Yes",
   "jdk.extension.cache.label.confirmation.cancel":"Cancel",
   "jdk.extension.cache.message.cacheCleared":"Cache cleared successfully for this workspace",
diff --git a/vscode/src/commands/cache.ts b/vscode/src/commands/cache.ts
@@ -33,7 +33,7 @@ const deleteCache = async () => {
     if (userDir && fs.existsSync(userDir)) {
         const yes = l10n.value("jdk.extension.cache.label.confirmation.yes")
         const cancel = l10n.value("jdk.extension.cache.label.confirmation.cancel")
-        const confirmation = await window.showInformationMessage('Are you sure you want to delete cache for this workspace  and reload the window ?',
+        const confirmation = await window.showInformationMessage(l10n.value("jdk.extension.cache.message.confirmToDeleteCache"),
             yes, cancel);
         if (confirmation === yes) {
             const reloadWindowActionLabel = l10n.value("jdk.extension.cache.label.reloadWindow");
diff --git a/vscode/src/commands/commands.ts b/vscode/src/commands/commands.ts
@@ -48,7 +48,7 @@ export const extCommands = {
     selectEditorProjs: appendPrefixToCommand('select.editor.projects'),
     attachDebugger: appendPrefixToCommand("java.attachDebugger.connector"),
     loadWorkspaceTests: appendPrefixToCommand("load.workspace.tests"),
-    projectDeleteEntry: "javals.foundProjects.deleteEntry"
+    projectDeleteEntry: appendPrefixToCommand("foundProjects.deleteEntry")
 }
 
 export const builtInCommands = {
@@ -78,5 +78,8 @@ export const nbCommands = {
     debuggerConfigurations: appendPrefixToCommand('java.attachDebugger.configurations'),
     runProjectAction: appendPrefixToCommand('project.run.action'),
     buildWorkspace: appendPrefixToCommand('build.workspace'),
-    cleanWorkspace: appendPrefixToCommand('clean.workspace')
+    cleanWorkspace: appendPrefixToCommand('clean.workspace'),
+    clearProjectCaches: appendPrefixToCommand('clear.project.caches'),
+    javaProjectPackages: appendPrefixToCommand('java.get.project.packages'),
+    openStackTrace: appendPrefixToCommand('open.stacktrace')
 }
diff --git a/vscode/src/configurations/configuration.ts b/vscode/src/configurations/configuration.ts
@@ -24,6 +24,7 @@ export const configKeys = {
     formatPrefs: 'format',
     hintPrefs: 'hints',
     importPrefs: 'java.imports',
+    runConfig: 'runCofig',
     runConfigVmOptions: 'runConfig.vmOptions',
     runConfigCwd: 'runConfig.cwd',
     verbose: 'verbose',
diff --git a/vscode/src/configurations/handlers.ts b/vscode/src/configurations/handlers.ts
@@ -13,7 +13,8 @@
   See the License for the specific language governing permissions and
   limitations under the License.
 */
-import { extensions, workspace } from "vscode";
+
+import { extensions, workspace, WorkspaceConfiguration } from "vscode";
 import { builtInConfigKeys, configKeys } from "./configuration";
 import { extConstants, NODE_WINDOWS_LABEL } from "../constants";
 import * as os from 'os';
@@ -22,6 +23,10 @@ import { LOGGER } from "../logger";
 import * as path from 'path';
 import * as fs from 'fs';
 
+export const getConfiguration = (key: string = extConstants.COMMAND_PREFIX): WorkspaceConfiguration => {
+    return workspace.getConfiguration(key);
+}
+
 export const getConfigurationValue = <T>(key: string, defaultValue: T | undefined = undefined): T => {
     const conf = workspace.getConfiguration(extConstants.COMMAND_PREFIX);
     return defaultValue != undefined ? conf.get(key, defaultValue) : conf.get(key) as T;
diff --git a/vscode/src/debugger/debugger.ts b/vscode/src/debugger/debugger.ts
@@ -36,7 +36,6 @@ export function registerDebugger(context: ExtensionContext): void {
     context.subscriptions.push(vscode.debug.registerDebugConfigurationProvider(extConstants.COMMAND_PREFIX, configDynamicProvider, vscode.DebugConfigurationProviderTriggerKind.Dynamic));
     let configResolver = new NetBeansConfigurationResolver();
     context.subscriptions.push(vscode.debug.registerDebugConfigurationProvider(extConstants.COMMAND_PREFIX, configResolver));
-    context.subscriptions.push(vscode.debug.onDidTerminateDebugSession(((session) => onDidTerminateSession(session))));
     let debugDescriptionFactory = new NetBeansDebugAdapterDescriptionFactory();
     context.subscriptions.push(vscode.debug.registerDebugAdapterDescriptorFactory(extConstants.COMMAND_PREFIX, debugDescriptionFactory));
     initializeRunConfiguration().then(initialized => {
@@ -267,14 +266,3 @@ class RunConfigurationProvider implements vscode.DebugConfigurationProvider {
     }
 
 }
-
-
-function onDidTerminateSession(session: vscode.DebugSession): any {
-    const config = session.configuration;
-    if (config.env) {
-        const file = config.env["MICRONAUT_CONFIG_FILES"];
-        if (file) {
-            vscode.workspace.fs.delete(vscode.Uri.file(file));
-        }
-    }
-}
diff --git a/vscode/src/extension.ts b/vscode/src/extension.ts
@@ -83,8 +83,9 @@ export function activate(context: ExtensionContext): VSNetBeansAPI {
 
 
 export function deactivate(): Thenable<void> {
-    if (globalVars.nbProcessManager?.getProcess() != null) {
-        globalVars.nbProcessManager?.getProcess()?.kill();
+    const process = globalVars.nbProcessManager?.getProcess();
+    if (process != null) {
+        process?.kill();
     }
     return globalVars.clientPromise.stopClient();
 }
diff --git a/vscode/src/logger.ts b/vscode/src/logger.ts
@@ -30,15 +30,18 @@ export class ExtensionLogger {
     }
 
     public log(message: string): void {
-        this.outChannel.appendLine(`[${LogLevel.INFO}]: ${message}`);
+        const formattedMessage = `[${LogLevel.INFO}]: ${message}`;
+        this.printLog(formattedMessage);
     }
 
     public warn(message: string): void {
-        this.outChannel.appendLine(`[${LogLevel.WARN}]: ${message}`);
+        const formattedMessage = `[${LogLevel.WARN}]: ${message}`;
+        this.printLog(formattedMessage);
     }
 
     public error(message: string): void {
-        this.outChannel.appendLine(`[${LogLevel.ERROR}]: ${message}`);
+        const formattedMessage = `[${LogLevel.ERROR}]: ${message}`;
+        this.printLog(formattedMessage);
     }
 
     public logNoNL(message: string): void {
@@ -56,6 +59,11 @@ export class ExtensionLogger {
     public dispose(): void {
         this.outChannel.dispose();
     }
+
+    private printLog(message: string): void{
+        const timestamp = new Date().toISOString();
+        this.outChannel.appendLine(`[${timestamp}] ${message}`);
+    }
 }
 
 export const LOGGER = new ExtensionLogger(extConstants.SERVER_NAME);
diff --git a/vscode/src/lsp/nbLanguageClient.ts b/vscode/src/lsp/nbLanguageClient.ts
@@ -62,8 +62,8 @@ export class NbLanguageClient extends LanguageClient {
                     'wantsJavaSupport': true,
                     'wantsGroovySupport': false,
                     'commandPrefix': extConstants.COMMAND_PREFIX,
-                    'configurationPrefix': 'jdk.',
-                    'altConfigurationPrefix': 'jdk.'
+                    'configurationPrefix': `${extConstants.COMMAND_PREFIX}.`,
+                    'altConfigurationPrefix': `${extConstants.COMMAND_PREFIX}.`
                 }
             },
             errorHandler: {
diff --git a/vscode/src/test/suite/general/extension.test.ts b/vscode/src/test/suite/general/extension.test.ts
@@ -30,7 +30,7 @@ import * as myExplorer from '../../../views/projects';
 import { CodeAction, commands, extensions, Selection, Uri, window, workspace, TreeItem } from 'vscode';
 import { assertWorkspace, awaitClient, dumpJava, findClusters, getFilePaths, openFile, prepareProject, replaceCode } from '../../testutils';
 import { FORMATTED_POM_XML, SAMPLE_CODE_FORMAT_DOCUMENT, SAMPLE_CODE_SORT_IMPORTS, SAMPLE_CODE_UNUSED_IMPORTS } from '../../constants';
-import { extConstants } from '../../../constants';
+import { extCommands } from '../../../commands/commands';
 
 suite('Extension Test Suite', function () {
   window.showInformationMessage('Start all tests.');
@@ -166,7 +166,7 @@ suite('Extension Test Suite', function () {
     if (refactorActions && refactorActions.length > 0) {
       for await (const action of refactorActions) {
         if (action.command && action.command.arguments) {
-          if (action.command.command === extConstants.COMMAND_PREFIX + ".surround.with") {
+          if (action.command.command === extCommands.surroundWith) {
             //this action has a popup where the user needs to
             //select a template that should be used for the surround:
             continue;
@@ -197,7 +197,7 @@ suite('Extension Test Suite', function () {
       assert.strictEqual(tests[1].tests[0].name, 'testTrue', `Invalid test name returned`);
 
       console.log("Test: run all workspace tests");
-      await vscode.commands.executeCommand(extConstants.COMMAND_PREFIX + '.run.test', workspaceFolder.uri.toString());
+      await vscode.commands.executeCommand(extCommands.runTest, workspaceFolder.uri.toString());
       console.log(`Test: run all workspace tests finished`);
     } catch (error) {
       dumpJava();
diff --git a/vscode/src/test/testutils.ts b/vscode/src/test/testutils.ts
@@ -35,6 +35,7 @@ import { NbLanguageClient } from "../lsp/nbLanguageClient";
 import { extConstants } from "../constants";
 import { l10n } from "../localiser";
 import { globalVars } from "../extension";
+import { nbCommands } from "../commands/commands";
 
 /**
  * Folder path currently opened in VSCode workspace
@@ -164,12 +165,12 @@ export async function waitProjectRecognized(someJavaFile: string): Promise<void>
 		const u: vscode.Uri = vscode.Uri.file(someJavaFile);
 		// clear out possible bad or negative caches.
 		return vscode.commands
-			.executeCommand(extConstants.COMMAND_PREFIX + ".clear.project.caches")
+			.executeCommand(nbCommands.clearProjectCaches)
 			.then(
 				// this should assure opening the root with the created project.
 				() =>
 					vscode.commands.executeCommand(
-						extConstants.COMMAND_PREFIX + ".java.get.project.packages",
+						nbCommands.javaProjectPackages,
 						u.toString()
 					)
 			);
diff --git a/vscode/src/views/TestViewController.ts b/vscode/src/views/TestViewController.ts
@@ -24,8 +24,7 @@
 import { commands, debug, tests, workspace, CancellationToken, TestController, TestItem, TestRunProfileKind, TestRunRequest, Uri, TestRun, TestMessage, Location, Position, MarkdownString } from "vscode";
 import * as path from 'path';
 import { asRange, TestCase, TestSuite } from "../lsp/protocol";
-import { extConstants } from "../constants";
-import { extCommands, builtInCommands } from "../commands/commands"
+import { extCommands, builtInCommands, nbCommands } from "../commands/commands"
 
 export class NbTestAdapter {
 
@@ -313,7 +312,7 @@ export class NbTestAdapter {
             }
             const result = regExp.exec(line);
             if (result) {
-                message.appendText(result[1]).appendText('(').appendMarkdown(`[${result[3]}](command:${extConstants.COMMAND_PREFIX}.open.stacktrace?${encodeURIComponent(JSON.stringify([currentTestUri, result[2], result[4], +result[5]]))})`).appendText(')');
+                message.appendText(result[1]).appendText('(').appendMarkdown(`[${result[3]}](command:${nbCommands.openStackTrace}?${encodeURIComponent(JSON.stringify([currentTestUri, result[2], result[4], +result[5]]))})`).appendText(')');
             } else {
                 message.appendText(line);
             }
diff --git a/vscode/src/views/runConfiguration.ts b/vscode/src/views/runConfiguration.ts
@@ -22,6 +22,8 @@
 import * as vscode from 'vscode';
 import { homedir } from 'os';
 import { l10n } from '../localiser';
+import { getConfiguration } from '../configurations/handlers';
+import { configKeys } from '../configurations/configuration';
 
 
 
@@ -108,7 +110,7 @@ class RunConfigurationNode extends vscode.TreeItem {
 	}
 
 	getConfig(): vscode.WorkspaceConfiguration {
-		return vscode.workspace.getConfiguration('jdk.runConfig');
+		return getConfiguration(configKeys.runConfig);
 	}
 
 }

Original file line number	Diff line number	Diff line change
`@@ -83,8 +83,9 @@ export function activate(context: ExtensionContext): VSNetBeansAPI {`
`83`	`83`
`84`	`84`
`85`	`85`	`export function deactivate(): Thenable<void> {`
`86`		`- if (globalVars.nbProcessManager?.getProcess() != null) {`
`87`		`- globalVars.nbProcessManager?.getProcess()?.kill();`
	`86`	`+ const process = globalVars.nbProcessManager?.getProcess();`
	`87`	`+ if (process != null) {`
	`88`	`+ process?.kill();`
`88`	`89`	`}`
`89`	`90`	`return globalVars.clientPromise.stopClient();`
`90`	`91`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,15 +30,18 @@ export class ExtensionLogger {`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`public log(message: string): void {`
`33`		- this.outChannel.appendLine(`[${LogLevel.INFO}]: ${message}`);
	`33`	+ const formattedMessage = `[${LogLevel.INFO}]: ${message}`;
	`34`	`+ this.printLog(formattedMessage);`
`34`	`35`	`}`
`35`	`36`
`36`	`37`	`public warn(message: string): void {`
`37`		- this.outChannel.appendLine(`[${LogLevel.WARN}]: ${message}`);
	`38`	+ const formattedMessage = `[${LogLevel.WARN}]: ${message}`;
	`39`	`+ this.printLog(formattedMessage);`
`38`	`40`	`}`
`39`	`41`
`40`	`42`	`public error(message: string): void {`
`41`		- this.outChannel.appendLine(`[${LogLevel.ERROR}]: ${message}`);
	`43`	+ const formattedMessage = `[${LogLevel.ERROR}]: ${message}`;
	`44`	`+ this.printLog(formattedMessage);`
`42`	`45`	`}`
`43`	`46`
`44`	`47`	`public logNoNL(message: string): void {`
`@@ -56,6 +59,11 @@ export class ExtensionLogger {`
`56`	`59`	`public dispose(): void {`
`57`	`60`	`this.outChannel.dispose();`
`58`	`61`	`}`
	`62`	`+`
	`63`	`+ private printLog(message: string): void{`
	`64`	`+ const timestamp = new Date().toISOString();`
	`65`	+ this.outChannel.appendLine(`[${timestamp}] ${message}`);
	`66`	`+ }`
`59`	`67`	`}`
`60`	`68`
`61`	`69`	`export const LOGGER = new ExtensionLogger(extConstants.SERVER_NAME);`