diff --git a/.eslintrc b/.eslintrc new file mode 100644 index 0000000..62503d3 --- /dev/null +++ b/.eslintrc @@ -0,0 +1,310 @@ +{ + "extends": [ + "plugin:react/recommended" + ], + "env": { + browser: true, + es6: true, + node: true, + "jest": true + }, + "parser": "babel-eslint", + "parserOptions": { + "parser": "babel-eslint", + "ecmaVersion": 2018, + "sourceType": "module", + "ecmaFeatures": { + "react": true, + "jsx": true + } + }, + "settings": { + "react": { + "createClass": "createReactClass", // Regex for Component Factory to use, + // default to "createReactClass" + "pragma": "React", // Pragma to use, default to "React" + "version": "detect", // React version. "detect" automatically picks the version you have installed. + // You can also use `16.0`, `16.3`, etc, if you want to override the detected value. + // default to latest and warns if missing + // It will default to "detect" in the future + "flowVersion": "0.53" // Flow version + }, + "propWrapperFunctions": [ + // The names of any function used to wrap propTypes, e.g. `forbidExtraProps`. If this isn't set, any propTypes wrapped in a function will be skipped. + "forbidExtraProps", + {"property": "freeze", "object": "Object"}, + {"property": "myFavoriteWrapper"} + ], + "linkComponents": [ + // Components used as alternatives to for linking, eg. + "Hyperlink", + {"name": "Link", "linkAttribute": "to"} + ] + }, + "plugins": [ + "standard", + "react", + "import", + "node", + "promise" + ], + "rules": { + ////////// Possible Errors ////////// + + "comma-dangle": 1, // disallow trailing commas in object literals + "no-cond-assign": 0, // disallow assignment in conditional expressions + "no-console": 0, // disallow use of console (off by default in the node environment) + "no-control-regex": 1, // disallow control characters in regular expressions + "no-debugger": 1, // disallow use of debugger + "no-dupe-args": 1, // disallow duplicate arguments in functions + "no-dupe-keys": 1, // disallow duplicate keys when creating object literals + "no-duplicate-case": 1, // disallow a duplicate case label + "no-empty-character-class": 1, // disallow the use of empty character classes in regular expressions + "no-empty": 1, // disallow empty statements + "no-empty-pattern": "error", + "no-ex-assign": 1, // disallow assigning to the exception in a catch block + "no-extra-boolean-cast": 0, // disallow double-negation boolean casts in a boolean context + "no-extra-parens": ["error", "functions"], // disallow unnecessary parentheses (off by default) + "no-extra-semi": 1, // disallow unnecessary semicolons + "no-func-assign": 1, // disallow overwriting functions written as function declarations + "no-inner-declarations": ["error", "functions"], // disallow function or variable declarations in nested blocks + "no-invalid-regexp": 1, // disallow invalid regular expression strings in the RegExp constructor + "no-irregular-whitespace": 1, // disallow irregular whitespace outside of strings and comments + "no-negated-in-lhs": 1, // disallow negation of the left operand of an in expression + "no-obj-calls": 1, // disallow the use of object properties of the global object (Math and JSON) as functions + "no-regex-spaces": 1, // disallow multiple spaces in a regular expression literal + "no-sparse-arrays": 1, // disallow sparse arrays + "no-unreachable": 1, // disallow unreachable statements after a return, throw, continue, or break statement + "use-isnan": 1, // disallow comparisons with the value NaN + "valid-jsdoc": 1, // Ensure JSDoc comments are valid (off by default) + "valid-typeof": ["error", { "requireStringLiterals": true }], // Ensure that the results of typeof are compared against a valid string + "no-unexpected-multiline": 1, // Avoid code that looks like two expressions but is actually one (off by default) + + ////////// Best Practices ////////// + + "accessor-pairs": 1, // enforces getter/setter pairs in objects (off by default) + "block-scoped-var": 0, // treat var statements as if they were block scoped (off by default) + "complexity": 1, // specify the maximum cyclomatic complexity allowed in a program (off by default) + "consistent-return": 1, // require return statements to either always or never specify values + "curly": 1, // specify curly brace conventions for all control statements + "default-case": 1, // require default case in switch statements (off by default) + "dot-notation": 1, // encourages use of dot notation whenever possible + "dot-location": ["error", "property"], // enforces consistent newlines before or after dots (off by default) + "eqeqeq": ["error", "always", { "null": "ignore" }], // require the use of === and !== + "guard-for-in": 1, // make sure for-in loops have an if statement (off by default) + "no-alert": 1, // disallow the use of alert, confirm, and prompt + "no-caller": 1, // disallow use of arguments.caller or arguments.callee + "no-div-regex": 1, // disallow division operators explicitly at beginning of regular expression (off by default) + "no-else-return": 1, // disallow else after a return in an if (off by default) + "no-eq-null": 1, // disallow comparisons to null without a type-checking operator (off by default) + "no-eval": 1, // disallow use of eval() + "no-extend-native": 1, // disallow adding to native types + "no-extra-bind": 1, // disallow unnecessary function binding + "no-fallthrough": 1, // disallow fallthrough of case statements + "no-floating-decimal": 1, // disallow the use of leading or trailing decimal points in numeric literals (off by default) + "no-implied-eval": 1, // disallow use of eval()-like methods + "no-iterator": 1, // disallow usage of __iterator__ property + "no-labels": 1, // disallow use of labeled statements + "no-lone-blocks": 1, // disallow unnecessary nested blocks + "no-loop-func": 1, // disallow creation of functions within loops + "no-multi-spaces": 1, // disallow use of multiple spaces + "no-multi-str": 1, // disallow use of multiline strings + "no-native-reassign": 1, // disallow reassignments of native objects + "no-new-func": 1, // disallow use of new operator for Function object + "no-new-wrappers": 1, // disallows creating new instances of String, Number, and Boolean + "no-new": 1, // disallow use of new operator when not part of the assignment or comparison + "no-octal-escape": 1, // disallow use of octal escape sequences in string literals, such as var foo = "Copyright \251"; + "no-octal": 1, // disallow use of octal literals + "no-param-reassign": 0, // disallow reassignment of function parameters (off by default) + "no-process-env": 0, // disallow use of process.env (off by default) + "no-proto": 1, // disallow usage of __proto__ property + "no-redeclare": 0, // disallow declaring the same variable more then once + "no-return-assign": ["error", "except-parens"], // disallow use of assignment in return statement + "no-script-url": 1, // disallow use of javascript: urls + "no-self-compare": 1, // disallow comparisons where both sides are exactly the same (off by default) + "no-sequences": 1, // disallow use of comma operator + "no-throw-literal": 1, // restrict what can be thrown as an exception (off by default) + "no-void": 1, // disallow use of void operator (off by default) + "no-warning-comments": 1, // disallow usage of configurable warning terms in comments, e.g. TODO or FIXME (off by default) + "no-with": 1, // disallow use of the with statement + "radix": 1, // require use of the second argument for parseInt() (off by default) + "vars-on-top": 0, // requires to declare all vars on top of their containing scope (off by default) + "wrap-iife": ["error", "any", { "functionPrototypeMethods": true }], // require immediate function invocation to be wrapped in parentheses (off by default) + "yoda": 1, // require or disallow Yoda conditions + + ////////// Strict Mode ////////// + + //"strict": 1, // controls location of Use Strict Directives + + ////////// Variables ////////// + + "no-catch-shadow": 0, // disallow the catch clause parameter name being the same as a variable in the outer scope (off by default in the node environment) + "no-delete-var": 1, // disallow deletion of variables + "no-label-var": 1, // disallow labels that share a name with a variable + "no-shadow": 0, // disallow declaration of variables already declared in the outer scope + "no-shadow-restricted-names": 1, // disallow shadowing of names such as arguments + "no-undef": ["error"], // disallow use of undeclared variables unless mentioned in a /*global */ block + "no-undef-init": 1, // disallow use of undefined when initializing variables + "no-undefined": 0, // disallow use of undefined variable (off by default) + "no-unused-vars": ["error", { "vars": "all", "args": "none", "ignoreRestSiblings": true, "no-unused-vars": ["_", "__", "___"] }], // disallow declaration of variables that are not used in the code + "no-use-before-define": ["error", { "functions": false, "classes": false, "variables": false }], // disallow use of variables before they are defined + + ////////// Node.js ////////// + + "handle-callback-err": ["error", "^(err|error)$" ], // enforces error handling in callbacks (off by default) (on by default in the node environment) + "no-mixed-requires": 1, // disallow mixing regular variable and require declarations (off by default) (on by default in the node environment) + "no-new-require": 1, // disallow use of new operator with the require function (off by default) (on by default in the node environment) + "no-path-concat": 1, // disallow string concatenation with __dirname and __filename (off by default) (on by default in the node environment) + "no-process-exit": 1, // disallow process.exit() (on by default in the node environment) + "no-restricted-modules": 1, // restrict usage of specified node modules (off by default) + "no-sync": 0, // disallow use of synchronous methods (off by default) + + ////////// Stylistic Issues ////////// + + "array-bracket-spacing": 1, // enforce spacing inside array brackets (off by default) + "brace-style": ["error", "1tbs", { "allowSingleLine": true }], // enforce one true brace style (off by default) + "camelcase": ["error", { "properties": "never" }], // require camel case names + "comma-spacing": ["error", { "before": false, "after": true }], // enforce spacing before and after comma + "comma-style": ["error", "last"], // enforce one true comma style (off by default) + "computed-property-spacing": 1, // require or disallow padding inside computed properties (off by default) + "consistent-this": ["error", "self"], // enforces consistent naming when capturing the current execution context (off by default) + "eol-last": 1, // enforce newline at the end of file, with no multiple empty lines + "func-names": 0, // require function expressions to have a name (off by default) + //"func-style": 1, // enforces use of function declarations or expressions (off by default) + "key-spacing": ["error", { "beforeColon": false, "afterColon": true }], // enforces spacing between keys and values in object literal properties + "lines-around-comment": 0, // enforces empty lines around comments (off by default) + "linebreak-style": 1, // disallow mixed 'LF' and 'CRLF' as linebreaks (off by default) + "max-nested-callbacks": 1, // specify the maximum depth callbacks can be nested (off by default) + "new-cap": ["error", { "newIsCap": true, "capIsNew": false }], // require a capital letter for constructors + "new-parens": 1, // disallow the omission of parentheses when invoking a constructor with no arguments + "newline-after-var": 0, // allow/disallow an empty newline after var statement (off by default) + "no-array-constructor": 1, // disallow use of the Array constructor + "no-continue": 0, // disallow use of the continue statement (off by default) + "no-inline-comments": 0, // disallow comments inline after code (off by default) + "no-lonely-if": 1, // disallow if as the only statement in an else block (off by default) + "no-mixed-spaces-and-tabs": 1, // disallow mixed spaces and tabs for indentation + "no-multiple-empty-lines": 1, // disallow multiple empty lines (off by default) + "no-nested-ternary": 0, // disallow nested ternary expressions (off by default) + "no-new-object": 1, // disallow use of the Object constructor + "no-spaced-func": 1, // disallow space between function identifier and application + "no-ternary": 0, // disallow the use of ternary operators (off by default) + "no-trailing-spaces": 1, // disallow trailing whitespace at the end of lines + "no-underscore-dangle": 0, // disallow dangling underscores in identifiers + "one-var": 0, // allow just one var statement per function (off by default) + "operator-assignment": 1, // require assignment operator shorthand where possible or prohibit it entirely (off by default) + "operator-linebreak": ["error", "after", { "overrides": { "?": "before", ":": "before" } }], // enforce operators to be placed before or after line breaks (off by default) + "padded-blocks": ["error", { "blocks": "never", "switches": "never", "classes": "never" }], // enforce padding within blocks (off by default) + "quote-props": 0, // require quotes around object literal property names (off by default) + "quotes": ["error", "double", { "avoidEscape": true, "allowTemplateLiterals": true }], // specify whether double or single quotes should be used + "semi-spacing": ["error", { "before": false, "after": true }], // enforce spacing before and after semicolons + "semi": ["error", "always"], // require or disallow use of semicolons instead of ASI + "sort-vars": 1, // sort variables within the same declaration block (off by default) + "space-before-blocks": 1, // require or disallow space before blocks (off by default) + "space-before-function-paren": 0, // require or disallow space before function opening parenthesis (off by default) + "space-in-parens": 1, // require or disallow spaces inside parentheses (off by default) + "space-infix-ops": 1, // require spaces around operators + "keyword-spacing": ["error", { "before": true, "after": true }], + "space-unary-ops": ["error", { "words": true, "nonwords": false }], // require or disallow spaces before/after unary operators (words on by default, nonwords off by default) + "spaced-comment": ["warn", "always", { + "line": { "markers": ["*package", "!", "/", ",", "="] }, + "block": { "balanced": true, "markers": ["*package", "!", ",", ":", "::", "flow-include", "*globals"], "exceptions": ["*", "globals.*", "eslint-env"] } + }], // require or disallow a space immediately following the // or /* in a comment (off by default) + "wrap-regex": 1, // require regex literals to be wrapped in parentheses (off by default) + + ////////// ECMAScript 6 ////////// + + "constructor-super": 1, // verify super() callings in constructors (off by default) + "generator-star-spacing": ["error", { "before": true, "after": true }], // enforce the spacing around the * in generator functions (off by default) + "no-this-before-super": 1, // disallow to use this/super before super() calling in constructors (off by default) + "no-var": 1, // require let or const instead of var (off by default) + "object-shorthand": 1, // require method and property shorthand syntax for object literals (off by default) + "prefer-const": 1, // suggest using of const declaration for variables that are never modified after declared (off by default) + + ////////// Legacy ////////// + + "max-depth": 1, // specify the maximum depth that blocks can be nested (off by default) + "max-len": ["warn", { "code": 120 }], // specify the maximum length of a line in your program (off by default) + "max-params": ["error", { max: 5 }], // limits the number of parameters that can be used in the function declaration. (off by default) + "max-statements": ["error", { "max": 50 }], // specify the maximum number of statement allowed in a function (off by default) + "no-bitwise": 1, // disallow use of bitwise operators (off by default) + "no-plusplus": 0, // disallow use of unary operators, ++ and -- (off by default) + + ////////// Standard 2019 /////// + "arrow-spacing": ["error", { "before": true, "after": true }], + "block-spacing": ["error", "always"], + "func-call-spacing": ["error", "never"], + "indent": ["error", 2, { + "SwitchCase": 1, + "VariableDeclarator": 1, + "outerIIFEBody": 1, + "MemberExpression": 1, + "FunctionDeclaration": { "parameters": 1, "body": 1 }, + "FunctionExpression": { "parameters": 1, "body": 1 }, + "CallExpression": { "arguments": 1 }, + "ArrayExpression": 1, + "ObjectExpression": 1, + "ImportDeclaration": 1, + "flatTernaryExpressions": false, + "ignoreComments": false + }], + "no-class-assign": "error", + "no-compare-neg-zero": "error", + "no-const-assign": "error", + "no-constant-condition": ["error", { "checkLoops": false }], + "no-dupe-class-members": "error", + "no-global-assign": "error", + "no-mixed-operators": ["error", { + "groups": [ + ["==", "!=", "===", "!==", ">", ">=", "<", "<="], + ["&&", "||"], + ["in", "instanceof"] + ], + "allowSamePrecedence": true + }], + "no-new-symbol": "error", + "no-return-await": "error", + "no-self-assign": "error", + "no-tabs": "error", + "no-template-curly-in-string": "error", + "no-unmodified-loop-condition": "error", + "no-unneeded-ternary": ["error", { "defaultAssignment": false }], + "no-unsafe-finally": "error", + "no-unsafe-negation": "error", + "no-unused-expressions": ["error", { "allowShortCircuit": true, "allowTernary": true, "allowTaggedTemplates": true }], + "no-useless-call": "error", + "no-useless-computed-key": "error", + "no-useless-constructor": "error", + "no-useless-escape": "error", + "no-useless-rename": "error", + "no-useless-return": "error", + "no-whitespace-before-property": "error", + "object-curly-spacing": ["error", "always"], + "object-property-newline": ["error", { "allowMultiplePropertiesPerLine": true }], + "prefer-promise-reject-errors": "error", + "rest-spread-spacing": ["error", "never"], + "symbol-description": "error", + "template-curly-spacing": ["error", "never"], + "template-tag-spacing": ["error", "never"], + "unicode-bom": ["error", "never"], + "yield-star-spacing": ["error", "both"], + + "import/export": "error", + "import/first": "error", + "import/no-duplicates": "error", + "import/no-named-default": "error", + "import/no-webpack-loader-syntax": "error", + + "node/no-deprecated-api": "error", + "node/process-exit-as-throw": "error", + + "promise/param-names": "error", + + "standard/array-bracket-even-spacing": ["error", "either"], + "standard/computed-property-even-spacing": ["error", "even"], + "standard/no-callback-literal": "error", + "standard/object-curly-even-spacing": ["error", "either"], + + "jsx-a11y/anchor-is-valid": 0, + "react/prop-types": 0 + } +} diff --git a/package.json b/package.json index be401f2..ef804ed 100644 --- a/package.json +++ b/package.json @@ -7,21 +7,37 @@ "dev": "webpack-dev-server --mode development --open", "build": "webpack --optimize-minimize --config webpack.production.config.js", "webpack": "webpack --config webpack.config.js --watch", + "lint": "eslint src", "test": "jest", - "test-watch": "jest --watch" + "test-watch": "jest --watch", + "test-coverage": "jest --coverage" }, + "pre-commit": [ + "lint", + "test", + "build" + ], "keywords": [], "author": "", "license": "ISC", "devDependencies": { + "@babel/cli": "^7.5.5", "@babel/core": "^7.5.5", "@babel/plugin-proposal-class-properties": "^7.5.5", "@babel/plugin-proposal-private-methods": "^7.4.4", "@babel/preset-env": "^7.5.5", "@babel/preset-react": "^7.0.0", + "babel-eslint": "^11.0.0-beta.0", "babel-loader": "^8.0.6", "babel-plugin-styled-components": "^1.10.6", + "eslint": "^6.3.0", + "eslint-plugin-import": "^2.18.2", + "eslint-plugin-node": "^10.0.0", + "eslint-plugin-promise": "^4.2.1", + "eslint-plugin-react": "^7.14.3", + "eslint-plugin-standard": "^4.0.1", "jest": "^24.9.0", + "pre-commit": "^1.2.2", "react-hot-loader": "^4.12.11", "webpack": "^4.39.2", "webpack-cli": "^3.3.7", diff --git a/src/components/Apple.js b/src/components/Apple.js index e2158ee..22016a4 100644 --- a/src/components/Apple.js +++ b/src/components/Apple.js @@ -1,3 +1,4 @@ +/* eslint-disable indent */ import styled, { keyframes, css } from "styled-components"; const fade = keyframes` @@ -20,12 +21,11 @@ const Apple = styled.div.attrs(({ theme, zoom }) => ({ &:after { content: "${props => (!props.died ? "🍎" : "🐛")}"; display: inline-block; - animation: ${props => - props.died - ? "none" - : css` - ${fade} 1s ease-out alternate infinite - `}; + animation: ${({ died }) => + !died && + css` + ${fade} 1s ease-out alternate infinite + `}; } `; diff --git a/src/containers/Snake.js b/src/containers/Snake.js index 1f82d51..4f8fb6c 100644 --- a/src/containers/Snake.js +++ b/src/containers/Snake.js @@ -87,7 +87,7 @@ const Snake = ({ (cell.type === "snake" && ) } - {!started && Press 'r' to start the game} + {!started && Press 'r' to start the game} {died && lastGameId !== gameId && hasHighscore(score) && ( )} diff --git a/src/redux/Module.js b/src/redux/Module.js index 93b3b9a..e94b916 100644 --- a/src/redux/Module.js +++ b/src/redux/Module.js @@ -1,6 +1,6 @@ export class Middleware { - #action = null; - #middleware = null; + _action = null; + _middleware = null; constructor(action, middleware) { if (!action) { @@ -15,26 +15,26 @@ export class Middleware { if (typeof middleware !== "function") { throw new Error("Argument 'middleware' must be a string."); } - this.#action = action; - this.#middleware = middleware; + this._action = action; + this._middleware = middleware; } get action() { - return this.#action; + return this._action; } get middleware() { - return this.#middleware; + return this._middleware; } invoke(dispatch, state, action, getState) { - return this.#middleware(dispatch, state, action, getState); + return this._middleware(dispatch, state, action, getState); } } export class After { - #action = null; - #handler = null; + _action = null; + _handler = null; constructor(action, handler) { if (!action) { @@ -49,26 +49,26 @@ export class After { if (typeof handler !== "function") { throw new Error("Argument 'handler' must be a string."); } - this.#action = action; - this.#handler = handler; + this._action = action; + this._handler = handler; } get action() { - return this.#action; + return this._action; } get handler() { - return this.#handler; + return this._handler; } invoke(dispatch, state, action, getState) { - return this.#handler(dispatch, state, action, getState); + return this._handler(dispatch, state, action, getState); } } export class Reducer { - #action = null; - #reducer = null; + _action = null; + _reducer = null; constructor(action, reducer) { if (!action) { @@ -83,55 +83,55 @@ export class Reducer { if (typeof reducer !== "function") { throw new Error("Argument 'reducer' must be a string."); } - this.#action = action; - this.#reducer = reducer; + this._action = action; + this._reducer = reducer; } get action() { - return this.#action; + return this._action; } get reducer() { - return this.#reducer; + return this._reducer; } invoke(state, action) { - return this.#reducer(state, action); + return this._reducer(state, action); } } export class Module { - #name = null; - #initialState = null; - #actions = {}; - #after = {}; + _name = null; + _initialState = null; + _actions = {}; + _after = {}; constructor(name, initialState) { if (!name) { throw new Error("Argument 'name' is required."); } - this.#name = name; - this.#initialState = initialState; + this._name = name; + this._initialState = initialState; this.middlewares = this.middlewares.bind(this); } get name() { - return this.#name; + return this._name; } middleware(action, middleware) { - this.#register(new Middleware(action, middleware)); + this._register(new Middleware(action, middleware)); } reducer(action, reducer) { - this.#register(new Reducer(action, reducer)); + this._register(new Reducer(action, reducer)); } after(action, middleware) { - this.#register(new After(action, middleware)); + this._register(new After(action, middleware)); } - #register(reducerOrMiddleware) { + _register(reducerOrMiddleware) { if ( !(reducerOrMiddleware instanceof Reducer) && !(reducerOrMiddleware instanceof Middleware) && @@ -140,12 +140,12 @@ export class Module { throw new Error("Only objects of type Reducer, Middleware or After can register"); } if (!(reducerOrMiddleware instanceof After)) { - if (this.#actions[reducerOrMiddleware.action]) { + if (this._actions[reducerOrMiddleware.action]) { throw new Error(`Reducer or middleware with action "${reducerOrMiddleware.action}" is already registered`); } - this.#actions[reducerOrMiddleware.action] = reducerOrMiddleware; + this._actions[reducerOrMiddleware.action] = reducerOrMiddleware; } else { - this.#after[reducerOrMiddleware.action] = reducerOrMiddleware; + this._after[reducerOrMiddleware.action] = reducerOrMiddleware; } return reducerOrMiddleware; } @@ -157,28 +157,28 @@ export class Module { if (typeof action !== "function") { throw new Error("Argument 'action' must be function"); } - const type = `[${this.#name}] ${name}`; + const type = `[${this._name}] ${name}`; const typedAction = (...args) => ({ ...action.apply(this, args), type }); return [type, typedAction]; } - #getActionByType(Type) { - return Object.keys(this.#actions) - .map(key => ({ key, action: this.#actions[key] })) + _getActionByType(Type) { + return Object.keys(this._actions) + .map(key => ({ key, action: this._actions[key] })) .reduce((actions, { key, action }) => (action instanceof Type ? { ...actions, [key]: action } : actions), {}); } middlewares({ dispatch, getState }) { return next => action => { next(action); - const middlewares = this.#getActionByType(Middleware); + const middlewares = this._getActionByType(Middleware); const middleware = middlewares[action.type]; if (middleware) { - middleware.invoke(dispatch, getState()[this.#name], action, getState); + middleware.invoke(dispatch, getState()[this._name], action, getState); } - const after = this.#after[action.type]; + const after = this._after[action.type]; if (after) { - after.invoke(dispatch, getState()[this.#name], action, getState); + after.invoke(dispatch, getState()[this._name], action, getState); } return null; }; @@ -186,8 +186,8 @@ export class Module { get reducers() { return { - [this.#name]: (state = this.#initialState, action) => { - const reducers = this.#getActionByType(Reducer); + [this._name]: (state = this._initialState, action) => { + const reducers = this._getActionByType(Reducer); const reducer = reducers[action.type]; if (reducer) { return reducer.invoke(state, action); @@ -199,16 +199,16 @@ export class Module { state(selector) { return state => { - if (!this.#name) { + if (!this._name) { throw new Error("Unable to detect module name"); } if (!state) { throw new Error("No state was passed to the state selector"); } - if (!state[this.#name]) { - throw new Error(`There is no state for module '${this.#name}'`); + if (!state[this._name]) { + throw new Error(`There is no state for module '${this._name}'`); } - return selector(state[this.#name]); + return selector(state[this._name]); }; } @@ -225,24 +225,3 @@ export class Module { } export default Module; - -export const attach = (modules, Component) => { - console.log("modules, Component", modules, Component); - return (...args) => { - console.log("args", args); - const component = Component({ - GAME: { - actions: { - startLoop() { - console.log("=== START LOOP ==="); - } - }, - state: {} - } - }); - return component; - component.props = { ...component.props }; - console.log("component", component); - return component.render.apply(this, args); - }; -};