lint

Author: mscolnick

frontend/src/core/codemirror/language/languages/sql/completion-builder.ts

@@ -53,6 +53,7 @@ export class CompletionBuilder {
    * Set a value at a nested path, creating intermediate objects as needed
    */
   private setAt(path: string[], value: SQLNamespace): void {
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
     let current: any = this.schema;
     for (const key of path.slice(0, -1)) {
       if (!current[key]) {
@@ -66,7 +67,7 @@ export class CompletionBuilder {
   /**
    * Build the final schema
    */
-  build(): Record<string, any> {
+  build(): SQLNamespace {
     return this.schema;
   }
 

frontend/src/core/codemirror/language/languages/sql/completion-store.ts

@@ -42,7 +42,7 @@ class SQLCompletionStore {
   }
 
   private getConnectionSchema(connection: DataSourceConnection): CachedSchema {
-    const { default_database, databases } = connection;
+    const { default_database, databases, default_schema } = connection;
     const builder = new CompletionBuilder();
 
     // When there is only one database, it is the default
@@ -89,7 +89,7 @@ class SQLCompletionStore {
     }
 
     // Otherwise, we need to use the fully qualified name
-    for (const database of connection.databases) {
+    for (const database of databases) {
       // Skip the default database, since we already added it
       if (database.name === defaultDb?.name) {
         continue;
@@ -112,7 +112,7 @@ class SQLCompletionStore {
     return {
       shouldAddLocalTables: true,
       schema: builder.build(),
-      defaultSchema: connection.default_schema ?? undefined,
+      defaultSchema: default_schema ?? undefined,
     };
   }