fixes #42

Author: atsyplenkov

CHANGELOG.md

@@ -2,7 +2,14 @@
 
 <!-- Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how to structure this file. -->
 
-## [0.3.0] - 2025-09-12
+## [0.3.2] - 2025-12-22
+
+Patch release that fixes some vulnerabilities in dependencies.
+
+### Fixed
+- quoted values are now parsed correctly (#42)
+
+## [0.3.1] - 2025-09-12
 
 ### Added
 - Support to generate SQL statements for querying and modifying database tables (in #41, thanks to @juarezr)

src/parse-table.js

@@ -219,10 +219,7 @@ function parseTextTable(textString) {
     let regex = new RegExp(pattern, 'gm');
     let matrix = rows.map((row) => row.split(regex));
     let cols = getNumSplitRows(matrix, i);
-    // Check if the pattern perfectly split all rows with same number of columns
-    if (cols[0] >= rlen && cols[1] >= rlen) {
-      return matrix;
-    }
+
     results[i] = matrix;
     columns[i] = cols;
   }
@@ -238,7 +235,14 @@ function parseTextTable(textString) {
       return row.concat(new Array(diff).fill(""));
     }
     return row;
-  });
+  }).map(row => row.map(cell => {
+    // Strip surrounding quotes if present
+    const trimmed = cell.trim();
+    if (trimmed.startsWith('"') && trimmed.endsWith('"') && trimmed.length >= 2) {
+      return trimmed.slice(1, -1);
+    }
+    return cell;
+  }));
   return normalized;
 }
 

test/quoted-values.test.js

@@ -0,0 +1,55 @@
+const assert = require('assert');
+const vscode = require('vscode');
+const parseTable = require('../src/parse-table');
+
+suite('Quoted Values Parsing Test Suite', () => {
+  // Mock configuration
+  const originalGetConfiguration = vscode.workspace.getConfiguration;
+
+  setup(() => {
+    vscode.workspace.getConfiguration = () => ({
+      get: (key) => {
+        switch (key) {
+          case 'decimalPoint': return '10,000.00';
+          case 'defaultConvention': return 'PascalCase';
+          default: return null;
+        }
+      }
+    });
+  });
+
+  teardown(() => {
+    vscode.workspace.getConfiguration = originalGetConfiguration;
+  });
+
+  test('should strip surrounding quotes from numeric values', () => {
+    const input = 'Col1\n"1"';
+    const result = parseTable.parseClipboard(input);
+
+    // Should catch the issue: currently "1" is parsed as string "1" (with quotes likely kept or treated as string)
+    // We expect it to be parsed as a number 1
+    assert.ok(result);
+    assert.strictEqual(result.data[0][0], 1);
+    // If it fails, it probably returns "1" (string) or key remains string type
+  });
+
+  test('should strip surrounding quotes from string values', () => {
+    const input = 'Col1\n"foo"';
+    const result = parseTable.parseClipboard(input);
+
+    assert.ok(result);
+    assert.strictEqual(result.data[0][0], 'foo');
+    // If it fails, it probably returns "\"foo\""
+  });
+
+  test('complex row with mixed quoted types', () => {
+    // From issue description: x Apple Pear Lemon "1"
+    const input = 'x\tApple\tPear\tLemon\n"1"\t"fruit"\t"fruit"\t"fruit"';
+    const result = parseTable.parseClipboard(input);
+
+    assert.ok(result);
+    assert.strictEqual(result.headers[0], 'X');
+    assert.strictEqual(result.data[0][0], 1); // "1" -> 1
+    assert.strictEqual(result.data[0][1], 'fruit'); // "fruit" -> fruit
+  });
+});