Add pipeline for linting and formatting (#120)

* Add pipeline for linting and formatting

* Add ruff

* Fix format

Author: jakubno

.eslintrc.cjs

@@ -0,0 +1,26 @@
+module.exports = {
+  env: {
+    node: true,
+    browser: true,
+    es6: true,
+  },
+  extends: ['eslint:recommended', 'plugin:@typescript-eslint/recommended'],
+  parser: '@typescript-eslint/parser',
+  parserOptions: {
+    ecmaVersion: 'latest',
+    sourceType: 'module',
+  },
+  ignorePatterns: ['dist/', 'node_modules/', '*.gen.ts'],
+  plugins: ['@typescript-eslint', 'unused-imports', '@stylistic/ts'],
+  rules: {
+    '@typescript-eslint/member-ordering': 'error',
+    '@typescript-eslint/ban-ts-comment': 'off', // "move fast" mode
+    '@typescript-eslint/no-explicit-any': 'off', // "move fast" mode
+    'linebreak-style': ['error', 'unix'],
+    'unused-imports/no-unused-imports': 'error',
+    // No double quotes
+    quotes: ['error', 'single', { avoidEscape: true }],
+    // No extra semicolon
+    '@stylistic/ts/semi': ['error', 'never'],
+  },
+}

.github/workflows/lint.yml

@@ -0,0 +1,68 @@
+name: Lint
+
+on:
+  pull_request:
+
+jobs:
+  lint:
+    name: Lint
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Checkout Repo
+        uses: actions/checkout@v4
+
+      - uses: pnpm/action-setup@v4
+        with:
+          version: 9.15.5
+
+      - name: Setup Node.js 20
+        uses: actions/setup-node@v4
+        with:
+          node-version: '20.x'
+          cache: pnpm
+
+      - name: Configure pnpm
+        run: |
+          pnpm config set auto-install-peers true
+          pnpm config set exclude-links-from-lockfile true
+
+      - name: Install dependencies
+        run: pnpm install --frozen-lockfile
+
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.9'
+
+      - name: Install and configure Poetry
+        uses: snok/install-poetry@v1
+        with:
+          version: 1.5.1
+          virtualenvs-create: true
+          virtualenvs-in-project: true
+          installer-parallel: true
+
+      - name: Install Python dependencies
+        working-directory: packages/python-sdk
+        run: |
+          poetry install --with dev
+
+      - name: Run linting
+        run: |
+          pnpm run lint
+
+      - name: Run formatting
+        run: |
+          pnpm run format
+
+      - name: Check for uncommitted changes
+        run: |
+          if [[ -n $(git status --porcelain) ]]; then
+            echo "❌ Files are not formatted properly:"
+            git status --short
+            git diff
+            exit 1
+          else
+            echo "✅ No changes detected."
+          fi

package.json

@@ -4,12 +4,19 @@
   "scripts": {
     "version": "changeset version && pnpm run -r postVersion",
     "publish": "changeset publish && pnpm run -r postPublish",
-    "rm-node-modules": "find . -name 'node_modules' -type d -prune -exec rm -rf '{}' +"
+    "rm-node-modules": "find . -name 'node_modules' -type d -prune -exec rm -rf '{}' +",
+    "lint": "pnpm --if-present --recursive run lint",
+    "format": "pnpm --if-present --recursive run format"
   },
-  "packageManager": "pnpm@8.7.6",
+  "packageManager": "pnpm@9.15.5",
   "devDependencies": {
     "@changesets/cli": "^2.26.2",
     "@changesets/read": "^0.6.5",
-    "changeset": "^0.2.6"
+    "changeset": "^0.2.6",
+    "@typescript-eslint/eslint-plugin": "^6.7.2",
+    "@typescript-eslint/parser": "^6.7.2",
+    "eslint": "^8.57.1",
+    "eslint-plugin-unused-imports": "^3.0.0",
+    "@stylistic/eslint-plugin-ts": "^1.6.2"
   }
 }

packages/js-sdk/.eslintrc.cjs

@@ -0,0 +1,4 @@
+module.exports = {
+  root: true,
+  extends: '../../.eslintrc.cjs',
+}

packages/js-sdk/example.mts

@@ -2,47 +2,44 @@ import { config } from 'dotenv'
 
 config()
 import { Sandbox } from './dist'
-import { writeFileSync } from 'fs';
+import { writeFileSync } from 'fs'
 
+console.log('Starting desktop sandbox...')
 
-console.log("Starting desktop sandbox...")
+console.time('--> Sandbox creation time')
+const desktop = await Sandbox.create()
+console.timeEnd('--> Sandbox creation time')
 
-console.time('--> Sandbox creation time');
-const desktop = await Sandbox.create();
-console.timeEnd('--> Sandbox creation time');
-
-console.log("Desktop Sandbox started, ID:", desktop.sandboxId)
-console.log("Screen size:", await desktop.getScreenSize())
+console.log('Desktop Sandbox started, ID:', desktop.sandboxId)
+console.log('Screen size:', await desktop.getScreenSize())
 
 await desktop.stream.start({
-  requireAuth: true
+  requireAuth: true,
 })
 
 const authKey = await desktop.stream.getAuthKey()
-console.log("Stream URL:", desktop.stream.getUrl({ authKey }))
+console.log('Stream URL:', desktop.stream.getUrl({ authKey }))
 
-await new Promise(resolve => setTimeout(resolve, 5000));
+await new Promise((resolve) => setTimeout(resolve, 5000))
 
 console.log("Moving mouse to 'Applications' and clicking...")
 await desktop.moveMouse(100, 100)
 await desktop.leftClick()
-console.log("Cursor position:", await desktop.getCursorPosition())
-
-await new Promise(resolve => setTimeout(resolve, 1000));
+console.log('Cursor position:', await desktop.getCursorPosition())
 
-const screenshot = await desktop.screenshot("bytes")
-writeFileSync('1.png', Buffer.from(screenshot));
+await new Promise((resolve) => setTimeout(resolve, 1000))
 
+const screenshot = await desktop.screenshot('bytes')
+writeFileSync('1.png', Buffer.from(screenshot))
 
 for (let i = 0; i < 20; i++) {
-  const x = Math.floor(Math.random() * 1024);
-  const y = Math.floor(Math.random() * 768);
-  await desktop.moveMouse(x, y);
-  await new Promise(resolve => setTimeout(resolve, 2000));
-  await desktop.rightClick();
+  const x = Math.floor(Math.random() * 1024)
+  const y = Math.floor(Math.random() * 768)
+  await desktop.moveMouse(x, y)
+  await new Promise((resolve) => setTimeout(resolve, 2000))
+  await desktop.rightClick()
   console.log('right clicked', i)
 }
 
-
 await desktop.stream.stop()
 await desktop.kill()

packages/js-sdk/package.json

@@ -49,7 +49,9 @@
     "dev": "tsup --watch",
     "test": "vitest run --disable-console-intercept",
     "example": "npx tsx example.mts",
-    "generate-ref": "./scripts/generate_sdk_ref.sh"
+    "generate-ref": "./scripts/generate_sdk_ref.sh",
+    "lint": "eslint src/ tests/",
+    "format": "prettier --write src/ tests/ example.mts"
   },
   "devDependencies": {
     "@types/node": "^22.13.9",

packages/js-sdk/src/index.ts

@@ -1,3 +1,3 @@
 export * from 'e2b'
 
-export { Sandbox } from './sandbox'
+export { Sandbox } from './sandbox'