codiedev 0.7.1 → 0.7.3

package/dist/commands/reverseTicket.js

@@ -211,19 +211,73 @@ async function runBranchMode(opts) {
     }
     const baseSha = git(`merge-base HEAD ${baseRef}`);
     const headSha = git("rev-parse HEAD");
-    if (baseSha === headSha) {
-        console.error(`No commits on ${branch} ahead of ${baseRef} — make a commit first.`);
-        process.exit(1);
-    }
-    const filesRaw = git(`diff --name-only ${baseSha}...HEAD`);
-    const filesChanged = filesRaw ? filesRaw.split("\n").filter(Boolean) : [];
+    // Filter generated/lockfile noise from the diff body. The --stat (collected
+    // separately) still includes every file untouched, so excluded files still
+    // surface in the ticket's "Other files touched" section.
+    const isNoise = (p) => /(^|\/)(node_modules|dist|build|\.next|\.turbo|coverage|out)\//.test(p) ||
+        /(^|\/)package-lock\.json$/.test(p) ||
+        /(^|\/)yarn\.lock$/.test(p) ||
+        /(^|\/)pnpm-lock\.yaml$/.test(p) ||
+        /(^|\/)bun\.lock(b)?$/.test(p) ||
+        /(^|\/)Cargo\.lock$/.test(p) ||
+        /(^|\/)Gemfile\.lock$/.test(p) ||
+        /(^|\/)poetry\.lock$/.test(p) ||
+        /(^|\/)go\.sum$/.test(p);
+    const shellTolerant = (cmd) => {
+        try {
+            return (0, child_process_1.execSync)(`${cmd} || true`, {
+                stdio: ["pipe", "pipe", "pipe"],
+                maxBuffer: 32 * 1024 * 1024,
+            })
+                .toString("utf8")
+                .trim();
+        }
+        catch {
+            return "";
+        }
+    };
+    const shQuote = (s) => `'${s.replace(/'/g, "'\\''")}'`;
+    // Capture all work relative to the base ref: committed + uncommitted-tracked
+    // + new untracked. Matches "ticket my current changes" intent.
+    const trackedRaw = git(`diff --name-only ${baseSha}`);
+    const trackedFiles = trackedRaw ? trackedRaw.split("\n").filter(Boolean) : [];
+    const untrackedRaw = safeGit(() => git("ls-files --others --exclude-standard"));
+    const untrackedFiles = untrackedRaw
+        ? untrackedRaw.split("\n").filter(Boolean)
+        : [];
+    const filesChanged = Array.from(new Set([...trackedFiles, ...untrackedFiles]));
     if (filesChanged.length === 0) {
-        console.error(`No files changed between ${baseRef} and ${branch} — nothing to write up.`);
+        console.error(`No changes detected between ${baseRef} and your working tree on ${branch}.`);
         process.exit(1);
     }
+    let stat = "";
+    try {
+        const trackedStat = git(`diff --stat=200 ${baseSha}`);
+        const untrackedStatLines = untrackedFiles.map((f) => {
+            const wc = shellTolerant(`wc -l < ${shQuote(f)}`);
+            const n = parseInt(wc.split(/\s+/)[0] || "0", 10) || 0;
+            return ` ${f} | ${n} ++++++++++ (new file)`;
+        });
+        stat = [trackedStat, ...untrackedStatLines]
+            .filter(Boolean)
+            .join("\n")
+            .slice(0, 8_000);
+    }
+    catch {
+        // best effort
+    }
     let diff = "";
     try {
-        diff = git(`diff ${baseSha}...HEAD`).slice(0, 12_000);
+        const sourceTracked = trackedFiles.filter((f) => !isNoise(f));
+        const trackedDiff = sourceTracked.length > 0
+            ? git(`diff ${baseSha} -- ${sourceTracked.map(shQuote).join(" ")}`)
+            : git(`diff ${baseSha}`);
+        const sourceUntracked = untrackedFiles.filter((f) => !isNoise(f));
+        const untrackedDiffs = sourceUntracked.map((f) => shellTolerant(`git diff --no-index --no-color -- /dev/null ${shQuote(f)}`));
+        diff = [trackedDiff, ...untrackedDiffs]
+            .filter(Boolean)
+            .join("\n")
+            .slice(0, 40_000);
     }
     catch {
         // best effort
@@ -266,6 +320,7 @@ async function runBranchMode(opts) {
                     authorEmail,
                     filesChanged,
                     diff: diff || undefined,
+                    stat: stat || undefined,
                 },
                 forcedArtifactKey: opts.forcedKey,
             },

package/dist/mcp.js

@@ -52,7 +52,17 @@ const path = __importStar(require("path"));
 const utils_1 = require("./utils");
 const shared_1 = require("./commands/shared");
 const PKG_NAME = "codiedev";
-const PKG_VERSION = "0.5.0";
+// Read version from the bundled package.json so the startup log + tool reports
+// always reflect the actually-installed version, rather than a stale constant.
+const PKG_VERSION = (() => {
+    try {
+        const pkgPath = path.resolve(__dirname, "..", "package.json");
+        return JSON.parse(fs.readFileSync(pkgPath, "utf8")).version ?? "unknown";
+    }
+    catch {
+        return "unknown";
+    }
+})();
 // ─────────────────────────────────────────────────────────────────────────────
 // Tool definitions — descriptions tuned so Claude/Codex resolve natural-language
 // requests into the right tool without manual steering.
