Add ESLint configuration as a package and setup for itself (#3)

* [eslint-config] add eslint-config package

* Bump-up pnpify

* [eslint-plugin] rename eslint-config to eslint-plugin

* Setup ESLint

* [eslint-plugin] use js instead of json to avoid yarn 2 resolution issue

* [eslint-plugin] disabled @typescript-eslint's indent rule

* bump eslint pnpify version

* Fix format

* [eslint-plugin] ignore max-len for comments

* [eslint-plugin] add default option for typescript projects

* [eslint-plugin] set quote-props rule as consistent-as-needed

* enable eslint for workspace by default

* cleanup gitattributes

* [eslint-plugin] release minor

----

Issues:
- Need to install plugins even eslint-plugin package provide it as dependencies. (yarnpkg/berry#8)
- Cannot check indentations of TypeScript files properly. (typescript-eslint/typescript-eslint#1824)

Author: cometkim

.eslintignore

@@ -0,0 +1,5 @@
+/.vscode/
+/.yarn/
+/.pnp.js
+
+**/lib/**

.eslintrc.json

@@ -0,0 +1,7 @@
+{
+  "env": {
+    "node": true
+  },
+  "plugins": ["@cometjs"],
+  "extends": "plugin:@cometjs/auto"
+}

.gitattributes

@@ -4,7 +4,7 @@
 /.yarn/releases/*             binary linguist-generated
 /.yarn/cache/*                binary linguist-vendored
 /.yarn/unplugged/*            linguist-vendored
-/.yarn/plugins/@yarnpkg/*     binary linguist-generated
+/.yarn/plugins/*              binary linguist-generated
+/.vscode/pnpify/*             binary linguist-generated
 /.vscode/*.json               linguist-language=JSON5
-/.vscode/pnpify/typescript/** binary linguist-generated
 /.tsconfig.json               linguist-language=JSON5

.pnp.js

@@ -0,0 +1,18 @@
+#!/usr/bin/env node
+
+const {existsSync} = require(`fs`);
+const {createRequire, createRequireFromPath} = require(`module`);
+const {resolve} = require(`path`);
+
+const relPnpApiPath = "../../../../.pnp.js";
+
+const absPnpApiPath = resolve(__dirname, relPnpApiPath);
+const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath);
+
+if (existsSync(absPnpApiPath)) {
+  // Setup the environment to be able to require eslint/bin/eslint.js
+  require(absPnpApiPath).setup();
+}
+
+// Defer to the real eslint/bin/eslint.js your application uses
+module.exports = absRequire(`eslint/bin/eslint.js`);

.vscode/pnpify/eslint/bin/eslint.js

@@ -0,0 +1,18 @@
+#!/usr/bin/env node
+
+const {existsSync} = require(`fs`);
+const {createRequire, createRequireFromPath} = require(`module`);
+const {resolve} = require(`path`);
+
+const relPnpApiPath = "../../../../.pnp.js";
+
+const absPnpApiPath = resolve(__dirname, relPnpApiPath);
+const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath);
+
+if (existsSync(absPnpApiPath)) {
+  // Setup the environment to be able to require eslint/lib/api.js
+  require(absPnpApiPath).setup();
+}
+
+// Defer to the real eslint/lib/api.js your application uses
+module.exports = absRequire(`eslint/lib/api.js`);

.vscode/pnpify/eslint/lib/api.js

@@ -0,0 +1,6 @@
+{
+  "name": "eslint",
+  "version": "7.0.0-rc.0-pnpify",
+  "main": "./lib/api.js",
+  "type": "commonjs"
+}

.vscode/pnpify/eslint/package.json

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 
+const {existsSync} = require(`fs`);
 const {createRequire, createRequireFromPath} = require(`module`);
 const {resolve} = require(`path`);
 
@@ -8,8 +9,10 @@ const relPnpApiPath = "../../../../.pnp.js";
 const absPnpApiPath = resolve(__dirname, relPnpApiPath);
 const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath);
 
-// Setup the environment to be able to require typescript/bin/tsc
-require(absPnpApiPath).setup();
+if (existsSync(absPnpApiPath)) {
+  // Setup the environment to be able to require typescript/bin/tsc
+  require(absPnpApiPath).setup();
+}
 
 // Defer to the real typescript/bin/tsc your application uses
 module.exports = absRequire(`typescript/bin/tsc`);

.vscode/pnpify/typescript/bin/tsc

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 
+const {existsSync} = require(`fs`);
 const {createRequire, createRequireFromPath} = require(`module`);
 const {resolve} = require(`path`);
 
@@ -8,8 +9,10 @@ const relPnpApiPath = "../../../../.pnp.js";
 const absPnpApiPath = resolve(__dirname, relPnpApiPath);
 const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath);
 
-// Setup the environment to be able to require typescript/bin/tsserver
-require(absPnpApiPath).setup();
+if (existsSync(absPnpApiPath)) {
+  // Setup the environment to be able to require typescript/bin/tsserver
+  require(absPnpApiPath).setup();
+}
 
 // Defer to the real typescript/bin/tsserver your application uses
 module.exports = absRequire(`typescript/bin/tsserver`);

.vscode/pnpify/typescript/bin/tsserver

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 
+const {existsSync} = require(`fs`);
 const {createRequire, createRequireFromPath} = require(`module`);
 const {resolve} = require(`path`);
 
@@ -8,8 +9,10 @@ const relPnpApiPath = "../../../../.pnp.js";
 const absPnpApiPath = resolve(__dirname, relPnpApiPath);
 const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath);
 
-// Setup the environment to be able to require typescript/lib/tsc.js
-require(absPnpApiPath).setup();
+if (existsSync(absPnpApiPath)) {
+  // Setup the environment to be able to require typescript/lib/tsc.js
+  require(absPnpApiPath).setup();
+}
 
 // Defer to the real typescript/lib/tsc.js your application uses
 module.exports = absRequire(`typescript/lib/tsc.js`);

.vscode/pnpify/typescript/lib/tsc.js

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 
+const {existsSync} = require(`fs`);
 const {createRequire, createRequireFromPath} = require(`module`);
 const {resolve} = require(`path`);
 
@@ -8,8 +9,10 @@ const relPnpApiPath = "../../../../.pnp.js";
 const absPnpApiPath = resolve(__dirname, relPnpApiPath);
 const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath);
 
-// Setup the environment to be able to require typescript/lib/tsserver.js
-require(absPnpApiPath).setup();
+if (existsSync(absPnpApiPath)) {
+  // Setup the environment to be able to require typescript/lib/tsserver.js
+  require(absPnpApiPath).setup();
+}
 
 // Defer to the real typescript/lib/tsserver.js your application uses
 module.exports = absRequire(`typescript/lib/tsserver.js`);

.vscode/pnpify/typescript/lib/tsserver.js

@@ -1,5 +1,6 @@
 #!/usr/bin/env node
 
+const {existsSync} = require(`fs`);
 const {createRequire, createRequireFromPath} = require(`module`);
 const {resolve} = require(`path`);
 
@@ -8,8 +9,10 @@ const relPnpApiPath = "../../../../.pnp.js";
 const absPnpApiPath = resolve(__dirname, relPnpApiPath);
 const absRequire = (createRequire || createRequireFromPath)(absPnpApiPath);
 
-// Setup the environment to be able to require typescript/lib/typescript.js
-require(absPnpApiPath).setup();
+if (existsSync(absPnpApiPath)) {
+  // Setup the environment to be able to require typescript/lib/typescript.js
+  require(absPnpApiPath).setup();
+}
 
 // Defer to the real typescript/lib/typescript.js your application uses
 module.exports = absRequire(`typescript/lib/typescript.js`);

.vscode/pnpify/typescript/lib/typescript.js

@@ -1,3 +1,6 @@
 {
-  "typescript.tsdk": ".vscode/pnpify/typescript/lib"
+  "typescript.tsdk": ".vscode/pnpify/typescript/lib",
+  "typescript.enablePromptUseWorkspaceTsdk": true,
+  "eslint.nodePath": ".vscode/pnpify",
+  "eslint.enable": true
 }

.vscode/settings.json

@@ -12,7 +12,15 @@
     "tsc:all": "tsc --noEmit"
   },
   "devDependencies": {
-    "@yarnpkg/pnpify": "^2.0.0-rc.20",
+    "@babel/core": "^7.9.6",
+    "@cometjs/eslint-plugin": "workspace:packages/eslint-plugin",
+    "@typescript-eslint/eslint-plugin": "^2.30.0",
+    "@yarnpkg/pnpify": "^2.0.0-rc.22",
+    "eslint": "7.0.0-rc.0",
+    "eslint-plugin-flowtype": "^4.7.0",
+    "eslint-plugin-jsx-a11y": "^6.2.3",
+    "eslint-plugin-react": "^7.19.0",
+    "eslint-plugin-react-hooks": "^4.0.0",
     "typescript": "^3.8.3"
   }
 }

.yarn/cache/@babel-runtime-corejs3-npm-7.9.6-fc4ed637c2-2.zip

@@ -12,7 +12,7 @@ type ErrorResult = {
   data: undefined,
   error: ApolloError,
   loading: false,
-}
+};
 
 type DataResult<T> = {
   data: Some<T>,
@@ -53,7 +53,9 @@ export function castQueryResult<T>(result: QueryResult<T>): Result<T> {
     reasons.push('is not compatible with DataResult<T> because `data` field is missing');
   }
   if (reasons.length) {
-    throw new Error(`Given object is not compatible with Result<T> because:\n${reasons.join('\n  -')}`);
+    throw new Error(
+      `Given object is not compatible with Result<T> because:\n${reasons.join('\n  -')}`,
+    );
   }
   return result as Result<T>;
 }
@@ -69,7 +71,7 @@ export function mapResult<
     data: RData | ((data: Some<TData>) => RData),
     error: RError | ((error: ApolloError) => RError),
     loading: RLoading | (() => RLoading),
-  }
+  },
 ): RData | RError | RLoading {
   const safeResult = castQueryResult(result);
   if (isLoadingResult(safeResult)) {

.yarn/cache/@babel-runtime-npm-7.9.6-93b1a0d418-2.zip

@@ -64,7 +64,7 @@ export function mapOption<T, RSome, RNone = None>(
       some: RSome | ((t: Some<T>) => RSome),
       none: RNone | (() => RNone),
     })
-  )
+  ),
 ): RSome | RNone {
   if (typeof map === 'function') {
     return isSome(option)
@@ -75,14 +75,14 @@ export function mapOption<T, RSome, RNone = None>(
 
   if (typeof map !== 'object') {
     throw new Error(
-      `The second argument only allows function or object but got: ${typeof map}`
+      `The second argument only allows function or object but got: ${typeof map}`,
     );
   }
 
   const matchedMap = map[matchOption(option)];
   if (!matchedMap) {
     throw new Error(
-      `The object doesn't have map to ${optionToString(option)} type`
+      `The object doesn't have map to ${optionToString(option)} type`,
     );
   }
   return isSome(option)

.yarn/cache/@nodelib-fs.scandir-npm-2.1.3-92e0b8f0d5-2.zip

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2020 Hyeseong Kim <[email protected]>
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

.yarn/cache/@nodelib-fs.stat-npm-2.0.3-be9eaa5ef7-2.zip

@@ -0,0 +1,33 @@
+module.exports = {
+  plugins: [
+    '@cometjs',
+  ],
+  extends: [
+    'plugin:@cometjs/base',
+  ],
+  overrides: [
+    {
+      files: ['**/*.ts?(x)'],
+      extends: ['plugin:@cometjs/typescript'],
+    },
+    {
+      files: ['**/*.js?(x)'],
+      extends: ['plugin:@cometjs/flow'],
+    },
+    {
+      files: ['**/*.{jsx,tsx}'],
+      extends: ['plugin:@cometjs/react'],
+    },
+    {
+      files: [
+        '**/*.test.{js,jsx,ts,tsx}',
+        '**/__test?(s)__/**/*.{js,jsx,ts,tsx}',
+        '**/__mock?(s)__/**/*.{js,jsx,ts,tsx}',
+      ],
+      env: {
+        node: true,
+        jest: true,
+      },
+    },
+  ],
+};

