Skip to content

build(test): bundle deps#333

Merged
Brooooooklyn merged 1 commit intomainfrom
12-04-build_test_bundle_deps
Dec 16, 2025
Merged

build(test): bundle deps#333
Brooooooklyn merged 1 commit intomainfrom
12-04-build_test_bundle_deps

Conversation

@Brooooooklyn
Copy link
Copy Markdown
Member

@Brooooooklyn Brooooooklyn commented Dec 4, 2025

Note

Bundles vitest and @vitest deps into @voidzero-dev/vite-plus-test with browser/Node separation, rewrites imports/exports, adds plugin shims, and updates ecosystem CI to use bundled packages.

  • @voidzero-dev/vite-plus-test:
    • Build pipeline: Copy vitest-dev and @vitest/* into dist/; parse and bundle only leaf deps to dist/vendor/; rewrite imports (map vite@voidzero-dev/vite-plus-core, @vitest/*, vitest/*); create browser-safe module-runner stub; fix path resolutions; patch @vitest/browser and @vitest/browser-playwright for browser-safe usage; add vendor stubs and validate externals.
    • Exports: Add Node-specific entry dist/index-node.js, ./browser-compat, ./context, ./client, ./locators, ./browser-playwright, and ./plugins/* shims mapping @vitest/* to dist/plugins/*.
    • Dependencies: Move bundled deps to devDeps; add runtime deps (sirv, ws, pixelmatch, pngjs); keep @voidzero-dev/vite-plus-core as dep; drop @vitest/browser-playwright from peers.
  • Ecosystem CI:
    • vibe-dashboard: Point workspace overrides to local tgz; remove @vitest/* devDeps from apps/dashboard/package.json; switch imports to vitest/browser-playwright; add @vitest/* version overrides in pnpm-workspace.yaml.
    • skeleton: Change test script to vite test; add overrides for vitest, @vitest/browser, and @vitest/browser-playwright; relax Node engine; update vite.config.ts imports to vitest/browser-playwright; remove @vitest/browser-playwright from package devDeps; patch workspace versions and append policy rules.

Written by Cursor Bugbot for commit de87af8. This will update automatically on new commits. Configure here.

Copy link
Copy Markdown
Member Author

Brooooooklyn commented Dec 4, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@Brooooooklyn Brooooooklyn force-pushed the 12-04-build_test_bundle_deps branch 5 times, most recently from a3c9e14 to 9dfda06 Compare December 10, 2025 05:00
@Brooooooklyn Brooooooklyn changed the base branch from main to graphite-base/333 December 10, 2025 05:22
@Brooooooklyn Brooooooklyn force-pushed the 12-04-build_test_bundle_deps branch from 9dfda06 to 07971e2 Compare December 10, 2025 05:22
@Brooooooklyn Brooooooklyn changed the base branch from graphite-base/333 to 12-04-ci-split-e2e December 10, 2025 05:22
@Brooooooklyn Brooooooklyn force-pushed the 12-04-build_test_bundle_deps branch from 07971e2 to ecf8b6f Compare December 10, 2025 05:54
@Brooooooklyn Brooooooklyn force-pushed the 12-04-build_test_bundle_deps branch from ecf8b6f to bbca4ea Compare December 10, 2025 05:57
@Brooooooklyn Brooooooklyn changed the base branch from 12-04-ci-split-e2e to graphite-base/333 December 10, 2025 06:10
@Brooooooklyn Brooooooklyn force-pushed the 12-04-build_test_bundle_deps branch from bbca4ea to c9b64fe Compare December 10, 2025 06:11
@graphite-app graphite-app Bot changed the base branch from graphite-base/333 to main December 10, 2025 06:11
@Brooooooklyn Brooooooklyn force-pushed the 12-04-build_test_bundle_deps branch 6 times, most recently from e683b95 to 1058a03 Compare December 16, 2025 06:15
@Brooooooklyn Brooooooklyn force-pushed the 12-04-build_test_bundle_deps branch from 1058a03 to de87af8 Compare December 16, 2025 06:16
@Brooooooklyn Brooooooklyn marked this pull request as ready for review December 16, 2025 06:27
Copilot AI review requested due to automatic review settings December 16, 2025 06:27
@Brooooooklyn
Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

To use Codex here, create a Codex account and connect to github.

@Brooooooklyn
Copy link
Copy Markdown
Member Author

cursor review

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no bugs!


Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR implements dependency bundling for the @voidzero-dev/vite-plus-test package (vitest wrapper). Instead of having users install all @vitest/* dependencies separately, the PR bundles them into the main package to reduce installation size and complexity.

Key changes:

  • Moves vitest and @vitest/* dependencies from runtime dependencies to devDependencies (they're now bundled)
  • Implements a comprehensive build script that copies @vitest/* packages, bundles leaf dependencies, and rewrites imports
  • Adds new runtime dependencies (sirv, ws, pixelmatch, pngjs) and exports for @vitest/browser compatibility

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 17 comments.

File Description
packages/test/package.json Restructures dependencies (moving @vitest/* to devDeps), adds new runtime deps, adds extensive plugin exports for bundled packages
pnpm-lock.yaml Reflects the dependency reorganization from package.json
packages/test/build.ts Adds 1700+ lines implementing the bundling logic: copies @vitest/* packages, bundles leaf dependencies, rewrites imports, and creates compatibility shims
ecosystem-ci/patch-project.ts Updates test project patching to handle bundled dependencies and configure pnpm overrides
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/test/build.ts
Comment thread ecosystem-ci/patch-project.ts
Comment thread ecosystem-ci/patch-project.ts
Comment thread ecosystem-ci/patch-project.ts
Comment thread packages/test/build.ts
Comment thread packages/test/build.ts
Comment thread packages/test/build.ts
Comment thread packages/test/build.ts
Comment thread packages/test/build.ts
Comment thread packages/test/build.ts
Copy link
Copy Markdown
Member Author

Brooooooklyn commented Dec 16, 2025

Merge activity

  • Dec 16, 7:23 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Dec 16, 7:23 AM UTC: @Brooooooklyn merged this pull request with Graphite.

@Brooooooklyn Brooooooklyn merged commit e7285a8 into main Dec 16, 2025
19 checks passed
@Brooooooklyn Brooooooklyn deleted the 12-04-build_test_bundle_deps branch December 16, 2025 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants