contribute-now 0.6.2-dev.313af88 → 0.6.2-dev.3d69403

package/dist/index.js

@@ -10103,12 +10103,12 @@ async function multiSelectPrompt(message, choices) {
 init_dist();
 var CONVENTIONAL_COMMIT_SYSTEM_PROMPT = `Git commit message generator. Format: <type>[!][(<scope>)]: <description>
 Types: feat, fix, docs, style, refactor, perf, test, build, ci, chore, revert
-Rules: breaking (!) only for feat/fix/refactor/perf; imperative mood; max 72 chars; lowercase start; scope optional camelCase/kebab-case. Return ONLY the message line.
+Rules: breaking (!) only for feat/fix/refactor/perf; imperative mood; max 72 chars; lowercase start; scope optional camelCase/kebab-case. Do NOT use backticks, quotes, or markdown formatting around filenames, functions, or identifiers. Return ONLY the message line.
 Examples: feat: add user auth | fix(auth): resolve token expiry | feat!: redesign auth API`;
 var CLEAN_COMMIT_SYSTEM_PROMPT = `Git commit message generator. EXACT format: <emoji> <type>[!][ (<scope>)]: <description>
 Spacing: EMOJI SPACE TYPE [SPACE OPENPAREN SCOPE CLOSEPAREN] COLON SPACE DESCRIPTION
 Types: \uD83D\uDCE6 new, \uD83D\uDD27 update, \uD83D\uDDD1️ remove, \uD83D\uDD12 security, ⚙️ setup, ☕ chore, \uD83E\uDDEA test, \uD83D\uDCD6 docs, \uD83D\uDE80 release
-Rules: breaking (!) only for new/update/remove/security; imperative mood; max 72 chars; lowercase start; scope optional. Return ONLY the message line.
+Rules: breaking (!) only for new/update/remove/security; imperative mood; max 72 chars; lowercase start; scope optional. Do NOT use backticks, quotes, or markdown formatting around filenames, functions, or identifiers. Return ONLY the message line.
 Correct: \uD83D\uDCE6 new: add user auth | \uD83D\uDD27 update (api): improve error handling | ⚙️ setup (ci): configure github actions
 WRONG: ⚙️setup(ci): ... | \uD83D\uDD27 update(api): ... ← always space before scope parenthesis`;
 function getGroupingSystemPrompt(convention) {
@@ -10133,6 +10133,7 @@ Rules:
 - Each group should represent ONE logical change
 - Every file must appear in exactly one group
 - Commit messages must follow the convention, be concise, imperative, max 72 chars
+- Do not use backticks, quotes, or markdown formatting in commit messages
 - Order groups so foundational changes come first (types, utils) and consumers come after
 - Return ONLY the JSON array, nothing else`;
 }
@@ -10351,6 +10352,9 @@ function extractJson(raw) {
   }
   return text;
 }
+function sanitizeGeneratedCommitMessage(message) {
+  return message.replace(/`+/g, "").replace(/\s+/g, " ").trim();
+}
 async function generateCommitMessage(diff, stagedFiles, model, convention = "clean-commit", context) {
   try {
     const isLarge = stagedFiles.length >= BATCH_CONFIG.LARGE_CHANGESET_THRESHOLD;
@@ -10368,7 +10372,7 @@ Files (${stagedFiles.length}): ${stagedFiles.join(", ")}
 Diff:
 ${diffContent}${multiFileHint}${squashHint}`;
     const result = await callCopilot(getCommitSystemPrompt(convention), userMessage, model, isLarge ? COPILOT_LONG_TIMEOUT_MS : COPILOT_TIMEOUT_MS);
-    return result?.trim() ?? null;
+    return result ? sanitizeGeneratedCommitMessage(result) : null;
   } catch {
     return null;
   }
@@ -10489,7 +10493,10 @@ ${diffContent}${largeHint}`;
       throw new Error("AI returned groups with invalid structure (missing files or message)");
     }
   }
