fix(workspace): ensure catalog promise always resolves and dedupe change events (#79)

Author: 9romise

src/composables/workspace-context.ts

@@ -30,8 +30,14 @@ export function useWorkspaceContext() {
   }
 
   useDisposable(workspace.onDidChangeTextDocument(({ document }) => {
-    if (document !== window.activeTextEditor?.document)
+    const activeEditor = window.activeTextEditor
+    if (
+      !activeEditor
+      || document !== activeEditor.document
+      || document.version === activeEditor.document.version
+    ) {
       return
+    }
 
     deleteCacheByUri(document.uri, false)
   }))

src/core/workspace.ts

@@ -54,20 +54,21 @@ class WorkspaceContext {
   }
 
   async loadWorkspace() {
-    this.#catalogs = undefined
+    this.#catalogs = Promise.withResolvers()
     this.packageManager = await getPackageManager(this.folder.uri)
 
     logger.info(`[workspace-context] detect package manager: ${this.packageManager}`)
 
     if (this.packageManager !== 'npm') {
-      this.#catalogs = Promise.withResolvers()
       const workspaceFilename = workspaceFileMapping[this.packageManager]
       this.workspaceFileUri = Uri.joinPath(this.folder.uri, workspaceFilename)
       this.#catalogs.resolve(
         await accessOk(this.workspaceFileUri)
           ? (await this.loadWorkspaceCatalogInfo(this.workspaceFileUri))?.catalogs
           : undefined,
       )
+    } else {
+      this.#catalogs.resolve(undefined)
     }
   }
 
@@ -85,7 +86,7 @@ class WorkspaceContext {
   }
 
   async getCatalogs(): Promise<CatalogsInfo | undefined> {
-    return this.#catalogs?.promise
+    return this.#catalogs!.promise
   }
 
   #createResolvedDependencyInfo(dependency: DependencyInfo, catalogs?: CatalogsInfo): ResolvedDependencyInfo {
@@ -131,7 +132,7 @@ class WorkspaceContext {
 
     const [info, catalogs] = await Promise.all([
       getDocumentText(uri).then((text) => extractor.getPackageManifestInfo(text)),
-      this.#catalogs?.promise,
+      this.getCatalogs(),
     ])
 
     if (!info)