[clauderic]

What are you adding in this PR?

In {{ 'hello' | upcase | | }}, LiquidHTMLSyntaxError reported two offenses whose fixes overlap:

{{ 'hello' | upcase | | }}
                    ^^^     Remove extra `|` character(s): replace "| |" with "| "
                     ^^     Remove the trailing `|` character: delete " |"

• Each fix on its own still leaves a trailing pipe.
• Autofix can't apply both, because both rewrite the last |. It throws Overlapping ranges are not allowed.

Pipes that end the markup are now reported only as a trailing pipe, and its fix removes the whole run: {{ 'hello' | upcase }}. A single trailing pipe is reported exactly as before.

What did you learn?

I found this by applying each test's fixes together, as autofix does. This was the only test where they conflicted. The existing test only checked that both offenses had a fix.

Tophatting

• The InvalidPipeSyntax test expected two offenses. It now expects one, plus the fixed result, and it fails without this change.
• I also checked {{ 'hello' | | | }}, {{ 'hello' | upcase | | | downcase | | }}, assign, echo, and pipes inside {% liquid %}. Each gets one fix per problem, and every result is valid Liquid.

Before you deploy

• I included a patch bump changeset

🤖 Generated with Claude Code