@discord/eslint-plugin-discord-intl 0.27.3 → 0.27.4

package/lib/native-validation.js

@@ -4,30 +4,38 @@
 /** @typedef {import('eslint').Rule.ReportDescriptorLocation} ReportDescriptorLocation */
 /** @typedef {import('eslint').AST.SourceLocation} SourceLocation */
 /** @typedef {import('eslint').SourceCode} SourceCode */
-/** @typedef {import("@discord/intl-loader-core/types").IntlDiagnostic} IntlDiagnostic */
+/** @typedef {import("@discord/intl-message-database").IntlDiagnostic} IntlDiagnostic */
 
 const crypto = require('node:crypto');
-const {
-  processDefinitionsFile,
-  database,
-  processAllMessagesFiles,
-  findAllMessagesFiles,
-} = require('@discord/intl-loader-core');
+const path = require('node:path');
 const { traverseMessageDefinitions, isDefinitionsFile } = require('./traverse');
-const diagnostics_channel = require('node:diagnostics_channel');
+const {
+  IntlMessagesDatabase,
+  IntlDatabaseInsertStrategy,
+} = require('@discord/intl-message-database');
 /** @type {Map<SourceCode, Record<string, IntlDiagnostic[]>>} */
 const FILE_VALIDATIONS = new Map();
 
-let isInitialized = false;
+/** @type {Map<string, IntlMessagesDatabase>} */
+const PROJECT_DATABASE_MAP = new Map();
 
 /**
  * @param {string} directory
+ * @returns {IntlMessagesDatabase}
  */
 function ensureInitialized(directory) {
-  if (isInitialized) return;
+  const key = path.resolve(directory);
+  const existing = PROJECT_DATABASE_MAP.get(key);
+  if (existing != null) return existing;
+
+  const database = new IntlMessagesDatabase();
+  database.processAllMessagesFiles(
+    database.findAllMessagesFiles([directory], 'en-US'),
+    IntlDatabaseInsertStrategy.Create,
+  );
 
-  processAllMessagesFiles(findAllMessagesFiles([directory]));
-  isInitialized = true;
+  PROJECT_DATABASE_MAP.set(key, database);
+  return database;
 }
 
 /**
@@ -38,12 +46,13 @@ function ensureInitialized(directory) {
  * validations, allowing them to be reused across multiple rules without wasting work validating
  * the same messages multiple times.
  *
+ * @param {IntlMessagesDatabase} database
  * @param {SourceCode} sourceCode
  * @param {string} fileName
  * @param {string} content
  * @return {Record<string, IntlDiagnostic[]>}
  */
-function processAndValidateNative(sourceCode, fileName, content) {
+function processAndValidateNative(database, sourceCode, fileName, content) {
   const fileKey = crypto.hash('sha1', content);
   const existing = FILE_VALIDATIONS.get(sourceCode);
   if (existing != null) return existing;
@@ -52,30 +61,31 @@ function processAndValidateNative(sourceCode, fileName, content) {
   // native database that this is a JS message definition _somehow_, so we'll make a random
   // name that should be relatively unique.
   const processingFileName = fileName === '<input>' ? `${fileKey}.messages.js` : fileName;
-  const processResult = processDefinitionsFile(processingFileName, content, {
-    processTranslations: false,
-  });
+  const processResult = database.processDefinitionsFileContent(
+    processingFileName,
+    content,
+    'en-US',
+    IntlDatabaseInsertStrategy.Update,
+  );
 
   /** @type {Record<string, IntlDiagnostic[]>} */
   const validations = {};
 
-  if (!processResult.succeeded) {
-    for (const error of processResult.errors) {
-      const key = error.key ?? 'file';
-      (validations[key] ??= []).push({
-        name: 'Processing::' + error.name,
-        description: error.message,
-        key: error.key ?? 'file',
-        file: error.file ?? processingFileName,
-        messageLine: error.line ?? 0,
-        messageCol: error.col ?? 0,
-        start: 0,
-        end: content.length,
-        locale: error.locale ?? 'definition',
-        category: 'correctness',
-        fixes: [],
-      });
-    }
+  for (const error of processResult.errors) {
+    const key = error.key ?? 'file';
+    (validations[key] ??= []).push({
+      name: 'Processing::' + error.name,
+      description: error.message,
+      key: error.key ?? 'file',
+      file: error.file ?? processingFileName,
+      messageLine: error.line ?? 0,
+      messageCol: error.col ?? 0,
+      start: 0,
+      end: 2,
+      locale: error.locale ?? 'definition',
+      category: 'correctness',
+      fixes: [],
+    });
   }
 
   for (const diagnostic of database.validateMessages()) {
@@ -100,8 +110,9 @@ function traverseAndReportMatchingNativeValidations(context, predicate) {
   if (!isDefinitionsFile(context.filename, context.sourceCode.text)) return {};
 
   const projectDirectory = context.settings['intl']?.['projectDirectory'] ?? context.cwd;
-  ensureInitialized(projectDirectory);
+  const database = ensureInitialized(projectDirectory);
   const validations = processAndValidateNative(
+    database,
     context.sourceCode,
     context.filename,
     context.sourceCode.text,

package/lib/traverse.js

@@ -1,4 +1,4 @@
-const { isMessageDefinitionsFile } = require('@discord/intl-loader-core');
+const { isMessageDefinitionsFile } = require('@discord/intl-message-database');
 
 /** @typedef {import('eslint').Rule.NodeParentExtension} NodeParentExtension */
 /** @typedef {import('eslint').SourceCode} SourceCode */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@discord/eslint-plugin-discord-intl",
-  "version": "0.27.3",
+  "version": "0.27.4",
   "license": "MIT",
   "type": "commonjs",
   "description": "ESLint plugin for validating and linting messages and translations using @discord/intl",
@@ -11,7 +11,7 @@
   },
   "dependencies": {
     "@typescript-eslint/scope-manager": "^8.8.1",
-    "@discord/intl-loader-core": "0.27.3"
+    "@discord/intl-message-database": "0.27.4"
   },
   "devDependencies": {
     "@types/eslint": "^8.56.12",