Skip to content

refactor: remove optimization#650

Open
Mister-Hope wants to merge 1 commit intomainfrom
vite
Open

refactor: remove optimization#650
Mister-Hope wants to merge 1 commit intomainfrom
vite

Conversation

@Mister-Hope
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings March 14, 2026 03:10
@coveralls
Copy link

Pull Request Test Coverage Report for Build 23079090950

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 71.818%

Totals Coverage Status
Change from base Build 23078232513: 0.02%
Covered Lines: 1544
Relevant Lines: 1976

💛 - Coveralls

Copy link
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 removes previously applied Vite optimization / SSR bundler-option adjustments across multiple plugins and the default theme, and updates a few client-side dynamic imports. It also tweaks dependency declarations for some packages.

Changes:

  • Removed multiple extendsBundlerOptions hooks that were adding Vite optimizeDeps.* and ssr.* overrides.
  • Updated some dynamic imports to include webpackChunkName annotations.
  • Adjusted package dependency declarations and updated pnpm-lock.yaml accordingly.

Reviewed changes

Copilot reviewed 21 out of 22 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
tools/highlighter-helper/package.json Moves @vuepress/helper from peer deps to dev deps.
themes/theme-default/src/node/defaultTheme.ts Removes Vite optimize-deps exclusion for @theme.
plugins/tools/plugin-redirect/src/node/redirectPlugin.ts Removes Vite SSR noExternal bundler hook.
plugins/search/plugin-slimsearch/src/node/slimsearchPlugin.ts Removes Vite optimize/SSR bundler hook.
plugins/search/plugin-meilisearch/src/node/meilisearchPlugin.ts Removes Vite SSR external bundler hook.
plugins/search/plugin-meilisearch/src/client/components/MeiliSearch.ts Adds webpackChunkName to dynamic import.
plugins/search/plugin-docsearch/src/client/components/DocSearch.ts Adds webpackChunkName to dynamic import.
plugins/pwa/plugin-pwa/src/node/pwaPlugin.ts Removes Vite optimize/SSR bundler adjustments; keeps dev-server customization.
plugins/markdown/plugin-revealjs/src/node/revealJsPlugin.ts Removes Vite optimize/SSR bundler adjustments.
plugins/markdown/plugin-markdown-file-tree/package.json Removes unused @vuepress/helper dependency.
plugins/markdown/plugin-markdown-chart/src/node/markdownChartPlugin.ts Removes Vite optimize/SSR bundler adjustments.
plugins/markdown/plugin-markdown-chart/src/client/components/Mermaid.ts Switches Mermaid dynamic import target to mermaid.
plugins/features/plugin-photo-swipe/src/node/photoSwipePlugin.ts Removes Vite optimize/SSR bundler adjustments.
plugins/features/plugin-medium-zoom/package.json Removes unused @vuepress/helper dependency.
plugins/features/plugin-icon/src/node/iconPlugin.ts Removes Vite SSR noExternal bundler hook and custom-element config.
plugins/features/plugin-copyright/src/node/copyrightPlugin.ts Removes Vite SSR noExternal bundler hook.
plugins/features/plugin-copy-code/src/node/copyCodePlugin.ts Removes Vite SSR noExternal bundler hook.
plugins/features/plugin-catalog/src/node/catalogPlugin.ts Removes Vite SSR noExternal bundler hook.
plugins/development/plugin-reading-time/src/node/readingTimePlugin.ts Removes Vite SSR noExternal bundler hook.
plugins/blog/plugin-comment/src/node/commentPlugin.ts Removes Vite optimize/SSR bundler adjustments.
plugins/blog/plugin-blog/src/node/blogPlugin.ts Removes Vite SSR noExternal bundler hook.
pnpm-lock.yaml Lockfile updates reflecting dependency/bundler-hook removals.
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

"devDependencies": {
"@vuepress/helper": "workspace:*"
},
"peerDependencies": {
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