Add pipeline for linting and formatting

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: python
+        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

chart_data_extractor/e2b_charts/charts/bars.py

@@ -37,7 +37,6 @@ def _extract_info(self, ax: Axes) -> None:
                 labels = [label.get_text() for label in ax.get_xticklabels()]
                 values = heights
             for label, value in zip(labels, values):
-
                 bar = BarData(label=label, value=value, group=group_label)
                 self.elements.append(bar)
 

chart_data_extractor/package.json

@@ -6,6 +6,13 @@
     "test": "poetry run pytest -n 4 --verbose -x",
     "example": "poetry run python3 example.py",
     "postVersion": "poetry version $(pnpm pkg get version --workspaces=false | tr -d \\\")",
-    "pretest": "poetry install"
+    "pretest": "poetry install",
+    "lint": "ruff check .",
+    "format": "ruff format ."
+  },
+  "devDependencies": {
+    "@typescript-eslint/eslint-plugin": "^7.11.0",
+    "@typescript-eslint/parser": "^7.11.0",
+    "eslint": "^8.57.1"
   }
 }

chart_data_extractor/pyproject.toml

@@ -27,3 +27,6 @@ build-backend = "poetry.core.masonry.api"
 
 [tool.poetry.urls]
 "Bug Tracker" = "https://github.com/e2b-dev/code-interpreter/issues"
+
+[tool.ruff.lint]
+ignore = ["F401", "F403"]

chart_data_extractor/tests/charts/test_blank.py

@@ -1,7 +1,6 @@
 import matplotlib.pyplot as plt
 
 from e2b_charts import chart_figure_to_chart
-from e2b_charts.charts import BarChart, ChartType
 
 
 def test_blank_chart():

chart_data_extractor/tests/charts/test_categorical_scale.py

@@ -1,6 +1,4 @@
-import numpy as np
 import matplotlib.pyplot as plt
-import datetime
 
 from e2b_charts import chart_figure_to_chart
 from e2b_charts.charts import LineChart

chart_data_extractor/tests/charts/test_log_graph.py

@@ -40,7 +40,7 @@ def test_log_chart():
     assert chart.x_label == "X-axis"
     assert chart.y_label == "Y-axis (log scale)"
 
-    assert chart.x_unit == None
+    assert chart.x_unit is None
     assert chart.y_unit == "log scale"
 
     assert chart.x_scale == "linear"

js/.eslintrc.cjs

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

js/example.mts

@@ -1,38 +1,35 @@
-import dotenv from 'dotenv'
+import { config } from 'dotenv'
 
 import { Sandbox } from './dist'
 
-dotenv.config()
+function log(...args: any[]) {
+  console.log(...args)
+}
 
-const code = `
-import matplotlib.pyplot as plt
-import numpy as np
+config()
 
-x = np.linspace(0, 20, 100)
-y = np.sin(x)
+const sbx = await Sandbox.create('bwyvo5fk343pbvxst536')
+log('ℹ️ sandbox created', sbx.sandboxId)
 
-plt.plot(x, y)
-plt.show()
+await sbx.runCode('x = 1')
+log('Sandbox code executed')
 
-x = np.linspace(0, 10, 100)
-plt.plot(x, y)
-plt.show()
+const sandboxId = await sbx.betaPause()
+log('Sandbox paused', sandboxId)
 
-import pandas
-pandas.DataFrame({"a": [1, 2, 3]})
-`
+// Resume the sandbox from the same state
+const sameSbx = await Sandbox.connect(sbx.sandboxId)
+log('Sandbox resumed', sameSbx.sandboxId)
 
-const sandbox = await Sandbox.create()
-console.log(sandbox.sandboxId)
+const execution = await sameSbx.runCode('x+=1; x')
+// Output result
+log(execution.text)
+log(execution.error)
+if (execution.text !== '2') {
+  log('Test failed:', 'Failed to resume sandbox')
+  throw new Error('Failed to resume sandbox')
+}
+log('Sandbox resumed successfully')
 
-const execution = await sandbox.runCode(code, {
-  onStdout(msg) {
-    console.log('stdout', msg)
-  },
-  onStderr(msg) {
-    console.log('stderr', msg)
-  },
-})
-console.log(execution.results[0].formats())
-console.log(execution.results[0].data)
-console.log(execution.results.length)
+await sbx.kill()
+log('Sandbox deleted')