build: add browser builds for @vue/compiler-sfc

packages/compiler-core/src/codegen.ts

@@ -31,7 +31,6 @@ import {
   advancePositionWithMutation,
   assert,
   isSimpleIdentifier,
-  loadDep,
   toValidAssetId
 } from './utils'
 import { isString, isArray, isSymbol } from '@vue/shared'
@@ -167,7 +166,7 @@ function createCodegenContext(
 
   if (!__BROWSER__ && sourceMap) {
     // lazy require source-map implementation, only in non-browser builds
-    context.map = new (loadDep('source-map')).SourceMapGenerator()
+    context.map = new SourceMapGenerator()
     context.map!.setSourceContent(filename, context.source)
   }
 

packages/compiler-core/src/utils.ts

@@ -30,8 +30,9 @@ import {
   KEEP_ALIVE,
   BASE_TRANSITION
 } from './runtimeHelpers'
-import { isString, isFunction, isObject, hyphenate } from '@vue/shared'
+import { isString, isObject, hyphenate } from '@vue/shared'
 import { parse } from '@babel/parser'
+import { walk } from 'estree-walker'
 import { Node } from '@babel/types'
 
 export const isBuiltInType = (tag: string, expected: string): boolean =>
@@ -49,31 +50,16 @@ export function isCoreComponent(tag: string): symbol | void {
   }
 }
 
-// cache node requires
-// lazy require dependencies so that they don't end up in rollup's dep graph
-// and thus can be tree-shaken in browser builds.
-let _parse: typeof parse
-let _walk: any
-
-export function loadDep(name: string) {
-  if (!__BROWSER__ && typeof process !== 'undefined' && isFunction(require)) {
-    return require(name)
-  } else {
-    // This is only used when we are building a dev-only build of the compiler
-    // which runs in the browser but also uses Node deps.
-    return (window as any)._deps[name]
-  }
-}
-
 export const parseJS: typeof parse = (code, options) => {
-  assert(
-    !__BROWSER__,
-    `Expression AST analysis can only be performed in non-browser builds.`
-  )
-  if (!_parse) {
-    _parse = loadDep('@babel/parser').parse
+  if (__BROWSER__) {
+    assert(
+      !__BROWSER__,
+      `Expression AST analysis can only be performed in non-browser builds.`
+    )
+    return null as any
+  } else {
+    return parse(code, options)
   }
-  return _parse(code, options)
 }
 
 interface Walker {
@@ -82,12 +68,15 @@ interface Walker {
 }
 
 export const walkJS = (ast: Node, walker: Walker) => {
-  assert(
-    !__BROWSER__,
-    `Expression AST analysis can only be performed in non-browser builds.`
-  )
-  const walk = _walk || (_walk = loadDep('estree-walker').walk)
-  return walk(ast, walker)
+  if (__BROWSER__) {
+    assert(
+      !__BROWSER__,
+      `Expression AST analysis can only be performed in non-browser builds.`
+    )
+    return null as any
+  } else {
+    return (walk as any)(ast, walker)
+  }
 }
 
 const nonIdentifierRE = /^\d|[^\$\w]/