fix: YAML path matching and legacy SSH URL support (#95)

* fix: YAML path matching and legacy SSH URL support

- Strip leading '/' from ADO yamlFilename before comparing to local
  relative paths (ADO stores e.g. '/.azdo/pipelines/agent.yml')
- Add legacy SSH URL format: git@vs-ssh.visualstudio.com:v3/...
- Fix step numbering in run() comments (3→4, was skipping 4)
- Add tests for YAML path matching with/without leading slash
- Add test for legacy SSH URL parsing

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix: extract normalize_ado_yaml_path helper and update docs

- Extract YAML path normalization into a named helper function so tests
  exercise the same code path as production (not duplicated logic)
- Normalize separators before trimming leading slash (handles backslash paths)
- Add backslash normalization test
- Add legacy SSH format to parse_ado_remote doc comment

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: jamesadevine

src/configure.rs

@@ -32,11 +32,16 @@ pub struct AdoContext {
 /// - HTTPS: `https://dev.azure.com/{org}/{project}/_git/{repo}`
 /// - HTTPS (legacy): `https://{org}.visualstudio.com/{project}/_git/{repo}`
 /// - SSH: `git@ssh.dev.azure.com:v3/{org}/{project}/{repo}`
+/// - SSH (legacy): `git@vs-ssh.visualstudio.com:v3/{org}/{project}/{repo}`
 pub fn parse_ado_remote(remote_url: &str) -> Result<AdoContext> {
     let url = remote_url.trim();
 
     // SSH format: git@ssh.dev.azure.com:v3/{org}/{project}/{repo}
-    if let Some(rest) = url.strip_prefix("git@ssh.dev.azure.com:v3/") {
+    // Also handles legacy: git@vs-ssh.visualstudio.com:v3/{org}/{project}/{repo}
+    if let Some(rest) = url
+        .strip_prefix("git@ssh.dev.azure.com:v3/")
+        .or_else(|| url.strip_prefix("git@vs-ssh.visualstudio.com:v3/"))
+    {
         let parts: Vec<&str> = rest.splitn(3, '/').collect();
         if parts.len() >= 3 {
             let repo_name = parts[2].trim_end_matches(".git");
@@ -265,6 +270,15 @@ fn fuzzy_match_by_name(agent_name: &str, definitions: &[DefinitionSummary]) -> F
     }
 }
 
+/// Normalize an ADO YAML filename for comparison with local paths.
+///
+/// ADO's Build Definitions API stores `yamlFilename` with a leading `/`
+/// (e.g., `/.azdo/pipelines/agent.yml`) and may use backslashes on Windows.
+/// This strips the leading `/` and normalizes separators to forward slashes.
+fn normalize_ado_yaml_path(path: &str) -> String {
+    path.replace('\\', "/").trim_start_matches('/').to_string()
+}
+
 /// Match detected pipeline YAML files to ADO pipeline definitions.
 ///
 /// Strategy:
@@ -290,12 +304,14 @@ async fn match_definitions(
         let yaml_path_str = pipeline.yaml_path.to_string_lossy();
         let yaml_path_normalized = yaml_path_str.replace('\\', "/");
 
-        // Strategy 1: Match by YAML filename in the definition
+        // Strategy 1: Match by YAML filename in the definition.
+        // ADO stores yamlFilename with a leading '/' (e.g., "/.azdo/pipelines/agent.yml"),
+        // so we strip it before comparing to the locally-detected relative path.
         let path_match = definitions.iter().find(|d| {
             d.process
                 .as_ref()
                 .and_then(|p| p.yaml_filename.as_ref())
-                .is_some_and(|f| f.replace('\\', "/") == yaml_path_normalized)
+                .is_some_and(|f| normalize_ado_yaml_path(f) == yaml_path_normalized)
         });
 
         if let Some(def) = path_match {
@@ -566,7 +582,7 @@ pub async fn run(
     }
     println!();
 
-    // Step 5: Update GITHUB_TOKEN
+    // Step 4: Update GITHUB_TOKEN
     if dry_run {
         println!("Dry run \u{2014} no changes applied.");
         println!(
@@ -650,6 +666,15 @@ mod tests {
         assert_eq!(ctx.repo_name, "myrepo");
     }
 
+    #[test]
+    fn test_parse_ado_remote_legacy_ssh() {
+        let url = "git@vs-ssh.visualstudio.com:v3/myorg/myproject/myrepo";
+        let ctx = parse_ado_remote(url).unwrap();
+        assert_eq!(ctx.org_url, "https://dev.azure.com/myorg");
+        assert_eq!(ctx.project, "myproject");
+        assert_eq!(ctx.repo_name, "myrepo");
+    }
+
     #[test]
     fn test_parse_ado_remote_invalid() {
         assert!(parse_ado_remote("https://github.com/user/repo").is_err());
@@ -666,6 +691,44 @@ mod tests {
         }
     }
 
+    fn make_def_with_yaml(id: u64, name: &str, yaml_filename: &str) -> DefinitionSummary {
+        DefinitionSummary {
+            id,
+            name: name.to_string(),
+            process: Some(ProcessInfo {
+                yaml_filename: Some(yaml_filename.to_string()),
+            }),
+        }
+    }
+
+    // ==================== YAML path matching ====================
+
+    #[test]
+    fn test_yaml_path_match_strips_leading_slash() {
+        // ADO stores yamlFilename with a leading '/'
+        assert_eq!(
+            normalize_ado_yaml_path("/.azdo/pipelines/agent.yml"),
+            ".azdo/pipelines/agent.yml"
+        );
+    }
+
+    #[test]
+    fn test_yaml_path_match_without_leading_slash() {
+        // Some ADO instances may store without leading '/'
+        assert_eq!(
+            normalize_ado_yaml_path(".azdo/pipelines/agent.yml"),
+            ".azdo/pipelines/agent.yml"
+        );
+    }
+
+    #[test]
+    fn test_yaml_path_match_backslash_normalization() {
+        assert_eq!(
+            normalize_ado_yaml_path("\\.azdo\\pipelines\\agent.yml"),
+            ".azdo/pipelines/agent.yml"
+        );
+    }
+
     #[test]
     fn test_fuzzy_match_single_unambiguous() {
         let defs = vec![