@agent-scope/cli 1.20.3 → 1.20.4

package/dist/index.d.cts

@@ -363,6 +363,18 @@ interface LayoutShiftData {
  * Heuristic flags from §6.1 catalog applicable to interaction profiling.
  */
 type ProfileHeuristicFlag = "WASTED_RENDER" | "MEMO_INEFFECTIVE" | "EFFECT_EVERY_RENDER" | "MISSING_CLEANUP" | "HIGH_RENDER_COUNT" | "LAYOUT_SHIFT_DETECTED" | "SLOW_INTERACTION";
+type ProfileValidity = "interaction" | "setup_only" | "no_interaction_requested";
+interface SkippedInteraction {
+    index: number;
+    action: InteractionStep$1["action"];
+    target?: string;
+    reason: "not_found" | "missing_target" | "missing_value" | "unsupported";
+}
+interface InteractionExecutionSummary {
+    attempted: number;
+    executed: number;
+    skipped: number;
+}
 /** Full interaction profile result per Spec §5. */
 interface InteractionProfile {
     /** Component name profiled. */
@@ -387,6 +399,18 @@ interface InteractionProfile {
     flags: ProfileHeuristicFlag[];
     /** Interaction steps that were replayed. */
     interaction: InteractionStep$1[];
+    /** Interaction execution counts for validity-aware profiling. */
+    interactionSteps: InteractionExecutionSummary;
+    /** Backwards-compatible attempted interaction count. */
+    interactionStepsAttempted: number;
+    /** Backwards-compatible executed interaction count. */
+    interactionStepsExecuted: number;
+    /** Backwards-compatible skipped interaction count. */
+    interactionStepsSkipped: number;
+    /** Details for each skipped interaction step. */
+    skippedInteractions: SkippedInteraction[];
+    /** Whether timing/flags reflect a real interaction or setup-only work. */
+    profileValidity: ProfileValidity;
 }
 
 /** Root cause categories per Instrumentation Spec §2.1 */

package/dist/index.d.ts

@@ -363,6 +363,18 @@ interface LayoutShiftData {
  * Heuristic flags from §6.1 catalog applicable to interaction profiling.
  */
 type ProfileHeuristicFlag = "WASTED_RENDER" | "MEMO_INEFFECTIVE" | "EFFECT_EVERY_RENDER" | "MISSING_CLEANUP" | "HIGH_RENDER_COUNT" | "LAYOUT_SHIFT_DETECTED" | "SLOW_INTERACTION";
+type ProfileValidity = "interaction" | "setup_only" | "no_interaction_requested";
+interface SkippedInteraction {
+    index: number;
+    action: InteractionStep$1["action"];
+    target?: string;
+    reason: "not_found" | "missing_target" | "missing_value" | "unsupported";
+}
+interface InteractionExecutionSummary {
+    attempted: number;
+    executed: number;
+    skipped: number;
+}
 /** Full interaction profile result per Spec §5. */
 interface InteractionProfile {
     /** Component name profiled. */
@@ -387,6 +399,18 @@ interface InteractionProfile {
     flags: ProfileHeuristicFlag[];
     /** Interaction steps that were replayed. */
     interaction: InteractionStep$1[];
+    /** Interaction execution counts for validity-aware profiling. */
+    interactionSteps: InteractionExecutionSummary;
+    /** Backwards-compatible attempted interaction count. */
+    interactionStepsAttempted: number;
+    /** Backwards-compatible executed interaction count. */
+    interactionStepsExecuted: number;
+    /** Backwards-compatible skipped interaction count. */
+    interactionStepsSkipped: number;
+    /** Details for each skipped interaction step. */
+    skippedInteractions: SkippedInteraction[];
+    /** Whether timing/flags reflect a real interaction or setup-only work. */
+    profileValidity: ProfileValidity;
 }
 
 /** Root cause categories per Instrumentation Spec §2.1 */

package/dist/index.js

@@ -9,6 +9,7 @@ import { createRequire } from 'module';
 import { mkdir, access, writeFile } from 'fs/promises';
 import * as readline from 'readline';
 import { loadTrace, generateTest, getBrowserEntryScript } from '@agent-scope/playwright';
+import { serialize } from '@agent-scope/core';
 import { chromium } from 'playwright';
 import { tmpdir } from 'os';
 import { createServer } from 'http';
@@ -2278,26 +2279,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;
@@ -2518,7 +2500,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) {
@@ -2714,44 +2699,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");
@@ -2762,7 +2782,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];
@@ -2801,13 +2821,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) {
@@ -2825,7 +2850,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,
@@ -2839,7 +2870,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();
@@ -2939,6 +2980,7 @@ function flattenSerializedValue(sv) {
     case "string":
     case "number":
     case "boolean":
+    case "date":
       return v.value;
     case "object": {
       if (!Array.isArray(v.entries)) return {};