codegen to allow inline substitution

Author: icecrasher321

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

@@ -148,7 +148,7 @@ describe('VariableResolver function block inputs', () => {
     )
 
     expect(result.resolvedInputs.code).toBe(
-      'const base64 = await sim.files.readBase64(globalThis["__blockRef_0"]);\nreturn base64'
+      'const base64 = (await sim.files.readBase64(globalThis["__blockRef_0"]));\nreturn base64'
     )
     expect(result.displayInputs.code).toBe('const base64 = <Producer.file.base64>;\nreturn base64')
     expect(result.contextVariables.__blockRef_0).toMatchObject({
@@ -158,6 +158,21 @@ describe('VariableResolver function block inputs', () => {
     expect(result.contextVariables.__blockRef_0).not.toHaveProperty('base64')
   })
 
+  it('wraps lazy JavaScript file base64 reads before member access', async () => {
+    const { block, ctx, resolver } = createResolver('javascript')
+
+    const result = await resolver.resolveInputsForFunctionBlock(
+      ctx,
+      'function',
+      { code: 'return <Producer.file.base64>.length' },
+      block
+    )
+
+    expect(result.resolvedInputs.code).toBe(
+      'return (await sim.files.readBase64(globalThis["__blockRef_0"])).length'
+    )
+  })
+
   it('uses existing inline base64 for keyless files instead of lazy storage reads', async () => {
     const { block, ctx, resolver } = createResolver('javascript')
     const state = new ExecutionState()
@@ -231,7 +246,7 @@ describe('VariableResolver function block inputs', () => {
     )
 
     expect(result.resolvedInputs.code).toBe(
-      'return await sim.values.read(globalThis["__blockRef_0"])'
+      'return (await sim.values.read(globalThis["__blockRef_0"]))'
     )
     expect(result.contextVariables.__blockRef_0).toMatchObject({
       __simLargeValueRef: true,

apps/sim/executor/variables/resolver.ts

@@ -501,7 +501,7 @@ export class VariableResolver {
     const { base64: _base64, ...fileMetadata } = file
     contextVarAccumulator[varName] = fileMetadata
     const fileExpression = `globalThis[${JSON.stringify(varName)}]`
-    const lazyExpression = `await sim.files.readBase64(${fileExpression})`
+    const lazyExpression = `(await sim.files.readBase64(${fileExpression}))`
 
     return {
       replacement: this.formatJavaScriptAsyncExpression(lazyExpression, template, matchIndex),
@@ -519,7 +519,7 @@ export class VariableResolver {
       return null
     }
 
-    const expression = `await sim.values.read(globalThis[${JSON.stringify(varName)}])`
+    const expression = `(await sim.values.read(globalThis[${JSON.stringify(varName)}]))`
     return this.formatJavaScriptAsyncExpression(expression, template, matchIndex, {
       stringifyInStringContext: true,
     })