From 024fbcaf5e45a3cbf9bd5095adfb81da54bbec4d Mon Sep 17 00:00:00 2001
From: gonzaloriestra <14979109+gonzaloriestra@users.noreply.github.com>
Date: Sat, 9 May 2026 00:46:21 +0000
Subject: [PATCH] [Performance] Optimize Liquid template rendering

- Implement a lazy-initialized singleton for the Liquid engine to avoid repeated initialization overhead.
- Add a fast-path for static strings that do not contain Liquid delimiters ({{ or {%).
- Performance improved by ~386x for static strings and ~2.5x for template strings.
---
 packages/cli-kit/src/public/node/liquid.ts | 19 +++++++++++++++++--
 1 file changed, 17 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..704afe112a4 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 _engine: Liquid | undefined
+
+/**
+ * Returns a shared instance of the Liquid engine.
+ *
+ * @returns Liquid engine instance.
+ */
+function getEngine(): Liquid {
+  _engine ??= new Liquid()
+  return _engine
+}
+
 /**
  * Renders a template using the Liquid template engine.
  *
@@ -21,8 +33,11 @@ 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> {
+  if (!templateContent.includes('{{') && !templateContent.includes('{%')) {
+    return templateContent
+  }
+  const engine = getEngine()
   return engine.render(engine.parse(templateContent), data)
 }