test(ui): make workflow paths portable

onutc · onutc · commit bc072eb7cf03 · 2026-03-13T07:26:45.000+01:00
diff --git a/e2e/acp-smoke-lib.mjs b/e2e/acp-smoke-lib.mjs
@@ -230,6 +230,9 @@ export function resolveACPEndpoint(spritz) {
  * Load a Node-compatible WebSocket client constructor from the CLI dependency set.
  */
 export function resolveWebSocketConstructor() {
+  if (typeof globalThis.WebSocket === 'function') {
+    return globalThis.WebSocket;
+  }
   const wsModule = cliRequire('ws');
   return wsModule.WebSocket || wsModule.default || wsModule;
 }
diff --git a/ui/entrypoint.test.mjs b/ui/entrypoint.test.mjs
@@ -4,14 +4,15 @@ import fs from 'node:fs';
 import os from 'node:os';
 import path from 'node:path';
 import vm from 'node:vm';
+import { uiPath, uiPublicPath } from './test-paths.mjs';
 import { execFileSync } from 'node:child_process';
 
 function renderConfig(env = {}) {
   const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'spritz-ui-'));
-  fs.copyFileSync('/Users/onur/repos/spritz/ui/public/config.js', path.join(tmpDir, 'config.js'));
-  fs.copyFileSync('/Users/onur/repos/spritz/ui/public/index.html', path.join(tmpDir, 'index.html'));
+  fs.copyFileSync(uiPublicPath('config.js'), path.join(tmpDir, 'config.js'));
+  fs.copyFileSync(uiPublicPath('index.html'), path.join(tmpDir, 'index.html'));
 
