@remotion/eslint-plugin 4.0.251 → 4.0.253

package/dist/index.d.ts

@@ -1,6 +1,5 @@
 declare const _default: {
-    rules: {
-        'no-mp4-import': import("@typescript-eslint/utils/dist/ts-eslint").RuleModule<"NoMP4Import", [], import("@typescript-eslint/utils/dist/ts-eslint").RuleListener>;
+    readonly rules: {
         'warn-native-media-tag': import("@typescript-eslint/utils/dist/ts-eslint").RuleModule<"NoNativeImgTag" | "NoNativeIFrameTag" | "NoNativeAudioTag" | "NoNativeVideoTag", [], import("@typescript-eslint/utils/dist/ts-eslint").RuleListener>;
         'deterministic-randomness': import("@typescript-eslint/utils/dist/ts-eslint").RuleModule<"DeterministicRandomness", [], import("@typescript-eslint/utils/dist/ts-eslint").RuleListener>;
         'no-string-assets': import("@typescript-eslint/utils/dist/ts-eslint").RuleModule<"NoStringAssets", [], import("@typescript-eslint/utils/dist/ts-eslint").RuleListener>;
@@ -14,22 +13,21 @@ declare const _default: {
         'no-background-image': import("@typescript-eslint/utils/dist/ts-eslint").RuleModule<"BackgroundImage", [], import("@typescript-eslint/utils/dist/ts-eslint").RuleListener>;
         'v4-config-import': import("@typescript-eslint/utils/dist/ts-eslint").RuleModule<"ImportConfig", [], import("@typescript-eslint/utils/dist/ts-eslint").RuleListener>;
     };
-    configs: {
-        recommended: {
-            rules: {
-                '@remotion/no-mp4-import': string;
-                '@remotion/warn-native-media-tag': string;
-                '@remotion/deterministic-randomness': string;
-                '@remotion/no-string-assets': string;
-                '@remotion/even-dimensions': string;
-                '@remotion/duration-in-frames': string;
-                '@remotion/from-0': string;
-                '@remotion/volume-callback': string;
-                '@remotion/use-gif-component': string;
-                '@remotion/staticfile-no-relative': string;
-                '@remotion/staticfile-no-remote': string;
-                '@remotion/no-background-image': string;
-                '@remotion/v4-config-import': string;
+    readonly configs: {
+        readonly recommended: {
+            readonly rules: {
+                readonly '@remotion/warn-native-media-tag': "error";
+                readonly '@remotion/deterministic-randomness': "error";
+                readonly '@remotion/no-string-assets': "error";
+                readonly '@remotion/even-dimensions': "error";
+                readonly '@remotion/duration-in-frames': "error";
+                readonly '@remotion/from-0': "error";
+                readonly '@remotion/volume-callback': "error";
+                readonly '@remotion/use-gif-component': "error";
+                readonly '@remotion/staticfile-no-relative': "error";
+                readonly '@remotion/staticfile-no-remote': "error";
+                readonly '@remotion/no-background-image': "error";
+                readonly '@remotion/v4-config-import': "error";
             };
         };
     };

package/dist/index.js

@@ -7,7 +7,6 @@ const even_dimensions_1 = __importDefault(require("./rules/even-dimensions"));
 const no_background_image_1 = __importDefault(require("./rules/no-background-image"));
 const no_duration_frames_infinity_1 = __importDefault(require("./rules/no-duration-frames-infinity"));
 const no_from_0_1 = __importDefault(require("./rules/no-from-0"));
-const no_mp4_import_1 = __importDefault(require("./rules/no-mp4-import"));
 const no_string_assets_1 = __importDefault(require("./rules/no-string-assets"));
 const staticfile_no_relative_1 = __importDefault(require("./rules/staticfile-no-relative"));
 const staticfile_no_remote_1 = __importDefault(require("./rules/staticfile-no-remote"));
@@ -16,7 +15,6 @@ const v4_import_1 = __importDefault(require("./rules/v4-import"));
 const volume_callback_1 = __importDefault(require("./rules/volume-callback"));
 const warn_native_media_tag_1 = __importDefault(require("./rules/warn-native-media-tag"));
 const rules = {
-    'no-mp4-import': no_mp4_import_1.default,
     'warn-native-media-tag': warn_native_media_tag_1.default,
     'deterministic-randomness': deterministic_randomness_1.default,
     'no-string-assets': no_string_assets_1.default,
@@ -35,7 +33,6 @@ module.exports = {
     configs: {
         recommended: {
             rules: {
-                '@remotion/no-mp4-import': 'off',
                 '@remotion/warn-native-media-tag': 'error',
                 '@remotion/deterministic-randomness': 'error',
                 '@remotion/no-string-assets': 'error',

package/dist/rules/missing-import.d.ts

@@ -0,0 +1,24 @@
+declare const _default: {
+    meta: {
+        type: string;
+        fixable: boolean;
+        docs: {
+            description: string;
+            category: string;
+            recommended: boolean;
+            url: string;
+        };
+        schema: {
+            type: string;
+            properties: {
+                imports: {
+                    type: string;
+                    default: {};
+                };
+            };
+            additionalProperties: boolean;
+        }[];
+    };
+    create(context: any): any;
+};
+export = _default;

package/dist/rules/missing-import.js

@@ -0,0 +1,165 @@
+/**
+ * @fileoverview Rule to flag references to undeclared variables.
+ * @author Mark Macdonald
+ */
+'use strict';
+const jsx_1 = require("../util/jsx");
+//------------------------------------------------------------------------------
+// Helpers
+//------------------------------------------------------------------------------
+/**
+ * Checks if the given node is the argument of a typeof operator.
+ * @param {ASTNode} node The AST node being checked.
+ * @returns {boolean} Whether or not the node is the argument of a typeof operator.
+ */
+function hasTypeOfOperator(node) {
+    const { parent } = node;
+    return parent.type === 'UnaryExpression' && parent.operator === 'typeof';
+}
+//------------------------------------------------------------------------------
+// Rule Definition
+//------------------------------------------------------------------------------
+const allowGlobals = false;
+module.exports = {
+    meta: {
+        type: 'problem',
+        fixable: true,
+        docs: {
+            description: 'Automatically imports stuff that you specify',
+            category: 'Variables',
+            recommended: true,
+            url: 'https://eslint.org/docs/rules/no-undef',
+        },
+        schema: [
+            {
+                type: 'object',
+                properties: {
+                    imports: {
+                        type: 'object',
+                        default: {},
+                    },
+                },
+                additionalProperties: false,
+            },
+        ],
+    },
+    create(context) {
+        const options = context.options[0] || {};
+        const considerTypeOf = (options && options.typeof === true) || false;
+        const map = options.imports || {};
+        function checkIdentifierInJSX(node) {
+            let scope = context.getScope();
+            const sourceCode = context.getSourceCode();
+            const { sourceType } = sourceCode.ast;
+            let { variables } = scope;
+            let scopeType = 'global';
+            let i;
+            let len;
+            // Ignore 'this' keyword (also maked as JSXIdentifier when used in JSX)
+            if (node.name === 'this') {
+                return;
+            }
+            if (!allowGlobals && sourceType === 'module') {
+                scopeType = 'module';
+            }
+            while (scope.type !== scopeType) {
+                scope = scope.upper;
+                variables = scope.variables.concat(variables);
+            }
+            if (scope.childScopes.length) {
+                variables = scope.childScopes[0].variables.concat(variables);
+                // Temporary fix for babel-eslint
+                if (scope.childScopes[0].childScopes.length) {
+                    variables =
+                        scope.childScopes[0].childScopes[0].variables.concat(variables);
+                }
+            }
+            for (i = 0, len = variables.length; i < len; i++) {
+                if (variables[i].name === node.name) {
+                    return;
+                }
+            }
+            const fixable = Boolean(map[node.name]);
+            if (fixable) {
+                context.report({
+                    node,
+                    message: [
+                        `'${node.name}' is not defined.`,
+                        `Run --fix to add \`${map[node.name]}\``,
+                    ]
+                        .filter(Boolean)
+                        .join(' '),
+                    ...(map[node.name]
+                        ? {
+                            fix: (fixer) => {
+                                return fixer.insertTextBefore(sourceCode.ast, map[node.name] + '\n');
+                            },
+                        }
+                        : {}),
+                });
+            }
+        }
+        return {
+            JSXOpeningElement(node) {
+                switch (node.name.type) {
+                    case 'JSXIdentifier':
+                        if ((0, jsx_1.isDOMComponent)(node)) {
+                            return;
+                        }
+                        node = node.name;
+                        break;
+                    case 'JSXMemberExpression':
+                        node = node.name;
+                        do {
+                            node = node.object;
+                        } while (node && node.type !== 'JSXIdentifier');
+                        break;
+                    case 'JSXNamespacedName':
+                        node = node.name.namespace;
+                        break;
+                    default:
+                        break;
+                }
+                checkIdentifierInJSX(node);
+            },
+            'Program:exit'( /* node */) {
+                const globalScope = context.getScope();
+                const sourceCode = context.getSourceCode();
+                if (Object.keys(map).length === 0) {
+                    context.report({
+                        node: sourceCode.ast,
+                        data: sourceCode.ast,
+                        message: 'Rule tenx/auto-import is enabled but there are no replacements specified. Get started by specifying an object as an option to the ESLint rule: ["error", {imports: {sortBy: `import sortBy from "lodash/sortBy"`}}]',
+                    });
+                    return;
+                }
+                globalScope.through.forEach((ref) => {
+                    const { identifier } = ref;
+                    if (!considerTypeOf && hasTypeOfOperator(identifier)) {
+                        return;
+                    }
+                    const fixable = Boolean(map[identifier.name]);
+                    if (fixable) {
+                        context.report({
+                            node: identifier,
+                            data: identifier,
+                            message: [
+                                `'${identifier.name}' is not defined.`,
+                                `Run --fix to add \`${map[identifier.name]}\``,
+                            ]
+                                .filter(Boolean)
+                                .join(' '),
+                            ...(fixable
+                                ? {
+                                    fix: (fixer) => {
+                                        return fixer.insertTextBefore(sourceCode.ast, map[identifier.name] + '\n');
+                                    },
+                                }
+                                : {}),
+                        });
+                    }
+                });
+            },
+        };
+    },
+};

package/package.json

@@ -3,7 +3,7 @@
     "url": "https://github.com/remotion-dev/remotion/tree/main/packages/eslint-plugin"
   },
   "name": "@remotion/eslint-plugin",
-  "version": "4.0.251",
+  "version": "4.0.253",
   "description": "Rules for writing Remotion code",
   "main": "dist/index.js",
   "bugs": {