Merge branch 'update-node' into update-core-package

Author: ValentaTomas

.changeset/curly-pumpkins-kick.md

@@ -0,0 +1,5 @@
+---
+'@e2b/code-interpreter-template': patch
+---
+
+Add retry

.changeset/many-planets-talk.md

@@ -0,0 +1,5 @@
+---
+'@e2b/code-interpreter': patch
+---
+
+Fix node version

.changeset/wicked-mirrors-punch.md

@@ -0,0 +1,5 @@
+---
+'@e2b/code-interpreter-python': patch
+---
+
+Fix issue with secure False

.github/workflows/js_tests.yml

@@ -36,8 +36,8 @@ jobs:
       - name: Setup Node
         uses: actions/setup-node@v3
         with:
-          node-version: '18.x'
-          registry-url: 'https://registry.npmjs.org'
+          node-version: "20.x"
+          registry-url: "https://registry.npmjs.org"
           cache: pnpm
           cache-dependency-path: pnpm-lock.yaml
 
@@ -81,4 +81,4 @@ jobs:
         env:
           E2B_API_KEY: ${{ secrets.E2B_API_KEY }}
           E2B_DOMAIN: ${{ vars.E2B_DOMAIN }}
-          E2B_TESTS_TEMPLATE: ${{ inputs.E2B_TESTS_TEMPLATE }}
+          E2B_TESTS_TEMPLATE: ${{ inputs.E2B_TESTS_TEMPLATE }}

.github/workflows/release.yml

@@ -29,7 +29,7 @@ jobs:
       - name: Setup Node
         uses: actions/setup-node@v3
         with:
-          node-version: "18.x"
+          node-version: "20.x"
           registry-url: "https://registry.npmjs.org"
           cache: pnpm
           cache-dependency-path: pnpm-lock.yaml
@@ -71,7 +71,7 @@ jobs:
       - name: Setup Node
         uses: actions/setup-node@v3
         with:
-          node-version: "18.x"
+          node-version: "20.x"
           registry-url: "https://registry.npmjs.org"
           cache: pnpm
           cache-dependency-path: pnpm-lock.yaml
@@ -108,7 +108,6 @@ jobs:
           IS_RELEASE=$(./.github/scripts/is_release_for_package.sh "@e2b/code-interpreter-template")
           echo "release=$IS_RELEASE" >> "$GITHUB_OUTPUT"
 
-
   charts-release:
     name: Charts release
     if: needs.changes.outputs.charts == 'true'
@@ -128,7 +127,7 @@ jobs:
       - name: Set up Python
         uses: actions/setup-python@v4
         with:
-          python-version: '3.12'
+          python-version: "3.12"
 
       - name: Install and configure Poetry
         uses: snok/install-poetry@v1
@@ -156,7 +155,7 @@ jobs:
           poetry build
           poetry config pypi-token.pypi ${PYPI_TOKEN}
           poetry publish --skip-existing
-        working-directory:  ./chart_data_extractor
+        working-directory: ./chart_data_extractor
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
           PYPI_TOKEN: ${{ secrets.CHARTS_PYPI_TOKEN }}
@@ -165,7 +164,7 @@ jobs:
         id: output_version
         working-directory: ./chart_data_extractor
         run: |
-            echo "::set-output name=version::$(pnpm pkg get version --workspaces=false | tr -d \\\")"
+          echo "::set-output name=version::$(pnpm pkg get version --workspaces=false | tr -d \\\")"
 
   build-docker-image:
     name: Build Docker Image
@@ -231,17 +230,17 @@ jobs:
   python-tests:
     name: Python Tests
     needs: [changes, build-template]
-    if: always() && 
+    if: always() &&
       !contains(needs.*.result, 'failure') &&
-      !contains(needs.*.result, 'cancelled') && 
+      !contains(needs.*.result, 'cancelled') &&
       needs.changes.outputs.template == 'true'
     uses: ./.github/workflows/python_tests.yml
     secrets: inherit
 
   js-tests:
     name: JS Tests
     needs: [changes, build-template]
