@agent-scope/cli 1.20.3 → 1.20.4

package/dist/cli.js

@@ -2333,6 +2333,7 @@ function createManifestCommand() {
 
 // src/instrument/hooks.ts
 import { resolve as resolve5 } from "path";
+import { serialize } from "@agent-scope/core";
 import { getBrowserEntryScript as getBrowserEntryScript2 } from "@agent-scope/playwright";
 import { Command as Cmd } from "commander";
 import { chromium as chromium2 } from "playwright";
@@ -2374,26 +2375,7 @@ function buildHookInstrumentationScript() {
   // Walk fiber tree and collect hook data per component
   var components = [];
 
-  function serializeValue(v) {
-    if (v === null) return null;
-    if (v === undefined) return undefined;
-    var t = typeof v;
-    if (t === "string" || t === "number" || t === "boolean") return v;
-    if (t === "function") return "[function " + (v.name || "anonymous") + "]";
-    if (Array.isArray(v)) {
-      try { return v.map(serializeValue); } catch(e) { return "[Array]"; }
-    }
-    if (t === "object") {
-      try {
-        var keys = Object.keys(v).slice(0, 5);
-        var out = {};
-        for (var k of keys) { out[k] = serializeValue(v[k]); }
-        if (Object.keys(v).length > 5) out["..."] = "(truncated)";
-        return out;
-      } catch(e) { return "[Object]"; }
-    }
-    return String(v);
-  }
+  var serializeValue = __SCOPE_SERIALIZE_VALUE__;
 
   // Hook node classifiers (mirrors hooks-extractor.ts logic)
   var HookLayout = 0b00100;
@@ -2614,7 +2596,10 @@ async function runHooksProfiling(componentName, filePath, props) {
     if (renderError !== null) {
       throw new Error(`Component render error: ${renderError}`);
     }
-    const instrumentScript = buildHookInstrumentationScript();
+    const instrumentScript = buildHookInstrumentationScript().replace(
+      "__SCOPE_SERIALIZE_VALUE__",
+      serialize.toString()
+    );
     const raw = await page.evaluate(instrumentScript);
     const result = raw;
     if (result.error) {
@@ -2816,44 +2801,79 @@ function buildProfilingCollectScript() {
 `;
 }
 async function replayInteraction(page, steps) {
-  for (const step of steps) {
+  let executed = 0;
+  const skippedInteractions = [];
+  const skip = (index, step, reason) => {
+    skippedInteractions.push({ index, action: step.action, target: step.target, reason });
+  };
+  for (const [index, step] of steps.entries()) {
     switch (step.action) {
       case "click":
-        if (step.target) {
-          await page.click(step.target, { timeout: 5e3 }).catch(() => {
-            process.stderr.write(`  \u26A0 click target "${step.target}" not found, skipping
+        if (!step.target) {
+          skip(index, step, "missing_target");
+          break;
+        }
+        try {
+          await page.click(step.target, { timeout: 5e3 });
+          executed++;
+        } catch {
+          process.stderr.write(`  \u26A0 click target "${step.target}" not found, skipping
 `);
-          });
+          skip(index, step, "not_found");
         }
         break;
       case "fill":
-        if (step.target && step.value !== void 0) {
-          await page.fill(step.target, step.value, { timeout: 5e3 }).catch(() => {
-            process.stderr.write(`  \u26A0 fill target "${step.target}" not found, skipping
+        if (!step.target) {
+          skip(index, step, "missing_target");
+          break;
+        }
+        if (step.value === void 0) {
+          skip(index, step, "missing_value");
+          break;
+        }
+        try {
+          await page.fill(step.target, step.value, { timeout: 5e3 });
+          executed++;
+        } catch {
+          process.stderr.write(`  \u26A0 fill target "${step.target}" not found, skipping
 `);
-          });
+          skip(index, step, "not_found");
         }
         break;
       case "hover":
-        if (step.target) {
-          await page.hover(step.target, { timeout: 5e3 }).catch(() => {
-            process.stderr.write(`  \u26A0 hover target "${step.target}" not found, skipping
+        if (!step.target) {
+          skip(index, step, "missing_target");
+          break;
+        }
+        try {
+          await page.hover(step.target, { timeout: 5e3 });
+          executed++;
+        } catch {
+          process.stderr.write(`  \u26A0 hover target "${step.target}" not found, skipping
 `);
-          });
+          skip(index, step, "not_found");
         }
         break;
       case "press":
-        if (step.value) {
-          await page.keyboard.press(step.value);
+        if (!step.value) {
+          skip(index, step, "missing_value");
+          break;
         }
+        await page.keyboard.press(step.value);
+        executed++;
         break;
       case "wait":
         await page.waitForTimeout(step.delay ?? 500);
+        executed++;
+        break;
+      default:
+        skip(index, step, "unsupported");
         break;
     }
   }
+  return { executed, skippedInteractions };
 }
-function analyzeProfileFlags(totalRenders, wastedRenders, timing, layoutShifts) {
+function analyzeProfileFlags(totalRenders, wastedRenders, timing, layoutShifts, profileValidity) {
   const flags = /* @__PURE__ */ new Set();
   if (wastedRenders !== null && wastedRenders > 0 && wastedRenders / Math.max(1, totalRenders) > 0.3) {
     flags.add("WASTED_RENDER");
@@ -2864,7 +2884,7 @@ function analyzeProfileFlags(totalRenders, wastedRenders, timing, layoutShifts)
   if (layoutShifts.cumulativeScore > 0.1) {
     flags.add("LAYOUT_SHIFT_DETECTED");
   }
-  if (timing.js > 100) {
+  if (profileValidity === "interaction" && timing.js > 100) {
     flags.add("SLOW_INTERACTION");
   }
   return [...flags];
@@ -2903,13 +2923,18 @@ async function runInteractionProfile(componentName, filePath, props, interaction
       throw new Error(`Profiling setup failed: ${setupData.error}`);
     }
     const jsStart = Date.now();
+    let executedInteractions = 0;
+    let skippedInteractions = [];
     if (interaction.length > 0) {
       process.stderr.write(`  Replaying ${interaction.length} interaction step(s)\u2026
 `);
-      await replayInteraction(page, interaction);
+      const replayResult = await replayInteraction(page, interaction);
+      executedInteractions = replayResult.executed;
+      skippedInteractions = replayResult.skippedInteractions;
       await page.waitForTimeout(300);
     }
     const jsDuration = Date.now() - jsStart;
+    const profileValidity = interaction.length === 0 ? "no_interaction_requested" : executedInteractions === 0 ? "setup_only" : "interaction";
     const collected = await page.evaluate(buildProfilingCollectScript());
     const profileData = collected;
     if (profileData.error) {
@@ -2927,7 +2952,13 @@ async function runInteractionProfile(componentName, filePath, props, interaction
     const totalRenders = profileData.commitCount ?? 0;
     const uniqueComponents = profileData.uniqueComponents ?? 0;
     const wastedRenders = profileData.wastedRenders ?? null;
-    const flags = analyzeProfileFlags(totalRenders, wastedRenders, timing, layoutShifts);
+    const flags = analyzeProfileFlags(
+      totalRenders,
+      wastedRenders,
+      timing,
+      layoutShifts,
+      profileValidity
+    );
     return {
       component: componentName,
       totalRenders,
@@ -2941,7 +2972,17 @@ async function runInteractionProfile(componentName, filePath, props, interaction
       timing,
       layoutShifts,
       flags,
-      interaction
+      interaction,
+      interactionSteps: {
+        attempted: interaction.length,
+        executed: executedInteractions,
+        skipped: skippedInteractions.length
+      },
+      interactionStepsAttempted: interaction.length,
+      interactionStepsExecuted: executedInteractions,
+      interactionStepsSkipped: skippedInteractions.length,
+      skippedInteractions,
+      profileValidity
     };
   } finally {
     await browser.close();
@@ -3047,6 +3088,7 @@ function flattenSerializedValue(sv) {
     case "string":
     case "number":
     case "boolean":
+    case "date":
       return v.value;
     case "object": {
       if (!Array.isArray(v.entries)) return {};