ghtml 1.2.1 → 1.2.3

package/.eslintrc.json

@@ -1,5 +1,4 @@
 {
   "root": true,
-  "env": { "node": true },
   "extends": ["plugin:grules/all"]
 }

package/.github/workflows/benchmark.yml

@@ -0,0 +1,93 @@
+name: Benchmark Comparison
+
+permissions:
+  contents: read
+  pull-requests: write
+
+on:
+  pull_request:
+    branches:
+      - main
+
+jobs:
+  benchmark-comparison:
+    runs-on: ubuntu-latest
+    steps:
+      # Checkout PR code
+      - name: Checkout PR code
+        uses: actions/checkout@v4
+
+      # Setup Node.js
+      - name: Setup Node.js
+        uses: actions/setup-node@v4
+        with:
+          node-version: "20"
+
+      # Install dependencies
+      - name: Install dependencies
+        run: npm install
+
+      # Run benchmark on PR code
+      - name: Run benchmark on PR code
+        run: node bench/index.js
+        id: benchmark_pr
+
+      # Save PR benchmark results
+      - name: Save PR benchmark results
+        run: echo "PR_RESULTS=$(cat bench/results.json)" >> "$GITHUB_ENV"
+
+      # Prepare for main branch benchmark
+      - name: Backup PR src directory
+        run: mkdir _pr_branch && mv src _pr_branch/src
+
+      # Checkout main branch src directory
+      - name: Checkout main branch src directory
+        uses: actions/checkout@v4
+        with:
+          ref: "main"
+          path: "_main_branch"
+
+      # Replace PR src with main src
+      - name: Replace PR src with main src
+        run: |
+          rm -rf src
+          cp -R _main_branch/src src
+
+      # Run benchmark on main branch src with PR's benchmark tooling
+      - name: Run benchmark on main branch src
+        run: node bench/index.js
+        id: benchmark_main
+
+      # Save main benchmark results
+      - name: Save main benchmark results
+        run: echo "MAIN_RESULTS=$(cat bench/results.json)" >> "$GITHUB_ENV"
+
+      # Comment PR with benchmark results comparison
+      - name: Comment PR with benchmark results comparison
+        uses: actions/github-script@v7
+        with:
+          github-token: ${{secrets.GITHUB_TOKEN}}
+          script: |
+            const prResults = JSON.parse(Buffer.from(process.env.PR_RESULTS, 'base64').toString('utf8'));
+            const mainResults = JSON.parse(Buffer.from(process.env.MAIN_RESULTS, 'base64').toString('utf8'));
+
+            const commentBody = `
+              Benchmark Results Comparison (${context.sha}):
+
+              **PR Branch:**
+              \`\`\`json
+              ${JSON.stringify(prResults, null, 2)}
+              \`\`\`
+
+              **Main Branch:**
+              \`\`\`json
+              ${JSON.stringify(mainResults, null, 2)}
+              \`\`\`
+            `;
+
+            github.rest.issues.createComment({
+              issue_number: context.issue.number,
+              owner: context.repo.owner,
+              repo: context.repo.repo,
+              body: commentBody
+            });

package/.github/workflows/{ci.yml → npm-test.yml}

@@ -1,4 +1,4 @@
-name: CI
+name: npm-test
 
 on:
   push:
@@ -23,4 +23,4 @@ jobs:
         with:
           node-version: ${{ matrix.node-version }}
       - run: npm install
-      - run: npm run test
+      - run: npm test

package/README.md

@@ -61,10 +61,8 @@ const htmlString = html`
       ? html`
           <p>Data:</p>
           <ul>
