@kkeve/suite 0.0.23 → 0.0.24

package/dist/dsl/toDecorator.js

@@ -160,7 +160,17 @@ function renderStep(d, ctx) {
         const executableLineCount = [...body, ...expLines].length;
         // P1 纵深防御：断言里若残留未解析占位符，说明解析链路有漏（正常已在
         // stringExpr 处编译期报错），一律不许短路 —— 自证式断言不能当作确定性事实。
-        const hasUnresolved = [...body, ...expLines].some((b) => /\$\{[^}]+\}/.test(b));
+        // stringExpr 对已知合法引用生成 ${String(X)} 模板，运行时变量在这里不是
+        // 「未解析占位符」，不应阻断确定性短路。其余残留 ${...} 仍是解析漏洞。
+        const hasUnresolved = [...body, ...expLines].some((b) => {
+            for (const m of String(b).matchAll(/\$\{([^}]+)\}/g)) {
+                const raw = String(m[1]).trim();
+                const name = /^String\(\s*([A-Za-z_$][A-Za-z0-9_$]*)\s*\)$/.exec(raw)?.[1] || raw;
+                if (!ctx.runtimeVars.has(name))
+                    return true;
+            }
+            return false;
+        });
         const deterministic = expectedFullyCovered && !pageLevelOp && executableLineCount > 0 && !hasUnresolved;
         out.push(`await keveGoal({ step: ${q(stepText)}, expected: ${q(expected)}${deterministic ? ', deterministic: true' : ''} }, async () => {`);
         for (const b of [...body, ...expLines])
@@ -285,7 +295,7 @@ function renderStep(d, ctx) {
                     : undefined);
             const targetCtx = valueCtx || d.child?.context || d.child;
             if (Array.isArray(targetCtx) && targetCtx.length) {
-                emit([`await engine.drag(${chain}, ${chainExpr(targetCtx, ctx)});`, ...waitExprs(d)]);
+                emit([`await engine.drag(${chainExpr(normalizeDragSource(d.context), ctx)}, ${chainExpr(targetCtx, ctx)});`, ...waitExprs(d)]);
             }
             else {
                 throw new Error(`DRAGGING 缺少目标元素，拒绝退化为悬停: ${stepText}`);
@@ -1063,6 +1073,23 @@ function collectHops(context) {
     }
     return out;
 }
+/**
+ * 拖拽源链归一化：录制器常见的「父容器 → 文本 → 列表项」链会把列表项限制在
+ * 文本叶节点后代内，导致确定性定位落空。同用例里可稳定命中的点击链是
+ * 「列表项 → 文本/includes」，拖拽源也应优先使用该形态。
+ */
+function normalizeDragSource(context) {
+    const hops = Array.isArray(context) ? context : [];
+    if (hops.length < 2)
+        return context;
+    const prev = hops[hops.length - 2];
+    const tail = hops[hops.length - 1];
+    const isListItemTail = String(tail?.value ?? '') === '[data-e2e-name="列表项"]';
+    const isTextHop = String(prev?.type || '') === 'includes' || String(prev?.type || '') === 'text';
+    if (isListItemTail && isTextHop)
+        return [tail, prev];
+    return context;
+}
 /** 截图基线 key 兜底：步骤文案 → 稳定 key（同步骤重复生成不变） */
 function sanitizeShotKey(text) {
     return String(text || 'shot').replace(/[^a-zA-Z0-9\u4e00-\u9fff]/g, '_').slice(0, 40);

package/dist/page-agent/tools.js

@@ -158,7 +158,9 @@ exports.tools.set('done', tool({
     execute: async function (input) {
         const icons = { pass: '✅', fail: '❌', blocked: '🚫' };
         let verdict = input.verdict;
-        // 强制校验：verdict 缺失或非法时从 text 推断（MacroTool 扁平化导致 verdict 变 optional）
+        // 强制校验：verdict 缺失或非法时从 text 推断（MacroTool 扁平化导致 verdict 变 optional）。
+        // verdict 与可推断文本都缺失时不能默认 pass —— Agent 已把动作做完但没给出结论，
+        // 交给上层按保守语义判定，而不是在工具层制造假绿。
         if (!icons[verdict]) {
             console.warn(`[done] ⚠️ verdict missing or invalid ("${input.verdict}"), inferring from text`);
             const t = String(input.text || '').toLowerCase();
@@ -168,9 +170,12 @@ exports.tools.set('done', tool({
             else if (t.includes('blocked') || t.includes('阻塞') || t.includes('阻止')) {
                 verdict = 'blocked';
             }
-            else {
+            else if (t.length > 0) {
                 verdict = 'pass'; // 有 text 且无否定词 → 默认 pass
             }
+            else {
+                throw new Error('done 缺少 verdict 且没有可推断的 text，不能默认判定通过');
+            }
         }
         return `${icons[verdict]} Task ${verdict}: ${input.text}`;
     },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kkeve/suite",
-  "version": "0.0.23",
+  "version": "0.0.24",
   "description": "AI-powered full-stack test agent CLI - static analysis, unit test, component test, E2E test, execute, evaluate, and report",
   "license": "MIT",
   "bin": {