openclaw - 💡(How to fix) Fix Feature: browser tool evaluate should support statements (var/let/const) [1 comments, 2 participants]

browser act kind=evaluate currently only accepts expressions, not statements. This means var, let, const declarations cause a syntax error: Error: Invalid evaluate function: Unexpected token 'var'

Fix Plan

To address the issue, we will implement an auto-wrap feature in an async arrow function.

Steps

• Check if the input is a statement or an expression
• If it's a statement, wrap it in an async arrow function
• Use a library like esprima or acorn to parse the input and determine if it's a statement or an expression

Example Code

const esprima = require('esprima');

function autoWrap(input) {
  try {
    const tree = esprima.parseScript(input);
    if (tree.body.length > 0 && tree.body[0].type === 'VariableDeclaration') {
      // Input is a statement, wrap it in an async arrow function
      return `(async () => { ${input} })()`;
    } else {
      // Input is an expression, return it as is
      return input;
    }
  } catch (e) {
    // If parsing fails, assume it's an expression
    return input;
  }
}

// Example usage:
const input = 'var x = new XMLHttpRequest();';
const wrappedInput = autoWrap(input);
console.log(wrappedInput);
// Output: (async () => { var x = new XMLHttpRequest(); })()

Verification

To verify the fix, test the autoWrap function with different inputs, including statements and expressions.

Extra Tips

• Use a try-catch block to handle parsing errors and assume the input is an expression if parsing fails.
• Consider using a more advanced parsing library like babel-parser for better support of modern JavaScript features.