Merge branch 'main' into agent-markdown

Author: nperez0111

.dockerignore

@@ -0,0 +1,17 @@
+# Keep the e2e image build context lean and Linux-clean: never copy host
+# (macOS) node_modules or build outputs — deps are installed fresh inside the
+# image. What actually lands in the image is selected explicitly by the COPY
+# steps in tests/Dockerfile (manifests + example apps), not by exclusions here;
+# these entries just stop the heavy/irrelevant trees from bloating the context.
+**/node_modules
+**/dist
+**/types
+**/.vite
+**/.vite-plus
+**/*.tsbuildinfo
+**/.DS_Store
+.git
+**/test-results
+**/blob-report
+**/playwright-report
+tests/.vitest-attachments

.github/workflows/build.yml

@@ -18,7 +18,7 @@ jobs:
     runs-on: ubuntu-latest
     timeout-minutes: 60
     steps:
-      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
         with:
           fetch-depth: 100
           persist-credentials: false
@@ -54,51 +54,26 @@ jobs:
         id: soft-release
         run: vp dlx pkg-pr-new publish './packages/*' # TODO disabled only for AI branch--compact
 
-  playwright-build:
-    name: "Playwright Build"
+  e2e:
+    # Vitest Browser Mode runs in the Playwright Linux container — the same
+    # environment as the local Docker run — so behaviour matches local dev.
+    # The suite resolves every `@blocknote/*` import to its `src/` via the
+    # aliases in `tests/vite.config.browser.ts` (vite transpiles the package
+    # sources on the fly), so the packages do NOT need to be built to `dist`
+    # first — `vp install` + the checked-out sources are enough.
+    name: "E2E - ${{ matrix.browser }} (${{ matrix.shardIndex }}/${{ matrix.shardTotal }})"
     runs-on: ubuntu-latest
     timeout-minutes: 30
