From edc6e477d93283084b738d3ae85e5dd7c6e99536 Mon Sep 17 00:00:00 2001
From: Balint Gabor <127662+gbalint@users.noreply.github.com>
Date: Thu, 20 Jun 2024 10:58:32 +0200
Subject: [PATCH] Fix off by one error with line numbers (#6007)

**Problem:**
Babel starts line numbers from 1, vscode starts them from 0.

**Fix:**
Subtract 1 from the Babel line numbers before storing it in the Bounds
object

**Manual Tests:**
I hereby swear that:

- [x] I opened a hydrogen project and it loaded
- [x] I could navigate to various routes in Preview mode
---
 .../component-descriptor-parser.ts            |  8 +--
 .../property-controls-local.spec.tsx          | 64 +++++++++----------
 2 files changed, 36 insertions(+), 36 deletions(-)

diff --git a/editor/src/core/property-controls/component-descriptor-parser.ts b/editor/src/core/property-controls/component-descriptor-parser.ts
index f6551afd86de..aaaaa674facb 100644
--- a/editor/src/core/property-controls/component-descriptor-parser.ts
+++ b/editor/src/core/property-controls/component-descriptor-parser.ts
@@ -75,10 +75,10 @@ export function generateComponentBounds(
           const { loc } = component
           if (loc != null) {
             componentBoundsByModule[moduleName][componentName] = {
-              startLine: loc.start.line,
-              startCol: loc.start.column,
-              endLine: loc.end.line,
-              endCol: loc.end.column,
+              startLine: loc.start.line - 1,
+              startCol: loc.start.column - 1,
+              endLine: loc.end.line - 1,
+              endCol: loc.end.column - 1,
             }
           }
         })
diff --git a/editor/src/core/property-controls/property-controls-local.spec.tsx b/editor/src/core/property-controls/property-controls-local.spec.tsx
index f337b0da4261..21c97d7bf46f 100644
--- a/editor/src/core/property-controls/property-controls-local.spec.tsx
+++ b/editor/src/core/property-controls/property-controls-local.spec.tsx
@@ -142,10 +142,10 @@ describe('registered property controls', () => {
           },
           "source": Object {
             "bounds": Object {
-              "endCol": 5,
-              "endLine": 42,
-              "startCol": 4,
-              "startLine": 5,
+              "endCol": 4,
+              "endLine": 41,
+              "startCol": 3,
+              "startLine": 4,
             },
             "sourceDescriptorFile": "/utopia/components.utopia.js",
             "type": "DESCRIPTOR_FILE",
@@ -378,10 +378,10 @@ describe('registered property controls', () => {
           },
           "source": Object {
             "bounds": Object {
-              "endCol": 5,
-              "endLine": 16,
-              "startCol": 4,
-              "startLine": 6,
+              "endCol": 4,
+              "endLine": 15,
+              "startCol": 3,
+              "startLine": 5,
             },
             "sourceDescriptorFile": "/utopia/components.utopia.js",
             "type": "DESCRIPTOR_FILE",
@@ -1011,10 +1011,10 @@ describe('registered property controls', () => {
           },
           "source": Object {
             "bounds": Object {
-              "endCol": 5,
-              "endLine": 32,
-              "startCol": 4,
-              "startLine": 5,
+              "endCol": 4,
+              "endLine": 31,
+              "startCol": 3,
+              "startLine": 4,
             },
             "sourceDescriptorFile": "/utopia/components.utopia.js",
             "type": "DESCRIPTOR_FILE",
@@ -1931,10 +1931,10 @@ describe('registered property controls', () => {
             },
             "source": Object {
               "bounds": Object {
-                "endCol": 5,
-                "endLine": 34,
-                "startCol": 4,
-                "startLine": 6,
+                "endCol": 4,
+                "endLine": 33,
+                "startCol": 3,
+                "startLine": 5,
               },
               "sourceDescriptorFile": "/utopia/components.utopia.js",
               "type": "DESCRIPTOR_FILE",
@@ -2688,10 +2688,10 @@ describe('Lifecycle management of registering components', () => {
         .toMatchInlineSnapshot(`
         Object {
           "bounds": Object {
-            "endCol": 5,
-            "endLine": 12,
-            "startCol": 4,
-            "startLine": 5,
+            "endCol": 4,
+            "endLine": 11,
+            "startCol": 3,
+            "startLine": 4,
           },
           "sourceDescriptorFile": "/utopia/components1.utopia.js",
           "type": "DESCRIPTOR_FILE",
@@ -2705,10 +2705,10 @@ describe('Lifecycle management of registering components', () => {
         .toMatchInlineSnapshot(`
         Object {
           "bounds": Object {
-            "endCol": 5,
-            "endLine": 12,
-            "startCol": 4,
-            "startLine": 5,
+            "endCol": 4,
+            "endLine": 11,
+            "startCol": 3,
+            "startLine": 4,
           },
           "sourceDescriptorFile": "/utopia/components2.utopia.js",
           "type": "DESCRIPTOR_FILE",
@@ -2746,10 +2746,10 @@ describe('Lifecycle management of registering components', () => {
         .toMatchInlineSnapshot(`
         Object {
           "bounds": Object {
-            "endCol": 5,
-            "endLine": 12,
-            "startCol": 4,
-            "startLine": 5,
+            "endCol": 4,
+            "endLine": 11,
+            "startCol": 3,
+            "startLine": 4,
           },
           "sourceDescriptorFile": "/utopia/components1.utopia.js",
           "type": "DESCRIPTOR_FILE",
@@ -2797,10 +2797,10 @@ describe('Lifecycle management of registering components', () => {
         .toMatchInlineSnapshot(`
         Object {
           "bounds": Object {
-            "endCol": 5,
-            "endLine": 12,
-            "startCol": 4,
-            "startLine": 5,
+            "endCol": 4,
+            "endLine": 11,
+            "startCol": 3,
+            "startLine": 4,
           },
           "sourceDescriptorFile": "/utopia/components1.utopia.js",
           "type": "DESCRIPTOR_FILE",