[chore]: refactor & fix lint for browse CLI (browserbase#1821)

Author: seanmcguire12

.github/workflows/ci.yml

@@ -257,6 +257,24 @@ jobs:
             packages/server-v4/openapi.v4.yaml
           retention-days: 1
 
+  run-cli-tests:
+    name: CLI Tests
+    runs-on: ubuntu-latest
+    needs: [run-build, determine-changes]
+    if: needs.determine-changes.outputs.cli == 'true'
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          fetch-depth: 1
+
+      - uses: ./.github/actions/setup-node-pnpm-turbo
+        with:
+          use-prebuilt-artifacts: "true"
+          restore-turbo-cache: "false"
+
+      - name: Run CLI Tests
+        run: pnpm exec turbo run test:cli --filter=@browserbasehq/browse-cli
+
   discover-core-tests:
     runs-on: ubuntu-latest
     needs: [determine-changes]

.prettierignore

@@ -10,6 +10,7 @@ lib/v3/dom/build/
 packages/core/dist/
 packages/core/lib/dom/build/
 packages/core/lib/v3/dom/build/
+packages/cli/dist/
 packages/evals/dist/
 packages/docs/
 *.min.js

eslint.config.mjs

@@ -18,6 +18,10 @@ export default [
     ],
     languageOptions: { globals: globals.node },
   },
+  {
+    files: ["packages/cli/**/*.{js,cjs,mjs,ts}"],
+    languageOptions: { globals: globals.node },
+  },
   {
     ignores: [
       "**/dist/**",
@@ -73,4 +77,10 @@ export default [
       ],
     },
   },
+  {
+    files: ["packages/cli/**/*.{js,cjs,mjs,ts}"],
+    rules: {
+      "no-empty": ["error", { allowEmptyCatch: true }],
+    },
+  },
 ];

package.json

@@ -17,7 +17,7 @@
     "format": "prettier --write .",
     "prettier": "prettier --write .",
     "eslint": "eslint .",
-    "test": "turbo run test:core test:e2e test:server test:evals",
+    "test": "turbo run test:core test:e2e test:server test:evals test:cli",
     "test:core": "turbo run test:core --",
     "test:core:local": "STAGEHAND_BROWSER_TARGET=local pnpm run test:core --",
     "test:core:bb": "STAGEHAND_BROWSER_TARGET=browserbase pnpm run test:core --",

packages/cli/.prettierignore

@@ -27,7 +27,7 @@
     "stagehand"
   ],
   "engines": {
-    "node": ">=18.0.0"
+    "node": "^20.19.0 || >=22.12.0"
   },
   "publishConfig": {
     "access": "public"
@@ -46,13 +46,12 @@
     "dev": "tsx src/index.ts",
     "browse": "tsx src/index.ts",
     "typecheck": "tsc --noEmit",
-    "lint": "eslint .",
-    "lint:fix": "eslint . --fix",
-    "format": "prettier --write .",
-    "format:check": "prettier --check .",
+    "eslint": "eslint .",
+    "lint": "cd ../.. && prettier --check packages/cli && cd packages/cli && pnpm run eslint && pnpm run typecheck",
     "test": "vitest run",
+    "test:cli": "vitest run",
     "test:watch": "vitest",
-    "prepublishOnly": "npm run build"
+    "prepublishOnly": "pnpm run build"
   },
   "dependencies": {
     "@browserbasehq/stagehand": "workspace:*",
@@ -62,24 +61,13 @@
     "pino-pretty": "^13.0.0",
     "ws": "^8.18.0"
   },
-  "peerDependencies": {
-    "playwright": "^1.50.0"
-  },
-  "peerDependenciesMeta": {
-    "playwright": {
-      "optional": true
-    }
-  },
   "devDependencies": {
-    "@eslint/js": "^9.16.0",
     "@types/node": "^20.11.30",
-    "eslint": "^9.16.0",
-    "globals": "^15.13.0",
-    "prettier": "^3.2.5",
+    "devtools-protocol": "^0.0.1464554",
+    "eslint": "^10.0.2",
     "tsup": "^8.2.1",
     "tsx": "^4.10.5",
-    "typescript": "^5.2.2",
-    "typescript-eslint": "^8.17.0",
-    "vitest": "^1.6.0"
+    "typescript": "5.8.3",
+    "vitest": "^4.0.8"
   }
 }