@snapcommit/cli 3.9.16 → 3.9.18

package/dist/commands/cursor-style.js

@@ -771,9 +771,71 @@ async function executeGitHubCommand(intent) {
             case 'new_pr':
                 const isDraft = intent.options?.draft || false;
                 console.log(chalk_1.default.blue(`\n🔄 Creating ${isDraft ? 'draft ' : ''}PR...`));
-                const pr = await github.createPullRequest(intent.options || {});
-                console.log(chalk_1.default.green(`✓ ${isDraft ? 'Draft ' : ''}PR #${pr.number} created`));
-                console.log(chalk_1.default.cyan(`  ${pr.html_url}\n`));
+                try {
+                    const pr = await github.createPullRequest(intent.options || {});
+                    console.log(chalk_1.default.green(`✓ ${isDraft ? 'Draft ' : ''}PR #${pr.number} created`));
+                    console.log(chalk_1.default.cyan(`  ${pr.html_url}\n`));
+                }
+                catch (error) {
+                    console.log(chalk_1.default.red('\n❌ Cannot create PR\n'));
+                    // Check if it's because we're on main branch
+                    try {
+                        const currentBranch = (0, child_process_1.execSync)('git rev-parse --abbrev-ref HEAD', { encoding: 'utf-8' }).trim();
+                        const defaultBranch = (0, child_process_1.execSync)('git symbolic-ref refs/remotes/origin/HEAD', { encoding: 'utf-8' }).trim().replace('refs/remotes/origin/', '');
+                        if (currentBranch === defaultBranch || currentBranch === 'main' || currentBranch === 'master') {
+                            console.log(chalk_1.default.yellow('⚠️  You\'re on the main branch!\n'));
+                            console.log(chalk_1.default.white('Why this fails:'));
+                            console.log(chalk_1.default.gray('  • PRs merge FROM a feature branch TO main'));
+                            console.log(chalk_1.default.gray('  • You can\'t create a PR from main to main\n'));
+                            console.log(chalk_1.default.cyan('💡 Here\'s what to do:\n'));
+                            console.log(chalk_1.default.white('1. Create a new branch:'));
+                            console.log(chalk_1.default.cyan('   create a new branch called feature/your-feature\n'));
+                            console.log(chalk_1.default.white('2. Make your changes'));
+                            console.log(chalk_1.default.white('3. Commit and push:'));
+                            console.log(chalk_1.default.cyan('   commit and push changes\n'));
+                            console.log(chalk_1.default.white('4. Then create PR:'));
+                            console.log(chalk_1.default.cyan('   create a PR to main\n'));
+                            return;
+                        }
+                    }
+                    catch (branchError) {
+                        // If we can't determine branch, show generic error
+                    }
+                    // Check if there are no commits to push
+                    try {
+                        const commitsAhead = (0, child_process_1.execSync)('git rev-list --count @{u}..HEAD 2>/dev/null || echo 0', { encoding: 'utf-8' }).trim();
+                        if (commitsAhead === '0') {
+                            console.log(chalk_1.default.yellow('⚠️  No new commits on this branch!\n'));
+                            console.log(chalk_1.default.white('Why this fails:'));
+                            console.log(chalk_1.default.gray('  • PRs need at least 1 commit that\'s not in main'));
+                            console.log(chalk_1.default.gray('  • Your branch is identical to main\n'));
+                            console.log(chalk_1.default.cyan('💡 Here\'s what to do:\n'));
+                            console.log(chalk_1.default.white('1. Make some changes to your code'));
+                            console.log(chalk_1.default.white('2. Commit and push:'));
+                            console.log(chalk_1.default.cyan('   commit and push changes\n'));
+                            console.log(chalk_1.default.white('3. Then create PR:'));
+                            console.log(chalk_1.default.cyan('   create a PR to main\n'));
+                            return;
+                        }
+                    }
+                    catch (commitError) {
+                        // If we can't check commits, continue to generic error
+                    }
+                    // Generic error with helpful message
+                    console.log(chalk_1.default.white('Common reasons:'));
+                    console.log(chalk_1.default.gray('  • Branch is up to date with main (no changes)'));
+                    console.log(chalk_1.default.gray('  • PR already exists for this branch'));
+                    console.log(chalk_1.default.gray('  • No commits on this branch\n'));
+                    console.log(chalk_1.default.cyan('💡 Try:\n'));
+                    console.log(chalk_1.default.white('• Check existing PRs:'));
+                    console.log(chalk_1.default.cyan('  show all PRs\n'));
+                    console.log(chalk_1.default.white('• Check your branch:'));
+                    console.log(chalk_1.default.cyan('  what branch am i on'));
+                    console.log(chalk_1.default.cyan('  show last 5 commits\n'));
+                    if (error.message) {
+                        console.log(chalk_1.default.gray(`\nError: ${error.message}\n`));
+                    }
+                }
                 break;
             case 'pr_list':
             case 'list_prs':
@@ -799,12 +861,73 @@ async function executeGitHubCommand(intent) {
                     prNumber = await github.findPRNumber('current');
                     if (!prNumber) {
                         console.log(chalk_1.default.red('\n❌ No PR found for current branch\n'));
+                        console.log(chalk_1.default.white('Why this fails:'));
+                        console.log(chalk_1.default.gray('  • This branch doesn\'t have an open PR\n'));
+                        console.log(chalk_1.default.cyan('💡 Here\'s what to do:\n'));
+                        console.log(chalk_1.default.white('1. Check if a PR exists:'));
+                        console.log(chalk_1.default.cyan('   show all PRs\n'));
+                        console.log(chalk_1.default.white('2. Or create a PR first:'));
+                        console.log(chalk_1.default.cyan('   create a PR to main\n'));
+                        console.log(chalk_1.default.white('3. Then merge it:'));
+                        console.log(chalk_1.default.cyan('   merge my PR\n'));
                         return;
                     }
                 }
                 console.log(chalk_1.default.blue(`\n🔄 Merging PR #${prNumber}...`));
-                await github.mergePullRequest(prNumber);
-                console.log(chalk_1.default.green(`✓ PR #${prNumber} merged\n`));
+                try {
+                    await github.mergePullRequest(prNumber);
+                    console.log(chalk_1.default.green(`✓ PR #${prNumber} merged\n`));
+                }
+                catch (error) {
+                    console.log(chalk_1.default.red('\n❌ Cannot merge PR\n'));
+                    const errorMsg = error.message?.toLowerCase() || '';
+                    if (errorMsg.includes('not mergeable') || errorMsg.includes('merge conflict')) {
+                        console.log(chalk_1.default.yellow('⚠️  PR has merge conflicts!\n'));
+                        console.log(chalk_1.default.white('Why this fails:'));
+                        console.log(chalk_1.default.gray('  • Changes in this PR conflict with main branch'));
+                        console.log(chalk_1.default.gray('  • You need to resolve conflicts first\n'));
+                        console.log(chalk_1.default.cyan('💡 Here\'s what to do:\n'));
+                        console.log(chalk_1.default.white('1. Pull latest changes from main:'));
+                        console.log(chalk_1.default.cyan('   merge main into this branch\n'));
+                        console.log(chalk_1.default.white('2. SnapCommit will auto-resolve conflicts'));
+                        console.log(chalk_1.default.white('3. Then try merging PR again:'));
+                        console.log(chalk_1.default.cyan('   merge my PR\n'));
+                    }
+                    else if (errorMsg.includes('required status check') || errorMsg.includes('checks')) {
+                        console.log(chalk_1.default.yellow('⚠️  CI checks haven\'t passed!\n'));
+                        console.log(chalk_1.default.white('Why this fails:'));
+                        console.log(chalk_1.default.gray('  • GitHub requires all checks to pass before merging'));
+                        console.log(chalk_1.default.gray('  • Your tests/build might be failing\n'));
+                        console.log(chalk_1.default.cyan('💡 Here\'s what to do:\n'));
+                        console.log(chalk_1.default.white('1. Check CI status:'));
+                        console.log(chalk_1.default.cyan('   check if my build passed\n'));
+                        console.log(chalk_1.default.white('2. Fix any failing tests'));
+                        console.log(chalk_1.default.white('3. Push fixes:'));
+                        console.log(chalk_1.default.cyan('   commit and push changes\n'));
+                        console.log(chalk_1.default.white('4. Then try merging again'));
+                    }
+                    else if (errorMsg.includes('review') || errorMsg.includes('approval')) {
+                        console.log(chalk_1.default.yellow('⚠️  PR needs approval!\n'));
+                        console.log(chalk_1.default.white('Why this fails:'));
+                        console.log(chalk_1.default.gray('  • Repository requires code review approval'));
+                        console.log(chalk_1.default.gray('  • You need a teammate to review and approve\n'));
+                        console.log(chalk_1.default.cyan('💡 Here\'s what to do:\n'));
+                        console.log(chalk_1.default.white('1. Ask a teammate to review your PR'));
+                        console.log(chalk_1.default.white('2. Wait for approval'));
+                        console.log(chalk_1.default.white('3. Then try merging again\n'));
+                    }
+                    else {
+                        console.log(chalk_1.default.white('Common reasons:'));
+                        console.log(chalk_1.default.gray('  • CI checks still running'));
+                        console.log(chalk_1.default.gray('  • Required reviews not approved'));
+                        console.log(chalk_1.default.gray('  • Merge conflicts exist\n'));
+                        console.log(chalk_1.default.cyan('💡 Try:\n'));
+                        console.log(chalk_1.default.cyan('  check if my build passed\n'));
+                        if (error.message) {
+                            console.log(chalk_1.default.gray(`Error: ${error.message}\n`));
+                        }
+                    }
+                }
                 break;
             case 'ci_check':
             case 'ci_status':
@@ -817,26 +940,52 @@ async function executeGitHubCommand(intent) {
                     const totalChecks = ciStatus.checks.length;
                     const failedChecks = ciStatus.checks.filter((c) => c.conclusion === 'failure').length;
                     if (state === 'success' || (totalChecks > 0 && failedChecks === 0)) {
-                        console.log(chalk_1.default.green(`✓ All checks passed (${totalChecks} checks)\n`));
+                        console.log(chalk_1.default.green(`✅ All checks passed (${totalChecks} checks)\n`));
                     }
                     else if (state === 'pending') {
                         console.log(chalk_1.default.yellow(`⏳ Checks running (${totalChecks} checks)\n`));
+                        console.log(chalk_1.default.gray('💡 Wait a few minutes and check again\n'));
                     }
                     else {
                         console.log(chalk_1.default.red(`❌ ${failedChecks}/${totalChecks} checks failed\n`));
+                        console.log(chalk_1.default.white('Why this matters:'));
+                        console.log(chalk_1.default.gray('  • Failed checks block PR merging'));
+                        console.log(chalk_1.default.gray('  • Something is broken in your code\n'));
+                        console.log(chalk_1.default.cyan('💡 Here\'s what to do:\n'));
+                        console.log(chalk_1.default.white('1. View full details on GitHub:'));
+                        console.log(chalk_1.default.cyan('   show my PR\n'));
+                        console.log(chalk_1.default.white('2. Fix the failing tests/build'));
+                        console.log(chalk_1.default.white('3. Push the fixes:'));
+                        console.log(chalk_1.default.cyan('   commit and push changes\n'));
+                        console.log(chalk_1.default.white('4. Check status again:'));
+                        console.log(chalk_1.default.cyan('   check if my build passed\n'));
                     }
                 }
                 catch (error) {
-                    console.log(chalk_1.default.gray('   No CI checks found\n'));
+                    console.log(chalk_1.default.gray('\n✓ No CI checks configured\n'));
+                    console.log(chalk_1.default.white('This is fine! Not all repos need CI.\n'));
                 }
                 break;
             case 'issue_create':
             case 'create_issue':
             case 'new_issue':
                 console.log(chalk_1.default.blue('\n🔄 Creating issue...'));
-                const issue = await github.createIssue(intent.options || {});
-                console.log(chalk_1.default.green(`✓ Issue #${issue.number} created`));
-                console.log(chalk_1.default.cyan(`  ${issue.html_url}\n`));
+                try {
+                    const issue = await github.createIssue(intent.options || {});
+                    console.log(chalk_1.default.green(`✓ Issue #${issue.number} created`));
+                    console.log(chalk_1.default.cyan(`  ${issue.html_url}\n`));
+                }
+                catch (error) {
+                    console.log(chalk_1.default.red('\n❌ Cannot create issue\n'));
+                    console.log(chalk_1.default.white('Common reasons:'));
+                    console.log(chalk_1.default.gray('  • Missing title or description'));
+                    console.log(chalk_1.default.gray('  • No permission to create issues\n'));
+                    console.log(chalk_1.default.cyan('💡 Try:\n'));
+                    console.log(chalk_1.default.cyan('  create an issue with title "Bug in login" and description "Users cant login"\n'));
+                    if (error.message) {
+                        console.log(chalk_1.default.gray(`Error: ${error.message}\n`));
+                    }
+                }
                 break;
             case 'issue_list':
             case 'list_issues':
@@ -1246,7 +1395,76 @@ async function executeGitHubCommand(intent) {
         }
     }
     catch (error) {
-        console.log(chalk_1.default.red(`\n❌ ${error.message}\n`));
+        // Intelligent error handling - NEVER show raw API errors
+        const errorMsg = error.message?.toLowerCase() || '';
+        // GitHub authentication errors
+        if (errorMsg.includes('bad credentials') || errorMsg.includes('unauthorized') || errorMsg.includes('401')) {
+            console.log(chalk_1.default.red('\n❌ GitHub authentication failed\n'));
+            console.log(chalk_1.default.white('Why this fails:'));
+            console.log(chalk_1.default.gray('  • Your GitHub connection expired'));
+            console.log(chalk_1.default.gray('  • Invalid or revoked access token\n'));
+            console.log(chalk_1.default.cyan('💡 Here\'s what to do:\n'));
+            console.log(chalk_1.default.white('1. Reconnect GitHub:'));
+            console.log(chalk_1.default.cyan('   github connect\n'));
+            console.log(chalk_1.default.white('2. Then try your command again\n'));
+            return;
+        }
+        // Permission errors
+        if (errorMsg.includes('not found') || errorMsg.includes('404')) {
+            console.log(chalk_1.default.red('\n❌ Repository or resource not found\n'));
+            console.log(chalk_1.default.white('Common reasons:'));
+            console.log(chalk_1.default.gray('  • Repository is private and you don\'t have access'));
+            console.log(chalk_1.default.gray('  • PR/issue number doesn\'t exist'));
+            console.log(chalk_1.default.gray('  • Repository name changed\n'));
+            console.log(chalk_1.default.cyan('💡 Try:\n'));
+            console.log(chalk_1.default.white('• Check repository access'));
+            console.log(chalk_1.default.white('• Verify PR/issue numbers:'));
+            console.log(chalk_1.default.cyan('  show all PRs\n'));
+            return;
+        }
+        // Rate limit errors
+        if (errorMsg.includes('rate limit') || errorMsg.includes('403')) {
+            console.log(chalk_1.default.red('\n❌ GitHub rate limit exceeded\n'));
+            console.log(chalk_1.default.white('Why this fails:'));
+            console.log(chalk_1.default.gray('  • Too many API requests in a short time'));
+            console.log(chalk_1.default.gray('  • GitHub limits API calls per hour\n'));
+            console.log(chalk_1.default.cyan('💡 Here\'s what to do:\n'));
+            console.log(chalk_1.default.white('Wait 10-15 minutes and try again'));
+            console.log(chalk_1.default.gray('(Rate limits reset hourly)\n'));
+            return;
+        }
+        // Validation errors
+        if (errorMsg.includes('validation') || errorMsg.includes('invalid')) {
+            console.log(chalk_1.default.red('\n❌ Invalid input\n'));
+            console.log(chalk_1.default.white('Common reasons:'));
+            console.log(chalk_1.default.gray('  • Missing required information (title, body, etc.)'));
+            console.log(chalk_1.default.gray('  • Invalid format or parameters\n'));
+            console.log(chalk_1.default.cyan('💡 Try:\n'));
+            console.log(chalk_1.default.white('Be more specific with your command'));
+            console.log(chalk_1.default.gray('Example: "create a PR with title \'Fix bug\' and description \'Fixes #123\'"\n'));
+            return;
+        }
+        // Network errors
+        if (errorMsg.includes('network') || errorMsg.includes('fetch failed') || errorMsg.includes('econnrefused')) {
+            console.log(chalk_1.default.red('\n❌ Network error\n'));
+            console.log(chalk_1.default.white('Why this fails:'));
+            console.log(chalk_1.default.gray('  • No internet connection'));
+            console.log(chalk_1.default.gray('  • GitHub API is down (rare)\n'));
+            console.log(chalk_1.default.cyan('💡 Here\'s what to do:\n'));
+            console.log(chalk_1.default.white('1. Check your internet connection'));
+            console.log(chalk_1.default.white('2. Try again in a few seconds\n'));
+            return;
+        }
+        // Generic fallback with helpful context
+        console.log(chalk_1.default.red('\n❌ Command failed\n'));
+        console.log(chalk_1.default.white('What happened:'));
+        console.log(chalk_1.default.gray(`  ${error.message}\n`));
+        console.log(chalk_1.default.cyan('💡 Try:\n'));
+        console.log(chalk_1.default.white('• Check your command:'));
+        console.log(chalk_1.default.cyan('  what branch am i on'));
+        console.log(chalk_1.default.cyan('  show last 5 commits\n'));
+        console.log(chalk_1.default.white('• Reconnect GitHub if needed:'));
+        console.log(chalk_1.default.cyan('  github connect\n'));
     }
 }
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@snapcommit/cli",
-  "version": "3.9.16",
+  "version": "3.9.18",
   "description": "Instant AI commits. Beautiful progress tracking. Never write commit messages again.",
   "main": "dist/index.js",
   "bin": {