@kitql/eslint-config 0.0.3 → 0.1.0-next.1

package/README.md

@@ -3,7 +3,7 @@
 [![](https://img.shields.io/npm/v/@kitql/eslint-config?color=&logo=npm)](https://www.npmjs.com/package/@kitql/eslint-config)
 [![](https://img.shields.io/npm/dm/@kitql/eslint-config?&logo=npm)](https://www.npmjs.com/package/@kitql/eslint-config)
 
-##  📖 Read the doc
+## 📖 Read the doc
 
 [![](https://img.shields.io/badge/Documentation%20of-kitql%20lint%20format-FF3E00.svg?style=flat&logo=stackblitz&logoColor=FF3E00)](https://kitql.dev/docs)
 
@@ -16,25 +16,24 @@ npm install @kitql/eslint-config --D
 ### eslint config
 
 `.eslintrc.js`
+
 ```js
 module.exports = {
-	extends: [
-		'@kitql',
-	],
+  extends: ['@kitql']
 }
 ```
 
 ### prettier config
 
 `.prettierrc.cjs`
+
 ```js
 const config = require('@kitql/eslint-config/.prettierrc.cjs')
 
 module.exports = {
-  ...config,
+  ...config
   // Some custom things?
 }
-
 ```
 
 ### usage
@@ -47,9 +46,9 @@ npm exec kitql-lint
 npm exec kitql-lint --format
 ```
 
-##  ⭐️ Join us
+## ⭐️ Join us
 
 [![GitHub Repo stars](https://img.shields.io/github/stars/jycouet/kitql?logo=github&label=KitQL&color=#4ACC31)](https://github.com/jycouet/kitql)
 
-💡 _[KitQL](https://www.kitql.dev/docs) itself is not a library, it's "nothing" but a collection of standalone libraries._
-
+💡 _[KitQL](https://www.kitql.dev/docs) itself is not a library, it's "nothing" but a collection of
+standalone libraries._

package/cmd.js

@@ -1,50 +1,35 @@
 #!/usr/bin/env node
+import { Log, red } from '@kitql/helpers'
 import { spawn, spawnSync } from 'child_process'
 import { program, Option } from 'commander'
 import fs from 'fs'
 
+const log = new Log('kitql-lint')
+
 program.addOption(new Option('-f, --format', 'format'))
 
 program.parse(process.argv)
 const options_cli = program.opts()
 
-// console.info(`options_cli`, options_cli)
-
-let pathPrettierIgnore = ''
-try {
-  fs.statSync('.prettierignore')
-  pathPrettierIgnore = '.prettierignore'
-} catch (error) {
-  try {
-    fs.statSync('../../.prettierignore')
-    pathPrettierIgnore = '../../.prettierignore'
-  } catch (error) {
-    // Still nothing
+const findFileOrUp = fileName => {
+  // Find file recursively 4 levels max up
+  for (let i = 0; i < 4; i++) {
+    try {
+      const path = '../'.repeat(i) + fileName
+      if (fs.statSync(path)) {
+        return path
+      }
+    } catch (error) {
+      // nothing to do
+    }
   }
-}
-if (!pathPrettierIgnore) {
-  console.error(`.prettierignore not found`)
-  process.exit(0)
-}
 
-let pathPrettierCjs = ''
-try {
-  fs.statSync('.prettierrc.cjs')
-  pathPrettierCjs = '.prettierrc.cjs'
-} catch (error) {
-  try {
-    fs.statSync('../../.prettierrc.cjs')
-    pathPrettierCjs = '../../.prettierrc.cjs'
-  } catch (error) {
-    // Still nothing
-  }
-}
-if (!pathPrettierCjs) {
-  console.error(`.prettierrc.cjs not found`)
-  process.exit(0)
+  log.error(red(`${fileName} not found`))
+  process.exit(1)
 }
 
-// --config <path>
+let pathPrettierIgnore = findFileOrUp('.prettierignore')
+let pathPrettierCjs = findFileOrUp('.prettierrc.cjs')
 
 const format = options_cli.format ?? false
 
@@ -60,26 +45,46 @@ const cmdPrettier =
   `${format ? ' --write' : ''}` +
   // exec
   ` .`
-spawnSync(cmdPrettier, {
+let result_prettier = spawn(cmdPrettier, {
   shell: true,
-  // cwd: process.cwd(),
-  stdio: 'inherit',
+  cwd: process.cwd(),
+  stdio: 'pipe',
 })
 
-// Then eslint
-const cmdEsLint =
-  `eslint` +
-  // ignore?
-  ` --ignore-path ${pathPrettierIgnore}` +
-  // format or not
-  `${format ? ' --fix' : ''} ` +
-  // exec
-  ` "*"`
-spawnSync(cmdEsLint, {
-  shell: true,
-  // cwd: process.cwd(),
-  stdio: 'inherit',
-})
+// let's not log anything when we are formating prettier
+if (!format) {
+  const logPrettier = new Log('kitql-lint prettier')
+  result_prettier.stdout.on('data', data => {
+    logPrettier.error(data.toString())
+  })
+}
+
+result_prettier.on('close', code => {
+  // Then eslint
+  const cmdEsLint =
+    `eslint` +
+    // ignore?
+    ` --ignore-path ${pathPrettierIgnore}` +
+    // format or not
+    `${format ? ' --fix' : ''}` +
+    // exec
+    ` .`
+
+  // log.info(cmdEsLint)
+
+  let result_eslint = spawnSync(cmdEsLint, {
+    shell: true,
+    cwd: process.cwd(),
+    stdio: 'inherit',
+  })
 
-// console.log(`cmdPrettier`, cmdPrettier)
-// console.log(`cmdEsLint`, cmdEsLint)
+  if (result_eslint.status) {
+    log.error(red(`eslint failed, check logs above.`))
+  }
+
+  if (code === 0 && result_eslint.status === 0) {
+    log.success(`All good, your files looks great!`)
+  }
+
+  process.exit(code || result_eslint.status)
+})

package/index.cjs

@@ -2,9 +2,8 @@ module.exports = {
   root: true,
   parser: '@typescript-eslint/parser',
   extends: [
-    '@theguild',
-    '@theguild/eslint-config/react',
-    '@theguild/eslint-config/mdx',
+    // '@theguild',
+    // '@theguild/eslint-config/mdx',
     '@theguild/eslint-config/json',
     '@theguild/eslint-config/yml',
     'eslint:recommended',
@@ -14,8 +13,17 @@ module.exports = {
   ],
   plugins: ['unused-imports', 'svelte', '@typescript-eslint'],
   rules: {
-    'no-console': ['error', { allow: ['info', 'warn', 'error', 'time', 'timeEnd'] }],
+    'no-console': ['error', { allow: ['info', 'warn', 'error', 'time', 'timeEnd', 'dir'] }],
     'unused-imports/no-unused-imports': 'error',
+    '@typescript-eslint/no-unused-vars': [
+      'warn',
+      {
+        argsIgnorePattern: '^_',
+        varsIgnorePattern: '^_',
+        caughtErrorsIgnorePattern: '^_',
+      },
+    ],
+    'no-empty': ['error', { allowEmptyCatch: true }],
     '@typescript-eslint/ban-ts-ignore': 'off',
     '@typescript-eslint/ban-ts-comment': 'off',
     '@typescript-eslint/no-explicit-any': 'off',
@@ -39,9 +47,11 @@ module.exports = {
       extends: ['plugin:@graphql-eslint/schema-all', 'plugin:@graphql-eslint/operations-all'],
       rules: {
         '@graphql-eslint/alphabetize': 'off',
+        '@graphql-eslint/no-one-place-fragments': 'off',
       },
     },
   ],
+  ignorePatterns: ['*.md'],
   parserOptions: {
     sourceType: 'module',
     ecmaVersion: 2020,

package/package.json

@@ -2,9 +2,11 @@
   "name": "@kitql/eslint-config",
   "description": "opinionated linting and formatting for projects",
   "keywords": [
-    "cli"
+    "cli",
+    "eslint",
+    "eslint-config"
   ],
-  "version": "0.0.3",
+  "version": "0.1.0-next.1",
   "license": "MIT",
   "type": "module",
   "repository": {
@@ -19,14 +21,15 @@
   "main": "index.cjs",
   "dependencies": {
     "@graphql-eslint/eslint-plugin": "3.20.1",
+    "@kitql/helpers": "0.8.8-next.1",
     "@theguild/eslint-config": "^0.11.1",
     "@theguild/prettier-config": "2.0.2",
     "@trivago/prettier-plugin-sort-imports": "4.3.0",
-    "@typescript-eslint/eslint-plugin": "6.14.0",
-    "@typescript-eslint/parser": "6.14.0",
+    "@typescript-eslint/eslint-plugin": "6.15.0",
+    "@typescript-eslint/parser": "6.15.0",
     "@vue/compiler-sfc": "3.3.4",
     "commander": "^11.1.0",
-    "eslint": "8.55.0",
+    "eslint": "8.56.0",
     "eslint-config-prettier": "9.1.0",
     "eslint-plugin-svelte": "2.35.0",
     "eslint-plugin-unused-imports": "3.0.0",
@@ -49,7 +52,8 @@
     ".prettierrc.cjs"
   ],
   "scripts": {
-    "dev": "node ./cmd.js",
+    "lint": "node ./cmd.js",
+    "format": "node ./cmd.js -f",
     "lint:example": "kitql-lint",
     "format:example": "kitql-lint --format"
   }