semlint-cli 0.1.3 → 0.1.5

package/dist/diagnostics.js

@@ -12,8 +12,13 @@ const VALID_SEVERITIES = new Set(["error", "warn", "info"]);
 function isPositiveInteger(value) {
     return typeof value === "number" && Number.isInteger(value) && value >= 1;
 }
-function normalizeDiagnostics(ruleId, diagnostics, debug) {
+/**
+ * @param resolveRoot - Directory to resolve diagnostic file paths against (e.g. git repo root).
+ *   Git diff paths are repo-relative; resolving from repo root ensures paths exist when running from subdirs.
+ */
+function normalizeDiagnostics(ruleId, diagnostics, debug, resolveRoot) {
     const out = [];
+    const baseDir = resolveRoot && resolveRoot.length > 0 ? resolveRoot : process.cwd();
     for (const raw of diagnostics) {
         if (typeof raw !== "object" || raw === null || Array.isArray(raw)) {
             if (debug) {
@@ -41,7 +46,7 @@ function normalizeDiagnostics(ruleId, diagnostics, debug) {
             }
             continue;
         }
-        const absolute = node_path_1.default.resolve(file);
+        const absolute = node_path_1.default.resolve(baseDir, file);
         if (!node_fs_1.default.existsSync(absolute)) {
             if (debug) {
                 process.stderr.write(`[debug] Dropped diagnostic for ${ruleId}: file does not exist (${file})\n`);

package/dist/git.js

@@ -1,6 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.getGitDiff = getGitDiff;
+exports.getRepoRoot = getRepoRoot;
 exports.getLocalBranchDiff = getLocalBranchDiff;
 const node_child_process_1 = require("node:child_process");
 const node_os_1 = require("node:os");
@@ -36,6 +37,20 @@ async function getGitDiff(base, head) {
     const result = await runGitCommand(["diff", base, head]);
     return result.stdout;
 }
+/**
+ * Returns the absolute path of the git repository root, or null if not in a git repo.
+ * Used to resolve diagnostic file paths, which are always repo-relative in git diff output.
+ */
+async function getRepoRoot() {
+    try {
+        const result = await runGitCommand(["rev-parse", "--show-toplevel"]);
+        const root = result.stdout.trim();
+        return root !== "" ? root : null;
+    }
+    catch {
+        return null;
+    }
+}
 async function getUntrackedFiles() {
     const result = await runGitCommand(["ls-files", "--others", "--exclude-standard"]);
     return result.stdout

package/dist/main.js

@@ -70,6 +70,8 @@ function buildBatchPrompt(rules, diff) {
         "- Each diagnostic must reference a changed file from the DIFF.",
         "- rule_id must match one of the RULE_ID values listed below.",
         "- Keep messages concise and actionable.",
+        "- Deduplicate semantically equivalent findings before returning output.",
+        "- When duplicates exist, keep a single diagnostic from the rule that semantically matches the issue best, even if that selected diagnostic has lower severity.",
         "",
         "RULES:",
         ruleBlocks,
@@ -94,6 +96,7 @@ async function runSemlint(options) {
             ? "Using local branch diff (staged + unstaged + untracked only)"
             : `Using explicit ref diff (${config.base}..${config.head})`);
         debugLog(config.debug, `Detected ${changedFiles.length} changed file(s)`);
+        const repoRoot = await timedAsync(config.debug, "Resolved git repo root", () => (0, git_1.getRepoRoot)());
         const backend = timed(config.debug, "Initialized backend runner", () => (0, backend_1.createBackendRunner)(config));
         const runnableRules = rules.filter((rule) => {
             const filterStartedAt = Date.now();
@@ -154,7 +157,7 @@ async function runSemlint(options) {
                     }
                 }
                 for (const rule of runnableRules) {
-                    const normalized = timed(config.debug, `Batch: diagnostics normalization for ${rule.id}`, () => (0, diagnostics_1.normalizeDiagnostics)(rule.id, groupedByRule.get(rule.id) ?? [], config.debug));
+                    const normalized = timed(config.debug, `Batch: diagnostics normalization for ${rule.id}`, () => (0, diagnostics_1.normalizeDiagnostics)(rule.id, groupedByRule.get(rule.id) ?? [], config.debug, repoRoot));
                     diagnostics.push(...normalized);
                 }
             }
@@ -179,7 +182,7 @@ async function runSemlint(options) {
                         prompt,
                         timeoutMs: config.timeoutMs
                     }));
-                    normalized = timed(config.debug, `Rule ${rule.id}: diagnostics normalization`, () => (0, diagnostics_1.normalizeDiagnostics)(rule.id, result.diagnostics, config.debug));
+                    normalized = timed(config.debug, `Rule ${rule.id}: diagnostics normalization`, () => (0, diagnostics_1.normalizeDiagnostics)(rule.id, result.diagnostics, config.debug, repoRoot));
                 }
                 catch (error) {
                     backendError = true;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "semlint-cli",
-  "version": "0.1.3",
+  "version": "0.1.5",
   "description": "Semantic lint CLI — runs LLM-backed rules on your git diff and returns CI-friendly exit codes",
   "type": "commonjs",
   "main": "dist/main.js",