refactor: Improve code formatting and structure across multiple files

Author: tech-sushant

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

@@ -2,8 +2,11 @@ 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 { getAppUploadInstruction, validateSupportforAppAutomate, SupportedFramework } from "./utils.js";
-import logger from "../../../logger.js";
+import {
+  getAppUploadInstruction,
+  validateSupportforAppAutomate,
+  SupportedFramework,
+} from "./utils.js";
 
 import {
   getAppSDKPrefixCommand,
@@ -23,7 +26,6 @@ export async function setupAppAutomateHandler(
   rawInput: unknown,
   config: BrowserStackConfig,
 ): Promise<CallToolResult> {
-  
   const input = z.object(SETUP_APP_AUTOMATE_SCHEMA).parse(rawInput);
   const auth = getBrowserStackAuth(config);
   const [username, accessKey] = auth.split(":");

src/tools/appautomate-utils/appium-sdk/languages/ruby.ts

@@ -28,7 +28,7 @@ browser_caps:
     "os_version": "9.0"
     "app": "<replace with the APK path from the upload step>"
     "name": "first_test"
-\`\`\``
+\`\`\``,
   );
 
   const envStep = createStep(
@@ -63,14 +63,14 @@ end
 at_exit do
   $driver.quit if $driver
 end
-\`\`\``
+\`\`\``,
   );
 
   const runStep = createStep(
     "Run the test:",
     `\`\`\`bash
 bundle exec cucumber
-\`\`\``
+\`\`\``,
   );
 
   return combineInstructions(configStep, envStep, runStep);
@@ -79,7 +79,7 @@ bundle exec cucumber
 export function getRubySDKCommand(
   framework: string,
   username: string,
-  accessKey: string
+  accessKey: string,
 ): string {
   const { isWindows, getPlatformLabel } = PLATFORM_UTILS;
 
@@ -88,7 +88,7 @@ export function getRubySDKCommand(
     accessKey,
     isWindows,
     getPlatformLabel(),
-    "Set your BrowserStack credentials as environment variables:"
+    "Set your BrowserStack credentials as environment variables:",
   );
 
   const installStep = createStep(
@@ -102,7 +102,7 @@ gem install appium_lib
 
 # Install Cucumber
 gem install cucumber
-\`\`\``
+\`\`\``,
   );
 
   const gemfileStep = createStep(
@@ -118,7 +118,7 @@ gem 'cucumber'
 Then run:
 \`\`\`bash
 bundle install
-\`\`\``
+\`\`\``,
   );
 
   return combineInstructions(envStep, installStep, gemfileStep);

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

@@ -9,7 +9,7 @@ export enum AppSDKSupportedLanguageEnum {
 export type AppSDKSupportedLanguage = keyof typeof AppSDKSupportedLanguageEnum;
 
 export enum AppSDKSupportedFrameworkEnum {
-  appium = "appium"
+  appium = "appium",
 }
 
 export type AppSDKSupportedFramework =
@@ -60,7 +60,14 @@ export interface AppSDKInstruction {
 export const SUPPORTED_CONFIGURATIONS = {
   appium: {
     ruby: ["cucumberRuby"],
-    java: ["junit5", "junit4", "testng", "cucumberTestng", "selenide", "jbehave"],
+    java: [
+      "junit5",
+      "junit4",
+      "testng",
+      "cucumberTestng",
+      "selenide",
+      "jbehave",
+    ],
     csharp: ["nunit", "xunit", "mstest", "specflow", "reqnroll"],
     python: ["pytest", "robot", "behave", "lettuce"],
     nodejs: ["jest", "mocha", "cucumberJs", "webdriverio", "nightwatch"],

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

@@ -1,9 +1,10 @@
+import { uploadApp } from "../native-execution/appautomate.js";
 import {
   AppSDKSupportedTestingFramework,
   AppSDKSupportedTestingFrameworkEnum,
   createStep,
 } from "./index.js";
-import {SUPPORTED_CONFIGURATIONS} from "./types.js"
+import { SUPPORTED_CONFIGURATIONS } from "./types.js";
 
 export function isBrowserStackAppUrl(appPath: string): boolean {
   return appPath.startsWith("bs://");
@@ -51,10 +52,12 @@ export async function getAppUploadInstruction(
   if (
     detectedTestingFramework ===
       AppSDKSupportedTestingFrameworkEnum.nightwatch ||
-    detectedTestingFramework === AppSDKSupportedTestingFrameworkEnum.webdriverio ||
-    detectedTestingFramework === AppSDKSupportedTestingFrameworkEnum.cucumberRuby
+    detectedTestingFramework ===
+      AppSDKSupportedTestingFrameworkEnum.webdriverio ||
+    detectedTestingFramework ===
+      AppSDKSupportedTestingFrameworkEnum.cucumberRuby
   ) {
-    const app_url = "bs://ff4e358328a3e914fe4f0e46ec7af73f9c08cd55";
+    const app_url = await uploadApp(appPath, username, accessKey);
     if (app_url) {
       return createStep(
         "Updating app_path with app_url",
@@ -66,32 +69,37 @@ export async function getAppUploadInstruction(
 }
 
 export type SupportedFramework = keyof typeof SUPPORTED_CONFIGURATIONS;
-type SupportedLanguage = keyof typeof SUPPORTED_CONFIGURATIONS[SupportedFramework];
+type SupportedLanguage =
+  keyof (typeof SUPPORTED_CONFIGURATIONS)[SupportedFramework];
 type SupportedTestingFramework = string;
 
 export function validateSupportforAppAutomate(
   framework: SupportedFramework,
   language: SupportedLanguage,
-  testingFramework: SupportedTestingFramework
+  testingFramework: SupportedTestingFramework,
 ) {
-  const frameworks = Object.keys(SUPPORTED_CONFIGURATIONS) as SupportedFramework[];
+  const frameworks = Object.keys(
+    SUPPORTED_CONFIGURATIONS,
+  ) as SupportedFramework[];
   if (!SUPPORTED_CONFIGURATIONS[framework]) {
     throw new Error(
-      `Unsupported framework '${framework}'. Supported frameworks: ${frameworks.join(", ")}`
+      `Unsupported framework '${framework}'. Supported frameworks: ${frameworks.join(", ")}`,
     );
   }
 
-  const languages = Object.keys(SUPPORTED_CONFIGURATIONS[framework]) as SupportedLanguage[];
+  const languages = Object.keys(
+    SUPPORTED_CONFIGURATIONS[framework],
+  ) as SupportedLanguage[];
   if (!SUPPORTED_CONFIGURATIONS[framework][language]) {
     throw new Error(
-      `Unsupported language '${language}' for framework '${framework}'. Supported languages: ${languages.join(", ")}`
+      `Unsupported language '${language}' for framework '${framework}'. Supported languages: ${languages.join(", ")}`,
     );
   }
 
   const testingFrameworks = SUPPORTED_CONFIGURATIONS[framework][language];
   if (!testingFrameworks.includes(testingFramework)) {
     throw new Error(
-      `Unsupported testing framework '${testingFramework}' for language '${language}' and framework '${framework}'. Supported testing frameworks: ${testingFrameworks.join(", ")}`
+      `Unsupported testing framework '${testingFramework}' for language '${language}' and framework '${framework}'. Supported testing frameworks: ${testingFrameworks.join(", ")}`,
     );
   }
 }

src/tools/appautomate.ts

@@ -393,7 +393,8 @@ export default function addAppAutomationTools(
       try {
         return await setupAppAutomateHandler(args, config);
       } catch (error) {
-        const error_message = error instanceof Error ? error.message : "Unknown error";
+        const error_message =
+          error instanceof Error ? error.message : "Unknown error";
         return {
           content: [
             {