eslint-plugin-sonar-config 0.1.2 → 0.1.3

package/index.js

@@ -1,33 +1,33 @@
-import noNestedTernary from "./rules/no-nested-ternary.js";
-import awaitOnlyPromises from "./rules/await-only-promises.js";
-import forLoopUpdateCounter from "./rules/for-loop-update-counter.js";
-import maxSwitchCases from "./rules/max-switch-cases.js";
-import noArrayDelete from "./rules/no-array-delete.js";
-import noExampleRule from "./rules/no-example-rule.js";
-import noInArray from "./rules/no-in-array.js";
-import noInWithPrimitives from "./rules/no-in-with-primitives.js";
-import noIndexOfComparePositive from "./rules/no-index-of-compare-positive.js";
-import noLabelInSwitch from "./rules/no-label-in-switch.js";
-import noThisInFunctionalComponent from "./rules/no-this-in-functional-component.js";
-import preferForOf from "./rules/prefer-for-of.js";
-import preferRegexExec from "./rules/prefer-regex-exec.js";
-import requireReduceInitialValue from "./rules/require-reduce-initial-value.js";
-import requireSortCompare from "./rules/require-sort-compare.js";
-import useStateCallback from "./rules/use-state-callback.js";
-import preferWhile from "./rules/prefer-while.js";
-import noIdenticalBranches from "./rules/no-identical-branches.js";
-import noEmptyRegexAlternatives from "./rules/no-empty-regex-alternatives.js";
-import regexAnchorWithAlternation from "./rules/regex-anchor-with-alternation.js";
-import noNonNumericArrayIndex from "./rules/no-non-numeric-array-index.js";
-import arrayMethodMissingArgs from "./rules/array-method-missing-args.js";
-import noMisleadingArrayMutation from "./rules/no-misleading-array-mutation.js";
-import noUnsafeWindowOpen from "./rules/no-unsafe-window-open.js";
-import noConstantBinaryExpression from "./rules/no-constant-binary-expression.js";
-import noBitwiseInBoolean from "./rules/no-bitwise-in-boolean.js";
-import noInvertedBooleanCheck from "./rules/no-inverted-boolean-check.js";
-import noGratuitousBoolean from "./rules/no-gratuitous-boolean.js";
-import noRedundantBooleanCondition from "./rules/no-redundant-boolean-condition.js";
-import noRedundantCallApply from "./rules/no-redundant-call-apply.js";
+const noNestedTernary = require("./rules/no-nested-ternary.js");
+const awaitOnlyPromises = require("./rules/await-only-promises.js");
+const forLoopUpdateCounter = require("./rules/for-loop-update-counter.js");
+const maxSwitchCases = require("./rules/max-switch-cases.js");
+const noArrayDelete = require("./rules/no-array-delete.js");
+const noExampleRule = require("./rules/no-example-rule.js");
+const noInArray = require("./rules/no-in-array.js");
+const noInWithPrimitives = require("./rules/no-in-with-primitives.js");
+const noIndexOfComparePositive = require("./rules/no-index-of-compare-positive.js");
+const noLabelInSwitch = require("./rules/no-label-in-switch.js");
+const noThisInFunctionalComponent = require("./rules/no-this-in-functional-component.js");
+const preferForOf = require("./rules/prefer-for-of.js");
+const preferRegexExec = require("./rules/prefer-regex-exec.js");
+const requireReduceInitialValue = require("./rules/require-reduce-initial-value.js");
+const requireSortCompare = require("./rules/require-sort-compare.js");
+const useStateCallback = require("./rules/use-state-callback.js");
+const preferWhile = require("./rules/prefer-while.js");
+const noIdenticalBranches = require("./rules/no-identical-branches.js");
+const noEmptyRegexAlternatives = require("./rules/no-empty-regex-alternatives.js");
+const regexAnchorWithAlternation = require("./rules/regex-anchor-with-alternation.js");
+const noNonNumericArrayIndex = require("./rules/no-non-numeric-array-index.js");
+const arrayMethodMissingArgs = require("./rules/array-method-missing-args.js");
+const noMisleadingArrayMutation = require("./rules/no-misleading-array-mutation.js");
+const noUnsafeWindowOpen = require("./rules/no-unsafe-window-open.js");
+const noConstantBinaryExpression = require("./rules/no-constant-binary-expression.js");
+const noBitwiseInBoolean = require("./rules/no-bitwise-in-boolean.js");
+const noInvertedBooleanCheck = require("./rules/no-inverted-boolean-check.js");
+const noGratuitousBoolean = require("./rules/no-gratuitous-boolean.js");
+const noRedundantBooleanCondition = require("./rules/no-redundant-boolean-condition.js");
+const noRedundantCallApply = require("./rules/no-redundant-call-apply.js");
 
 const rulesConfig = {
   "no-nested-ternary": { rule: noNestedTernary, level: "error" },
@@ -114,4 +114,4 @@ const plugin = {
   },
 };
 
