pixiebrix-extension icon indicating copy to clipboard operation
pixiebrix-extension copied to clipboard

Generate AST for variable path parsing/generation to handle property access edge cases in variable popover

Open twschiller opened this issue 1 year ago • 1 comments

Context

  • https://github.com/pixiebrix/pixiebrix-extension/pull/8637
  • The current implementation has a corner case for bracketed expressions that end in ?, e.g., ["hello world?"]
  • The check /[ .]/ for which accessors need brackets also needs to be fixed. Characters such as -, etc., would also necessitate brackets
  • Ideally would use a JS parser, but @ is not a valid first character in identifiers

twschiller avatar Jun 17 '24 18:06 twschiller