@jrpool/kilotest 57.2.0 → 57.2.3

package/web/listDiagnoses/listDiagnoses.test.ts

@@ -14,6 +14,7 @@ import {parse} from 'node-html-parser';
 // Mock the util functions called by index.ts before requiring it, so that
 // it imports the mocked versions. Other exports delegate to the real module.
 import * as realUtil from '../../util.ts';
+import * as realWebUtil from '../util.ts';
 let getReportCallCount = 0;
 let failGetReportOnCall = -1;
 let pageDataStringsOverride: any = null;
@@ -26,12 +27,17 @@ mock.module('../../util.ts', {
         return {error: 'Report is invalid (test override)'};
       }
       return (realUtil.getReport as any)(...args);
-    },
+    }
+  }
+} as any);
+mock.module('../util.ts', {
+  exports: {
+    ...realWebUtil,
     getPageDataStrings: async (...args: any[]) => {
       if (pageDataStringsOverride !== null) {
         return pageDataStringsOverride;
       }
-      return (realUtil.getPageDataStrings as any)(...args);
+      return (realWebUtil.getPageDataStrings as any)(...args);
     }
   }
 } as any);

package/web/listIssues/index.ts

@@ -6,19 +6,21 @@
 // IMPORTS
 
 import {
-  getPageData,
-  getPageDataStrings,
   getReport,
-  getEngineNamesString,
   getTestActInstances,
-  getWCAGLink,
-  getWeightName,
   htmlSafe,
   isUsableReport,
   objectSort,
   populateTemplate,
   ruleEngines
 } from '../../util.ts';
+import {
+  getPageData,
+  getPageDataStrings,
+  getEngineNamesString,
+  getWCAGLink,
+  getWeightName
+} from '../util.ts';
 import {issues as issueSpecs} from 'testaro-issues';
 
 // FUNCTIONS

package/web/listIssues/listIssues.test.ts

@@ -13,15 +13,16 @@ import {parse} from 'node-html-parser';
 // Mock the util functions called by index.ts before requiring it, so that
 // it imports the mocked versions. Other exports delegate to the real module.
 import * as realUtil from '../../util.ts';
+import * as realWebUtil from '../util.ts';
 let pageDataStringsOverride: any = null;