-  execFileSync('/bin/sh', ['/Users/onur/repos/spritz/ui/entrypoint.sh'], {
+  execFileSync('/bin/sh', [uiPath('entrypoint.sh')], {
     env: {
       ...process.env,
       SPRITZ_UI_HTML_DIR: tmpDir,
diff --git a/ui/nginx-config.test.mjs b/ui/nginx-config.test.mjs
@@ -1,8 +1,9 @@
 import test from 'node:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs';
+import { uiPath } from './test-paths.mjs';
 
-const nginxConfig = fs.readFileSync('/Users/onur/repos/spritz/ui/nginx.conf', 'utf8');
+const nginxConfig = fs.readFileSync(uiPath('nginx.conf'), 'utf8');
 
 test('nginx config disables caching for index and runtime config', () => {
   assert.match(nginxConfig, /location = \/index\.html \{[\s\S]*Cache-Control "no-store, max-age=0"/);
diff --git a/ui/public/acp-client.test.mjs b/ui/public/acp-client.test.mjs
@@ -2,6 +2,7 @@ import test from 'node:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs';
 import vm from 'node:vm';
+import { uiPublicPath } from '../test-paths.mjs';
 
 function loadACPClientModule() {
   const sockets = [];
@@ -57,7 +58,7 @@ function loadACPClientModule() {
   });
   context.globalThis = context.window;
 
-  vm.runInContext(fs.readFileSync('/Users/onur/repos/spritz/ui/public/acp-client.js', 'utf8'), context, {
+  vm.runInContext(fs.readFileSync(uiPublicPath('acp-client.js'), 'utf8'), context, {
     filename: 'acp-client.js',
   });
 
diff --git a/ui/public/acp-layout-css.test.mjs b/ui/public/acp-layout-css.test.mjs
@@ -1,8 +1,9 @@
 import test from 'node:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs';
+import { uiPublicPath } from '../test-paths.mjs';
 
-const styles = fs.readFileSync('/Users/onur/repos/spritz/ui/public/styles.css', 'utf8');
+const styles = fs.readFileSync(uiPublicPath('styles.css'), 'utf8');
 
 function getRule(selector) {
   const escapedSelector = selector.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
diff --git a/ui/public/acp-page-cache.test.mjs b/ui/public/acp-page-cache.test.mjs
@@ -2,6 +2,7 @@ import test from 'node:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs';
 import vm from 'node:vm';
+import { uiPublicPath } from '../test-paths.mjs';
 
 function createStorage(seed = {}) {
   const values = new Map(Object.entries(seed));
@@ -110,10 +111,10 @@ function loadModules(storageSeed = {}, createACPClient = null) {
   window.window = window;
   const context = vm.createContext({ window, document, console, setTimeout, clearTimeout, URL, URLSearchParams });
   context.globalThis = context.window;
-  vm.runInContext(fs.readFileSync('/Users/onur/repos/spritz/ui/public/acp-render.js', 'utf8'), context, {
+  vm.runInContext(fs.readFileSync(uiPublicPath('acp-render.js'), 'utf8'), context, {
     filename: 'acp-render.js',
   });
-  vm.runInContext(fs.readFileSync('/Users/onur/repos/spritz/ui/public/acp-page.js', 'utf8'), context, {
+  vm.runInContext(fs.readFileSync(uiPublicPath('acp-page.js'), 'utf8'), context, {
     filename: 'acp-page.js',
   });
   return window;
diff --git a/ui/public/acp-page-layout.test.mjs b/ui/public/acp-page-layout.test.mjs
@@ -2,6 +2,7 @@ import test from 'node:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs';
 import vm from 'node:vm';
+import { uiPublicPath } from '../test-paths.mjs';
 
 function createElement(tagName) {
   return {
@@ -75,9 +76,9 @@ function loadModules() {
   window.window = window;
   const context = vm.createContext({ window, document, console, setTimeout, clearTimeout });
   context.globalThis = context.window;
-  const renderScript = fs.readFileSync('/Users/onur/repos/spritz/ui/public/acp-render.js', 'utf8');
+  const renderScript = fs.readFileSync(uiPublicPath('acp-render.js'), 'utf8');
   vm.runInContext(renderScript, context, { filename: 'acp-render.js' });
-  const pageScript = fs.readFileSync('/Users/onur/repos/spritz/ui/public/acp-page.js', 'utf8');
+  const pageScript = fs.readFileSync(uiPublicPath('acp-page.js'), 'utf8');
   vm.runInContext(pageScript, context, { filename: 'acp-page.js' });
   return { window, document };
 }
diff --git a/ui/public/acp-page-notice.test.mjs b/ui/public/acp-page-notice.test.mjs
@@ -2,6 +2,7 @@ import test from 'node:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs';
 import vm from 'node:vm';
+import { uiPublicPath } from '../test-paths.mjs';
 
 function createElement(tagName) {
   return {
@@ -64,10 +65,10 @@ function loadModules(createACPClient) {
   window.window = window;
   const context = vm.createContext({ window, document, console, setTimeout, clearTimeout, URL, URLSearchParams });
   context.globalThis = context.window;
-  vm.runInContext(fs.readFileSync('/Users/onur/repos/spritz/ui/public/acp-render.js', 'utf8'), context, {
+  vm.runInContext(fs.readFileSync(uiPublicPath('acp-render.js'), 'utf8'), context, {
     filename: 'acp-render.js',
   });
-  vm.runInContext(fs.readFileSync('/Users/onur/repos/spritz/ui/public/acp-page.js', 'utf8'), context, {
+  vm.runInContext(fs.readFileSync(uiPublicPath('acp-page.js'), 'utf8'), context, {
     filename: 'acp-page.js',
   });
   return window;
diff --git a/ui/public/acp-page-session-binding.test.mjs b/ui/public/acp-page-session-binding.test.mjs
@@ -2,6 +2,7 @@ import test from 'node:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs';
 import vm from 'node:vm';
+import { uiPublicPath } from '../test-paths.mjs';
 
 function createElement(tagName) {
   const listeners = new Map();
@@ -81,10 +82,10 @@ function loadModules(createACPClient) {
   window.window = window;
   const context = vm.createContext({ window, document, console, setTimeout, clearTimeout, URL, URLSearchParams });
   context.globalThis = context.window;
-  vm.runInContext(fs.readFileSync('/Users/onur/repos/spritz/ui/public/acp-render.js', 'utf8'), context, {
+  vm.runInContext(fs.readFileSync(uiPublicPath('acp-render.js'), 'utf8'), context, {
     filename: 'acp-render.js',
   });
-  vm.runInContext(fs.readFileSync('/Users/onur/repos/spritz/ui/public/acp-page.js', 'utf8'), context, {
+  vm.runInContext(fs.readFileSync(uiPublicPath('acp-page.js'), 'utf8'), context, {
     filename: 'acp-page.js',
   });
   return window;
diff --git a/ui/public/acp-render.test.mjs b/ui/public/acp-render.test.mjs
@@ -2,6 +2,7 @@ import test from 'node:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs';
 import vm from 'node:vm';
+import { uiPublicPath } from '../test-paths.mjs';
 
 function createElement(tagName) {
   return {
@@ -50,7 +51,7 @@ function loadRenderModule() {
   window.window = window;
   const context = vm.createContext({ window, document, console });
   context.globalThis = context.window;
-  const script = fs.readFileSync('/Users/onur/repos/spritz/ui/public/acp-render.js', 'utf8');
+  const script = fs.readFileSync(uiPublicPath('acp-render.js'), 'utf8');
   vm.runInContext(script, context, { filename: 'acp-render.js' });
   return window.SpritzACPRender;
 }
diff --git a/ui/public/app-chat-route.test.mjs b/ui/public/app-chat-route.test.mjs
@@ -2,6 +2,7 @@ import test from 'node:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs';
 import vm from 'node:vm';
+import { uiPublicPath } from '../test-paths.mjs';
 
 function createStorage() {
   const values = new Map();
@@ -151,7 +152,7 @@ test('chat hash route initializes without throwing', () => {
   });
   context.globalThis = context.window;
 
-  const script = fs.readFileSync('/Users/onur/repos/spritz/ui/public/app.js', 'utf8');
+  const script = fs.readFileSync(uiPublicPath('app.js'), 'utf8');
   assert.doesNotThrow(() => {
     vm.runInContext(script, context, { filename: 'app.js' });
   });
diff --git a/ui/public/app-list-actions.test.mjs b/ui/public/app-list-actions.test.mjs
@@ -2,6 +2,7 @@ import test from 'node:test';
 import assert from 'node:assert/strict';
 import fs from 'node:fs';
 import vm from 'node:vm';
+import { uiPublicPath } from '../test-paths.mjs';
 
 function createStorage() {
   const values = new Map();
@@ -174,7 +175,7 @@ test('spritz list shows a transitional chat action while workspace chat is still
   });
   context.globalThis = context.window;
 
-  const script = fs.readFileSync('/Users/onur/repos/spritz/ui/public/app.js', 'utf8');
+  const script = fs.readFileSync(uiPublicPath('app.js'), 'utf8');
   vm.runInContext(script, context, { filename: 'app.js' });
 
   await new Promise((resolve) => setTimeout(resolve, 0));
diff --git a/ui/public/preset-config.test.mjs b/ui/public/preset-config.test.mjs
@@ -1,9 +1,10 @@
 import test from 'node:test';
 import assert from 'node:assert/strict';
 import { createRequire } from 'node:module';
+import { uiPublicPath } from '../test-paths.mjs';
 
 const require = createRequire(import.meta.url);
-const { parsePresets } = require('/Users/onur/repos/spritz/ui/public/preset-config.js');
+const { parsePresets } = require(uiPublicPath('preset-config.js'));
 
 test('parsePresets returns raw arrays directly', () => {
   const presets = [{ name: 'OpenClaw', image: 'example/openclaw' }];
diff --git a/ui/test-paths.mjs b/ui/test-paths.mjs
@@ -0,0 +1,18 @@
+import path from 'node:path';
+import { fileURLToPath } from 'node:url';
+
+const uiDir = path.dirname(fileURLToPath(import.meta.url));
+
+/**
+ * Resolve a path relative to the ui/ directory.
+ */
+export function uiPath(...parts) {
+  return path.join(uiDir, ...parts);
+}
+
+/**
+ * Resolve a path relative to the ui/public/ directory.
+ */
+export function uiPublicPath(...parts) {
+  return path.join(uiDir, 'public', ...parts);
+}
