Merge branch 'next'

Author: kellyjosephprice

.github/workflows/ci.yml

@@ -46,7 +46,7 @@ jobs:
         run: make ci
 
       - name: Upload snapshot diffs
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v5
         if: ${{ failure() }}
         with:
           name: snapshots-diffs
@@ -58,7 +58,7 @@ jobs:
         run: make updateSnapshot
 
       - name: Upload snapshots
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v5
         if: ${{ failure() }}
         with:
           name: image-snapshots

__tests__/migration/tables.test.ts

@@ -1,6 +1,21 @@
 import { migrate } from '../helpers';
 
 describe('mdx migration of tables', () => {
+  it('compiles tables with empty cells', () => {
+    const md = `
+| Name | Date |
+| ---- | ---- |
+| test |      |
+`;
+    const mdx = migrate(md);
+    expect(mdx).toMatchInlineSnapshot(`
+      "| Name | Date |
+      | ---- | ---- |
+      | test |      |
+      "
+    `);
+  });
+
   it('compiles tables with newlines and inline code', () => {
     const md = `
 [block:parameters]

processor/transform/tables-to-jsx.ts

@@ -34,6 +34,9 @@ const visitor = (table: Table, index: number, parent: Parents) => {
         ? (cell.children[0] as unknown as Paragraph).children[0]
         : cell.children[0];
 
+    // If cell is empty there's nothing to visit
+    if (!content) return EXIT;
+
     // @note: Compatibility with RDMD. Ideally, I'd put this in a separate
     // transformer, but then there'd be some duplication.
     visit(cell, 'break', (_, breakIndex, breakParent) => {