Skip to content

test(e2e): add more projects to ecosystem CI tests#366

Merged
fengmk2 merged 1 commit intomainfrom
more-migration-e2e-projects
Dec 24, 2025
Merged

test(e2e): add more projects to ecosystem CI tests#366
fengmk2 merged 1 commit intomainfrom
more-migration-e2e-projects

Conversation

@fengmk2
Copy link
Copy Markdown
Member

@fengmk2 fengmk2 commented Dec 24, 2025

Note

Broadens ecosystem CI coverage and makes migration/install flows compatible with local tgz builds.

  • Adds frm-stack and vue-mini to E2E matrix; updates commands; keeps skeleton commented; adjusts vibe-dashboard steps and retains rollipop
  • Replaces pnpm install with vite install in workflow; installs Playwright where needed
  • Simplifies ecosystem-ci/patch-project.ts to a single vite migrate path using file: tgz overrides; removes project-specific patchers
  • Updates ecosystem-ci/repo.json (new repos, updated hashes) and .gitignore
  • Improves migrator logic (packages/global/src/migration/migrator.ts) to:
    • Use catalog: only when versions aren’t file:; write file: directly to overrides/devDeps
    • Skip catalog entries for file: versions and safely delete existing catalog items
    • Correct vite-plus version selection in monorepos vs npm
    • Respect file: when deciding catalog: usage in overrides and vite-plus injection

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

@fengmk2 fengmk2 force-pushed the more-migration-e2e-projects branch from 6712690 to 465ff94 Compare December 24, 2025 10:03
@fengmk2 fengmk2 changed the title test: support pnpm e2e migration project test(e2e): add frm-stack project to ecosystem CI tests Dec 24, 2025
@fengmk2 fengmk2 force-pushed the more-migration-e2e-projects branch 2 times, most recently from 1411b33 to dff4316 Compare December 24, 2025 12:31
@fengmk2 fengmk2 changed the base branch from 12-22-test_e2e_add_vite_migration_e2e to graphite-base/366 December 24, 2025 12:42
@fengmk2 fengmk2 force-pushed the more-migration-e2e-projects branch from dff4316 to 509d98e Compare December 24, 2025 12:43
@fengmk2 fengmk2 changed the base branch from graphite-base/366 to main December 24, 2025 12:43
@fengmk2 fengmk2 force-pushed the more-migration-e2e-projects branch from 509d98e to 3dd44a1 Compare December 24, 2025 12:56
Comment thread .github/workflows/e2e-test.yml Outdated
@fengmk2 fengmk2 changed the title test(e2e): add frm-stack project to ecosystem CI tests test(e2e): add more projects to ecosystem CI tests Dec 24, 2025
@fengmk2 fengmk2 force-pushed the more-migration-e2e-projects branch from 3dd44a1 to 3f08039 Compare December 24, 2025 13:04
@fengmk2 fengmk2 marked this pull request as ready for review December 24, 2025 13:05
@fengmk2 fengmk2 requested review from Brooooooklyn and Copilot and removed request for Copilot December 24, 2025 13:05
@fengmk2 fengmk2 self-assigned this Dec 24, 2025
@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!

@fengmk2 fengmk2 merged commit c4d9a09 into main Dec 24, 2025
28 of 30 checks passed
Copy link
Copy Markdown
Member Author

fengmk2 commented Dec 24, 2025

Merge activity

@fengmk2 fengmk2 deleted the more-migration-e2e-projects branch December 24, 2025 13:58
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