contribute-now 0.4.0-dev.d24b735 → 0.4.0-dev.d48d9e6

package/dist/index.js

@@ -970,6 +970,68 @@ function withTimeout(promise, ms) {
 }
 var COPILOT_TIMEOUT_MS = 30000;
 var COPILOT_LONG_TIMEOUT_MS = 90000;
+var BATCH_CONFIG = {
+  LARGE_CHANGESET_THRESHOLD: 15,
+  COMPACT_PER_FILE_CHARS: 300,
+  MAX_COMPACT_PAYLOAD: 1e4,
+  FALLBACK_BATCH_SIZE: 15
+};
+function parseDiffByFile(rawDiff) {
+  const sections = new Map;
+  const headerPattern = /^diff --git a\/(.+?) b\//gm;
+  const positions = [];
+  for (let match = headerPattern.exec(rawDiff);match !== null; match = headerPattern.exec(rawDiff)) {
+    positions.push({ file: match[1], start: match.index });
+  }
+  for (let i = 0;i < positions.length; i++) {
+    const { file, start } = positions[i];
+    const end = i + 1 < positions.length ? positions[i + 1].start : rawDiff.length;
+    sections.set(file, rawDiff.slice(start, end));
+  }
+  return sections;
+}
+function extractDiffStats(diffSection) {
+  let added = 0;
+  let removed = 0;
+  for (const line of diffSection.split(`
+`)) {
+    if (line.startsWith("+") && !line.startsWith("+++"))
+      added++;
+    if (line.startsWith("-") && !line.startsWith("---"))
+      removed++;
+  }
+  return { added, removed };
+}
+function createCompactDiff(files, rawDiff, maxTotalChars = BATCH_CONFIG.MAX_COMPACT_PAYLOAD) {
+  if (files.length === 0)
+    return "";
+  const diffSections = parseDiffByFile(rawDiff);
+  const perFileBudget = Math.min(BATCH_CONFIG.COMPACT_PER_FILE_CHARS, Math.floor(maxTotalChars / files.length));
+  const parts = [];
+  for (const file of files) {
+    const section = diffSections.get(file);
+    if (section) {
+      const stats = extractDiffStats(section);
+      const header = `[${file}] (+${stats.added}/-${stats.removed})`;
+      if (section.length <= perFileBudget) {
+        parts.push(`${header}
+${section}`);
+      } else {
+        const truncated = section.slice(0, perFileBudget - header.length - 20);
+        parts.push(`${header}
+${truncated}
+...(truncated)`);
+      }
+    } else {
+      parts.push(`[${file}] (new/binary file — no diff available)`);
+    }
+  }
+  const result = parts.join(`
+
+`);
+  return result.length > maxTotalChars ? `${result.slice(0, maxTotalChars - 15)}
+...(truncated)` : result;
+}
 async function checkCopilotAvailable() {
   try {
     const client = await getManagedClient();
@@ -1070,16 +1132,18 @@ function extractJson(raw) {
 }
 async function generateCommitMessage(diff, stagedFiles, model, convention = "clean-commit") {
   try {
+    const isLarge = stagedFiles.length >= BATCH_CONFIG.LARGE_CHANGESET_THRESHOLD;
     const multiFileHint = stagedFiles.length > 1 ? `
 
 IMPORTANT: Multiple files are staged. Generate ONE commit message that captures the high-level purpose of ALL changes together. Focus on the overall intent, not individual file changes. Be specific but concise — do not list every file.` : "";
+    const diffContent = isLarge ? createCompactDiff(stagedFiles, diff) : diff.slice(0, 4000);
     const userMessage = `Generate a commit message for these staged changes:
 
-Files: ${stagedFiles.join(", ")}
+Files (${stagedFiles.length}): ${stagedFiles.join(", ")}
 
 Diff:
-${diff.slice(0, 4000)}${multiFileHint}`;
-    const result = await callCopilot(getCommitSystemPrompt(convention), userMessage, model);
+${diffContent}${multiFileHint}`;
+    const result = await callCopilot(getCommitSystemPrompt(convention), userMessage, model, isLarge ? COPILOT_LONG_TIMEOUT_MS : COPILOT_TIMEOUT_MS);
     return result?.trim() ?? null;
   } catch {
     return null;
@@ -1128,16 +1192,23 @@ ${conflictDiff.slice(0, 4000)}`;
   }
 }
 async function generateCommitGroups(files, diffs, model, convention = "clean-commit") {
+  const isLarge = files.length >= BATCH_CONFIG.LARGE_CHANGESET_THRESHOLD;
+  const diffContent = isLarge ? createCompactDiff(files, diffs) : diffs.slice(0, 6000);
+  const largeHint = isLarge ? `
+
+NOTE: This is a large changeset (${files.length} files). Compact diffs are provided for every file. Focus on creating well-organized logical groups.` : "";
   const userMessage = `Group these changed files into logical atomic commits:
 
 Files:
 ${files.join(`
 `)}
 
-Diffs (truncated):
-${diffs.slice(0, 6000)}`;
+Diffs:
+${diffContent}${largeHint}`;
   const result = await callCopilot(getGroupingSystemPrompt(convention), userMessage, model, COPILOT_LONG_TIMEOUT_MS);
   if (!result) {
+    if (isLarge)
+      return generateCommitGroupsInBatches(files, diffs, model, convention);
     throw new Error("AI returned an empty response");
   }
   const cleaned = extractJson(result);
@@ -1145,10 +1216,14 @@ ${diffs.slice(0, 6000)}`;
   try {
     parsed = JSON.parse(cleaned);
   } catch {
+    if (isLarge)
+      return generateCommitGroupsInBatches(files, diffs, model, convention);
     throw new Error(`AI response is not valid JSON. Raw start: "${result.slice(0, 120)}..."`);
   }
   const groups = parsed;
   if (!Array.isArray(groups) || groups.length === 0) {
+    if (isLarge)
+      return generateCommitGroupsInBatches(files, diffs, model, convention);
     throw new Error("AI response was not a valid JSON array of commit groups");
   }
   for (const group of groups) {
@@ -1158,7 +1233,51 @@ ${diffs.slice(0, 6000)}`;
   }
   return groups;
 }
+async function generateCommitGroupsInBatches(files, diffs, model, convention = "clean-commit") {
+  const batchSize = BATCH_CONFIG.FALLBACK_BATCH_SIZE;
+  const allGroups = [];
+  const diffSections = parseDiffByFile(diffs);
+  for (let i = 0;i < files.length; i += batchSize) {
+    const batchFiles = files.slice(i, i + batchSize);
+    const batchDiff = batchFiles.map((f) => diffSections.get(f) ?? "").filter(Boolean).join(`
+`);
+    const batchDiffContent = batchFiles.length >= BATCH_CONFIG.LARGE_CHANGESET_THRESHOLD ? createCompactDiff(batchFiles, batchDiff) : batchDiff.slice(0, 6000);
+    const batchNum = Math.floor(i / batchSize) + 1;
+    const totalBatches = Math.ceil(files.length / batchSize);
+    const userMessage = `Group these changed files into logical atomic commits:
+
+Files:
+${batchFiles.join(`
+`)}
+
+Diffs:
+${batchDiffContent}
+
+NOTE: Processing batch ${batchNum}/${totalBatches} of a large changeset. Group only the files listed above.`;
+    try {
+      const result = await callCopilot(getGroupingSystemPrompt(convention), userMessage, model, COPILOT_LONG_TIMEOUT_MS);
+      if (!result)
+        continue;
+      const cleaned = extractJson(result);
+      const parsed = JSON.parse(cleaned);
+      if (Array.isArray(parsed)) {
+        for (const group of parsed) {
+          if (Array.isArray(group.files) && typeof group.message === "string") {
+            allGroups.push(group);
+          }
+        }
+      }
+    } catch {}
+  }
+  if (allGroups.length === 0) {
+    throw new Error("AI could not group any files even with batch processing");
+  }
+  return allGroups;
+}
 async function regenerateAllGroupMessages(groups, diffs, model, convention = "clean-commit") {
+  const totalFiles = groups.reduce((sum, g) => sum + g.files.length, 0);
+  const isLarge = totalFiles >= BATCH_CONFIG.LARGE_CHANGESET_THRESHOLD;
+  const diffContent = isLarge ? createCompactDiff(groups.flatMap((g) => g.files), diffs) : diffs.slice(0, 6000);
   const groupSummary = groups.map((g, i) => `Group ${i + 1}: [${g.files.join(", ")}]`).join(`
 `);
   const userMessage = `Regenerate ONLY the commit messages for these pre-defined file groups. Do NOT change the file groupings.
@@ -1166,8 +1285,8 @@ async function regenerateAllGroupMessages(groups, diffs, model, convention = "cl
 Groups:
 ${groupSummary}
 
-Diffs (truncated):
-${diffs.slice(0, 6000)}`;
+Diffs:
+${diffContent}`;
   const result = await callCopilot(getGroupingSystemPrompt(convention), userMessage, model, COPILOT_LONG_TIMEOUT_MS);
   if (!result)
     return groups;
@@ -1186,12 +1305,14 @@ ${diffs.slice(0, 6000)}`;
 }
 async function regenerateGroupMessage(files, diffs, model, convention = "clean-commit") {
   try {
+    const isLarge = files.length >= BATCH_CONFIG.LARGE_CHANGESET_THRESHOLD;
+    const diffContent = isLarge ? createCompactDiff(files, diffs) : diffs.slice(0, 4000);
     const userMessage = `Generate a single commit message for these files:
 
 Files: ${files.join(", ")}
 
 Diff:
-${diffs.slice(0, 4000)}`;
+${diffContent}`;
     const result = await callCopilot(getCommitSystemPrompt(convention), userMessage, model);
     return result?.trim() ?? null;
   } catch {
@@ -1749,7 +1870,8 @@ ${pc6.bold("Changed files:")}`);
         warn(`AI unavailable: ${copilotError}`);
         warn("Falling back to manual commit message entry.");
       } else {
-        const spinner = createSpinner("Generating commit message with AI...");
+        const spinnerMsg = stagedFiles.length >= 15 ? `Generating commit message with AI (${stagedFiles.length} files — using optimized batching)...` : "Generating commit message with AI...";
+        const spinner = createSpinner(spinnerMsg);
         commitMessage = await generateCommitMessage(diff, stagedFiles, args.model, config.commitConvention);
         if (commitMessage) {
           spinner.success("AI commit message generated.");
@@ -1840,7 +1962,7 @@ ${pc6.bold("Changed files:")}`);
   for (const f of changedFiles) {
     console.log(`  ${pc6.dim("•")} ${f}`);
   }
-  const spinner = createSpinner(`Asking AI to group ${changedFiles.length} file(s) into logical commits...`);
+  const spinner = createSpinner(changedFiles.length >= 15 ? `Asking AI to group ${changedFiles.length} file(s) into logical commits (using optimized batching)...` : `Asking AI to group ${changedFiles.length} file(s) into logical commits...`);
   const diffs = await getFullDiffForFiles(changedFiles);
   if (!diffs.trim()) {
     spinner.stop();
@@ -2018,7 +2140,7 @@ import pc7 from "picocolors";
 // package.json
 var package_default = {
   name: "contribute-now",
-  version: "0.4.0-dev.d24b735",
+  version: "0.4.0-dev.d48d9e6",
   description: "Git workflow CLI for squash-merge two-branch models. Keeps dev in sync with main after squash merges.",
   type: "module",
   bin: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "contribute-now",
-  "version": "0.4.0-dev.d24b735",
+  "version": "0.4.0-dev.d48d9e6",
   "description": "Git workflow CLI for squash-merge two-branch models. Keeps dev in sync with main after squash merges.",
   "type": "module",
   "bin": {