refactor: remove constructor visibility in ModuleParser and update error message in ConfigurationManager

#1 (review)

Author: ketupia

src/parsers/moduleParser.ts

@@ -49,7 +49,6 @@ export function getAllAvailableConfigurations(): ModuleInterface[] {
  */
 export class ModuleParser implements Parser {
   private static instance: ModuleParser;
-  private constructor() {}
 
   static getInstance(): ModuleParser {
     if (!ModuleParser.instance) {
@@ -102,7 +101,6 @@ export const moduleParser = ModuleParser.getInstance();
  * Extract code lenses from matched modules and parsed sections.
  * These will be shown in the editor as clickable links to documentation.
  */
-// For testing only
 export function extractCodeLenses(
   source: string,
   matchedModules: ModuleInterface[]

src/utils/config.ts

@@ -52,7 +52,7 @@ export class ConfigurationManager {
       case "enableHoverInfo":
         return config.get("enableHoverInfo", false) as AshStudioConfig[T];
       default:
-        throw new Error(`Unknown configuration key: ${String(key)}`);
+        throw new Error(`Unknown configuration key: ${key}`);
     }
   }