-mock.module('../../util.ts', {
+mock.module('../util.ts', {
   exports: {
-    ...realUtil,
+    ...realWebUtil,
     getPageDataStrings: async (...args: any[]) => {
       if (pageDataStringsOverride !== null) {
         return pageDataStringsOverride;
       }
-      return (realUtil.getPageDataStrings as any)(...args);
+      return (realWebUtil.getPageDataStrings as any)(...args);
     }
   }
 } as any);

package/web/listReports/index.ts

@@ -7,18 +7,20 @@
 
 import {
   getAgoDays,
-  getCountString,
   getJobNames,
-  getMultiReportWhats,
   getObject,
-  getPageDataStrings,
   getTestRequests,
-  getReportData,
   getReportExtracts,
   jobsPath,
   objectSort,
   populateTemplate
 } from '../../util.ts';
+import {
+  getCountString,
+  getMultiReportWhats,
+  getPageDataStrings,
+  getReportData
+} from '../util.ts';
 import path from 'node:path';
 
 // FUNCTIONS

package/web/listRules/index.ts

@@ -6,11 +6,13 @@
 // IMPORTS
 
 import {
-  getWeightName,
   htmlSafe,
   populateTemplate,
   ruleEngines
 } from '../../util.ts';
+import {
+  getWeightName
+} from '../util.ts';
 import {issues as issueSpecs, issueRules, rules as ruleSpecs} from 'testaro-issues';
 
 // FUNCTIONS

package/web/listTopIssues/index.ts

@@ -7,16 +7,18 @@
 
 import {sendAlert} from '../../alerts.ts';
 import {
-  getEngineNamesString,
   getReportExtracts,
   getReport,
   getTestActInstances,
-  getWCAGLink,
-  getWeightName,
   isReportError,
   objectSort,
-  populateTemplate,
+  populateTemplate
 } from '../../util.ts';
+import {
+  getEngineNamesString,
+  getWCAGLink,
+  getWeightName
+} from '../util.ts';
 import {issues as issueSpecs} from 'testaro-issues';
 
 // FUNCTIONS

package/web/listViolators/index.ts

@@ -5,20 +5,22 @@
 
 // IMPORTS
 
+import {
+  getReport,
+  getTestActInstances,
+  htmlSafe,
+  isReportError,
+  populateTemplate
+} from '../../util.ts';
 import {
   getPageDataStrings,
   getPathID,
-  getReport,
   getEngineNamesString,
-  getTestActInstances,
   getTextFragmentHref,
   getWCAGLink,
   getWeightName,
-  htmlSafe,
-  isReportError,
-  makeBreakable,
-  populateTemplate,
-} from '../../util.ts';
+  makeBreakable
+} from '../util.ts';
 import {issues as issueSpecs} from 'testaro-issues';
 
 // FUNCTIONS

package/web/listViolators/listViolators.test.ts

@@ -14,6 +14,7 @@ import {parse} from 'node-html-parser';
 // Mock the util functions called by index.ts before requiring it, so that
 // it imports the mocked versions. Other exports delegate to the real module.
 import * as realUtil from '../../util.ts';
+import * as realWebUtil from '../util.ts';
 let getReportCallCount = 0;
 let failGetReportOnCall = -1;
 let pageDataStringsOverride: any = null;
@@ -26,12 +27,17 @@ mock.module('../../util.ts', {
         return {error: 'Report is invalid (test override)'};
       }
       return (realUtil.getReport as any)(...args);
-    },
+    }
+  }
+} as any);
+mock.module('../util.ts', {
+  exports: {
+    ...realWebUtil,
     getPageDataStrings: async (...args: any[]) => {
       if (pageDataStringsOverride !== null) {
         return pageDataStringsOverride;
       }
-      return (realUtil.getPageDataStrings as any)(...args);
+      return (realWebUtil.getPageDataStrings as any)(...args);
     }
   }
 } as any);

package/web/metrics/index.ts

@@ -7,9 +7,16 @@
 // IMPORTS
 
 import {
-  clearMetrics, getDateTimeString, getExclusionCookieValue, getMetrics, isValidAuthCode,
-  metricsExclusionCookieName, populateTemplate
+  clearMetrics,
+  getExclusionCookieValue,
+  getMetrics,
+  isValidAuthCode,
+  metricsExclusionCookieName,
+  populateTemplate
 } from '../../util.ts';
+import {
+  getDateTimeString
+} from '../util.ts';
 
 // FUNCTIONS
 

package/web/reportDeletion.ts

@@ -6,9 +6,16 @@
 // IMPORTS
 
 import {
-  errorMessage, getReportData, isValidAuthCode, objectSort, populateTemplate, readdirOrCreate,
+  errorMessage,
+  isValidAuthCode,
+  objectSort,
+  populateTemplate,
+  readdirOrCreate,
   reportsPath
 } from '../util.ts';
+import {
+  getReportData
+} from './util.ts';
 import fs from 'node:fs/promises';
 import path from 'node:path';
 

package/web/requestRetestForm/index.ts

@@ -5,7 +5,8 @@
 
 // IMPORTS
 
-import {getAgoString, getDateTimeString, getReportExtracts, populateTemplate} from '../../util.ts';
+import {getReportExtracts, populateTemplate} from '../../util.ts';
+import {getAgoString, getDateTimeString} from '../util.ts';
 
 // FUNCTIONS
 

package/web/tutorialAI/index.html

@@ -17,47 +17,34 @@
       <h2>Purpose</h2>
       <p>This tutorial shows you how to make Kilotest available to AI assistants. Suppose:</p>
       <ul>
