@ai-ecoverse/slicc-shared-web 1.6.0 → 1.7.1

package/README.md

@@ -67,8 +67,9 @@ test('boots', async (t) => {
 
 `chrome.page(t)` opens a tab and returns the page API: `goto(path)`, `reload()`, `evaluate(fn, ...args)`, `until(fn, ...args)`, `press(key, ...modifiers)` (a key name like `Enter` or `F5`, or a single character typed with its US-layout key code), `type(text)`, `insert(text)`, `enter()`, `init(fn)` (runs in every new document), `expose(name, handler)` (a binding the page calls with a JSON string), `screenshot(file)`, `tab()` (another tab in the same context), `close()`, `errors`, `responses` and `dir` (the test's artifact directory). `goto` and `reload` wait until the new document is complete and no longer `about:blank`, and retry the navigation once if that never happens. `until` polls until the function returns a truthy value (not only strict `true`) and resolves with that value; the timeout error names the last result. `within(ms, fn, ...args)` is `until` with its own deadline, for a stage that should take a known time. Both stop at the deadline even if one evaluation never answers. Every CDP call has a deadline: `connect(url).send(method, params, sessionId, ms)` rejects with `<method>: no answer in <ms> ms` (60 s unless given), so a stuck page fails its test instead of hanging the run. When one evaluation runs past `stallAfter`, the harness probes once, without interrupting the wait:
 - it saves `stall-<n>.png` (a screenshot of the page, if the renderer still answers) in the test's artifact directory;
-- it saves a CPU profile sample of every page and worker in the test as `stall-<n>-<target>-<session>.cpuprofile`;
+- it saves a CPU profile sample of every page and worker in the test, including workers started by other workers, as `stall-<n>-<target>-<session>.cpuprofile`;
 - it adds a line per target to `console.log`, saying whether that target is running or paused;
+- on the first stall, it records a 10 s browser trace as `stall-1-trace.json` (open it in DevTools' Performance panel or Perfetto). The browser collects it with V8 CPU samples per thread, so it shows what a page's main thread is doing even when that thread no longer answers CDP. It also logs each process's CPU time over the window;
 - it adds a line for every debugger pause still held in the browser, in any target, with its id, URL, start time, reason and top call frames.
 
 A pause in one target blocks every page in the same renderer process, so those pause lines name the culprit even when it isn't the page under test. The harness resumes its own coverage pauses (`beforeunload`, worker exits) within 10 s, even when collecting coverage fails. Downloads of intercepted URLs time out after 60 s and are tried up to three times; a `4xx` is not retried. `chrome.requests` lists the paths the server saw in this test, `chrome.overrides` is a `Map` of paths to bodies served instead of files, and `chrome.cdn` takes `status` (answer every intercepted request with it), `corrupt` (flip the last byte) and lists the intercepted `requests`. All of them reset with each test. `serve(options)` starts the same server on its own, for an `npm start`.

package/harness/cdn.mjs

@@ -24,7 +24,7 @@ export async function fetched(url, { attempts = 3, ms = DOWNLOAD_TIMEOUT, fetche
 export async function download(url) {
   const { host, pathname, search } = new URL(url);
   const query = search ? encodeURIComponent(search) : '';
-  const file = new URL(`./${host}${pathname}${query}`, store);
+  const file = new URL(`./${host}${pathname}${query}`.replaceAll('%', '%25'), store);
   const hit = await readFile(file).catch(ignore);
   if (hit) return hit;
   const body = await fetched(url);
@@ -49,7 +49,9 @@ export async function cdn(cdp, remotes) {
       const failure = { requestId, responseCode: state.status, responseHeaders: cors };
       return cdp.send('Fetch.fulfillRequest', failure);
     }
-    const body = await download(request.url).catch(ignore);
+    const body = await download(request.url).catch((error) => {
+      console.warn(`harness: could not serve ${request.url}: ${error.message}`);
+    });
     if (!body) return cdp.send('Fetch.failRequest', { requestId, errorReason: 'Failed' });
     const sent = Buffer.from(body);
     if (state.corrupt) sent[sent.length - 1] ^= 0xff;

package/harness/chrome.mjs

@@ -142,7 +142,7 @@ export async function launch({
           stallAfter,
           stall: async (tag, png) => {
             if (png) await writeFile(new URL(`${tag}.png`, dir), png).catch(ignore);
-            await bounded(record.stall(dir, tag), tag, 20000);
+            await bounded(record.stall(dir, tag), tag, 45000);
           },
         });
         pages.push(opened);

package/harness/recorder.mjs

@@ -104,9 +104,74 @@ function pauseOf(target = {}, params) {
   };
 }
 
+const TRACED = [
+  'toplevel',
+  'devtools.timeline',
+  'disabled-by-default-devtools.timeline',
+  'v8.execute',
+  'disabled-by-default-v8.cpu_profiler',
+];
+
+async function cpuTimes(cdp) {
+  const { processInfo = [] } = await cdp
+    .send('SystemInfo.getProcessInfo', {}, undefined, 5000)
+    .catch(() => ({}));
+  return new Map(processInfo.map((info) => [`${info.type} ${info.id}`, info.cpuTime]));
+}
+
+export async function trace(cdp, dir, tag, ms = 10000) {
+  const events = [];
+  const done = Promise.withResolvers();
+  const off = cdp.on(({ method, params }) => {
+    if (method === 'Tracing.dataCollected') events.push(...params.value);
+    if (method === 'Tracing.tracingComplete') done.resolve();
+  });
+  try {
+    const before = await cpuTimes(cdp);
+    await cdp.send(
+      'Tracing.start',
+      { traceConfig: { includedCategories: TRACED }, transferMode: 'ReportEvents' },
+      undefined,
+      10000
+    );
+    await sleep(ms);
+    await cdp.send('Tracing.end', {}, undefined, 10000);
+    await bounded(done.promise, 'trace', 20000);
+    const after = await cpuTimes(cdp);
+    await writeFile(new URL(`${tag}-trace.json`, dir), JSON.stringify({ traceEvents: events }));
+    const busy = [...after]
+      .map(([name, time]) => [name, time - (before.get(name) ?? 0)])
+      .filter(([, used]) => used > 0.05)
+      .map(([name, used]) => `${name} ${used.toFixed(1)}s`);
+    return [
+      `trace ${tag}-trace.json (${events.length} events)`,
+      `cpu over ${ms / 1000}s: ${busy.join(', ') || 'none'}`,
+    ];
+  } catch (error) {
+    return [`trace failed: ${error.message}`];
+  } finally {
+    off();
+  }
+}
+
+function claim(run) {
+  if (!run || run.traced) return false;
+  run.traced = true;
+  return true;
+}
+
+function contextOf(sessions, parents, sessionId) {
+  for (let id = sessionId; id; id = parents.get(id)) {
+    const context = sessions.get(id)?.browserContextId;
+    if (context) return context;
+  }
+  return undefined;
+}
+
 export function recorder(cdp, server, { coverage = ['/'], exits = new Map() } = {}) {
   const sessions = new Map();
   const pauses = new Map();
+  const parents = new Map();
   const tabs = new Map();
   const ours = covered(server, coverage);
   let run = null;
@@ -208,15 +273,24 @@ export function recorder(cdp, server, { coverage = ['/'], exits = new Map() } =
   }
 
   async function stall(dir, tag) {
-    const current = [...sessions].filter(([, target]) => target.browserContextId === run?.context);
-    const lines = await Promise.all(current.map((entry) => sample(cdp, pauses, entry, dir, tag)));
-    const all = [...lines, ...held(pauses)];
+    const first = claim(run);
+    const current = [...sessions].filter(
+      ([sessionId]) => contextOf(sessions, parents, sessionId) === run?.context
+    );
+    const [lines, traced] = await Promise.all([
+      Promise.all(current.map((entry) => sample(cdp, pauses, entry, dir, tag))),
+      first ? trace(cdp, dir, tag) : [],
+    ]);
+    const all = [...lines, ...traced, ...held(pauses)];
     for (const line of all) run?.console.push(`${tag}: ${line}`);
     return all;
   }
 
   cdp.on(async ({ method, params, sessionId }) => {
-    if (method === 'Target.attachedToTarget') await attach(params);
+    if (method === 'Target.attachedToTarget') {
+      if (sessionId) parents.set(params.sessionId, sessionId);
+      await attach(params);
+    }
     if (method === 'Target.detachedFromTarget') {
       sessions.delete(params.sessionId);
       pauses.delete(params.sessionId);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ai-ecoverse/slicc-shared-web",
-  "version": "1.6.0",
+  "version": "1.7.1",
   "description": "Shared Renovate, Biome, TypeScript, lefthook and GitHub Actions configuration, plus the CDP integration-test harness, for SLICC's web repos",
   "type": "module",
   "license": "Apache-2.0",