mirror of
https://github.com/nexu-io/open-design.git
synced 2026-10-27 09:51:12 +01:00
shangxinyu1andGitHub
751c9de56d
* test: add e2e ui automation suite * fix review feedback for ui e2e suite Resolved the FileWorkspace.tsx merge-marker issue and kept the intended combination of multiple, accept="image/*", and data-testid. Updated the e2e port handling so the test config no longer relies on a single hardcoded app port. It now resolves an available port first and passes the same port selection through the dev server and Playwright base URL. Since main has moved to the Next.js dev stack, this was also adapted from the old Vite-based flow to NEXT_PORT. Kept test:ui serialized so cleanup completes before Playwright starts. Updated reset-e2e-artifacts.mjs so cleanup failures are surfaced with a warning instead of being silently swallowed, except for the expected ENOENT case.
73 lines
2.5 KiB
JavaScript
Executable File
73 lines
2.5 KiB
JavaScript
Executable File
#!/usr/bin/env node
|
|
// Launcher for `npm run dev:all`.
|
|
//
|
|
// Probes for free ports for the daemon (OD_PORT, default 7456) and the
|
|
// Next.js dev server (NEXT_PORT, default 3000) before spawning
|
|
// `concurrently`, so a stray process holding either port doesn't kill the
|
|
// whole boot. The resolved ports are exported into the child env, which
|
|
// means:
|
|
// * the daemon's cli.js sees the new OD_PORT and binds to it
|
|
// * next.config.ts reads the same OD_PORT and proxies /api, /artifacts,
|
|
// /frames to the daemon's actual port
|
|
// * Next.js binds to NEXT_PORT (we pass `next dev -p $NEXT_PORT` so the
|
|
// `dev` script can stay parameter-free for the common single-process
|
|
// case where the user runs just `pnpm dev`)
|
|
//
|
|
// If a port is busy we walk forward up to PORT_SEARCH_RANGE steps and log
|
|
// the switch so the user notices.
|
|
|
|
import { spawn } from 'node:child_process';
|
|
import { findFreePort } from './resolve-dev-ports.mjs';
|
|
|
|
const desiredDaemon = Number(process.env.OD_PORT) || 7456;
|
|
const desiredNext = Number(process.env.NEXT_PORT) || 3000;
|
|
const strictDaemonPort = process.env.OD_PORT_STRICT === '1';
|
|
const strictNextPort = process.env.NEXT_PORT_STRICT === '1';
|
|
|
|
const daemonPort = strictDaemonPort
|
|
? desiredDaemon
|
|
: await findFreePort(desiredDaemon, 'daemon');
|
|
const nextPort = strictNextPort
|
|
? desiredNext
|
|
: await findFreePort(desiredNext, 'next');
|
|
|
|
if (daemonPort !== desiredDaemon) {
|
|
console.log(
|
|
`[dev:all] daemon port ${desiredDaemon} is busy, switching to ${daemonPort}`,
|
|
);
|
|
}
|
|
if (nextPort !== desiredNext) {
|
|
console.log(
|
|
`[dev:all] next port ${desiredNext} is busy, switching to ${nextPort}`,
|
|
);
|
|
}
|
|
|
|
const env = {
|
|
...process.env,
|
|
OD_PORT: String(daemonPort),
|
|
NEXT_PORT: String(nextPort),
|
|
PORT: String(nextPort),
|
|
};
|
|
|
|
// `npm:daemon` is the shorthand for the daemon script, and `next dev -p
|
|
// <port>` is invoked directly so we can pass the resolved port without
|
|
// round-tripping through npm scripts. Keep the port numeric before it reaches
|
|
// the command string, and avoid shell interpretation on POSIX; Windows needs
|
|
// shell mode so the local `.cmd` shim can resolve.
|
|
const child = spawn(
|
|
'concurrently',
|
|
['-k', '-n', 'daemon,web', '-c', 'cyan,magenta', 'npm:daemon', `next dev -p ${nextPort}`],
|
|
{ env, stdio: 'inherit', shell: process.platform === 'win32' },
|
|
);
|
|
|
|
child.on('exit', (code, signal) => {
|
|
if (signal) process.kill(process.pid, signal);
|
|
else process.exit(code ?? 0);
|
|
});
|
|
|
|
for (const sig of ['SIGINT', 'SIGTERM']) {
|
|
process.on(sig, () => {
|
|
if (!child.killed) child.kill(sig);
|
|
});
|
|
}
|