fix: minor bugs

Author: Lokesh-Garg-22

docs/PROJECT_STRUCTURE.md

@@ -33,7 +33,8 @@ css-modules-intellisense/
 │   │   ├── classNameCache.ts   # Class name caching
 │   │   ├── cssModuleDependencyCache.ts
 │   │   ├── loadCaches.ts       # Cache initialization
-│   │   └── processConfig.ts    # Configuration processing
+│   │   ├── processConfig.ts    # Configuration processing
+│   │   └── vsConfig.ts         # VS Code configuration helpers
 │   ├── providers/              # Language feature providers
 │   │   ├── completionProvider.ts    # Auto-completion
 │   │   ├── definitionProvider.ts    # Go-to-definition
@@ -70,6 +71,7 @@ css-modules-intellisense/
 │   └── scss.tmLanguage.json
 ├── .editorconfig               # Editor configuration
 ├── .gitignore                  # Git ignore rules
+├── .prettierrc                 # Prettier formatting rules
 ├── .markdownlint.json          # Markdown linting rules
 ├── .vscode-test.mjs            # VS Code test configuration
 ├── .vscodeignore               # Files to exclude from extension package
@@ -120,6 +122,7 @@ Static resources including test fixtures and extension icons
 ## Configuration Files
 
 - `.editorconfig` - Code style consistency
+- `.prettierrc` - Code formatting rules
 - `tsconfig.json` - TypeScript compiler settings
 - `eslint.config.mjs` - Linting rules
 - `package.json` - Extension metadata and dependencies

src/libs/classNameCache.ts

