From 8bb6c8ea8bd39ff889428106755a349fe5c57fbd Mon Sep 17 00:00:00 2001
From: gonzaloriestra <14979109+gonzaloriestra@users.noreply.github.com>
Date: Wed, 6 May 2026 01:10:07 +0000
Subject: [PATCH] [Performance] Optimize Liquid template rendering

- Reuse Liquid engine instance via shared singleton
- Add fast-path for static strings without Liquid tags
- Static strings are ~480x faster, templates ~3x faster
---
 packages/cli-kit/src/public/node/liquid.ts | 22 ++++++++++++++++++++--
 1 file changed, 20 insertions(+), 2 deletions(-)

diff --git a/packages/cli-kit/src/public/node/liquid.ts b/packages/cli-kit/src/public/node/liquid.ts
index 47cdb18ff27..58d0cbe1b7c 100644
--- a/packages/cli-kit/src/public/node/liquid.ts
+++ b/packages/cli-kit/src/public/node/liquid.ts
@@ -14,6 +14,18 @@ import {joinPath, dirname, relativePath} from './path.js'
 import {outputContent, outputToken, outputDebug} from './output.js'
 import {Liquid} from 'liquidjs'
 
+let liquidEngine: Liquid | undefined
+
+/**
+ * Returns a shared instance of the Liquid template engine.
+ *
+ * @returns Shared Liquid instance.
+ */
+function getLiquidEngine(): Liquid {
+  liquidEngine ??= new Liquid()
+  return liquidEngine
+}
+
 /**
  * Renders a template using the Liquid template engine.
  *
@@ -21,8 +33,14 @@ import {Liquid} from 'liquidjs'
  * @param data - Data to feed the template engine.
  * @returns Rendered template.
  */
-export function renderLiquidTemplate(templateContent: string, data: object): Promise<string> {
-  const engine = new Liquid()
+export async function renderLiquidTemplate(templateContent: string, data: object): Promise<string> {
+  // Optimization: If the content doesn't contain any Liquid tags, we can return it as is.
+  // This avoids the overhead of parsing and rendering the template.
+  if (!templateContent.includes('{{') && !templateContent.includes('{%')) {
+    return templateContent
+  }
+
+  const engine = getLiquidEngine()
   return engine.render(engine.parse(templateContent), data)
 }