-        <li>You use an AI platform, like Claude, ChatGPT, Perplexity, or Mistral Vibe.</li>
+        <li>You use an AI platform, like Claude, Mistral Vibe, or Perplexity Pro.</li>
         <li>You want to audit the quality of a web page.</li>
       </ul>
       <p>Then the models on your AI platform can use Kilotest <strong>if</strong> you make Kilotest available on the platform.</p>
       <h2>What AI platform do you use?</h2>
       <ul>
         <li><a href="#claude">Claude.ai in a browser or Claude Desktop</a></li>
-        <li><a href="#chatgpt">ChatGPT</a></li>
-        <li><a href="#perplexity">Perplexity</a></li>
         <li><a href="#vibe">Mistral Vibe</a></li>
+        <li><a href="#perplexity">Perplexity Pro in a browser</a></li>
       </ul>
-      <h2 id="claude">Claude.ai <a href="https://claude.ai">in a browser</a> or <a href="https://downloads.claude.com">Claude Desktop</a></h2>
-      <p>To make Kilotest available on either Claude platform:</p>
+      <h2 id="claude"><a href="https://claude.ai">Claude.ai in a browser</a> or <a href="https://downloads.claude.com">Claude Desktop</a></h2>
       <ol>
         <li><a href="https://claude.ai/new">Start a new task</a>.</li>
         <li>Expand the left sidebar to see the labels.</li>
-        <li>Click <q>Customize</q>.</li>
-        <li>Click <q>Connectors</q>.</li>
-        <li>In the <q>Add</q> list, click <q>Add custom connector</q>.</li>
+        <li>Click the <q>Customize</q> button.</li>
+        <li>Click the <q>Connectors</q> button.</li>
+        <li>In the <q>Add</q> list, click the <q>Add custom connector</q> button.</li>
         <li>Enter <q>Kilotest</q> into the <q>Name</q> input.</li>
         <li>Enter <q>https://kilotest.com/mcp</q> into the <q>Remote MCP server URL</q> input.</li>
         <li>Leave the OAuth inputs empty.</li>
-        <li>Click <q>Add</q>.</li>
+        <li>Click the <q>Add</q> button.</li>
         <li>For maximum autonomy, click <q>Always allow</q> for each tool.</li>
         <li>Click the <code>x</code> button or press the <code>Escape</code> key to close the dialog.</li>
       </ol>
-      <h2 id="chatgpt">ChatGPT</h2>
-      <p>To make Kilotest available on ChatGPT:</p>
-      <ol>
-        <li>Forthcoming</li>
-      </ol>
-      <h2 id="perplexity">Perplexity</h2>
-      <p>To make Kilotest available on Perplexity:</p>
-      <ol>
-        <li>Forthcoming</li>
-      </ol>
-      <h2 id="Vibe">Mistral Vibe</h2>
-      <p>To make Kilotest available on Mistral Vibe:</p>
+      <h2 id="vibe">Mistral Vibe</h2>
       <ol>
         <li>Choose the <a href="https://chat.mistral.ai/work">Work tab</a>.</li>
-        <li>Click <q>Connect your apps for better answers</q></li>
+        <li>Click <q>Connect your apps for better answers</q>.</li>
         <li>Click the <q>+ Add connector</q> button.</li>
         <li>Click the <q>Add a custom connector</q> button.</li>
         <li>Enter <q>Kilotest</q> into the <q>Connector name</q> input.</li>
@@ -65,6 +52,22 @@
         <li>Click the <q>Add connector</q> button.</li>
         <li>Toggle the <q>Ask/Allow</q> buttons to show <q>Allow</q>.</li>
       </ol>
