@datalayer/agent-runtimes 1.5.71 → 1.5.73

package/lib/voice/serverHearing.js

@@ -27,8 +27,8 @@
  *
  * @module voice/serverHearing
  */
-import { transcriberFor } from '../specs/voices';
-import { HEARING_RATE } from './capture';
+import { transcriberFor } from '../specs/voices.js';
+import { HEARING_RATE } from './capture.js';
 /** Where the speech service hears a recording, on ai-agents' host. */
 export const TRANSCRIPTIONS_PATH = '/api/ai-agents/v1/speech/transcriptions';
 /** Where the speech service hears speech as it is said. */

package/lib/voice/useSpokenAnswers.js

@@ -11,7 +11,7 @@
  * @module voice/useSpokenAnswers
  */
 import { useEffect, useRef } from 'react';
-import { SentenceCutter } from './sentences';
+import { SentenceCutter } from './sentences.js';
 /** An item's text, when it is the agent's own answer and not a tool call. */
 export function answerText(item) {
     const message = item;

package/lib/voice/useVoiceSettings.js

@@ -9,7 +9,7 @@
  * @module voice/useVoiceSettings
  */
 import { useCallback, useEffect, useState } from 'react';
-import { heldVoiceSettings, keepVoiceSettings, onVoiceSettingsChange, } from './settings';
+import { heldVoiceSettings, keepVoiceSettings, onVoiceSettingsChange, } from './settings.js';
 /** The person's settings, and a setter that keeps them and tells every chat. */
 export function useVoiceSettings() {
     const [settings, setSettings] = useState(heldVoiceSettings);

package/lib/voice/useVoiceSpeaker.js

@@ -12,8 +12,8 @@
  * @module voice/useVoiceSpeaker
  */
 import { useCallback, useEffect, useMemo, useState } from 'react';
-import { readAloud } from './ReadAloud';
-import { ServerSpeaker } from './speaker';
+import { readAloud } from './ReadAloud.js';
+import { ServerSpeaker } from './speaker.js';
 /**
  * The speaker of a chat's voice, or none: `voice` undefined, its `output`
  * `off`, or `on` false (a host that speaks for the chat).

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@datalayer/agent-runtimes",
-    "version": "1.5.71",
+    "version": "1.5.73",
     "type": "module",
     "workspaces": [
         ".",
@@ -54,16 +54,17 @@
     },
     "scripts": {
         "apply:patches": "bash scripts/apply-patches.sh",
-        "build": "npm run clean && gulp resources-to-lib && tsc -b && cross-env NODE_OPTIONS=--max-old-space-size=8192 vite build && npm run build:embed",
+        "build": "npm run clean && gulp resources-to-lib && tsc -b && node scripts/fully-specify-lib.mjs && cross-env NODE_OPTIONS=--max-old-space-size=8192 vite build && npm run build:embed",
         "build-storybook": "storybook build",
         "build:all": "npm run build && npm run build:examples",
         "build:embed": "cross-env NODE_OPTIONS=--max-old-space-size=8192 vite build --config vite.embed.config.ts",
         "dist-embed": "node scripts/dist-embed.mjs",
         "dist-embed:release": "node scripts/dist-embed.mjs --release",
         "build:examples": "npm run build:nextjs",
-        "build:lib": "npm run clean:lib && gulp resources-to-lib && tsc -b",
+        "build:lib": "npm run clean:lib && gulp resources-to-lib && tsc -b && node scripts/fully-specify-lib.mjs",
         "build:nextjs": "npm run build --workspace=nextjs-notebook-example",
         "build:types": "npm run clean:lib && tsc -b",
+        "check:pack": "node scripts/check-npm-pack.mjs",
         "check": "npm run format:check && npm run lint:all && npm run type-check:all && npm run typedoc",
         "check:environments-generated": "python scripts/generate-environments-types.py --check",
         "check:fix": "npm run format:all && npm run lint:fix && npm run type-check:all",

package/scripts/check-npm-pack.mjs

@@ -0,0 +1,77 @@
+#!/usr/bin/env node
+/*
+ * Copyright (c) 2025-2026 Datalayer, Inc.
+ * Distributed under the terms of the Modified BSD License.
+ */
+
+/**
+ * Refuses a package whose tarball does not carry what its `lib/` imports.
+ *
+ * Lists the tarball with `npm pack --dry-run --json` and reads every
+ * `lib/**\/*.js` in it: each relative specifier must name a file the
+ * tarball carries, written whole (`'./approvals.js'`, not `'./approvals'`:
+ * Node's ESM loader adds no extension and looks up no `index.js`). And no
+ * two files of the tarball may have paths that differ only in case: on a
+ * case-insensitive file system (macOS, Windows) one overwrites the other
+ * when the package is installed (`voice/HandsFree.js` and
+ * `voice/handsFree.js` did). Prints each refusal and exits 1. Run it after
+ * the build and before `npm pack`.
+ *
+ * Usage: node scripts/check-npm-pack.mjs
+ */
+
+import { execFileSync } from 'node:child_process';
+import { readFileSync } from 'node:fs';
+import { relativeSpecifiers, resolveSpecifier } from './esm-specifiers.mjs';
+
+const listing = JSON.parse(
+  execFileSync('npm', ['pack', '--dry-run', '--json', '--ignore-scripts'], {
+    encoding: 'utf8',
+    maxBuffer: 256 * 1024 * 1024,
+    stdio: ['ignore', 'pipe', 'inherit'],
+  }),
+);
+const packed = new Set(listing[0].files.map(({ path }) => path));
+const isFile = path => packed.has(path);
+
+const refused = [];
+
+const byFolded = new Map();
+for (const path of packed) {
+  const folded = path.toLowerCase();
+  const other = byFolded.get(folded);
+  if (other) {
+    refused.push(`${other} and ${path} differ only in case`);
+  } else {
+    byFolded.set(folded, path);
+  }
+}
+
+let read = 0;
+let specifiers = 0;
+for (const file of [...packed].filter(
+  path => path.startsWith('lib/') && path.endsWith('.js'),
+)) {
+  read += 1;
+  for (const { specifier } of relativeSpecifiers(readFileSync(file, 'utf8'))) {
+    specifiers += 1;
+    const { status, whole } = resolveSpecifier(file, specifier, isFile);
+    if (status === 'missing') {
+      refused.push(`${file}: '${specifier}' names no file in the tarball`);
+    } else if (status === 'partial') {
+      refused.push(
+        `${file}: '${specifier}' is not written whole (Node needs '${whole}')`,
+      );
+    }
+  }
+}
+console.log(
+  `check-npm-pack: ${listing[0].name}@${listing[0].version}, ${packed.size} files, ${specifiers} relative specifiers in ${read} lib/ modules`,
+);
+if (refused.length > 0) {
+  console.error(refused.slice(0, 200).join('\n'));
+  if (refused.length > 200) console.error(`… and ${refused.length - 200} more`);
+  console.error(`check-npm-pack: ${refused.length} refused`);
+  process.exit(1);
+}
+console.log('check-npm-pack: every relative import of lib/ is in the tarball');

package/scripts/esm-specifiers.mjs

@@ -0,0 +1,76 @@
+/*
+ * Copyright (c) 2025-2026 Datalayer, Inc.
+ * Distributed under the terms of the Modified BSD License.
+ */
+
+/**
+ * The relative module specifiers of a compiled ES module, as Node reads them.
+ *
+ * `tsc` with `moduleResolution: bundler` keeps a source's specifiers as they
+ * are written (`'./approvals'`, `'./chat'`), which a bundler resolves and
+ * Node's ESM loader refuses: it takes a relative specifier as a file path,
+ * with no extension added and no directory's `index.js` looked up. So
+ * `lib/` is made loadable by Node (vitest's externalized dependencies, a
+ * Node service importing `lib/portable`) by writing each specifier whole
+ * (`fully-specify-lib.mjs`), and the packed tarball is checked for a
+ * specifier that names a file it does not carry (`check-npm-pack.mjs`).
+ *
+ * Comments are not read: TypeScript's own pre-processor lists the imports,
+ * the exports and the dynamic imports of a file.
+ */
+
+import ts from 'typescript';
+import { posix } from 'node:path';
+
+/**
+ * Every relative specifier of `text`, with the offset of its first
+ * character in `text`.
+ *
+ * @param {string} text A compiled JavaScript module.
+ * @returns {{ specifier: string, start: number }[]}
+ */
+export function relativeSpecifiers(text) {
+  const { importedFiles } = ts.preProcessFile(text, true, true);
+  const found = [];
+  for (const { fileName, pos } of importedFiles) {
+    if (!fileName.startsWith('./') && !fileName.startsWith('../')) continue;
+    const start = text.indexOf(fileName, pos);
+    if (start < 0) {
+      throw new Error(`the specifier ${fileName} is not at ${pos}`);
+    }
+    found.push({ specifier: fileName, start });
+  }
+  return found;
+}
+
+/**
+ * What a specifier of `from` names, given the files there are.
+ *
+ * @param {string} from The importing file, a posix path.
+ * @param {string} specifier A relative specifier.
+ * @param {(path: string) => boolean} isFile Whether a file is there.
+ * @returns {{ status: 'whole' | 'partial' | 'missing', path?: string, whole?: string }}
+ *   `whole`: the specifier names a file; `partial`: it names a file only
+ *   once `.js` or `/index.js` is added (`whole` is the specifier written
+ *   so); `missing`: no file.
+ */
+export function resolveSpecifier(from, specifier, isFile) {
+  const query = specifier.search(/[?#]/);
+  const bare = query < 0 ? specifier : specifier.slice(0, query);
+  const target = posix.normalize(posix.join(posix.dirname(from), bare));
+  if (isFile(target)) return { status: 'whole', path: target };
+  if (query < 0) {
+    if (isFile(`${target}.js`)) {
+      return { status: 'partial', path: `${target}.js`, whole: `${bare}.js` };
+    }
+    const index = posix.join(target, 'index.js');
+    if (isFile(index)) {
+      return {
+        status: 'partial',
+        path: index,
+        whole: `${bare.replace(/\/$/, '')}/index.js`,
+      };
+    }
+  }
+  return { status: 'missing' };
+}

package/scripts/fully-specify-lib.mjs

@@ -0,0 +1,61 @@
+#!/usr/bin/env node
+/*
+ * Copyright (c) 2025-2026 Datalayer, Inc.
+ * Distributed under the terms of the Modified BSD License.
+ */
+
+/**
+ * Writes each relative specifier of `lib/**\/*.js` whole, after `tsc`:
+ * `'./approvals'` becomes `'./approvals.js'`, `'./chat'` (a folder)
+ * `'./chat/index.js'`, so Node's ESM loader reads `lib/` as a bundler does.
+ * A specifier that already names a file, or names none, is left as it is
+ * (`check-npm-pack.mjs` refuses the second). Run again, it changes nothing.
+ *
+ * Usage: node scripts/fully-specify-lib.mjs [lib]
+ */
+
+import { readdirSync, readFileSync, statSync, writeFileSync } from 'node:fs';
+import { join, relative, resolve, sep } from 'node:path';
+import { relativeSpecifiers, resolveSpecifier } from './esm-specifiers.mjs';
+
+const root = resolve(process.argv[2] ?? 'lib');
+const posixOf = path => relative(root, path).split(sep).join('/');
+
+const files = [];
+const walk = dir => {
+  for (const name of readdirSync(dir)) {
+    const path = join(dir, name);
+    if (statSync(path).isDirectory()) walk(path);
+    else files.push(posixOf(path));
+  }
+};
+walk(root);
+const present = new Set(files);
+const isFile = path => present.has(path);
+
+let rewritten = 0;
+let changedFiles = 0;
+for (const file of files.filter(name => name.endsWith('.js'))) {
+  const path = join(root, file);
+  const text = readFileSync(path, 'utf8');
+  const edits = relativeSpecifiers(text)
+    .map(({ specifier, start }) => ({
+      specifier,
+      start,
+      ...resolveSpecifier(file, specifier, isFile),
+    }))
+    .filter(({ status }) => status === 'partial');
+  if (edits.length === 0) continue;
+  let out = text;
+  for (const { specifier, start, whole } of edits.sort(
+    (a, b) => b.start - a.start,
+  )) {
+    out = out.slice(0, start) + whole + out.slice(start + specifier.length);
+  }
+  writeFileSync(path, out);
+  rewritten += edits.length;
+  changedFiles += 1;
+}
+console.log(
+  `fully-specify-lib: ${rewritten} specifiers written whole in ${changedFiles} of ${files.length} files under ${posixOf(root) || root}`,
+);