express-hbs 2.5.0 → 4.0.0

package/LICENSE

@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2012-2023 Barc, Inc., Ghost Foundation
+Copyright (c) 2012-2025 Barc, Inc., Ghost Foundation
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of
 this software and associated documentation files (the "Software"), to deal in

package/README.md

@@ -1,6 +1,17 @@
 # express-hbs
 
-Express handlebars template engine with multiple layouts, blocks and cached partials.
+Express Handlebars template engine for Express apps, with nested layouts,
+named content blocks, cached partials, i18n helpers, and asynchronous helpers.
+
+`express-hbs` exposes Express-compatible view engine functions and a shared
+Handlebars instance. It is used by apps that want Handlebars templates with
+layout inheritance and block-style content regions while keeping the familiar
+`app.engine()` integration.
+
+## Requirements
+
+- Node.js 22 or later
+- the pinned pnpm version when working on this repository
 
 ## v2.0.0
 
@@ -17,7 +28,8 @@ If you're upgrading from v0.8.4 to v1.0.0 there are some potentially breaking ch
 
 ## Usage
 
-To use with express 4.
+Register the engine with Express using `hbs.express4()`.
+
 ```js
 var hbs = require('express-hbs');
 
@@ -28,7 +40,9 @@ app.engine('hbs', hbs.express4({
 app.set('view engine', 'hbs');
 app.set('views', __dirname + '/views');
 ```
-To use with express 3 is the same as above, except use hbs.express3
+
+`hbs.express3()` is still available for legacy apps and accepts the same
+options:
 
 ```js
 app.engine('hbs', hbs.express3({
@@ -36,7 +50,7 @@ app.engine('hbs', hbs.express3({
 }));
 ```
 
