chore: move path.dirname out of considerAdditionalPath

atom-community · Jun 10, 2021 · dba8dc1 · dba8dc1
1 parent 63f7de8
commit dba8dc1
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/lib/auto-languageclient.ts b/lib/auto-languageclient.ts
@@ -707,7 +707,7 @@ export default class AutoLanguageClient {
 
     if (query !== null && this.serversSupportDefinitionDestinations() && server.additionalPaths !== undefined) {
       for (const def of query.definitions) {
-        considerAdditionalPath(server as ActiveServer & { additionalPaths: Set<string> }, def.path)
+        considerAdditionalPath(server as ActiveServer & { additionalPaths: Set<string> }, path.dirname(def.path))
       }
     }
 

diff --git a/lib/server-manager.ts b/lib/server-manager.ts
@@ -347,8 +347,11 @@ export function normalizePath(projectPath: string): string {
 }
 
 /** Considers a path for inclusion in `additionalPaths`. */
-export function considerAdditionalPath(server: ActiveServer & { additionalPaths: Set<string> }, additionalPath: string): void {
+export function considerAdditionalPath(
+  server: ActiveServer & { additionalPaths: Set<string> },
+  additionalPath: string
+): void {
   if (!additionalPath.startsWith(server.projectPath)) {
-    server.additionalPaths.add(path.dirname(additionalPath))
+    server.additionalPaths.add(additionalPath)
   }
 }