Repository navigation
chore: add vinext as dev server - #32559
Conversation
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
There was a problem hiding this comment.
Pull request overview
This PR adds vinext as an alternative development server for the web workspace. Vinext is a Vite-based dev server that can run Next.js applications. The changes consolidate Vite and Vitest configurations into a single file, add vinext-specific workaround plugins, and introduce new npm scripts for vinext-based development, build, and start workflows.
Changes:
- Consolidated vitest.config.ts into vite.config.ts with mode-based plugin loading
- Added vinext package and related dependencies (image-size, @mdx-js/rollup, react-server-dom-webpack, @vitejs/plugin-rsc)
- Created custom Vite plugins to patch vinext's Google Font, image metadata, and constants shims
- Modified next.config.ts to use direct process.env access instead of centralized env object
- Changed API_PREFIX default from localhost to cloud.dify.ai
- Updated test file to use useRouter mock instead of AppRouterContext
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| web/vite.config.ts | Added vinext plugins, merged vitest config, added conditional plugin loading based on mode |
| web/vitest.config.ts | Deleted - configuration merged into vite.config.ts |
| web/package.json | Added vinext scripts (dev:vinext, build:vinext, start:vinext) and dependencies |
| web/pnpm-lock.yaml | Added dependency entries for vinext and related packages |
| web/next.config.ts | Replaced centralized env imports with direct process.env access |
| web/config/index.ts | Changed API_PREFIX default from localhost:5001 to cloud.dify.ai |
| web/app/components/header/account-dropdown/index.spec.tsx | Migrated from AppRouterContext to useRouter mock |
Files not reviewed (1)
- web/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 6 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.
Files not reviewed (1)
- web/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 4 comments.
Files not reviewed (1)
- web/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 3 comments.
Files not reviewed (1)
- web/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 13 out of 14 changed files in this pull request and generated no new comments.
Files not reviewed (1)
- web/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 13 out of 14 changed files in this pull request and generated 2 comments.
Files not reviewed (1)
- web/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 14 out of 15 changed files in this pull request and generated 3 comments.
Files not reviewed (1)
- web/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 16 out of 17 changed files in this pull request and generated 4 comments.
Files not reviewed (1)
- web/pnpm-lock.yaml: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: yyh <92089059+lyzno1@users.noreply.github.com>
Important
Fixes #<issue number>.Summary
fixes #32558
Screenshots
Checklist
make lintandmake type-check(backend) andcd web && npx lint-staged(frontend) to appease the lint gods