Meituan-Dianping / mpvue

基于 Vue.js 的小程序开发框架,从底层支持 Vue.js 语法和构建工具体系。
http://mpvue.com
MIT License
20.41k stars 2.07k forks source link

sourcemap生成格式错误,导致还原失败 #1822

Open GendSmith opened 3 years ago

GendSmith commented 3 years ago

productionSourceMap: true的时候,sourcemap生成格式错误,mappings缺少分号

**mpvue 版本号:2.0.0

最小化复现代码:

[建议提供最小化可运行的代码:附件或文本代码] 直接用快速上手的demo就可以复现http://mpvue.com/mpvue/quickstart.html

// 还原脚本
const fs = require('fs')
const SourceMap = require('source-map')

async function originalPositionFor(line, column) {
   const sourceMapFilePath = "/Users/smithdeng/Desktop/testd/mpvue_demo_to_reappear_sourcemap_bug/reappear_sourcemap_bug/dist/wx/app.js.map"
   const sourceMapConsumer = await new SourceMap.SourceMapConsumer
                            (JSON.parse(fs.readFileSync(sourceMapFilePath, 'utf8')))
   const res = sourceMapConsumer.originalPositionFor({
      line,
      column,
   })
   console.log(res)
}

originalPositionFor(3,506)

问题复现步骤:

  1. 下载快速上手的demo,http://mpvue.com/mpvue/quickstart.html
  2. 把config/index.js的productionSourceMap改成true
  3. npm run build:wx

观察到的表现: dist目录下生成的map文件的mappings缺少分号,无法使用map来还原

我手动把分号补齐之后,就可以还原成功了

请看下图:

截图或动态图: image

image

savioryu commented 3 years ago

webpack-mpvue-asset-plugin 作为 mpvue 的资源路径解析插件,处理依赖模块的引用,增加了 require(xxx) 的代码,为什么会对 UglifyJsPluginSourceMap 造成影响呢?

    entryChunk.files.forEach(filePath => {
      const assetFile = compilation.assets[filePath]
      const extname = path.extname(filePath)
      let content = assetFile.source()

      chunks.reverse().forEach(chunk => {
        chunk.files.forEach(subFile => {
          if (path.extname(subFile) === extname && assetFile) {
            let relativePath = upath.normalize(relative(filePath, subFile))

            // 百度小程序 js 引用不支持绝对路径,改为相对路径
            if (extname === '.js') {
              relativePath = getRelativePath(relativePath)
            }

            if (/^(\.wxss)|(\.ttss)|(\.acss)|(\.css)$/.test(extname)) {
              relativePath = getRelativePath(relativePath)
              content = `@import "${relativePath}";\n${content}`
            } else if (!(/^\.map$/.test(extname))) {
              content = `require("${relativePath}")\n${content}`
            }
          }
        })
        assetFile.source = () => content
      })
    })

====================================================================================== 2021-02-07 更新

MpvuePlugin.prototype.apply = compiler => compiler.plugin('emit', emitHandle)

问题出在插件是在 emit 阶段执行的,即准备生成文件时注入了 require(xxx) 的代码,生成 sourceMap 的环节之前已经完成了

GendSmith commented 3 years ago

所以得让生成sourcemap这个步骤后置对吧

savioryu commented 3 years ago

解决思路:针对入口区块(entryChunk) 的 entryChunk.files 中的 .map 文件做处理, chunk 数对应插入的 require 代码行数,即添加相应数量的分号。

修改后的 webpack-mpvue-asset-plugin 如下:

const path = require('path')
const upath = require('upath')
const relative = require('relative')

const getRelativePath = (filePath) => {
  if (!/^\.(\.)?\//.test(filePath)) {
    filePath = `./${filePath}`
  }
  return filePath
}

const emitHandle = (compilation, callback) => {
  Object.keys(compilation.entrypoints).forEach(key => {
    const { chunks } = compilation.entrypoints[key]
    const entryChunk = chunks.pop()

    entryChunk.files.forEach(filePath => {
      const assetFile = compilation.assets[filePath]
      const extname = path.extname(filePath)
      let content = assetFile.source()

      chunks.reverse().forEach(chunk => {
        chunk.files.forEach(subFile => {
          if (path.extname(subFile) === extname && assetFile) {
            let relativePath = upath.normalize(relative(filePath, subFile))

            // 百度小程序 js 引用不支持绝对路径,改为相对路径
            if (extname === '.js') {
              relativePath = getRelativePath(relativePath)
            }

            if (/^(\.wxss)|(\.ttss)|(\.acss)|(\.css)$/.test(extname)) {
              relativePath = getRelativePath(relativePath)
              content = `@import "${relativePath}";\n${content}`
            } else if (!(/^\.map$/.test(extname))) {
              content = `require("${relativePath}")\n${content}`
            }
          }
        })
        assetFile.source = () => content
      })

      if ((/^\.map$/.test(extname))) {
        content = JSON.parse(content)
        content.mappings = new Array(chunks.length).fill(';').join('') + content.mappings
        content = JSON.stringify(content)
        assetFile.source = () => content
      }

    })

  })
  callback()
}

function MpvuePlugin() {}
MpvuePlugin.prototype.apply = compiler => compiler.plugin('emit', emitHandle)

module.exports = MpvuePlugin
ohyeahhh commented 8 months ago

所以最后怎么解决的

ANT-CYJ commented 8 months ago

放心吧,我已经收到啦。