interpolate block params in AST?

[stevenvachon]

{{#path 1 "2" path.path as |param0 param1 param2|}} text {{param0}} {{param1}} {{param2}} {{/path}}

currently parses into something very similar to:

{ type: 'Program',
  body: 
   [ { type: 'BlockStatement',
       path: { type: 'PathExpression', parts: ['path'] },
       params: 
        [ { type: 'NumberLiteral', value: 1 },
          { type: 'StringLiteral', value: '2' },
          { type: 'PathExpression', parts: ['path', 'path'] } ],
       program: 
        { type: 'Program',
          body: 
           [ { type: 'ContentStatement', value: ' text ' },
             { type: 'MustacheStatement', path: { type: 'PathExpression', parts: ['param0'] } },
             { type: 'MustacheStatement', path: { type: 'PathExpression', parts: ['param1'] } },
             { type: 'MustacheStatement', path: { type: 'PathExpression', parts: ['param2'] } } ],
          blockParams: ['param0', 'param1', 'param2'] },
       } ] }

which relies on the compiler to interpolate the block parameters. Why not do that in the parser to simplify both the compiler and the AST? It could instead parse to something like:

{ type: 'Program',
  body: 
   [ { type: 'BlockStatement',
       path: { type: 'PathExpression', parts: ['path'] },
       params: 
        [ { type: 'NumberLiteral', value: 1 },
          { type: 'StringLiteral', value: '2' },
          { type: 'PathExpression', parts: ['path', 'path'] } ],
       program: 
        { type: 'Program',
          body: 
           [ { type: 'ContentStatement', value: ' text ' } },
             { type: 'MustacheStatement', path: { type: 'NumberLiteral', value: 1 } },
             { type: 'MustacheStatement', path: { type: 'StringLiteral', value: '2' } },
             { type: 'MustacheStatement', path: { type: 'PathExpression', parts: ['path', 'path'] } } ] },
       } ] }

[kpdecker]

This is not how block parameters work. They can only be resolved at runtime after the block helper runs, not at parse time.