ep_table_of_contents 0.3.89 → 0.3.102

package/.github/workflows/backend-tests.yml

@@ -16,20 +16,20 @@ jobs:
     steps:
       -
         name: Install libreoffice
-        uses: awalsh128/cache-apt-pkgs-action@v1.4.2
+        uses: awalsh128/cache-apt-pkgs-action@v1.6.0
         with:
           packages: libreoffice libreoffice-pdfimport
           version: 1.0
       -
         name: Install etherpad core
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           repository: ether/etherpad-lite
           path: etherpad-lite
       - uses: pnpm/action-setup@v3
         name: Install pnpm
         with:
-          version: 8
+          version: 10
           run_install: false
       - name: Get pnpm store directory
         shell: bash
@@ -44,20 +44,9 @@ jobs:
             ${{ runner.os }}-pnpm-store-
       -
         name: Checkout plugin repository
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           path: plugin
-      -
-        name: Determine plugin name
-        id: plugin_name
-        working-directory: ./plugin
-        run: |
-          npx -c 'printf %s\\n "::set-output name=plugin_name::${npm_package_name}"'
-      -
-        name: Link plugin directory
-        working-directory: ./plugin
-        run: |
-          pnpm link --global
       - name: Remove tests
         working-directory: ./etherpad-lite
         run: rm -rf ./src/tests/backend/specs
@@ -65,28 +54,17 @@ jobs:
         name: Install Etherpad core dependencies
         working-directory: ./etherpad-lite
         run: bin/installDeps.sh
-      - name: Link plugin to etherpad-lite
+      - name: Install plugin
         working-directory: ./etherpad-lite
         run: |
-          pnpm link --global $PLUGIN_NAME
-          pnpm run install-plugins --path  ../../plugin
-        env:
-          PLUGIN_NAME: ${{ steps.plugin_name.outputs.plugin_name }}
-      - name: Link ep_etherpad-lite
-        working-directory: ./etherpad-lite/src
-        run: |
-          pnpm link --global
-      - name: Link etherpad to plugin
-        working-directory: ./plugin
-        run: |
-          pnpm link --global ep_etherpad-lite
+          pnpm run plugins i --path  ../../plugin
       -
         name: Run the backend tests
-        working-directory: ./etherpad-lite
+        working-directory: ./etherpad-lite/src
         run: |
-          res=$(find .. -path "./node_modules/ep_*/static/tests/backend/specs/**" | wc -l)
+          res=$(find ./plugin_packages -path "*/static/tests/backend/specs/*" 2>/dev/null | wc -l)
           if [ $res -eq 0 ]; then
           echo "No backend tests found"
           else