-export default plugin;
+module.exports = plugin;

package/package.json

@@ -1,9 +1,8 @@
 {
   "name": "eslint-plugin-sonar-config",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "ESLint plugin com regras customizadas para melhorar a qualidade de código em projeto com Sonar implementado.",
   "main": "index.js",
-  "type": "module",
   "keywords": [
     "eslint",
     "eslintplugin",
@@ -13,12 +12,9 @@
   ],
   "license": "MIT",
   "peerDependencies": {
-    "eslint": "8.57.0"
+    "eslint": ">=8.0.0"
   },
   "engines": {
-    "node": "16.0.0"
-  },
-  "exports": {
-    ".": "./index.js"
+    "node": ">=16.0.0"
   }
 }

package/rules/array-method-missing-args.js

@@ -2,7 +2,7 @@
  * @fileoverview Métodos push/unshift devem receber pelo menos 1 argumento
  * [].push() ou arr.push() sem argumentos é inútil e provavelmente um bug
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/await-only-promises.js

@@ -8,7 +8,7 @@
  */
 
 /** @type {import('eslint').Rule.RuleModule} */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/for-loop-update-counter.js

@@ -2,7 +2,7 @@
  * @fileoverview O update do for loop deve modificar a variável usada na condição
  * Evita bugs onde o incremento modifica uma variável diferente da condição.
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/max-switch-cases.js

@@ -4,7 +4,7 @@
  */
 const MAX_CASES = 30;
 
