完善代码（避免空调用）

Author: rqaefaqa

src/PawSQLSidebarProvider.ts

@@ -8,6 +8,7 @@ import {
 import { LanguageService } from "./LanguageService";
 import * as path from "path";
 import { getUrls } from "./constants";
+import { isBlank } from "./utils/pawsqlUtils";
 
 // Previous classes remain unchanged...
 class WorkspaceManagerItem extends vscode.TreeItem {
@@ -350,7 +351,7 @@ export class PawSQLTreeProvider
     try {
       // Reset all validation states first
       this.isConfigValid = false;
-      if (!backendUrl || !frontendUrl || !apiKey) {
+      if (isBlank(backendUrl) || isBlank(frontendUrl) || isBlank(apiKey)) {
         this.isConfigValid = false;
         vscode.commands.executeCommand("setContext", "isConfigured", false);
         this.workspaces = [];
@@ -441,14 +442,14 @@ export class PawSQLTreeProvider
       // Reset all validation states first
       this.isConfigValid = false;
 
-      if (!backendUrl || !frontendUrl || !apiKey) {
+      if (isBlank(backendUrl) || isBlank(frontendUrl) || isBlank(apiKey)) {
         this.isConfigValid = false;
         vscode.commands.executeCommand("setContext", "isConfigured", false);
         this.workspaces = [];
         return false;
       }
 
-      const backendResult = await validateBackend(backendUrl);
+      const backendResult = await validateBackend(backendUrl ?? "");
       const isBackendConnected = backendResult.isAvailable;
 
       if (!isBackendConnected) {
@@ -458,7 +459,7 @@ export class PawSQLTreeProvider
         return false;
       }
 
-      const frontendReuslt = await validateFrontend(frontendUrl);
+      const frontendReuslt = await validateFrontend(frontendUrl ?? "");
       const isFrontendConnected = frontendReuslt.isAvailable;
 
       if (!isFrontendConnected) {
@@ -469,7 +470,7 @@ export class PawSQLTreeProvider
         return false;
       }
 
-      const isApikeyValid = await ApiService.validateUserKey(apiKey);
+      const isApikeyValid = await ApiService.validateUserKey(apiKey ?? "");
 
       if (!isApikeyValid) {
         this.isConfigValid = false;
@@ -523,6 +524,10 @@ export class PawSQLTreeProvider
     const apiKey = config.get<string>("apiKey") ?? "";
     const frontendUrl = config.get<string>("frontendUrl") ?? "";
     const backendUrl = config.get<string>("backendUrl") ?? "";
+    if (isBlank(backendUrl) || isBlank(frontendUrl) || isBlank(apiKey)) {
+      return false;
+    }
+
     let failedCommand = [];
     try {
       const [backendResult, frontendResult, isApikeyValid] = await Promise.all([
@@ -569,6 +574,16 @@ export class PawSQLTreeProvider
   public async validateConfigurationByKey(key: string): Promise<void> {
     const config = vscode.workspace.getConfiguration("pawsql");
     const value = config.get<string>(key);
+    if (isBlank(value)) {
+      this.workspaces = [];
+      this.isConfigValid = false;
+      vscode.commands.executeCommand(
+        "setContext",
+        "isConfigured",
+        this.isConfigValid
+      );
+      return;
+    }
     try {
       if (key === "backendUrl") {
         // Validate backend connectivity

src/SqlCodeLensProvider.ts

@@ -3,6 +3,7 @@ import { ConfigurationService } from "./configurationService";
 import { LanguageService } from "./LanguageService";
 import { ApiService, WorkspaceItem } from "./apiService";
 import parse from "./utils/parse";
+import { isBlank } from "./utils/pawsqlUtils";
 
 interface OptimizingRange {
   range: vscode.Range;
@@ -73,13 +74,24 @@ export class SqlCodeLensProvider implements vscode.CodeLensProvider {
     try {
       this.cleanupStaleOptimizingStates();
 
+      const config = vscode.workspace.getConfiguration("pawsql");
+
+      const apiKey = config.get<string>("apiKey");
+      let isApikeyValid = false;
+      if (isBlank(apiKey)) {
+        isApikeyValid = false;
+      } else {
+        isApikeyValid = await ApiService.validateUserKey(apiKey ?? "");
+      }
+
       const codeLenses: vscode.CodeLens[] = [];
       const fileWorkspace = await this.addFileConfigurationLens(
         document,
-        codeLenses
+        codeLenses,
+        isApikeyValid
       );
 
-      if (!token.isCancellationRequested) {
+      if (isApikeyValid && !token.isCancellationRequested) {
         await this.addSqlQueryLenses(
           document,
           codeLenses,
@@ -97,31 +109,34 @@ export class SqlCodeLensProvider implements vscode.CodeLensProvider {
 
   private async addFileConfigurationLens(
     document: vscode.TextDocument,
-    codeLenses: vscode.CodeLens[]
+    codeLenses: vscode.CodeLens[],
+    isApikeyValid: boolean
   ): Promise<WorkspaceItem | null> {
     const fileUri = document.uri.toString();
     const fileWorkspace = await ConfigurationService.getFileDefaultWorkspace(
       fileUri
     );
+
     const config = vscode.workspace.getConfiguration("pawsql");
+
     const defaultWorkspace = config.get<WorkspaceItem>("defaultWorkspace");
-    const apiKey = config.get<string>("apiKey");
 
     const separatorRange = new vscode.Range(0, 0, 1, 0);
-    const isApikeyValid = await ApiService.validateUserKey(apiKey ?? "");
 
     if (!isApikeyValid) {
       codeLenses.push(this.createInitConfigLens(separatorRange, document));
     } else if (fileWorkspace?.workspaceId || defaultWorkspace?.workspaceId) {
       codeLenses.push(
-        this.createWorkspaceLens(
+        this.createAnalysisWithDefaultWorkspaceLens(
           separatorRange,
           document,
           fileWorkspace ?? defaultWorkspace!
         )
       );
     } else {
-      codeLenses.push(this.createSelectWorkspaceLens(separatorRange, document));
+      codeLenses.push(
+        this.createAnalysisWithSelectWorkspaceLens(separatorRange, document)
+      );
     }
 
     return fileWorkspace;
@@ -184,7 +199,7 @@ export class SqlCodeLensProvider implements vscode.CodeLensProvider {
     });
   }
 
-  private createWorkspaceLens(
+  private createAnalysisWithDefaultWorkspaceLens(
     range: vscode.Range,
     document: vscode.TextDocument,
     workspace: Workspace
@@ -198,7 +213,7 @@ export class SqlCodeLensProvider implements vscode.CodeLensProvider {
     });
   }
 
-  private createSelectWorkspaceLens(
+  private createAnalysisWithSelectWorkspaceLens(
     range: vscode.Range,
     document: vscode.TextDocument
   ): vscode.CodeLens {