Fix CI test failure and CommitCommand edit flow validation

[Copilot]

Description

Two unresolved CodeRabbit issues causing CI failure and broken UX in the commit workflow.

CI fix (package.json)

• "test" was hardcoded to exit 1, failing every CI run. Changed to npm run lint (TypeScript type-checking) as a meaningful smoke test until a real test suite is added.

CommitCommand edit flow (CommitCommand.ts)

• Invalid AI suggestion previously aborted the command entirely — now warns and routes user into the edit flow.
• Invalid manual edit previously returned to the outer accept/edit/reject loop, where a blank Enter would silently commit the stale AI suggestion. Now keeps prompting within the edit loop until a valid message is entered.
• Added a defensive null-check before git.commit() since commitMessage is now typed as string | null.

-  if (suggestedValidationError) {
-    logger.error(`Invalid AI commit message ...`);
-    return;  // abort
-  }
+  if (suggestedValidationError) {
+    logger.warn(`Invalid AI commit message: ${suggestedValidationError}`);
+    console.warn(`⚠️ AI returned an invalid message: ${suggestedValidationError}`);
+    // fall through to edit loop
+  }

   if (choice === 'e' || choice === 'edit') {
-    const editedMessage = await rl.question(...);
-    if (editedValidationError) { continue; }  // back to outer loop
+    while (true) {
+      const editedMessage = await rl.question(...);
+      if (!editedValidationError) { commitMessage = editedMessage.trim(); break; }
+      console.error(`❌ Invalid commit message: ${editedValidationError}`);
+    }
     break;
   }

Related Issue

Closes #

Type of Change

• 🐛 Bug fix (non-breaking change that fixes an issue)
• ✨ New feature (non-breaking change that adds functionality)
• 💥 Breaking change (fix or feature that would cause existing functionality to change)
• 📝 Documentation update
• 🔧 Refactor / internal improvement
• 🤖 CI/CD or tooling change
• ⬆️ Dependency update

Checklist

• I have read the CONTRIBUTING guidelines (if present)
• My code follows the existing code style
• I have added or updated tests that cover my changes
• All existing tests pass locally
• I have updated the documentation where necessary
• My changes do not introduce new security vulnerabilities

How Has This Been Tested?

npm run lint and npm run build pass. npm test (now aliased to lint) exits 0. Edit-flow logic verified by code inspection against all input branches.

Screenshots (if applicable)

N/A

Additional Notes

CodeQL scan found 0 alerts.

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.