-    steps:
-      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
-        with:
-          fetch-depth: 100
-          persist-credentials: false
-
-      - uses: voidzero-dev/setup-vp@2dec1e33f4ab2c6d5bce1b0c4607961bb1a3f7a1 # v1
-        with:
-          node-version-file: ".node-version"
-          cache: true
-
-      - name: Install dependencies
-        run: vp install
-
-      - name: Build packages
-        run: vp run -r build
-
-      - name: Upload build artifacts
-        uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7
-        with:
-          name: playwright-build
-          path: |
-            packages/*/dist
-            playground/dist
-          retention-days: 1
-
-  playwright:
-    name: "Playwright Tests - ${{ matrix.browser }} (${{ matrix.shardIndex }}/${{ matrix.shardTotal }})"
-    runs-on: ubuntu-latest
-    needs: playwright-build
-    timeout-minutes: 30
     container:
-      image: mcr.microsoft.com/playwright:v1.51.1-noble
+      image: mcr.microsoft.com/playwright:v1.60.0-noble
     strategy:
       fail-fast: false
       matrix:
         browser: [chromium, firefox, webkit]
         shardIndex: [1, 2]
         shardTotal: [2]
     steps:
-      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
         with:
           fetch-depth: 100
           persist-credentials: false
@@ -108,44 +83,53 @@ jobs:
           node-version-file: ".node-version"
           cache: true
 
-      - name: Download build artifacts
-        uses: actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8
-        with:
-          name: playwright-build
-
       - name: Install dependencies
         run: vp install
 
-      - name: Run server and Playwright tests
-        run: |
-          HOME=/root vp exec concurrently --success=first -r --kill-others \
-            "vp run --filter @blocknote/example-editor preview" \
-            "wait-on http://localhost:3000 && cd tests && vp exec playwright test --project ${{ matrix.browser }} --shard=${{ matrix.shardIndex }}/${{ matrix.shardTotal }}"
+      # No preview server: Vitest Browser Mode serves the tests + mounted example
+      # apps itself. `--browser` selects this matrix job's browser and
+      # `--shard=<index>/<total>` splits that browser's test files across two
+      # parallel machines. Each shard records a machine-readable `blob` report
+      # (named per browser+shard so they don't collide); the `merge-reports` job
+      # stitches every browser's shards into one HTML report afterwards.
+      - name: Run e2e tests (${{ matrix.browser }} ${{ matrix.shardIndex }}/${{ matrix.shardTotal }})
+        run: HOME=/root vp test -c vite.config.browser.ts --run --browser ${{ matrix.browser }} --shard=${{ matrix.shardIndex }}/${{ matrix.shardTotal }} --reporter=default --reporter=blob --outputFile.blob=blob-report/blob-${{ matrix.browser }}-${{ matrix.shardIndex }}.json
+        working-directory: tests
 
-      - name: Upload blob report
+      - name: Upload blob report (${{ matrix.browser }} ${{ matrix.shardIndex }}/${{ matrix.shardTotal }})
         uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7
         if: ${{ !cancelled() }}
         with:
           name: blob-report-${{ matrix.browser }}-${{ matrix.shardIndex }}
           path: tests/blob-report/
           retention-days: 1
 
-      - name: Upload HTML report
+      - name: Upload failure artifacts
         uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7
-        if: ${{ !cancelled() }}
+        if: ${{ failure() }}
         with:
-          name: playwright-report-${{ matrix.browser }}-${{ matrix.shardIndex }}
-          path: tests/playwright-report/
-          retention-days: 30
+          name: e2e-attachments-${{ matrix.browser }}-${{ matrix.shardIndex }}
+          path: tests/.vitest-attachments/
+          retention-days: 7
 
   merge-reports:
-    name: "Merge Playwright Reports"
-    if: ${{ !cancelled() }}
-    needs: playwright
+    # Stitch every browser+shard blob report into a single navigable HTML
+    # report (the Vitest equivalent of `playwright merge-reports`). Runs even
+    # when a shard failed, so the report always covers every browser.
+    name: "E2E Report"
     runs-on: ubuntu-latest
+    needs: e2e
+    if: ${{ !cancelled() }}
+    timeout-minutes: 15
+    # Same container as the e2e shards: `--mergeReports` doesn't run tests or
+    # launch browsers, but the HTML reporter still resolves the test files'
+    # browser environment, so the browser config must stay enabled (and present).
+    container:
+      image: mcr.microsoft.com/playwright:v1.60.0-noble
     steps:
-      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
         with:
+          fetch-depth: 100
           persist-credentials: false
 
       - uses: voidzero-dev/setup-vp@2dec1e33f4ab2c6d5bce1b0c4607961bb1a3f7a1 # v1
@@ -156,20 +140,62 @@ jobs:
       - name: Install dependencies
         run: vp install
 
+      # Gather every shard's blob into one directory (blob-report-chromium-1, …).
       - name: Download blob reports
         uses: actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8
         with:
-          path: tests/all-blob-reports
+          path: tests/blob-report
           pattern: blob-report-*
           merge-multiple: true
 
-      - name: Merge reports
-        run: vp exec playwright merge-reports --reporter html ./all-blob-reports
+      # Re-emit a single HTML report from the merged blobs (no tests are run).
+      - name: Merge into HTML report
+        run: HOME=/root vp test -c vite.config.browser.ts --mergeReports=blob-report --reporter=html
         working-directory: tests
 
       - name: Upload merged HTML report
         uses: actions/upload-artifact@043fb46d1a93c77aae656e7c1c64a875d1fc6a0a # v7
+        if: ${{ !cancelled() }}
         with:
-          name: playwright-report-merged
+          name: e2e-report
           path: tests/playwright-report/
-          retention-days: 30
+          retention-days: 7
+
+  deploy-report:
+    # Publish the merged report to a GitHub Pages PR preview and comment the
+    # link on the PR (Vercel-style). Pages serves it over HTTP, so the report's
+    # data loads correctly — unlike opening the downloaded artifact over file://
+    # (the @vitest/ui report fetches its data and is blocked by CORS there).
+    #
+    # Default `needs` semantics: this runs only when `merge-reports` succeeded
+    # (i.e. a report exists) — but regardless of whether the tests passed, so a
+    # red run still gets a navigable preview. Skipped for fork PRs, whose
+    # read-only token can't push to gh-pages or comment (they keep the artifact).
+    name: "E2E Report Preview"
+    runs-on: ubuntu-latest
+    needs: merge-reports
+    if: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.repo.full_name == github.repository }}
+    permissions:
+      contents: write
+      pull-requests: write
+    steps:
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
+        with:
+          persist-credentials: false
+
+      - name: Download merged report
+        uses: actions/download-artifact@3e5f45b2cfb9172054b4087a40e8e0b5a5461e7c # v8
+        with:
+          name: e2e-report
+          path: e2e-report
+
+      # Deploys to the `gh-pages` branch under `pr-preview/pr-<N>/` and posts a
+      # sticky comment with the URL. The companion `pr-preview-cleanup` workflow
+      # removes it when the PR closes.
+      - name: Deploy report to PR preview
+        uses: rossjrw/pr-preview-action@ffa7509e91a3ec8dfc2e5536c4d5c1acdf7a6de9 # v1
+        with:
+          source-dir: e2e-report
+          preview-branch: gh-pages
+          umbrella-dir: pr-preview
+          action: deploy

.github/workflows/pr-preview-cleanup.yml

@@ -0,0 +1,30 @@
+name: pr-preview-cleanup
+
+# Removes the GitHub Pages e2e-report preview that `build.yml`'s `deploy-report`
+# job published for a PR, once that PR is closed/merged. Kept separate from
+# `build.yml` so closing a PR doesn't re-run the whole build + e2e suite.
+on:
+  pull_request:
+    types: [closed]
+
+permissions:
+  contents: write
+  pull-requests: write
+
+jobs:
+  remove-preview:
+    name: "Remove E2E Report Preview"
+    runs-on: ubuntu-latest
+    # Fork PRs never got a preview (read-only token), so nothing to remove.
+    if: ${{ github.event.pull_request.head.repo.full_name == github.repository }}
+    steps:
+      - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6
+        with:
+          persist-credentials: false
+
+      - name: Remove PR preview
+        uses: rossjrw/pr-preview-action@ffa7509e91a3ec8dfc2e5536c4d5c1acdf7a6de9 # v1
+        with:
+          preview-branch: gh-pages
+          umbrella-dir: pr-preview
+          action: remove

.gitignore

@@ -10,6 +10,10 @@ examples/*/types
 tsconfig.tsbuildinfo
 # testing
 coverage
