gitpadi 2.0.5 → 2.0.7

package/dist/cli.js

@@ -12,7 +12,7 @@ import { execSync } from 'child_process';
 import boxen from 'boxen';
 import { createSpinner } from 'nanospinner';
 import { Octokit } from '@octokit/rest';
-import { initGitHub, setRepo, getOwner, getRepo, getOctokit, saveConfig, getToken } from './core/github.js';
+import { initGitHub, setRepo, getOwner, getRepo, getOctokit, saveConfig, getToken, getAuthenticatedUser, getRepoPermissions } from './core/github.js';
 import * as issues from './commands/issues.js';
 import * as prs from './commands/prs.js';
 import * as repos from './commands/repos.js';
@@ -20,6 +20,7 @@ import * as contributors from './commands/contributors.js';
 import * as releases from './commands/releases.js';
 import * as contribute from './commands/contribute.js';
 const VERSION = '2.0.0';
+let targetConfirmed = false;
 // ── Styling ────────────────────────────────────────────────────────────
 const cyber = gradient(['#ff00ff', '#00ffff', '#ff00ff']);
 const neon = gradient(['#00ff87', '#60efff']);
@@ -157,35 +158,102 @@ async function ensureAuthenticated() {
 /**
  * Ensures we have a target repository (Owner/Repo)
  */
-async function ensureTargetRepo() {
+async function ensureTargetRepo(force = false) {
     let owner = getOwner();
     let repo = getRepo();
-    if (owner && repo) {
-        return;
+    if (!force && owner && repo) {
+        if (targetConfirmed)
+            return;
+        const { confirm } = await inquirer.prompt([{
+                type: 'confirm',
+                name: 'confirm',
+                message: cyan('🎯 Targeting ') + bold(`${owner}/${repo}`) + cyan('. Correct?'),
+                default: true
+            }]);
+        if (confirm) {
+            targetConfirmed = true;
+            return;
+        }
     }
     console.log(neon('\n  📦 Project Targeting — which repo are we working on?\n'));
-    const answers = await inquirer.prompt([
-        {
+    // 1. Get the Owner/Org
+    const { targetOwner } = await inquirer.prompt([{
             type: 'input',
-            name: 'owner',
+            name: 'targetOwner',
             message: cyan('👤 GitHub Owner/Org:'),
-            default: owner || '',
-            validate: (v) => v.length > 0 || 'Required',
-        },
-        {
-            type: 'input',
-            name: 'repo',
-            message: cyan('📦 Repository name:'),
-            default: repo || '',
+            default: owner || await getAuthenticatedUser(),
             validate: (v) => v.length > 0 || 'Required',
-        },
-    ]);
-    setRepo(answers.owner, answers.repo);
-    saveConfig({ token: getToken(), owner: answers.owner, repo: answers.repo });
-    const spinner = createSpinner(dim('Connecting...')).start();
-    await sleep(400);
-    spinner.success({ text: green(`Locked in → ${cyan(`${answers.owner}/${answers.repo}`)}`) });
-    console.log('');
+        }]);
+    // 2. Fetch Repos for that Owner
+    const spin = createSpinner(dim(`Fetching repositories for ${targetOwner}...`)).start();
+    const fetchedRepos = await repos.listRepos({ owner: targetOwner, silent: true });
+    if (fetchedRepos.length > 0) {
+        spin.success({ text: green(`Found ${fetchedRepos.length} repositories for ${targetOwner}`) });
+        const { selectedRepo } = await inquirer.prompt([{
+                type: 'list',
+                name: 'selectedRepo',
+                message: cyan('📦 Select Repository:'),
+                choices: [
+                    ...fetchedRepos.map((r) => ({ name: r.name, value: r.name })),
+                    new inquirer.Separator(),
+                    { name: '✍️  Enter manually...', value: '__manual__' }
+                ],
+                loop: false
+            }]);
+        if (selectedRepo !== '__manual__') {
+            repo = selectedRepo;
+        }
+        else {
+            const { manualRepo } = await inquirer.prompt([{
+                    type: 'input',
+                    name: 'manualRepo',
+                    message: cyan('📦 Repository name:'),
+                    validate: (v) => v.length > 0 || 'Required',
+                }]);
+            repo = manualRepo;
+        }
+    }
+    else {
+        spin.warn({ text: yellow(`No public repositories found for ${targetOwner}.`) });
+        const { manualRepo } = await inquirer.prompt([{
+                type: 'input',
+                name: 'manualRepo',
+                message: cyan('📦 Enter Repository name manually:'),
+                validate: (v) => v.length > 0 || 'Required',
+            }]);
+        repo = manualRepo;
+    }
+    setRepo(targetOwner, repo);
+    saveConfig({ token: getToken(), owner: targetOwner, repo: repo });
+    targetConfirmed = true;
+    // 4. Permission Check (for Maintainer safety)
+    const checkSpin = createSpinner(dim('Checking maintainer permissions...')).start();
+    try {
+        const perms = await getRepoPermissions(targetOwner, repo);
+        if (!perms.push && !perms.admin) {
+            checkSpin.warn({ text: yellow(`Limited Access → ${cyan(`${targetOwner}/${repo}`)}`) });
+            console.log(boxen(chalk.yellow(`🛡️  Maintainer Mode Warning\n\n`) +
+                chalk.dim(`You do not have push/admin permissions for this repository.\n`) +
+                chalk.dim(`Most maintainer actions (bulk issues, merging PRs) will fail.`), { padding: 1, borderColor: 'yellow', borderStyle: 'round' }));
+            const { proceed } = await inquirer.prompt([{
+                    type: 'confirm',
+                    name: 'proceed',
+                    message: 'Continue anyway?',
+                    default: false
+                }]);
+            if (!proceed) {
+                targetConfirmed = false;
+                throw new BackToMenu();
+            }
+        }
+        else {
+            checkSpin.success({ text: green(`Permissions verified for ${cyan(`${targetOwner}/${repo}`)}`) });
+        }
+    }
+    catch {
+        checkSpin.warn({ text: yellow('Could not verify permissions (API limit or private)') });
+    }
+    console.log(green(`\n  ✅ Locked in → ${cyan(`${targetOwner}/${repo}`)}\n`));
 }
 // ── Mode Selector ──────────────────────────────────────────────────────
 async function mainMenu() {
@@ -333,16 +401,7 @@ async function maintainerMenu() {
         if (category === 'back')
             break;
         if (category === 'switch') {
-            await safeMenu(async () => {
-                const a = await inquirer.prompt([
-                    { type: 'input', name: 'owner', message: cyan('👤 New Owner/Org:'), default: getOwner() },
-                    { type: 'input', name: 'repo', message: cyan('📦 New Repo:'), default: getRepo() },
-                ]);
-                setRepo(a.owner, a.repo);
-                const s = createSpinner(dim('Switching...')).start();
-                await sleep(400);
-                s.success({ text: green(`Now targeting ${cyan(`${a.owner}/${a.repo}`)}`) });
-            });
+            await ensureTargetRepo(true);
             continue;
         }
         if (category === 'issues')

package/dist/commands/repos.js

@@ -91,9 +91,23 @@ export async function listRepos(opts) {
     const spinner = !opts.silent ? ora('Fetching repos...').start() : null;
     const octokit = getOctokit();
     try {
-        let repos;
-        if (opts.org) {
-            ({ data: repos } = await octokit.repos.listForOrg({ org: opts.org, per_page: opts.limit || 100, sort: 'updated' }));
+        let repos = [];
+        const owner = opts.org || opts.owner;
+        if (owner) {
+            // Check if it's an org or user
+            try {
+                const { data: entity } = await octokit.users.getByUsername({ username: owner });
+                if (entity.type === 'Organization') {
+                    ({ data: repos } = await octokit.repos.listForOrg({ org: owner, per_page: opts.limit || 100, sort: 'updated' }));
+                }
+                else {
+                    ({ data: repos } = await octokit.repos.listForUser({ username: owner, per_page: opts.limit || 100, sort: 'updated' }));
+                }
+            }
+            catch {
+                // Fallback to org if lookup fails (might be a private org only visible to token)
+                ({ data: repos } = await octokit.repos.listForOrg({ org: owner, per_page: opts.limit || 100, sort: 'updated' }));
+            }
         }
         else {
             ({ data: repos } = await octokit.repos.listForAuthenticatedUser({ per_page: opts.limit || 100, sort: 'updated' }));

package/dist/core/github.js

@@ -94,6 +94,10 @@ export async function getRepoDetails(owner, repo) {
     const { data } = await getOctokit().repos.get({ owner, repo });
     return data;
 }
+export async function getRepoPermissions(owner, repo) {
+    const data = await getRepoDetails(owner, repo);
+    return data.permissions || { admin: false, push: false, pull: true };
+}
 export async function getLatestCheckRuns(owner, repo, ref) {
     const octokit = getOctokit();
     const { data: checks } = await octokit.checks.listForRef({ owner, repo, ref });

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "gitpadi",
-    "version": "2.0.5",
+    "version": "2.0.7",
     "description": "GitPadi — Your AI-powered GitHub management CLI. Create repos, manage issues & PRs, score contributors, and automate everything.",
     "type": "module",
     "bin": {

package/src/cli.ts

@@ -15,7 +15,7 @@ import { execSync } from 'child_process';
 import boxen from 'boxen';
 import { createSpinner } from 'nanospinner';
 import { Octokit } from '@octokit/rest';
-import { initGitHub, setRepo, getOwner, getRepo, getOctokit, loadConfig, saveConfig, getToken } from './core/github.js';
+import { initGitHub, setRepo, getOwner, getRepo, getOctokit, loadConfig, saveConfig, getToken, getAuthenticatedUser, getRepoPermissions } from './core/github.js';
 
 import * as issues from './commands/issues.js';
 import * as prs from './commands/prs.js';
@@ -25,6 +25,7 @@ import * as releases from './commands/releases.js';
 import * as contribute from './commands/contribute.js';
 
 const VERSION = '2.0.0';
+let targetConfirmed = false;
 
 // ── Styling ────────────────────────────────────────────────────────────
 const cyber = gradient(['#ff00ff', '#00ffff', '#ff00ff']);
@@ -178,40 +179,113 @@ async function ensureAuthenticated() {
 /**
  * Ensures we have a target repository (Owner/Repo)
  */
-async function ensureTargetRepo() {
+async function ensureTargetRepo(force = false) {
     let owner = getOwner();
     let repo = getRepo();
 
-    if (owner && repo) {
-        return;
+    if (!force && owner && repo) {
+        if (targetConfirmed) return;
+
+        const { confirm } = await inquirer.prompt([{
+            type: 'confirm',
+            name: 'confirm',
+            message: cyan('🎯 Targeting ') + bold(`${owner}/${repo}`) + cyan('. Correct?'),
+            default: true
+        }]);
+        if (confirm) {
+            targetConfirmed = true;
+            return;
+        }
     }
 
     console.log(neon('\n  📦 Project Targeting — which repo are we working on?\n'));
 
-    const answers = await inquirer.prompt([
-        {
-            type: 'input',
-            name: 'owner',
-            message: cyan('👤 GitHub Owner/Org:'),
-            default: owner || '',
-            validate: (v: string) => v.length > 0 || 'Required',
-        },
-        {
+    // 1. Get the Owner/Org
+    const { targetOwner } = await inquirer.prompt([{
+        type: 'input',
+        name: 'targetOwner',
+        message: cyan('👤 GitHub Owner/Org:'),
+        default: owner || await getAuthenticatedUser(),
+        validate: (v: string) => v.length > 0 || 'Required',
+    }]);
+
+    // 2. Fetch Repos for that Owner
+    const spin = createSpinner(dim(`Fetching repositories for ${targetOwner}...`)).start();
+    const fetchedRepos = await repos.listRepos({ owner: targetOwner, silent: true });
+
+    if (fetchedRepos.length > 0) {
+        spin.success({ text: green(`Found ${fetchedRepos.length} repositories for ${targetOwner}`) });
+
+        const { selectedRepo } = await inquirer.prompt([{
+            type: 'list',
+            name: 'selectedRepo',
+            message: cyan('📦 Select Repository:'),
+            choices: [
+                ...fetchedRepos.map((r: any) => ({ name: r.name, value: r.name })),
+                new inquirer.Separator(),
+                { name: '✍️  Enter manually...', value: '__manual__' }
+            ],
+            loop: false
+        }]);
+
+        if (selectedRepo !== '__manual__') {
+            repo = selectedRepo;
+        } else {
+            const { manualRepo } = await inquirer.prompt([{
+                type: 'input',
+                name: 'manualRepo',
+                message: cyan('📦 Repository name:'),
+                validate: (v: string) => v.length > 0 || 'Required',
+            }]);
+            repo = manualRepo;
+        }
+    } else {
+        spin.warn({ text: yellow(`No public repositories found for ${targetOwner}.`) });
+        const { manualRepo } = await inquirer.prompt([{
             type: 'input',
-            name: 'repo',
-            message: cyan('📦 Repository name:'),
-            default: repo || '',
+            name: 'manualRepo',
+            message: cyan('📦 Enter Repository name manually:'),
             validate: (v: string) => v.length > 0 || 'Required',
-        },
-    ]);
+        }]);
+        repo = manualRepo;
+    }
 
-    setRepo(answers.owner, answers.repo);
-    saveConfig({ token: getToken(), owner: answers.owner, repo: answers.repo });
+    setRepo(targetOwner, repo as string);
+    saveConfig({ token: getToken(), owner: targetOwner, repo: repo as string });
+    targetConfirmed = true;
 
-    const spinner = createSpinner(dim('Connecting...')).start();
-    await sleep(400);
-    spinner.success({ text: green(`Locked in → ${cyan(`${answers.owner}/${answers.repo}`)}`) });
-    console.log('');
+    // 4. Permission Check (for Maintainer safety)
+    const checkSpin = createSpinner(dim('Checking maintainer permissions...')).start();
+    try {
+        const perms = await getRepoPermissions(targetOwner, repo as string);
+        if (!perms.push && !perms.admin) {
+            checkSpin.warn({ text: yellow(`Limited Access → ${cyan(`${targetOwner}/${repo}`)}`) });
+            console.log(boxen(
+                chalk.yellow(`🛡️  Maintainer Mode Warning\n\n`) +
+                chalk.dim(`You do not have push/admin permissions for this repository.\n`) +
+                chalk.dim(`Most maintainer actions (bulk issues, merging PRs) will fail.`),
+                { padding: 1, borderColor: 'yellow', borderStyle: 'round' }
+            ));
+
+            const { proceed } = await inquirer.prompt([{
+                type: 'confirm',
+                name: 'proceed',
+                message: 'Continue anyway?',
+                default: false
+            }]);
+
+            if (!proceed) {
+                targetConfirmed = false;
+                throw new BackToMenu();
+            }
+        } else {
+            checkSpin.success({ text: green(`Permissions verified for ${cyan(`${targetOwner}/${repo}`)}`) });
+        }
+    } catch {
+        checkSpin.warn({ text: yellow('Could not verify permissions (API limit or private)') });
+    }
+
+    console.log(green(`\n  ✅ Locked in → ${cyan(`${targetOwner}/${repo}`)}\n`));
 }
 
 // ── Mode Selector ──────────────────────────────────────────────────────
@@ -365,16 +439,7 @@ async function maintainerMenu() {
         if (category === 'back') break;
 
         if (category === 'switch') {
-            await safeMenu(async () => {
-                const a = await inquirer.prompt([
-                    { type: 'input', name: 'owner', message: cyan('👤 New Owner/Org:'), default: getOwner() },
-                    { type: 'input', name: 'repo', message: cyan('📦 New Repo:'), default: getRepo() },
-                ]);
-                setRepo(a.owner, a.repo);
-                const s = createSpinner(dim('Switching...')).start();
-                await sleep(400);
-                s.success({ text: green(`Now targeting ${cyan(`${a.owner}/${a.repo}`)}`) });
-            });
+            await ensureTargetRepo(true);
             continue;
         }
 

package/src/commands/repos.ts

@@ -101,14 +101,27 @@ export async function setTopics(name: string, topics: string[], opts: { org?: st
     } catch (e: any) { spinner.fail(e.message); }
 }
 
-export async function listRepos(opts: { org?: string; limit?: number; silent?: boolean }) {
+export async function listRepos(opts: { org?: string; owner?: string; limit?: number; silent?: boolean }) {
     const spinner = !opts.silent ? ora('Fetching repos...').start() : null;
     const octokit = getOctokit();
 
     try {
-        let repos: any[];
-        if (opts.org) {
-            ({ data: repos } = await octokit.repos.listForOrg({ org: opts.org, per_page: opts.limit || 100, sort: 'updated' }));
+        let repos: any[] = [];
+        const owner = opts.org || opts.owner;
+
+        if (owner) {
+            // Check if it's an org or user
+            try {
+                const { data: entity } = await octokit.users.getByUsername({ username: owner });
+                if (entity.type === 'Organization') {
+                    ({ data: repos } = await octokit.repos.listForOrg({ org: owner, per_page: opts.limit || 100, sort: 'updated' }));
+                } else {
+                    ({ data: repos } = await octokit.repos.listForUser({ username: owner, per_page: opts.limit || 100, sort: 'updated' }));
+                }
+            } catch {
+                // Fallback to org if lookup fails (might be a private org only visible to token)
+                ({ data: repos } = await octokit.repos.listForOrg({ org: owner, per_page: opts.limit || 100, sort: 'updated' }));
+            }
         } else {
             ({ data: repos } = await octokit.repos.listForAuthenticatedUser({ per_page: opts.limit || 100, sort: 'updated' }));
         }

package/src/core/github.ts

@@ -114,6 +114,11 @@ export async function getRepoDetails(owner: string, repo: string) {
     return data;
 }
 
+export async function getRepoPermissions(owner: string, repo: string) {
+    const data = await getRepoDetails(owner, repo);
+    return data.permissions || { admin: false, push: false, pull: true };
+}
+
 export async function getLatestCheckRuns(owner: string, repo: string, ref: string) {
     const octokit = getOctokit();
     const { data: checks } = await octokit.checks.listForRef({ owner, repo, ref });