.yarn/cache/@nodelib-fs.walk-npm-1.2.4-1ae6c97d00-2.zip

@@ -0,0 +1,53 @@
+module.exports = {
+  parserOptions: {
+    ecmaVersion: 2018,
+    sourceType: 'module',
+  },
+  env: {
+    es6: true,
+  },
+  extends: [
+    'eslint:recommended',
+  ],
+  rules: {
+    'max-len': ['error', {
+      code: 100,
+      tabWidth: 2,
+      ignoreComments: true,
+      ignoreTrailingComments: true,
+      ignoreUrls: true,
+      ignoreStrings: true,
+      ignoreTemplateLiterals: true,
+      ignoreRegExpLiterals: true,
+    }],
+    'indent': ['error', 2],
+    'semi': ['error', 'always'],
+    'quotes': ['error', 'single'],
+    'quote-props': ['error', 'consistent-as-needed'],
+    'eol-last': ['error', 'always'],
+    'eqeqeq': ['error', 'always', { null: 'ignore' }],
+    'comma-spacing': ['error', { before: false, after: true }],
+    'comma-dangle': ['error', 'always-multiline'],
+    'array-bracket-newline': ['error', 'consistent'],
+    'arrow-spacing': ['error', { before: true, after: true }],
+    'array-element-newline': ['off', { multiline: true, minItems: 3 }],
+    'object-curly-spacing': ['error', 'always'],
+    'object-curly-newline': ['error', {
+      ObjectExpression: {
+        consistent: true,
+      },
+      ObjectPattern: {
+        consistent: true,
+      },
+      ImportDeclaration: {
+        multiline: true,
+        minProperties: 3,
+      },
+      ExportDeclaration: {
+        multiline: true,
+        minProperties: 3,
+      },
+    }],
+    'no-console': 'warn',
+  },
+};

.yarn/cache/@sindresorhus-is-npm-2.1.1-f90d94a68e-2.zip

@@ -0,0 +1,16 @@
+module.exports = {
+  parser: require.resolve('babel-eslint'),
+  plugins: [
+    '@cometjs',
+    'flowtype',
+  ],
+  extends: [
+    'plugin:@cometjs/base',
+    'plugin:flowtype/recommended',
+  ],
+  settings: {
+    flowtype: {
+      onlyFilesWithFlowAnnotation: true,
+    },
+  },
+};