-Options for `#express3` and `#express4`
+Options for `express3()` and `express4()`:
 
 ```js
 hbs.express4({
@@ -67,15 +81,15 @@ hbs.express4({
 
 ## Syntax
 
-To mark where layout should insert page
+To mark where a layout should insert page content:
 
     {{{body}}}
 
-To declare a block placeholder in layout
+To declare a block placeholder in a layout:
 
     {{{block "pageScripts"}}}
 
-To define block content in a page
+To define block content in a page:
 
     {{#contentFor "pageScripts"}}
       CONTENT HERE
@@ -83,7 +97,7 @@ To define block content in a page
 
 ## Layouts
 
-There are three ways to use a layout, listed in precedence order
+There are three ways to use a layout, listed in precedence order:
 
 1.  Declarative within a page. Use handlebars comment
 
@@ -100,9 +114,10 @@ There are three ways to use a layout, listed in precedence order
 
 2.  As an option to render
 
-    ## ⚠️ This creates a potential security vulnerability if used without a `restrictLayoutsTo`:
-
-    The `restrictLayoutsTo` option will restrict reading layouts to a particular directory, if you do not pass this option then do not use the `layout` option in conjunction with passing user submitted data to res.render e.g. `res.render('index', req.query)`. This allows users to read arbitrary files from your filesystem!
+    ⚠️ Passing user-controlled data into the `layout` option can read arbitrary
+    files unless `restrictLayoutsTo` confines layout resolution to a safe
+    directory. Do not call `res.render('index', req.query)` or similar without
+    setting `restrictLayoutsTo`.
 
     ```js
     res.render('veggies', {
@@ -112,7 +127,8 @@ There are three ways to use a layout, listed in precedence order
     });
     ```
 
-    This option also allows for layout suppression (both the default layout and when specified declaratively in a page) by passing in a falsey Javascript value as the value of the `layout` property:
+    This option also allows layout suppression, including the default layout and
+    template-declared layouts, by passing a falsey JavaScript value:
 
     ```js
     res.render('veggies', {
@@ -215,9 +231,12 @@ var instance2 = hbs.create();
 
 ## Template options
 
-The main use case for template options is setting the handlebars "data" object - this creates global template variables accessible with an `@` prefix.
+The main use case for template options is setting the Handlebars `data` object,
+which creates global template variables accessible with an `@` prefix.
 
-Template options can be set in 3 ways. When setting global template options they can be [passed as config on creation of an instance](https://github.com/barc/express-hbs#usage), and they can also be updated used the `updateTemplateOptions(templateOptions)` method of an instance. To set template options for an individual request they can be set on `res.locals` using the helper method `updateLocalTemplateOptions(locals, templateOptions)`.
+Template options can be passed when creating an engine instance, updated with
+`updateTemplateOptions(templateOptions)`, or set for one request with
+`updateLocalTemplateOptions(locals, templateOptions)` on `res.locals`.
 
 Both of these methods have a companion method `getTemplateOptions()` and `getLocalTemplateOptions(locals)`, which should be used when extending or merging the current options.
 
@@ -275,20 +294,29 @@ File `views/index.hbs`
 
 To run example project
 
-    npm install -d
-    node example/app.js
+```sh
+pnpm install
+node example/app.js
+```
 
+The example server listens on <http://localhost:3000>.
 
 ## Testing
 
-The test suite requires the `grunt-cli` package:
+Install dependencies and run the test suite:
 
-    npm install -g grunt-cli
-    npm install -d
+```sh
+pnpm install
+pnpm test
+```
 
-Once everything's installed, just run:
+Useful maintainer commands:
 
-    npm test
+```sh
+pnpm lint          # oxlint plus oxfmt --check
+pnpm lint:fix      # apply safe lint fixes and formatting
+pnpm coverage      # mocha under nyc with enforced coverage thresholds
+```
 
 
 ## Credits
@@ -302,4 +330,4 @@ Big thanks to all [CONTRIBUTORS](https://github.com/TryGhost/express-hbs/contrib
 
 The MIT License (MIT)
 
-Copyright (c) 2012-2023 Barc, Inc., Ghost Foundation - Released under the [MIT license](LICENSE).
+Copyright (c) 2012-2026 Barc, Inc., Ghost Foundation - Released under the [MIT license](LICENSE).

package/lib/hbs.js

@@ -10,12 +10,12 @@ var _ = require('lodash');
 /**
  * Regex pattern for layout directive. {{!< layout }}
  */
-var layoutPattern = /{{!<\s+([A-Za-z0-9\._\-\/]+)\s*}}/;
+var layoutPattern = /{{!<\s+([A-Za-z0-9._\-/]+)\s*}}/;
 
 /**
  * Constructor
  */
-var ExpressHbs = function() {
+var ExpressHbs = function () {
   this.handlebars = handlebars.create();
   this.SafeString = this.handlebars.SafeString;
   this.Utils = this.handlebars.Utils;
@@ -33,7 +33,7 @@ var ExpressHbs = function() {
  * <link rel="stylesheet" href='{{{URL "css/style.css"}}}' />
  * {{/contentFor}}
  */
-ExpressHbs.prototype.content = function(name, options, context) {
+ExpressHbs.prototype.content = function (name, options, context) {
   var block = options.data.root.blockCache[name] || (options.data.root.blockCache[name] = []);
   block.push(options.fn(context));
 };
@@ -46,9 +46,8 @@ ExpressHbs.prototype.content = function(name, options, context) {
  * @param {String} filename Path to template file.
  * @param {String} layout Layout path.
  */
-ExpressHbs.prototype.layoutPath = function(filename, layout) {
-  var dirs,
-    layoutPath;
+ExpressHbs.prototype.layoutPath = function (filename, layout) {
+  var dirs, layoutPath;
   if (layout[0] === '.') {
     dirs = path.dirname(filename);
   } else if (this.layoutsDir) {
@@ -71,7 +70,7 @@ ExpressHbs.prototype.layoutPath = function(filename, layout) {
  * @param {String} filename Path to template
  * @returns {String|undefined} Returns the path to layout.
  */
-ExpressHbs.prototype.declaredLayoutFile = function(str, filename) {
+ExpressHbs.prototype.declaredLayoutFile = function (str, filename) {
   var matches = str.match(layoutPattern);
   if (matches) {
     var layout = matches[1];
@@ -96,12 +95,20 @@ ExpressHbs.prototype.declaredLayoutFile = function(str, filename) {
  * @param {Boolean}     useCache    Cache the compiled layout?
  * @param {Function}    cb          Callback called with layouts stack
  */
-ExpressHbs.prototype.cacheLayout = function(layoutFile, useCache, cb) {
+ExpressHbs.prototype.cacheLayout = function (layoutFile, useCache, cb) {
   var self = this;
 
   if (this.restrictLayoutsTo) {
-    if (!layoutFile.startsWith(this.restrictLayoutsTo)) {
-      var err = new Error('Cannot read ' + layoutFile + ' it does not reside in ' + this.restrictLayoutsTo);
+    var relativePath = path.relative(this.restrictLayoutsTo, layoutFile);
+    var outside =
+      relativePath === '..' ||
+      relativePath.startsWith('..' + path.sep) ||
+      path.isAbsolute(relativePath);
+
+    if (outside) {
+      var err = new Error(
+        'Cannot read ' + layoutFile + ' it does not reside in ' + this.restrictLayoutsTo,
+      );
       return cb(err, null);
     }
   }
@@ -113,14 +120,14 @@ ExpressHbs.prototype.cacheLayout = function(layoutFile, useCache, cb) {
   var layoutTemplates = this.cache[layoutFile];
   if (layoutTemplates) return cb(null, layoutTemplates);
 
-  fs.readFile(layoutFile, 'utf8', function(err, str) {
+  fs.readFile(layoutFile, 'utf8', function (err, str) {
     if (err) return cb(err);
 
     //  File path of eventual declared parent layout
     var parentLayoutFile = self.declaredLayoutFile(str, layoutFile);
 
     // This function returns the current layout stack to the caller
-    var _returnLayouts = function(layouts) {
+    var _returnLayouts = function (layouts) {
       var currentLayout;
       layouts = layouts.slice(0);
       currentLayout = self.compile(str, layoutFile);
@@ -133,7 +140,7 @@ ExpressHbs.prototype.cacheLayout = function(layoutFile, useCache, cb) {
 
     if (parentLayoutFile) {
       // Recursively compile/cache parent layouts
-      self.cacheLayout(parentLayoutFile, useCache, function(err, parentLayouts) {
+      self.cacheLayout(parentLayoutFile, useCache, function (err, parentLayouts) {
         if (err) return cb(err);
         _returnLayouts(parentLayouts);
       });
@@ -147,7 +154,7 @@ ExpressHbs.prototype.cacheLayout = function(layoutFile, useCache, cb) {
 /**
  * Cache partial templates found under directories configure in partialsDir.
  */
-ExpressHbs.prototype.cachePartials = function(cb) {
+ExpressHbs.prototype.cachePartials = function (cb) {
   var self = this;
 
   if (!(this.partialsDir instanceof Array)) {
@@ -158,36 +165,36 @@ ExpressHbs.prototype.cachePartials = function(cb) {
   var count = 0;
 
   function readNext() {
-    readdirp(self.partialsDir[count], {fileFilter: '*' + self._options.extname})
-    .on('warn', function(err) {
-      console.warn('Non-fatal error in express-hbs cachePartials.', err);
-    })
-    .on('error', function(err) {
-      console.error('Fatal error in express-hbs cachePartials', err);
-      return cb(err);
-    })
-    .on('data', function(entry) {
-      if (!entry) return;
-      var source = fs.readFileSync(entry.fullPath, 'utf8');
-      var dirname = path.dirname(entry.path);
-      dirname = dirname === '.' ? '' : dirname + '/';
-
-      var name = dirname + path.basename(entry.basename, self._options.extname);
-      // fix the path in windows
-      name = name.split('\\').join('/');
-      self.registerPartial(name, source, entry.fullPath);
-    })
-    .on('end', function() {
-      count += 1;
-
-      // If all directories aren't read, read the next directory
-      if (count < self.partialsDir.length) {
-        readNext();
-      } else {
-        self.isPartialCachingComplete = true;
-        if (cb) cb(null, true);
-      }
-    });
+    readdirp(self.partialsDir[count], { fileFilter: '*' + self._options.extname })
+      .on('warn', function (err) {
+        console.warn('Non-fatal error in express-hbs cachePartials.', err);
+      })
+      .on('error', function (err) {
+        console.error('Fatal error in express-hbs cachePartials', err);
+        return cb(err);
+      })
+      .on('data', function (entry) {
+        if (!entry) return;
+        var source = fs.readFileSync(entry.fullPath, 'utf8');
+        var dirname = path.dirname(entry.path);
+        dirname = dirname === '.' ? '' : dirname + '/';
+
+        var name = dirname + path.basename(entry.basename, self._options.extname);
+        // fix the path in windows
+        name = name.split('\\').join('/');
+        self.registerPartial(name, source, entry.fullPath);
+      })
+      .on('end', function () {
+        count += 1;
+
+        // If all directories aren't read, read the next directory
+        if (count < self.partialsDir.length) {
+          readNext();
+        } else {
+          self.isPartialCachingComplete = true;
+          if (cb) cb(null, true);
+        }
+      });
   }
 
   readNext();
@@ -210,7 +217,7 @@ ExpressHbs.prototype.cachePartials = function(cb) {
  *   }
  * }
  */
-ExpressHbs.prototype.express3 = function(options) {
+ExpressHbs.prototype.express3 = function (options) {
   var self = this;
 
   // Set defaults
@@ -227,31 +234,31 @@ ExpressHbs.prototype.express3 = function(options) {
 
   if (options.i18n) {
     var i18n = options.i18n;
-    this.handlebars.registerHelper('__', function() {
+    this.handlebars.registerHelper('__', function () {
       var args = Array.prototype.slice.call(arguments);
       var options = args.pop();
       return i18n.__.apply(options.data.root, args);
     });
-    this.handlebars.registerHelper('__n', function() {
+    this.handlebars.registerHelper('__n', function () {
       var args = Array.prototype.slice.call(arguments);
       var options = args.pop();
       return i18n.__n.apply(options.data.root, args);
     });
   }
 
-  this.handlebars.registerHelper(this._options.blockHelperName, function(name, options) {
+  this.handlebars.registerHelper(this._options.blockHelperName, function (name, options) {
     var val = options.data.root.blockCache[name];
     if (val === undefined && typeof options.fn === 'function') {
       val = options.fn(this);
     }
     if (Array.isArray(val)) {
-      val = val.join('\n'); 
+      val = val.join('\n');
     }
     return val;
   });
 
   // Pass 'this' as context of helper function to don't lose context call of helpers.
-  this.handlebars.registerHelper(this._options.contentHelperName, function(name, options) {
+  this.handlebars.registerHelper(this._options.contentHelperName, function (name, options) {
     return self.content(name, options, this);
   });
 
@@ -300,12 +307,12 @@ ExpressHbs.prototype.express4 = ExpressHbs.prototype.express3;
  *
  * @param {Boolean} useCache Whether to cache.
  */
-ExpressHbs.prototype.loadDefaultLayout = function(useCache, cb) {
+ExpressHbs.prototype.loadDefaultLayout = function (useCache, cb) {
   var self = this;
   if (!this._options.defaultLayout) return cb();
   if (useCache && this.defaultLayoutTemplates) return cb(null, this.defaultLayoutTemplates);
 
-  this.cacheLayout(this._options.defaultLayout, useCache, function(err, templates) {
+  this.cacheLayout(this._options.defaultLayout, useCache, function (err, templates) {
     if (err) return cb(err);
     self.defaultLayoutTemplates = templates.slice(0);
     return cb(null, templates);
@@ -315,7 +322,7 @@ ExpressHbs.prototype.loadDefaultLayout = function(useCache, cb) {
 /**
  * Expose useful methods.
  */
-ExpressHbs.prototype.registerHelper = function(name, fn) {
+ExpressHbs.prototype.registerHelper = function (name, fn) {
   this.handlebars.registerHelper(name, fn);
 };
 
@@ -325,7 +332,7 @@ ExpressHbs.prototype.registerHelper = function(name, fn) {
  * @param {String} name The name of the partial as used in a template.
  * @param {String} source String source of the partial.
  */
-ExpressHbs.prototype.registerPartial = function(name, source, filename) {
+ExpressHbs.prototype.registerPartial = function (name, source, filename) {
   this.handlebars.registerPartial(name, this.compile(source, filename));
 };
 
@@ -335,7 +342,7 @@ ExpressHbs.prototype.registerPartial = function(name, source, filename) {
  * @param {String} source The source to compile.
  * @param {String} filename The path used to embed into __filename for errors.
  */
-ExpressHbs.prototype.compile = function(source, filename) {
+ExpressHbs.prototype.compile = function (source, filename) {
   // Handlebars has a bug with comment only partial causes errors. This must
   // be a string so the block below can add a space.
   if (typeof source !== 'string') {
@@ -368,53 +375,46 @@ ExpressHbs.prototype.compile = function(source, filename) {
  * @param {String} name The name of the partial as used in a template.
  * @param {String} fn The `function(options, cb)`
  */
-ExpressHbs.prototype.registerAsyncHelper = function(name, fn) {
-  this.handlebars.registerHelper(name, function(context, options) {
-    var resolverCache = this.resolverCache ||
+ExpressHbs.prototype.registerAsyncHelper = function (name, fn) {
+  this.handlebars.registerHelper(name, function (context, options) {
+    var resolverCache =
+      this.resolverCache ||
       _.get(context, 'data.root.resolverCache') ||
       _.get(options, 'data.root.resolverCache');
     if (!resolverCache) {
       throw new Error('Could not find resolver cache in async helper ' + name + '.');
     }
     if (options && fn.length > 2) {
-      var resolveFunc = function(arr, cb) {
+      var resolveFunc = function (arr, cb) {
         return fn.call(this, arr[0], arr[1], cb);
       };
 
-      return resolver.resolve(
-        resolverCache,
-        resolveFunc.bind(this),
-        [context, options]
-      );
+      return resolver.resolve(resolverCache, resolveFunc.bind(this), [context, options]);
     }
-    return resolver.resolve(
-      resolverCache,
-      fn.bind(this),
-      context
-    );
+    return resolver.resolve(resolverCache, fn.bind(this), context);
   });
 };
 
-ExpressHbs.prototype.getTemplateOptions = function() {
+ExpressHbs.prototype.getTemplateOptions = function () {
   return this._options.templateOptions;
 };
 
-ExpressHbs.prototype.updateTemplateOptions = function(templateOptions) {
+ExpressHbs.prototype.updateTemplateOptions = function (templateOptions) {
   this._options.templateOptions = templateOptions;
 };
 
-ExpressHbs.prototype.getLocalTemplateOptions = function(locals) {
+ExpressHbs.prototype.getLocalTemplateOptions = function (locals) {
   return locals._templateOptions || {};
 };
 
-ExpressHbs.prototype.updateLocalTemplateOptions = function(locals, localTemplateOptions) {
-  return locals._templateOptions = localTemplateOptions;
+ExpressHbs.prototype.updateLocalTemplateOptions = function (locals, localTemplateOptions) {
+  return (locals._templateOptions = localTemplateOptions);
 };
 
 /**
  * Creates a new instance of ExpressHbs.
  */
-ExpressHbs.prototype.create = function() {
+ExpressHbs.prototype.create = function () {
   return new ExpressHbs();
 };
 
@@ -508,7 +508,6 @@ ExpressHbs.prototype.___express = function ___express(filename, source, options,
     cb(null, res);
   }
 
-
   /**
    * Renders `template` with an optional set of nested `layoutTemplates` using
    * data in `locals`.
@@ -520,7 +519,7 @@ ExpressHbs.prototype.___express = function ___express(filename, source, options,
     // being passed the rendered string of the previous ones as `body`
     var i = layoutTemplates.length - 1;
 
-    var _stackRenderer = function(err, htmlStr) {
+    var _stackRenderer = function (err, htmlStr) {
       if (err) return cb(err);
 
       if (i >= 0) {
@@ -535,7 +534,6 @@ ExpressHbs.prototype.___express = function ___express(filename, source, options,
     renderTemplate(template, locals, _stackRenderer);
   }
 
-
   /**
    * Lazy loads js-beautify, which should not be used in production env.
    */
@@ -561,14 +559,14 @@ ExpressHbs.prototype.___express = function ___express(filename, source, options,
       }
     }
 
-    fs.readFile(filename, 'utf8', function(err, source) {
+    fs.readFile(filename, 'utf8', function (err, source) {
       if (err) return cb(err);
 
       var template = self.compile(source, filename);
       if (options.cache) {
         self.cache[filename] = {
           source: source,
-          template: template
+          template: template,
         };
       }
       return cb(null, source, template);
@@ -579,16 +577,16 @@ ExpressHbs.prototype.___express = function ___express(filename, source, options,
    * Compiles a file into a template and a layoutTemplate, then renders it above.
    */
   function compileFile(locals, cb) {
-    getSourceTemplate(function(err, source, template) {
+    getSourceTemplate(function (err, source, template) {
       if (err) return cb(err);
 
       // Try to get the layout
-      parseLayout(source, filename, function(err, layoutTemplates) {
+      parseLayout(source, filename, function (err, layoutTemplates) {
         if (err) return cb(err);
 
         function renderIt(layoutTemplates) {
           if (self._options.beautify) {
-            return render(template, locals, layoutTemplates, function(err, html) {
+            return render(template, locals, layoutTemplates, function (err, html) {
               if (err) return cb(err);
               loadBeautify();
               return cb(null, self.beautify(html, self.beautifyrc));
@@ -608,7 +606,7 @@ ExpressHbs.prototype.___express = function ___express(filename, source, options,
         } else if (typeof options.layout !== 'undefined' && options.layout) {
           // 2. Layout specified by options from render
           var layoutFile = self.layoutPath(filename, options.layout);
-          self.cacheLayout(layoutFile, options.cache, function(err, layoutTemplates) {
+          self.cacheLayout(layoutFile, options.cache, function (err, layoutTemplates) {
             if (err) return cb(err);
             renderIt(layoutTemplates);
           });
@@ -625,11 +623,11 @@ ExpressHbs.prototype.___express = function ___express(filename, source, options,
 
   function replaceValue(values, text) {
     if (typeof text === 'string') {
-      Object.keys(values).forEach(function(id) {
-        text = text.replace(id, function() {
+      Object.keys(values).forEach(function (id) {
+        text = text.replace(id, function () {
           return values[id];
         });
-        text = text.replace(self.Utils.escapeExpression(id), function() {
+        text = text.replace(self.Utils.escapeExpression(id), function () {
           return self.Utils.escapeExpression(values[id]);
         });
       });
@@ -640,9 +638,9 @@ ExpressHbs.prototype.___express = function ___express(filename, source, options,
   // Handles waiting for async helpers
   function handleAsync(err, res) {
     if (err) return cb(err);
-    resolver.done(options.resolverCache, function(err, values) {
+    resolver.done(options.resolverCache, function (err, values) {
       if (err) return cb(err);
-      Object.keys(values).forEach(function(key) {
+      Object.keys(values).forEach(function (key) {
         values[key] = replaceValue(values, values[key]);
       });
       res = replaceValue(values, res);
@@ -654,13 +652,13 @@ ExpressHbs.prototype.___express = function ___express(filename, source, options,
   }
 
   // kick it off by loading default template (if any)
-  this.loadDefaultLayout(options.cache, function(err) {
+  this.loadDefaultLayout(options.cache, function (err) {
     if (err) return cb(err);
 
     // Force reloading of all partials if caching is not used. Inefficient but there
     // is no loading partial event.
     if (self.partialsDir && (!options.cache || !self.isPartialCachingComplete)) {
-      return self.cachePartials(function(err) {
+      return self.cachePartials(function (err) {
         if (err) return cb(err);
         return compileFile(options, handleAsync);
       });

package/lib/resolver.js

@@ -14,9 +14,9 @@ var ID_SUFFIX = '__';
 
 function resolve(cache, fn, context) {
   var id = ID_PREFIX + ID_ESCAPED_STRING + generateId(ID_LENGTH) + ID_SUFFIX;
-  cache[id] = new Promise(function(passed, failed) {
+  cache[id] = new Promise(function (passed, failed) {
     try {
-      fn(context, function(res) {
+      fn(context, function (res) {
         passed(res);
       });
     } catch (error) {
@@ -29,23 +29,24 @@ function resolve(cache, fn, context) {
 // cache is an object where the keys are cache keys and values are promises. See above.
 function done(cache, callback) {
   return Promise.all(Object.values(cache))
-    .then(function(values) {
+    .then(function (values) {
       var resolvedCache = {};
       var keys = Object.keys(cache);
-      values.forEach(function(value, index) {
+      values.forEach(function (value, index) {
         resolvedCache[keys[index]] = value;
       });
       callback(null, resolvedCache);
-  }).catch(function(error) {
-    callback(error);
-  });
+    })
+    .catch(function (error) {
+      callback(error);
+    });
 }
 
 function hasResolvers(text) {
   // NOTE: We specifically search the text for the ID_PREFIX **NOT** including the escapable character
   // This is because that character can be escaped in the text, and lead us to not finding unresolved
   // async helper outputs.
-  if (text.search(ID_PREFIX) > 0) {
+  if (text.search(ID_PREFIX) >= 0) {
     return true;
   }
   return false;
@@ -54,5 +55,5 @@ function hasResolvers(text) {
 module.exports = {
   done: done,
   hasResolvers: hasResolvers,
-  resolve: resolve
+  resolve: resolve,
 };