fix: add semicolons to compiled template statements for consistency

src/compile-string.ts

@@ -93,7 +93,7 @@ export function compileBody(this: Eta, buff: Array<AstObject>): string {
       const str = currentBlock;
 
       // we know string exists
-      returnStr += "__eta.res+='" + str + "'\n";
+      returnStr += "__eta.res+='" + str + "';\n";
     } else {
       const type = currentBlock.t; // "r", "e", or "i"
       let content = currentBlock.val || "";
@@ -107,7 +107,7 @@ export function compileBody(this: Eta, buff: Array<AstObject>): string {
           content = "__eta.f(" + content + ")";
         }
 
-        returnStr += "__eta.res+=" + content + "\n";
+        returnStr += "__eta.res+=" + content + ";\n";
       } else if (type === "i") {
         // interpolate
 
@@ -119,7 +119,7 @@ export function compileBody(this: Eta, buff: Array<AstObject>): string {
           content = "__eta.e(" + content + ")";
         }
 
-        returnStr += "__eta.res+=" + content + "\n";
+        returnStr += "__eta.res+=" + content + ";\n";
       } else if (type === "e") {
         // execute
         returnStr += content + "\n";

test/compile-string.spec.ts

@@ -26,8 +26,8 @@ function layout(path, data) {
 
 function output(s){__eta.res+=s;}
 
-__eta.res+='hi '
-__eta.res+=__eta.e(it.name)
+__eta.res+='hi ';
+__eta.res+=__eta.e(it.name);
 
 if (__eta.layout) {
   __eta.res = include (__eta.layout, {...it, body: __eta.res, ...__eta.layoutData});
@@ -52,8 +52,8 @@ function layout(path, data) {
 
 function output(s){__eta.res+=s;}
 
-__eta.res+='hi '
-__eta.res+=it.name
+__eta.res+='hi ';
+__eta.res+=it.name;
 
 if (__eta.layout) {
   __eta.res = include (__eta.layout, {...it, body: __eta.res, ...__eta.layoutData});
@@ -80,9 +80,9 @@ function layout(path, data) {
 
 function output(s){__eta.res+=s;}
 
-__eta.res+='hi'
-__eta.res+=__eta.e(it.firstname)
-__eta.res+=__eta.e(it.lastname)
+__eta.res+='hi';
+__eta.res+=__eta.e(it.firstname);
+__eta.res+=__eta.e(it.lastname);
 
 if (__eta.layout) {
   __eta.res = include (__eta.layout, {...it, body: __eta.res, ...__eta.layoutData});
@@ -107,29 +107,29 @@ function layout(path, data) {
 
 function output(s){__eta.res+=s;}
 
-__eta.res+='Hi\\n'
+__eta.res+='Hi\\n';
 console.log("Hope you like Eta!")
-__eta.res+=__eta.e(it.htmlstuff)
-__eta.res+='\\n'
+__eta.res+=__eta.e(it.htmlstuff);
+__eta.res+='\\n';
 for (var key in it.obj) {
-__eta.res+='Value: '
-__eta.res+=__eta.e(it.obj[key])
-__eta.res+=', Key: '
-__eta.res+=__eta.e(key)
-__eta.res+='\\n'
+__eta.res+='Value: ';
+__eta.res+=__eta.e(it.obj[key]);
+__eta.res+=', Key: ';
+__eta.res+=__eta.e(key);
+__eta.res+='\\n';
 if (key === 'thirdchild') {
-__eta.res+='  '
+__eta.res+='  ';
 for (var i = 0, arr = it.obj[key]; i < arr.length; i++) {
-__eta.res+='      Salutations! Index: '
-__eta.res+=__eta.e(i)
-__eta.res+=', parent key: '
-__eta.res+=__eta.e(key)
-__eta.res+='      \\n  '
+__eta.res+='      Salutations! Index: ';
+__eta.res+=__eta.e(i);
+__eta.res+=', parent key: ';
+__eta.res+=__eta.e(key);
+__eta.res+='      \\n  ';
 }
 }
 }
-__eta.res+='\\nThis is a partial: '
-__eta.res+=include("mypartial")
+__eta.res+='\\nThis is a partial: ';
+__eta.res+=include("mypartial");
 
 if (__eta.layout) {
   __eta.res = include (__eta.layout, {...it, body: __eta.res, ...__eta.layoutData});

test/render.spec.ts

@@ -312,3 +312,54 @@ describe("import values merging with the useWith", () => {
     expect(res).toEqual("This is a partial.\nHello friend");
   });
 });
+
+describe("forEach loops in included templates", () => {
+  const eta = new Eta();
+
+  it("renders forEach loop inside included partial", () => {
+    eta.loadTemplate(
+      "@loop-partial",
+      `<ul>
+<% (it.items || []).forEach(item => { %>
+<li><%= item %></li>
+<% }) %>
+</ul>`,
+    );
+
+    eta.loadTemplate(
+      "@loop-main",
+      `<%~ include('@loop-partial', {items: ['a', 'b', 'c']}) %>`,
+    );
+
+    const result = eta.render("@loop-main", {});
+
+    expect(result).toContain("<li>a</li>");
+    expect(result).toContain("<li>b</li>");
+    expect(result).toContain("<li>c</li>");
+  });
+
+  it("renders includes called from within a forEach loop", () => {
+    eta.loadTemplate(
+      "@nested-partial",
+      `<div>
+<% (it.question.items || []).forEach(opt => { %>
+<span><%= opt %></span>
+<% }) %>
+</div>`,
+    );
+
+    eta.loadTemplate(
+      "@nested-main",
+      `<% [{items: ['a', 'b']}, {items: ['c', 'd']}].forEach(q => { %>
+<%~ include('@nested-partial', {question: q}) %>
+<% }) %>`,
+    );
+
+    const result = eta.render("@nested-main", {});
+
+    expect(result).toContain("<span>a</span>");
+    expect(result).toContain("<span>b</span>");
+    expect(result).toContain("<span>c</span>");
+    expect(result).toContain("<span>d</span>");
+  });
+});