Add support for multiple locale directories (#114)

* WIP: Add support for multiple locale directories

* Add testcases

Author: ota-meshi

docs/started.md

@@ -44,6 +44,17 @@ module.export = {
       //   pattern: './path/to/locales/*.{json,json5,yaml,yml}', // extension is glob formatting!
       //   localeKey: 'file' // or 'key'
       // }
+      // or
+      // localeDir: [
+      //   {
+      //     pattern: './path/to/locales1/*.{json,json5,yaml,yml}',
+      //     localeKey: 'file' // or 'key'
+      //   },
+      //   {
+      //     pattern: './path/to/locales2/*.{json,json5,yaml,yml}',
+      //     localeKey: 'file' // or 'key'
+      //   },
+      // ]
     }
   }
 }
@@ -53,13 +64,14 @@ See [the rule list](../rules/)
 
 ### `settings['vue-i18n']`
 
-- `localeDir` ... You can specify a string or an object.
+- `localeDir` ... You can specify a string or an object or an array.
   - String option ... A glob for specifying files that store localization messages of project.
   - Object option
     - `pattern` (`string`) ... A glob for specifying files that store localization messages of project.
     - `localeKey` (`'file' | 'key'`) ... Specifies how to determine the locale for localization messages.
       - `'file'` ... Determine the locale name from the filename. The resource file should only contain messages for that locale. Use this option if you use `vue-cli-plugin-i18n`. This option is also used when String option is specified.
       - `'key'` ...  Determine the locale name from the root key name of the file contents. The value of that key should only contain messages for that locale. Used when the resource file is in the format given to the `messages` option of the `VueI18n` constructor option.
+  - Array option ... An array of String option and Object option. Useful if you have multiple locale directories.
 
 ### Running ESLint from command line
 

lib/types/settings.ts

@@ -12,6 +12,7 @@ export type LocaleKeyType = 'file' | 'key'
 export type SettingsVueI18nLocaleDir =
   | SettingsVueI18nLocaleDirGlob
   | SettingsVueI18nLocaleDirObject
+  | (SettingsVueI18nLocaleDirGlob | SettingsVueI18nLocaleDirObject)[]
 /**
  * A glob for specifying files that store localization messages of project.
  */

lib/utils/index.ts

@@ -17,9 +17,15 @@ import type {
   TemplateListener,
   RuleListener,
   LocaleKeyType,
-  SettingsVueI18nLocaleDir
+  SettingsVueI18nLocaleDir,
+  SettingsVueI18nLocaleDirObject,
+  SettingsVueI18nLocaleDirGlob
 } from '../types'
 
