From f0708aeba87b1b44fa8251056ea98bda8ca307c9 Mon Sep 17 00:00:00 2001 From: Ryan Bahan Date: Thu, 2 Apr 2026 18:02:45 -0600 Subject: [PATCH] Remove debug dependency from eslint-plugin-cli The only usage was a single diagnostic log line in no-inline-graphql. Replace with a conditional console.error gated on process.env.DEBUG. Co-Authored-By: Claude Opus 4.6 (1M context) --- packages/eslint-plugin-cli/package.json | 3 +-- packages/eslint-plugin-cli/rules/no-inline-graphql.js | 4 ++-- pnpm-lock.yaml | 4 ---- 3 files changed, 3 insertions(+), 8 deletions(-) diff --git a/packages/eslint-plugin-cli/package.json b/packages/eslint-plugin-cli/package.json index 549177bf99e..3183b3105ef 100644 --- a/packages/eslint-plugin-cli/package.json +++ b/packages/eslint-plugin-cli/package.json @@ -29,8 +29,7 @@ "eslint-plugin-unused-imports": "4.1.4", "@vitest/eslint-plugin": "1.1.44", "globals": "16.2.0", - "execa": "7.2.0", - "debug": "4.4.0" + "execa": "7.2.0" }, "devDependencies": { "prettier": "3.8.1" diff --git a/packages/eslint-plugin-cli/rules/no-inline-graphql.js b/packages/eslint-plugin-cli/rules/no-inline-graphql.js index 7acf5006eaa..8959ccda80b 100644 --- a/packages/eslint-plugin-cli/rules/no-inline-graphql.js +++ b/packages/eslint-plugin-cli/rules/no-inline-graphql.js @@ -2,7 +2,7 @@ const path = require('path') const fs = require('fs') const crypto = require('crypto') -const debug = require('debug')('eslint-plugin-cli:no-inline-graphql') +const debugEnabled = process.env.DEBUG && process.env.DEBUG.includes('eslint-plugin-cli') /** * Check if using a gql`` template literal @@ -38,7 +38,7 @@ function checkKnownFailuresIfShouldFail(context) { const shouldFail = !knownFailures[relativePath] || knownFailures[relativePath] !== fileHash if (shouldFail) { - debug(`Reporting inline GraphQL tag fail for - '${relativePath}': '${fileHash}',`) + if (debugEnabled) console.error(`eslint-plugin-cli:no-inline-graphql Reporting inline GraphQL tag fail for - '${relativePath}': '${fileHash}',`) } return shouldFail diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml index 6b0c7b1cf40..b76d2f5954e 100644 --- a/pnpm-lock.yaml +++ b/pnpm-lock.yaml @@ -569,9 +569,6 @@ importers: '@vitest/eslint-plugin': specifier: 1.1.44 version: 1.1.44(@typescript-eslint/utils@8.56.1(eslint@9.39.3(jiti@2.6.1))(typescript@5.9.3))(eslint@9.39.3(jiti@2.6.1))(typescript@5.9.3)(vitest@3.2.4(@types/node@22.19.15)(jiti@2.6.1)(jsdom@28.1.0)(msw@2.12.10(@types/node@22.19.15)(typescript@5.9.3))(sass@1.97.3)(yaml@2.8.3)) - debug: - specifier: 4.4.0 - version: 4.4.0(supports-color@8.1.1) eslint: specifier: ^9.0.0 version: 9.39.3(jiti@2.6.1) @@ -12595,7 +12592,6 @@ snapshots: '@typescript-eslint/eslint-plugin': 8.56.1(@typescript-eslint/parser@8.56.1(eslint@9.39.3(jiti@2.6.1))(typescript@5.9.3))(eslint@9.39.3(jiti@2.6.1))(typescript@5.9.3) '@typescript-eslint/parser': 8.56.1(eslint@9.39.3(jiti@2.6.1))(typescript@5.9.3) '@vitest/eslint-plugin': 1.1.44(@typescript-eslint/utils@8.56.1(eslint@9.39.3(jiti@2.6.1))(typescript@5.9.3))(eslint@9.39.3(jiti@2.6.1))(typescript@5.9.3)(vitest@3.2.4(@types/node@18.19.70)(jiti@2.6.1)(jsdom@28.1.0)(msw@2.12.10(@types/node@18.19.70)(typescript@5.9.3))(sass@1.97.3)(yaml@2.8.3)) - debug: 4.4.0(supports-color@8.1.1) eslint: 9.39.3(jiti@2.6.1) eslint-config-prettier: 10.1.5(eslint@9.39.3(jiti@2.6.1)) eslint-plugin-jsdoc: 50.7.1(eslint@9.39.3(jiti@2.6.1))