Merge branch 'dev' into feat/migrate-basic-e2e-to-playwright

Author: Debajeet-1411

.agents/context/backend.md

@@ -14,7 +14,7 @@
 ## Coding Rules
 
 - Follow PEP 8, format with `black py` (v23.1.0), 80-char lines
-- Python >= 3.8 compatibility
+- Python >= 3.12 compatibility
 - Use `@pytorch_wrap` on all `Visdom` methods
 - Use `warn_once()` for deprecation warnings
 

.agents/context/testing.md

@@ -1,8 +1,20 @@
 # Testing Instructions
 
-Cypress 9 for E2E and visual regression. No Python unit tests — validation is via demo scripts and Cypress.
+Two layers: **pytest** for Python unit tests (pure functions, server utils, window/env lifecycle) and **Cypress 9** for E2E and visual regression. Demo scripts remain useful for manual/visual validation.
 
-## Run Tests
+## Run Python Tests (pytest)
+
+```bash
+pip install -r test-requirements.txt   # includes pytest, pytest-cov
+pytest                                 # runs the tracked suite under py/tests/
+pytest -m "not server"                 # skip tests that need a live server (CI default)
+```
+
+Config lives in `pyproject.toml` (`[tool.pytest.ini_options]`): discovery is scoped to
+`py/tests/` and `pythonpath = ["py"]` makes `import visdom` work without an editable install.
+Experimental `test_*.py` scripts in the repo root (and `test/`) are intentionally out of scope.
+
+## Run E2E / Visual Tests (Cypress)
 
 ```bash
 visdom -port 8098 -env_path /tmp   # Always start fresh server first
@@ -24,9 +36,19 @@ Always use port `8098` and `-env_path /tmp` for isolation.
 
 `basic.js` (connection), `pane.js` (CRUD), `text.js`, `image.js`, `properties.js`, `modal.js`, `misc.js`, `screenshots.init.js` (baseline), `screenshots.js` (comparison).
 
+## Writing Python Tests
+
+- Place in `py/tests/`, name files `test_*.py`, classes `Test*` (unittest `TestCase` or plain
+  pytest functions both work — pytest auto-discovers unittest).
+- Keep them hermetic (no running server). A test that genuinely needs a live server must be
+  marked `@pytest.mark.server` so CI can deselect it.
+- Start with simple hermetic tests (e.g., `test_smoke.py`) and add focused unit tests for
+  server/window/env lifecycle code as coverage grows.
+
 ## CI
 
-- Python 3.8, 3.9, 3.10 matrix, both WebSocket and polling modes
+- `python-tests.yml` runs `pytest -m "not server"` on a Python version matrix for every pull
+  request (and on pushes to master/dev)
 - Visual regression compares PR screenshots against base branch
 - `update-js-build-files.yml` auto-compiles JS on master
 - `pypi.yml` publishes to PyPI when VERSION changes

.github/actions/prepare/action.yml

@@ -12,7 +12,7 @@ inputs:
   python-version:
     description: 'Python version to use'
     required: false
-    default: '3.10'
+    default: '3.12'
 
 runs:
   using: "composite"
@@ -43,7 +43,7 @@ runs:
 
 
       # now cache pip
-      - uses: actions/cache@v3
+      - uses: actions/cache@v4
         id: cache-pip
         with:
           path: ~/.cache/pip
@@ -57,7 +57,7 @@ runs:
           pip3 install -e .
 
       # now compile the new version
-      - uses: actions/cache@v3
+      - uses: actions/cache@v4
         id: cache-npm
         with:
           path: |

.github/workflows/deploy-docs.yml

@@ -25,8 +25,8 @@ jobs:
       run:
         working-directory: ./website
     steps:
-      - uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5
-      - uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020
+      - uses: actions/checkout@9c091bb21b7c1c1d1991bb908d89e4e9dddfe3e0
+      - uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e
         with:
           node-version: 24
           cache: npm
@@ -39,7 +39,7 @@ jobs:
         run: npm run build
 
       - name: Upload artifact
-        uses: actions/upload-pages-artifact@56afc609e74202658d3ffba0e8f6dda462b719fa
+        uses: actions/upload-pages-artifact@fc324d3547104276b827a68afc52ff2a11cc49c9
         with:
           path: website/build
 
@@ -52,4 +52,4 @@ jobs:
     steps:
       - name: Deploy to GitHub Pages
         id: deployment
-        uses: actions/deploy-pages@d6db90164ac5ed86f2b6aed7e0febac5b3c0c03e
+        uses: actions/deploy-pages@cd2ce8fcbc39b97be8ca5fce6e763baed58fa128

.github/workflows/process-changes.yml

@@ -11,9 +11,9 @@ jobs:
     runs-on: ubuntu-latest
     steps:
     - name: "Checkout Repository"
-      uses: actions/checkout@v7
+      uses: actions/checkout@v7.0.0
     - name: "Install Node"
-      uses: actions/setup-node@v6
+      uses: actions/setup-node@v6.4.0
       with:
           node-version: '24'
     - name: "Install Dependencies"
@@ -25,7 +25,7 @@ jobs:
     name: "Python Linter Check"
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v7
+      - uses: actions/checkout@v7.0.0
       - uses: psf/black@26.5.1
         with:
           src: "./py"
@@ -41,7 +41,7 @@ jobs:
       jsfileschanged: ${{ steps.checkout.outputs.jsfileschanged }}
     steps:
       - name: "Checkout Repository"
-        uses: actions/checkout@v7
+        uses: actions/checkout@v7.0.0
       - uses: ./.github/actions/prepare
         with:
           loadprbuild: false
@@ -78,7 +78,7 @@ jobs:
     steps:
 
       - name: "Checkout Repository"
-        uses: actions/checkout@v7
+        uses: actions/checkout@v7.0.0
 
       - name: "Save current Head-Ref as PR-branch"
         shell: bash
@@ -109,6 +109,7 @@ jobs:
           browser: chrome
           start: visdom -port 8098 -env_path /tmp
           wait-on: 'http://localhost:8098'
+          env: CYPRESS_CHROME_ARGS="--disable-gpu --disable-dev-shm-usage --no-sandbox --disable-setuid-sandbox"
           spec: cypress/integration/*.init.js
 
       - uses: actions/upload-artifact@v7
@@ -123,7 +124,7 @@ jobs:
     needs: visual-regression-test-init
     steps:
       - name: "Checkout Repository"
-        uses: actions/checkout@v7
+        uses: actions/checkout@v7.0.0
       - uses: ./.github/actions/prepare
 
       - uses: actions/download-artifact@v8
@@ -138,6 +139,7 @@ jobs:
           browser: chrome
           start: visdom -port 8098 -env_path /tmp
           wait-on: 'http://localhost:8098'
+          env: CYPRESS_CHROME_ARGS="--disable-gpu --disable-dev-shm-usage --no-sandbox --disable-setuid-sandbox"
           spec: cypress/integration/screenshots.js
 
       - uses: actions/upload-artifact@v7
@@ -152,10 +154,10 @@ jobs:
     needs: install-and-build
     strategy:
       matrix:
-        python: ['3.10', '3.11', '3.12']
+        python: ['3.12', '3.13']
     steps:
       - name: "Checkout Repository"
-        uses: actions/checkout@v7
+        uses: actions/checkout@v7.0.0
       - uses: ./.github/actions/prepare
         with:
           python-version: ${{ matrix.python }}
@@ -167,6 +169,7 @@ jobs:
           browser: chrome
           start: visdom -port 8098 -env_path /tmp
           wait-on: 'http://localhost:8098'
+          env: CYPRESS_CHROME_ARGS="--disable-gpu --disable-dev-shm-usage --no-sandbox --disable-setuid-sandbox"
           config: excludeSpecPattern=**/screenshots.*
 
       - uses: actions/upload-artifact@v7
@@ -181,7 +184,7 @@ jobs:
     needs: install-and-build
     steps:
       - name: 'Checkout Repository'
-        uses: actions/checkout@v7
+        uses: actions/checkout@v7.0.0
       - uses: ./.github/actions/prepare
 
       - name: Cypress test
@@ -191,10 +194,35 @@ jobs:
           browser: chrome
           start: visdom -port 8098 -env_path /tmp -use_frontend_client_polling
           wait-on: 'http://localhost:8098'
+          env: CYPRESS_CHROME_ARGS="--disable-gpu --disable-dev-shm-usage --no-sandbox --disable-setuid-sandbox"
           config: excludeSpecPattern=**/screenshots.*
 
       - uses: actions/upload-artifact@v7
         if: failure()
         with:
           name: cypress-screenshots-functional-polling
           path: cypress/screenshots
+
+  playwright-test:
+    name: "Playwright Functional Test"
+    runs-on: ubuntu-latest
+    needs: install-and-build
+    steps:
+      - name: "Checkout Repository"
+        uses: actions/checkout@v7.0.0
+      - uses: ./.github/actions/prepare
+
+      - name: Install Playwright Chromium
+        run: npx playwright install --with-deps chromium
+
+
+
+      - name: Run Playwright tests
+        run: npm run test:pw
+
+      - name: Upload Playwright Report
+        uses: actions/upload-artifact@v7
+        if: failure()
+        with:
+          name: playwright-report
+          path: playwright/playwright-report/

.github/workflows/pypi.yml

@@ -14,7 +14,7 @@ jobs:
     name: "Deploy to PyPI"
     steps:
       - name: Checkout
-        uses: actions/checkout@v7
+        uses: actions/checkout@v7.0.0
       - name: Get git-tags
         run: |
           git fetch --prune --unshallow --tags

.github/workflows/python-tests.yml

@@ -0,0 +1,31 @@
+name: Python Tests
+
+on:
+  pull_request:
+  push:
+    branches:
+      - master
+      - dev
+
+jobs:
+  pytest:
+    name: "pytest (Python ${{ matrix.python-version }})"
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        python-version: ["3.12", "3.13"]
+    steps:
+      - name: Checkout Repository
+        uses: actions/checkout@v7.0.0
+      - name: Set up Python
+        uses: actions/setup-python@v6
+        with:
+          python-version: ${{ matrix.python-version }}
+      - name: Install package and test dependencies
+        run: |
+          python -m pip install --upgrade pip
+          python -m pip install -e .
+          python -m pip install -r test-requirements.txt
+      - name: Run pytest
+        run: pytest -m "not server"

.github/workflows/update-js-build-files.yml

@@ -17,12 +17,12 @@ jobs:
     runs-on: ubuntu-latest
     name: "Update Static JS-Files"
     steps:
-      - uses: actions/checkout@v7
+      - uses: actions/checkout@v7.0.0
         # > if the push needs to trigger other workflows, use a repo scoped Personal Acces Token
         # see: https://stackoverflow.com/questions/57921401/push-to-origin-from-github-action/58393457#58393457
         # with:
         #   token: ${{ secrets.PAT }}
-      - uses: actions/setup-node@v6
+      - uses: actions/setup-node@v6.4.0
         with:
           node-version: '24'
       - run: ELECTRON_SKIP_BINARY_DOWNLOAD=1 npm install

.gitignore

@@ -30,3 +30,10 @@ website/.docusaurus
 venv
 test-bin
 bin
+
+playwright/playwright-report/
+playwright/test-results/
+playwright/tmp/
+playwright-report/
+test-results/
+tmp/

.npmrc

@@ -0,0 +1 @@
+registry=https://registry.npmjs.org/