wyreframe 0.7.4 → 0.7.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wyreframe",
-  "version": "0.7.4",
+  "version": "0.7.6",
   "description": "ASCII wireframe + interaction DSL to HTML converter with scene transitions",
   "author": "wickedev",
   "repository": {

package/src/parser/Detector/BoxTracer.mjs

@@ -153,17 +153,30 @@ function traceBox(grid, topLeft) {
       };
     }
     let nextRow = topLeft.row + 1 | 0;
+    let prevRow = topLeft.row - 1 | 0;
+    let isMiddleDivider;
+    if (prevRow >= 0) {
+      let prevLeftPos = Types.Position.make(prevRow, topLeft.col);
+      let prevRightPos = Types.Position.make(prevRow, topRightOpt.col);
+      let leftChar = Grid.get(grid, prevLeftPos);
+      let rightChar = Grid.get(grid, prevRightPos);
+      isMiddleDivider = leftChar === "VLine" && typeof leftChar !== "object" ? rightChar === "VLine" && typeof rightChar !== "object" : false;
+    } else {
+      isMiddleDivider = false;
+    }
     let isValidTopEdge;
-    if (nextRow < grid.height) {
+    if (isMiddleDivider || nextRow >= grid.height) {
+      isValidTopEdge = false;
+    } else {
       let nextLeftPos = Types.Position.make(nextRow, topLeft.col);
       let nextRightPos = Types.Position.make(nextRow, topRightOpt.col);
-      let leftChar = Grid.get(grid, nextLeftPos);
-      let rightChar = Grid.get(grid, nextRightPos);
-      if (leftChar !== undefined && typeof leftChar !== "object") {
-        switch (leftChar) {
+      let leftChar$1 = Grid.get(grid, nextLeftPos);
+      let rightChar$1 = Grid.get(grid, nextRightPos);
+      if (leftChar$1 !== undefined && typeof leftChar$1 !== "object") {
+        switch (leftChar$1) {
           case "Corner" :
-            if (rightChar !== undefined && typeof rightChar !== "object") {
-              switch (rightChar) {
+            if (rightChar$1 !== undefined && typeof rightChar$1 !== "object") {
+              switch (rightChar$1) {
                 case "Corner" :
                   isValidTopEdge = false;
                   break;
@@ -175,8 +188,8 @@ function traceBox(grid, topLeft) {
             }
             break;
           case "VLine" :
-            if (rightChar !== undefined && typeof rightChar !== "object") {
-              switch (rightChar) {
+            if (rightChar$1 !== undefined && typeof rightChar$1 !== "object") {
+              switch (rightChar$1) {
                 default:
                   isValidTopEdge = true;
               }
@@ -190,8 +203,6 @@ function traceBox(grid, topLeft) {
       } else {
         isValidTopEdge = true;
       }
-    } else {
-      isValidTopEdge = false;
     }
     if (!isValidTopEdge) {
       return {

package/src/parser/Detector/BoxTracer.res

@@ -237,8 +237,38 @@ let traceBox = (grid: Grid.t, topLeft: Position.t): traceResult => {
           // in the next row at the left and right columns, NOT Corner characters (+).
           // If the next row has corners at both positions, this is likely the
           // bottom edge of one box immediately followed by the top edge of another.
+          //
+          // Issue #20 fix: Also check if this corner is a MIDDLE divider of a table.
+          // A middle divider has VLine characters in the PREVIOUS row at both corners.
+          // Example table structure:
+          //   +-----------+-----------+    <- Row 3: Top edge (valid)
+          //   |  Header1  |  Header2  |    <- Row 4: Content row
+          //   +-----------+-----------+    <- Row 5: Middle divider (INVALID as top edge)
+          //   |  Cell 1   |  Cell 2   |    <- Row 6: Content row
+          //   +-----------+-----------+    <- Row 7: Bottom edge
+          // Row 5 has VLines in both previous (row 4) AND next (row 6) rows,
+          // so it's a middle divider, not a valid top edge.
           let nextRow = topLeft.row + 1
-          let isValidTopEdge = if nextRow < grid.height {
+          let prevRow = topLeft.row - 1
+
+          // Check if previous row has VLines at both corners (indicates middle divider)
+          let isMiddleDivider = if prevRow >= 0 {
+            let prevLeftPos = Position.make(prevRow, topLeft.col)
+            let prevRightPos = Position.make(prevRow, topRight.col)
+            let leftChar = Grid.get(grid, prevLeftPos)
+            let rightChar = Grid.get(grid, prevRightPos)
+            // If previous row has VLine at BOTH positions, this is a middle divider
+            switch (leftChar, rightChar) {
+            | (Some(VLine), Some(VLine)) => true  // Previous row has content = middle divider
+            | _ => false
+            }
+          } else {
+            false  // No previous row = could be valid top edge
+          }
+
+          let isValidTopEdge = if isMiddleDivider {
+            false  // Middle dividers are not valid top edges
+          } else if nextRow < grid.height {
             let nextLeftPos = Position.make(nextRow, topLeft.col)
             let nextRightPos = Position.make(nextRow, topRight.col)
             let leftChar = Grid.get(grid, nextLeftPos)

package/src/parser/Semantic/SemanticParser.mjs

@@ -642,6 +642,17 @@ function parseContentLine(line, lineIndex, contentStartRow, box, registry) {
   if (trimmed === "") {
     let row = contentStartRow + lineIndex | 0;
     let baseCol = box.bounds.left + 1 | 0;
+    let rowWithinChildBox = box.children.some(child => {
+      let b = child.bounds;
+      if (row >= b.top) {
+        return row <= b.bottom;
+      } else {
+        return false;
+      }
+    });
+    if (rowWithinChildBox) {
+      return;
+    }
     let position = Types.Position.make(row, baseCol);
     return {
       TAG: "Spacer",

package/src/parser/Semantic/SemanticParser.res

@@ -1083,11 +1083,25 @@ let parseContentLine = (
   let trimmed = line->String.trim
 
   // Issue #16: Preserve empty lines as Spacer elements for vertical spacing
+  // Issue #19: But don't create Spacers for rows that are within child box bounds
   if trimmed === "" {
     let row = contentStartRow + lineIndex
     let baseCol = box.bounds.left + 1
-    let position = Position.make(row, baseCol)
-    Some(Spacer({position: position}))
+
+    // Check if this row is within any child box's vertical bounds
+    // If so, skip creating a Spacer (Issue #19: incorrect spacer count)
+    let rowWithinChildBox = box.children->Array.some(child => {
+      let b = child.bounds
+      row >= b.top && row <= b.bottom
+    })
+
+    if rowWithinChildBox {
+      // Skip spacer for rows occupied by child boxes
+      None
+    } else {
+      let position = Position.make(row, baseCol)
+      Some(Spacer({position: position}))
+    }
   } else if isNoiseText(trimmed) {
     // Filter out border/noise text - don't create any element
     None