+      <h2 id="perplexity">Perplexity Pro in a browser</h2>
+      <ol>
+        <li>After signing in, click the <q>Customize</q> button in the left sidebar.</li>
+        <li>Click the <q>+ Custom connector</q> menu button.</li>
+        <li>Click the <q>Add MCP connector</q> menu item.</li>
+        <li>Enter <q>Kilotest</q> into the <q>Name</q> input.</li>
+        <li>Enter <q>https://kilotest.com/mcp</q> into the <q>MCP server URL</q> input.</li>
+        <li>Click the <q>Advanced</q> menu button to open the advanced settings.</li>
+        <li>Under <q>Authentication</q>, change the authentication type to <q>None</q>.</li>
+        <li>Check the <q>I understand &hellip;</q> checkbox.</li>
+        <li>Click the <q>Add</q> button.</li>
+      </ol>
+      <p>Once you add Kilotest, it <strong>may</strong> become available in the Perplexity app on your computer or mobile device, too.</p>
+      <h2>Other AI platforms</h2>
+      <p>You can connect Kilotest to some other AI platforms, too. The instructions above are limited to platforms that are either free or relatively inexpensive (US$20 per month or less). If you discover an AI platform that should be added to this page, please submit a comment below.</p>
+      <h2>Trouble?</h2>
       <p>What if this did not work? Use the comment form below to tell us about it.</p>
       <p>You can also submit an issue on the <a href="https://github.com/jrpool/kilotest/issues">Kilotest GitHub repository</a>.</p>
       <section id="suggest-improvement" class="main">

package/web/tutorialAI/index.ts

@@ -6,7 +6,8 @@
 // IMPORTS
 
 import {sendAlert} from '../../alerts.ts';
-import {checkCommentDuplicate, checkCommentLength, describeMax, getEnvMax, getJSON, getNowStamp} from '../../util.ts';
+import {checkCommentDuplicate, describeMax, getEnvMax, getJSON, getNowStamp} from '../../util.ts';
+import {checkCommentLength} from '../util.ts';
 import fs from 'node:fs/promises';
 import path from 'node:path';
 

package/web/tutorialWeb/index.ts

@@ -6,7 +6,8 @@
 // IMPORTS
 
 import {sendAlert} from '../../alerts.ts';
-import {checkCommentDuplicate, checkCommentLength, describeMax, getEnvMax, getJSON, getNowStamp} from '../../util.ts';
+import {checkCommentDuplicate, describeMax, getEnvMax, getJSON, getNowStamp} from '../../util.ts';
+import {checkCommentLength} from '../util.ts';
 import fs from 'node:fs/promises';
 import path from 'node:path';
 

package/web/util.test.ts

