plum-e2e 2.6.7 → 2.6.9

package/.env

@@ -15,5 +15,10 @@
 # along with Plum. If not, see https://www.gnu.org/licenses/.
 #
 
+#
+# This file is part of Plum.
+# Licensed under the MIT License. See LICENSE file in the project root for details.
+#
+
 
 IS_HEADLESS=true

package/backend/config/scripts/generate-report.js

@@ -32,6 +32,14 @@ if (!fs.existsSync(jsonReportFile)) {
 // and saves the combined report after all lanes finish.
 if (process.env.PLUM_MODE === 'node') process.exit(0);
 
+// `run-test` runs on the host without Docker/Postgres, so DATABASE_URL is
+// never set there — skip the DB save instead of letting Prisma's connection
+// attempt surface as an unhandled-looking stack trace after tests already ran.
+if (!process.env.DATABASE_URL) {
+	console.log('No DATABASE_URL configured — skipping report save to database.');
+	process.exit(0);
+}
+
 (async () => {
 	try {
 		const raw = JSON.parse(fs.readFileSync(jsonReportFile, 'utf8'));

package/backend/lib/runnerProcess.js

@@ -153,24 +153,22 @@ function statusOf(id, registry = loadRegistry()) {
 }
 
 /**
- * Ensures a local node can actually run tests: backend dependencies must be
- * installed and the Playwright browser binaries present. A node with neither
- * starts fine but every scenario fails at the browser-launch hook. Idempotent —
- * npm install is skipped when node_modules already exists, and `playwright
- * install` no-ops when the browser is already downloaded.
+ * Installs backend npm dependencies if missing or stale. Shared by `prepareEnv`
+ * (which additionally provisions Playwright browsers for test execution) and
+ * any caller that only needs `backend/node_modules` to exist, such as `plum
+ * mcp` — spawning the MCP server without this guard fails outright if the
+ * package was reinstalled/updated since the last time deps were installed
+ * (npm install -g wipes node_modules on upgrade) and the user never ran a
+ * command that happens to call prepareEnv first.
  *
- * npm/npx are .cmd shims on Windows, so these must run through a shell.
+ * npm is a .cmd shim on Windows, so this must run through a shell.
  *
- * stdin is left as 'ignore' (not inherited): these installers only need to
- * print progress, and letting them touch stdin corrupts the raw-mode state of
+ * stdin is left as 'ignore' (not inherited): the installer only needs to
+ * print progress, and letting it touch stdin corrupts the raw-mode state of
  * an interactive prompt running in the same terminal (the caller's menu would
  * wedge after the install finishes).
  */
-function prepareEnv() {
-	const stdio = ['ignore', 'inherit', 'inherit'];
-
-	// Re-run npm install when node_modules is missing OR when the plum package
-	// version has changed (npm install -g wipes node_modules on upgrade).
+function ensureBackendDeps() {
 	const markerPath = path.join(BACKEND_DIR, 'node_modules', '.plum-version');
 	let installedVersion = null;
 	try {
@@ -184,9 +182,24 @@ function prepareEnv() {
 		!fs.existsSync(path.join(BACKEND_DIR, 'node_modules')) ||
 		installedVersion !== currentVersion
 	) {
-		execSync('npm install', { cwd: BACKEND_DIR, stdio, shell: true });
+		execSync('npm install', {
+			cwd: BACKEND_DIR,
+			stdio: ['ignore', 'inherit', 'inherit'],
+			shell: true
+		});
 		fs.writeFileSync(markerPath, currentVersion, 'utf8');
 	}
+}
+
+/**
+ * Ensures a local node can actually run tests: backend dependencies must be
+ * installed and the Playwright browser binaries present. A node with neither
+ * starts fine but every scenario fails at the browser-launch hook. Idempotent —
+ * npm install is skipped when node_modules already exists, and `playwright
+ * install` no-ops when the browser is already downloaded.
+ */
+function prepareEnv() {
+	ensureBackendDeps();
 
 	// --with-deps also installs the OS-level shared libraries (libnss3, libatk,
 	// etc.) Chromium/Firefox need to actually launch — without it, a fresh
@@ -194,7 +207,7 @@ function prepareEnv() {
 	// "Host system is missing dependencies to run browsers."
 	execSync('npx playwright install --with-deps chromium firefox', {
 		cwd: BACKEND_DIR,
-		stdio,
+		stdio: ['ignore', 'inherit', 'inherit'],
 		shell: true
 	});
 }
@@ -316,6 +329,7 @@ module.exports = {
 	killPort,
 	pruneDead,
 	statusOf,
+	ensureBackendDeps,
 	prepareEnv,
 	startNode,
 	stopNode

package/backend/logs/runner-cmqneqerz0000qq01i5ap2rvl.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 3002 (node/runner mode)
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests

package/backend/logs/runner-cmqnfv7kr0000r101aeocm8eu.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 3002 (node/runner mode)
 (node:23570) [DEP0190] DeprecationWarning: Passing args to a child process with shell option true can lead to security vulnerabilities, as the arguments are not escaped, only concatenated.

package/backend/logs/runner-cmqnfvb560001r101qoi0phau.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 node:events:487
       throw er; // Unhandled 'error' event

package/backend/logs/runner-cmqnfvlm20002r101gsyqb837.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 3003 (node/runner mode)
 (node:23686) [DEP0190] DeprecationWarning: Passing args to a child process with shell option true can lead to security vulnerabilities, as the arguments are not escaped, only concatenated.

package/backend/logs/runner-cmqnfvqfy0003r101fh41pzx3.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 3004 (node/runner mode)
 (node:23733) [DEP0190] DeprecationWarning: Passing args to a child process with shell option true can lead to security vulnerabilities, as the arguments are not escaped, only concatenated.

package/backend/logs/runner-cmqnfvvwo0004r101q4dtqxd2.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 3005 (node/runner mode)
 (node:23776) [DEP0190] DeprecationWarning: Passing args to a child process with shell option true can lead to security vulnerabilities, as the arguments are not escaped, only concatenated.

package/backend/logs/runner-cmr8o2x3n0000mg01410tpuck.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 3002 (node/runner mode)
 (node:89713) [DEP0190] DeprecationWarning: Passing args to a child process with shell option true can lead to security vulnerabilities, as the arguments are not escaped, only concatenated.

package/backend/logs/runner-cmr90hvcx0000n2016edtmrtv.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 node:events:487
       throw er; // Unhandled 'error' event

package/backend/logs/runner-cmr90i7h20001n201g2wg6v3n.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 node:events:487
       throw er; // Unhandled 'error' event

package/backend/logs/runner-cmr90iilm0002n201mmisva95.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 node:events:487
       throw er; // Unhandled 'error' event

package/backend/logs/runner-cmr90j70r0003n201xz8525dz.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 5177 (node/runner mode)
 (node:98310) [DEP0190] DeprecationWarning: Passing args to a child process with shell option true can lead to security vulnerabilities, as the arguments are not escaped, only concatenated.

package/backend/logs/runner-cmr90yup70000qp017ianc2gp.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 node:events:487
       throw er; // Unhandled 'error' event

package/backend/logs/runner-cmr90zbtm0001qp01u9tfdkh1.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 5178 (node/runner mode)
 (node:825) [DEP0190] DeprecationWarning: Passing args to a child process with shell option true can lead to security vulnerabilities, as the arguments are not escaped, only concatenated.

package/backend/logs/runner-cmr91a3mw0000tb01414rd1df.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 5178 (node/runner mode)
 (node:2599) [DEP0190] DeprecationWarning: Passing args to a child process with shell option true can lead to security vulnerabilities, as the arguments are not escaped, only concatenated.

package/backend/logs/runner-cmr91dhla0001tb01jp7thjnl.log

@@ -14,6 +14,10 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 5180 (node/runner mode)
 (node:3077) [DEP0190] DeprecationWarning: Passing args to a child process with shell option true can lead to security vulnerabilities, as the arguments are not escaped, only concatenated.

package/backend/logs/runner-cmrkk6fcl0000qj012kve79f4.log

@@ -14,5 +14,9 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 3999 (node/runner mode)

package/backend/logs/runner-cmrkl8sym0000qy01euz4270n.log

@@ -14,5 +14,9 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 3996 (node/runner mode)

package/backend/logs/runner-cmrkl9oyx0001qy0137nzr94d.log

@@ -14,5 +14,9 @@ GNU General Public License for more details.
 You should have received a copy of the GNU General Public License
 along with Plum. If not, see https://www.gnu.org/licenses/.
 */
+/*
+This file is part of Plum.
+Licensed under the MIT License. See LICENSE file in the project root for details.
+*/
 📂 Loading tests from: /Users/silverlunah/Projects/plum/backend/tests
 Backend running on port 3996 (node/runner mode)

package/bin/plum.js

@@ -1283,6 +1283,19 @@ switch (command) {
 	}
 
 	case 'mcp': {
+		// Claude launches `plum mcp` directly, skipping every other command
+		// (`server start`, `node start`, `run-test`) that would otherwise have
+		// already called ensureBackendDeps/prepareEnv. Without this guard, a
+		// stale/missing backend/node_modules (e.g. after `npm i -g plum-e2e@latest`
+		// wipes it) makes the MCP server fail outright on its own requires.
+		try {
+			const { ensureBackendDeps } = runnerProcessLib();
+			ensureBackendDeps();
+		} catch (e) {
+			console.error(`Could not verify backend dependencies: ${e.message}`);
+			process.exit(1);
+		}
+
 		const mcpScript = path.join(plumRoot, 'backend', 'mcp', 'server.js');
 		spawn(process.execPath, [mcpScript], {
 			stdio: 'inherit',

package/bin/postinstall.js

@@ -0,0 +1,46 @@
+/*
+ * This file is part of Plum.
+ *
+ * Plum is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation, either version 3 of the License, or
+ * (at your option) any later version.
+ *
+ * Plum is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License
+ * along with Plum. If not, see https://www.gnu.org/licenses/.
+ */
+
+/**
+ * Runs on every `npm install` of this package (including `npm i -g plum-e2e`
+ * and re-installs on update), so `plum mcp` and other backend-dependent
+ * commands work immediately without the user first having to run `plum
+ * server start` or `plum node start` — the only commands that used to trigger
+ * a backend `npm install` on their own.
+ *
+ * Best-effort: a failure here must not fail the overall `npm install`, so
+ * errors are logged, not thrown.
+ */
+
+import path from 'path';
+import { fileURLToPath } from 'url';
+import { createRequire } from 'module';
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const require = createRequire(import.meta.url);
+
+try {
+	const { ensureBackendDeps } = require(
+		path.join(__dirname, '..', 'backend', 'lib', 'runnerProcess.js')
+	);
+	ensureBackendDeps();
+} catch (e) {
+	console.warn(`[plum] Could not install backend dependencies automatically: ${e.message}`);
+	console.warn(
+		'[plum] If `plum mcp` fails to start, run `npm install` inside its backend/ folder.'
+	);
+}