plum-e2e 2.6.9 → 2.7.1

package/backend/routes/node.routes.js

@@ -1,35 +1,13 @@
 /*
  * 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/.
+ * Licensed under the MIT License. See LICENSE file in the project root for details.
  */
 
 const express = require('express');
 const router = express.Router();
-const { spawn } = require('child_process');
-const crypto = require('crypto');
-const fs = require('fs');
-const os = require('os');
-const path = require('path');
 const { authGuard } = require('../middleware/auth');
-const { TRIGGER_REMOTE } = require('../constants/triggers');
-
-const BACKEND_DIR = path.resolve(__dirname, '..');
-
-// In-memory job store for active remote executions.
-// Jobs are purged after 10 minutes post-completion.
-const jobs = {};
+const { isNodeMode, DEFAULT_PORT } = require('../constants/env');
+const nodeExecutionService = require('../services/nodeExecutionService');
 
 // Health check — primary server uses this to confirm the node is reachable
 router.get('/ping', authGuard, (req, res) => {
@@ -38,7 +16,7 @@ router.get('/ping', authGuard, (req, res) => {
 
 // Graceful shutdown for node-mode processes (no-op on the primary server)
 router.post('/shutdown', authGuard, (req, res) => {
-	if (process.env.PLUM_MODE !== 'node') {
+	if (!isNodeMode()) {
 		return res.status(403).json({ error: 'Not a node runner' });
 	}
 	res.json({ ok: true });
@@ -50,7 +28,7 @@ router.post('/shutdown', authGuard, (req, res) => {
 // binds the same port, retrying past the brief EADDRINUSE window left by this
 // process shutting down (see the listen retry in server.js).
 router.post('/restart', authGuard, (req, res) => {
-	if (process.env.PLUM_MODE !== 'node') {
+	if (!isNodeMode()) {
 		return res.status(403).json({ error: 'Not a node runner' });
 	}
 	const id = process.env.RUNNER_ID;
@@ -60,128 +38,20 @@ router.post('/restart', authGuard, (req, res) => {
 	res.json({ ok: true });
 	setTimeout(() => {
 		const { startNode } = require('../lib/runnerProcess');
-		startNode({ id, port: process.env.PORT || '3001', token: process.env.NODE_TOKEN });
+		startNode({ id, port: process.env.PORT || DEFAULT_PORT, token: process.env.NODE_TOKEN });
 		process.exit(0);
 	}, 200);
 });
 
 // Start a remote test job
 router.post('/execute', authGuard, (req, res) => {
-	const { tags, browser = 'chromium', workers = 1, tests = null, env: userEnv = {} } = req.body;
-	const jobId = crypto.randomUUID();
-
-	// path.resolve ensures absolute even if TMPDIR env var is set to a relative path
-	const tmpdir = path.resolve(os.tmpdir());
-
-	// Write test files sent by the primary into a per-job temp dir
-	let tempTestsDir = null;
-	if (tests && Object.keys(tests).length > 0) {
-		tempTestsDir = path.join(tmpdir, `plum-job-${jobId}`);
-		for (const [rel, content] of Object.entries(tests)) {
-			const dest = path.join(tempTestsDir, rel);
-			fs.mkdirSync(path.dirname(dest), { recursive: true });
-			fs.writeFileSync(dest, Buffer.from(content, 'base64'));
-		}
-	}
-
-	// Each job writes to its own temp file so concurrent jobs on the same node
-	// cannot clobber each other's reports (shared cucumber_report.json race condition).
-	const reportFile = path.join(tmpdir, `plum-report-${jobId}.json`);
-	const ssDir = path.join(tmpdir, `plum-ss-${jobId}`);
-	fs.mkdirSync(ssDir, { recursive: true });
-
-	jobs[jobId] = {
-		status: 'running',
-		logs: '',
-		exitCode: null,
-		startedAt: Date.now(),
-		meta: { tags: tags || '', browser, workers },
-		tempTestsDir,
-		reportFile,
-		ssDir,
-		pendingScreenshots: []
-	};
-
-	const env = {
-		...process.env,
-		// User/test vars (BASE_URL, IS_HEADLESS, custom secrets) forwarded from the
-		// primary's own .env — nodes are stateless runners and shouldn't need their
-		// own copy. Spread before the job-control vars below so a stray same-named
-		// var in the user's .env can never override how this job actually runs.
-		...userEnv,
-		TAG: tags || '',
-		TRIGGER: TRIGGER_REMOTE,
-		BROWSER: browser,
-		REPORT_RUNNERS: String(workers),
-		CUCUMBER_REPORT_FILE: reportFile,
-		PLUM_SS_DIR: ssDir,
-		...(tempTestsDir ? { TESTS_ROOT: tempTestsDir } : {})
-	};
-	if (workers > 1) env.PARALLEL = String(workers);
-
-	const seenSsFiles = new Set();
-	const ssPoller = setInterval(() => {
-		const job = jobs[jobId];
-		if (!job) {
-			clearInterval(ssPoller);
-			return;
-		}
-		try {
-			const files = fs
-				.readdirSync(ssDir)
-				.filter((f) => f.endsWith('.ss.json'))
-				.sort();
-			for (const f of files) {
-				if (seenSsFiles.has(f)) continue;
-				seenSsFiles.add(f);
-				const filePath = path.join(ssDir, f);
-				const data = JSON.parse(fs.readFileSync(filePath, 'utf8'));
-				job.pendingScreenshots.push(data);
-				try {
-					fs.unlinkSync(filePath);
-				} catch {}
-			}
-		} catch {}
-	}, 400);
-
-	const proc = spawn('npm', ['run', 'test'], { env, shell: true, cwd: BACKEND_DIR });
-	proc.stdout.on('data', (d) => {
-		jobs[jobId].logs += d.toString();
-	});
-	proc.stderr.on('data', (d) => {
-		jobs[jobId].logs += d.toString();
-	});
-	proc.on('close', (code) => {
-		clearInterval(ssPoller);
-		jobs[jobId].status = code === 0 ? 'done' : 'error';
-		jobs[jobId].exitCode = code;
-
-		try {
-			if (fs.existsSync(reportFile)) {
-				jobs[jobId].reportContent = fs.readFileSync(reportFile, 'utf8');
-			}
-		} catch {}
-
-		fs.rm(ssDir, { recursive: true, force: true }, () => {});
-
-		if (jobs[jobId].tempTestsDir) {
-			fs.rm(jobs[jobId].tempTestsDir, { recursive: true, force: true }, () => {});
-		}
-
-		setTimeout(() => {
-			try {
-				fs.unlinkSync(reportFile);
-			} catch {}
-			delete jobs[jobId];
-		}, 600_000);
-	});
-
+	const jobId = nodeExecutionService.startJob(req.body);
 	res.json({ jobId, status: 'started' });
 });
 
 // Fetch the final report content after a job completes
 router.get('/report/:jobId', authGuard, (req, res) => {
-	const job = jobs[req.params.jobId];
+	const job = nodeExecutionService.getJob(req.params.jobId);
 	if (!job) return res.status(404).json({ error: 'Job not found' });
 	if (!job.reportContent) return res.status(404).json({ error: 'Report not ready' });
 	res.json({ content: job.reportContent, meta: job.meta });
@@ -189,17 +59,10 @@ router.get('/report/:jobId', authGuard, (req, res) => {
 
 // Poll job status and streamed logs
 router.get('/execute/:jobId', authGuard, (req, res) => {
-	const job = jobs[req.params.jobId];
-	if (!job) return res.status(404).json({ error: 'Job not found' });
-
 	const offset = parseInt(req.query.offset || '0', 10);
-	const screenshots = job.pendingScreenshots.splice(0);
-	res.json({
-		status: job.status,
-		logs: job.logs.slice(offset),
-		exitCode: job.exitCode,
-		screenshots
-	});
+	const result = nodeExecutionService.pollJob(req.params.jobId, offset);
+	if (!result) return res.status(404).json({ error: 'Job not found' });
+	res.json(result);
 });
 
 module.exports = router;

package/backend/routes/reports.routes.js

@@ -1,18 +1,6 @@
 /*
  * 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/.
+ * Licensed under the MIT License. See LICENSE file in the project root for details.
  */
 
 const express = require('express');

package/backend/routes/runners.routes.js

@@ -1,25 +1,15 @@
 /*
  * 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/.
+ * Licensed under the MIT License. See LICENSE file in the project root for details.
  */
 
 const express = require('express');
 const router = express.Router();
 const runnerService = require('../services/runnerService');
+const { jwtAuth } = require('../middleware/jwtAuth');
+const { requireAdmin } = require('../middleware/requireAdmin');
 
-router.get('/', async (req, res) => {
+router.get('/', jwtAuth, requireAdmin, async (req, res) => {
 	try {
 		const runners = await runnerService.getAll();
 		res.json({ runners });
@@ -28,7 +18,7 @@ router.get('/', async (req, res) => {
 	}
 });
 
-router.post('/probe', async (req, res) => {
+router.post('/probe', jwtAuth, requireAdmin, async (req, res) => {
 	try {
 		const { url, token } = req.body;
 		if (!url || !token)
@@ -40,7 +30,7 @@ router.post('/probe', async (req, res) => {
 	}
 });
 
-router.post('/', async (req, res) => {
+router.post('/', jwtAuth, requireAdmin, async (req, res) => {
 	try {
 		const { name, url, token, browser } = req.body;
 		if (!name || !url || !token)
@@ -52,7 +42,7 @@ router.post('/', async (req, res) => {
 	}
 });
 
-router.put('/:id', async (req, res) => {
+router.put('/:id', jwtAuth, requireAdmin, async (req, res) => {
 	try {
 		const { name, url, token, browser } = req.body;
 		const runner = await runnerService.update(req.params.id, { name, url, token, browser });
@@ -62,7 +52,7 @@ router.put('/:id', async (req, res) => {
 	}
 });
 
-router.delete('/:id', async (req, res) => {
+router.delete('/:id', jwtAuth, requireAdmin, async (req, res) => {
 	try {
 		await runnerService.stop(req.params.id);
 		await runnerService.remove(req.params.id);
@@ -72,7 +62,7 @@ router.delete('/:id', async (req, res) => {
 	}
 });
 
-router.post('/:id/ping', async (req, res) => {
+router.post('/:id/ping', jwtAuth, requireAdmin, async (req, res) => {
 	try {
 		const result = await runnerService.ping(req.params.id);
 		res.json(result);
@@ -81,7 +71,7 @@ router.post('/:id/ping', async (req, res) => {
 	}
 });
 
-router.post('/:id/stop', async (req, res) => {
+router.post('/:id/stop', jwtAuth, requireAdmin, async (req, res) => {
 	try {
 		const result = await runnerService.stop(req.params.id);
 		res.json(result);
@@ -90,7 +80,7 @@ router.post('/:id/stop', async (req, res) => {
 	}
 });
 
-router.post('/:id/restart', async (req, res) => {
+router.post('/:id/restart', jwtAuth, requireAdmin, async (req, res) => {
 	try {
 		const result = await runnerService.restart(req.params.id);
 		res.json(result);

package/backend/routes/settings.routes.js

@@ -1,18 +1,6 @@
 /*
  * 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/.
+ * Licensed under the MIT License. See LICENSE file in the project root for details.
  */
 
 const express = require('express');
@@ -21,8 +9,9 @@ const settingsService = require('../services/settingsService');
 const testSuiteService = require('../services/testSuiteService');
 const testCaseService = require('../services/testCaseService');
 const { jwtAuth } = require('../middleware/jwtAuth');
+const { requireAdmin } = require('../middleware/requireAdmin');
 
-router.get('/project', async (req, res, next) => {
+router.get('/project', jwtAuth, requireAdmin, async (req, res, next) => {
 	try {
 		const project = await settingsService.getProject();
 		res.json(project);
@@ -31,7 +20,7 @@ router.get('/project', async (req, res, next) => {
 	}
 });
 
-router.post('/project', async (req, res, next) => {
+router.post('/project', jwtAuth, requireAdmin, async (req, res, next) => {
 	try {
 		const { name, logoUrl, timezone, maxRetries } = req.body;
 		const project = await settingsService.updateProject({ name, logoUrl, timezone, maxRetries });
@@ -76,7 +65,7 @@ router.post('/test-prefixes/migrate', jwtAuth, async (req, res, next) => {
 	}
 });
 
-router.get('/integrations', async (req, res, next) => {
+router.get('/integrations', jwtAuth, requireAdmin, async (req, res, next) => {
 	try {
 		const webhooks = await settingsService.getWebhooks();
 		res.json(webhooks);
@@ -85,7 +74,7 @@ router.get('/integrations', async (req, res, next) => {
 	}
 });
 
-router.post('/integrations', async (req, res, next) => {
+router.post('/integrations', jwtAuth, requireAdmin, async (req, res, next) => {
 	try {
 		const { discordWebhookUrl, slackWebhookUrl, notifyPublicUrl } = req.body;
 		const project = await settingsService.updateWebhooks({

package/backend/routes/test-cases.routes.js

@@ -1,18 +1,6 @@
 /*
  * 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/.
+ * Licensed under the MIT License. See LICENSE file in the project root for details.
  */
 
 const express = require('express');

package/backend/routes/test-runs.routes.js

@@ -1,18 +1,6 @@
 /*
  * 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/.
+ * Licensed under the MIT License. See LICENSE file in the project root for details.
  */
 
 const express = require('express');

package/backend/routes/test-suites.routes.js

@@ -1,18 +1,6 @@
 /*
  * 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/.
+ * Licensed under the MIT License. See LICENSE file in the project root for details.
  */
 
 const express = require('express');

package/backend/routes/tests.routes.js

@@ -1,18 +1,6 @@
 /*
  * 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/.
+ * Licensed under the MIT License. See LICENSE file in the project root for details.
  */
 
 const express = require('express');

package/backend/routes/trigger.routes.js

@@ -1,189 +1,47 @@
 /*
  * 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/.
+ * Licensed under the MIT License. See LICENSE file in the project root for details.
  */
 
-const path = require('path');
-const fs = require('fs');
-const os = require('os');
-const { spawn } = require('child_process');
-const { randomUUID } = require('crypto');
 const express = require('express');
 const router = express.Router();
 const { jwtAuth } = require('../middleware/jwtAuth');
-const prisma = require('../services/prisma');
-const { startSsPoller } = require('../lib/screenshotPoller');
 const { TRIGGER_TYPE } = require('../constants/triggers');
-const settingsService = require('../services/settingsService');
-const reportService = require('../services/reportService');
-const { readCucumberReportFile } = require('../lib/reportFilename');
-const { runWithRetries } = require('../lib/retryRunner');
-
-const BACKEND_DIR = path.resolve(__dirname, '..');
-const JOB_TTL_MS = 60 * 60 * 1000; // 1 hour
-
-// In-memory job store: jobId → { status, exitCode, reportId, startedAt }
-const jobs = new Map();
-
-let _io = null;
-function setSocketIO(io) {
-	_io = io;
-}
-
-function pruneOldJobs() {
-	const cutoff = Date.now() - JOB_TTL_MS;
-	for (const [id, job] of jobs) {
-		if (job.startedAt < cutoff) jobs.delete(id);
-	}
-}
+const { DEFAULT_BROWSER } = require('../constants/defaults');
+const { JOB_STATUS } = require('../constants/jobStatus');
+const triggerService = require('../services/triggerService');
 
 router.post('/', jwtAuth, async (req, res, next) => {
 	try {
-		pruneOldJobs();
-
-		const { tag = '', browser = 'chromium', workers = 1, baseUrl, testRunId, source } = req.body;
+		const {
+			tag = '',
+			browser = DEFAULT_BROWSER,
+			workers = 1,
+			baseUrl,
+			testRunId,
+			source
+		} = req.body;
 		const trigger = source === 'mcp' ? TRIGGER_TYPE.MCP : TRIGGER_TYPE.EXTERNAL;
-		const { maxRetries } = await settingsService.getProject();
-
-		const jobId = randomUUID();
-		const startedAt = Date.now();
-		jobs.set(jobId, { status: 'running', exitCode: null, reportId: null, startedAt });
-
-		if (_io) {
-			_io.emit('bg-run-start', {
-				runId: jobId,
-				kind: trigger,
-				label: trigger === TRIGGER_TYPE.MCP ? 'MCP run' : 'External run',
-				meta: { tag, browser, workers }
-			});
-		}
-
-		const onLog = (text) => {
-			if (_io) _io.emit('bg-run-log', { runId: jobId, log: text });
-		};
-
-		function runAttempt(currentTag, suppressSave) {
-			return new Promise((resolve) => {
-				const ssDir = path.join(os.tmpdir(), `plum-trigger-ss-${jobId}-${Date.now()}`);
-				fs.mkdirSync(ssDir, { recursive: true });
-
-				const env = {
-					...process.env,
-					TAG: currentTag,
-					TRIGGER: trigger,
-					BROWSER: browser,
-					REPORT_RUNNERS: String(workers),
-					PLUM_SS_DIR: ssDir
-				};
-				if (Number(workers) > 1) env.PARALLEL = String(workers);
-				if (testRunId) env.TEST_RUN_ID = testRunId;
-				if (baseUrl) env.BASE_URL = baseUrl;
-				if (suppressSave) env.PLUM_MODE = 'node';
-
-				const proc = spawn('npm', ['run', 'test'], { env, shell: true, cwd: BACKEND_DIR });
-
-				const ssPoller = startSsPoller(ssDir, ({ stepName, data }) => {
-					if (_io) _io.emit('bg-run-screenshot', { runId: jobId, stepName, data });
-				});
-
-				proc.stdout.on('data', (d) => onLog(d.toString()));
-				proc.stderr.on('data', (d) => onLog(`[ERROR] ${d.toString()}`));
-
-				proc.on('close', (code) => {
-					clearInterval(ssPoller);
-					fs.rm(ssDir, { recursive: true, force: true }, () => {});
-					resolve({ code, raw: suppressSave ? readCucumberReportFile() : null });
-				});
-			});
-		}
-
-		if (maxRetries === 0) {
-			runAttempt(tag, false).then(async ({ code }) => {
-				let reportId = null;
-				try {
-					// Find the latest report created after this job started
-					const report = await prisma.report.findFirst({
-						where: { createdAt: { gte: new Date(startedAt) } },
-						orderBy: { createdAt: 'desc' },
-						select: { id: true, status: true }
-					});
-					reportId = report?.id ?? null;
-					if (reportId) {
-						await prisma.report.update({
-							where: { id: reportId },
-							data: { duration: Date.now() - startedAt }
-						});
-					}
-					jobs.set(jobId, {
-						status: code === 130 ? 'cancelled' : 'done',
-						exitCode: code,
-						reportId,
-						startedAt
-					});
-				} catch {
-					jobs.set(jobId, { status: 'done', exitCode: code, reportId: null, startedAt });
-				}
-
-				if (_io) _io.emit('bg-run-done', { runId: jobId, code, reportId });
-			});
-		} else {
-			runWithRetries({
-				maxRetries,
-				spawnAttempt: async (tagOverride) => {
-					const { code, raw } = await runAttempt(tagOverride ?? tag, true);
-					return { code, rawJson: raw ? JSON.parse(raw) : [] };
-				},
-				onLog
-			}).then(async ({ code, rawJson, attempts }) => {
-				let reportId = null;
-				try {
-					const report = await reportService.saveReport({
-						rawCucumberJson: rawJson,
-						tags: tag,
-						triggerType: trigger,
-						browser,
-						testRunId: testRunId ?? null,
-						duration: Date.now() - startedAt,
-						attempts
-					});
-					reportId = report.id;
-					jobs.set(jobId, {
-						status: code === 130 ? 'cancelled' : 'done',
-						exitCode: code,
-						reportId,
-						startedAt
-					});
-				} catch {
-					jobs.set(jobId, { status: 'done', exitCode: code, reportId: null, startedAt });
-				}
-
-				if (_io) _io.emit('bg-run-done', { runId: jobId, code, reportId });
-			});
-		}
 
-		res.status(202).json({ jobId, status: 'running' });
+		const jobId = await triggerService.startRun({
+			tag,
+			browser,
+			workers,
+			baseUrl,
+			testRunId,
+			trigger
+		});
+		res.status(202).json({ jobId, status: JOB_STATUS.RUNNING });
 	} catch (e) {
 		next(e);
 	}
 });
 
 router.get('/:jobId', jwtAuth, (req, res) => {
-	const job = jobs.get(req.params.jobId);
+	const job = triggerService.getJob(req.params.jobId);
 	if (!job) return res.status(404).json({ error: 'Job not found' });
 	res.json(job);
 });
 
 module.exports = router;
-router.setSocketIO = setSocketIO;
+router.setSocketIO = triggerService.setSocketIO;