eslint-plugin-cvsdk-rules 8.2.4 → 8.2.5-preview-no-audio-fix

package/config/file.ts

@@ -1 +1 @@
-// See: https://typescript-eslint.io/packages/rule-tester/#type-aware-testing
+// See: https://typescript-eslint.io/packages/rule-tester/#type-aware-testing

package/config/tsconfig.json

@@ -1,11 +1,11 @@
-{
-  // https://typescript-eslint.io/packages/rule-tester/#type-aware-testing
-  "compilerOptions": {
-    // Otherwise TS will complain it'll override the base project's output
-    "outDir": "./",
-    "strict": true
-  },
-  "exclude": [],
-  // See: https://typescript-eslint.io/packages/rule-tester/#type-aware-testing
-  "include": ["file.ts"]
+{
+  // https://typescript-eslint.io/packages/rule-tester/#type-aware-testing
+  "compilerOptions": {
+    // Otherwise TS will complain it'll override the base project's output
+    "outDir": "./",
+    "strict": true
+  },
+  "exclude": [],
+  // See: https://typescript-eslint.io/packages/rule-tester/#type-aware-testing
+  "include": ["file.ts"]
 }

package/index.js

@@ -1,6 +1,6 @@
-module.exports = {
-    rules: {
-        'prefer-self-over-window': require('./rules/prefer-self-over-window'),
-        'prefer-txml-over-domparser': require('./rules/prefer-txml-over-domparser'),
-    },
-};
+module.exports = {
+    rules: {
+        'prefer-self-over-window': require('./rules/prefer-self-over-window'),
+        'prefer-txml-over-domparser': require('./rules/prefer-txml-over-domparser'),
+    },
+};

package/jest.config.js

@@ -1,2 +1,2 @@
-{
-}
+{
+}

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "eslint-plugin-cvsdk-rules",
-    "version": "8.2.4",
+    "version": "8.2.5-preview-no-audio-fix",
     "main": "index.js",
     "scripts": {
         "test": "jest",
@@ -13,5 +13,5 @@
         "@typescript-eslint/rule-tester": "6.21.0",
         "jest": "29.7.0"
     },
-    "gitHead": "f195a9203ccf184a1d632fc6ee2eaa0297d9b198"
+    "gitHead": "be9dd3e0acd90c08777a0a0ed1edf9c248359001"
 }

package/rules/prefer-self-over-window.js

@@ -1,41 +1,41 @@
-const { ESLintUtils } = require('@typescript-eslint/utils');
-
-const meta = {
-    docs: {
-        description: 'Disallow window access.',
-    },
-    messages: {
-        forbiddenWindowAccess: '`window` cannot be accessed inside some containers, we prefer using `self` as it has more compatibility.',
-    },
-    schema: [], // no options
-    type: 'problem',
-    fixable: 'code',
-};
-
-function create(context) {
-    const parserServices = ESLintUtils.getParserServices(context);
-
-    function reportWindowError(node) {
-        if (node.parent.type !== 'VariableDeclarator') {
-            const nodeType = parserServices.getTypeAtLocation(node);
-            if (Object.prototype.hasOwnProperty.call(nodeType, 'aliasSymbol')) {
-                context.report({
-                    messageId: 'forbiddenWindowAccess',
-                    node,
-                    fix: function (fixer) {
-                        return fixer.replaceText(node, 'self');
-                    },
-                });
-            }
-        }
-    }
-
-    return {
-        "Identifier[name='window']": reportWindowError,
-    };
-}
-
-module.exports = {
-    create,
-    meta,
-};
+const { ESLintUtils } = require('@typescript-eslint/utils');
+
+const meta = {
+    docs: {
+        description: 'Disallow window access.',
+    },
+    messages: {
+        forbiddenWindowAccess: '`window` cannot be accessed inside some containers, we prefer using `self` as it has more compatibility.',
+    },
+    schema: [], // no options
+    type: 'problem',
+    fixable: 'code',
+};
+
+function create(context) {
+    const parserServices = ESLintUtils.getParserServices(context);
+
+    function reportWindowError(node) {
+        if (node.parent.type !== 'VariableDeclarator') {
+            const nodeType = parserServices.getTypeAtLocation(node);
+            if (Object.prototype.hasOwnProperty.call(nodeType, 'aliasSymbol')) {
+                context.report({
+                    messageId: 'forbiddenWindowAccess',
+                    node,
+                    fix: function (fixer) {
+                        return fixer.replaceText(node, 'self');
+                    },
+                });
+            }
+        }
+    }
+
+    return {
+        "Identifier[name='window']": reportWindowError,
+    };
+}
+
+module.exports = {
+    create,
+    meta,
+};

