semlint-cli 0.1.2 → 0.1.4

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,28 +37,19 @@ async function getGitDiff(base, head) {
     const result = await runGitCommand(["diff", base, head]);
     return result.stdout;
 }
-function isMissingRefError(message) {
-    return /(not a valid object name|unknown revision|bad revision|no upstream configured|no upstream branch)/i.test(message);
-}
-async function resolveLocalComparisonBase() {
-    const candidates = ["@{upstream}", "origin/main", "main"];
-    for (const candidate of candidates) {
-        try {
-            const result = await runGitCommand(["merge-base", "HEAD", candidate]);
-            const mergeBase = result.stdout.trim();
-            if (mergeBase !== "") {
-                return mergeBase;
-            }
-        }
-        catch (error) {
-            const message = error instanceof Error ? error.message : String(error);
-            if (!isMissingRefError(message)) {
-                throw error;
-            }
-            continue;
-        }
+/**
+ * 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;
     }
-    return "HEAD";
 }
 async function getUntrackedFiles() {
     const result = await runGitCommand(["ls-files", "--others", "--exclude-standard"]);
@@ -70,9 +62,16 @@ async function getNoIndexDiffForFile(filePath) {
     const result = await runGitCommand(["diff", "--no-index", "--", node_os_1.devNull, filePath], [0, 1]);
     return result.stdout;
 }
+/**
+ * Returns the diff for the current branch limited to:
+ * - Staged changes (--cached: index vs HEAD)
+ * - Unstaged changes (working tree vs index)
+ * - Untracked files (as full-file diffs)
+ * Does not include already-committed changes on the branch.
+ */
 async function getLocalBranchDiff() {
-    const base = await resolveLocalComparisonBase();
-    const trackedDiff = await runGitCommand(["diff", base]);
+    const stagedResult = await runGitCommand(["diff", "--cached"]);
+    const unstagedResult = await runGitCommand(["diff"]);
     const untrackedFiles = await getUntrackedFiles();
     const untrackedDiffChunks = [];
     for (const filePath of untrackedFiles) {
@@ -81,5 +80,7 @@ async function getLocalBranchDiff() {
             untrackedDiffChunks.push(fileDiff);
         }
     }
-    return [trackedDiff.stdout, ...untrackedDiffChunks].filter((chunk) => chunk !== "").join("\n");
+    return [stagedResult.stdout, unstagedResult.stdout, ...untrackedDiffChunks]
+        .filter((chunk) => chunk !== "")
+        .join("\n");
 }

package/dist/main.js

@@ -91,9 +91,10 @@ async function runSemlint(options) {
         const diff = await timedAsync(config.debug, "Computed git diff", () => useLocalBranchDiff ? (0, git_1.getLocalBranchDiff)() : (0, git_1.getGitDiff)(config.base, config.head));
         const changedFiles = timed(config.debug, "Parsed changed files from diff", () => (0, filter_1.extractChangedFilesFromDiff)(diff));
         debugLog(config.debug, useLocalBranchDiff
-            ? "Using local branch diff (tracked + staged + unstaged + untracked)"
+            ? "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 +155,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 +180,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.2",
+  "version": "0.1.4",
   "description": "Semantic lint CLI — runs LLM-backed rules on your git diff and returns CI-friendly exit codes",
   "type": "commonjs",
   "main": "dist/main.js",