fix(cli): allow --info flag to work without git repository

Fixes bounty issue #1382

Added --repo flag to allow specifying the GitHub repository directly
(in owner/repo format) when using the --info flag. This enables users
to view PR details without needing to be inside a git repository,
which is useful for quickly checking PR information.

Author: factorydroid

cortex-cli/src/pr_cmd.rs

@@ -95,6 +95,10 @@ pub struct PrCli {
     /// GitHub token for API access (for private repos).
     #[arg(long)]
     pub token: Option<String>,
+
+    /// GitHub repository in "owner/repo" format. Required when using --info outside a git repository.
+    #[arg(short, long)]
+    pub repo: Option<String>,
 }
 
 impl PrCli {
@@ -104,31 +108,57 @@ impl PrCli {
     }
 }
 
+/// Parse a repository string in "owner/repo" format.
+fn parse_repo_arg(repo_arg: &str) -> Result<(String, String)> {
+    let parts: Vec<&str> = repo_arg.split('/').collect();
+    if parts.len() != 2 || parts[0].is_empty() || parts[1].is_empty() {
+        bail!(
+            "Invalid repository format: '{}'. Expected 'owner/repo' format.",
+            repo_arg
+        );
+    }
+    Ok((parts[0].to_string(), parts[1].to_string()))
+}
+
 /// Checkout a pull request branch.
 async fn run_pr_checkout(args: PrCli) -> Result<()> {
     use cortex_engine::github::GitHubClient;
 
-    let repo_path = args.path.unwrap_or_else(|| PathBuf::from("."));
     let pr_number = args.number;
 
     // Validate PR number locally before making any API calls
     validate_pr_number(pr_number)?;
 
-    // Change to repo directory
-    std::env::set_current_dir(&repo_path)
-        .with_context(|| format!("Failed to change to directory: {}", repo_path.display()))?;
-
-    // Check if we're in a git repo
-    if !repo_path.join(".git").exists() {
-        bail!("Not a git repository. Run this command from a git repository root.");
-    }
+    // Determine the repository - either from --repo flag or from git remote
+    let repository = if let Some(ref repo_arg) = args.repo {
+        // Use the provided --repo flag
+        let (owner, repo) = parse_repo_arg(repo_arg)?;
+        format!("{}/{}", owner, repo)
+    } else {
+        // Need to be in a git repository to determine owner/repo
+        let repo_path = args.path.clone().unwrap_or_else(|| PathBuf::from("."));
+
+        // Change to repo directory
+        std::env::set_current_dir(&repo_path)
+            .with_context(|| format!("Failed to change to directory: {}", repo_path.display()))?;
+
+        // Check if we're in a git repo
+        if !repo_path.join(".git").exists() {
+            if args.info {
+                bail!(
+                    "Not a git repository. Use --repo owner/repo to specify the repository when using --info outside a git repository."
+                );
+            }
+            bail!("Not a git repository. Run this command from a git repository root.");
+        }
 
-    // Get the remote URL to determine owner/repo
-    let remote_url = get_git_remote_url()?;
-    let (owner, repo) = parse_github_url(&remote_url)
-        .with_context(|| format!("Failed to parse GitHub URL: {}", remote_url))?;
+        // Get the remote URL to determine owner/repo
+        let remote_url = get_git_remote_url()?;
+        let (owner, repo) = parse_github_url(&remote_url)
+            .with_context(|| format!("Failed to parse GitHub URL: {}", remote_url))?;
 
-    let repository = format!("{}/{}", owner, repo);
+        format!("{}/{}", owner, repo)
+    };
 
     println!("🔀 Pull Request #{}", pr_number);
     println!("{}", "=".repeat(40));
@@ -173,6 +203,17 @@ async fn run_pr_checkout(args: PrCli) -> Result<()> {
         return Ok(());
     }
 
+    // For checkout operations, we need to be in a git repository
+    // This check is needed when --repo was provided but --info was not
+    if args.repo.is_some() {
+        let repo_path = args.path.clone().unwrap_or_else(|| PathBuf::from("."));
+        std::env::set_current_dir(&repo_path)
+            .with_context(|| format!("Failed to change to directory: {}", repo_path.display()))?;
+        if !repo_path.join(".git").exists() {
+            bail!("Not a git repository. Checkout operations require a git repository.");
+        }
+    }
+
     // Check for uncommitted changes
     if !args.force {
         let status_output = Command::new("git")
@@ -405,4 +446,35 @@ mod tests {
         let result = validate_pr_number(MAX_REASONABLE_PR_NUMBER + 1);
         assert!(result.is_err());
     }
+
+    #[test]
+    fn test_parse_repo_arg_valid() {
+        let (owner, repo) = parse_repo_arg("CortexLM/cortex-cli").unwrap();
+        assert_eq!(owner, "CortexLM");
+        assert_eq!(repo, "cortex-cli");
+    }
+
+    #[test]
+    fn test_parse_repo_arg_invalid_no_slash() {
+        let result = parse_repo_arg("invalid");
+        assert!(result.is_err());
+    }
+
+    #[test]
+    fn test_parse_repo_arg_invalid_empty_owner() {
+        let result = parse_repo_arg("/repo");
+        assert!(result.is_err());
+    }
+
+    #[test]
+    fn test_parse_repo_arg_invalid_empty_repo() {
+        let result = parse_repo_arg("owner/");
+        assert!(result.is_err());
+    }
+
+    #[test]
+    fn test_parse_repo_arg_invalid_too_many_slashes() {
+        let result = parse_repo_arg("owner/repo/extra");
+        assert!(result.is_err());
+    }
 }