@postalsys/email-text-tools 2.4.26 → 2.6.0

package/.github/VULNERABILITY_REPORT.yml

@@ -1,75 +0,0 @@
-name: '@postalsys/email-text-tools security vulnerability report'
-description: 'Report an exploitable security vulnerability in @postalsys/email-text-tools'
-
-body:
-    - type: markdown
-      attributes:
-          value: |
-              Before reporting, read our
-              [Security Policy](https://github.com/postalsys/email-text-tools/security/policy).
-
-              Reports must demonstrate a credible security impact.
-              Technically valid bugs without a realistic attack
-              scenario belong in the regular issue tracker.
-
-    - type: textarea
-      id: summary
-      attributes:
-          label: Vulnerability summary
-          description: Describe the vulnerability and affected versions.
-      validations:
-          required: true
-
-    - type: textarea
-      id: attacker
-      attributes:
-          label: Attacker capabilities
-          description: |
-              What can an attacker control without already having
-              privileged access to the application or its configuration?
-      validations:
-          required: true
-          min_length: 100
-
-    - type: textarea
-      id: exploit
-      attributes:
-          label: Realistic exploitation scenario
-          description: |
-              Describe how this can be exploited in a real application.
-              Explain the prerequisites and security boundary crossed.
-      validations:
-          required: true
-          min_length: 150
-
-    - type: textarea
-      id: poc
-      attributes:
-          label: Proof of concept
-          description: Provide reproducible steps or working code.
-      validations:
-          required: true
-
-    - type: textarea
-      id: impact
-      attributes:
-          label: Security impact
-          description: |
-              What can the attacker achieve? Who is affected?
-              Explain why this is a security vulnerability rather
-              than an ordinary bug.
-      validations:
-          required: true
-          min_length: 100
-
-    - type: checkboxes
-      id: confirmation
-      attributes:
-          label: Reporting requirements
-          options:
-              - label: I have read the Security Policy.
-                required: true
-              - label: I have verified the finding against a supported version.
-                required: true
-              - label: I have described a credible attacker-controlled scenario.
-                required: true

package/.github/workflows/release.yaml

@@ -1,38 +0,0 @@
-on:
-    push:
-        branches:
-            - main
-
-permissions:
-    contents: write
-    pull-requests: write
-    issues: write
-    id-token: write
-
-name: release
-jobs:
-    release-please:
-        runs-on: ubuntu-latest
-        outputs:
-            release_created: ${{ steps.release.outputs.release_created }}
-        steps:
-            - uses: googleapis/release-please-action@v5
-              id: release
-
-    publish:
-        needs: release-please
-        if: ${{ needs.release-please.outputs.release_created }}
-        runs-on: ubuntu-latest
-        environment: npm-publish
-        permissions:
-            contents: read
-            id-token: write
-        steps:
-            - uses: actions/checkout@v7
-            - uses: actions/setup-node@v7
-              with:
-                  node-version: 24
-                  package-manager-cache: false
-                  registry-url: 'https://registry.npmjs.org'
-            - run: npm ci
-            - run: npm publish --provenance --access public

package/.github/workflows/test.yaml

@@ -1,35 +0,0 @@
-name: Run tests
-
-on:
-    push:
-    pull_request:
-
-permissions:
-    contents: read
-
-concurrency:
-    group: ${{ github.workflow }}-${{ github.ref }}
-    cancel-in-progress: true
-
-jobs:
-    test:
-        strategy:
-            matrix:
-                node: [20.x, 22.x, 24.x, latest]
-                os: [ubuntu-latest]
-        runs-on: ${{ matrix.os }}
-        timeout-minutes: 10
-        steps:
-            - uses: actions/checkout@v7
-            - name: Use Node.js ${{ matrix.node }}
-              uses: actions/setup-node@v7
-              with:
-                  node-version: ${{ matrix.node }}
-                  cache: npm
-            - run: npm ci
-            - run: npm run lint
-            - run: npm run format:check
-            - run: npm test
-            - name: CommonJS dependency graph (require(esm) disabled)
-              if: matrix.node != '20.x'
-              run: npm run test:cjs

package/.ncurc.js

@@ -1,28 +0,0 @@
-'use strict';
-
-module.exports = {
-    upgrade: true,
-    reject: [
-        // Block package upgrades that moved to ESM
-        'nanoid',
-        'jsdom', // parse5 dependency in 27.0.1 is ESM only
-        'juice', // 12+ is pure ESM (requires Node >=22.12) and breaks pkg builds
-        // node-html-parser was rejected here on the grounds that 8+ depends on the pure-ESM entities@8.
-        // That is not how it ships: node-html-parser 9 bundles entities inline into dist/index.cjs and
-        // never require()s the package, so the ESM copy is installed but never loaded. Unpinned 2026-08,
-        // which also drops a duplicate parser from EmailEngine's tree (@postalsys/email-content already
-        // pulls 9.x). Keep it unpinned unless a release starts require()ing entities for real.
-        // 6.0.0 changed the CommonJS export shape (named bindings instead of the constructor) and,
-        // more importantly, stopped linkifying bare domains such as 'example.com' or 'www.example.org'
-        // by default, which silently breaks autolinking in plain text emails. Stay on 5.x.
-        'linkify-it',
-        // 2.0.0 is ESM-only (exports only src/he.mjs) and requires Node >=22, so require('he') throws
-        // and pkg cannot bundle it. 1.2.0 is the last CommonJS release; EmailEngine and mailparser
-        // hold it there too, and a different copy here would be a second he in their trees
-        'he',
-        // @asamuzakjp/css-color >=4.1.2 pulls in @csstools/* which are pure ESM, so require(jsdom) fails
-        // without require(esm) (jsdom -> cssstyle -> @asamuzakjp/css-color). Held at 4.1.1 through
-        // package.json "overrides"; overrides do not propagate to consumers, EmailEngine carries its own.
-        '@asamuzakjp/css-color'
-    ]
-};

package/.prettierignore

@@ -1,14 +0,0 @@
-# Auto-generated files
-CHANGELOG.md
-
-# Dependencies
-node_modules/
-package-lock.json
-
-# Build output
-dist/
-build/
-coverage/
-
-# Logs
-*.log

package/.prettierrc.js

@@ -1,10 +0,0 @@
-'use strict';
-
-module.exports = {
-    printWidth: 160,
-    tabWidth: 4,
-    singleQuote: true,
-    endOfLine: 'lf',
-    trailingComma: 'none',
-    arrowParens: 'avoid'
-};

package/.release-please-manifest.json

@@ -1,3 +0,0 @@
-{
-    ".": "2.4.26"
-}

package/CLAUDE.md

@@ -1,105 +0,0 @@
-# CLAUDE.md
-
-This file provides guidance to Claude Code (claude.ai/code) when working with code in this repository.
-
-## Commands
-
-```bash
-npm test                  # run the full suite (node --test test/*.test.js)
-npm run test:cjs          # the suite with require(esm) disabled (CommonJS graph guard)
-npm run test:coverage     # with experimental coverage
-npm run test:watch        # watch mode
-npm run lint              # eslint .
-npm run lint:fix          # eslint . --fix
-npm run format            # prettier --write .
-npm run format:check      # prettier --check .
-npm run update            # rm -rf node_modules package-lock.json && ncu -u && npm install
-```
-
-Run a single test file or a single test by name (uses Node's built-in test runner):
-
-```bash
-node --test test/mime-html.test.js
-node --test --test-name-pattern="should process HTML in worker thread"
-```
-
-Benchmarks are not wired into npm scripts; run them directly, e.g. `node benchmark/benchmark.js` (see `benchmark/` and `docs/PERFORMANCE.md`).
-
-## Critical constraints
-
-- **CommonJS only, Node 20 minimum.** See Platform constraints below. All source uses `require`/`module.exports` with a global `'use strict'` (ESLint enforces `sourceType: 'commonjs'`). The constraint is about the module format, not the syntax level: ESLint parses `ecmaVersion: 2022`, which every supported Node (>=20) and pkg target runs natively. The `require('juice').default || require('juice')` idiom at the require sites is a deliberate CJS-interop fallback, not dead code.
-- **Releases are automated via release-please.** Version in `package.json` and `CHANGELOG.md` are owned by the bot, do not bump them by hand. Conventional Commits drive it (`fix:` -> patch, `feat:` -> minor); `fix:`/`feat:` commits land directly on `main` and release-please opens the release PR. See Conventions below for the full commit rules.
-- **ESLint uses the shared flat config** (`eslint.config.js`, ESLint 10). It is the CommonJS template shared across the Postal Systems / Nodemailer packages (eslint:recommended plus the retired eslint-config-nodemailer rules, eslint-config-prettier last); keep it identical to the other repos except for the "Repo-specific" block at the end, where each override carries a one-line reason. The only override here turns `no-await-in-loop` off for `benchmark/`, which awaits sequentially on purpose. CI (`.github/workflows/test.yaml`) runs lint, `format:check` and the tests on Node 20, 22 and 24, plus `npm run test:cjs` on 22 and 24.
-
-## Platform constraints
-
-- **Node.js 20 is the minimum supported version** (`engines.node` is `>=20.0.0`). Runtime code, tests and tooling must work on Node 20, 22 and 24.
-- **Everything that reaches runtime must be genuinely CommonJS-loadable.** EmailEngine consumes this library and ships as a single-file binary built with [`@yao-pkg/pkg`](https://github.com/yao-pkg/pkg), which snapshots a CommonJS module graph and cannot load ES modules. `require()` of this package and of every runtime dependency must work with `--no-experimental-require-module`: Node 22+ (and 20.19) enable require(esm) by default, which hides an ESM-only dependency until the pkg binary fails. A CommonJS entry that only wraps a dynamic `import()` of ESM code does not count.
-- **How to check:** `npm run test:cjs` runs the whole suite under `NODE_OPTIONS=--no-experimental-require-module`; CI runs it on Node 22 and 24.
-- **Pins:** never add or upgrade to an ESM-only dependency. Hold the last CommonJS major in `.ncurc.js` (`reject` or `target`) with a one-line reason; `npm run update` honors it. Currently rejected: `nanoid`, `jsdom` (27.0.1 pulls an ESM-only parse5), `juice` (12+ is pure ESM and needs Node >=22.12), `linkify-it` (6.0.0 changed exports and stopped linkifying bare domains), `he` (2.0.0 is ESM-only), `@asamuzakjp/css-color`.
-- **The jsdom chain:** jsdom -> cssstyle -> `@asamuzakjp/css-color` >=4.1.2 pulls in `@csstools/*` v4, which are pure ESM. `package.json` `"overrides"` holds `@asamuzakjp/css-color` at 4.1.1 so this repo's own tests and CI exercise the CommonJS graph. Overrides do not propagate to consumers: an app installing this package resolves css-color itself, and EmailEngine carries its own identical override and `.ncurc.js` reject.
-
-## Conventions
-
-These mirror the conventions used across Postal Systems projects (see `../emailengine`).
-
-**Git commits**
-
-- Use Conventional Commits: `fix:` (patch release), `feat:` (minor release), and `chore:`/`build:`/`ci:`/`docs:`/`refactor:`/`style:`/`test:` for changes that should not trigger a release. release-please derives the version and changelog from these prefixes.
-- Do **not** list Claude or any AI assistant as a co-author / co-contributor in commit messages or PR descriptions (no `Co-Authored-By` trailer).
-- Append `[skip ci]` to commits that do not change runtime behavior (docs, comments, CI/workflow tweaks, formatting) to avoid spending CI minutes. Exception: never add `[skip ci]` to a `fix:` or `feat:` commit, those must run so the release workflow triggers.
-
-**Code & docs style**
-
-- No emojis anywhere in code or documentation, printable ASCII only.
-- Use a single hyphen-minus (`-`) as a dash in user-facing strings and docs. Never use double hyphens (`--`), em dashes, or en dashes.
-
-**Before committing code changes**
-
-1. `npm run format` and `npm run lint`
-2. `npm test`
-3. For non-trivial changes, consider `/simplify` (quality cleanup) and `/security-review`. This library sanitizes untrusted HTML, so anything touching the `mimeHtml` pipeline is security-sensitive.
-
-**After pushing**
-
-- Check the workflow runs for the push (`gh run list --branch <branch>`) and report their status. If a run fails for an unrelated/infrastructure reason (checkout reporting "account suspended", HTTP 403, other auth errors), check <https://www.githubstatus.com/> for an active GitHub incident before assuming the change caused it.
-
-## Architecture
-
-A small CommonJS library exposing five text-transformation functions for email content, all re-exported from `index.js` (`mimeHtml` through a lazy getter, so jsdom only loads on first use):
-
-| Export       | File                  | Purpose                                                     |
-| ------------ | --------------------- | ----------------------------------------------------------- |
-| `textToHtml` | `lib/text-to-html.js` | plain text -> HTML (linkified, quote-aware)                 |
-| `htmlToText` | `lib/html-to-text.js` | HTML -> plain text                                          |
-| `inlineHtml` | `lib/inline-html.js`  | build reply/forward HTML body with quoted original          |
-| `inlineText` | `lib/inline-text.js`  | build reply/forward plain-text body with quoted original    |
-| `mimeHtml`   | `lib/mime-html.js`    | sanitize + inline-CSS arbitrary email HTML for safe display |
-
-`lib/html-guard.js` holds what `mimeHtml` and `inlineHtml` share before they parse anything: the nesting-depth estimate (`exceedsNestingDepth`, `MAX_NESTING_DEPTH`) and the plain-text fallback projection (`fallbackText`). It does not load jsdom, so `inline-html.js` can require it eagerly.
-
-### `mimeHtml`: the core, and where the complexity lives
-
-Pipeline (same for sync and async): **DOMPurify sanitize (whole document)** -> **pre-filter problematic CSS** -> **juice (inline `<style>` into element `style=` attributes)** -> **DOMPurify sanitize again (return DOM body)** -> strip forbidden style properties / size constraints, force `target="_blank"` on links, wrap in a styled `<div>`. When the input has only `text` (no `html`), it is first run through `textToHtml` and given a default `<style>` block (`INLINE_STYLE_BLOCK`).
-
-The non-obvious part is **juice hang protection**. Certain modern CSS selectors (`:is()`, `:where()`, `:has()`, `:not(... : ...)`) can make `juice` hang. Two layers defend against this:
-
-1. A linear pre-filter (`filterCss()`, split on `}`, never a regex: the old regex backtracked catastrophically) strips rules using those selectors from head `<style>` tags before juice runs, after capping style text at 256 KB. Input nested deeper than `MAX_NESTING_DEPTH` is not parsed (jsdom tree building is quadratic in depth) and, like any pipeline error, renders the escaped-text `fallbackHtml()`.
-2. The async path additionally runs juice inside a **worker thread with a timeout** as a safety net for unknown edge cases. On timeout/failure it falls back to returning the HTML without inlined CSS (unless `fallbackOnError: false`).
-
-`mimeHtml` exposes three forms. Note the default is sync for backward compatibility, but only the async form is fully hang-protected:
-
-- `mimeHtml(message)` / `mimeHtml.sync(message)`: synchronous, runs juice in-process (can still hang on unknown selectors).
-- `mimeHtml.async(message, options)`: async, juice in a worker pool with timeout. Options: `timeout`, `useWorkerPool`, `fallbackOnError`, `minWorkers`/`maxWorkers`/`idleTimeout`.
-- `mimeHtml.closeWorkerPool()` / `mimeHtml.getWorkerPoolStats()`: manage the lazily-created global worker pool.
-
-`lib/worker-pool.js` (`WorkerPool`) manages a min/max pool of `lib/juice-worker.js` workers with task queueing, per-task timeouts, worker replacement on error/timeout, and idle-worker reaping. `lib/juice-worker.js` is a tiny worker entry that just runs `juice(html)` and posts the result back.
-
-### Other modules
-
-- `text-to-html.js` parses plain text into a nested tree of `text`/`quote` nodes (by `>` prefixes) so multi-level email quotes become nested `<blockquote class="ee-block-N">`. Linkification uses `linkify-it` + full `tlds` list (plus `.onion` and a `git:` alias).
-- `inline-html.js` / `inline-text.js` produce Apple-Mail-style reply/forward bodies, formatting From/Subject/Date/To/Cc/Bcc headers and quoting the original (HTML via `<blockquote>`, text via `> ` prefixes). Dates use `Intl.DateTimeFormat` via `lib/format-date.js`, honoring `options.locale` (unsupported -> 'en') and `options.tz` (unknown -> UTC).
-
-### Testing notes
-
-Tests use Node's built-in `node:test` runner. Suites that exercise `mimeHtml.async` or `WorkerPool` **must call `await mimeHtml.closeWorkerPool()` (or close the pool) in an `after()` hook**, otherwise live workers keep the process alive and the run hangs.

package/benchmark/benchmark.js

@@ -1,348 +0,0 @@
-#!/usr/bin/env node
-'use strict';
-
-const fs = require('fs');
-const path = require('path');
-const mimeHtml = require('../lib/mime-html');
-
-// Test data sets
-const testCases = {
-    simple: {
-        name: 'Simple HTML',
-        html: `
-            <!DOCTYPE html>
-            <html>
-            <head>
-                <style>
-                    body { font-family: Arial; }
-                    p { color: blue; }
-                </style>
-            </head>
-            <body>
-                <h1>Hello World</h1>
-                <p>This is a simple test email.</p>
-            </body>
-            </html>
-        `
-    },
-
-    moderate: {
-        name: 'Moderate HTML with inline styles',
-        html: `
-            <!DOCTYPE html>
-            <html>
-            <head>
-                <style>
-                    body { font-family: Arial, sans-serif; background: #f0f0f0; }
-                    .container { max-width: 600px; margin: 0 auto; padding: 20px; }
-                    h1 { color: #333; border-bottom: 2px solid #007bff; }
-                    p { line-height: 1.6; color: #666; }
-                    .button { display: inline-block; padding: 10px 20px; background: #007bff; color: white; text-decoration: none; }
-                    .footer { margin-top: 40px; padding-top: 20px; border-top: 1px solid #ddd; }
-                </style>
-            </head>
-            <body>
-                <div class="container">
-                    <h1>Newsletter Title</h1>
-                    <p>This is a moderately complex email template with various styles.</p>
-                    <p>Lorem ipsum dolor sit amet, consectetur adipiscing elit.</p>
-                    <a href="#" class="button">Call to Action</a>
-                    <div class="footer">
-                        <p>Footer content here</p>
-                    </div>
-                </div>
-            </body>
-            </html>
-        `
-    },
-
-    complex: {
-        name: 'Complex HTML with media queries',
-        html: `
-            <!DOCTYPE html>
-            <html>
-            <head>
-                <style>
-                    * { margin: 0; padding: 0; box-sizing: border-box; }
-                    body { font-family: -apple-system, BlinkMacSystemFont, 'Segoe UI', Roboto, Oxygen, Ubuntu, sans-serif; }
-                    .wrapper { max-width: 600px; margin: 0 auto; }
-                    .header { background: linear-gradient(135deg, #667eea 0%, #764ba2 100%); padding: 40px 20px; }
-                    .header h1 { color: white; font-size: 32px; }
-                    .content { padding: 40px 20px; background: white; }
-                    .card { border: 1px solid #e0e0e0; border-radius: 8px; padding: 20px; margin-bottom: 20px; }
-                    .card h2 { color: #333; margin-bottom: 10px; }
-                    .grid { display: grid; grid-template-columns: repeat(2, 1fr); gap: 20px; }
-                    @media only screen and (max-width: 600px) {
-                        .grid { grid-template-columns: 1fr; }
-                        .header h1 { font-size: 24px; }
-                    }
-                    table { width: 100%; border-collapse: collapse; }
-                    th, td { padding: 10px; text-align: left; border-bottom: 1px solid #ddd; }
-                    .button { display: inline-block; padding: 12px 24px; background: #007bff; color: white; text-decoration: none; border-radius: 4px; }
-                    .footer { background: #f8f9fa; padding: 30px 20px; text-align: center; color: #6c757d; }
-                </style>
-            </head>
-            <body>
-                <div class="wrapper">
-                    <div class="header">
-                        <h1>Complex Email Template</h1>
-                    </div>
-                    <div class="content">
-                        <div class="card">
-                            <h2>Feature One</h2>
-                            <p>Description of the first feature with detailed information.</p>
-                        </div>
-                        <div class="grid">
-                            <div class="card">
-                                <h2>Item 1</h2>
-                                <p>Grid item content</p>
-                            </div>
-                            <div class="card">
-                                <h2>Item 2</h2>
-                                <p>Grid item content</p>
-                            </div>
-                        </div>
-                        <table>
-                            <tr><th>Product</th><th>Price</th><th>Quantity</th></tr>
-                            <tr><td>Item A</td><td>$10</td><td>2</td></tr>
-                            <tr><td>Item B</td><td>$20</td><td>1</td></tr>
-                        </table>
-                        <a href="#" class="button">Shop Now</a>
-                    </div>
-                    <div class="footer">
-                        <p>Copyright 2024. All rights reserved.</p>
-                    </div>
-                </div>
-            </body>
-            </html>
-        `
-    },
-
-    problematic: {
-        name: 'HTML with problematic CSS (modern selectors)',
-        html: `
-            <!DOCTYPE html>
-            <html>
-            <head>
-                <style>
-                    body { font-family: system-ui, sans-serif; }
-                    /* Modern CSS that might cause issues */
-                    .container:is(.active, .highlighted) { background: yellow; }
-                    .item:where(.primary) { color: blue; }
-                    .parent:has(> .child) { border: 1px solid red; }
-                    /* Standard CSS */
-                    h1 { color: #333; }
-                    p { line-height: 1.6; }
-                    .box:not(.special) { padding: 10px; }
-                </style>
-            </head>
-            <body>
-                <div class="container active">
-                    <h1>Test with Modern CSS</h1>
-                    <div class="parent">
-                        <div class="child">Child element</div>
-                    </div>
-                    <p class="item primary">This uses modern selectors</p>
-                    <div class="box">Regular box</div>
-                    <div class="box special">Special box</div>
-                </div>
-            </body>
-            </html>
-        `
-    }
-};
-
-// Load the failing.html if it exists
-try {
-    const failingHtml = fs.readFileSync(path.join(__dirname, '../examples/failing.html'), 'utf8');
-    testCases.realWorld = {
-        name: 'Real-world problematic HTML',
-        html: failingHtml
-    };
-} catch (_err) {
-    // File doesn't exist, skip it
-}
-
-// Benchmark runner
-class Benchmark {
-    constructor() {
-        this.results = [];
-    }
-
-    async runTest(name, fn, iterations = 100) {
-        const times = [];
-        const errors = [];
-
-        // Warmup
-        for (let i = 0; i < 5; i++) {
-            try {
-                await fn();
-            } catch (_err) {
-                // Ignore warmup errors
-            }
-        }
-
-        // Actual test
-        for (let i = 0; i < iterations; i++) {
-            const start = process.hrtime.bigint();
-            try {
-                await fn();
-                const end = process.hrtime.bigint();
-                times.push(Number(end - start) / 1000000); // Convert to milliseconds
-            } catch (err) {
-                errors.push(err.message);
-            }
-        }
-
-        if (times.length > 0) {
-            times.sort((a, b) => a - b);
-            const avg = times.reduce((a, b) => a + b, 0) / times.length;
-            const median = times[Math.floor(times.length / 2)];
-            const min = times[0];
-            const max = times[times.length - 1];
-            const p95 = times[Math.floor(times.length * 0.95)];
-            const p99 = times[Math.floor(times.length * 0.99)];
-
-            return {
-                name,
-                iterations,
-                successful: times.length,
-                failed: errors.length,
-                avg: avg.toFixed(2),
-                median: median.toFixed(2),
-                min: min.toFixed(2),
-                max: max.toFixed(2),
-                p95: p95.toFixed(2),
-                p99: p99.toFixed(2),
-                errors: errors.length > 0 ? errors[0] : null
-            };
-        } else {
-            return {
-                name,
-                iterations,
-                successful: 0,
-                failed: errors.length,
-                errors: errors[0] || 'All iterations failed'
-            };
-        }
-    }
-
-    formatResults(results) {
-        console.log('\n' + '='.repeat(80));
-        console.log(`Test Case: ${results.testCase}`);
-        console.log('='.repeat(80));
-
-        const table = [];
-        results.tests.forEach(test => {
-            if (test.successful > 0) {
-                table.push({
-                    Method: test.name,
-                    'Success Rate': `${test.successful}/${test.iterations}`,
-                    'Avg (ms)': test.avg,
-                    'Median (ms)': test.median,
-                    'Min (ms)': test.min,
-                    'Max (ms)': test.max,
-                    'P95 (ms)': test.p95,
-                    'P99 (ms)': test.p99
-                });
-            } else {
-                table.push({
-                    Method: test.name,
-                    'Success Rate': `${test.successful}/${test.iterations}`,
-                    Error: test.errors
-                });
-            }
-        });
-
-        console.table(table);
-
-        // Calculate speedup if both succeeded
-        const syncTest = results.tests.find(t => t.name === 'Sync');
-        const asyncTest = results.tests.find(t => t.name === 'Async (with pool)');
-
-        if (syncTest && asyncTest && syncTest.successful > 0 && asyncTest.successful > 0) {
-            const speedup = (parseFloat(syncTest.avg) / parseFloat(asyncTest.avg)).toFixed(2);
-            if (speedup > 1) {
-                console.log(`\nAsync is ${speedup}x faster than Sync`);
-            } else {
-                console.log(`\nSync is ${(1 / speedup).toFixed(2)}x faster than Async`);
-            }
-        }
-    }
-
-    async runBenchmarks() {
-        console.log('Starting Mime HTML Benchmarks...');
-        console.log('================================\n');
-
-        for (const [_key, testCase] of Object.entries(testCases)) {
-            const results = {
-                testCase: testCase.name,
-                tests: []
-            };
-
-            // Test sync version
-            results.tests.push(await this.runTest('Sync', () => Promise.resolve(mimeHtml.sync({ html: testCase.html })), 100));
-
-            // Test async version with pool
-            results.tests.push(
-                await this.runTest(
-                    'Async (with pool)',
-                    async () =>
-                        await mimeHtml.async(
-                            { html: testCase.html },
-                            {
-                                useWorkerPool: true,
-                                timeout: 5000,
-                                minWorkers: 2,
-                                maxWorkers: 4
-                            }
-                        ),
-                    100
-                )
-            );
-
-            // Test async version without pool
-            results.tests.push(
-                await this.runTest(
-                    'Async (no pool)',
-                    async () =>
-                        await mimeHtml.async(
-                            { html: testCase.html },
-                            {
-                                useWorkerPool: false,
-                                timeout: 5000
-                            }
-                        ),
-                    100
-                )
-            );
-
-            this.formatResults(results);
-        }
-
-        // Show worker pool stats
-        const stats = mimeHtml.getWorkerPoolStats();
-        if (stats) {
-            console.log('\n' + '='.repeat(80));
-            console.log('Worker Pool Stats:');
-            console.log('='.repeat(80));
-            console.table([stats]);
-        }
-
-        // Clean up
-        await mimeHtml.closeWorkerPool();
-    }
-}
-
-// Run benchmarks
-const benchmark = new Benchmark();
-benchmark
-    .runBenchmarks()
-    .then(() => {
-        console.log('\nBenchmark complete!');
-        process.exit(0);
-    })
-    .catch(err => {
-        console.error('Benchmark failed:', err);
-        process.exit(1);
-    });