fix: shared template context injection

Author: bnussbau

app/Liquid/Tags/PluginRenderTag.php

@@ -0,0 +1,45 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Liquid\Tags;
+
+use Keepsuit\Liquid\Render\RenderContext;
+use Keepsuit\Liquid\Support\MissingValue;
+use Keepsuit\Liquid\Tags\RenderTag;
+
+/**
+ * Render tag that injects plugin context (trmnl, size, data, config) into partials
+ * so shared templates can use variables like trmnl.user.name without passing them explicitly.
+ */
+class PluginRenderTag extends RenderTag
+{
+    /**
+     * Root-level keys from the plugin render context that should be available in partials.
+     *
+     * @var list<string>
+     */
+    private const PARENT_CONTEXT_KEYS = ['trmnl', 'size', 'data', 'config'];
+
+    protected function buildPartialContext(RenderContext $rootContext, string $templateName, array $variables = []): RenderContext
+    {
+        $partialContext = $rootContext->newIsolatedSubContext($templateName);
+
+        foreach (self::PARENT_CONTEXT_KEYS as $key) {
+            $value = $rootContext->get($key);
+            if ($value !== null && ! $value instanceof MissingValue) {
+                $partialContext->set($key, $value);
+            }
+        }
+
+        foreach ($variables as $key => $value) {
+            $partialContext->set($key, $value);
+        }
+
+        foreach ($this->attributes as $key => $value) {
+            $partialContext->set($key, $rootContext->evaluate($value));
+        }
+
+        return $partialContext;
+    }
+}

app/Models/Plugin.php

@@ -10,6 +10,7 @@
 use App\Liquid\Filters\StandardFilters;
 use App\Liquid\Filters\StringMarkup;
 use App\Liquid\Filters\Uniqueness;
+use App\Liquid\Tags\PluginRenderTag;
 use App\Liquid\Tags\TemplateTag;
 use App\Services\Plugin\Parsers\ResponseParserRegistry;
 use App\Services\PluginImportService;
@@ -499,6 +500,8 @@ public function render(string $size = 'full', bool $standalone = true, ?Device $
 
                     // Register the template tag for inline templates
                     $environment->tagRegistry->register(TemplateTag::class);
+                    // Use plugin render tag so partials receive trmnl, size, data, config
+                    $environment->tagRegistry->register(PluginRenderTag::class);
 
                     // Apply Liquid replacements (including 'with' syntax conversion)
                     $processedMarkup = $this->applyLiquidReplacements($markup);

tests/Feature/PluginInlineTemplatesTest.php

@@ -1,6 +1,7 @@
 <?php
 
 use App\Models\Plugin;
+use App\Models\User;
 use Illuminate\Foundation\Testing\RefreshDatabase;
 
 uses(RefreshDatabase::class);
@@ -238,3 +239,34 @@
     $this->assertStringContainsString('"35":[{"name":"Ryan","age":35}]', $result);
     $this->assertStringContainsString('"29":[{"name":"Sara","age":29},{"name":"Jimbob","age":29}]', $result);
 });
+
+test('shared template receives trmnl context when', function (): void {
+    $user = User::factory()->create(['name' => 'Jane Smith']);
+
+    $plugin = Plugin::factory()->create([
+        'user_id' => $user->id,
+        'name' => 'Departures',
+        'markup_language' => 'liquid',
+        'render_markup_shared' => <<<'LIQUID'
+{% template departures_view %}
+<div class="title_bar">
+  <span class="title">Departures</span>
+  <span class="instance">{{ trmnl.user.name }}</span>
+</div>
+{% endtemplate %}
+LIQUID
+        ,
+        'render_markup' => <<<'LIQUID'
+<div class="view">
+{% render "departures_view", station: "Hauptbahnhof" %}
+</div>
+LIQUID
+        ,
+        'data_payload' => [],
+    ]);
+
+    $result = $plugin->render('full');
+
+    $this->assertStringContainsString('Jane Smith', $result);
+    $this->assertStringContainsString('class="instance"', $result);
+});