-          pnpm run test
+          npx cross-env NODE_ENV=production mocha --import=tsx --timeout 120000 --recursive node_modules/ep_*/static/tests/backend/specs/**
           fi

package/.github/workflows/frontend-tests.yml

@@ -12,13 +12,13 @@ jobs:
     steps:
       -
         name: Check out Etherpad core
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           repository: ether/etherpad-lite
       - uses: pnpm/action-setup@v3
         name: Install pnpm
         with:
-          version: 8
+          version: 10
           run_install: false
       - name: Get pnpm store directory
         shell: bash
@@ -33,7 +33,7 @@ jobs:
             ${{ runner.os }}-pnpm-store-
       -
         name: Check out the plugin
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           path: ./node_modules/__tmp
       -

package/.github/workflows/npmpublish.yml

@@ -15,13 +15,13 @@ jobs:
           node-version: 20
           registry-url: https://registry.npmjs.org/
       - name: Check out Etherpad core
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           repository: ether/etherpad-lite
       - uses: pnpm/action-setup@v3
         name: Install pnpm
         with:
-          version: 8
+          version: 10
           run_install: false
       - name: Get pnpm store directory
         shell: bash
@@ -35,7 +35,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-pnpm-store-
       -
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           fetch-depth: 0
       -

package/locales/be-tarask.json

@@ -0,0 +1,9 @@
+{
+	"@metadata": {
+		"authors": [
+			"Renessaince"
+		]
+	},
+	"ep_table_of_contents.toc.title": "Паказаць зьмест",
+	"ep_table_of_contents.toc": "Паказаць зьмест"
+}

package/locales/da.json

@@ -0,0 +1,9 @@
+{
+	"@metadata": {
+		"authors": [
+			"Peterleth"
+		]
+	},
+	"ep_table_of_contents.toc.title": "Vis indholdsfortegnelsen",
+	"ep_table_of_contents.toc": "Vis indholdsfortegnelsen"
+}

package/locales/el.json

@@ -1,8 +1,10 @@
 {
 	"@metadata": {
 		"authors": [
+			"Jimkats",
 			"Norhorn"
 		]
 	},
+	"ep_table_of_contents.toc.title": "Προβολή πίνακα περιεχομένων",
 	"ep_table_of_contents.toc": "Προβολή του Πίνακα Περιεχομένων"
 }

package/locales/fa.json

@@ -0,0 +1,9 @@
+{
+	"@metadata": {
+		"authors": [
+			"Darafsh"
+		]
+	},
+	"ep_table_of_contents.toc.title": "نمایش فهرست مطالب",
+	"ep_table_of_contents.toc": "نمایش فهرست مطالب"
+}

package/locales/ga.json

@@ -0,0 +1,9 @@
+{
+	"@metadata": {
+		"authors": [
+			"Aindriu80"
+		]
+	},
+	"ep_table_of_contents.toc.title": "Taispeáin Clár Ábhair",
+	"ep_table_of_contents.toc": "Taispeáin Clár Ábhair"
+}

package/locales/nl.json

@@ -1,7 +1,9 @@
 {
 	"@metadata": {
-		"authors": []
+		"authors": [
+			"McDutchie"
+		]
 	},
-	"ep_table_of_contents.toc.title": "Toon Inhoudsopgave",
-	"ep_table_of_contents.toc": "Toon Inhoudsopgave"
+	"ep_table_of_contents.toc.title": "Inhoudsopgave weergeven",
+	"ep_table_of_contents.toc": "Inhoudsopgave weergeven"
 }

package/locales/pl.json

@@ -0,0 +1,9 @@
+{
+	"@metadata": {
+		"authors": [
+			"Usagi.02808"
+		]
+	},
+	"ep_table_of_contents.toc.title": "Pokaż spis treści",
+	"ep_table_of_contents.toc": "Pokaż spis treści"
+}

package/locales/ps.json

@@ -0,0 +1,9 @@
+{
+	"@metadata": {
+		"authors": [
+			"شاه زمان پټان"
+		]
+	},
+	"ep_table_of_contents.toc.title": "نيوليک ښودل",
+	"ep_table_of_contents.toc": "نيوليک ښودل"
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ep_table_of_contents",
-  "version": "0.3.89",
+  "version": "0.3.102",
   "description": "View a table of contents for your pad",
   "author": {
     "name": "John McLear",
@@ -34,6 +34,7 @@
     "node": ">=18.0.0"
   },
   "scripts": {
+    "test": "node --test tests/*.test.js",
     "lint": "eslint .",
     "lint:fix": "eslint --fix ."
   }

package/static/css/toc.css

@@ -17,74 +17,31 @@
   cursor:pointer;
 }
 
-.toch1{
+.tocDepth1{
   margin-left:0px;
   font-size: 1.5rem;
 }
-.toch2{
+.tocDepth2{
   margin-left:10px;
   font-size: 1.2rem;
 }
-.toch3{
+.tocDepth3{
   margin-left:20px;
   font-size: 1rem;
 }
-.toch4{
+.tocDepth4{
   margin-left:30px;
   font-size:1rem;
 }
-.toch5{
+.tocDepth5{
   margin-left:30px;
   font-size:1rem;
 }
-.toch6{
+.tocDepth6{
   margin-left:30px;
   font-size:1rem;
 }
 
-
-/* Add styling to the first item in a list */
-
-#tocItems{counter-reset: first}
-.toch1 { counter-reset: second third fourth fifth sixth seventh;}
-.toch2 { counter-reset: third fourth fifth sixth seventh; }
-.toch3 { counter-reset: fourth fifth sixth seventh; }
-.toch4 { counter-reset: fifth sixth seventh; }
-.toch5 { counter-reset: sixth seventh; }
-.toch6 { counter-reset: seventh; }
-
-
-/* The behavior for incrementing and the prefix */
-.toch1:before {
-  content: counter(first) ". " ;
-  counter-increment: first;
-}
-
-.toch2:before {
-  content: counter(first) "." counter(second) ". ";
-  counter-increment: second;
-}
-
-.toch3:before {
-  content: counter(first) "." counter(second) "." counter(third) ". ";
-  counter-increment: third 1;
-}
-
-.toch4:before {
-  content: counter(first) "." counter(second) "." counter(third) "." counter(fourth) ". ";
-  counter-increment: fourth 1;
-}
-
-.toch5:before {
-  content: counter(first) "." counter(second) "." counter(third) "." counter(fourth) "." counter(fifth) ". ";
-  counter-increment: fifth 1;
-}
-
-.toch6:before {
-  content: counter(first) "." counter(second) "." counter(third) "." counter(fourth) "." counter(fifth) "." counter(sixth) ". ";
-  counter-increment: sixth 1;
-}
-
 .activeTOC{
   font-weight: 800;
 }

