codiedev 0.7.0 → 0.7.2

package/dist/commands/reverseTicket.js

@@ -215,15 +215,41 @@ async function runBranchMode(opts) {
         console.error(`No commits on ${branch} ahead of ${baseRef} — make a commit first.`);
         process.exit(1);
     }
+    // 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 filesRaw = git(`diff --name-only ${baseSha}...HEAD`);
     const filesChanged = filesRaw ? filesRaw.split("\n").filter(Boolean) : [];
     if (filesChanged.length === 0) {
         console.error(`No files changed between ${baseRef} and ${branch} — nothing to write up.`);
         process.exit(1);
     }
+    let stat = "";
+    try {
+        stat = git(`diff --stat=200 ${baseSha}...HEAD`).slice(0, 8_000);
+    }
+    catch {
+        // best effort
+    }
     let diff = "";
     try {
-        diff = git(`diff ${baseSha}...HEAD`).slice(0, 12_000);
+        const sourceFiles = filesChanged.filter((f) => !isNoise(f));
+        if (sourceFiles.length > 0) {
+            const pathspecArgs = sourceFiles.map((f) => `'${f.replace(/'/g, "'\\''")}'`).join(" ");
+            diff = git(`diff ${baseSha}...HEAD -- ${pathspecArgs}`).slice(0, 40_000);
+        }
+        else {
+            diff = git(`diff ${baseSha}...HEAD`).slice(0, 40_000);
+        }
     }
     catch {
         // best effort
@@ -266,6 +292,7 @@ async function runBranchMode(opts) {
                     authorEmail,
                     filesChanged,
                     diff: diff || undefined,
+                    stat: stat || undefined,
                 },
                 forcedArtifactKey: opts.forcedKey,
             },
@@ -274,13 +301,17 @@ async function runBranchMode(opts) {
             console.error("Reverse-ticket generation returned no result. The writer may have skipped due to missing context.");
             process.exit(1);
         }
+        const portalHost = (process.env.CODIEDEV_PORTAL_URL || "https://codiedev.com").replace(/\/$/, "");
+        const portalLink = res.portalUrl.startsWith("http")
+            ? res.portalUrl
+            : `${portalHost}${res.portalUrl.startsWith("/") ? "" : "/"}${res.portalUrl}`;
         const verb = res.action === "overwritten" ? "Updated" : "Created";
         console.log(`✓ ${verb} reverse-ticket draft.`);
         console.log(`  artifact: ${res.artifactKey ?? res.artifactId}`);
-        console.log(`  portal:   ${res.portalUrl}`);
+        console.log(`  portal:   ${portalLink}`);
         console.log("");
         console.log("Open in your portal to review, edit, and push to Jira.");
-        console.log("When the PR opens later, the webhook dedupes against this draft — one ticket, not two.");
+        console.log("Each call creates a fresh draft — clean up duplicates from the portal if needed.");
     }
     catch (err) {
         console.error(`Reverse-ticket failed: ${err.message}`);

package/dist/mcp.js

@@ -1089,15 +1089,46 @@ async function handleReverseTicket(args, config) {
     if (baseSha === headSha) {
         throw new Error(`No commits on ${branch} ahead of ${baseRef} — make a commit first.`);
     }
+    // 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);
     const filesRaw = git(`diff --name-only ${baseSha}...HEAD`);
     const filesChanged = filesRaw ? filesRaw.split("\n").filter(Boolean) : [];
     if (filesChanged.length === 0) {
         throw new Error(`No files changed between ${baseRef} and ${branch} — nothing to write up.`);
     }
-    // Truncated diff (12k chars) — writer works without it but it sharpens output.
+    // Always capture --stat — compact (path + line counts), fits even for huge
+    // PRs, and serves as the writer's ground truth for "every file touched"
+    // when the diff body below is truncated.
+    let stat = "";
+    try {
+        stat = git(`diff --stat=200 ${baseSha}...HEAD`).slice(0, 8_000);
+    }
+    catch {
+        // Best effort.
+    }
+    // Diff body: 40k cap, noise-filtered via pathspec exclusions so lockfiles
+    // and generated dirs don't eat the budget.
     let diff = "";
     try {
-        diff = git(`diff ${baseSha}...HEAD`).slice(0, 12_000);
+        const sourceFiles = filesChanged.filter((f) => !isNoise(f));
+        if (sourceFiles.length > 0) {
+            const pathspecArgs = sourceFiles.map((f) => `'${f.replace(/'/g, "'\\''")}'`).join(" ");
+            diff = git(`diff ${baseSha}...HEAD -- ${pathspecArgs}`).slice(0, 40_000);
+        }
+        else {
+            diff = git(`diff ${baseSha}...HEAD`).slice(0, 40_000);
+        }
     }
     catch {
         // Best effort.
@@ -1132,6 +1163,7 @@ async function handleReverseTicket(args, config) {
                 authorEmail,
                 filesChanged,
                 diff: diff || undefined,
+                stat: stat || undefined,
             },
             forcedArtifactKey: forcedKey,
         },
@@ -1150,11 +1182,13 @@ async function handleReverseTicket(args, config) {
             ],
         };
     }
-    const portalBase = (config.backendUrl || "")
-        .replace(/\/$/, "")
-        .replace(/\.convex\.cloud$/, ".convex.cloud"); // backendUrl is the Convex site; portal is separate
-    // Best effort — surface the portal path; the user can prefix with their portal host.
-    const portalPath = res.portalUrl;
+    // Always return an absolute URL so MCP clients (Copilot Chat, Claude Code,
+    // etc.) can render the link correctly without guessing a host. Default to
+    // codiedev.com; allow override via CODIEDEV_PORTAL_URL for self-hosted.
+    const portalHost = (process.env.CODIEDEV_PORTAL_URL || "https://codiedev.com").replace(/\/$/, "");
+    const portalLink = res.portalUrl.startsWith("http")
+        ? res.portalUrl
+        : `${portalHost}${res.portalUrl.startsWith("/") ? "" : "/"}${res.portalUrl}`;
     const verb = res.action === "overwritten" ? "Updated" : "Created";
     return {
         content: [
@@ -1166,9 +1200,9 @@ async function handleReverseTicket(args, config) {
                     `  diff size: ${diff.length} chars\n` +
                     `  artifact: ${res.artifactKey ?? res.artifactId}\n\n` +
                     `Open in your portal to review, edit, and push to Jira:\n` +
-                    `  ${portalPath}\n\n` +
-                    `When the PR opens later, the webhook dedupes against this draft — ` +
-                    `you'll get one ticket, not two.`,
+                    `  ${portalLink}\n\n` +
+                    `Each call creates a fresh draft — if you've made several for the ` +
+                    `same branch, clean them up from the portal.`,
             },
         ],
     };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codiedev",
-  "version": "0.7.0",
+  "version": "0.7.2",
   "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",