@tea-agent/loop-agent 0.25.1 → 0.25.2

package/CHANGELOG.md

@@ -2,6 +2,22 @@
 
 ## [Unreleased]
 
+## [0.25.2] - 2026-07-30
+
+### 重点更新
+
+- 加固后端测试中文 HTML 报告的用例关联与解析逻辑，提升报告的准确性与可读性
+
+### 改进
+
+- 改进基于 class 的 pytest node id 解析，准确提取真实脚本路径，避免显示为 unknown.py
+- 当用例名称缺失后端 ID 时，支持按函数名与 Markdown catalog 进行回退关联
+- 规范化 [REQ]/[RESP] HTTP 日志块，增强日志兼容性
+
+### 修复
+
+- 修复成功执行路径下原生 pytest-html 报告未正确保留的问题，现会将其另存为 backend-test-pytest-native.html
+
 ## [0.25.1] - 2026-07-30
 
 ### 重点更新

package/dist/executors/shell-executor.js

@@ -587,11 +587,13 @@ async function executeBackendTestPipeline(input, meta) {
                 throw new Error(`pytest did not complete with a reportable exit code: ${pytestExitCode}`);
             }
             const parsed = parsePytestHtmlReport(pytestHtmlContent);
+            // Keep native pytest-html (data-jsonblob) for audit before styled overwrite.
+            const nativeHtmlPath = await writeRunReport(meta.runDir, "backend-test-pytest-native.html", pytestHtmlContent);
             // Bind Result v1 from the native pytest-html report BEFORE overwriting with the
             // styled renderer (which drops the data-jsonblob island).
             const resultArtifact = await materializeBackendTestResultFromPytestHtml({
                 runDir: meta.runDir,
-                htmlRelativePath: "reports/backend-test.html",
+                htmlRelativePath: "reports/backend-test-pytest-native.html",
                 htmlContent: pytestHtmlContent,
                 pytestExitCode,
             });
@@ -682,7 +684,7 @@ async function executeBackendTestPipeline(input, meta) {
             });
             const l5Path = await writeRunReport(meta.runDir, "backend-test-l5-dashboard.html", l5Html);
             const sanitizedOutputs = results.map((result) => redactBackendTestOutput(result.stdout));
