archive/rsnext
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

chore: reduce modified code

Browse source
This commit is contained in:
hardfist 2024-07-21 18:20:58 +08:00
parent 1fa178b627
commit 03eba8f996
4 changed files with 45 additions and 46 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
packages/next/src/build/webpack-config.ts
View file

@ -1929,9 +1929,9 @@ export default async function getBaseWebpackConfig(
new NextFontManifestPlugin({
appDir,
}),
!dev &&
isClient &&
new CssChunkingPlugin(config.experimental.cssChunking === 'strict'),
// !dev &&
// isClient &&
// new CssChunkingPlugin(config.experimental.cssChunking === 'strict'),
!dev &&
isClient &&
new (require('./webpack/plugins/telemetry-plugin').TelemetryPlugin)(

82
packages/next/src/build/webpack/config/blocks/css/index.ts
View file

@ -588,47 +588,47 @@ export const css = curry(async function css(
}
// Enable full mini-css-extract-plugin hmr for prod mode pages or app dir
// if (ctx.isClient && (ctx.isProduction || ctx.hasAppDir)) {
// // Extract CSS as CSS file(s) in the client-side production bundle.
// const MiniCssExtractPlugin =
// require('../../../plugins/mini-css-extract-plugin').default
// fns.push(
// plugin(
// // @ts-ignore webpack 5 compat
// new MiniCssExtractPlugin({
// filename: ctx.isProduction
// ? 'static/css/[contenthash].css'
// : 'static/css/[name].css',
// chunkFilename: ctx.isProduction
// ? 'static/css/[contenthash].css'
// : 'static/css/[name].css',
// // Next.js guarantees that CSS order "doesn't matter", due to imposed
// // restrictions:
// // 1. Global CSS can only be defined in a single entrypoint (_app)
// // 2. CSS Modules generate scoped class names by default and cannot
// // include Global CSS (:global() selector).
// //
// // While not a perfect guarantee (e.g. liberal use of `:global()`
// // selector), this assumption is required to code-split CSS.
// //
// // If this warning were to trigger, it'd be unactionable by the user,
// // but likely not valid -- so we disable it.
// ignoreOrder: true,
// insert: function (linkTag: HTMLLinkElement) {
// if (typeof _N_E_STYLE_LOAD === 'function') {
// const { href, onload, onerror } = linkTag
// _N_E_STYLE_LOAD(new URL(href).pathname).then(
// () => onload?.call(linkTag, { type: 'load' } as Event),
// () => onerror?.call(linkTag, {} as Event)
// )
// } else {
// document.head.appendChild(linkTag)
// }
// },
// })
// )
// )
// }
if (ctx.isClient && (ctx.isProduction || ctx.hasAppDir)) {
// Extract CSS as CSS file(s) in the client-side production bundle.
const MiniCssExtractPlugin =
require('@rspack/core').CssExtractRspackPlugin;
fns.push(
plugin(
// @ts-ignore webpack 5 compat
new MiniCssExtractPlugin({
filename: ctx.isProduction
? 'static/css/[contenthash].css'
: 'static/css/[name].css',
chunkFilename: ctx.isProduction
? 'static/css/[contenthash].css'
: 'static/css/[name].css',
// Next.js guarantees that CSS order "doesn't matter", due to imposed
// restrictions:
// 1. Global CSS can only be defined in a single entrypoint (_app)
// 2. CSS Modules generate scoped class names by default and cannot
// include Global CSS (:global() selector).
//
// While not a perfect guarantee (e.g. liberal use of `:global()`
// selector), this assumption is required to code-split CSS.
//
// If this warning were to trigger, it'd be unactionable by the user,
// but likely not valid -- so we disable it.
ignoreOrder: true,
insert: function (linkTag: HTMLLinkElement) {
if (typeof _N_E_STYLE_LOAD === 'function') {
const { href, onload, onerror } = linkTag
_N_E_STYLE_LOAD(new URL(href).pathname).then(
() => onload?.call(linkTag, { type: 'load' } as Event),
() => onerror?.call(linkTag, {} as Event)
)
} else {
document.head.appendChild(linkTag)
}
},
})
)
)
}
const fn = pipe(...fns)
return fn(config)

2
packages/next/src/build/webpack/loaders/utils.ts
View file

@ -33,7 +33,7 @@ export function isCSSMod(mod: {
mod.loaders?.some(
({ loader }) =>
loader.includes('next-style-loader/index.js') ||
// loader.includes('mini-css-extract-plugin/loader.js') ||
loader.includes('rspack.CssExtractRspackPlugin.loader') ||
loader.includes('@vanilla-extract/webpack-plugin/loader/')
)
)

1
packages/next/src/build/webpack/plugins/css-chunking-plugin.ts
View file

@ -29,7 +29,6 @@ export class CssChunkingPlugin {
}
public apply(compiler: Compiler) {
return
const strict = this.strict
const summary = !!process.env.CSS_CHUNKING_SUMMARY
compiler.hooks.thisCompilation.tap(PLUGIN_NAME, (compilation) => {