@rpcbase/cli 0.68.0 → 0.69.0

package/bin.js

@@ -52,6 +52,7 @@ yargs(hideBin(process.argv))
   .command(["lint-staged"], "Run lint-staged", (yargs) => {}, (argv) => {
     lint_staged(argv)
   })
+  // TODO: rm this
   .command(["agent"], "Run the agent", (yargs) => {
     // yargs
       // .option("option1", {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rpcbase/cli",
-  "version": "0.68.0",
+  "version": "0.69.0",
   "license": "SSPL-1.0",
   "bin": {
     "rb": "./bin.js"
@@ -50,11 +50,6 @@
     "debug": "4.3.6",
     "dotenv": "16.4.5",
     "eslint": "8.57.0",
-    "eslint-plugin-flowtype": "8.0.3",
-    "eslint-plugin-import": "2.29.1",
-    "eslint-plugin-lodash": "8.0.0",
-    "eslint-plugin-react": "7.35.0",
-    "eslint-plugin-react-hooks": "4.6.2",
     "lint-staged": "15.2.7",
     "octokit": "3.1.2",
     "parse-dotenv": "2.1.0",

package/eslint/config.js

@@ -1,92 +0,0 @@
-/* @flow */
-const mobile_overrides = require("./src/mobile_overrides")
-
-
-module.exports = {
-  parser: "@babel/eslint-parser",
-  parserOptions: {
-    requireConfigFile: false,
-  },
-  // workingDirectories:{mode: "auto"},
-  ignorePatterns: [
-    "**/*.{css,scss}",
-    "*.eslintrc.js",
-    "**/node_modules/**",
-    "**/.nyc_output/**",
-    "**/coverage/**",
-    "**/build/**",
-    "client/static/**",
-  ],
-  env: {
-    browser: true,
-    es6: true,
-    node: true,
-    jest: true,
-  },
-  settings: {
-    react: {
-      version: "18"
-    }
-  },
-  plugins: ["react", "react-hooks", "flowtype", "import", "lodash"],
-  extends: [
-    "eslint:recommended",
-    "plugin:react/recommended",
-    "plugin:react/jsx-runtime",
-    "plugin:flowtype/recommended",
-  ],
-  globals: {
-    cx: "readonly",
-    monaco: "readonly",
-    // tests
-    page: "readonly",
-    browser: "readonly",
-    context: "readonly",
-    __DEV__: "readonly",
-    __DETOX__: "readonly",
-  },
-  rules: {
-    // "@rpcbase/lint/no-unused-imports": "warn",
-    "@rpcbase/lint/no-rb-restricted-imports": ["error", [
-      {"name": "mongodb", "message": "do not import the 'mongodb' module, this can cause issues, use mongoose.mongo"},
-      {"name": "mongoose", "message": "use @rpcbase/server/mongoose"},
-    ]],
-    "array-callback-return": "error",
-    "comma-dangle": ["error", "always-multiline"],
-    "import/newline-after-import": ["error", {"count": 2}],
-    "import/extensions": ["error", "never", {"svg": "always", "scss": "always", "css": "always", "json": "always"}],
-    "import/no-relative-parent-imports": ["warn"],
-    "lodash/import-scope": ["error", "method"],
-    "max-len": "off",
-    "no-async-promise-executor": "warn",
-    "no-case-declarations": "warn",
-    "no-console": ["warn", {allow: ["warn", "error"]}],
-    "no-constant-condition": ["warn", {"checkLoops": false}],
-    "no-empty": ["error", { "allowEmptyCatch": true }],
-    "no-irregular-whitespace": ["error", {skipRegExps: false, skipStrings: false, skipComments: false}],
-    "no-misleading-character-class": ["error"],
-    "no-return-await": "error",
-    "no-shadow": ["error", {"allow": ["err", "error"]}],
-    "no-redeclare": ["error", {"builtinGlobals": true}],
-    "no-undefined": "warn",
-    "no-useless-escape": "warn",
-    "no-use-before-define": ["error", {variables: true, functions: true, classes: true}],
-    "no-var": "error",
-    "flowtype/boolean-style": ["error", "boolean"],
-    "prefer-template": ["error"],
-    "quotes": ["error", "double", {"allowTemplateLiterals": true}],
-    "react-hooks/exhaustive-deps": "warn",
-    "react-hooks/rules-of-hooks": "error",
-    "react/display-name": "warn",
-    "react/no-access-state-in-setstate": "error",
-    "react/no-find-dom-node": "warn",
-    "react/no-unescaped-entities": "warn",
-    "react/no-unknown-property": "warn",
-    "react/react-in-jsx-scope": "off",
-    "spaced-comment": ["error", "always"],
-    "space-before-function-paren": ["error", "never"],
-  },
-  overrides: [
-    mobile_overrides,
-  ]
-}

package/eslint/index.js

@@ -1,8 +0,0 @@
-/* @flow */
-
-module.exports = {
-  rules: {
-    "no-unused-imports": require("./src/rules/no-unused-imports"),
-    "no-rb-restricted-imports": require("./src/rules/no-rb-restricted-imports")
-  }
-}

package/eslint/src/mobile_overrides.js

@@ -1,10 +0,0 @@
-/* @flow */
-
-module.exports = {
-  files: [ "mobile/spec/**" ],
-  globals: {
-    device: "readonly",
-    element: "readonly",
-    by: "readonly",
-  },
-}

package/eslint/src/rules/no-rb-restricted-imports/index.js

@@ -1,31 +0,0 @@
-/* @flow */
-
-module.exports = {
-  create(context) {
-    const restrictedImports = context.options[0] || []
-
-    function checkNode(node, moduleName) {
-      const restrictedImport = restrictedImports.find(r => r.name === moduleName)
-      if (restrictedImport) {
-        context.report({
-          node,
-          message: restrictedImport.message,
-          data: {
-            name: moduleName,
-          },
-        })
-      }
-    }
-
-    return {
-      "ImportDeclaration"(node) {
-        checkNode(node, node.source.value)
-      },
-      "CallExpression"(node) {
-        if (node.callee.name === "require" && node.arguments[0] && node.arguments[0].type === "Literal") {
-          checkNode(node, node.arguments[0].value)
-        }
-      },
-    }
-  },
-}

package/eslint/src/rules/no-unused-imports/index.js

@@ -1,67 +0,0 @@
-/* @flow */
-
-const noUnusedImports = {
-  meta: {
-    fixable: "code",
-  },
-  create(context) {
-    const getRemovalRange = (importSpecifier, sourceCode) => {
-
-      const tokenBefore = sourceCode.getTokenBefore(importSpecifier)
-      const tokenAfter = sourceCode.getTokenAfter(importSpecifier)
-
-      // if there is a comma before the punctuator, remove the comma as well
-      if (tokenBefore.value === "{") {
-        const tokenBeforeBefore = sourceCode.getTokenBefore(tokenBefore)
-        if (tokenBeforeBefore.value === ",") {
-          return [
-            sourceCode.getTokenBefore(tokenBeforeBefore).range[1],
-            tokenAfter.range[1],
-          ]
-        }
-      }
-
-      if (tokenBefore.value === ",") {
-        return [tokenBefore.range[0], importSpecifier.range[1]]
-      } else if (tokenAfter.value === ",") {
-        return [importSpecifier.range[0] - 1, tokenAfter.range[1]] // Include the space before the specifier
-      }
-
-      // When there is no comma, remove the whole specifier including surrounding spaces
-      return [
-        sourceCode.getTokenBefore(tokenBefore).range[1],
-        sourceCode.getTokenAfter(tokenAfter).range[0],
-      ]
-    }
-
-    return {
-      "ImportDeclaration"(node) {
-        const sourceCode = context.getSourceCode()
-        const importSpecifiers = node.specifiers
-
-        importSpecifiers.forEach((importSpecifier) => {
-          const localName = importSpecifier.local.name
-
-          const variable = context.getScope().variables.find((v) => v.name === localName)
-
-          if (variable && variable.references.length === 0) {
-            context.report({
-              node: importSpecifier,
-              message: `'${localName}' is defined but never used.`,
-              fix(fixer) {
-                if (importSpecifiers.length === 1) {
-                  return fixer.remove(node)
-                } else {
-                  const removalRange = getRemovalRange(importSpecifier, sourceCode)
-                  return fixer.removeRange(removalRange)
-                }
-              },
-            })
-          }
-        })
-      },
-    }
-  },
-}
-
-module.exports = noUnusedImports

package/eslint/src/rules/no-unused-imports/test.js

@@ -1,54 +0,0 @@
-/* @flow */
-const { RuleTester } = require("eslint")
-
-const noUnusedImports = require("./index")
-
-const ruleTester = new RuleTester({
-  parserOptions: {
-    ecmaVersion: 2021,
-    sourceType: "module",
-  },
-});
-
-
-ruleTester.run('no-unused-imports', noUnusedImports, {
-  valid: [
-    "import used from './module'; used();",
-    "import * as used from './module'; used.foo();",
-    "import { used } from './module'; used();",
-    "import { used as alias } from './module'; alias();",
-  ],
-
-  invalid: [
-    {
-      code: "import unused from './module';",
-      errors: [{ message: "'unused' is defined but never used." }],
-      output: '',
-    },
-    {
-      code: "import * as unused from './module';",
-      errors: [{ message: "'unused' is defined but never used." }],
-      output: '',
-    },
-    {
-      code: "import { unused } from './module';",
-      errors: [{ message: "'unused' is defined but never used." }],
-      output: '',
-    },
-    {
-      code: "import { unused as alias } from './module';",
-      errors: [{ message: "'alias' is defined but never used." }],
-      output: '',
-    },
-    {
-      code: "import used, { unused } from './module'; used();",
-      errors: [{ message: "'unused' is defined but never used." }],
-      output: "import used from './module'; used();",
-    },
-    {
-      code: "import { used, unused } from './module'; used();",
-      errors: [{ message: "'unused' is defined but never used." }],
-      output: "import { used } from './module'; used();",
-    },
-  ],
-});