Linting ++

Author: tech-sushant

src/lib/version-resolver.ts

@@ -3,10 +3,7 @@
  * Else if exact match, returns that
  * Else picks the numerically closest (or first)
  */
-export function resolveVersion(
-  requested: string,
-  available: string[],
-): string {
+export function resolveVersion(requested: string, available: string[]): string {
   // strip duplicates & sort
   const uniq = Array.from(new Set(available));
 

src/tools/appautomate-utils/appium-sdk/config-generator.ts

@@ -42,8 +42,9 @@ platforms:
 ${platformConfigs}
 parallelsPerPlatform: 1
 browserstackLocal: true
-buildName: ${buildName}
+// TODO: replace projectName and buildName according to actual project
 projectName: ${projectName}
+buildName: ${buildName}
 debug: true
 networkLogs: true
 percy: false

src/tools/appautomate-utils/appium-sdk/constants.ts

@@ -91,40 +91,3 @@ export const SETUP_APP_AUTOMATE_SCHEMA = {
     .default("BStack-AppAutomate-Suite")
     .describe("Project name for organizing test runs on BrowserStack."),
 };
-
-export const SETUP_APP_AUTOMATE_SCHEMA_LEGACY = {
-  detectedFramework: z
-    .nativeEnum(AppSDKSupportedFrameworkEnum)
-    .describe(
-      "The mobile automation framework configured in the project. Example: 'appium'",
-    ),
-
-  detectedTestingFramework: z
-    .nativeEnum(AppSDKSupportedTestingFrameworkEnum)
-    .describe(
-      "The testing framework used in the project. Be precise with framework selection Example: 'testng', 'behave', 'pytest', 'robot'",
-    ),
-
-  detectedLanguage: z
-    .nativeEnum(AppSDKSupportedLanguageEnum)
-    .describe(
-      "The programming language used in the project. Supports Java and C#. Example: 'java', 'csharp'",
-    ),
-
-  desiredPlatforms: z
-    .array(z.nativeEnum(AppSDKSupportedPlatformEnum))
-    .describe(
-      "The mobile platforms the user wants to test on. Always ask this to the user, do not try to infer this. Example: ['android', 'ios']",
-    ),
-
-  appPath: z
-    .string()
-    .describe(
-      "Path to the mobile app file (.apk for Android, .ipa for iOS). Can be a local file path or a BrowserStack app URL (bs://)",
-    ),
-  project: z
-    .string()
-    .optional()
-    .default("BStack-AppAutomate-Suite")
-    .describe("Project name for organizing test runs on BrowserStack."),
-};

src/tools/appautomate-utils/appium-sdk/handler.ts

@@ -2,6 +2,8 @@ import { z } from "zod";
 import { CallToolResult } from "@modelcontextprotocol/sdk/types.js";
 import { BrowserStackConfig } from "../../../lib/types.js";
 import { getBrowserStackAuth } from "../../../lib/get-auth.js";
+import { validateAppAutomateDevices } from "../../sdk-utils/common/device-validator.js";
+
 import {
   getAppUploadInstruction,
   validateSupportforAppAutomate,
@@ -13,8 +15,6 @@ import {
   generateAppBrowserStackYMLInstructions,
 } from "./index.js";
 
-import { validateAppAutomateDevices } from "../../sdk-utils/common/device-validator.js";
-
 import {
   AppSDKSupportedLanguage,
   AppSDKSupportedTestingFramework,

src/tools/appautomate-utils/native-execution/constants.ts

@@ -29,7 +29,7 @@ export const RUN_APP_AUTOMATE_SCHEMA = {
         "  zip -r Tests.zip *.xctestrun *-Runner.app\n\n" +
         "If in other directory, provide existing test file path",
     ),
-    devices: z
+  devices: z
     .array(
       z.union([
         // Android: [android, deviceName, osVersion]

src/tools/sdk-utils/bstack/configUtils.ts

@@ -7,46 +7,33 @@ export function generateBrowserStackYMLInstructions(config: {
   projectName: string;
 }): string {
   const enablePercy = config.enablePercy || false;
-  const projectName = config.projectName;
+  const projectName = config.projectName || "BrowserStack Automate Build";
 
   // Generate platform configurations using the utility function
   const platformConfigs = generatePlatformConfigs(config);
 
-  // Determine build name and step title
-  const buildName =
-    config.validatedEnvironments && config.validatedEnvironments.length > 0
-      ? `${projectName}-Build`
-      : "Sample-Build";
-
   const stepTitle =
-    config.validatedEnvironments && config.validatedEnvironments.length > 0
-      ? "Create a browserstack.yml file in the project root with your validated device configurations:"
-      : "Create a browserstack.yml file in the project root. The file should be in the following format:";
+    "Create a browserstack.yml file in the project root with your validated device configurations:";
+
+  const buildName = `${projectName}-Build`;
 
   let ymlContent = `
 # ======================
 # BrowserStack Reporting
 # ======================
+
+# TODO: Replace these sample values with your actual project details
 projectName: ${projectName}
 buildName: ${buildName}
 
 # =======================================
 # Platforms (Browsers / Devices to test)
 # =======================================`;
 
-  if (config.validatedEnvironments && config.validatedEnvironments.length > 0) {
-    ymlContent += `
-# Auto-generated from validated device configurations
-platforms:
-${platformConfigs}`;
-  } else {
-    ymlContent += `
+  ymlContent += `
 # Platforms object contains all the browser / device combinations you want to test on.
-# Generate this on the basis of the following platforms requested by the user:
-# Requested platforms: ${config.platforms || []}
 platforms:
 ${platformConfigs}`;
-  }
 
   ymlContent += `
 
@@ -55,6 +42,7 @@ ${platformConfigs}`;
 # =======================
 # The number of parallel threads to be used for each platform set.
 # BrowserStack's SDK runner will select the best strategy based on the configured value
+# The number of parallel threads to be used for each platform set.
 parallelsPerPlatform: 1
 
 # =================

src/tools/sdk-utils/percy-bstack/handler.ts

@@ -107,7 +107,6 @@ export function runPercyWithBrowserstackSDK(
   }
 
   const ymlInstructions = generateBrowserStackYMLInstructions({
-    // For now, feed a normalized summary string from devices for the comment
     platforms:
       ((input as any).devices as string[][] | undefined)?.map((t) =>
         t.join(" "),