Merge pull request #5 from JoelMiller74/update-codecov-results

Update CI configuration and enhance test coverage reporting

Author: JoelMiller74

.github/workflows/ci.yml

@@ -15,6 +15,8 @@ jobs:
     steps:
       - name: Checkout
         uses: actions/checkout@v4
+        with:
+          fetch-depth: 2
 
       - name: Setup Node.js
         uses: actions/setup-node@v4
@@ -55,8 +57,11 @@ jobs:
         run: npm run coverage --if-present
 
       - name: Upload coverage to Codecov
-        uses: codecov/codecov-action@v4
+        uses: codecov/codecov-action@v5
+        with:
+          token: ${{ secrets.CODECOV_TOKEN }}
+      - name: Upload test results to Codecov
+        if: ${{ !cancelled() }}
+        uses: codecov/test-results-action@v1
         with:
-          files: ./coverage/lcov.info,./coverage/coverage-final.json
-          fail_ci_if_error: true
-          verbose: true
+          token: ${{ secrets.CODECOV_TOKEN }}

src/formatter/engine.ts

@@ -268,7 +268,7 @@ export function formatTsql(text: string, { options, config, profile }: EngineCon
   // DDL: align CREATE TABLE column definitions (basic)
   const alignCols = config.get?.('alignColumnDefinitions', true);
   if (alignCols) {
-    out = out.replace(/CREATE\s+TABLE\s+([\s\S]*?)\(([\s\S]*?)\)/gi, (m, tbl, cols) => {
+    out = out.replace(/CREATE\s+TABLE\s+([\s\S]*?)\s*\(((?:[^()]*|\([^()]*\))*)\)/gi, (m, tbl, cols) => {
       const lines = cols.split(',').map((s: string) => s.trim());
       const maxName = Math.max(...lines.map((l: string) => (l.split(/\s+/)[0] || '').length));
       const padded = lines.map((l: string) => {

src/test/formatter.vitest.ts

@@ -205,7 +205,8 @@ describe('Formatter Engine (Vitest)', () => {
   it('align column definitions true', () => {
     const t = 'CREATE TABLE t (id INT, name VARCHAR(50))';
     const out = formatTsql(t, { options: {} as any, config: cfg({ alignColumnDefinitions: true }), profile: {} });
-    expect(out).toBe('CREATE TABLE t (\n  id   INT,\n  name VARCHAR(50)\n))');
+    console.log('align-cols OUT:\n' + out);
+    expect(out).toBe('CREATE TABLE t (\n  id   INT,\n  name VARCHAR(50)\n)');
   });
 
   it('bracket identifiers remove', () => {