package/rules/prefer-self-over-window.spec.js

@@ -1,40 +1,40 @@
-const { RuleTester } = require('@typescript-eslint/rule-tester');
-const path = require('path');
-
-const rule = require('./prefer-self-over-window');
-
-const ruleName = 'prefer-self-over-window';
-
-// https://typescript-eslint.io/packages/rule-tester/#type-aware-testing
-const ruleTester = new RuleTester({
-    parser: require.resolve('@typescript-eslint/parser'),
-    parserOptions: {
-        project: true,
-        tsconfigRootDir: path.join(__dirname, '../config'),
-    },
-});
-
-describe('Disallow window access', () => {
-    ruleTester.run(ruleName, rule, {
-        invalid: [
-            {
-                code: 'window;',
-                output: 'self;',
-                errors: [{ messageId: 'forbiddenWindowAccess' }],
-            },
-            {
-                code: 'declare let window: any; window;',
-                output: 'declare let window: any; self;',
-                errors: [{ messageId: 'forbiddenWindowAccess' }],
-            },
-        ],
-        valid: [
-            'const window = 1;',
-            'function someFunc() { const window = 1; window; }',
-            'function someFunc() { let window: number; window; };',
-            'function someFunc() { let window: unknown; window; };',
-            'function someFunc() { let window: any; window; };',
-            'function someFunc() { let window; window; };',
-        ],
-    });
-});
+const { RuleTester } = require('@typescript-eslint/rule-tester');
+const path = require('path');
+
+const rule = require('./prefer-self-over-window');
+
+const ruleName = 'prefer-self-over-window';
+
+// https://typescript-eslint.io/packages/rule-tester/#type-aware-testing
+const ruleTester = new RuleTester({
+    parser: require.resolve('@typescript-eslint/parser'),
+    parserOptions: {
+        project: true,
+        tsconfigRootDir: path.join(__dirname, '../config'),
+    },
+});
+
+describe('Disallow window access', () => {
+    ruleTester.run(ruleName, rule, {
+        invalid: [
+            {
+                code: 'window;',
+                output: 'self;',
+                errors: [{ messageId: 'forbiddenWindowAccess' }],
+            },
+            {
+                code: 'declare let window: any; window;',
+                output: 'declare let window: any; self;',
+                errors: [{ messageId: 'forbiddenWindowAccess' }],
+            },
+        ],
+        valid: [
+            'const window = 1;',
+            'function someFunc() { const window = 1; window; }',
+            'function someFunc() { let window: number; window; };',
+            'function someFunc() { let window: unknown; window; };',
+            'function someFunc() { let window: any; window; };',
+            'function someFunc() { let window; window; };',
+        ],
+    });
+});

package/rules/prefer-txml-over-domparser.js

@@ -1,26 +1,26 @@
-const meta = {
-    docs: 'Disallow DOMParser usage.',
-    messages: {
-        forbiddenDomParserUsage: `Prefer using txml (https://www.npmjs.com/package/txml) over DOMParser as DOMParser isn't supported on ION.`,
-    },
-    schema: [],
-    type: 'problem',
-};
-
-function create(context) {
-    return {
-        Identifier(node) {
-            if (node.type === 'Identifier' && node.name === 'DOMParser' && node.parent.type === 'NewExpression') {
-                context.report({
-                    node,
-                    messageId: 'forbiddenDomParserUsage',
-                });
-            }
-        },
-    };
-}
-
-module.exports = {
-    create,
-    meta,
-};
+const meta = {
+    docs: 'Disallow DOMParser usage.',
+    messages: {
+        forbiddenDomParserUsage: `Prefer using txml (https://www.npmjs.com/package/txml) over DOMParser as DOMParser isn't supported on ION.`,
+    },
+    schema: [],
+    type: 'problem',
+};
+
+function create(context) {
+    return {
+        Identifier(node) {
+            if (node.type === 'Identifier' && node.name === 'DOMParser' && node.parent.type === 'NewExpression') {
+                context.report({
+                    node,
+                    messageId: 'forbiddenDomParserUsage',
+                });
+            }
+        },
+    };
+}
+
+module.exports = {
+    create,
+    meta,
+};