+interface LocaleFiles {
+  files: string[]
+  localeKey: LocaleKeyType
+}
 const UNEXPECTED_ERROR_LOCATION = { line: 1, column: 0 }
 /**
  * Register the given visitor to parser services.
@@ -48,16 +54,24 @@ export function defineTemplateBodyVisitor(
   )
 }
 
-/**
- * @param {string[]} files
- * @param {LocaleKeyType} localeKey
- * @returns {LocaleMessage[]}
- */
-function loadLocaleMessages(files: string[], localeKey: LocaleKeyType) {
-  return files.map(file => {
-    const fullpath = resolve(process.cwd(), file)
-    return new FileLocaleMessage({ fullpath, localeKey })
-  })
+function loadLocaleMessages(
+  localeFilesList: LocaleFiles[],
+  cwd: string
+): FileLocaleMessage[] {
+  const results: FileLocaleMessage[] = []
+  const checkDupeMap: { [file: string]: LocaleKeyType[] } = {}
+  for (const { files, localeKey } of localeFilesList) {
+    for (const file of files) {
+      const localeKeys = checkDupeMap[file] || (checkDupeMap[file] = [])
+      if (localeKeys.includes(localeKey)) {
+        continue
+      }
+      localeKeys.push(localeKey)
+      const fullpath = resolve(cwd, file)
+      results.push(new FileLocaleMessage({ fullpath, localeKey }))
+    }
+  }
+  return results
 }
 
 /** @type {Set<RuleContext>} */
@@ -105,34 +119,49 @@ export function getLocaleMessages(context: RuleContext): LocaleMessages {
 class LocaleDirLocaleMessagesCache {
   private _targetFilesLoader: CacheLoader<[string, string], string[]>
   private _loadLocaleMessages: (
-    files: string[],
-    localeKey: LocaleKeyType,
+    localeFilesList: LocaleFiles[],
     cwd: string
   ) => FileLocaleMessage[]
   constructor() {
     this._targetFilesLoader = new CacheLoader(pattern => glob.sync(pattern))
 
-    this._loadLocaleMessages = defineCacheFunction((files, localeKey) => {
-      return loadLocaleMessages(files, localeKey)
-    })
+    this._loadLocaleMessages = defineCacheFunction(
+      (localeFilesList: LocaleFiles[], cwd) => {
+        return loadLocaleMessages(localeFilesList, cwd)
+      }
+    )
   }
   /**
    * @param {SettingsVueI18nLocaleDir} localeDir
    * @returns {LocaleMessage[]}
    */
   getLocaleMessagesFromLocaleDir(localeDir: SettingsVueI18nLocaleDir) {
     const cwd = process.cwd()
+    let localeFilesList: LocaleFiles[]
+    if (Array.isArray(localeDir)) {
+      localeFilesList = localeDir.map(dir => this._toLocaleFiles(dir, cwd))
+    } else {
+      localeFilesList = [this._toLocaleFiles(localeDir, cwd)]
+    }
+    return this._loadLocaleMessages(localeFilesList, cwd)
+  }
+
+  private _toLocaleFiles(
+    localeDir: SettingsVueI18nLocaleDirGlob | SettingsVueI18nLocaleDirObject,
+    cwd: string
+  ): LocaleFiles {
     const targetFilesLoader = this._targetFilesLoader
-    let files
-    let localeKey: LocaleKeyType
     if (typeof localeDir === 'string') {
-      files = targetFilesLoader.get(localeDir, cwd)
-      localeKey = 'file'
+      return {
+        files: targetFilesLoader.get(localeDir, cwd),
+        localeKey: 'file'
+      }
     } else {
-      files = targetFilesLoader.get(localeDir.pattern, cwd)
-      localeKey = String(localeDir.localeKey ?? 'file') as LocaleKeyType
+      return {
+        files: targetFilesLoader.get(localeDir.pattern, cwd),
+        localeKey: String(localeDir.localeKey ?? 'file') as LocaleKeyType
+      }
     }
-    return this._loadLocaleMessages(files, localeKey, cwd)
   }
 }
 

tests/fixtures/no-duplicate-keys-in-locale/invalid/multiple-locales/locales1/en.1.json

@@ -0,0 +1,10 @@
+{
+  "hello": "hello",
+  "messages": {
+    "foo": "foo",
+    "bar": "bar",
+    "baz": "baz",
+    "dupe": "dupe"
+  },
+  "dupe": "dupe"
+}

tests/fixtures/no-duplicate-keys-in-locale/invalid/multiple-locales/locales2/en.2.json

@@ -0,0 +1,10 @@
+{
+  "good-bye": "good bye",
+  "messages": {
+    "qux": "qux",
+    "quux": "quux",
+    "corge": "corge",
+    "dupe": "dupe"
+  },
+  "dupe": "dupe"
+}

tests/fixtures/no-duplicate-keys-in-locale/invalid/multiple-locales/locales3/index.1.json

@@ -0,0 +1,11 @@
+{
+  "ja": {
+    "hello": "こんにちは",
+    "messages": {
+      "foo": "foo",
+      "bar": "bar",
+      "baz": "baz"
+    },
+    "dupe": "dupe"
+  }
+}

tests/fixtures/no-duplicate-keys-in-locale/invalid/multiple-locales/locales3/index.2.yaml

@@ -0,0 +1,8 @@
+'ja':
+  'good-bye': 'さようなら'
+  'messages':
+    'qux': 'qux'
+    'quux': 'quux'
+    'corge': 'corge'
+    'dupe': 'dupe'
+  'dupe': 'dupe'

tests/fixtures/no-duplicate-keys-in-locale/invalid/multiple-locales/src/App.vue

@@ -0,0 +1,33 @@
+<i18n lang="yaml">
+en:
+  block: block
+  nest:
+    foo
+ja:
+  block: block
+  nest:
+    foo: bar
+</i18n>
+<i18n lang="yaml">
+en:
+  nest:
+    foo: bar
+ja:
+  dupe: dupe
+</i18n>
+<i18n lang="json">
+{
+  "en": {
+    "json-dupe": "dupe",
+    "nest": {
+      "json-dupe": "dupe"
+    },
+    "nest": {
+      "json-dupe": "dupe"
+    },
+    "json-dupe": "dupe"
+  }
+}
+</i18n>
+<template></template>
+<script></script>

tests/fixtures/no-duplicate-keys-in-locale/valid/multiple-locales/locales1/en.1.json

@@ -0,0 +1,8 @@
+{
+  "hello": "hello",
+  "messages": {
+    "foo": "foo",
+    "bar": "bar",
+    "baz": "baz"
+  }
+}

tests/fixtures/no-duplicate-keys-in-locale/valid/multiple-locales/locales2/en.2.json

@@ -0,0 +1,8 @@
+{
+  "good-bye": "good bye",
+  "messages": {
+    "qux": "qux",
+    "quux": "quux",
+    "corge": "corge"
+  }
+}