-export default {
+module.exports = {
   meta: {
     type: "suggestion",
     docs: {

package/rules/no-array-delete.js

@@ -8,7 +8,7 @@
  */
 
 /** @type {import('eslint').Rule.RuleModule} */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-bitwise-in-boolean.js

@@ -2,7 +2,7 @@
  * @fileoverview Operadores bitwise não devem ser usados em contextos booleanos
  * if (a & b) provavelmente deveria ser if (a && b)
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-constant-binary-expression.js

@@ -2,7 +2,7 @@
  * @fileoverview Expressões que sempre retornam o mesmo valor
  * Cobre: a < a, true && x, x === new Object(), if ({})
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-empty-regex-alternatives.js

@@ -2,7 +2,7 @@
  * @fileoverview Alternativas em regex não devem ser vazias
  * /a|/ ou /|a/ ou /a||b/ são problemáticos
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-example-rule.js

@@ -8,7 +8,7 @@
  */
 
 /** @type {import('eslint').Rule.RuleModule} */
-export default {
+module.exports = {
   meta: {
     type: "suggestion", // "problem" | "suggestion" | "layout"
     docs: {

package/rules/no-gratuitous-boolean.js

@@ -2,7 +2,7 @@
  * @fileoverview Expressões booleanas não devem ser gratuitas
  * Cobre: a === true, a ? true : false, !a ? false : true
  */
-export default {
+module.exports = {
   meta: {
     type: "suggestion",
     docs: {

package/rules/no-identical-branches.js

@@ -1,7 +1,7 @@
 /**
  * @fileoverview Branches de if/else/ternário/switch não devem ter implementações idênticas
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-in-array.js

@@ -3,7 +3,7 @@
  * O operador "in" verifica propriedades do objeto, não índices do array.
  * Use includes(), indexOf() ou hasOwnProperty() para arrays.
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-in-with-primitives.js

@@ -2,7 +2,7 @@
  * @fileoverview Não usar operador "in" com tipos primitivos
  * O operador "in" com primitivos (string, number, boolean) causa TypeError.
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-index-of-compare-positive.js

@@ -2,7 +2,7 @@
  * @fileoverview indexOf não deve ser comparado com números positivos
  * Comparar indexOf > 0 ignora o primeiro elemento. Use >= 0 ou includes().
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-inverted-boolean-check.js

@@ -2,7 +2,7 @@
  * @fileoverview Checks booleanos não devem ser invertidos desnecessariamente
  * !(a === b) deveria ser a !== b
  */
-export default {
+module.exports = {
   meta: {
     type: "suggestion",
     docs: {

package/rules/no-label-in-switch.js

@@ -2,7 +2,7 @@
  * @fileoverview Switch não deve conter labels que não são case
  * Misturar labels e cases no switch é confuso e pode ser erro de digitação.
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-misleading-array-mutation.js

@@ -2,7 +2,7 @@
  * @fileoverview Métodos que mutam arrays não devem ser usados de forma enganosa
  * const reversed = arr.reverse() é problemático pois arr também é mutado
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-nested-ternary.js

@@ -6,7 +6,7 @@
  */
 
 /** @type {import('eslint').Rule.RuleModule} */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-non-numeric-array-index.js

@@ -2,7 +2,7 @@
  * @fileoverview Índices de arrays devem ser numéricos
  * arr["foo"] é problemático, deve usar objeto ao invés de array
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-redundant-boolean-condition.js

@@ -2,7 +2,7 @@
  * @fileoverview Condições booleanas não devem ser redundantes
  * if (a) { if (a) { ... } } - o segundo if é redundante
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-redundant-call-apply.js

@@ -2,7 +2,7 @@
  * @fileoverview Chamadas .call() e .apply() não devem ser redundantes
  * fn.call(undefined, a, b) deveria ser fn(a, b)
  */
-export default {
+module.exports = {
   meta: {
     type: "suggestion",
     docs: {

package/rules/no-this-in-functional-component.js

@@ -2,7 +2,7 @@
  * @fileoverview Não usar "this" em componentes funcionais React
  * Componentes funcionais não têm instância, então "this" não faz sentido.
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/no-unsafe-window-open.js

@@ -2,7 +2,7 @@
  * @fileoverview window.open() deve usar 'noopener' para segurança
  * Sem noopener, a janela aberta pode acessar window.opener
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/prefer-for-of.js

@@ -2,7 +2,7 @@
  * @fileoverview Preferir for...of ao invés de for tradicional com índice
  * Quando o índice é usado apenas para acessar elementos do array, for...of é mais legível.
  */
-export default {
+module.exports = {
   meta: {
     type: "suggestion",
     docs: {

package/rules/prefer-regex-exec.js

@@ -2,7 +2,7 @@
  * @fileoverview Preferir RegExp.exec() ao invés de String.match()
  * RegExp.exec() é mais rápido e produz o mesmo resultado quando não há flag 'g'.
  */
-export default {
+module.exports = {
   meta: {
     type: "suggestion",
     docs: {

package/rules/prefer-while.js

@@ -2,7 +2,7 @@
  * @fileoverview Use "while" ao invés de "for" quando não há init e update
  * for (; condition; ) {} → while (condition) {}
  */
-export default {
+module.exports = {
   meta: {
     type: "suggestion",
     docs: {

package/rules/regex-anchor-with-alternation.js

@@ -2,7 +2,7 @@
  * @fileoverview Alternativas em regex devem ser agrupadas quando usadas com anchors (^ ou $)
  * /^a|b/ deveria ser /^(a|b)/ ou /^a|^b/
  */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/require-reduce-initial-value.js

@@ -5,7 +5,7 @@
  */
 
 /** @type {import('eslint').Rule.RuleModule} */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/require-sort-compare.js

@@ -5,7 +5,7 @@
  */
 
 /** @type {import('eslint').Rule.RuleModule} */
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {

package/rules/use-state-callback.js

@@ -35,7 +35,7 @@ function containsIdentifier(node, name) {
   return false;
 }
 
-export default {
+module.exports = {
   meta: {
     type: "problem",
     docs: {