From f555acfa5a70e94786ac61460e39137c7922642c Mon Sep 17 00:00:00 2001 From: Pierre Le Fevre Date: Wed, 4 Mar 2026 19:47:07 +0100 Subject: [PATCH] Fix AT-URI parsing in PR merge The parse_target_repo_info function expected 4 parts in a split AT-URI but valid AT-URIs only have 3 parts after stripping the at:// prefix. This caused all PR merges to fail with 'Invalid target repo AT-URI'. Fixed the length check from < 4 to < 3 and the rkey index from 3 to 2. --- crates/tangled-cli/src/commands/pr.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crates/tangled-cli/src/commands/pr.rs b/crates/tangled-cli/src/commands/pr.rs index 88d2666..c031d1a 100644 --- a/crates/tangled-cli/src/commands/pr.rs +++ b/crates/tangled-cli/src/commands/pr.rs @@ -507,12 +507,12 @@ async fn parse_target_repo_info( .split('/') .collect(); - if parts.len() < 4 { + if parts.len() < 3 { return Err(anyhow!("Invalid target repo AT-URI: {}", target_repo)); } let repo_did = parts[0].to_string(); - let repo_rkey = parts[3]; + let repo_rkey = parts[2]; // Get repo name #[derive(serde::Deserialize)] -- 2.51.2