-            ${Object.values(data).map(
-              ([key, val]) => `
-                ${key}: ${val}
-              `,
+            !${Object.values(data).map(
+              ([key, val]) => html`<li>${key}: ${val}</li>`,
             )}
           </ul>
         `

package/bench/index.js

@@ -0,0 +1,101 @@
+/* eslint-disable no-unused-expressions */
+import { html } from "../src/index.js";
+import { Bench } from "tinybench";
+import { writeFileSync } from "node:fs";
+import { Buffer } from "node:buffer";
+
+const bench = new Bench({ time: 500 });
+
+bench.add("Simple formatting", () => {
+  html`<div>Hello, world!</div>`;
+});
+
+const username = "User";
+bench.add("Using string variable", () => {
+  html`<p>${username}</p>`;
+});
+
+const value = null;
+const undef = undefined;
+bench.add("Handling null and undefined", () => {
+  html`<p>${value} and ${undef}</p>`;
+});
+
+const user = { id: 1, name: "John Doe" };
+bench.add("Multiple types of expressions", () => {
+  html`
+    ${undefined}
+    <div>User: <span>${user.name}</span></div>
+    <div>Id: <span>${user.id}</span></div>
+    ${null}
+  `;
+});
+
+const items = ["Item 1", "Item 2", "Item 3"];
+bench.add("Arrays and iteration", () => {
+  html`<ul>
+    ${items.map((item) => {
+      return html`<li>${item}</li>`;
+    })}
+  </ul>`;
+});
+
+const items2 = ["Item 1", undefined, "Item 2", null, 2000];
+bench.add("Arrays and iteration with multiple types", () => {
+  html`<ul>
+    ${items2.map((item) => {
+      return html`<li>${item}</li>`;
+    })}
+  </ul>`;
+});
+
+const loggedIn = true;
+bench.add("Complex/nested expressions", () => {
+  html`<nav>
+    ${loggedIn
+      ? html`<a href="/logout">Logout</a>`
+      : html`<a href="/login">Login</a>`}
+  </nav>`;
+});
+
+const largeString = Array.from({ length: 1000 }).join("Lorem ipsum ");
+bench.add("Large strings", () => {
+  html`<p>${largeString}</p>`;
+});
+
+bench.add("High iteration count", () => {
+  for (let i = 0; i !== 1000; i++) {
+    html`<span>${i}</span>`;
+  }
+});
+
+const scriptContent =
+  "<script>console.log('This should not execute');</script>";
+bench.add("Escape HTML", () => {
+  html`<div>${scriptContent} ${scriptContent}</div>`;
+});
+
+// Render raw HTML
+const rawHTML = "<em>Italic</em> and <strong>bold</strong>";
+const markup = "<mark>Highlighted</mark>";
+bench.add("Unescaped expressions", () => {
+  html`
+    <div>!${rawHTML}</div>
+    <div>!${rawHTML}</div>
+    <div>!${markup}</div>
+    <div>!${markup}</div>
+    <div>!${rawHTML}</div>
+    <div>!${rawHTML}</div>
+  `;
+});
+
+await bench.warmup();
+await bench.run();
+
+const table = bench.table();
+console.table(table);
+
+writeFileSync(
+  "bench/results.json",
+  Buffer.from(JSON.stringify(table), "utf8").toString("base64"),
+);

package/package.json

@@ -3,7 +3,7 @@
   "description": "Replace your template engine with fast JavaScript by leveraging the power of tagged templates.",
   "author": "Gürgün Dayıoğlu",
   "license": "MIT",
-  "version": "1.2.1",
+  "version": "1.2.3",
   "type": "module",
   "main": "./src/index.js",
   "exports": {
@@ -14,13 +14,15 @@
     "node": ">=18"
   },
   "scripts": {
-    "test": "npm run lint && node --experimental-test-coverage test/index.js",
+    "test": "npm run lint && c8 --100 node --test test/**",
     "lint": "eslint . && prettier --check .",
     "lint:fix": "eslint --fix . && prettier --write ."
   },
   "devDependencies": {
     "@fastify/pre-commit": "^2.1.0",
-    "grules": "^0.14.1"
+    "c8": "^9.1.0",
+    "grules": "^0.15.0",
+    "tinybench": "^2.6.0"
   },
   "repository": {
     "type": "git",

package/src/html.js

@@ -20,12 +20,12 @@ const escapeFunction = (key) => {
  * @param {...any} expressions Expressions to interpolate.
  * @returns {string} The HTML string.
  */
-const html = (literals, ...expressions) => {
+const html = ({ raw: literals }, ...expressions) => {
   let accumulator = "";
   let index = 0;
 
-  for (; index < expressions.length; ++index) {
-    let literal = literals.raw[index];
+  for (; index !== expressions.length; ++index) {
+    let literal = literals[index];
     let expression =
       typeof expressions[index] === "string"
         ? expressions[index]
@@ -44,7 +44,7 @@ const html = (literals, ...expressions) => {
     accumulator += literal + expression;
   }
 
-  return (accumulator += literals.raw[index]);
+  return (accumulator += literals[index]);
 };
 
 /**
@@ -52,11 +52,11 @@ const html = (literals, ...expressions) => {
  * @param {...any} expressions Expressions to interpolate.
  * @yields {string} The HTML strings.
  */
-const htmlGenerator = function* (literals, ...expressions) {
+const htmlGenerator = function* ({ raw: literals }, ...expressions) {
   let index = 0;
 
-  for (; index < expressions.length; ++index) {
-    let literal = literals.raw[index];
+  for (; index !== expressions.length; ++index) {
+    let literal = literals[index];
     let expression;
 
     if (typeof expressions[index] === "string") {
@@ -114,8 +114,8 @@ const htmlGenerator = function* (literals, ...expressions) {
     }
   }
 
-  if (literals.raw[index].length) {
-    yield literals.raw[index];
+  if (literals[index].length) {
+    yield literals[index];
   }
 };
 

package/test/index.js

@@ -1,7 +1,6 @@
-// eslint-disable-next-line n/no-missing-import
+import { html, htmlGenerator } from "../src/index.js";
 import test from "node:test";
 import assert from "node:assert";
-import { html, htmlGenerator } from "../src/index.js";
 
 const conditionTrue = true;
 const conditionFalse = false;
@@ -33,6 +32,10 @@ test("renders normal input", () => {
   assert.strictEqual(html`Hey, ${username}!`, `Hey, ${username}!`);
 });
 
+test("renders undefined and null as empty string", () => {
+  assert.strictEqual(html`<p>${null}${undefined}</p>`, "<p></p>");
+});
+
 test("renders safe content", () => {
   assert.strictEqual(
     html`<p>${descriptionSafe}</p>`,