fix(executor): start a new identifier at the character before it

A token continues only when the character immediately before it belongs to
the same token. Asking the previous *significant* character instead made a
name after a line break look like a continuation, so it kept whatever
property-access answer the last token had: `const seen = params.a.b` on one
line left the `if` on the next carrying `b`'s, which turned the statement
head into a method call and the regex after it into division.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: icecrasher321

apps/sim/executor/variables/resolver.test.ts

@@ -1000,6 +1000,25 @@ describe('VariableResolver function block inputs', () => {
     expect(code).toContain(`Number('' + JSON.stringify(globalThis["__blockRef_2"]) + '')`)
   })
 
+  it('starts a new identifier at a line break rather than continuing the last one', async () => {
+    const { block, ctx, resolver } = createResolver('javascript')
+
+    const result = await resolver.resolveInputsForFunctionBlock(
+      ctx,
+      'function',
+      {
+        // `if` begins a statement here; reading the previous *significant* character would
+        // see the `b` of `params.b` and carry its property-access answer into this token.
+        code: ['const seen = params.a.b', `if (seen) /['"]/.test('<producer.result>')`].join('\n'),
+      },
+      block
+    )
+
+    expect(result.resolvedInputs.code).toContain(
+      `.test('' + JSON.stringify(globalThis["__blockRef_0"]) + '')`
+    )
+  })
+
   it('divides after a postfix update rather than opening a regex', async () => {
     const { block, ctx, resolver } = createResolver('javascript')
 

apps/sim/executor/variables/resolver.ts

@@ -1479,10 +1479,10 @@ export class VariableResolver {
           lastSignificantIndex = i
         }
         if (this.isJavaScriptIdentifierChar(char)) {
-          if (
-            previousSignificantIndex < 0 ||
-            !this.isJavaScriptIdentifierChar(template[previousSignificantIndex])
-          ) {
+          // An identifier continues only when the character right before it is part of the same
+          // token. Asking the previous *significant* character instead treats a name after a
+          // line break as a continuation and leaves it carrying the last one's answer.
+          if (i === 0 || !this.isJavaScriptIdentifierChar(template[i - 1])) {
             identifierFollowsPropertyAccess = template[previousSignificantIndex] === '.'
           }
         } else if (char === '(') {
@@ -1561,10 +1561,10 @@ export class VariableResolver {
         lastSignificantIndex = i
       }
       if (this.isJavaScriptIdentifierChar(char)) {
-        if (
-          previousSignificantIndex < 0 ||
-          !this.isJavaScriptIdentifierChar(template[previousSignificantIndex])
-        ) {
+        // An identifier continues only when the character right before it is part of the same
+        // token. Asking the previous *significant* character instead treats a name after a
+        // line break as a continuation and leaves it carrying the last one's answer.
+        if (i === 0 || !this.isJavaScriptIdentifierChar(template[i - 1])) {
           identifierFollowsPropertyAccess = template[previousSignificantIndex] === '.'
         }
       } else if (char === '(') {