+# Vitest's attachment store (failure screenshots / diffs). Accumulates across
+# runs; the HTML reporter copies what it needs into playwright-report/data,
+# so the originals are unreferenced cruft.
+.vitest-attachments/
 
 # production
 /build

.vscode/extensions.json

@@ -1,5 +1,5 @@
 {
   // See http://go.microsoft.com/fwlink/?LinkId=827846
   // for the documentation about the extensions.json format
-  "recommendations": ["esbenp.prettier-vscode", "dbaeumer.vscode-eslint"]
+  "recommendations": ["VoidZero.vite-plus-extension-pack"]
 }

.vscode/settings.json

@@ -1,10 +1,16 @@
 {
-  "editor.defaultFormatter": "esbenp.prettier-vscode",
+  "editor.defaultFormatter": "oxc.oxc-vscode",
   "editor.formatOnSave": true,
-  "[javascript]": { "editor.defaultFormatter": "esbenp.prettier-vscode" },
-  "[typescriptreact]": {
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
+  "editor.formatOnSaveMode": "file",
+  "editor.codeActionsOnSave": {
+    "source.fixAll.oxc": "explicit"
   },
+  "oxc.fmt.configPath": "./vite.config.ts",
+  "npm.scriptRunner": "vp",
+  "[javascript]": { "editor.defaultFormatter": "oxc.oxc-vscode" },
+  "[javascriptreact]": { "editor.defaultFormatter": "oxc.oxc-vscode" },
+  "[typescript]": { "editor.defaultFormatter": "oxc.oxc-vscode" },
+  "[typescriptreact]": { "editor.defaultFormatter": "oxc.oxc-vscode" },
   "search.exclude": {
     "packages/editor/public/types": true,
     "packages/website/docs/.vitepress": false,
@@ -15,11 +21,5 @@
     "editor.defaultFormatter": "redhat.vscode-xml"
   },
   "scm.defaultViewMode": "tree",
-  "search.defaultViewMode": "tree",
-  "[typescript]": {
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
-  },
-  "[mdx]": {
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
-  }
+  "search.defaultViewMode": "tree"
 }