-            outputs.push(...sanitizedOutputs, `html=${htmlPath}`, `markdown=${markdownPath}`, `facts=${factsPath}`, `l5-dashboard=${l5Path}`, `result=${resultArtifact.path}`, facts);
+            outputs.push(...sanitizedOutputs, `html=${htmlPath}`, `nativeHtml=${nativeHtmlPath}`, `markdown=${markdownPath}`, `facts=${factsPath}`, `l5-dashboard=${l5Path}`, `result=${resultArtifact.path}`, facts);
         }
         else if (pipeline === "contracts") {
             const wrapperPath = path.join(meta.runDir, "analyze-and-discover-backend-test-pi.json");

package/dist/workflows/dag/backend-test-markdown-workflow.js

@@ -774,15 +774,57 @@ export async function collectBackendTestHumanCaseCatalog(workspaceRoot) {
 function junitCaseId(name) {
     return symbolCaseId(name) ?? name.match(CASE_ID)?.[0];
 }
+/** Prefer function-name Case ID; fall back to first BE-* in free text (docstring/log). */
+function extractCaseIdFromText(value) {
+    if (!value?.trim())
+        return undefined;
+    return junitCaseId(value) ?? value.match(CASE_ID)?.[0];
+}
+function resolveReportCaseId(result, catalog) {
+    const fromName = extractCaseIdFromText(result.name);
+    if (fromName && catalog.has(fromName))
+        return fromName;
+    if (fromName)
+        return fromName;
+    // Match catalog by pytest function/method name when Case ID is only in docstring/md.
+    const byFunction = [...catalog.values()].find((item) => item.testFunctions.includes(result.name));
+    if (byFunction)
+        return byFunction.id;
+    const fromStdout = extractCaseIdFromText(result.stdout);
+    if (fromStdout && catalog.has(fromStdout))
+        return fromStdout;
+    if (fromStdout)
+        return fromStdout;
+    const fromDetails = extractCaseIdFromText(result.details);
+    if (fromDetails && catalog.has(fromDetails))
+        return fromDetails;
+    return "未关联";
+}
 function humanStatus(status) {
     return status === "passed" ? "通过" : status === "failure" ? "失败" : status === "error" ? "错误" : "跳过";
 }
 function formatDuration(durationMs) {
     return durationMs === undefined ? "未记录" : `${(durationMs / 1000).toFixed(3)} 秒`;
 }
-function inferredScriptPath(classname) {
-    const moduleName = classname.split(".").filter((part) => part && !/^Test/.test(part)).join("/");
-    return `${moduleName || "unknown"}.py`;
+function inferredScriptPath(classname, filePath) {
+    if (filePath?.trim()) {
+        return filePath.replaceAll("\\", "/").replace(/^\.\//, "");
+    }
+    const moduleName = classname
+        .split(".")
+        .filter((part) => part && !/^Test[A-Z_]/.test(part) && part !== "Test")
+        .join("/");
+    if (!moduleName || moduleName === "unknown")
+        return "unknown.py";
+    return moduleName.endsWith(".py") ? moduleName : `${moduleName}.py`;
+}
+function resolveReportScriptPath(result, item) {
+    if (item?.scriptPath)
+        return item.scriptPath;
+    if (result.filePath?.trim()) {
+        return result.filePath.replaceAll("\\", "/").replace(/^\.\//, "");
+    }
+    return inferredScriptPath(result.classname, result.filePath);
 }
 export function redactBackendTestOutput(value) {
     return value
@@ -815,8 +857,97 @@ function reportOutputLines(value, prefix) {
         return redactBackendTestOutput(trimmed.trim()).slice(0, 4000);
     });
 }
+/**
+ * Normalize alternate HTTP log dialects into HTTP_REQUEST / HTTP_RESPONSE lines.
+ * Supports common project helpers that emit [REQ]/[RESP] block style instead of
+ * the contract prefixes (keeps original HTTP_* lines unchanged).
+ */
+export function normalizeBackendTestHttpLogText(value) {
+    if (!value.trim())
+        return value;
+    const lines = value.split(/\r?\n/);
+    const out = [];
+    let i = 0;
+    while (i < lines.length) {
+        const raw = lines[i];
+        const line = raw.trim();
+        // Already-contract lines: pass through.
+        if (/^(HTTP_REQUEST|HTTP REQUEST|HTTP_RESPONSE|HTTP RESPONSE)\b/i.test(line)) {
+            out.push(raw);
+            i += 1;
+            continue;
+        }
+        // [REQ] METHOD path  OR  [REQ] METHOD url
+        const reqHead = line.match(/^\[REQ\]\s+([A-Z]+)\s+(\S+)\s*$/i);
+        if (reqHead) {
+            const method = reqHead[1].toUpperCase();
+            const url = reqHead[2];
+            let body = "";
+            let j = i + 1;
+            // Optional [REQ body] then JSON/text until separator or next tag.
+            if (j < lines.length && /^\[REQ\s*body\]/i.test(lines[j].trim())) {
+                j += 1;
+                const bodyLines = [];
+                while (j < lines.length &&
+                    !/^[=-]{3,}\s*$/.test(lines[j].trim()) &&
+                    !/^\[(?:REQ|RESP)/i.test(lines[j].trim()) &&
+                    !/^(HTTP_REQUEST|HTTP_RESPONSE|HTTP REQUEST|HTTP RESPONSE)\b/i.test(lines[j].trim())) {
+                    bodyLines.push(lines[j]);
+                    j += 1;
+                }
+                body = bodyLines.join("\n").trim();
+            }
+            const payload = { method, url };
+            if (body) {
+                try {
+                    payload.json = JSON.parse(body);
+                }
+                catch {
+                    payload.body = body.slice(0, 4000);
+                }
+            }
+            out.push(`HTTP_REQUEST ${JSON.stringify(payload)}`);
+            i = j;
+            continue;
+        }
+        // [RESP status] 200  OR  [RESP] 200
+        const respHead = line.match(/^\[RESP(?:\s+status)?\]\s+(\d{3})\b/i);
+        if (respHead) {
+            const status_code = Number(respHead[1]);
+            let body = "";
+            let j = i + 1;
+            if (j < lines.length && /^\[RESP\s*body\]/i.test(lines[j].trim())) {
+                j += 1;
+                const bodyLines = [];
+                while (j < lines.length &&
+                    !/^[=-]{3,}\s*$/.test(lines[j].trim()) &&
+                    !/^\[(?:REQ|RESP)/i.test(lines[j].trim()) &&
+                    !/^(HTTP_REQUEST|HTTP_RESPONSE|HTTP REQUEST|HTTP RESPONSE)\b/i.test(lines[j].trim())) {
+                    bodyLines.push(lines[j]);
+                    j += 1;
+                }
+                body = bodyLines.join("\n").trim();
+            }
+            const payload = { status_code };
+            if (body) {
+                try {
+                    payload.result = JSON.parse(body);
+                }
+                catch {
+                    payload.result = body.slice(0, 4000);
+                }
+            }
+            out.push(`HTTP_RESPONSE ${JSON.stringify(payload)}`);
+            i = j;
+            continue;
+        }
+        out.push(raw);
+        i += 1;
+    }
+    return out.join("\n");
+}
 function requestResponseSummary(result) {
-    const combined = [result.stdout, result.stderr].filter(Boolean).join("\n");
+    const combined = normalizeBackendTestHttpLogText([result.stdout, result.stderr].filter(Boolean).join("\n"));
     return {
         requests: reportOutputLines(combined, "HTTP_REQUEST"),
         responses: reportOutputLines(combined, "HTTP_RESPONSE"),
@@ -1053,8 +1184,8 @@ export function renderBackendTestHtml(input) {
     const failures = input.parsed.cases.filter((result) => result.status !== "passed");
     const headColor = failed ? { fg: "#b42318", bg: "linear-gradient(135deg,#fef3f2 0,#fee4e2 100%)", border: "#fda29b", icon: "✗" } : { fg: "#067647", bg: "linear-gradient(135deg,#ecfdf3 0,#d1fadf 100%)", border: "#abefc6", icon: "✓" };
     const caseCards = input.parsed.cases.map((result) => {
-        const caseId = junitCaseId(result.name) ?? "未关联";
-        const item = catalog.get(caseId);
+        const caseId = resolveReportCaseId(result, catalog);
+        const item = caseId === "未关联" ? undefined : catalog.get(caseId);
         const io = requestResponseSummary(result);
         const calls = pairHttpCalls(io.requests, io.responses);
         const statusColors = statusColor(result.status);
@@ -1064,7 +1195,7 @@ export function renderBackendTestHtml(input) {
         const failureBlock = result.status === "passed"
             ? ""
             : `<div style="margin-top:12px;padding:10px 12px;border-radius:8px;background:#fef3f2;border:1px solid #fda29b;color:#b42318;font-size:0.84rem"><strong>${escapeHtml(result.message || humanStatus(result.status))}</strong>${result.details ? `<details style="margin-top:6px"><summary style="cursor:pointer;color:#b42318;font-size:0.78rem">失败详情</summary><pre style="white-space:pre-wrap;background:#f6f8fa;color:#1f2937;padding:10px 12px;border-radius:8px;font-family:ui-monospace,SFMono-Regular,Menlo,Consolas,monospace;font-size:0.78rem;margin-top:6px;border:1px solid #eceef2;line-height:1.6">${escapeHtml(result.details)}</pre></details>` : ""}</div>`;
-        const scriptPath = item?.scriptPath ?? inferredScriptPath(result.classname);
+        const scriptPath = resolveReportScriptPath(result, item);
         return `<details style="background:#fff;border:1px solid #e6eaf0;border-radius:12px;margin-top:10px"><summary style="display:flex;align-items:center;gap:12px;padding:12px 16px;cursor:pointer;list-style:none"><span style="background:#e8edf5;color:#2b3a55;padding:2px 9px;border-radius:6px;font-family:ui-monospace,SFMono-Regular,Menlo,Consolas,monospace;font-size:0.76rem;font-weight:600;white-space:nowrap">${escapeHtml(caseId)}</span><div style="flex:1;min-width:0"><div style="font-weight:600;color:#172033;font-size:0.92rem">${escapeHtml(item?.title ?? result.name)}</div><code style="font-family:ui-monospace,SFMono-Regular,Menlo,Consolas,monospace;font-size:0.74rem;color:#98a2b3">${escapeHtml(result.name)}</code></div><span style="color:#98a2b3;font-size:0.8rem;white-space:nowrap">${formatDuration(result.durationMs)}</span><span style="display:inline-block;padding:2px 10px;border-radius:999px;font-weight:700;font-size:0.72rem;color:${statusColors.fg};background:${statusColors.bg}">${humanStatus(result.status)}</span><span style="color:#aab2bf;font-size:0.8rem">▾</span></summary><div style="padding:4px 16px 14px 16px;border-top:1px solid #f0f3f7"><div style="margin-top:10px"><div style="color:#667085;font-size:0.74rem;font-weight:600;margin-bottom:3px">自动化脚本</div><code style="background:#eef2f7;color:#344054;padding:1px 6px;border-radius:5px;font-family:ui-monospace,SFMono-Regular,Menlo,Consolas,monospace;font-size:0.8rem">${escapeHtml(scriptPath)}</code></div>${item?.scenario ? `<div style="margin-top:12px"><div style="color:#667085;font-size:0.74rem;font-weight:600;margin-bottom:3px">验收场景</div><div style="color:#475467;font-size:0.86rem;line-height:1.75">${escapeHtml(item.scenario)}</div></div>` : ""}${logSection}${failureBlock}</div></details>`;
     }).join("");
     const metricCard = (label, value, valueColor = "#172033") => `<div style="padding:14px 16px;border:1px solid #e3e8ef;border-radius:12px;background:#fbfcfe"><span style="color:#667085;font-size:0.78rem">${escapeHtml(label)}</span><div style="font-size:22px;font-weight:700;color:${valueColor}">${value}</div></div>`;
@@ -1073,7 +1204,7 @@ export function renderBackendTestHtml(input) {
     const qualityBlock = `<div style="margin-top:18px"><div style="color:#17365d;font-size:16px;font-weight:700;margin:0 4px 6px">质量校验</div><div style="display:grid;grid-template-columns:repeat(auto-fit,minmax(240px,1fr));gap:12px;margin-top:8px">${renderQualityCard("Markdown 用例校验", caseValidation)}${renderQualityCard("Markdown → pytest 追溯", traceability)}</div></div>`;
     const failureOverview = failures.length > 0
         ? `<div style="margin-top:18px"><div style="color:#17365d;font-size:16px;font-weight:700;margin:0 4px 6px">失败概览</div><div style="display:grid;gap:10px;margin-top:8px">${failures.map((result) => {
-            const caseId = junitCaseId(result.name) ?? "未关联";
+            const caseId = resolveReportCaseId(result, catalog);
             const c = statusColor(result.status);
             return `<div style="border-left:4px solid ${c.fg};background:${c.bg};padding:10px 12px;border-radius:8px"><div style="font-weight:600;color:#172033;font-size:0.88rem">${escapeHtml(caseId)} · ${escapeHtml(catalog.get(caseId)?.title ?? result.name)}</div><div style="color:${c.fg};font-size:0.82rem;margin-top:2px">${escapeHtml(result.message || humanStatus(result.status))}</div></div>`;
         }).join("")}</div></div>`
@@ -1342,9 +1473,9 @@ export function renderBackendTestFacts(input) {
         "| 用例编号 | 用例名称 | 自动化脚本 | 测试函数 | 结果 | 耗时 | 失败原因 |",
         "|---|---|---|---|---|---:|---|",
         ...input.parsed.cases.map((result) => {
-            const caseId = junitCaseId(result.name) ?? "未关联";
-            const item = catalog.get(caseId);
-            const script = item?.scriptPath ?? inferredScriptPath(result.classname);
+            const caseId = resolveReportCaseId(result, catalog);
+            const item = caseId === "未关联" ? undefined : catalog.get(caseId);
+            const script = resolveReportScriptPath(result, item);
             return `| ${caseId} | ${item?.title ?? result.name} | \`${script}\` | \`${result.name}\` | ${humanStatus(result.status)} | ${formatDuration(result.durationMs)} | ${(result.message ?? "—").replaceAll("|", "\\|")} |`;
         }),
         "",

package/dist/workflows/dag/backend-test-result-contract.js

@@ -428,7 +428,8 @@ export function parsePytestHtmlReport(html) {
             continue;
         const rawResult = (record.result ?? "").toLowerCase();
         const testId = record.testId ?? nodeId;
-        const { classname, name } = splitPytestNodeId(testId);
+        const { classname, name, filePath } = splitPytestNodeId(testId);
+        const filePathFields = filePath ? { filePath } : {};
         const durationMs = parseDurationLabelMs(record.duration);
         const capturedLog = record.log ?? "";
         // pytest-html collapses captured stdout/stderr into a single `log` field
@@ -441,6 +442,8 @@ export function parsePytestHtmlReport(html) {
             cases.push({
                 classname,
                 name,
+                ...filePathFields,
+                ...filePathFields,
                 durationMs,
                 status: "passed",
                 ...(stdout ? { stdout } : {}),
@@ -455,6 +458,7 @@ export function parsePytestHtmlReport(html) {
             cases.push({
                 classname,
                 name,
+                ...filePathFields,
                 durationMs,
                 status: "failure",
                 message: summary,
@@ -471,6 +475,7 @@ export function parsePytestHtmlReport(html) {
             cases.push({
                 classname,
                 name,
+                ...filePathFields,
                 durationMs,
                 status: "error",
                 message: summary,
@@ -484,6 +489,7 @@ export function parsePytestHtmlReport(html) {
             cases.push({
                 classname,
                 name,
+                ...filePathFields,
                 durationMs,
                 status: "skipped",
                 ...(stdout ? { stdout } : {}),
@@ -499,6 +505,7 @@ export function parsePytestHtmlReport(html) {
             cases.push({
                 classname,
                 name,
+                ...filePathFields,
                 durationMs,
                 status: "error",
                 message: summary,
@@ -520,20 +527,44 @@ export function parsePytestHtmlReport(html) {
         failures: failures.slice(0, MAX_FAILURES),
     };
 }
-function splitPytestNodeId(testId) {
-    // pytest-html uses node ids like "path/to/test_x.py::TestClass::test_name".
-    const sepIndex = testId.lastIndexOf("::");
-    if (sepIndex < 0) {
-        return { classname: "unknown", name: testId };
-    }
-    const prefix = testId.slice(0, sepIndex);
-    const name = testId.slice(sepIndex + 2);
-    const lastParen = name.indexOf("(");
-    const cleanName = lastParen >= 0 ? name.slice(0, lastParen) : name;
-    const moduleSep = prefix.lastIndexOf("::");
-    const filePart = moduleSep >= 0 ? prefix.slice(moduleSep + 2) : prefix;
-    const moduleWithoutExt = filePart.replace(/\.py$/i, "");
-    return { classname: moduleWithoutExt.replace(/\//g, ".") || "unknown", name: cleanName };
+/**
+ * Split a pytest node id into module classname, test name, and optional file path.
+ *
+ * Handles:
+ * - `testcase/test_x.py::test_name`
+ * - `testcase/test_x.py::TestClass::test_name` (class-based; previous code
+ *   treated `TestClass` as the file segment and later filtered it to unknown.py)
+ * - Windows separators and parametrized names `test_name[param]`
+ */
+export function splitPytestNodeId(testId) {
+    const normalized = testId.replaceAll("\\", "/").trim();
+    if (!normalized) {
+        return { classname: "unknown", name: "unknown" };
+    }
+    const parts = normalized.split("::").filter(Boolean);
+    if (parts.length < 2) {
+        // Bare id without "::" — keep as name; path unknown.
+        return { classname: "unknown", name: normalized };
+    }
+    let rawName = parts[parts.length - 1];
+    // Drop call args if present; keep parametrize brackets in display name strip for id match.
+    const paren = rawName.indexOf("(");
+    if (paren >= 0)
+        rawName = rawName.slice(0, paren);
+    const cleanName = rawName.trim() || "unknown";
+    // Prefer the first segment that looks like a .py file path.
+    const filePart = parts.find((part) => /\.py$/i.test(part)) ??
+        (parts[0].includes("/") || parts[0].includes(".") ? parts[0] : undefined);
+    const filePath = filePart
+        ? filePart.replace(/\\/g, "/")
+        : undefined;
+    const moduleWithoutExt = (filePath ?? "unknown").replace(/\.py$/i, "");
+    const classname = moduleWithoutExt.replaceAll("/", ".") || "unknown";
+    return {
+        classname,
+        name: cleanName,
+        ...(filePath ? { filePath } : {}),
+    };
 }
 function splitPytestHtmlLogSections(log) {
     if (!log)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tea-agent/loop-agent",
-  "version": "0.25.1",
+  "version": "0.25.2",
   "type": "module",
   "bin": {
     "loop-agent": "bin/loop-agent.js",