remove extra logic

Signed-off-by: Ihor Dykhta <dikhta.igor@gmail.com>

Author: igorDykhta

src/duckdb/src/table/duckdb-table-utils.ts

@@ -75,19 +75,12 @@ export function quoteTableName(columnName: string): string {
     return columnName;
   }
 
-  // If it contains dots and quotes, it might be a fully qualified name
+  // If it contains both dots and quotes, assume it's a qualified name and return as-is
+  // This avoids complex parsing that breaks with dots inside quotes like "my.schema"."table"
   if (columnName.includes('.') && columnName.includes('"')) {
-    // Simple heuristic: if it looks like "part1"."part2" or "part1"."part2"."part3"
-    // Split on dots and check if each part is quoted
-    const parts = columnName.split('.');
-    const allPartsQuoted = parts.every(
-      part => part.trim().startsWith('"') && part.trim().endsWith('"') && part.trim().length > 2
-    );
-
-    if (allPartsQuoted && parts.length >= 2) {
-      return columnName;
-    }
+    return columnName;
   }
+
   return `"${columnName.replace(/"/g, '""')}"`;
 }
 

test/node/utils/duckdb-utils-test.js

@@ -514,19 +514,30 @@ test('duckdb-utils -> quoteTableName', t => {
     'should preserve qualified name with special chars'
   );
 
+  // Test qualified names with dots inside quotes (would break old parsing logic)
+  t.equal(
+    quoteTableName('"my.schema"."table.name"'),
+    '"my.schema"."table.name"',
+    'should preserve qualified name with dots inside quotes'
+  );
+
   // Test invalid fully qualified names (should get quoted)
   t.equal(quoteTableName('schema.table'), '"schema.table"', 'should quote unquoted qualified name');
   t.equal(
     quoteTableName('"schema".table'),
-    '"""schema"".table"',
-    'should quote partially quoted name'
+    '"schema".table',
+    'should preserve partially quoted name (trust user intent)'
   );
   t.equal(
     quoteTableName('schema."table"'),
-    '"schema.""table"""',
-    'should quote partially quoted name'
+    'schema."table"',
+    'should preserve partially quoted name (trust user intent)'
+  );
+  t.equal(
+    quoteTableName('bad."format'),
+    'bad."format',
+    'should preserve name with dots and quotes (trust user intent)'
   );
-  t.equal(quoteTableName('bad."format'), '"bad.""format"', 'should quote malformed qualified name');
 
   // Test edge cases
   t.equal(quoteTableName(''), '""', 'should handle empty string');