Skip to content

test(e2e): add vite migration e2e#359

Merged
fengmk2 merged 2 commits intomainfrom
12-22-test_e2e_add_vite_migration_e2e
Dec 24, 2025
Merged

test(e2e): add vite migration e2e#359
fengmk2 merged 2 commits intomainfrom
12-22-test_e2e_add_vite_migration_e2e

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented Dec 22, 2025

Note

  • E2E/CI: Adds rollipop to the matrix with custom commands and skip-install, sets GITHUB_TOKEN in job env, and avoids incorrect .gitignore usage during migration.
  • Ecosystem helper: patch-project.ts gains a vite migrate path using tgz via VITE_PLUS_OVERRIDE_PACKAGES and VITE_PLUS_VERSION; repo.json and .gitignore updated to include rollipop.
  • Migration tooling: Replaces hardcoded override map with env-driven VITE_PLUS_OVERRIDE_PACKAGES and VITE_PLUS_VERSION, updating package.json and workspace rewrite logic across pnpm/yarn/npm.
  • Core package: Fixes exports["./lib"].default to ./dist/tsdown/index.js and adds @babel/* dependencies.
  • Lockfile: Updated for new dependencies.

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

@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch 7 times, most recently from ae85fe3 to d552608 Compare December 23, 2025 01:36
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch 2 times, most recently from d809095 to 2521cbc Compare December 23, 2025 02:28
@fengmk2 fengmk2 force-pushed the rewrite-declare-module-namespace branch from 1a4348e to cc93454 Compare December 23, 2025 02:28
@fengmk2 fengmk2 self-assigned this Dec 23, 2025
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch 3 times, most recently from b07668e to 4dc59a9 Compare December 23, 2025 03:22
Comment thread .github/workflows/e2e-test.yml Outdated
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch from 4dc59a9 to cb9b075 Compare December 23, 2025 08:24
@fengmk2 fengmk2 force-pushed the rewrite-declare-module-namespace branch from cc93454 to 4da046c Compare December 23, 2025 08:24
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch from cb9b075 to 0560081 Compare December 23, 2025 08:31
@fengmk2 fengmk2 force-pushed the rewrite-declare-module-namespace branch from 4da046c to 25bac5c Compare December 23, 2025 08:31
@fengmk2 fengmk2 changed the base branch from rewrite-declare-module-namespace to graphite-base/359 December 23, 2025 08:48
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch from 0560081 to 8e2fbe1 Compare December 23, 2025 08:48
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch from 5b4f99f to 9ec2d28 Compare December 24, 2025 05:35
@fengmk2 fengmk2 force-pushed the rewrite-tsdown-to-vite-lib branch from b01886a to 43287ba Compare December 24, 2025 05:42
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch from 9ec2d28 to 81f77c1 Compare December 24, 2025 05:43
@fengmk2 fengmk2 force-pushed the rewrite-tsdown-to-vite-lib branch from 43287ba to 73b0808 Compare December 24, 2025 06:10
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch from 81f77c1 to d7324f3 Compare December 24, 2025 06:10
@fengmk2 fengmk2 force-pushed the rewrite-tsdown-to-vite-lib branch from 73b0808 to f0e76cc Compare December 24, 2025 07:02
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch from d7324f3 to b6debb3 Compare December 24, 2025 07:03
@Brooooooklyn
Copy link
Copy Markdown
Member

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!

@graphite-app graphite-app Bot changed the base branch from rewrite-tsdown-to-vite-lib to graphite-base/359 December 24, 2025 07:42
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch from b6debb3 to 4afb7c4 Compare December 24, 2025 07:45
@graphite-app graphite-app Bot changed the base branch from graphite-base/359 to rewrite-tsdown-to-vite-lib December 24, 2025 07:45
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch 2 times, most recently from b608c49 to 2432380 Compare December 24, 2025 07:52
@fengmk2 fengmk2 force-pushed the rewrite-tsdown-to-vite-lib branch from 97f9757 to c2da86a Compare December 24, 2025 07:56
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch from 2432380 to 9b76b61 Compare December 24, 2025 07:56
@fengmk2 fengmk2 changed the base branch from rewrite-tsdown-to-vite-lib to graphite-base/359 December 24, 2025 08:08
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch from 9b76b61 to 045b95f Compare December 24, 2025 08:11
@graphite-app graphite-app Bot changed the base branch from graphite-base/359 to main December 24, 2025 08:12
- Change ./lib export from index.mjs to index.js (matches actual file)
- Add @babel/generator, @babel/parser, @babel/types as dependencies
  (required at runtime by rolldown-plugin-dts bundled in tsdown)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@fengmk2 fengmk2 force-pushed the 12-22-test_e2e_add_vite_migration_e2e branch from 045b95f to 2aef050 Compare December 24, 2025 08:12
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying vite-plus with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2aef050
Status:⚡️  Build in progress...

View logs

@fengmk2 fengmk2 merged commit afbb297 into main Dec 24, 2025
12 of 13 checks passed
Copy link
Copy Markdown
Member Author

fengmk2 commented Dec 24, 2025

Merge activity

@fengmk2 fengmk2 deleted the 12-22-test_e2e_add_vite_migration_e2e branch December 24, 2025 12:42
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.

2 participants