fix: use 1-indexed line numbers in error/warning positions (#9)

Changed adjustLineOffset to always add +1 when adjusting positions,
converting from 0-indexed grid rows to 1-indexed file line numbers.

Before: position.row = gridRow + lineOffset (0-indexed)
After:  position.row = gridRow + lineOffset + 1 (1-indexed)

This ensures warnings like "MisalignedClosingBorder" report the actual
file line number users see in their editor, making debugging easier.

Example for a misalignment on Line 5 with scene directive:
- Grid row 3 (0-indexed in grid)
- lineOffset = 1 (for @scene directive)
- position.row = 3 + 1 + 1 = 5 (matches editor Line 5)

Updated E2E-13 test to verify position.row = 5 for Line 5.

Closes #9

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: wickedev

src/parser/Errors/ErrorTypes.res

@@ -217,18 +217,21 @@ let adjustCodeLineOffset = (code: errorCode, offset: int): errorCode => {
  * Adjust the line offset in an error's position.
  * Creates a new error with the position adjusted by the given offset.
  *
+ * This function always adds +1 to convert from 0-indexed grid rows
+ * to 1-indexed file line numbers, plus any offset for stripped directive lines.
+ *
  * @param error - The error to adjust
- * @param offset - The number of lines to add to the row
- * @returns A new error with adjusted position
+ * @param offset - The number of directive lines stripped before grid processing
+ * @returns A new error with 1-indexed line position
  */
 let adjustLineOffset = (error: t, offset: int): t => {
-  if offset === 0 {
-    error
-  } else {
-    {
-      code: adjustCodeLineOffset(error.code, offset),
-      severity: error.severity,
-      context: error.context,
-    }
+  // Always add 1 to convert from 0-indexed to 1-indexed line numbers
+  // offset accounts for stripped directive lines (e.g., @scene: login)
+  // +1 converts from 0-indexed grid rows to 1-indexed file lines
+  let totalOffset = offset + 1
+  {
+    code: adjustCodeLineOffset(error.code, totalOffset),
+    severity: error.severity,
+    context: error.context,
   }
 }

src/parser/__tests__/WyreframeParser_integration.test.res

@@ -1022,22 +1022,21 @@ describe("E2E-13: Line number offset in warnings with scene directives (Issue #5
         // We MUST have a misaligned warning for this test to be valid
         t->expect(Array.length(misalignedWarnings))->Expect.toBe(1)
 
-        // Verify the line number is correct
+        // Verify the line number is correct (1-indexed)
         switch misalignedWarnings->Array.get(0) {
         | Some(warning) => {
             switch warning.code {
             | ErrorTypes.MisalignedClosingBorder({position, _}) => {
-                // Position.row is 0-indexed:
-                // - Line 1 (0-idx 0): @scene: login - directive, stripped
-                // - Line 2 (0-idx 1): empty - first content line, lineOffset = 1
-                // - Line 3 (0-idx 2): +---+ - grid row 1 (bounds.top)
-                // - Line 4 (0-idx 3): |...| - grid row 2
-                // - Line 5 (0-idx 4): misaligned - grid row 3
-                // - Line 6 (0-idx 5): +---+ - grid row 4 (bounds.bottom)
+                // Position.row is 1-indexed for user display:
+                // - Line 1: @scene: login (directive, stripped - lineOffset = 1)
+                // - Line 2: empty (first content line - grid row 0)
+                // - Line 3: +---+ (grid row 1)
+                // - Line 4: |...| (grid row 2)
+                // - Line 5: misaligned (grid row 3) ← warning points here
+                // - Line 6: +---+ (grid row 4)
                 //
-                // Grid row 3 + lineOffset 1 = row 4 (0-indexed)
-                // Which is Line 5 (1-indexed)
-                t->expect(position.row)->Expect.toBe(4)
+                // Formula: position.row = gridRow + lineOffset + 1 = 3 + 1 + 1 = 5
+                t->expect(position.row)->Expect.toBe(5)
               }
             | _ => t->expect(true)->Expect.toBe(false) // Should not reach here
             }