fix-md-tables 1.1.1 → 1.1.3

package/README.md

@@ -27,7 +27,7 @@ Renders misaligned:
 Add ideographic spaces (U+3000) to cells with fewer emoji. Ideographic spaces also display as 2 columns, compensating for the width difference:
 
 ```markdown
-| Status　 | Description　  | Comments　 |
+| Status　 | Description　  | Comments   |
 | -------- | -------------- | ---------- |
 | ✅       | ✅ Complete    | ❌         |
 | 🚧       | 🚧 In Progress | ⚠️         |
@@ -134,9 +134,9 @@ Main function to fix table alignment in markdown content.
 
 Count emoji characters in a string.
 
-### `stripIdeographicSpaces(str: string): string`
+### `normalizeIdeographicSpaces(str: string): string`
 
-Remove all ideographic spaces from a string.
+Replace all ideographic spaces with 2 regular spaces (same visual width).
 
 ### `processTable(tableRows: string[]): string[]`
 

package/lib/index.mjs

@@ -28,9 +28,9 @@ export function countEmoji(str) {
   return (str.match(EMOJI_REGEX) || []).length;
 }
 
-/** Remove all ideographic spaces from a string */
-export function stripIdeographicSpaces(str) {
-  return str.replaceAll("\u3000", "");
+/** Replace all ideographic spaces with 2 regular spaces (same visual width) */
+export function normalizeIdeographicSpaces(str) {
+  return str.replaceAll("\u3000", "  ");
 }
 
 /** Check if a line is a markdown table separator */
@@ -78,8 +78,11 @@ export function compensateSeparatorCell(cell, compensation) {
   // Remove 2 dashes per ideographic space (both are 2 cols visually)
   const dashesToRemove = compensation * 2;
   const newDashes = dashes.length > dashesToRemove ? dashes.slice(0, -dashesToRemove) : dashes;
+  // Also remove trailing spaces to maintain column width
+  const spacesToRemove = compensation * 2;
+  const newTrailingSpace = trailingSpace.length > spacesToRemove ? trailingSpace.slice(spacesToRemove) : "";
 
-  return leadingSpace + leftColon + newDashes + IDEOGRAPHIC_SPACE.repeat(compensation) + rightColon + trailingSpace;
+  return leadingSpace + leftColon + newDashes + IDEOGRAPHIC_SPACE.repeat(compensation) + rightColon + newTrailingSpace;
 }
 
 /** Split cell into [leadingSpace, content, trailingSpace] without regex (ReDoS-safe) */
@@ -124,25 +127,53 @@ export function buildTableRow(cells) {
   return "|" + cells.join("|") + "|";
 }
 
+/** Count ideographic spaces in a string */
+export function countIdeographicSpaces(str) {
+  return (str.match(/\u3000/g) || []).length;
+}
+
 /** Process a complete table, applying emoji compensation to all rows */
 export function processTable(tableRows) {
-  // Strip existing ideographic spaces to start fresh
-  const cleanedRows = tableRows.map(stripIdeographicSpaces);
-  const parsedRows = cleanedRows.map(parseTableRow);
+  // Parse original rows to count existing ideographic spaces
+  const originalParsedRows = tableRows.map(parseTableRow);
+
+  // Normalize for emoji counting (but we'll use original cells when adequate)
+  const cleanedRows = tableRows.map(normalizeIdeographicSpaces);
+  const cleanedParsedRows = cleanedRows.map(parseTableRow);
 
   // Check if table has any emoji
   const hasEmoji = cleanedRows.some((row) => countEmoji(row) > 0);
-  if (!hasEmoji || parsedRows.length < 2) {
+  if (!hasEmoji || cleanedParsedRows.length < 2) {
     return cleanedRows;
   }
 
-  const numCols = Math.max(...parsedRows.map((r) => r.length));
-  const maxEmojiPerCol = calculateMaxEmojiPerColumn(parsedRows, numCols);
+  const numCols = Math.max(...cleanedParsedRows.map((r) => r.length));
+  const maxEmojiPerCol = calculateMaxEmojiPerColumn(cleanedParsedRows, numCols);
 
   // Rebuild all rows with compensation
-  return parsedRows.map((row, rowIdx) => {
+  return originalParsedRows.map((originalRow, rowIdx) => {
+    const cleanedRow = cleanedParsedRows[rowIdx];
     const isSeparatorRow = rowIdx === 1;
-    const processedCells = row.map((cell, col) => processCell(cell, col, isSeparatorRow, maxEmojiPerCol));
+
+    const processedCells = originalRow.map((originalCell, col) => {
+      const cleanedCell = cleanedRow[col];
+      const existingCompensation = countIdeographicSpaces(originalCell);
+      const maxEmoji = maxEmojiPerCol[col] || 0;
+      const cellEmoji = countEmoji(cleanedCell);
+      const neededCompensation = maxEmoji - cellEmoji;
+
+      // If existing compensation is adequate, keep original cell
+      if (existingCompensation >= neededCompensation) {
+        return originalCell;
+      }
+
+      // Need more compensation - use normalized cell and add what's needed
+      const additionalCompensation = neededCompensation - existingCompensation;
+      return isSeparatorRow
+        ? compensateSeparatorCell(cleanedCell, additionalCompensation)
+        : compensateRegularCell(cleanedCell, additionalCompensation);
+    });
+
     return buildTableRow(processedCells);
   });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fix-md-tables",
-  "version": "1.1.1",
+  "version": "1.1.3",
   "description": "Fix markdown table alignment for emoji using ideographic spaces",
   "type": "module",
   "bin": {