eslint-plugin-chrome-extensions 2.3.0 → 2.4.1

package/src/rules/valid-version-field.js

@@ -30,14 +30,16 @@ export default {
                 },
                 timezone: {
                     type: 'string',
-                    description: 'IANA timezone for determining the current date (when omitted, only year and month are checked)'
+                    description: 'IANA timezone for determining the current date '
+                               + '(when omitted, only year and month are checked)'
                 }
             },
             additionalProperties: false
         }],
         defaultOptions: [{}],
         messages: {
-            invalidVersion: 'Invalid manifest `version`: must be a valid {{versionType}} version (expected format: {{example}}).',
+            invalidVersion: 'Invalid manifest `version`: must be a valid {{versionType}} version '
+                          + '(expected format: {{example}}).',
             futureVersion: 'Invalid manifest `version`: date cannot be in the future.'
         }
     },

package/src/utils.js

@@ -10,17 +10,6 @@ export function bundleRules(rules, predicate, pluginName) {
         .map(([ruleId]) => [`${pluginName}/${ruleId}`, 'error']))
 }
 
-export function initBundleDeprecation(plugin, [ruleId, bundleNames], pluginName) {
-    for (const bundleName of bundleNames) {
-        /* eslint-disable security/detect-object-injection */
-        if (plugin.configs[bundleName])
-            plugin.configs[bundleName].rules[`${pluginName}/${ruleId}`] = 'error'
-        /* eslint-enable security/detect-object-injection */
-        if (process.env.DEBUG_MODE)
-            console.warn(`[${pluginName}] '${ruleId}' will be removed from '${bundleName}' in a future major release`)
-    }
-}
-
 export function findExtensionRoot(context) {
     let dir = path.dirname(getFilename(context))
     while (dir) {
@@ -55,6 +44,17 @@ export function getMemberPath(node) {
     return parts.join('.')
 }
 
+export function initBundleDeprecation(plugin, [ruleId, bundleNames], pluginName) {
+    for (const bundleName of bundleNames) {
+        /* eslint-disable security/detect-object-injection */
+        if (plugin.configs[bundleName])
+            plugin.configs[bundleName].rules[`${pluginName}/${ruleId}`] = 'error'
+        /* eslint-enable security/detect-object-injection */
+        if (process.env.DEBUG_MODE)
+            console.warn(`[${pluginName}] '${ruleId}' will be removed from '${bundleName}' in a future major release`)
+    }
+}
+
 export function isContentScriptFile(context, manifest) {
     const scripts = (manifest?.content_scripts || []).flatMap(cs => cs.js || [])
     if (!scripts.length) return false
@@ -87,8 +87,7 @@ export function readManifest(context) {
     if (!root) return null
     if (manifestCache.has(root)) return manifestCache.get(root)
     let parsed
-    try {
-        // eslint-disable-next-line security/detect-non-literal-fs-filename
+    try { // eslint-disable-next-line security/detect-non-literal-fs-filename
         parsed = JSON.parse(fs.readFileSync(path.join(root, 'manifest.json'), 'utf8'))
     } catch { parsed = null }
     manifestCache.set(root, parsed)