-    if: always() && 
+    if: always() &&
       !contains(needs.*.result, 'failure') &&
       !contains(needs.*.result, 'cancelled') &&
       needs.changes.outputs.template == 'true'
@@ -250,7 +249,7 @@ jobs:
 
   release:
     needs: [python-tests, js-tests]
-    if: always() && 
+    if: always() &&
       !contains(needs.*.result, 'failure') &&
       !contains(needs.*.result, 'cancelled') &&
       (needs.changes.outputs.js == 'true' || needs.changes.outputs.python == 'true' || needs.changes.outputs.charts == 'true' || needs.changes.outputs.template == 'true')
@@ -268,7 +267,6 @@ jobs:
         with:
           token: ${{ steps.app-token.outputs.token }}
 
-
       - name: Set up Python
         uses: actions/setup-python@v4
         with:
@@ -286,10 +284,10 @@ jobs:
         with:
           version: 9.5
 
-      - name: Setup Node.js 18
+      - name: Setup Node.js 20
         uses: actions/setup-node@v3
         with:
-          node-version: '18.x'
+          node-version: "20.x"
           cache: pnpm
 
       - name: Configure pnpm
@@ -304,11 +302,11 @@ jobs:
         run: pnpm run version
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-      
+
       - name: Generate SDK reference
         id: sdk-ref
         run: pnpm run --recursive generate-ref
-      
+
       - name: Show docs file structure
         run: |
           if [ -d "./sdk-reference" ]; then

.github/workflows/release_candidates.yml

@@ -22,11 +22,11 @@ jobs:
         with:
           version: 9.5
 
-      - name: Setup Node.js 18
+      - name: Setup Node.js 20
         uses: actions/setup-node@v4
         if: ${{ contains( github.event.pull_request.labels.*.name, 'js-rc') }}
         with:
-          node-version: '18.x'
+          node-version: "20.x"
           registry-url: https://registry.npmjs.org
           cache: pnpm
 

CODEOWNERS

@@ -1,3 +1,3 @@
 # These owners will be the default owners for everything in
 # the repo. Unless a later match takes precedence.
-* @jakubno @ValentaTomas @0div @mishushakov
+* @jakubno @ValentaTomas @mishushakov

js/package.json

@@ -39,7 +39,7 @@
     "format": "prettier --write src/ tests/ example.mts"
   },
   "devDependencies": {
-    "@types/node": "^18.18.6",
+    "@types/node": "^20.19.19",
     "dotenv": "^16.4.5",
     "knip": "^5.25.1",
     "npm-check-updates": "^17.1.14",
@@ -68,7 +68,7 @@
     "vm"
   ],
   "engines": {
-    "node": ">=18"
+    "node": ">=20"
   },
   "browserslist": [
     "defaults"

pnpm-lock.yaml

@@ -191,6 +191,10 @@ async def run_code(
         request_timeout = request_timeout or self.connection_config.request_timeout
         context_id = context.id if context else None
 
+        headers: Dict[str, str] = {}
+        if self._envd_access_token:
+            headers = {"X-Access-Token": self._envd_access_token}
+
         try:
             async with self._client.stream(
                 "POST",
@@ -201,7 +205,7 @@ async def run_code(
                     "language": language,
                     "env_vars": envs,
                 },
-                headers={"X-Access-Token": self._envd_access_token},
+                headers=headers,
                 timeout=(request_timeout, timeout, request_timeout, request_timeout),
             ) as response:
                 err = await aextract_exception(response)
@@ -249,10 +253,14 @@ async def create_code_context(
         if cwd:
             data["cwd"] = cwd
 
+        headers: Dict[str, str] = {}
+        if self._envd_access_token:
+            headers = {"X-Access-Token": self._envd_access_token}
+
         try:
             response = await self._client.post(
                 f"{self._jupyter_url}/contexts",
-                headers={"X-Access-Token": self._envd_access_token},
+                headers=headers,
                 json=data,
                 timeout=request_timeout or self.connection_config.request_timeout,
             )