]> git.ipfire.org Git - thirdparty/vuejs/core.git/commitdiff
refactor: remove prefixIdentifier and mode usage in compiler-vapor
authorEvan You <evan@vuejs.org>
Thu, 30 Jan 2025 12:10:40 +0000 (20:10 +0800)
committerEvan You <evan@vuejs.org>
Thu, 30 Jan 2025 12:10:40 +0000 (20:10 +0800)
vapor compilation always assume module mode + prefixIdentifier: true

packages/compiler-vapor/__tests__/transforms/vOnce.spec.ts
packages/compiler-vapor/src/compile.ts
packages/compiler-vapor/src/generate.ts
packages/compiler-vapor/src/generators/expression.ts
packages/compiler-vapor/src/transform.ts

index 124e3369497e8394e053f5ec63e23b7d6bc4838f..1f40424e5f7ffca59f038dc6eecc785c128d707e 100644 (file)
@@ -3,7 +3,7 @@ import { IRNodeTypes } from '../../src'
 import { getBaseTransformPreset } from '../../src/compile'
 import { makeCompile } from './_utils'
 
-const [nodeTransforms, directiveTransforms] = getBaseTransformPreset(true)
+const [nodeTransforms, directiveTransforms] = getBaseTransformPreset()
 const compileWithOnce = makeCompile({
   nodeTransforms,
   directiveTransforms,
index bc51e5c2c326cb50a51506982f7f725cb72cd390..f84eafcbe0bfc9b0fe3e2c98071ca7f56f7ad5a7 100644 (file)
@@ -1,9 +1,6 @@
 import {
   type CompilerOptions as BaseCompilerOptions,
-  ErrorCodes,
   type RootNode,
-  createCompilerError,
-  defaultOnError,
   parse,
 } from '@vue/compiler-dom'
 import { extend, isString } from '@vue/shared'
@@ -38,20 +35,9 @@ export function compile(
   source: string | RootNode,
   options: CompilerOptions = {},
 ): VaporCodegenResult {
-  const onError = options.onError || defaultOnError
-  const isModuleMode = options.mode === 'module'
-  const prefixIdentifiers = options.prefixIdentifiers === true || isModuleMode
-
-  if (options.scopeId && !isModuleMode) {
-    onError(createCompilerError(ErrorCodes.X_SCOPE_ID_NOT_SUPPORTED))
-  }
-
-  const resolvedOptions = extend({}, options, {
-    prefixIdentifiers,
-  })
+  const resolvedOptions = extend({}, options)
   const ast = isString(source) ? parse(source, resolvedOptions) : source
-  const [nodeTransforms, directiveTransforms] =
-    getBaseTransformPreset(prefixIdentifiers)
+  const [nodeTransforms, directiveTransforms] = getBaseTransformPreset()
 
   if (options.isTS) {
     const { expressionPlugins } = options
@@ -87,9 +73,7 @@ export type TransformPreset = [
   Record<string, DirectiveTransform>,
 ]
 
-export function getBaseTransformPreset(
-  prefixIdentifiers?: boolean,
-): TransformPreset {
+export function getBaseTransformPreset(): TransformPreset {
   return [
     [
       transformVOnce,
index 012ea55d5866f3511fcb1c8c65531757d426f750..319aac9e48289bf4f53f4d08a2f4bcc9d41dfe9e 100644 (file)
@@ -68,8 +68,8 @@ export class CodegenContext {
     options: CodegenOptions,
   ) {
     const defaultOptions: Required<CodegenOptions> = {
-      mode: 'function', // TODO
-      prefixIdentifiers: options.mode === 'module',
+      mode: 'module',
+      prefixIdentifiers: true,
       sourceMap: false,
       filename: `template.vue.html`,
       scopeId: null,
index 8e71458337e7d950c9d02c49af1dff91877496df..6ece58fa80f9b31f504ba9574f8276ade6794a7e 100644 (file)
@@ -22,7 +22,6 @@ export function genExpression(
   context: CodegenContext,
   assignment?: string,
 ): CodeFragment[] {
-  const { prefixIdentifiers } = context.options
   const { content, ast, isStatic, loc } = node
 
   if (isStatic) {
@@ -30,7 +29,6 @@ export function genExpression(
   }
 
   if (
-    !prefixIdentifiers ||
     !node.content.trim() ||
     // there was a parsing error
     ast === false ||
index f8ff395ede5df9cc8d10655cef81a0c17c4adf4c..1cace2e47b41bc8612c113025f3d44ed48159879 100644 (file)
@@ -194,7 +194,7 @@ export class TransformContext<T extends AllNode = AllNode> {
 
 const defaultOptions = {
   filename: '',
-  prefixIdentifiers: false,
+  prefixIdentifiers: true,
   hoistStatic: false,
   hmr: false,
   cacheHandlers: false,