-  return groups;
+  return groups.map((group) => ({
+    ...group,
+    message: sanitizeGeneratedCommitMessage(group.message)
+  }));
 }
 async function generateCommitGroupsInBatches(files, diffs, model, convention = "clean-commit") {
   const batchSize = BATCH_CONFIG.FALLBACK_BATCH_SIZE;
@@ -10524,7 +10531,11 @@ NOTE: Processing batch ${batchNum}/${totalBatches} of a large changeset. Group o
             const batchFileSet = new Set(batchFiles);
             const filteredFiles = group.files.filter((f3) => batchFileSet.has(f3));
             if (filteredFiles.length > 0) {
-              allGroups.push({ ...group, files: filteredFiles });
+              allGroups.push({
+                ...group,
+                files: filteredFiles,
+                message: sanitizeGeneratedCommitMessage(group.message)
+              });
             }
           }
         }
@@ -10567,7 +10578,7 @@ ${diffContent}`;
       return groups;
     return groups.map((g3, i2) => ({
       files: g3.files,
-      message: typeof parsed[i2]?.message === "string" ? parsed[i2].message : g3.message
+      message: typeof parsed[i2]?.message === "string" ? sanitizeGeneratedCommitMessage(parsed[i2].message) : g3.message
     }));
   } catch {
     return groups;
@@ -10584,7 +10595,7 @@ Files: ${files.join(", ")}
 Diff:
 ${diffContent}`;
     const result = await callCopilot(getCommitSystemPrompt(convention), userMessage, model);
-    return result?.trim() ?? null;
+    return result ? sanitizeGeneratedCommitMessage(result) : null;
   } catch {
     return null;
   }
@@ -11020,17 +11031,24 @@ var CONVENTION_FORMAT_HINTS = {
   conventional: [
     "Format: <type>[!][(<scope>)]: <description>",
     "Types: feat, fix, docs, style, refactor, perf, test, build, ci, chore, revert",
-    "Examples: feat: add login page | fix(auth): resolve token expiry | docs: update README"
+    "Examples: feat: add login page | fix(auth): resolve token expiry | docs: update README",
+    "Do not use backticks or markdown formatting in the message."
   ],
   "clean-commit": [
     "Format: <emoji> <type>[!][(<scope>)]: <description>",
     "Types: \uD83D\uDCE6 new | \uD83D\uDD27 update | \uD83D\uDDD1️ remove | \uD83D\uDD12 security | ⚙️ setup | ☕ chore | \uD83E\uDDEA test | \uD83D\uDCD6 docs | \uD83D\uDE80 release",
-    "Examples: \uD83D\uDCE6 new: user auth | \uD83D\uDD27 update (api): improve errors | ⚙️ setup (ci): add workflow"
+    "Examples: \uD83D\uDCE6 new: user auth | \uD83D\uDD27 update (api): improve errors | ⚙️ setup (ci): add workflow",
+    "Do not use backticks or markdown formatting in the message."
   ]
 };
