common-qa-mcp 0.8.1 → 0.10.0

package/dist/adapters/simplifyqa/adapter.js

@@ -1,7 +1,7 @@
 import os from 'node:os';
 import { makeClient } from '../http.js';
 import { RemoteMcpAdapter } from '../remoteMcp/adapter.js';
-import { applyCollectionSteps, buildTestcasePayload, deriveTcType, extractParamNames, fromSqaStep, looksSensitive, normalizeSuiteEntry, numericId, paramDefault, pickArray, str, SUITE_UPDATE_ALLOWED_KEYS, TECH_BY_STEPTYPE, toDefectSummary, toEpic, toEpicSummary, toFeature, toFeatureSummary, toFunction, toFunctionSummary, toModuleSaved, toModuleSummary, toObject, toReleaseSummary, toSavedRef, toSprintSummary, toStory, toStorySummary, toSuite, toSuiteSummary, toTaskSummary, toTcSummary, toTestCase, toVersionEntries, withTotal, } from './mapper.js';
+import { applyCollectionSteps, buildTestcasePayload, deriveTcType, extractApiParamNames, extractParamNames, fromSqaStep, looksSensitive, normalizeSuiteEntry, numericId, paramDefault, pickArray, str, SUITE_UPDATE_ALLOWED_KEYS, TECH_BY_STEPTYPE, toDefectSummary, toEpic, toEpicSummary, toFeature, toFeatureSummary, toFunction, toFunctionSummary, toModuleSaved, toModuleSummary, toObject, toImageUrl, toReleaseSummary, toSavedRef, toSprintSummary, toStory, toStorySummary, toSuite, toSuiteSummary, toTaskSummary, toTcSummary, toTestCase, toVersionEntries, withTotal, } from './mapper.js';
 /** SQA run-row status → neutral RunStatus.state. */
 function mapRunState(s) {
     switch (s.toUpperCase()) {
@@ -65,13 +65,17 @@ function normalizeFormula(doc, projectId) {
         parameters: params,
     };
 }
-/** SQA per-step status → neutral step status. */
+/** SQA per-step status → neutral step status. Matches RunState terminals ('passed'/'failed'/
+ *  'skipped'/'stopped') so the same icon/formatting logic works for top-level state AND per-step
+ *  rows (run.tools.ts:180 keys off these values — the earlier 'pass'/'fail' names left every
+ *  step icon defaulting to '•'). Non-terminal SQA states (INFO, INPROGRESS, NOTEXECUTED) pass
+ *  through lowercased so the tool can still surface them as diagnostic text. */
 function mapStepStatus(s) {
     switch (s.toUpperCase()) {
-        case 'PASSED': return 'pass';
-        case 'FAILED': return 'fail';
-        case 'SKIPPED': return 'skip';
-        case 'STOPPED': return 'stop';
+        case 'PASSED': return 'passed';
+        case 'FAILED': return 'failed';
+        case 'SKIPPED': return 'skipped';
+        case 'STOPPED': return 'stopped';
         default: return s.toLowerCase();
     }
 }
@@ -608,7 +612,7 @@ export class SimplifyQaAdapter {
     async hydrateObjectNames(tc) {
         const ids = new Set();
         for (const s of tc.steps) {
-            if (s.objectRef?.id && !s.objectRef.name)
+            if (s.objectRef?.id && (!s.objectRef.name || !s.objectRef.thumbnailUrl))
                 ids.add(s.objectRef.id);
         }
         if (!ids.size)
@@ -623,6 +627,7 @@ export class SimplifyQaAdapter {
             const raw = await this.http.postJson('/tm/object/search', body);
             const rows = pickArray(raw, 'data', 'content', 'objects');
             const byId = new Map();
+            const imgById = new Map();
             for (const r of rows) {
                 const rid = str(r, 'id', '_id');
                 if (!rid)
@@ -630,9 +635,19 @@ export class SimplifyQaAdapter {
                 const name = str(r, 'disname', 'description', 'placeholder');
                 if (name)
                     byId.set(rid, name);
+                const thumbnailUrl = toImageUrl(r?.thumbnail_url, this.cfg.baseUrl);
+                const pageScreenshotUrl = toImageUrl(r?.pageScreenShot_url, this.cfg.baseUrl);
+                if (thumbnailUrl || pageScreenshotUrl) {
+                    imgById.set(rid, { ...(thumbnailUrl ? { thumbnailUrl } : {}), ...(pageScreenshotUrl ? { pageScreenshotUrl } : {}) });
+                }
             }
             for (const s of tc.steps) {
-                if (!s.objectRef?.id || s.objectRef.name)
+                if (!s.objectRef?.id)
+                    continue;
+                const img = imgById.get(s.objectRef.id);
+                if (img)
+                    Object.assign(s.objectRef, img);
+                if (s.objectRef.name)
                     continue;
                 const name = byId.get(s.objectRef.id);
                 if (name)
@@ -1016,15 +1031,41 @@ export class SimplifyQaAdapter {
         return str(rows[0], 'id', '_id', 'featureId') ?? ref;
     }
     // ---------------------------------------------------------------------- Modules, releases, sprints
+    /** Pages internally until the server returns a short page (or the total is reached) — the tool
+     *  has no limit/offset of its own and always wants the WHOLE project tree; `searchBody`'s default
+     *  page size (100) previously truncated any project with more modules than that silently. */
     async listModules() {
         const projectId = this.requireProjectId();
-        const body = this.searchBody([
-            { column: 'projectId', value: projectId },
-            { column: 'deleted', value: false },
-        ], { selectFields: ['id', 'code', 'name', 'parentId', 'isRoot', 'childIds'] });
-        const raw = await this.http.postJson('/pm/module/search', body);
-        const rows = pickArray(raw, 'data', 'content', 'modules');
-        return withTotal(rows.map(toModuleSummary), raw);
+        const PAGE = 100;
+        const SAFETY_CAP = 5000;
+        const rows = [];
+        let totalCount;
+        let startIndex = 0;
+        for (;;) {
+            const body = this.searchBody([
+                { column: 'projectId', value: projectId },
+                { column: 'deleted', value: false },
+            ], { selectFields: ['id', 'code', 'name', 'parentId', 'isRoot', 'childIds'], startIndex, limit: PAGE });
+            const raw = await this.http.postJson('/pm/module/search', body);
+            const page = pickArray(raw, 'data', 'content', 'modules');
+            if (totalCount === undefined) {
+                const t = raw?.totalCount ?? raw?.total ?? raw?.totalRecords;
+                if (typeof t === 'number')
+                    totalCount = t;
+            }
+            rows.push(...page);
+            if (page.length < PAGE)
+                break;
+            if (rows.length >= SAFETY_CAP)
+                break;
+            if (totalCount !== undefined && rows.length >= totalCount)
+                break;
+            startIndex += PAGE;
+        }
+        const mapped = rows.map(toModuleSummary);
+        if (totalCount !== undefined)
+            mapped.totalCount = totalCount;
+        return mapped;
     }
     /** GET /pm/module/:id — server returns an array; empty array → NotFound. */
     async getModule(id) {
@@ -1227,23 +1268,73 @@ export class SimplifyQaAdapter {
         // reaches that branch after emptyIds.push).
         return { dryRun: false, deleted: true, dependencies: [], hasBlockers: false };
     }
+    /** `list_releases` has no limit/offset of its own — it always wants the WHOLE visible set — so
+     *  page internally past `searchBody`'s default 100 instead of silently truncating (same fix as
+     *  listModules). */
     async listReleases(q) {
         const fields = [{ column: 'deleted', value: false }];
         if (q.scope === 'project')
             fields.push({ column: 'projectIds', value: this.requireProjectId() });
-        const body = this.searchBody(fields);
-        const raw = await this.http.postJson('/pm/release/search', body);
-        const rows = pickArray(raw, 'data', 'content', 'releases');
-        return withTotal(rows.map(toReleaseSummary), raw);
+        const PAGE = 100;
+        const SAFETY_CAP = 5000;
+        const rows = [];
+        let totalCount;
+        let startIndex = 0;
+        for (;;) {
+            const body = this.searchBody(fields, { startIndex, limit: PAGE });
+            const raw = await this.http.postJson('/pm/release/search', body);
+            const page = pickArray(raw, 'data', 'content', 'releases');
+            if (totalCount === undefined) {
+                const t = raw?.totalCount ?? raw?.total ?? raw?.totalRecords;
+                if (typeof t === 'number')
+                    totalCount = t;
+            }
+            rows.push(...page);
+            if (page.length < PAGE)
+                break;
+            if (rows.length >= SAFETY_CAP)
+                break;
+            if (totalCount !== undefined && rows.length >= totalCount)
+                break;
+            startIndex += PAGE;
+        }
+        const mapped = rows.map(toReleaseSummary);
+        if (totalCount !== undefined)
+            mapped.totalCount = totalCount;
+        return mapped;
     }
+    /** Same unconditional-pagination fix as listReleases — `list_sprints` has no limit/offset either. */
     async listSprints(q) {
         const fields = [{ column: 'deleted', value: false }];
         if (q.releaseId)
             fields.push({ column: 'releaseId', value: Number.parseInt(q.releaseId, 10) });
-        const body = this.searchBody(fields);
-        const raw = await this.http.postJson('/pm/sprint/search', body);
-        const rows = pickArray(raw, 'data', 'content', 'sprints');
-        return withTotal(rows.map(toSprintSummary), raw);
+        const PAGE = 100;
+        const SAFETY_CAP = 5000;
+        const rows = [];
+        let totalCount;
+        let startIndex = 0;
+        for (;;) {
+            const body = this.searchBody(fields, { startIndex, limit: PAGE });
+            const raw = await this.http.postJson('/pm/sprint/search', body);
+            const page = pickArray(raw, 'data', 'content', 'sprints');
+            if (totalCount === undefined) {
+                const t = raw?.totalCount ?? raw?.total ?? raw?.totalRecords;
+                if (typeof t === 'number')
+                    totalCount = t;
+            }
+            rows.push(...page);
+            if (page.length < PAGE)
+                break;
+            if (rows.length >= SAFETY_CAP)
+                break;
+            if (totalCount !== undefined && rows.length >= totalCount)
+                break;
+            startIndex += PAGE;
+        }
+        const mapped = rows.map(toSprintSummary);
+        if (totalCount !== undefined)
+            mapped.totalCount = totalCount;
+        return mapped;
     }
     /** POST /pm/users/search — the customer's users (router mounted at `/pm/users`, plural — see
      *  project-management/express-app.js:181). Backend auto-scopes non-admin callers to users in
@@ -1285,7 +1376,7 @@ export class SimplifyQaAdapter {
         let doc = raw?.data ?? raw;
         if (Array.isArray(doc))
             doc = doc.length > 0 ? doc[0] : {};
-        const obj = toObject(doc && typeof doc === 'object' ? doc : {});
+        const obj = toObject(doc && typeof doc === 'object' ? doc : {}, this.cfg.baseUrl);
         if (opts?.enrichNames)
             await this.enrichObjectNames(obj);
         return obj;
@@ -2328,6 +2419,22 @@ export class SimplifyQaAdapter {
                     paramIdByName.set(name, id);
                 }
             }
+            // API steps keep their `{{name}}` tokens INSIDE the request object (url/headers/body/auth),
+            // not in the description — scan those too so the mapper can rewrite them to `{{<paramId>}}`
+            // and register parameterIds. Without this, an API request containing {{var}} saves unbound.
+            if (s.stepType === 'api') {
+                const apiObj = (s.api ?? s.sourceRaw?.['apiData']);
+                for (const name of extractApiParamNames(apiObj)) {
+                    // A purely-numeric token is an ALREADY-BOUND `{{<paramId>}}` (round-tripped from the
+                    // backend), not a parameter name — never create a parameter for it.
+                    if (/^\d+$/.test(name))
+                        continue;
+                    if (paramIdByName.has(name))
+                        continue;
+                    const id = await this.getOrCreateParameter(projectId, moduleId, name, paramDefault(name, tc.data, undefined));
+                    paramIdByName.set(name, id);
+                }
+            }
         }
         return paramIdByName;
     }
@@ -2433,6 +2540,14 @@ export class SimplifyQaAdapter {
             parameterId: [],
             description: '',
         };
+        // Backend expects raw base64 PNG (no `data:` prefix) and stores it under /download/recording.
+        const rawImage = (v) => (v ?? '').replace(/^data:image\/\w+;base64,/i, '').trim();
+        const thumb = rawImage(objectRef.thumbnail_url);
+        const pageShot = rawImage(objectRef.pageScreenShot_url);
+        if (thumb)
+            body['thumbnail_url'] = thumb;
+        if (pageShot)
+            body['pageScreenShot_url'] = pageShot;
         const raw = await this.http.postJson('/tm/object/createObjectSeq', body);
         const flat = raw?.data ?? raw;
         const id = str(flat, 'id', '_id', 'objectId');
@@ -2457,7 +2572,11 @@ export class SimplifyQaAdapter {
             // ponytail: control-flow / function / switch / function_end steps have no action to resolve.
             // Push a null-actionId slot so index alignment survives; mapper checks and skips action/object
             // emit for these. Without this, IF/FOR/WHILE/FUNCTION steps throw "missing actionType" on save.
-            if (s.stepType === 'control_flow' || s.stepType === 'function' || s.stepType === 'function_end' || s.stepType === 'switch' || s.stepType === 'manual') {
+            // API steps are action-less too: Postman import and the SQA UI both persist them with
+            // actionId:null and identify them by stepType:'API' + apiData. Resolving an action name here
+            // would fail for a step that legitimately has none (e.g. the recorder's "API Call" is not a
+            // backend action name — the seeded one is executeRestAPI).
+            if (s.stepType === 'control_flow' || s.stepType === 'function' || s.stepType === 'function_end' || s.stepType === 'switch' || s.stepType === 'manual' || s.stepType === 'api') {
                 out.push({ actionId: null, objectId: null });
                 continue;
             }
@@ -2767,6 +2886,25 @@ export class SimplifyQaAdapter {
             doc = doc.length > 0 ? doc[0] : {};
         return doc && typeof doc === 'object' ? doc : {};
     }
+    /** Mirrors sqa-ui `buildExecutionPlatform` (suites.component.ts:779) so MCP-added TCs save with
+     *  the same per-row platform the UI would set — otherwise the suite trips the "select execution
+     *  platform" warning at run time. Special types self-identify; WEB falls back to Chrome so
+     *  suite_run doesn't send `platform: []`; mobile with no bound device stays empty (UI does same). */
+    buildExecutionPlatformForEntry(raw, tcType) {
+        const special = new Set(['API', 'MAINFRAME', 'SAP', 'DESKTOP', 'DATABASE']);
+        if (special.has(tcType)) {
+            const p = { id: tcType, name: tcType, technologyType: tcType, type: 'NONE' };
+            return { LOCAL: p, REMOTE: { ...p }, CLOUD: { ...p } };
+        }
+        const own = raw?.['executionPlatform'];
+        if (own && typeof own === 'object' && own.id) {
+            return { LOCAL: own, REMOTE: {}, CLOUD: {} };
+        }
+        if (tcType === 'WEB') {
+            return { LOCAL: { id: 'CHROME', name: 'Chrome', technologyType: 'WEB', type: 'NONE' }, REMOTE: {}, CLOUD: {} };
+        }
+        return { LOCAL: {}, REMOTE: {}, CLOUD: {} };
+    }
     /** Assemble the strict allow-listed suite entry for an ADD op. Required fields per the SQA
      *  validator: testcaseId, code, seq (set by the caller), version, versionId, tcType. */
     async buildSuiteEntryFromTestcase(tcNum) {
@@ -2789,7 +2927,7 @@ export class SimplifyQaAdapter {
             ...(raw?.projectId !== undefined ? { projectId: raw.projectId } : {}),
             version: versionName ?? 'v1.0',
             versionId: versionId ?? '',
-            executionPlatform: { LOCAL: {}, REMOTE: {}, CLOUD: {} },
+            executionPlatform: this.buildExecutionPlatformForEntry(raw, tcType),
         };
     }
     async resolveObjectTemplateId(technology) {
@@ -2833,7 +2971,7 @@ export class SimplifyQaAdapter {
         const body = this.searchBody(fields, { startIndex: q.offset ?? 0, limit: q.limit ?? 100 });
         const raw = await this.http.postJson('/tm/object/search', body);
         const rows = pickArray(raw, 'data', 'content', 'objects');
-        return rows.map(toObject);
+        return rows.map((r) => toObject(r, this.cfg.baseUrl));
     }
     /**
      * Create a new object — `POST /tm/object`, plain JSON (NOT multipart, unlike testcase/suite writes;
@@ -2882,7 +3020,7 @@ export class SimplifyQaAdapter {
         };
         const raw = await this.http.postJson('/tm/object', body);
         const doc = (raw?.data && typeof raw.data === 'object' ? raw.data : raw) ?? {};
-        return toObject(doc);
+        return toObject(doc, this.cfg.baseUrl);
     }
     /** The current project's module literally named "Default Module" (case-insensitive) — mirrors the
      *  real sqa-ui `object.component.ts` `openForm()` auto-selection. Best-effort: returns undefined
@@ -3019,7 +3157,7 @@ export class SimplifyQaAdapter {
         };
         const res = await this.http.putJson(`/tm/object/${numId}`, body);
         const updated = (res?.data && typeof res.data === 'object' ? res.data : res) ?? doc;
-        const obj = toObject(updated);
+        const obj = toObject(updated, this.cfg.baseUrl);
         if (opts?.enrichNames)
             await this.enrichObjectNames(obj);
         return obj;
@@ -3344,7 +3482,24 @@ export class SimplifyQaAdapter {
             executionId, executionRunId, parentId, projectId,
         });
         const rawSteps = Array.isArray(stepsResp?.data) ? stepsResp.data : [];
-        return rawSteps.map((s) => {
+        // Backend logs control-flow markers as INFO rows with `description: null` (see
+        // executionStepResult.service.js insertLoopIterationStart/insertFunctionEnd/… — the same
+        // executionRunId carries LOOP_ITERATION_START/END, FUNCTION_START/END, FUNCTION_ITERATION_*,
+        // WHILE_START/END alongside the real action rows). Left in, they fill the text response
+        // with blank `• step N:  (info)` lines that hide the actual failing step. Filter here so
+        // stepResults contains only debuggable rows.
+        const NOISE_STEP_TYPES = new Set([
+            'LOOP_ITERATION_START', 'LOOP_ITERATION_END',
+            'FUNCTION_START', 'FUNCTION_END',
+            'FUNCTION_ITERATION_START', 'FUNCTION_ITERATION_END',
+            'FOR_START', 'FOR_END', 'WHILE_START', 'WHILE_END',
+            'ITERATOR_START', 'ITERATOR_END', 'DO',
+        ]);
+        const filtered = rawSteps.filter((s) => {
+            const st = String(s?.stepType ?? '').toUpperCase();
+            return !NOISE_STEP_TYPES.has(st);
+        });
+        return filtered.map((s) => {
             const step = {
                 n: typeof s?.seq === 'number' ? s.seq : 0,
                 description: String(s?.description ?? ''),
@@ -3414,15 +3569,15 @@ export class SimplifyQaAdapter {
             reportUrl: `${base}/user/reports/reports-page?type=TESTCASE&parentId=${parentId}&id=${runIdNum}&projectId=${projectId}`,
         };
     }
-    /** Most recent execution runId for a SUITE — same `/exec/execution/search` the /reports UI's
-     *  current-executions view drives, filtered to `type=SUITE` + this suite's numeric id, sorted by
-     *  id desc, limit 1. Returns the numeric run id (what `getRun` accepts), or null if the suite has
-     *  never been executed. */
-    async getLatestSuiteRun(suiteId) {
+    /** Most recent execution runId for a SUITE or TC — same `/exec/execution/search` the /reports UI's
+     *  current-executions view drives, filtered by `type` + the entity's numeric id, sorted id desc,
+     *  limit 1. `childExecution:false` + the `parentExecutionCopy nor` clause together drop suite-child
+     *  TC rows, so a TC lookup returns its LATEST STANDALONE run, not the child spawned inside a suite. */
+    async getLatestRunFor(type, id) {
         const projectId = this.requireProjectId();
-        const suiteNum = Number.parseInt(numericId(suiteId), 10);
-        if (!Number.isFinite(suiteNum))
-            throw new Error(`getLatestSuiteRun: invalid suiteId ${suiteId}.`);
+        const num = Number.parseInt(numericId(id), 10);
+        if (!Number.isFinite(num))
+            throw new Error(`getLatestRunFor(${type}): invalid id ${id}.`);
         // Hand-rolled payload — searchBody doesn't emit `sort` or `operator` keys the /reports search wants.
         const body = {
             searchFields: [
@@ -3431,16 +3586,83 @@ export class SimplifyQaAdapter {
                 { column: 'projectId', value: projectId, regEx: false },
                 { column: 'childExecution', value: false },
                 { column: 'parentExecutionCopy', value: true, operator: 'nor' },
-                { column: 'type', value: 'SUITE', regEx: false },
-                { column: 'executionTypeId', value: suiteNum, regEx: false },
+                { column: 'type', value: type, regEx: false },
+                { column: 'executionTypeId', value: num, regEx: false },
             ],
             startIndex: 0,
             limit: 1,
         };
         const raw = await this.http.postJson('/exec/execution/search', body);
         const rows = pickArray(raw, 'data', 'content', 'executions');
-        const runId = str(rows[0], 'id', '_id');
-        return runId ?? null;
+        return str(rows[0], 'id', '_id') ?? null;
+    }
+    async getLatestSuiteRun(suiteId) { return this.getLatestRunFor('SUITE', suiteId); }
+    async getLatestTestcaseRun(tcId) { return this.getLatestRunFor('TC', tcId); }
+    /** Top-N recent runs — one HTTP call, same standalone-only filters as `getLatestRunFor`. For each
+     *  SUITE row the exec doc's `executionItems[]` already carries per-TC status, so the tally is
+     *  computed inline (no extra call per run). TC rows omit the tally counters. Sorted newest-first
+     *  by `id` (execution ids are monotonic). Bounded — callers should cap `limit` at the tool layer
+     *  so a runaway request can't pull thousands of rows. */
+    async listRecentRuns(type, id, limit) {
+        const projectId = this.requireProjectId();
+        const num = Number.parseInt(numericId(id), 10);
+        if (!Number.isFinite(num))
+            throw new Error(`listRecentRuns(${type}): invalid id ${id}.`);
+        const base = (this.cfg.baseUrl ?? '').replace(/\/+$/, '');
+        const body = {
+            searchFields: [
+                { column: 'id', sort: 'dsc' },
+                { column: 'deleted', value: false },
+                { column: 'projectId', value: projectId, regEx: false },
+                { column: 'childExecution', value: false },
+                { column: 'parentExecutionCopy', value: true, operator: 'nor' },
+                { column: 'type', value: type, regEx: false },
+                { column: 'executionTypeId', value: num, regEx: false },
+            ],
+            startIndex: 0,
+            limit,
+        };
+        const raw = await this.http.postJson('/exec/execution/search', body);
+        const rows = pickArray(raw, 'data', 'content', 'executions');
+        return rows.map((r) => {
+            const runId = str(r, 'id', '_id') ?? '';
+            const runIdNum = Number.parseInt(runId, 10);
+            const summary = {
+                runId,
+                state: mapRunState(str(r, 'status') ?? ''),
+            };
+            const startedAt = str(r, 'startTime', 'createdAt');
+            const endedAt = str(r, 'endTime');
+            if (startedAt)
+                summary.startedAt = startedAt;
+            if (endedAt)
+                summary.endedAt = endedAt;
+            if (Number.isFinite(runIdNum)) {
+                const entityNum = Number(r?.executionTypeId ?? num);
+                summary.reportUrl = `${base}/user/reports/reports-page?type=${type === 'SUITE' ? 'SUITE' : 'TESTCASE'}&parentId=${entityNum}&id=${runIdNum}&projectId=${projectId}`;
+            }
+            if (type === 'SUITE') {
+                const items = Array.isArray(r?.executionItems) ? r.executionItems : [];
+                let passed = 0, failed = 0, skipped = 0, notExecuted = 0;
+                for (const it of items) {
+                    const s = String(it?.status ?? '').toUpperCase();
+                    if (s === 'PASSED')
+                        passed++;
+                    else if (s === 'FAILED')
+                        failed++;
+                    else if (s === 'SKIPPED')
+                        skipped++;
+                    else if (s === 'NOTEXECUTED')
+                        notExecuted++;
+                }
+                summary.totalTestCases = items.length;
+                summary.passed = passed;
+                summary.failed = failed;
+                summary.skipped = skipped;
+                summary.notExecuted = notExecuted;
+            }
+            return summary;
+        });
     }
     /** Targeted name lookup for a single project by numeric id — the whole-list `listProjects`
      *  is paged (default 100) and API-key scoping can hide the selected project, which left
@@ -3635,11 +3857,15 @@ export class SimplifyQaAdapter {
         const executionId = Number.parseInt(numericId(runId), 10);
         if (!Number.isFinite(executionId))
             throw new Error(`stopRun: invalid runId ${runId}.`);
-        await this.http.postJson('/exec/execution/stopExecution', {
-            projectId,
-            executionId,
-            killType: 'STOP_EXECUTION',
-        });
+        // Mirror sqa-ui reports.component.ts:2041-2054: cloud-parallel needs a dedicated killType so
+        // the backend fans the Redis kill out to every child agent (executions/…/execution.service.js
+        // :4610-4657). Plain STOP_EXECUTION only pushes to the parent's agentId, leaving cloud-parallel
+        // child VMs running even though their DB rows read FAILED.
+        const doc = await this.fetchExecDoc(executionId, projectId);
+        const mode = String(doc?.mode ?? '').toUpperCase();
+        const style = String(doc?.executionStyle ?? '').toUpperCase();
+        const killType = mode === 'CLOUD' && style === 'PARALLEL' ? 'STOP_EXECUTION_PARALLEL_CLOUD' : 'STOP_EXECUTION';
+        await this.http.postJson('/exec/execution/stopExecution', { projectId, executionId, killType });
     }
     /** SUITE dispatch. Mirrors `startRun` above (which handles TC) — same `/exec/execution` endpoint,
      *  same LOCAL-agent fast-fail, same project-name / environment resolution. What differs is on the