@validate.qa/runner 1.0.20 → 1.0.22

package/dist/cli.js

@@ -66068,6 +66068,8 @@ ${constraints.join("\n")}`);
     function buildMobileFallbackScenarios(input) {
       if (input.mode === "survey")
         return [];
+      if (input.featureName?.trim())
+        return [];
       const orderedScreens = targetAppScreens(input.graph, input.appId).filter((state2) => state2.screenType?.toLowerCase() !== "error").filter((state2) => state2.visited || state2.firstSeenIteration === 0).sort((a, b) => a.firstSeenIteration - b.firstSeenIteration);
       if (orderedScreens.length === 0)
         return [];
@@ -66215,6 +66217,15 @@ ${constraints.join("\n")}`);
             fallbackReason: result.fallbackReason ?? "Mobile planner produced no scenarios"
           };
         }
+        if (input.featureName?.trim()) {
+          return {
+            ...result,
+            featureGroups: mobileFeatureGroups,
+            warning: [result.warning, `Deep discovery could not plan any scenarios for "${input.featureName}" \u2014 the explorer did not reach that feature's screens this run. If the feature requires a signed-in session, verify TEST_USER_EMAIL / TEST_USER_PASSWORD are valid for this app, then re-run.`].filter(Boolean).join(" | "),
+            fallbackUsed: true,
+            fallbackReason: result.fallbackReason ?? `Planner produced no scenarios for feature "${input.featureName}"`
+          };
+        }
       }
       if (input.mode === "survey" && result.featureGroups.length === 0 && mobileFeatureGroups.length > 0) {
         return {
@@ -66495,7 +66506,8 @@ Max iterations: ${MAX_SCENARIO_ITERATIONS}. You will be warned near the end.`;
 This app has stored login credentials under these keys:
 ${keyList}
 
-When a scenario needs a signed-in session or fills a login/sign-up form, fill each credential field with \`mobile_fill_credential\`: pass the field ref and the KEY (e.g. \`{ ref: "e5", credentialKey: "TEST_USER_EMAIL" }\`). The runner types the real value on the device and the generated test stores a \`{{KEY}}\` placeholder that replay resolves \u2014 you never see, log, or echo the secret.
+When a scenario needs a signed-in session, complete the FULL login sequence \u2014 filling the fields alone does NOT log you in:
+1. Reach the email/password form. 2. Fill each credential field with \`mobile_fill_credential\` (pass the field ref and the KEY, e.g. \`{ ref: "e5", credentialKey: "TEST_USER_EMAIL" }\`) \u2014 the runner types the real value on the device and the generated test stores a \`{{KEY}}\` placeholder that replay resolves; you never see the secret. 3. TAP THE SUBMIT button (Sign In / Log In / Continue). 4. Re-snapshot and VERIFY an authenticated screen appeared. If login fails twice, do NOT keep retrying or wander: call finish_generate with verified:false and say the login could not be completed (the credentials may be invalid for this app).
 
 SECURITY RULES:
 - NEVER call \`mobile_type\` with a real email, password, or token value \u2014 the runner will REJECT the call and you lose an iteration. Use \`mobile_fill_credential\` for every credential field.
@@ -328471,7 +328483,7 @@ var require_package4 = __commonJS({
   "package.json"(exports2, module2) {
     module2.exports = {
       name: "@validate.qa/runner",
-      version: "1.0.20",
+      version: "1.0.22",
       description: "validate local test runner - execute Playwright tests on your machine, connected to validate.qa cloud.",
       bin: {
         "validate-runner": "dist/cli.js"
@@ -330240,6 +330252,10 @@ async function startMobileNetworkCapture(options) {
 
 // src/services/appium-executor.ts
 var DEFAULT_STEP_TIMEOUT_MS = 1e4;
+var MOBILE_NEW_COMMAND_TIMEOUT_SECONDS = (() => {
+  const raw = Number(process.env.VALIDATEQA_MOBILE_NEW_COMMAND_TIMEOUT);
+  return Number.isFinite(raw) && raw > 0 ? Math.floor(raw) : 600;
+})();
 var VERIFICATION_ACTIONS = /* @__PURE__ */ new Set(["assertVisible", "assertText", "waitForElement"]);
 var WD_SESSION_CREATE_TIMEOUT_MS = 12e4;
 var WD_ACTION_TIMEOUT_MS = 3e4;
@@ -330339,6 +330355,14 @@ async function createSession(target) {
     ...target.deviceId ? { "appium:udid": target.deviceId } : {},
     ...target.platformVersion ? { "appium:platformVersion": target.platformVersion } : {},
     "appium:noReset": true,
+    // Appium's default newCommandTimeout is 60s — it kills the session if no
+    // WebDriver command arrives in that window. A discovery session is driven
+    // by an AI loop where a single model turn (or a pause on a permission
+    // dialog) can exceed 60s, so the default silently terminates the session
+    // mid-run and every later command 404s ("invalid session id"), failing the
+    // whole survey. The runner owns session teardown explicitly, so a long
+    // idle window is safe. Env-overridable.
+    "appium:newCommandTimeout": MOBILE_NEW_COMMAND_TIMEOUT_SECONDS,
     ...realCaps
   } : {
     platformName: "Android",
@@ -330347,6 +330371,8 @@ async function createSession(target) {
     ...target.deviceId ? { "appium:udid": target.deviceId } : {},
     ...target.platformVersion ? { "appium:platformVersion": target.platformVersion } : {},
     "appium:noReset": true,
+    // See the iOS branch: the 60s default kills AI-driven discovery sessions.
+    "appium:newCommandTimeout": MOBILE_NEW_COMMAND_TIMEOUT_SECONDS,
     ...realCaps
   };
   const result = await wdRequest("/session", {

package/dist/cli.mjs

@@ -66073,6 +66073,8 @@ ${constraints.join("\n")}`);
     function buildMobileFallbackScenarios(input) {
       if (input.mode === "survey")
         return [];
+      if (input.featureName?.trim())
+        return [];
       const orderedScreens = targetAppScreens(input.graph, input.appId).filter((state2) => state2.screenType?.toLowerCase() !== "error").filter((state2) => state2.visited || state2.firstSeenIteration === 0).sort((a, b) => a.firstSeenIteration - b.firstSeenIteration);
       if (orderedScreens.length === 0)
         return [];
@@ -66220,6 +66222,15 @@ ${constraints.join("\n")}`);
             fallbackReason: result.fallbackReason ?? "Mobile planner produced no scenarios"
           };
         }
+        if (input.featureName?.trim()) {
+          return {
+            ...result,
+            featureGroups: mobileFeatureGroups,
+            warning: [result.warning, `Deep discovery could not plan any scenarios for "${input.featureName}" \u2014 the explorer did not reach that feature's screens this run. If the feature requires a signed-in session, verify TEST_USER_EMAIL / TEST_USER_PASSWORD are valid for this app, then re-run.`].filter(Boolean).join(" | "),
+            fallbackUsed: true,
+            fallbackReason: result.fallbackReason ?? `Planner produced no scenarios for feature "${input.featureName}"`
+          };
+        }
       }
       if (input.mode === "survey" && result.featureGroups.length === 0 && mobileFeatureGroups.length > 0) {
         return {
@@ -66500,7 +66511,8 @@ Max iterations: ${MAX_SCENARIO_ITERATIONS}. You will be warned near the end.`;
 This app has stored login credentials under these keys:
 ${keyList}
 
-When a scenario needs a signed-in session or fills a login/sign-up form, fill each credential field with \`mobile_fill_credential\`: pass the field ref and the KEY (e.g. \`{ ref: "e5", credentialKey: "TEST_USER_EMAIL" }\`). The runner types the real value on the device and the generated test stores a \`{{KEY}}\` placeholder that replay resolves \u2014 you never see, log, or echo the secret.
+When a scenario needs a signed-in session, complete the FULL login sequence \u2014 filling the fields alone does NOT log you in:
+1. Reach the email/password form. 2. Fill each credential field with \`mobile_fill_credential\` (pass the field ref and the KEY, e.g. \`{ ref: "e5", credentialKey: "TEST_USER_EMAIL" }\`) \u2014 the runner types the real value on the device and the generated test stores a \`{{KEY}}\` placeholder that replay resolves; you never see the secret. 3. TAP THE SUBMIT button (Sign In / Log In / Continue). 4. Re-snapshot and VERIFY an authenticated screen appeared. If login fails twice, do NOT keep retrying or wander: call finish_generate with verified:false and say the login could not be completed (the credentials may be invalid for this app).
 
 SECURITY RULES:
 - NEVER call \`mobile_type\` with a real email, password, or token value \u2014 the runner will REJECT the call and you lose an iteration. Use \`mobile_fill_credential\` for every credential field.
@@ -328476,7 +328488,7 @@ var require_package4 = __commonJS({
   "package.json"(exports, module) {
     module.exports = {
       name: "@validate.qa/runner",
-      version: "1.0.20",
+      version: "1.0.22",
       description: "validate local test runner - execute Playwright tests on your machine, connected to validate.qa cloud.",
       bin: {
         "validate-runner": "dist/cli.js"
@@ -330248,6 +330260,10 @@ async function startMobileNetworkCapture(options) {
 
 // src/services/appium-executor.ts
 var DEFAULT_STEP_TIMEOUT_MS = 1e4;
+var MOBILE_NEW_COMMAND_TIMEOUT_SECONDS = (() => {
+  const raw = Number(process.env.VALIDATEQA_MOBILE_NEW_COMMAND_TIMEOUT);
+  return Number.isFinite(raw) && raw > 0 ? Math.floor(raw) : 600;
+})();
 var VERIFICATION_ACTIONS = /* @__PURE__ */ new Set(["assertVisible", "assertText", "waitForElement"]);
 var WD_SESSION_CREATE_TIMEOUT_MS = 12e4;
 var WD_ACTION_TIMEOUT_MS = 3e4;
@@ -330347,6 +330363,14 @@ async function createSession(target) {
     ...target.deviceId ? { "appium:udid": target.deviceId } : {},
     ...target.platformVersion ? { "appium:platformVersion": target.platformVersion } : {},
     "appium:noReset": true,
+    // Appium's default newCommandTimeout is 60s — it kills the session if no
+    // WebDriver command arrives in that window. A discovery session is driven
+    // by an AI loop where a single model turn (or a pause on a permission
+    // dialog) can exceed 60s, so the default silently terminates the session
+    // mid-run and every later command 404s ("invalid session id"), failing the
+    // whole survey. The runner owns session teardown explicitly, so a long
+    // idle window is safe. Env-overridable.
+    "appium:newCommandTimeout": MOBILE_NEW_COMMAND_TIMEOUT_SECONDS,
     ...realCaps
   } : {
     platformName: "Android",
@@ -330355,6 +330379,8 @@ async function createSession(target) {
     ...target.deviceId ? { "appium:udid": target.deviceId } : {},
     ...target.platformVersion ? { "appium:platformVersion": target.platformVersion } : {},
     "appium:noReset": true,
+    // See the iOS branch: the 60s default kills AI-driven discovery sessions.
+    "appium:newCommandTimeout": MOBILE_NEW_COMMAND_TIMEOUT_SECONDS,
     ...realCaps
   };
   const result = await wdRequest("/session", {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@validate.qa/runner",
-  "version": "1.0.20",
+  "version": "1.0.22",
   "description": "validate local test runner - execute Playwright tests on your machine, connected to validate.qa cloud.",
   "bin": {
     "validate-runner": "dist/cli.js"