package/static/js/toc.js

@@ -1,8 +1,67 @@
 'use strict';
 
-$('#tocButton').click(() => {
-  $('#toc').toggle();
-});
+const getHeadingLevel = (tag) => {
+  const match = /^h([1-6])$/.exec(tag);
+  return match ? Number(match[1]) : null;
+};
+
+const getOutlineEntries = (toc) => {
+  const stack = [];
+  const counters = [];
+
+  const outlineEntries = toc.map((entry) => {
+    const level = getHeadingLevel(entry.tag);
+    if (level == null) {
+      return {
+        ...entry,
+        displayDepth: 1,
+        numbering: '',
+        numberParts: [],
+      };
+    }
+
+    while (stack.length > 0 && stack[stack.length - 1] >= level) {
+      stack.pop();
+    }
+
+    const depth = stack.length + 1;
+    counters.length = depth;
+    counters[depth - 1] = (counters[depth - 1] || 0) + 1;
+    stack.push(level);
+
+    return {
+      ...entry,
+      displayDepth: depth,
+      numbering: counters.join('.'),
+      numberParts: [...counters],
+    };
+  });
+
+  const topLevelHeadings = outlineEntries.filter((entry) => entry.numberParts.length === 1);
+  if (topLevelHeadings.length !== 1) return outlineEntries;
+
+  return outlineEntries.map((entry) => {
+    if (entry.numberParts.length === 0) return entry;
+    if (entry.numberParts.length === 1) {
+      return {
+        ...entry,
+        numbering: '',
+      };
+    }
+
+    return {
+      ...entry,
+      displayDepth: entry.displayDepth - 1,
+      numbering: entry.numberParts.slice(1).join('.'),
+    };
+  });
+};
+
+if (typeof $ !== 'undefined') {
+  $('#tocButton').click(() => {
+    $('#toc').toggle();
+  });
+}
 
 const tableOfContents = {
 
@@ -17,9 +76,7 @@ const tableOfContents = {
 
   // Find Tags
   findTags: () => {
-    const toc = {}; // The main object we will use
-    const tocL = {}; // A per line record of each TOC item
-    let count = 0;
+    const toc = [];
     let delims = ['h1', 'h2', 'h3', 'h4', 'h5', 'h6', '.h1', '.h2', '.h3', '.h4', '.h5', '.h6'];
     if (clientVars.plugins.plugins.ep_context) {
       if (clientVars.plugins.plugins.ep_context.styles) {
@@ -47,37 +104,29 @@ const tableOfContents = {
         linkText = linkText.replace(/\s*#*/, '');
       }
 
-      // Create an object of lineNumbers that include the tag
-      tocL[lineNumber] = tag;
-
-      // Does the previous line already have this delim?
-      // If so do nothing..
-      if (tocL[lineNumber - 1]) {
-        if (tocL[lineNumber - 1] === tag) return;
-      }
-
-      toc[count] = {
+      toc.push({
         tag,
         y: newY,
         text: linkText,
         focusId,
         lineNumber,
-      };
-      count++;
+      });
     });
 
+    const outlineEntries = getOutlineEntries(toc);
     clientVars.plugins.plugins.ep_table_of_context = toc;
     $('#tocItems').html('');
-    $.each(toc, (h, v) => { // for each item we should display
+    $.each(outlineEntries, (index, entry) => {
+      const label = entry.numbering ? `${entry.numbering}. ${entry.text}` : entry.text;
       const $link = $('<a>', {
-        text: v.text,
-        title: v.text,
+        text: label,
+        title: entry.text,
         href: '#',
-        class: `tocItem toc${v.tag}`,
-        click: () => { tableOfContents.scroll(`${v.y}`); return false; },
+        class: `tocItem tocDepth${Math.min(entry.displayDepth, 6)}`,
+        click: () => { tableOfContents.scroll(`${entry.y}`); return false; },
       });
-      $link.data('class', `toc${v.tag}`);
-      $link.data('offset', `${v.y}`);
+      $link.attr('data-toc-index', index);
+      $link.data('offset', `${entry.y}`);
       $link.appendTo('#tocItems');
     });
   },
@@ -98,22 +147,16 @@ const tableOfContents = {
     const repLineNumber = rep.selEnd[0]; // line Number
 
     // So given a line number of 10 and a toc of [4,8,12] we want to find 8..
+    let activeTocIndex = null;
     $.each(toc, (k, line) => {
       if (repLineNumber >= line.lineNumber) {
-        // we might be showing this..
-        const nextLine = toc[k];
-        if (nextLine.lineNumber <= repLineNumber) {
-          const activeToc = parseInt(k) + 1;
-
-          // Seems expensive, we go through each item and remove class
-          $('.tocItem').each(function () {
-            $(this).removeClass('activeTOC');
-          });
-
-          $(`.toch${activeToc}`).addClass('activeTOC');
-        }
+        activeTocIndex = Number(k);
       }
     });
+
+    $('.tocItem').removeClass('activeTOC');
+    if (activeTocIndex === null) return;
+    $(`.tocItem[data-toc-index="${activeTocIndex}"]`).addClass('activeTOC');
   },
 
   update: (rep) => {

package/static/tests/backend/specs/exportHTML.ts

@@ -0,0 +1,84 @@
+'use strict';
+
+import {strict as assert} from 'assert';
+import * as common from 'ep_etherpad-lite/tests/backend/common';
+import {randomString} from 'ep_etherpad-lite/static/js/pad_utils';
+
+let agent:any;
+const apiVersion = 1;
+
+const authGet = async (url: string) =>
+    agent.get(url).set('authorization', await common.generateJWTToken());
+
+const createPad = async (padId: string) => {
+  const res = await authGet(`/api/${apiVersion}/createPad?padID=${padId}`);
+  assert.equal(res.body.code, 0, 'Unable to create new pad');
+  return padId;
+};
+
+const setHTML = async (padId: string, html: string) => {
+  const res = await authGet(`/api/${apiVersion}/setHTML?padID=${padId}&html=${encodeURIComponent(html)}`);
+  assert.equal(res.body.code, 0, 'Unable to set pad HTML');
+  return padId;
+};
+
+const getHTMLEndpointFor = (padId: string) => `/api/${apiVersion}/getHTML?padID=${padId}`;
+
+const buildHTML = (body: string) => `<!doctype html><html><body>${body}</body></html>`;
+
+describe('TOC export/import behavior', function () {
+  let padId:string;
+
+  before(async function () {
+    agent = await common.init();
+  });
+
+  beforeEach(async function () {
+    padId = randomString(10);
+    await createPad(padId);
+    await setHTML(padId, buildHTML('<h1>Title</h1><h2>Section A</h2><h2>Section B</h2>'));
+  });
+
+  it('keeps the import/export HTML free of TOC sidebar markup', async function () {
+    const res = await authGet(getHTMLEndpointFor(padId));
+    assert.equal(res.status, 200);
+
+    const html = res.body.data.html;
+    assert.match(html, /Title/);
+    assert.match(html, /Section A/);
+    assert.match(html, /Section B/);
+    assert.doesNotMatch(html, /id="toc"/);
+    assert.doesNotMatch(html, /id="tocItems"/);
+    assert.doesNotMatch(html, /tocItem/);
+    assert.doesNotMatch(html, /Table of Contents/);
+    assert.doesNotMatch(html, /ep_table_of_contents\/static\/js\/toc\.js/);
+  });
+
+  it('does not inject TOC sidebar markup into HTML export', async function () {
+    const res = await authGet(`/p/${padId}/export/html`);
+    assert.equal(res.status, 200);
+
+    const html = res.text;
+    assert.match(html, /Title/);
+    assert.match(html, /Section A/);
+    assert.match(html, /Section B/);
+    assert.doesNotMatch(html, /id="toc"/);
+    assert.doesNotMatch(html, /id="tocItems"/);
+    assert.doesNotMatch(html, /tocItem/);
+    assert.doesNotMatch(html, /Table of Contents/);
+    assert.doesNotMatch(html, /ep_table_of_contents\/static\/js\/toc\.js/);
+    assert.doesNotMatch(html, /0\.1/);
+  });
+
+  it('does not inject TOC sidebar text into plain text export', async function () {
+    const res = await authGet(`/p/${padId}/export/txt`);
+    assert.equal(res.status, 200);
+
+    const text = res.text;
+    assert.match(text, /Title/);
+    assert.match(text, /Section A/);
+    assert.match(text, /Section B/);
+    assert.doesNotMatch(text, /Table of Contents/);
+    assert.doesNotMatch(text, /0\.1/);
+  });
+});

package/static/tests/frontend-new/playwright.config.js

@@ -0,0 +1,15 @@
+'use strict';
+
+const path = require('node:path');
+
+module.exports = {
+  testDir: path.join(__dirname, 'specs'),
+  testMatch: '**/*.spec.js',
+  timeout: 90 * 1000,
+  retries: process.env.CI ? 2 : 0,
+  use: {
+    baseURL: 'http://127.0.0.1:9001',
+    viewport: {width: 1280, height: 3000},
+    trace: 'on-first-retry',
+  },
+};

package/static/tests/frontend-new/specs/toc.spec.js

@@ -0,0 +1,84 @@
+const {expect, test} = require('@playwright/test');
+const {
+  clearPadContent,
+  getPadBody,
+  goToNewPad,
+  writeToPad,
+} = require('ep_etherpad-lite/tests/frontend-new/helper/padHelper');
+
+const enableToc = async (page) => {
+  await page.locator('.buttonicon-settings').click();
+  await page.locator('label[for="options-toc"]').click();
+  await expect(page.locator('#toc')).toBeVisible();
+};
+
+const writeSections = async (page, count, prefix) => {
+  for (let i = 0; i < count; i++) {
+    await writeToPad(page, `${prefix} ${i + 1}`);
+    if (i < count - 1) await page.keyboard.press('Enter');
+  }
+};
+
+const applyHeading = async (page, lineNumber, headingLevel) => {
+  const padBody = await getPadBody(page);
+  await padBody.locator('div').nth(lineNumber).selectText();
+  await page.evaluate((value) => {
+    $('#heading-selection').val(String(value)).trigger('change');
+  }, headingLevel - 1);
+};
+
+test.beforeEach(async ({page}) => {
+  await goToNewPad(page);
+  await enableToc(page);
+  await clearPadContent(page);
+});
+
+test.describe('table of contents numbering', () => {
+  test('starts sibling h2 headings at 1 when there is no h1', async ({page}) => {
+    await writeToPad(page, 'First section');
+    await page.keyboard.press('Enter');
+    await writeToPad(page, 'Second section');
+
+    await applyHeading(page, 0, 2);
+    await applyHeading(page, 1, 2);
+
+    const tocItems = page.locator('#tocItems .tocItem');
+    await expect(tocItems).toHaveCount(2);
+    await expect(tocItems.nth(0)).toHaveText('1. First section');
+    await expect(tocItems.nth(1)).toHaveText('2. Second section');
+  });
+
+  test('keeps a single top-level heading unnumbered and starts children at 1', async ({page}) => {
+    await writeToPad(page, 'Document title');
+    await page.keyboard.press('Enter');
+    await writeToPad(page, 'First section');
+    await page.keyboard.press('Enter');
+    await writeToPad(page, 'Second section');
+
+    await applyHeading(page, 0, 1);
+    await applyHeading(page, 1, 2);
+    await applyHeading(page, 2, 2);
+
+    const tocItems = page.locator('#tocItems .tocItem');
+    await expect(tocItems).toHaveCount(3);
+    await expect(tocItems.nth(0)).toHaveText('Document title');
+    await expect(tocItems.nth(1)).toHaveText('1. First section');
+    await expect(tocItems.nth(2)).toHaveText('2. Second section');
+  });
+
+  test('keeps all TOC entries for larger sets of sibling headings', async ({page}) => {
+    const headingCount = 25;
+    await writeSections(page, headingCount, 'Section');
+
+    const tocItems = page.locator('#tocItems .tocItem');
+    for (let i = 0; i < headingCount; i++) {
+      await applyHeading(page, i, 2);
+      await expect(page.locator(`#tocItems .tocItem[title="Section ${i + 1}"]`)).toBeVisible({timeout: 15_000});
+    }
+
+    await expect(tocItems).toHaveCount(headingCount);
+    await expect(tocItems.nth(0)).toHaveText('1. Section 1');
+    await expect(tocItems.nth(9)).toHaveText('10. Section 10');
+    await expect(tocItems.nth(24)).toHaveText('25. Section 25');
+  });
+});

package/tests/toc-numbering.test.js

@@ -0,0 +1,90 @@
+'use strict';
+
+const assert = require('node:assert/strict');
+const fs = require('node:fs');
+const path = require('node:path');
+const vm = require('node:vm');
+const test = require('node:test');
+
+const loadTocHelpers = () => {
+  const tocPath = path.join(__dirname, '..', 'static', 'js', 'toc.js');
+  const source = fs.readFileSync(tocPath, 'utf8');
+  const sandbox = {
+    console,
+    URLSearchParams,
+    globalThis: {},
+    $: () => ({
+      click() {},
+    }),
+  };
+  sandbox.globalThis = sandbox;
+  vm.runInNewContext(`${source}
+globalThis.__tocTestExports = {getHeadingLevel, getOutlineEntries};`, sandbox, {filename: tocPath});
+  return sandbox.__tocTestExports;
+};
+
+const {getOutlineEntries} = loadTocHelpers();
+
+const makeEntries = (tags) => tags.map((tag, index) => ({
+  tag,
+  text: `Heading ${index + 1}`,
+  lineNumber: index,
+}));
+
+const summarize = (entries) => getOutlineEntries(makeEntries(entries)).map((entry) => ({
+  depth: entry.displayDepth,
+  numbering: entry.numbering,
+}));
+
+test('starts first visible heading at 1 when h1 is missing', () => {
+  assert.deepEqual(summarize(['h2', 'h3', 'h3']), [
+    {depth: 1, numbering: ''},
+    {depth: 1, numbering: '1'},
+    {depth: 1, numbering: '2'},
+  ]);
+});
+
+test('increments sibling headings instead of repeating 0.1', () => {
+  assert.deepEqual(summarize(['h2', 'h2', 'h2']), [
+    {depth: 1, numbering: '1'},
+    {depth: 1, numbering: '2'},
+    {depth: 1, numbering: '3'},
+  ]);
+});
+
+test('keeps a single top-level heading unnumbered and starts children at 1', () => {
+  assert.deepEqual(summarize(['h1', 'h2', 'h2']), [
+    {depth: 1, numbering: ''},
+    {depth: 1, numbering: '1'},
+    {depth: 1, numbering: '2'},
+  ]);
+});
+
+test('preserves hierarchical numbering when there are multiple top-level sections', () => {
+  assert.deepEqual(summarize(['h1', 'h2', 'h1', 'h2']), [
+    {depth: 1, numbering: '1'},
+    {depth: 2, numbering: '1.1'},
+    {depth: 1, numbering: '2'},
+    {depth: 2, numbering: '2.1'},
+  ]);
+});
+
+test('counts nested sections correctly across section changes', () => {
+  assert.deepEqual(summarize(['h2', 'h3', 'h2', 'h3', 'h4']), [
+    {depth: 1, numbering: '1'},
+    {depth: 2, numbering: '1.1'},
+    {depth: 1, numbering: '2'},
+    {depth: 2, numbering: '2.1'},
+    {depth: 3, numbering: '2.1.1'},
+  ]);
+});
+
+test('keeps numbering stable across many sibling headings', () => {
+  const headingCount = 200;
+  const summary = summarize(Array.from({length: headingCount}, () => 'h2'));
+
+  assert.equal(summary.length, headingCount);
+  assert.deepEqual(summary[0], {depth: 1, numbering: '1'});
+  assert.deepEqual(summary[99], {depth: 1, numbering: '100'});
+  assert.deepEqual(summary[199], {depth: 1, numbering: '200'});
+});