@codacy/tools-reek-6 0.2.4 → 0.2.6

package/dist/index.js

@@ -48,6 +48,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Public Writable Attributes",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -62,6 +63,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Boolean Parameter",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -76,6 +78,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Class Variables",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -90,6 +93,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Control Parameter",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -104,6 +108,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Detect Data Clumps in Method Parameters",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -128,6 +133,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Duplicate Method Calls Within a Method",
     enabled: false,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "allow_calls",
@@ -152,6 +158,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Detect Feature Envy",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -166,6 +173,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Instance Variable Assumption",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -180,6 +188,7 @@ var patterns_default = [
     severityLevel: "Info",
     title: "Enforce Descriptive Comments on Modules and Classes",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -194,6 +203,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Long Yield List",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -213,6 +223,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Manual Dispatch",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -227,6 +238,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Detect Missing Safe Method",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -241,6 +253,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid defining initialize method in modules",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -255,6 +268,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Nested Iterators",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -279,6 +293,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Nil Checks",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -293,6 +308,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Repeated Conditional",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -312,6 +328,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Subclassing Core Classes",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -326,6 +343,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Too Many Constants in a Class",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -345,6 +363,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Too Many Instance Variables",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -364,6 +383,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Detect Too Many Methods in Class",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -383,6 +403,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Methods with Too Many Statements",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -402,6 +423,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Uncommunicative Method Names",
     enabled: false,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "accept",
@@ -421,6 +443,7 @@ var patterns_default = [
     severityLevel: "Info",
     title: "Avoid Uncommunicative Module Names",
     enabled: false,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "accept",
@@ -440,6 +463,7 @@ var patterns_default = [
     severityLevel: "Info",
     title: "Avoid Uncommunicative Parameter Names",
     enabled: false,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "accept",
@@ -459,6 +483,7 @@ var patterns_default = [
     severityLevel: "Info",
     title: "Avoid Uncommunicative Variable Names",
     enabled: false,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "accept",
@@ -478,6 +503,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Unused Parameters in Methods",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -492,6 +518,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Avoid Unused Private Methods",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -506,6 +533,7 @@ var patterns_default = [
     severityLevel: "Warning",
     title: "Detect Utility Functions",
     enabled: true,
+    languages: ["Ruby"],
     parameters: [
       {
         name: "exclude",
@@ -521,6 +549,18 @@ var patterns_default = [
   }
 ];
 
+// src/metadata.ts
+var descriptor = {
+  id: "Reek",
+  displayName: "Reek",
+  preferredVersion: "6.5.0",
+  languages: ["Ruby"],
+  filePatterns: ["**/*.rb"],
+  supportedOS: ["linux", "darwin", "win32"],
+  supportedArch: ["x64", "arm64"],
+  supportedRunners: ["local", "container"]
+};
+
 // src/adapter.ts
 var TOOL_ID = "Reek";
 var ADAPTER_ID = "reek-6";
@@ -539,37 +579,6 @@ var PATTERN_META = new Map(
 function ruleMetadata(smellType) {
   return PATTERN_META.get(smellType) ?? { category: "CodeStyle", severity: "Info" };
 }
-function getLineContent(lines, lineNumber) {
-  return lines[lineNumber - 1] ?? "";
-}
-async function readFileLines(filePath) {
-  try {
-    const content = await import_promises.default.readFile(filePath, "utf-8");
-    return content.split("\n");
-  } catch {
-    return [];
-  }
-}
-function buildResult(startedAt, completedAt, issues, errors, filesAnalyzed) {
-  const durationMs = completedAt.getTime() - startedAt.getTime();
-  return {
-    metadata: {
-      startedAt: startedAt.toISOString(),
-      completedAt: completedAt.toISOString(),
-      durationMs
-    },
-    issues,
-    errors,
-    summary: {
-      toolId: TOOL_ID,
-      status: errors.some((e) => e.level === "error") ? "partial" : "success",
-      issueCount: issues.length,
-      errorCount: errors.length,
-      durationMs,
-      filesAnalyzed
-    }
-  };
-}
 async function resolveToolInvocation(ctx) {
   const gemPaths = (0, import_tooling.getGemPaths)(ctx.globalDir, ADAPTER_ID);
   const scriptPath = (0, import_tooling.resolveGemBinary)(gemPaths, "reek");
@@ -593,16 +602,9 @@ async function resolveToolInvocation(ctx) {
   return { binary: "reek", prefixArgs: [], rubyPath: rubyInfo?.path };
 }
 var adapter = {
-  id: TOOL_ID,
-  displayName: "Reek",
-  preferredVersion: REEK_VERSION,
+  ...descriptor,
   compatibleVersions: ">=6.0.0",
   website: "https://github.com/troessner/reek",
-  languages: ["Ruby"],
-  filePatterns: ["**/*.rb"],
-  supportedOS: ["linux", "darwin", "win32"],
-  supportedArch: ["x64", "arm64"],
-  supportedRunners: ["local", "container"],
   prerequisites: [
     {
       name: "Ruby",
@@ -767,7 +769,7 @@ var adapter = {
     const issues = [];
     const errors = [];
     if (ctx.targetFiles.length === 0) {
-      return buildResult(startedAt, /* @__PURE__ */ new Date(), issues, errors, 0);
+      return (0, import_tooling.buildResult)(TOOL_ID, startedAt, /* @__PURE__ */ new Date(), issues, errors, 0);
     }
     const invocation = await resolveToolInvocation(ctx);
     const absoluteFiles = ctx.targetFiles.map((f) => import_path.default.resolve(ctx.repositoryRoot, f));
@@ -779,7 +781,7 @@ var adapter = {
       ctx.logger.info("Skipping analysis: no local config and no patterns enabled", {
         toolId: TOOL_ID
       });
-      return buildResult(startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
+      return (0, import_tooling.buildResult)(TOOL_ID, startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
     }
     args.push(...absoluteFiles);
     try {
@@ -798,10 +800,10 @@ var adapter = {
           message: `Reek exited with code ${exitCode}${stderr ? `: ${stderr.trim()}` : ""}`,
           level: "error"
         });
-        return buildResult(startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
+        return (0, import_tooling.buildResult)(TOOL_ID, startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
       }
       if (!stdout.trim()) {
-        return buildResult(startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
+        return (0, import_tooling.buildResult)(TOOL_ID, startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
       }
       let smells;
       try {
@@ -814,7 +816,7 @@ var adapter = {
           message: `Failed to parse Reek JSON output: ${err instanceof Error ? err.message : String(err)}`,
           level: "error"
         });
-        return buildResult(startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
+        return (0, import_tooling.buildResult)(TOOL_ID, startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
       }
       if (!Array.isArray(smells)) {
         errors.push({
@@ -824,7 +826,7 @@ var adapter = {
           message: "Reek output is not a JSON array",
           level: "error"
         });
-        return buildResult(startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
+        return (0, import_tooling.buildResult)(TOOL_ID, startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
       }
       const enabledPatterns = new Set(codacyConfig.patterns.map((p) => p.patternId));
       const fileLineCache = /* @__PURE__ */ new Map();
@@ -840,7 +842,7 @@ var adapter = {
         const meta = ruleMetadata(patternId);
         const absPath = import_path.default.resolve(ctx.repositoryRoot, filePath);
         if (!fileLineCache.has(absPath)) {
-          fileLineCache.set(absPath, await readFileLines(absPath));
+          fileLineCache.set(absPath, await (0, import_tooling.readFileLines)(absPath));
         }
         const fileLines = fileLineCache.get(absPath);
         issues.push({
@@ -849,7 +851,7 @@ var adapter = {
           toolId: TOOL_ID,
           line,
           message: smell.message,
-          lineContent: getLineContent(fileLines, line),
+          lineContent: (0, import_tooling.getLineContent)(fileLines, line),
           category: meta.category,
           severity: meta.severity
         });
@@ -880,7 +882,7 @@ var adapter = {
       errorCount: errors.length,
       durationMs
     });
-    return buildResult(startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
+    return (0, import_tooling.buildResult)(TOOL_ID, startedAt, /* @__PURE__ */ new Date(), issues, errors, ctx.targetFiles.length);
   }
 };
 

package/dist/metadata.d.ts

@@ -0,0 +1,13 @@
+import { AdapterDescriptor } from '@codacy/tooling';
+
+/**
+ * Lightweight adapter descriptor for lazy loading.
+ *
+ * This module contains only plain-data metadata fields and has zero heavy
+ * imports (no patterns.json, no runtime utilities). The CLI imports this
+ * at startup for adapter registration without loading the full adapter module.
+ */
+
+declare const descriptor: AdapterDescriptor;
+
+export { descriptor };

package/dist/metadata.js

@@ -0,0 +1,39 @@
+"use strict";
+var __defProp = Object.defineProperty;
+var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
+var __getOwnPropNames = Object.getOwnPropertyNames;
+var __hasOwnProp = Object.prototype.hasOwnProperty;
+var __export = (target, all) => {
+  for (var name in all)
+    __defProp(target, name, { get: all[name], enumerable: true });
+};
+var __copyProps = (to, from, except, desc) => {
+  if (from && typeof from === "object" || typeof from === "function") {
+    for (let key of __getOwnPropNames(from))
+      if (!__hasOwnProp.call(to, key) && key !== except)
+        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
+  }
+  return to;
+};
+var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
+
+// src/metadata.ts
+var metadata_exports = {};
+__export(metadata_exports, {
+  descriptor: () => descriptor
+});
+module.exports = __toCommonJS(metadata_exports);
+var descriptor = {
+  id: "Reek",
+  displayName: "Reek",
+  preferredVersion: "6.5.0",
+  languages: ["Ruby"],
+  filePatterns: ["**/*.rb"],
+  supportedOS: ["linux", "darwin", "win32"],
+  supportedArch: ["x64", "arm64"],
+  supportedRunners: ["local", "container"]
+};
+// Annotate the CommonJS export names for ESM import in node:
+0 && (module.exports = {
+  descriptor
+});

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@codacy/tools-reek-6",
-  "version": "0.2.4",
+  "version": "0.2.6",
   "description": "Reek adapter — CLI-mode Ruby code smell detector",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",
@@ -8,7 +8,7 @@
     "dist"
   ],
   "dependencies": {
-    "@codacy/tooling": "0.4.0"
+    "@codacy/tooling": "0.6.0"
   },
   "license": "MIT",
   "author": "Codacy <support@codacy.com> (https://www.codacy.com)",
@@ -21,8 +21,18 @@
   "engines": {
     "node": ">=20.0.0"
   },
+  "exports": {
+    ".": {
+      "types": "./dist/index.d.ts",
+      "default": "./dist/index.js"
+    },
+    "./metadata": {
+      "types": "./dist/metadata.d.ts",
+      "default": "./dist/metadata.js"
+    }
+  },
   "scripts": {
-    "build": "tsup src/index.ts --format cjs --dts --clean",
+    "build": "tsup src/index.ts src/metadata.ts --format cjs --dts --clean",
     "test": "vitest run",
     "lint": "eslint src/",
     "clean": "rm -rf dist",