+function hasUnsupportedCommitMessageChars(message) {
+  return message.includes("`");
+}
 function validateCommitMessage(message, convention) {
   if (convention === "none")
     return true;
+  if (hasUnsupportedCommitMessageChars(message))
+    return false;
   if (convention === "clean-commit")
     return CLEAN_COMMIT_PATTERN.test(message);
   if (convention === "conventional")
@@ -11042,6 +11060,7 @@ function getValidationError(convention) {
     return [];
   return [
     `Commit message does not follow ${CONVENTION_LABELS[convention]} format.`,
+    "Do not use backticks or markdown formatting in commit messages.",
     ...CONVENTION_FORMAT_HINTS[convention]
   ];
 }
@@ -11469,7 +11488,7 @@ var import_picocolors9 = __toESM(require_picocolors(), 1);
 // package.json
 var package_default = {
   name: "contribute-now",
-  version: "0.6.2-dev.313af88",
+  version: "0.6.2-dev.3d69403",
   description: "Developer CLI that automates git workflows — branching, syncing, committing, and PRs — with multi-workflow and commit convention support.",
   type: "module",
   bin: {
@@ -11880,13 +11899,13 @@ esac
 
 # Detect available package runner
 if command -v contrib >/dev/null 2>&1; then
-  contrib validate "$commit_msg"
+  contrib validate --file "$commit_msg_file"
 elif command -v bunx >/dev/null 2>&1; then
-  bunx contrib validate "$commit_msg"
+  bunx contrib validate --file "$commit_msg_file"
 elif command -v pnpx >/dev/null 2>&1; then
-  pnpx contrib validate "$commit_msg"
+  pnpx contrib validate --file "$commit_msg_file"
 elif command -v npx >/dev/null 2>&1; then
-  npx contrib validate "$commit_msg"
+  npx contrib validate --file "$commit_msg_file"
 else
   echo "Warning: No package runner found. Skipping commit message validation."
   exit 0
@@ -13039,16 +13058,27 @@ async function performSquashMerge(origin, baseBranch, featureBranch, options) {
   if (!message) {
     const copilotError = await checkCopilotAvailable();
     if (!copilotError) {
-      const spinner = createSpinner("Generating AI commit message for squash merge...");
-      const [stagedDiff, stagedFiles] = await Promise.all([getStagedDiff(), getStagedFiles()]);
-      const aiMsg = await generateCommitMessage(stagedDiff, stagedFiles, options?.model, options?.convention ?? "clean-commit", "squash-merge");
-      if (aiMsg) {
-        message = aiMsg;
-        spinner.success("AI commit message generated.");
-        console.log(`
+      while (!message) {
+        const spinner = createSpinner("Generating AI commit message for squash merge...");
+        const [stagedDiff, stagedFiles] = await Promise.all([getStagedDiff(), getStagedFiles()]);
+        const aiMsg = await generateCommitMessage(stagedDiff, stagedFiles, options?.model, options?.convention ?? "clean-commit", "squash-merge");
+        if (aiMsg) {
+          message = aiMsg;
+          spinner.success("AI commit message generated.");
+          console.log(`
   ${import_picocolors16.default.dim("AI suggestion:")} ${import_picocolors16.default.bold(import_picocolors16.default.cyan(message))}`);
-      } else {
+          break;
+        }
         spinner.fail("AI did not return a commit message.");
+        const retryAction = await selectPrompt("AI could not generate a commit message. What would you like to do?", ["Try again with AI", "Write manually", "Cancel"]);
+        if (retryAction === "Try again with AI") {
+          continue;
+        }
+        if (retryAction === "Cancel") {
+          warn("Squash merge commit cancelled.");
+          process.exit(0);
+        }
+        break;
       }
     } else {
       warn(`AI unavailable: ${copilotError}`);
@@ -13078,7 +13108,7 @@ async function performSquashMerge(origin, baseBranch, featureBranch, options) {
   ${import_picocolors16.default.dim("AI suggestion:")} ${import_picocolors16.default.bold(import_picocolors16.default.cyan(regen))}`);
         } else {
           spinner.fail("Regeneration failed.");
-          finalMsg = await inputPrompt("Enter commit message");
+          continue;
         }
       } else {
         finalMsg = await inputPrompt("Enter commit message");
@@ -14114,6 +14144,7 @@ ${import_picocolors19.default.bold("\uD83D\uDCA1 AI Conflict Resolution Guidance
 
 // src/commands/validate.ts
 var import_picocolors20 = __toESM(require_picocolors(), 1);
+import { readFileSync as readFileSync5 } from "node:fs";
 var validate_default = defineCommand({
   meta: {
     name: "validate",
@@ -14123,7 +14154,11 @@ var validate_default = defineCommand({
     message: {
       type: "positional",
       description: "The commit message to validate",
-      required: true
+      required: false
+    },
+    file: {
+      type: "string",
+      description: "Path to a commit message file; only the first line is validated"
     }
   },
   async run({ args }) {
@@ -14137,7 +14172,11 @@ var validate_default = defineCommand({
       info('Commit convention is set to "none". All messages are accepted.');
       process.exit(0);
     }
-    const message = args.message;
+    const message = args.file ? readFileSync5(args.file, "utf-8").split(/\r?\n/, 1)[0] ?? "" : args.message;
+    if (!message) {
+      error("No commit message provided. Pass a message or use --file <path>.");
+      process.exit(1);
+    }
     if (validateCommitMessage(message, convention)) {
       success(`Valid ${CONVENTION_LABELS[convention]} message.`);
       process.exit(0);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "contribute-now",
-  "version": "0.6.2-dev.313af88",
+  "version": "0.6.2-dev.3d69403",
   "description": "Developer CLI that automates git workflows — branching, syncing, committing, and PRs — with multi-workflow and commit convention support.",
   "type": "module",
   "bin": {