fix: bring bug fixes from v9.2.0-beta.11 (#720)

* fix: Maximum call stack size (#713)

* fix: vue-i18n components scope resolving in Legacy API mode (#715)

Co-authored-by: Fabian Kranewitter <[email protected]>

Author: kazupon

packages/message-compiler/src/tokenizer.ts

@@ -438,45 +438,40 @@ export function createTokenizer(
   }
 
   function readText(scnr: Scanner): string {
-    const fn = (buf: string): string => {
+    let buf = ''
+    while (true) {
       const ch = scnr.currentChar()
       if (
         ch === TokenChars.BraceLeft ||
         ch === TokenChars.BraceRight ||
         ch === TokenChars.LinkedAlias ||
+        ch === TokenChars.Pipe ||
         !ch
       ) {
-        return buf
+        break
       } else if (ch === TokenChars.Modulo) {
         if (isTextStart(scnr)) {
           buf += ch
           scnr.next()
-          return fn(buf)
         } else {
-          return buf
+          break
         }
-      } else if (ch === TokenChars.Pipe) {
-        return buf
       } else if (ch === SPACE || ch === NEW_LINE) {
         if (isTextStart(scnr)) {
           buf += ch
           scnr.next()
-          return fn(buf)
         } else if (isPluralStart(scnr)) {
-          return buf
+          break
         } else {
           buf += ch
           scnr.next()
-          return fn(buf)
         }
       } else {
         buf += ch
         scnr.next()
-        return fn(buf)
       }
     }
-
-    return fn('')
+    return buf
   }
 
   function readNamedIdentifier(scnr: Scanner): string {

packages/message-compiler/test/fixtures/20_newsgroups_alt_atheism_51060.txt

@@ -1,6 +1,8 @@
 import { parse } from '../src/tokenizer'
 import { TokenizeOptions } from '../src/options'
 import { CompileError } from '../src/errors'
+import path from 'path'
+import { promises as fs } from 'fs'
 
 test('token analysis', () => {
   const cases = [
@@ -98,3 +100,20 @@ test('token analysis', () => {
     }
   }
 })
+
+describe('edge cases', () => {
+  test('long text', async () => {
+    const data = await fs.readFile(
+      path.join(__dirname, './fixtures/20_newsgroups_alt_atheism_51060.txt'),
+      'utf8'
+    )
+    let err = null
+    try {
+      parse(data)
+    } catch (e) {
+      console.error(e)
+      err = e
+    }
+    expect(err).toEqual(null)
+  })
+})

packages/message-compiler/test/tokenizer.test.ts

@@ -78,7 +78,8 @@ export const DatetimeFormat = {
   setup(props: DatetimeFormatProps, context: SetupContext): RenderFunction {
     const i18n =
       props.i18n ||
-      (useI18n({ useScope: 'parent' }) as Composer & ComposerInternal)
+      (useI18n({ useScope: 'parent', __useComponent: true }) as Composer &
+        ComposerInternal)
 
     return renderFormatter<
       FormattableProps<number | Date, Intl.DateTimeFormatOptions>,

packages/vue-i18n/src/components/DatetimeFormat.ts

@@ -73,7 +73,8 @@ export const NumberFormat = {
   setup(props: NumberFormatProps, context: SetupContext): RenderFunction {
     const i18n =
       props.i18n ||
-      (useI18n({ useScope: 'parent' }) as Composer & ComposerInternal)
+      (useI18n({ useScope: 'parent', __useComponent: true }) as Composer &
+        ComposerInternal)
 
     return renderFormatter<
       FormattableProps<number, Intl.NumberFormatOptions>,

packages/vue-i18n/src/components/NumberFormat.ts

@@ -100,7 +100,10 @@ export const Translation = {
     const { slots, attrs } = context
     const i18n =
       props.i18n ||
-      (useI18n({ useScope: props.scope }) as Composer & ComposerInternal)
+      (useI18n({
+        useScope: props.scope as 'global' | 'parent',
+        __useComponent: true
+      }) as Composer & ComposerInternal)
     const keys = Object.keys(slots).filter(key => key !== '_')
 
     return (): VNodeChild => {

packages/vue-i18n/src/components/Translation.ts

@@ -98,6 +98,7 @@ export const EnableEmitter = makeSymbol('__enableEmitter')
 export const DisableEmitter = makeSymbol('__disableEmitter')
 export const SetPluralRulesSymbol = makeSymbol('__setPluralRules')
 export const DevToolsMetaSymbol = makeSymbol('__intlifyMeta')
+export const InejctWithOption = makeSymbol('__injectWithOption')
 
 /** @VueI18nComposition */
 export type VueMessageType = string | VNode
@@ -326,6 +327,7 @@ export interface ComposerInternalOptions<
   __i18n?: CustomBlocks<Message>
   __i18nGlobal?: CustomBlocks<Message>
   __root?: Composer<Messages, DateTimeFormats, NumberFormats, Message>
+  __injectWithOption?: boolean
 }
 
 /**
@@ -1862,7 +1864,8 @@ export function createComposer<
     [TransrateVNodeSymbol]: transrateVNode,
     [NumberPartsSymbol]: numberParts,
     [DatetimePartsSymbol]: datetimeParts,
-    [SetPluralRulesSymbol]: setPluralRules
+    [SetPluralRulesSymbol]: setPluralRules,
+    [InejctWithOption]: (options as any).__injectWithOption // eslint-disable-line @typescript-eslint/no-explicit-any
   }
 
   // for vue-devtools timeline event

packages/vue-i18n/src/composer.ts

@@ -19,7 +19,8 @@ import {
   getLocaleMessages,
   createComposer,
   EnableEmitter,
-  DisableEmitter
+  DisableEmitter,
+  InejctWithOption
 } from './composer'
 import { createVueI18n } from './legacy'
 import { I18nWarnCodes, getWarnMessage } from './warnings'
@@ -598,7 +599,8 @@ export function useI18n<
   }
 
   if (scope === 'parent') {
-    let composer = getComposer(i18n, instance)
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    let composer = getComposer(i18n, instance, (options as any).__useComponent)
     if (composer == null) {
       if (__DEV__) {
         warn(getWarnMessage(I18nWarnCodes.NOT_FOUND_PARENT_SCOPE))
@@ -666,7 +668,8 @@ function getComposer<
   Legacy extends boolean
 >(
   i18n: I18n<Messages, DateTimeFormats, NumberFormats, Legacy>,
-  target: ComponentInternalInstance
+  target: ComponentInternalInstance,
+  useComponent = false
 ): Composer<Messages, DateTimeFormats, NumberFormats> | null {
   let composer: Composer<Messages, DateTimeFormats, NumberFormats> | null = null
   const root = target.root
@@ -696,6 +699,10 @@ function getComposer<
           VueI18nInternal<Messages, DateTimeFormats, NumberFormats>)
           .__composer as Composer<Messages, DateTimeFormats, NumberFormats>
       }
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any
+      if (useComponent && !(composer as any)[InejctWithOption]) {
+        composer = null
+      }
     }
     if (composer != null) {
       break

packages/vue-i18n/src/i18n.ts

@@ -1113,7 +1113,7 @@ function convertComposerOptions<
       return messages
     }, (messages || {}) as LocaleMessages<VueMessageType>) as typeof options.messages
   }
-  const { __i18n, __root } = options
+  const { __i18n, __root, __injectWithOption } = options
 
   const datetimeFormats = options.datetimeFormats
   const numberFormats = options.numberFormats
@@ -1138,7 +1138,8 @@ function convertComposerOptions<
     escapeParameter,
     inheritLocale,
     __i18n,
-    __root
+    __root,
+    __injectWithOption
   }
 }
 

packages/vue-i18n/src/legacy.ts

@@ -52,6 +52,7 @@ export function defineMixin<Messages, DateTimeFormats, NumberFormats>(
         if (this === this.$root) {
           this.$i18n = mergeToRoot(vuei18n, optionsI18n)
         } else {
+          optionsI18n.__injectWithOption = true
           this.$i18n = createVueI18n(optionsI18n)
         }
       } else if (options.__i18n) {
@@ -60,6 +61,7 @@ export function defineMixin<Messages, DateTimeFormats, NumberFormats>(
         } else {
           this.$i18n = createVueI18n({
             __i18n: (options as ComposerInternalOptions<Messages>).__i18n,
+            __injectWithOption: true,
             __root: composer
           } as VueI18nOptions)
         }