pr-checkmate 1.21.7 → 1.21.8

package/.eslintrc.js

@@ -21,7 +21,7 @@ function loadUserConfig() {
   if (fs.existsSync(configPath)) {
     try {
       const config = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
-      console.log(`[ESLint]: ✅ Loaded config from: ${configPath}`);
+      console.error(`[ESLint]: ✅ Loaded config from: ${configPath}`);
       return config;
     } catch (err) {
       console.warn(`[ESLint]: ⚠️  Failed to parse ${configPath}:`, err.message);

package/cspell.json

@@ -2,11 +2,11 @@
   "version": "0.2",
   "language": "en,ru,uk,he,fr,es",
   "import": [
-    "./node_modules/@cspell/dict-ru_ru/cspell-ext.json",
-    "./node_modules/@cspell/dict-uk-ua/cspell-ext.json",
-    "./node_modules/@cspell/dict-he/cspell-ext.json",
-    "./node_modules/@cspell/dict-fr-fr/cspell-ext.json",
-    "./node_modules/@cspell/dict-es-es/cspell-ext.json"
+    "@cspell/dict-ru_ru/cspell-ext.json",
+    "@cspell/dict-uk-ua/cspell-ext.json",
+    "@cspell/dict-he/cspell-ext.json",
+    "@cspell/dict-fr-fr/cspell-ext.json",
+    "@cspell/dict-es-es/cspell-ext.json"
   ],
   "words": [
     "ACMR",

package/dist/config/loader.js

@@ -59,6 +59,16 @@ function mergeWithDefaults(userConfig) {
         prBody: { ...defaults.prBody, ...userConfig.prBody },
         duplicate: { ...defaults.duplicate, ...userConfig.duplicate },
         commitlint: { ...defaults.commitlint, ...userConfig.commitlint },
+        licenseCheck: {
+            allowPackages: [
+                ...(defaults.licenseCheck?.allowPackages || []),
+                ...(userConfig.licenseCheck?.allowPackages || []),
+            ],
+            allowLicenses: [
+                ...(defaults.licenseCheck?.allowLicenses || []),
+                ...(userConfig.licenseCheck?.allowLicenses || []),
+            ],
+        },
     };
 }
 /**

package/dist/scripts/commitlint.js

@@ -39,7 +39,11 @@ const DEFAULT_TYPES = [
 ];
 function buildTempConfig(types) {
     const config = {
-        extends: [(0, utils_1.resolvePackageBin)('@commitlint/config-conventional', 'index.js')],
+        // Resolve the config package to its real entry point via Node module
+        // resolution. It works regardless of npm hoisting in the client project and
+        // regardless of the package's internal layout (e.g. lib/index.js vs index.js)
+        // or exports map — unlike hardcoding a subpath.
+        extends: [require.resolve('@commitlint/config-conventional')],
         rules: {
             'type-enum': [2, 'always', types],
         },

package/dist/scripts/copy-configs.js

@@ -11,7 +11,6 @@ const configFiles = [
     { source: 'src/config/eslint.config.mjs', dest: 'eslint.config.mjs' },
     { source: 'src/config/.eslintrc.js', dest: '.eslintrc.js' },
     { source: 'src/config/.prettierrc', dest: '.prettierrc' },
-    { source: 'src/config/.eslintignore', dest: '.eslintignore' },
     { source: 'src/config/cspell.json', dest: 'cspell.json' },
     { source: 'tsconfig.json', dest: 'tsconfig.json' },
     { source: 'src/config/pr-checkmate-workflow.yml', dest: 'dist/config/pr-checkmate-workflow.yml' },

package/dist/scripts/dependency-check.js

@@ -30,9 +30,14 @@ async function runDependencyCheck() {
             throw err;
         }
     }
+    // Tools loaded by depcheck (e.g. eslint configs) may print to stdout ahead of
+    // the JSON, so slice from the first "{" to the last "}" before parsing.
+    const start = stdout.indexOf('{');
+    const end = stdout.lastIndexOf('}');
+    const json = start !== -1 && end !== -1 ? stdout.slice(start, end + 1) : stdout;
     let result;
     try {
-        result = JSON.parse(stdout);
+        result = JSON.parse(json);
     }
     catch {
         utils_1.logger.error('[runDependencyCheck]: ❌ Failed to parse depcheck output');

package/dist/scripts/license-check.js

@@ -6,11 +6,23 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.runLicenseCheck = runLicenseCheck;
 const execa_1 = require("execa");
 const utils_1 = require("../utils");
+const loader_1 = require("../config/loader");
 const node_path_1 = __importDefault(require("node:path"));
 const node_fs_1 = __importDefault(require("node:fs"));
 const FORBIDDEN_LICENSES = ['GPL-2.0', 'GPL-3.0', 'AGPL-3.0', 'LGPL-2.0', 'LGPL-2.1', 'LGPL-3.0'];
+/**
+ * Strip the trailing "@version" from a license-checker key to get the bare
+ * package name. Handles scoped packages, e.g. "@cspell/dict-ru_ru@2.3.2" → "@cspell/dict-ru_ru".
+ */
+function bareName(pkgKey) {
+    const at = pkgKey.lastIndexOf('@');
+    return at > 0 ? pkgKey.slice(0, at) : pkgKey;
+}
 async function runLicenseCheck() {
     utils_1.logger.info('[runLicenseCheck]: Checking dependency licenses...');
+    const cfg = (0, loader_1.loadProjectConfig)();
+    const allowPackages = new Set(cfg.licenseCheck?.allowPackages ?? []);
+    const allowLicenses = cfg.licenseCheck?.allowLicenses ?? [];
     if (!node_fs_1.default.existsSync(node_path_1.default.join(process.cwd(), 'package.json'))) {
         utils_1.logger.warn('[runLicenseCheck]: ⚠️ No package.json found, skipping');
         return;
@@ -22,8 +34,12 @@ async function runLicenseCheck() {
         const licenses = JSON.parse(stdout);
         const violations = [];
         for (const [pkg, info] of Object.entries(licenses)) {
+            if (allowPackages.has(bareName(pkg)))
+                continue;
             const raw = info.licenses ?? '';
             const license = Array.isArray(raw) ? raw.join(' OR ') : raw;
+            if (allowLicenses.some(a => license.includes(a)))
+                continue;
             if (FORBIDDEN_LICENSES.some(f => license.includes(f))) {
                 violations.push(`  ${pkg}: ${license}`);
             }

package/dist/scripts/spellcheck.js

@@ -35,8 +35,22 @@ function createTempCspellConfig() {
     const packageConfigDir = node_path_1.default.dirname(packageConfigPath);
     const defaultConfig = JSON.parse(node_fs_1.default.readFileSync(packageConfigPath, 'utf-8'));
     const userConfig = loadUserCspellConfig();
-    // Resolve relative import paths to absolute so they work from any client cwd
-    const resolvedImports = (defaultConfig.import ?? []).map(p => p.startsWith('.') ? node_path_1.default.resolve(packageConfigDir, p) : p);
+    // Resolve import paths to absolute so they work from any client cwd.
+    // Relative paths are resolved against the package config dir; bare package
+    // specifiers (e.g. "@cspell/dict-ru_ru/cspell-ext.json") are resolved via Node
+    // module resolution so they work regardless of npm hoisting in the client
+    // project (dicts may live in the root node_modules, not nested under us).
+    const resolvedImports = (defaultConfig.import ?? []).map(p => {
+        if (p.startsWith('.')) {
+            return node_path_1.default.resolve(packageConfigDir, p);
+        }
+        try {
+            return require.resolve(p);
+        }
+        catch {
+            return p;
+        }
+    });
     // Merge configurations
     const mergedConfig = {
         ...defaultConfig,

package/eslint.config.mjs

@@ -76,13 +76,13 @@ function loadUserConfig() {
   if (fs.existsSync(configPath)) {
     try {
       const config = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
-      console.log(`[loadUserConfig]: ✅ Loaded user config from: ${configPath}`);
+      console.error(`[loadUserConfig]: ✅ Loaded user config from: ${configPath}`);
       return config;
     } catch (err) {
       console.warn(`[loadUserConfig]: ⚠️ Failed to parse ${configPath}:`, err.message);
     }
   } else {
-    console.log(`[loadUserConfig]: ℹ️ No pr-checkmate.json found, using defaults`);
+    console.error(`[loadUserConfig]: ℹ️ No pr-checkmate.json found, using defaults`);
   }
   
   return { sourcePath: 'src', eslint: {} };
@@ -110,13 +110,13 @@ const customRulesCount = Object.keys(userConfig.eslint?.rules || {}).length;
 const customIgnoresCount = (userConfig.eslint?.ignorePatterns || []).length;
 
 if (customRulesCount > 0) {
-  console.log(`🔧 Applied ${customRulesCount} custom ESLint rules`);
+  console.error(`🔧 Applied ${customRulesCount} custom ESLint rules`);
 }
 if (customIgnoresCount > 0) {
-  console.log(`🚫 Applied ${customIgnoresCount} custom ignore patterns`);
+  console.error(`🚫 Applied ${customIgnoresCount} custom ignore patterns`);
 }
 
-console.log(`📋 ESLint checking: ${SRC}/**/*.{ts,tsx,js,jsx}`);
+console.error(`📋 ESLint checking: ${SRC}/**/*.{ts,tsx,js,jsx}`);
 
 export default [
   // Global ignores must be in a separate config object

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "pr-checkmate",
-    "version": "1.21.7",
+    "version": "1.21.8",
     "description": "Automated PR quality checks: linting, formatting, dependency analysis, and spellcheck",
     "keywords": [
         "github-actions",
@@ -23,7 +23,6 @@
         "eslint.config.mjs",
         ".eslintrc.js",
         ".prettierrc",
-        ".eslintignore",
         "cspell.json",
         "src/config/pr-checkmate-workflow.yml",
         ".markdownlint.json",
@@ -67,10 +66,10 @@
         "@cspell/dict-he": "^4.0.2",
         "@cspell/dict-ru_ru": "^2.2.1",
         "@cspell/dict-uk-ua": "^4.0.0",
+        "@types/node": "^25.9.4",
         "@typescript-eslint/eslint-plugin": "^8.47.0",
         "@typescript-eslint/parser": "^8.47.0",
         "cspell": "^9.3.2",
-        "dotenv": "^17.2.3",
         "eslint": "^9.26.0",
         "eslint-config-prettier": "^9.1.2",
         "eslint-plugin-import": "^2.29.0",

package/src/config/.eslintrc.js

@@ -21,7 +21,7 @@ function loadUserConfig() {
   if (fs.existsSync(configPath)) {
     try {
       const config = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
-      console.log(`[ESLint]: ✅ Loaded config from: ${configPath}`);
+      console.error(`[ESLint]: ✅ Loaded config from: ${configPath}`);
       return config;
     } catch (err) {
       console.warn(`[ESLint]: ⚠️  Failed to parse ${configPath}:`, err.message);

package/src/config/cspell.json

@@ -2,11 +2,11 @@
   "version": "0.2",
   "language": "en,ru,uk,he,fr,es",
   "import": [
-    "./node_modules/@cspell/dict-ru_ru/cspell-ext.json",
-    "./node_modules/@cspell/dict-uk-ua/cspell-ext.json",
-    "./node_modules/@cspell/dict-he/cspell-ext.json",
-    "./node_modules/@cspell/dict-fr-fr/cspell-ext.json",
-    "./node_modules/@cspell/dict-es-es/cspell-ext.json"
+    "@cspell/dict-ru_ru/cspell-ext.json",
+    "@cspell/dict-uk-ua/cspell-ext.json",
+    "@cspell/dict-he/cspell-ext.json",
+    "@cspell/dict-fr-fr/cspell-ext.json",
+    "@cspell/dict-es-es/cspell-ext.json"
   ],
   "words": [
     "ACMR",

package/src/config/eslint.config.mjs

@@ -76,13 +76,13 @@ function loadUserConfig() {
   if (fs.existsSync(configPath)) {
     try {
       const config = JSON.parse(fs.readFileSync(configPath, 'utf-8'));
-      console.log(`[loadUserConfig]: ✅ Loaded user config from: ${configPath}`);
+      console.error(`[loadUserConfig]: ✅ Loaded user config from: ${configPath}`);
       return config;
     } catch (err) {
       console.warn(`[loadUserConfig]: ⚠️ Failed to parse ${configPath}:`, err.message);
     }
   } else {
-    console.log(`[loadUserConfig]: ℹ️ No pr-checkmate.json found, using defaults`);
+    console.error(`[loadUserConfig]: ℹ️ No pr-checkmate.json found, using defaults`);
   }
   
   return { sourcePath: 'src', eslint: {} };
@@ -110,13 +110,13 @@ const customRulesCount = Object.keys(userConfig.eslint?.rules || {}).length;
 const customIgnoresCount = (userConfig.eslint?.ignorePatterns || []).length;
 
 if (customRulesCount > 0) {
-  console.log(`🔧 Applied ${customRulesCount} custom ESLint rules`);
+  console.error(`🔧 Applied ${customRulesCount} custom ESLint rules`);
 }
 if (customIgnoresCount > 0) {
-  console.log(`🚫 Applied ${customIgnoresCount} custom ignore patterns`);
+  console.error(`🚫 Applied ${customIgnoresCount} custom ignore patterns`);
 }
 
-console.log(`📋 ESLint checking: ${SRC}/**/*.{ts,tsx,js,jsx}`);
+console.error(`📋 ESLint checking: ${SRC}/**/*.{ts,tsx,js,jsx}`);
 
 export default [
   // Global ignores must be in a separate config object

package/.eslintignore

@@ -1,72 +0,0 @@
-# Dependencies
-node_modules/
-
-# Build outputs
-dist/
-build/
-out/
-.next/
-.nuxt/
-
-# Test reports & artifacts
-allure-report/
-allure-results/
-reports/
-report/
-test-reports/
-test-report/
-test-output/
-test-results/
-results/
-result/
-coverage/
-.coverage/
-htmlcov/
-
-# Jest
-jest-html-reporters-*/
-jest-stare/
-jest.config.js
-
-# Cypress
-cypress/reports/
-cypress/videos/
-cypress/screenshots/
-
-# Playwright
-playwright-report/
-test-results/
-
-# WebdriverIO
-.wdio/
-wdio-logs/
-
-# Mocha/Mochawesome
-mochawesome-report/
-mochawesome-reports/
-
-# Maven/JUnit
-target/
-
-# Logs
-logs/
-*.log
-npm-debug.log*
-yarn-debug.log*
-
-# Python
-__pycache__/
-.pytest_cache/
-.venv/
-venv/
-
-# Git
-.git/
-
-# Config files
-*.config.js
-*.config.ts
-jest.config.*
-wdio.config.*
-playwright.config.*
-cypress.config.*

package/src/config/.eslintignore

@@ -1,72 +0,0 @@
-# Dependencies
-node_modules/
-
-# Build outputs
-dist/
-build/
-out/
-.next/
-.nuxt/
-
-# Test reports & artifacts
-allure-report/
-allure-results/
-reports/
-report/
-test-reports/
-test-report/
-test-output/
-test-results/
-results/
-result/
-coverage/
-.coverage/
-htmlcov/
-
-# Jest
-jest-html-reporters-*/
-jest-stare/
-jest.config.js
-
-# Cypress
-cypress/reports/
-cypress/videos/
-cypress/screenshots/
-
-# Playwright
-playwright-report/
-test-results/
-
-# WebdriverIO
-.wdio/
-wdio-logs/
-
-# Mocha/Mochawesome
-mochawesome-report/
-mochawesome-reports/
-
-# Maven/JUnit
-target/
-
-# Logs
-logs/
-*.log
-npm-debug.log*
-yarn-debug.log*
-
-# Python
-__pycache__/
-.pytest_cache/
-.venv/
-venv/
-
-# Git
-.git/
-
-# Config files
-*.config.js
-*.config.ts
-jest.config.*
-wdio.config.*
-playwright.config.*
-cypress.config.*