Prevent SelfDrop context mutation across render boundaries

[karreiro]

This PR fixes SelfDrop scoping so a self assigned in one template preserves its original scope when passed to a rendered template.

# template1.liquid
{% assign var = 42 %}
{% assign s = self %}
{% render 'template2', other_self: s %}

# template2.liquid
{% assign var = 43 %}
{{ other_self.var }}  => 43 (wrong — should be 42)

The fix renames @context to @self_context in SelfDrop, so the drop has a specific reference to its originating context instead of reusing the inherited context that gets overwritten at the context.rb level on line 220.

[aswamy]

Not sure if liquid-spec also allows render tags, but ill update it to capture your test. Thank you for this ❤️

0Shopify/liquid-spec#124

[ianks]

We likely want to undef context as well, which serves as a signal that the drop doesn’t support contextualization

[karreiro]

Thank you for the review, @ianks!

I did consider undef_method :context=, but that felt a bit unexpected, also because some places (e.g., template.rb:155) call Drop#context= without any guard. I went with def context=(_); end instead to keep the interface intact. Please let me know your thoughts!

[jg-rp]

Hi 👋

This looks like a subtle change in policy for the {% render %} tag. A deliberate mechanism for template authors to bring all of the parent template's scope into a partial template, albeit namespaced under an arbitrary variable.

I understand if you've decided this is a positive new feature. It's just that with this change partial templates are less isolated. They are potentially more tightly coupled to their parent templates, which I thought was one of the reasons for introducing {% render %} and depreciating {% include %}.

And, of course, having arbitrary aliases of self does make static analysis much harder.