@@ -43,7 +43,7 @@ export default class ClassNameCache {
         for (const workspacePath of dependents) {
           const resolvedPath = resolveWorkspaceRelativePath(workspacePath);
           if (!resolvedPath) {
-            return;
+            continue;
           }
           const document = await vscode.workspace.openTextDocument(
             resolvedPath

src/libs/processConfig.ts

@@ -1,5 +1,6 @@
 import * as vscode from "vscode";
 import { CONFIGURATION_KEY, CONFIGURATIONS } from "../config";
+import { getVsConfig } from "./vsConfig";
 
 function registerConditionalListener<T>(
   event: vscode.Event<T>,
@@ -9,8 +10,7 @@ function registerConditionalListener<T>(
   let disposable: vscode.Disposable | undefined;
 
   const applyConfig = () => {
-    const config = vscode.workspace.getConfiguration(CONFIGURATION_KEY);
-    const enabled = config.get<boolean>(settingKey, true);
+    const enabled = getVsConfig().get<boolean>(settingKey, true);
 
     if (enabled && !disposable) {
       disposable = event(func);

src/libs/vsConfig.ts

@@ -0,0 +1,11 @@
+import * as vscode from "vscode";
+import { CONFIGURATION_KEY, CONFIGURATIONS } from "../config";
+
+export const getVsConfig = () =>
+  vscode.workspace.getConfiguration(CONFIGURATION_KEY);
+
+export const getAliasMap = (): Record<string, string> =>
+  getVsConfig().get<Record<string, string>>(CONFIGURATIONS.ALIASES, {});
+
+export const getBlacklistPatterns = (): string[] =>
+  getVsConfig().get<string[]>(CONFIGURATIONS.BLACKLIST_PATTERNS, []);

src/providers/completionProvider.ts

@@ -1,5 +1,5 @@
 import * as vscode from "vscode";
-import { CONFIGURATION_KEY, CONFIGURATIONS, MESSAGES } from "../config";
+import { MESSAGES } from "../config";
 import ClassNameCache from "../libs/classNameCache";
 import isPositionInString from "../utils/isPositionInString";
 import getImportModulePath from "../utils/getImportModulePath";
@@ -9,11 +9,6 @@ import { getWorkspaceRelativeImportPath } from "../utils/getPath";
 export default class CompletionItemProvider
   implements vscode.CompletionItemProvider
 {
-  static config = vscode.workspace.getConfiguration(CONFIGURATION_KEY);
-  static aliasMap = CompletionItemProvider.config.get<Record<string, string>>(
-    CONFIGURATIONS.ALIASES,
-    {}
-  );
 
   provideCompletionItems = async (
     document: vscode.TextDocument,

src/providers/definitionProvider.ts

@@ -96,7 +96,7 @@ export class ModuleDefinitionProvider implements vscode.DefinitionProvider {
     }
 
     const className = cssDoc.getText(wordRange).slice(1); // remove leading dot
-    const cssPath = resolveImportPathWithAliases(cssDoc, cssDoc.uri.path);
+    const cssPath = resolveImportPathWithAliases(cssDoc, cssDoc.uri.fsPath);
     if (!fs.existsSync(cssPath)) {
       return;
     }

src/providers/renameProvider.ts

@@ -41,7 +41,7 @@ const provideRenameEdits = async ({
       const resolvedPath = resolveImportPathWithAliases(doc, match[2]);
 
       if (resolvedPath !== filePath) {
-        return;
+        continue;
       }
 
       const classNamePositions = await getDataOfClassName(

src/types/cache.ts

@@ -39,6 +39,10 @@ export class PathMapCache extends Array<string> {
     });
   }
 
+  hasKey(key: string): boolean {
+    return this.reverseMap.has(key);
+  }
+
   getKeyFormIndex(index: number) {
     return this[index];
   }
@@ -95,8 +99,11 @@ class BaseCache<V extends {}, FC = unknown> {
   }
 
   hasByKey(key: string): boolean {
-    const keyIndex = this.pathMapCache.getIndexFormKey(key);
-    return this.has(keyIndex);
+    if (!this.pathMapCache.hasKey(key)) {
+      return false;
+    }
+
+    return this.has(this.pathMapCache.getIndexFormKey(key));
   }
 
   getByKey(key: string) {
@@ -110,7 +117,7 @@ class BaseCache<V extends {}, FC = unknown> {
   }
 
   setMap(entries: readonly (readonly [string, V])[]) {
-    this.clear;
+    this.clear();
     entries.forEach((entry) => {
       this.setByKey(entry[0], entry[1]);
     });

src/utils/getAllFiles.ts

@@ -3,18 +3,11 @@ import {
   getModuleFileRegex,
   getScriptFileRegex,
 } from "./getFileExtensionRegex";
-import { CONFIGURATION_KEY, CONFIGURATIONS } from "../config";
-
-const config = vscode.workspace.getConfiguration(CONFIGURATION_KEY);
-const blacklistPatterns = config.get<string[]>(
-  CONFIGURATIONS.BLACKLIST_PATTERNS,
-  []
-);
+import { getBlacklistPatterns } from "../libs/vsConfig";
 
 export const getAllScriptFiles = async () => {
   const includePattern = `**/*.{${getScriptFileRegex()}}`;
-
-  const excludePattern = `{${blacklistPatterns.join(",")}}`;
+  const excludePattern = `{${getBlacklistPatterns().join(",")}}`;
 
   const files = await vscode.workspace.findFiles(
     includePattern,
@@ -26,8 +19,7 @@ export const getAllScriptFiles = async () => {
 
 export const getAllModuleFiles = async () => {
   const includePattern = `**/*.module.{${getModuleFileRegex(",")}}`;
-
-  const excludePattern = `{${blacklistPatterns.join(",")}}`;
+  const excludePattern = `{${getBlacklistPatterns().join(",")}}`;
 
   const files = await vscode.workspace.findFiles(
     includePattern,

src/utils/getPath.ts

@@ -1,10 +1,6 @@
 import * as path from "path";
 import * as vscode from "vscode";
-import { CONFIGURATION_KEY, CONFIGURATIONS } from "../config";
-
-// Load alias configuration from workspace settings
-const config = vscode.workspace.getConfiguration(CONFIGURATION_KEY);
-const aliasMap = config.get<Record<string, string>>(CONFIGURATIONS.ALIASES, {});
+import { getAliasMap } from "../libs/vsConfig";
 
 /**
  * Resolves an import path by checking configured aliases and falling back to relative resolution from the current document.
@@ -24,6 +20,7 @@ export const resolveImportPathWithAliases = (
 
   const workspaceRoot = workspaceFolders[0].uri.fsPath;
   let resolvedPath = importPath;
+  const aliasMap = getAliasMap();
 
   // Replace alias with absolute path if match is found
   for (const [alias, aliasRelativePath] of Object.entries(aliasMap)) {