pi-agent-browser-native 0.9.3 → 0.9.4

package/dist/extensions/agent-browser/lib/input-modes/semantic-action.js

@@ -9,145 +9,235 @@ export function getCompiledSemanticActionSessionPrefix(compiled) {
     return commandIndex > 0 ? compiled.args.slice(0, commandIndex) : [];
 }
 export function isCompiledSemanticActionFindCommand(compiled) {
-    if (!compiled || compiled.action === "select")
-        return false;
-    return compiled.args[getCompiledSemanticActionCommandIndex(compiled)] === "find";
+    return (compiled !== undefined &&
+        compiled.action !== "select" &&
+        compiled.args[getCompiledSemanticActionCommandIndex(compiled)] === "find");
 }
-export function compileAgentBrowserSemanticAction(input) {
-    if (!isRecord(input)) {
-        return { error: "semanticAction must be an object." };
+function sessionArgs(session, args) {
+    return typeof session === "string" ? ["--session", session, ...args] : [...args];
+}
+function textError(action, text) {
+    if (text !== undefined && typeof text !== "string") {
+        return "semanticAction.text must be a string when provided.";
     }
-    const action = input.action;
-    const locator = input.locator;
-    const value = input.value;
-    const values = input.values;
-    const selector = input.selector;
-    const text = input.text;
-    const role = input.role;
-    const name = typeof input.name === "string" && input.name.trim().length === 0 ? undefined : input.name;
-    const session = input.session;
-    if (typeof action !== "string" || !AGENT_BROWSER_SEMANTIC_ACTIONS.includes(action)) {
-        return { error: `semanticAction.action must be one of: ${AGENT_BROWSER_SEMANTIC_ACTIONS.join(", ")}.` };
-    }
-    if (session !== undefined && (typeof session !== "string" || session.trim().length === 0)) {
-        return { error: "semanticAction.session must be a non-empty string when provided." };
+    if (action === "fill" && (typeof text !== "string" || text.length === 0)) {
+        return `semanticAction.text is required for ${action}.`;
     }
-    if (action === "select") {
-        if (text !== undefined) {
-            return { error: "semanticAction.text is not supported for select; use value or values for option values." };
-        }
-        if (typeof selector === "string" && selector.trim().length > 0) {
-            if (locator !== undefined || role !== undefined || name !== undefined) {
-                return { error: "semanticAction.selector cannot be combined with locator, role, or name for select; use selector plus value/values, or locator fields plus values." };
-            }
-            const selectedValues = getSelectValues(input, "semanticAction");
-            if (selectedValues.error)
-                return { error: selectedValues.error };
-            const args = typeof session === "string" ? ["--session", session, "select", selector, ...selectedValues.values] : ["select", selector, ...selectedValues.values];
-            return { compiled: { action: "select", selector, values: selectedValues.values, args } };
-        }
-        if (selector !== undefined) {
-            return { error: "semanticAction.selector must be a non-empty string when provided." };
-        }
-        if (locator === undefined) {
-            return { error: "semanticAction.selector or semanticAction.locator is required for select." };
-        }
-        if (locator !== "role" && locator !== "label") {
-            return { error: "semanticAction select locator must be role or label; use selector plus value/values for other targets." };
-        }
-        if (locator === "role") {
-            if (typeof role !== "string" || !/^(?:combobox|listbox)$/i.test(role)) {
-                return { error: "semanticAction.role must be combobox or listbox for locator=role select." };
-            }
-            if (typeof name !== "string" || name.trim().length === 0) {
-                return { error: "semanticAction.name is required for locator=role select." };
-            }
-            const optionValues = getSelectValues({ value, values }, "semanticAction");
-            if (optionValues.error)
-                return { error: optionValues.error };
-            const args = typeof session === "string"
-                ? ["--session", session, "find", "role", role, "select", ...optionValues.values, "--name", name]
-                : ["find", "role", role, "select", ...optionValues.values, "--name", name];
-            return { compiled: { action: "select", locator: "role", values: optionValues.values, args } };
-        }
-        if (typeof value !== "string" || value.trim().length === 0) {
-            return { error: "semanticAction.value must be the accessible label text for locator=label select." };
-        }
-        if (role !== undefined || name !== undefined) {
-            return { error: "semanticAction.role and name are only supported for locator=role select." };
-        }
-        const optionValues = getSelectValues({ values }, "semanticAction");
-        if (optionValues.error) {
-            return { error: optionValues.error.includes("required")
-                    ? "semanticAction.values is required for locator=label select (value is the label text)."
-                    : optionValues.error };
-        }
-        const args = typeof session === "string"
-            ? ["--session", session, "find", "label", value, "select", ...optionValues.values]
-            : ["find", "label", value, "select", ...optionValues.values];
-        return { compiled: { action: "select", locator: "label", values: optionValues.values, args } };
-    }
-    if (values !== undefined) {
-        return { error: "semanticAction.values is only supported for select actions." };
+    if (action !== "fill" && text !== undefined) {
+        return "semanticAction.text is only supported for fill actions.";
     }
-    if (selector !== undefined) {
-        if (typeof selector !== "string" || selector.trim().length === 0) {
-            return { error: "semanticAction.selector must be a non-empty string when provided." };
-        }
-        if (locator !== undefined || value !== undefined || role !== undefined || name !== undefined) {
-            return { error: "semanticAction.selector cannot be combined with locator, value, role, or name; use selector for a direct click/check/fill target or locator fields for find-based actions." };
-        }
-        if (text !== undefined && typeof text !== "string") {
-            return { error: "semanticAction.text must be a string when provided." };
-        }
-        if (action === "fill" && (typeof text !== "string" || text.length === 0)) {
-            return { error: `semanticAction.text is required for ${action}.` };
-        }
-        if (action !== "fill" && text !== undefined) {
-            return { error: "semanticAction.text is only supported for fill actions." };
-        }
-        const directArgs = typeof session === "string" ? ["--session", session, action, selector] : [action, selector];
-        if (action === "fill")
-            directArgs.push(text);
-        return { compiled: { action: action, selector, args: directArgs } };
-    }
-    if (typeof locator !== "string" || !AGENT_BROWSER_SEMANTIC_LOCATORS.includes(locator)) {
-        return { error: `semanticAction.locator must be one of: ${AGENT_BROWSER_SEMANTIC_LOCATORS.join(", ")}.` };
-    }
-    if (value !== undefined && (typeof value !== "string" || value.trim().length === 0)) {
-        return { error: "semanticAction.value must be a non-empty string when provided." };
-    }
-    if (role !== undefined && (typeof role !== "string" || role.trim().length === 0)) {
-        return { error: "semanticAction.role must be a non-empty string when provided." };
-    }
-    const locatorValue = locator === "role" && typeof role === "string" ? role : value;
-    if (typeof locatorValue !== "string" || locatorValue.trim().length === 0) {
-        return { error: locator === "role" ? "semanticAction.value or semanticAction.role must be a non-empty string for locator=role." : "semanticAction.value must be a non-empty string." };
+    return undefined;
+}
+function directAction(input, action, selector) {
+    if (input.locator !== undefined ||
+        input.value !== undefined ||
+        input.role !== undefined ||
+        input.name !== undefined) {
+        return {
+            error: "semanticAction.selector cannot be combined with locator, value, role, or name; use selector for a direct click/check/fill target or locator fields for find-based actions.",
+        };
+    }
+    const error = textError(action, input.text);
+    if (error !== undefined) {
+        return { error };
+    }
+    const args = sessionArgs(input.session, [action, selector]);
+    if (action === "fill" && typeof input.text === "string") {
+        args.push(input.text);
+    }
+    return { compiled: { action, selector, args } };
+}
+function roleSelect(input) {
+    const { role, name } = input;
+    if (typeof role !== "string" || !/^(?:combobox|listbox)$/i.test(role)) {
+        return { error: "semanticAction.role must be combobox or listbox for locator=role select." };
+    }
+    if (typeof name !== "string" || name.trim().length === 0) {
+        return { error: "semanticAction.name is required for locator=role select." };
+    }
+    const options = getSelectValues({ value: input.value, values: input.values }, "semanticAction");
+    if (options.error !== undefined) {
+        return { error: options.error };
+    }
+    return {
+        compiled: {
+            action: "select",
+            locator: "role",
+            values: options.values,
+            args: sessionArgs(input.session, [
+                "find",
+                "role",
+                role,
+                "select",
+                ...options.values,
+                "--name",
+                name,
+            ]),
+        },
+    };
+}
+function labelSelect(input) {
+    if (typeof input.value !== "string" || input.value.trim().length === 0) {
+        return {
+            error: "semanticAction.value must be the accessible label text for locator=label select.",
+        };
+    }
+    if (input.role !== undefined || input.name !== undefined) {
+        return { error: "semanticAction.role and name are only supported for locator=role select." };
+    }
+    const options = getSelectValues({ values: input.values }, "semanticAction");
+    if (options.error !== undefined) {
+        return {
+            error: options.error.includes("required")
+                ? "semanticAction.values is required for locator=label select (value is the label text)."
+                : options.error,
+        };
+    }
+    return {
+        compiled: {
+            action: "select",
+            locator: "label",
+            values: options.values,
+            args: sessionArgs(input.session, ["find", "label", input.value, "select", ...options.values]),
+        },
+    };
+}
+function selectAction(input) {
+    if (input.text !== undefined) {
+        return {
+            error: "semanticAction.text is not supported for select; use value or values for option values.",
+        };
     }
-    if (text !== undefined && typeof text !== "string") {
-        return { error: "semanticAction.text must be a string when provided." };
+    if (typeof input.selector === "string" && input.selector.trim().length > 0) {
+        return directSelect(input, input.selector);
     }
-    if (action === "fill" && (typeof text !== "string" || text.length === 0)) {
-        return { error: `semanticAction.text is required for ${action}.` };
+    if (input.selector !== undefined) {
+        return { error: "semanticAction.selector must be a non-empty string when provided." };
     }
-    if (action !== "fill" && text !== undefined) {
-        return { error: "semanticAction.text is only supported for fill actions." };
+    if (input.locator === undefined) {
+        return { error: "semanticAction.selector or semanticAction.locator is required for select." };
     }
-    if (role !== undefined && locator !== "role") {
-        return { error: "semanticAction.role is only supported for locator=role." };
+    if (input.locator !== "role" && input.locator !== "label") {
+        return {
+            error: "semanticAction select locator must be role or label; use selector plus value/values for other targets.",
+        };
     }
-    if (role !== undefined && value !== undefined && role !== value) {
-        return { error: "semanticAction.role must match value when both are provided for locator=role." };
+    return input.locator === "role" ? roleSelect(input) : labelSelect(input);
+}
+function directSelect(input, selector) {
+    if (input.locator !== undefined || input.role !== undefined || input.name !== undefined) {
+        return {
+            error: "semanticAction.selector cannot be combined with locator, role, or name for select; use selector plus value/values, or locator fields plus values.",
+        };
+    }
+    const options = getSelectValues(input, "semanticAction");
+    if (options.error !== undefined) {
+        return { error: options.error };
+    }
+    return {
+        compiled: {
+            action: "select",
+            selector,
+            values: options.values,
+            args: sessionArgs(input.session, ["select", selector, ...options.values]),
+        },
+    };
+}
+function locatorValueError(input) {
+    if (input.value !== undefined &&
+        (typeof input.value !== "string" || input.value.trim().length === 0)) {
+        return "semanticAction.value must be a non-empty string when provided.";
     }
-    if (name !== undefined && (locator !== "role" || typeof name !== "string" || name.length === 0)) {
-        return { error: "semanticAction.name is only supported as a non-empty string for locator=role." };
+    if (input.role !== undefined &&
+        (typeof input.role !== "string" || input.role.trim().length === 0)) {
+        return "semanticAction.role must be a non-empty string when provided.";
     }
-    const args = typeof session === "string" ? ["--session", session, "find", locator, locatorValue, action] : ["find", locator, locatorValue, action];
-    if (action === "fill") {
-        args.push(text);
+    return undefined;
+}
+function roleFieldsError(input, locator) {
+    if (input.role !== undefined && locator !== "role") {
+        return "semanticAction.role is only supported for locator=role.";
+    }
+    if (input.role !== undefined && input.value !== undefined && input.role !== input.value) {
+        return "semanticAction.role must match value when both are provided for locator=role.";
+    }
+    if (input.name !== undefined &&
+        (locator !== "role" || typeof input.name !== "string" || input.name.length === 0)) {
+        return "semanticAction.name is only supported as a non-empty string for locator=role.";
+    }
+    return undefined;
+}
+function findAction(input, action) {
+    const locator = AGENT_BROWSER_SEMANTIC_LOCATORS.find((value) => value === input.locator);
+    if (locator === undefined) {
+        return {
+            error: `semanticAction.locator must be one of: ${AGENT_BROWSER_SEMANTIC_LOCATORS.join(", ")}.`,
+        };
+    }
+    const valueError = locatorValueError(input);
+    if (valueError !== undefined) {
+        return { error: valueError };
+    }
+    const locatorValue = locator === "role" && typeof input.role === "string" ? input.role : input.value;
+    if (typeof locatorValue !== "string" || locatorValue.trim().length === 0) {
+        return {
+            error: locator === "role"
+                ? "semanticAction.value or semanticAction.role must be a non-empty string for locator=role."
+                : "semanticAction.value must be a non-empty string.",
+        };
+    }
+    const error = textError(action, input.text) ?? roleFieldsError(input, locator);
+    if (error !== undefined) {
+        return { error };
+    }
+    return {
+        compiled: { action, locator, args: findActionArgs(input, { action, locator, locatorValue }) },
+    };
+}
+function findActionArgs(input, target) {
+    const args = sessionArgs(input.session, [
+        "find",
+        target.locator,
+        target.locatorValue,
+        target.action,
+    ]);
+    if (target.action === "fill" && typeof input.text === "string") {
+        args.push(input.text);
+    }
+    if (target.locator === "role" && typeof input.name === "string") {
+        args.push("--name", input.name);
+    }
+    return args;
+}
+function nonSelectAction(input, action) {
+    if (input.values !== undefined) {
+        return { error: "semanticAction.values is only supported for select actions." };
     }
-    if (locator === "role" && typeof name === "string") {
-        args.push("--name", name);
+    if (input.selector === undefined) {
+        return findAction(input, action);
+    }
+    if (typeof input.selector !== "string" || input.selector.trim().length === 0) {
+        return { error: "semanticAction.selector must be a non-empty string when provided." };
+    }
+    return directAction(input, action, input.selector);
+}
+export function compileAgentBrowserSemanticAction(input) {
+    if (!isRecord(input)) {
+        return { error: "semanticAction must be an object." };
+    }
+    const action = AGENT_BROWSER_SEMANTIC_ACTIONS.find((value) => value === input.action);
+    if (action === undefined) {
+        return {
+            error: `semanticAction.action must be one of: ${AGENT_BROWSER_SEMANTIC_ACTIONS.join(", ")}.`,
+        };
+    }
+    if (input.session !== undefined &&
+        (typeof input.session !== "string" || input.session.trim().length === 0)) {
+        return { error: "semanticAction.session must be a non-empty string when provided." };
     }
-    return { compiled: { action: action, locator: locator, args } };
+    const normalized = {
+        ...input,
+        name: typeof input.name === "string" && input.name.trim().length === 0 ? undefined : input.name,
+    };
+    return action === "select" ? selectAction(normalized) : nonSelectAction(normalized, action);
 }

package/dist/extensions/agent-browser/lib/input-modes/shared.js

@@ -1,5 +1,11 @@
 import { isRecord } from "../parsing.js";
-import { SOURCE_LOOKUP_DEFAULT_MAX_WORKSPACE_FILES, SOURCE_LOOKUP_MAX_WORKSPACE_FILES } from "./types.js";
+import { SOURCE_LOOKUP_DEFAULT_MAX_WORKSPACE_FILES, SOURCE_LOOKUP_MAX_WORKSPACE_FILES, } from "./types.js";
+function isString(value) {
+    return typeof value === "string";
+}
+export function isStringArray(value) {
+    return Array.isArray(value) && value.every(isString);
+}
 export function getSelectValues(input, context) {
     const rawValue = input.value;
     const rawValues = input.values;
@@ -7,8 +13,12 @@ export function getSelectValues(input, context) {
         return { error: `${context}.value and ${context}.values cannot both be provided for select.` };
     }
     if (rawValues !== undefined) {
-        if (!Array.isArray(rawValues) || rawValues.length === 0 || rawValues.some((value) => typeof value !== "string" || value.trim().length === 0)) {
-            return { error: `${context}.values must be a non-empty array of non-empty strings for select.` };
+        if (!isStringArray(rawValues) ||
+            rawValues.length === 0 ||
+            rawValues.some((value) => value.trim().length === 0)) {
+            return {
+                error: `${context}.values must be a non-empty array of non-empty strings for select.`,
+            };
         }
         return { values: rawValues };
     }
@@ -25,8 +35,9 @@ export function getCommandNameFromBatchItem(item) {
     return Array.isArray(command) && typeof command[0] === "string" ? command[0] : undefined;
 }
 export function validateLookupMaxWorkspaceFiles(value, fieldName) {
-    if (value === undefined)
+    if (value === undefined) {
         return { value: SOURCE_LOOKUP_DEFAULT_MAX_WORKSPACE_FILES };
+    }
     if (typeof value !== "number" || !Number.isInteger(value) || value <= 0) {
         return { error: `${fieldName} must be a positive integer when provided.` };
     }

package/dist/extensions/agent-browser/lib/input-modes/types.js

@@ -1,16 +1,65 @@
 export const DEFAULT_SESSION_MODE = "auto";
 export const AGENT_BROWSER_CODE_MAX_TIMEOUT_MS = 300_000;
 export const AGENT_BROWSER_SEMANTIC_ACTIONS = ["check", "click", "fill", "select"];
-export const AGENT_BROWSER_SEMANTIC_LOCATORS = ["alt", "label", "placeholder", "role", "testid", "text", "title"];
-const AGENT_BROWSER_JOB_STEP_ACTIONS = ["open", "click", "fill", "type", "select", "wait", "assertText", "assertUrl", "waitForDownload", "screenshot", "snapshot"];
+export const AGENT_BROWSER_SEMANTIC_LOCATORS = [
+    "alt",
+    "label",
+    "placeholder",
+    "role",
+    "testid",
+    "text",
+    "title",
+];
+const AGENT_BROWSER_JOB_STEP_ACTIONS = [
+    "open",
+    "click",
+    "fill",
+    "type",
+    "select",
+    "wait",
+    "assertText",
+    "assertUrl",
+    "waitForDownload",
+    "screenshot",
+    "snapshot",
+];
 export const AGENT_BROWSER_QA_LOAD_STATES = ["domcontentloaded", "load", "networkidle"];
-export const AGENT_BROWSER_ELECTRON_ACTIONS = ["list", "launch", "status", "cleanup", "probe"];
+export const AGENT_BROWSER_ELECTRON_ACTIONS = [
+    "list",
+    "launch",
+    "status",
+    "cleanup",
+    "probe",
+];
 export const AGENT_BROWSER_ELECTRON_HANDOFFS = ["connect", "tabs", "snapshot"];
 export const AGENT_BROWSER_ELECTRON_TARGET_TYPES = ["page", "webview", "any"];
-export const AGENT_BROWSER_ELECTRON_LIST_FIELDS = new Set(["action", "query", "maxResults"]);
-export const AGENT_BROWSER_ELECTRON_PROBE_FIELDS = new Set(["action", "launchId", "timeoutMs"]);
-export const AGENT_BROWSER_ELECTRON_RESERVED_APP_ARGS = ["--user-data-dir", "--remote-debugging-port", "--remote-debugging-address", "--remote-debugging-pipe"];
+export const AGENT_BROWSER_ELECTRON_LIST_FIELDS = [
+    "action",
+    "query",
+    "maxResults",
+];
+export const AGENT_BROWSER_ELECTRON_PROBE_FIELDS = [
+    "action",
+    "launchId",
+    "timeoutMs",
+];
+export const AGENT_BROWSER_ELECTRON_RESERVED_APP_ARGS = [
+    "--user-data-dir",
+    "--remote-debugging-port",
+    "--remote-debugging-address",
+    "--remote-debugging-pipe",
+];
 export const SOURCE_LOOKUP_WORKSPACE_EXTENSIONS = new Set([".ts", ".tsx", ".js", ".jsx"]);
-export const SOURCE_LOOKUP_IGNORED_DIRECTORIES = new Set([".git", "node_modules", "dist", "build", "coverage", ".next", "out", "tmp", "temp"]);
+export const SOURCE_LOOKUP_IGNORED_DIRECTORIES = new Set([
+    ".git",
+    "node_modules",
+    "dist",
+    "build",
+    "coverage",
+    ".next",
+    "out",
+    "tmp",
+    "temp",
+]);
 export const SOURCE_LOOKUP_DEFAULT_MAX_WORKSPACE_FILES = 2_000;
 export const SOURCE_LOOKUP_MAX_WORKSPACE_FILES = 5_000;

package/dist/extensions/agent-browser/lib/json-schema.js

@@ -1,17 +1,27 @@
 const OPTIONAL_SCHEMA = Symbol("pi-agent-browser-optional-schema");
 function withOptions(schema, options) {
-    return { ...schema, ...(options ?? {}) };
+    return { ...schema, ...options };
 }
 function literalType(value) {
     const valueType = typeof value;
-    return valueType === "string" || valueType === "number" || valueType === "boolean" ? valueType : undefined;
+    return valueType === "string" || valueType === "number" || valueType === "boolean"
+        ? valueType
+        : undefined;
 }
 function propertySchema(schema) {
     const clone = { ...schema };
-    delete clone[OPTIONAL_SCHEMA];
+    if (OPTIONAL_SCHEMA in clone) {
+        delete clone[OPTIONAL_SCHEMA];
+    }
     return clone;
 }
-export const JsonSchema = {
+function isOptional(schema) {
+    return OPTIONAL_SCHEMA in schema && schema[OPTIONAL_SCHEMA] === true;
+}
+function unsafeSchema(schema) {
+    return schema;
+}
+const nativeJsonSchemaBuilder = {
     Array(items, options) {
         return withOptions({ type: "array", items }, options);
     },
@@ -23,18 +33,21 @@ export const JsonSchema = {
     },
     Literal(value, options) {
         const type = literalType(value);
-        return withOptions(type ? { type, const: value } : { const: value }, options);
+        return withOptions(type !== undefined ? { type, const: value } : { const: value }, options);
     },
     Number(options) {
         return withOptions({ type: "number" }, options);
     },
     Object(properties, options) {
         const required = globalThis.Object.entries(properties)
-            .filter(([, schema]) => schema[OPTIONAL_SCHEMA] !== true)
+            .filter(([, schema]) => !isOptional(schema))
             .map(([key]) => key);
         return withOptions({
             type: "object",
-            properties: globalThis.Object.fromEntries(globalThis.Object.entries(properties).map(([key, schema]) => [key, propertySchema(schema)])),
+            properties: globalThis.Object.fromEntries(globalThis.Object.entries(properties).map(([key, schema]) => [
+                key,
+                propertySchema(schema),
+            ])),
             ...(required.length > 0 ? { required } : {}),
         }, options);
     },
@@ -47,7 +60,6 @@ export const JsonSchema = {
     Union(types, options) {
         return withOptions({ anyOf: types }, options);
     },
-    Unsafe(schema) {
-        return schema;
-    },
+    Unsafe: unsafeSchema,
 };
+export const JsonSchema = nativeJsonSchemaBuilder;

package/dist/extensions/agent-browser/lib/launch-scoped-flags.js

@@ -112,7 +112,12 @@ export const LAUNCH_SCOPED_FLAG_DEFINITIONS = [
 ];
 export const LAUNCH_SCOPED_FLAGS = LAUNCH_SCOPED_FLAG_DEFINITIONS.map((definition) => definition.flag);
 export const LAUNCH_SCOPED_FLAG_LABEL = LAUNCH_SCOPED_FLAGS.join(", ");
-export const OPEN_RESULT_TAB_CORRECTION_FLAGS = new Set(["--profile", "--restore", "--session-name", "--state"]);
+export const OPEN_RESULT_TAB_CORRECTION_FLAGS = new Set([
+    "--profile",
+    "--restore",
+    "--session-name",
+    "--state",
+]);
 /** Launch modes that must never be combined with wrapper-managed restore state. */
 export const MANAGED_RESTORE_INCOMPATIBLE_FLAGS = [
     "--restore",
@@ -187,14 +192,21 @@ export function hasLaunchScopedFlagToken(args, flag) {
     const commandStartIndex = findCommandStartIndex(args);
     const command = commandStartIndex === undefined ? undefined : args[commandStartIndex];
     return args.some((token, index) => {
-        if (token.startsWith(`${flag}=`))
+        if (token.startsWith(`${flag}=`)) {
             return flag === "--restore";
-        if (token !== flag)
+        }
+        if (token !== flag) {
             return false;
-        if (flag === "--auto-connect")
+        }
+        if (flag === "--auto-connect") {
             return isBooleanFlagEnabled(args, flag);
-        if (flag === "--state" && command === "wait" && commandStartIndex !== undefined && index > commandStartIndex)
+        }
+        if (flag === "--state" &&
+            command === "wait" &&
+            commandStartIndex !== undefined &&
+            index > commandStartIndex) {
             return false;
+        }
         return true;
     });
 }