@@ -1086,18 +1096,85 @@ async function handleReverseTicket(args, config) {
     }
     const baseSha = git(`merge-base HEAD ${baseRef}`);
     const headSha = git("rev-parse HEAD");
-    if (baseSha === headSha) {
-        throw new Error(`No commits on ${branch} ahead of ${baseRef} — make a commit first.`);
-    }
-    const filesRaw = git(`diff --name-only ${baseSha}...HEAD`);
-    const filesChanged = filesRaw ? filesRaw.split("\n").filter(Boolean) : [];
+    // Filter generated/lockfile noise so the LLM's diff budget gets spent on
+    // real source. The --stat block (collected separately below) still includes
+    // every file untouched, so excluded files still appear in the ticket's
+    // "Other files touched" section if relevant.
+    const isNoise = (p) => /(^|\/)(node_modules|dist|build|\.next|\.turbo|coverage|out)\//.test(p) ||
+        /(^|\/)package-lock\.json$/.test(p) ||
+        /(^|\/)yarn\.lock$/.test(p) ||
+        /(^|\/)pnpm-lock\.yaml$/.test(p) ||
+        /(^|\/)bun\.lock(b)?$/.test(p) ||
+        /(^|\/)Cargo\.lock$/.test(p) ||
+        /(^|\/)Gemfile\.lock$/.test(p) ||
+        /(^|\/)poetry\.lock$/.test(p) ||
+        /(^|\/)go\.sum$/.test(p);
+    // Helper: run a shell command tolerating non-zero exit codes (git diff
+    // --no-index exits 1 when files differ, which is the normal case).
+    // Reuses the execSync already imported above for the git() helper.
+    const shellTolerant = (cmd) => {
+        try {
+            return execSync(`${cmd} || true`, {
+                stdio: ["pipe", "pipe", "pipe"],
+                maxBuffer: 32 * 1024 * 1024,
+            })
+                .toString("utf8")
+                .trim();
+        }
+        catch {
+            return "";
+        }
+    };
+    const shQuote = (s) => `'${s.replace(/'/g, "'\\''")}'`;
+    // Capture all work the developer has done relative to the base ref:
+    //   - committed changes between base and HEAD
+    //   - uncommitted modifications to tracked files
+    //   - new untracked files (path listed + content via synthetic diff)
+    // Matches the user's intent ("ticket my current changes") rather than the
+    // narrower "only what's been committed" interpretation.
+    const trackedRaw = git(`diff --name-only ${baseSha}`);
+    const trackedFiles = trackedRaw ? trackedRaw.split("\n").filter(Boolean) : [];
+    const untrackedRaw = safeGit(() => git("ls-files --others --exclude-standard"));
+    const untrackedFiles = untrackedRaw
+        ? untrackedRaw.split("\n").filter(Boolean)
+        : [];
+    const filesChanged = Array.from(new Set([...trackedFiles, ...untrackedFiles]));
     if (filesChanged.length === 0) {
-        throw new Error(`No files changed between ${baseRef} and ${branch} — nothing to write up.`);
+        throw new Error(`No changes detected between ${baseRef} and your working tree on ${branch}.`);
+    }
+    // --stat block: compact ground truth for "every file touched". Tracked
+    // changes from `git diff --stat baseSha`; untracked files appended as
+    // synthesized stat rows so the LLM sees them with line counts.
+    let stat = "";
+    try {
+        const trackedStat = git(`diff --stat=200 ${baseSha}`);
+        const untrackedStatLines = untrackedFiles.map((f) => {
+            const wc = shellTolerant(`wc -l < ${shQuote(f)}`);
+            const n = parseInt(wc.split(/\s+/)[0] || "0", 10) || 0;
+            return ` ${f} | ${n} ++++++++++ (new file)`;
+        });
+        stat = [trackedStat, ...untrackedStatLines]
+            .filter(Boolean)
+            .join("\n")
+            .slice(0, 8_000);
+    }
+    catch {
+        // Best effort.
     }
-    // Truncated diff (12k chars) — writer works without it but it sharpens output.
+    // Diff body: 40k cap, noise-filtered. Tracked changes via `git diff baseSha`,
+    // plus synthetic add-only diffs for untracked files via `git diff --no-index`.
     let diff = "";
     try {
-        diff = git(`diff ${baseSha}...HEAD`).slice(0, 12_000);
+        const sourceTracked = trackedFiles.filter((f) => !isNoise(f));
+        const trackedDiff = sourceTracked.length > 0
+            ? git(`diff ${baseSha} -- ${sourceTracked.map(shQuote).join(" ")}`)
+            : git(`diff ${baseSha}`);
+        const sourceUntracked = untrackedFiles.filter((f) => !isNoise(f));
+        const untrackedDiffs = sourceUntracked.map((f) => shellTolerant(`git diff --no-index --no-color -- /dev/null ${shQuote(f)}`));
+        diff = [trackedDiff, ...untrackedDiffs]
+            .filter(Boolean)
+            .join("\n")
+            .slice(0, 40_000);
     }
     catch {
         // Best effort.
@@ -1132,6 +1209,7 @@ async function handleReverseTicket(args, config) {
                 authorEmail,
                 filesChanged,
                 diff: diff || undefined,
+                stat: stat || undefined,
             },
             forcedArtifactKey: forcedKey,
         },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codiedev",
-  "version": "0.7.1",
+  "version": "0.7.3",
   "description": "Connect Claude Code, Codex, Cursor, or VS Code Copilot to CodieDev for org-wide session capture and artifact collaboration",
   "bin": {
     "codiedev": "./dist/cli.js",