[clauderic]

What are you adding in this PR?

When unclosed blocks are allowed, as in toLiquidAST and the tolerant parses, a block in {% liquid %} that's never closed got the position of an end tag after its last line:

{%- liquid
  assign var1 = product
  if use_variant
    assign var2 = var1
-%}

• The if's blockEndPosition started at the end of the last line and ran for the length of endif, over \n-%} and one character past the end of the 78-character template. Its position ended there too.
• Inside an element, it ran over whatever came next. In <div>{% liquid … %}</div>, it covered \n%}</d.
• It's now empty, at the end of the last line, as for an unclosed block outside {% liquid %}. The if covers if use_variant to var1.
• A closed block is unchanged.

The test of this template now checks where the if ends. Every other test passes unchanged, and prettier formats these templates the same as before.

What's next? Any followup issues?

#1347 is stacked on this. #1359, the parser specs, on top of #1347, needs this fix: a spec gives a position as the text it covers, cut out of the template, and this one ran past its end.

Tophatting

• npx vitest run packages/liquid-html-parser/src/document/parser.test.ts
• Or: toLiquidAST the template above. Before, children[0].markup[1].blockEndPosition was { start: 74, end: 79 }; now it's { start: 74, end: 74 }.

Before you deploy

• I included a patch bump changeset
• My feature is backward compatible

🤖 Generated with Claude Code