@@ -0,0 +1,240 @@
+/*
+  util.test.ts
+  Tests of the utility functions used only by the web pages.
+*/
+
+// IMPORTS
+
+import {test, before, after} from 'node:test';
+import assert from 'node:assert/strict';
+import path from 'node:path';
+import fs from 'node:fs/promises';
+import {reportsPath} from '../util.ts';
+import {
+  checkCommentLength,
+  getAgoString,
+  getCountString,
+  getEngineNamesString,
+  getMultiReportWhats,
+  getPageData,
+  getPageDataStrings,
+  getTextFragmentHref,
+  getWCAGLink,
+  getWeightName,
+  makeBreakable
+} from './util.ts';
+
+// SETUP AND TEARDOWN
+
+// These tests use the fixture database directory.
+import {fixtureDBDir as fixtureDbDir} from '../test/dbFixture.ts';
+const savedDbDir = process.env.DB_DIR;
+
+before(() => {
+  process.env.DB_DIR = fixtureDbDir;
+});
+
+after(() => {
+  if (savedDbDir !== undefined) {
+    process.env.DB_DIR = savedDbDir;
+  }
+  else {
+    delete process.env.DB_DIR;
+  }
+});
+
+// TESTS
+
+test('checkCommentLength accepts a comment of exactly 20 characters', () => {
+  const result = checkCommentLength('a'.repeat(20));
+  assert.deepEqual(result, {status: 'ok'});
+});
+
+test('checkCommentLength accepts a comment of exactly 1000 characters', () => {
+  const result = checkCommentLength('a'.repeat(1000));
+  assert.deepEqual(result, {status: 'ok'});
+});
+
+test('checkCommentLength rejects a comment shorter than 20 characters', () => {
+  const result = checkCommentLength('a'.repeat(19));
+  assert.deepEqual(result, {
+    status: 'error',
+    message: 'Your comment was shorter than 20 characters'
+  });
+});
+
+test('checkCommentLength rejects a comment longer than 1000 characters', () => {
+  const result = checkCommentLength('a'.repeat(1001));
+  assert.deepEqual(result, {
+    status: 'error',
+    message: 'Your comment was longer than 1000 characters'
+  });
+});
+
+test('getAgoString returns "1 day" for a 1-day-old timestamp', () => {
+  const date = new Date(Date.now() - 1 * 24 * 60 * 60 * 1000);
+  assert.equal(
+    getAgoString(date.toISOString().slice(2).replace(/[-:]/g, '').slice(0, 11)), '1 day'
+  );
+});
+
+test('getAgoString returns "3 days" for a 3-day-old timestamp', () => {
+  const date = new Date(Date.now() - 3 * 24 * 60 * 60 * 1000);
+  assert.equal(
+    getAgoString(date.toISOString().slice(2).replace(/[-:]/g, '').slice(0, 11)), '3 days'
+  );
+});
+
+test('getCountString returns singular for count 1', () => {
+  assert.equal(getCountString(1, 'report', 'reports'), '1 report');
+});
+
+test('getCountString returns plural for count 0', () => {
+  assert.equal(getCountString(0, 'report', 'reports'), '0 reports');
+});
+
+test('getTextFragmentHref returns a text-fragment URL', () => {
+  const href = getTextFragmentHref('Hello', 'https://example.com/page');
+  assert.ok(href.startsWith('https://example.com/page#:~:text='));
+});
+
+test('getWCAGLink returns a URL for a numeric WCAG ID', () => {
+  const link = getWCAGLink('4.1.2');
+  assert.ok(link.startsWith('https://www.w3.org/WAI/WCAG22/Understanding/'));
+});
+
+test('getWeightName returns the correct name for each weight', () => {
+  assert.equal(getWeightName(1), 'lowest');
+  assert.equal(getWeightName(2), 'low');
+  assert.equal(getWeightName(3), 'high');
+  assert.equal(getWeightName(4), 'highest');
+  assert.equal(getWeightName(5), 'unknown');
+});
+
+test('makeBreakable inserts wbr before non-initial slashes', () => {
+  assert.equal(makeBreakable('/api/listReports'), '/api<wbr>/listReports');
+});
+
+test('getPageData returns page data for a valid report', async () => {
+  const data = await getPageData('260101T0000', 'mix') as any;
+  assert.equal(data.description, 'Mixed Outcomes Page');
+  assert.equal(data.url, 'https://example.com/mixed');
+  assert.equal(typeof data.daysAgo, 'number');
+});
+
+test('getPageData returns an error for a nonexistent report', async () => {
+  const data = await getPageData('999999T9999', 'xxx') as any;
+  assert.ok(data.error);
+});
+
+test('getPageDataStrings returns HTML strings for a valid report', async () => {
+  const strings = await getPageDataStrings('260101T0000', 'mix') as any;
+  assert.equal(strings.description, 'Mixed Outcomes Page');
+  assert.equal(strings.url, 'https://example.com/mixed');
+  assert.equal(strings.urlLink, '<a href="https://example.com/mixed">https://example.com/mixed</a>');
+  assert.ok(strings.testInfo.includes('by job <code>mix</code>'));
+  assert.ok(strings.testInfo.includes('2026-01-01 at 00:00'));
+});
+
+test('getPageDataStrings returns different testInfo for a different timeStamp', async () => {
+  const strings = await getPageDataStrings('260101T0001', 'ct') as any;
+  assert.equal(strings.description, 'All CantTell Page');
+  assert.ok(strings.testInfo.includes('by job <code>ct</code>'));
+  assert.ok(strings.testInfo.includes('2026-01-01 at 00:01'));
+});
+
+test('getPageDataStrings returns an error for a nonexistent report', async () => {
+  const strings = await getPageDataStrings('999999T9999', 'xxx');
+  assert.ok(strings.error);
+});
+
+test('getPageDataStrings uses provided pageData instead of reading the report', async () => {
+  const strings = await getPageDataStrings('260101T0000', 'mix', {
+    description: 'Custom Page',
+    url: 'https://custom.com',
+    daysAgo: 1
+  }) as any;
+  assert.equal(strings.description, 'Custom Page');
+  assert.equal(strings.url, 'https://custom.com');
+  assert.ok(strings.testInfo.includes('1 day ago'));
+});
+
+test('getReportData returns an error for a nonexistent report', async () => {
+  const {getReportData} = await import('./util.ts');
+  const result: any = await getReportData('990101T0000', 'xxx');
+  assert.ok(result.error);
+});
+
+test('getReportData falls back to the engine ID for an unknown prevented engine', async () => {
+  const {getReportData} = await import('./util.ts');
+  const prvJSON = await fs.readFile(path.join(reportsPath(), '260101T0006-prv.json'), 'utf8');
+  const report = JSON.parse(prvJSON);
+  report.jobData.preventions.unknownEngine = 'mystery failure';
+  const reportPath = path.join(reportsPath(), '260103T0000-unk.json');
+  await fs.writeFile(reportPath, JSON.stringify(report));
+  try {
+    const result: any = await getReportData('260103T0000', 'unk');
+    assert.ok(result.preventedEngineNames.includes('unknownEngine'));
+  }
+  finally {
+    await fs.unlink(reportPath);
+  }
+});
+
+test('getAgoString returns "1 day" for exactly 1 day ago', () => {
+  // Construct a time stamp 1 day and 1 hour ago, so Math.round gives exactly 1.
+  const date = new Date(Date.now() - (86400000 + 3600000));
+  const stamp = date.toISOString().slice(2).replace(/[-:]/g, '').slice(0, 11);
+  assert.equal(getAgoString(stamp), '1 day');
+});
+
+test('getDateTimeString describes an invalid time stamp as unknown', async () => {
+  const {getDateTimeString} = await import('./util.ts');
+  const result = getDateTimeString('999999T9999');
+  assert.equal(result, 'an unknown date at an unknown time');
+});
+
+test('getEngineNamesString falls back to the engine ID for an unknown engine', async () => {
+  const {getEngineNamesString} = await import('./util.ts');
+  const result = getEngineNamesString(new Set(['unknownEngine']));
+  assert.equal(result, 'unknownEngine');
+});
+
+test('getPathID returns the catalog pathID when catalogIndex is truthy', async () => {
+  const {getPathID} = await import('./util.ts');
+  const catalog = {'0': {tagName: 'div', pathID: '/html/body/div'}};
+  assert.equal(getPathID(catalog, '0', '/fallback'), '/html/body/div');
+});
+
+test('getPathID returns the fallback pathID when catalogIndex is truthy but catalogItem has no pathID', async () => {
+  const {getPathID} = await import('./util.ts');
+  const catalog = {'0': {tagName: 'div', pathID: ''}};
+  assert.equal(getPathID(catalog, '0', '/fallback'), '/fallback');
+});
+
+test('getPathID returns /html when catalogIndex is truthy but catalogItem and pathID are both missing', async () => {
+  const {getPathID} = await import('./util.ts');
+  const catalog = {};
+  assert.equal(getPathID(catalog, '0', null as any), '/html');
+});
+
+test('getPathID returns /html when catalogIndex is falsy and pathID is null', async () => {
+  const {getPathID} = await import('./util.ts');
+  assert.equal(getPathID({}, null as any, null as any), '/html');
+});
+
+test('getEngineNamesString returns a sorted +-delimited list of engine names', () => {
+  const result = getEngineNamesString(new Set(['axe', 'wave', 'nuVal']));
+  const names = result.split(' + ');
+  assert.ok(names.length === 3);
+  assert.ok(names.includes('WAVE'));
+});
+
+test('getEngineNamesString falls back to the ID for an unknown engine', () => {
+  assert.equal(getEngineNamesString(new Set(['unknownEngine'])), 'unknownEngine');
+});
+
+test('getMultiReportWhats returns descriptions that have multiple reports', async () => {
+  const whats = await getMultiReportWhats();
+  assert.ok(whats.includes('Mixed Outcomes Page'));
+});