plum-e2e 2.5.3 → 2.5.5

package/bin/plum.js

@@ -285,6 +285,22 @@ function applyServerConfig(cfg) {
 // 127.0.0.1 sidesteps the DNS/happy-eyeballs mismatch entirely.
 const READY_POLL_INTERVAL_MS = 2000;
 const READY_POLL_MAX_ATTEMPTS = 90; // ~3 minutes
+// Each poll attempt must itself be bounded — a single fetch() with no timeout
+// can hang on a stale/dead connection (e.g. right after a container restart)
+// and stall the whole loop indefinitely, regardless of the attempt budget
+// above. This is what caused the intermittent endless "Waiting for server to
+// be ready…" even when the server was already reachable elsewhere.
+const FETCH_ATTEMPT_TIMEOUT_MS = 3000;
+
+async function fetchWithTimeout(url, options = {}) {
+	const controller = new AbortController();
+	const timer = setTimeout(() => controller.abort(), FETCH_ATTEMPT_TIMEOUT_MS);
+	try {
+		return await fetch(url, { ...options, signal: controller.signal });
+	} finally {
+		clearTimeout(timer);
+	}
+}
 
 async function waitForServerReady(apiBase) {
 	const s = clack.spinner();
@@ -293,7 +309,7 @@ async function waitForServerReady(apiBase) {
 	for (let i = 0; i < READY_POLL_MAX_ATTEMPTS; i++) {
 		await new Promise((r) => setTimeout(r, READY_POLL_INTERVAL_MS));
 		try {
-			const res = await fetch(`${apiBase}/auth/needs-setup`);
+			const res = await fetchWithTimeout(`${apiBase}/auth/needs-setup`);
 			if (res.ok) {
 				ready = true;
 				break;
@@ -316,7 +332,7 @@ async function waitForServerReady(apiBase) {
 async function runFirstUserSetup(apiBase, uiUrl) {
 	let needsSetup = false;
 	try {
-		const res = await fetch(`${apiBase}/auth/needs-setup`);
+		const res = await fetchWithTimeout(`${apiBase}/auth/needs-setup`);
 		const data = await res.json();
 		needsSetup = data.needsSetup;
 	} catch {}
@@ -349,7 +365,7 @@ async function runFirstUserSetup(apiBase, uiUrl) {
 	}
 
 	try {
-		const res = await fetch(`${apiBase}/auth/setup`, {
+		const res = await fetchWithTimeout(`${apiBase}/auth/setup`, {
 			method: 'POST',
 			headers: { 'Content-Type': 'application/json' },
 			body: JSON.stringify({ name, email, password })
@@ -463,14 +479,19 @@ async function serverUpdate() {
 		isAlive(registry[String(nodeCfg.id)].pid)
 	);
 
+	// Re-exec `plum` as a fresh process for the restart steps rather than
+	// calling serverRestart()/nodeRestart() directly — this same process
+	// already loaded the OLD code into memory before npm install ran above,
+	// so calling them in-process would rebuild using stale logic no matter
+	// how new the just-installed files on disk actually are.
 	if (hasServerCfg) {
-		clack.log.step('Rebuilding server with new version…');
-		await serverRestart();
+		clack.log.step('Rebuilding server with the newly installed version…');
+		execSync('plum server restart', { stdio: 'inherit' });
 	}
 
 	if (nodeRunning) {
-		clack.log.step('Restarting node runner with new version…');
-		await nodeRestart();
+		clack.log.step('Restarting node runner with the newly installed version…');
+		execSync('plum node restart', { stdio: 'inherit' });
 	}
 
 	if (!hasServerCfg && !nodeRunning) {
@@ -710,6 +731,14 @@ async function openManageRunnersMenu(primaryUrl) {
  * 		"plum <command>" to run the desired command.
  * ------------------------------------------------------ */
 switch (command) {
+	case '--version':
+	case '-v':
+	case 'version': {
+		const pkg = JSON.parse(fs.readFileSync(path.join(plumRoot, 'package.json'), 'utf8'));
+		console.log(pkg.version);
+		break;
+	}
+
 	case 'init': {
 		clack.intro(pc.bgMagenta(pc.white('  🟣 Plum — Init  ')));
 
@@ -1169,6 +1198,7 @@ switch (command) {
 	default:
 		console.log('--------------------------------------\n');
 		console.log('Usage: plum <command>\n');
+		console.log('  --version, -v        Print the installed Plum version');
 		console.log('  init                 Set up a new Plum project');
 		console.log('  server start         Start the full UI stack (interactive)');
 		console.log('    --headless <bool>  Run browsers headless (true/false)');

package/frontend/src/routes/+layout.svelte

@@ -45,9 +45,11 @@
 
 		try {
 			const needsSetup = await checkNeedsSetup();
-			goto(needsSetup ? '/setup' : '/login');
+			await goto(needsSetup ? '/setup' : '/login');
 		} catch {
-			goto('/login');
+			await goto('/login');
+		} finally {
+			ready = true;
 		}
 	});
 </script>

package/frontend/src/routes/+page.svelte

@@ -91,7 +91,9 @@
 	$: filtered = suites
 		.map((suite) => {
 			if (!q) return suite;
-			const suiteMatches = suite.suiteName.toLowerCase().includes(q);
+			const suiteMatches =
+				suite.suiteName.toLowerCase().includes(q) ||
+				suiteIds(suite).some((id) => id.toLowerCase().includes(q));
 			const matchedTests = suite.tests.filter(
 				(t) =>
 					t.testCase.toLowerCase().includes(q) ||

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "plum-e2e",
-	"version": "2.5.3",
+	"version": "2.5.5",
 	"repository": {
 		"type": "git",
 		"url": "https://github.com/silverlunah/plum.git"