pr-checkmate 1.21.11 → 1.21.12

package/dist/cli.js

@@ -27,6 +27,8 @@ const COMMAND_TO_CHECK = {
     duplicate: 'Duplicate Code',
     commitlint: 'Commitlint',
     'cpp-format': 'C++ Format',
+    'python-lint': 'Ruff Lint',
+    'python-format': 'Ruff Format',
 };
 const cmd = process.argv[2];
 void (async () => {

package/dist/config/loader.js

@@ -61,6 +61,11 @@ function mergeWithDefaults(userConfig) {
             ...userConfig.cpp,
             clangFormat: { ...defaults.cpp?.clangFormat, ...userConfig.cpp?.clangFormat },
         },
+        python: {
+            ...defaults.python,
+            ...userConfig.python,
+            ruff: { ...defaults.python?.ruff, ...userConfig.python?.ruff },
+        },
         diffSmell: { ...defaults.diffSmell, ...userConfig.diffSmell },
         prBody: { ...defaults.prBody, ...userConfig.prBody },
         duplicate: { ...defaults.duplicate, ...userConfig.duplicate },
@@ -133,6 +138,7 @@ function getDefaultConfig() {
         },
         typecheck: { enabled: true },
         cpp: { enabled: true, clangFormat: { style: 'LLVM' } },
+        python: { enabled: true, ruff: { lint: true, format: true } },
         diffSmell: { enabled: true },
         prBody: { enabled: true, minLength: 20, requireTicket: true },
         duplicate: { enabled: true, minLines: 5, minTokens: 50 },

package/dist/core/checks/python-format.js

@@ -0,0 +1,68 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.pythonFormatCheck = void 0;
+const node_fs_1 = __importDefault(require("node:fs"));
+const node_path_1 = __importDefault(require("node:path"));
+const utils_1 = require("../../utils");
+const targets_1 = require("../targets");
+const types_1 = require("../types");
+const PY_GLOBS = ['*.py'];
+/**
+ * Formats changed Python files with Ruff's formatter (WASM, no Python/pip needed).
+ * Read-only in check mode (`warn` when files differ from formatted output); rewrites
+ * files in place when `write` so CI can auto-commit — mirrors the Prettier/C++ checks.
+ */
+exports.pythonFormatCheck = {
+    name: 'Ruff Format',
+    phase: 'format',
+    applies(ctx) {
+        return (ctx.config.python?.enabled !== false &&
+            ctx.config.python?.ruff?.format !== false &&
+            ctx.languages.includes('python'));
+    },
+    async run(ctx) {
+        const targets = await (0, targets_1.resolveTargetFiles)(ctx, PY_GLOBS);
+        if (targets === null)
+            return (0, types_1.skip)('git unavailable');
+        if (targets.length === 0)
+            return (0, types_1.pass)('no Python files changed');
+        const { Workspace, PositionEncoding } = require('@astral-sh/ruff-wasm-nodejs');
+        const workspace = new Workspace(Workspace.defaultSettings(), PositionEncoding.Utf16);
+        let unformatted = 0;
+        for (const rel of targets) {
+            const abs = node_path_1.default.join(ctx.cwd, rel);
+            let content;
+            try {
+                content = node_fs_1.default.readFileSync(abs, 'utf-8');
+            }
+            catch {
+                continue; // renamed/removed between diff and read — skip
+            }
+            let formatted;
+            try {
+                formatted = workspace.format(content);
+            }
+            catch (err) {
+                // A syntax error makes Ruff throw — that's a lint concern, not a format failure.
+                utils_1.logger.warn(`[Ruff Format]: ⚠️ Could not format ${rel} (syntax error?)\n${err}`);
+                continue;
+            }
+            if (formatted === content)
+                continue;
+            unformatted += 1;
+            if (ctx.write)
+                node_fs_1.default.writeFileSync(abs, formatted);
+        }
+        if (unformatted === 0)
+            return (0, types_1.pass)();
+        if (!ctx.write) {
+            utils_1.logger.warn(`[Ruff Format]: ⚠️ ${unformatted} Python file(s) need formatting`);
+            return (0, types_1.warn)(`${unformatted} file(s) need formatting (run with write to fix)`);
+        }
+        utils_1.logger.info(`[Ruff Format]: ✅ Reformatted ${unformatted} Python file(s)`);
+        return (0, types_1.pass)(`reformatted ${unformatted} file(s)`);
+    },
+};

package/dist/core/checks/python-lint.js

@@ -0,0 +1,62 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.pythonLintCheck = void 0;
+const node_fs_1 = __importDefault(require("node:fs"));
+const node_path_1 = __importDefault(require("node:path"));
+const utils_1 = require("../../utils");
+const targets_1 = require("../targets");
+const types_1 = require("../types");
+const PY_GLOBS = ['*.py'];
+/**
+ * Lints changed Python files with Ruff, bundled as a self-contained WASM module
+ * (`@astral-sh/ruff-wasm-nodejs`) so it runs on a bare Node image with no Python
+ * or pip install. Informational by design — reports issues without failing the
+ * build, matching how formatting checks roll out; promote to blocking per repo
+ * once teams have adopted it. Uses Ruff's default rule set (pyflakes + pycodestyle).
+ */
+exports.pythonLintCheck = {
+    name: 'Ruff Lint',
+    phase: 'informational',
+    applies(ctx) {
+        return (ctx.config.python?.enabled !== false &&
+            ctx.config.python?.ruff?.lint !== false &&
+            ctx.languages.includes('python'));
+    },
+    async run(ctx) {
+        const targets = await (0, targets_1.resolveTargetFiles)(ctx, PY_GLOBS);
+        if (targets === null)
+            return (0, types_1.skip)('git unavailable');
+        if (targets.length === 0)
+            return (0, types_1.pass)('no Python files changed');
+        const { Workspace, PositionEncoding } = require('@astral-sh/ruff-wasm-nodejs');
+        const workspace = new Workspace(Workspace.defaultSettings(), PositionEncoding.Utf16);
+        let totalIssues = 0;
+        let filesWithIssues = 0;
+        for (const rel of targets) {
+            let content;
+            try {
+                content = node_fs_1.default.readFileSync(node_path_1.default.join(ctx.cwd, rel), 'utf-8');
+            }
+            catch {
+                continue; // renamed/removed between diff and read — skip
+            }
+            const diagnostics = workspace.check(content);
+            if (diagnostics.length === 0)
+                continue;
+            filesWithIssues += 1;
+            totalIssues += diagnostics.length;
+            utils_1.logger.warn(`[Ruff Lint]: ⚠️ ${rel}: ${diagnostics.length} issue(s)`);
+            diagnostics
+                .slice(0, 5)
+                .forEach((d) => utils_1.logger.warn(`    ${rel}:${d.start_location.row} ${d.code ?? '?'} ${d.message}`));
+            if (diagnostics.length > 5)
+                utils_1.logger.warn(`    ... and ${diagnostics.length - 5} more`);
+        }
+        if (totalIssues === 0)
+            return (0, types_1.pass)();
+        return (0, types_1.warn)(`${totalIssues} issue(s) in ${filesWithIssues} file(s)`);
+    },
+};

package/dist/core/registry.js

@@ -21,9 +21,11 @@ const pr_title_lint_1 = require("./checks/pr-title-lint");
 const outdated_deps_1 = require("./checks/outdated-deps");
 const markdown_lint_1 = require("./checks/markdown-lint");
 const spellcheck_1 = require("./checks/spellcheck");
+const python_lint_1 = require("./checks/python-lint");
 // format — runs last, may write files
 const prettier_1 = require("./checks/prettier");
 const cpp_format_1 = require("./checks/cpp-format");
+const python_format_1 = require("./checks/python-format");
 /**
  * Every check, in declaration order. The orchestrator groups them by `phase`
  * (blocking → informational → format) and runs each phase's checks in parallel.
@@ -49,7 +51,9 @@ exports.REGISTRY = [
     outdated_deps_1.outdatedDepsCheck,
     markdown_lint_1.markdownLintCheck,
     spellcheck_1.spellcheckCheck,
+    python_lint_1.pythonLintCheck,
     // format
     prettier_1.prettierCheck,
     cpp_format_1.cppFormatCheck,
+    python_format_1.pythonFormatCheck,
 ];

package/dist/core/targets.js

@@ -0,0 +1,40 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.resolveTargetFiles = resolveTargetFiles;
+const execa_1 = require("execa");
+const utils_1 = require("../utils");
+/**
+ * Lists the files a language check should operate on, matched by pathspec `globs`
+ * (e.g. `['*.py']`). Uses the PR diff range when available (delta mode), else
+ * scans all tracked files. Returns an empty array when nothing matches, and
+ * `null` when git itself is unusable (caller should `skip`).
+ *
+ * Only ACMR files are returned (Added/Copied/Modified/Renamed) — deletions are
+ * excluded, so every returned path exists on disk and is safe to read.
+ */
+async function resolveTargetFiles(ctx, globs) {
+    try {
+        const args = ctx.baseSha
+            ? [
+                'diff',
+                '--name-only',
+                '--diff-filter=ACMR',
+                ctx.baseSha,
+                ctx.headSha ?? 'HEAD',
+                '--',
+                ...globs,
+            ]
+            : ['ls-files', ...globs];
+        const { stdout } = await (0, execa_1.execa)('git', args, { cwd: ctx.cwd, reject: false });
+        const files = stdout
+            .split('\n')
+            .map(f => f.trim())
+            .filter(Boolean);
+        utils_1.logger.info(`[targets]: ${ctx.baseSha ? '📂 Delta' : 'Full scan'}: ${files.length} file(s) for ${globs.join(', ')}`);
+        return files;
+    }
+    catch (err) {
+        utils_1.logger.warn(`[targets]: ⚠️ Could not list files\n${err}`);
+        return null;
+    }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "pr-checkmate",
-    "version": "1.21.11",
+    "version": "1.21.12",
     "description": "Automated PR quality checks: linting, formatting, dependency analysis, and spellcheck",
     "keywords": [
         "github-actions",
@@ -59,6 +59,7 @@
     "author": "Artur Polishchuk",
     "license": "LicenseRef-Proprietary",
     "dependencies": {
+        "@astral-sh/ruff-wasm-nodejs": "^0.15.20",
         "@commitlint/cli": "^21.2.0",
         "@commitlint/config-conventional": "^21.2.0",
         "@cspell/dict-es-es": "^3.0.8",