Fix regex to use explicit adb states and support tab separators

- Replace \s{2,} with \s+ to handle tab-separated adb output
- Use explicit state list (device|offline|unauthorized|etc.) instead
  of \S+ to prevent false positives from non-device lines
- Add ParseAdbDevicesOutput_TabSeparator test

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

Author: rmarinho

src/Xamarin.Android.Tools.AndroidSdk/Runners/AdbRunner.cs

@@ -24,10 +24,11 @@ public class AdbRunner
 
 	// Pattern to match device lines: <serial> <state> [key:value ...]
 	// Requires 2+ spaces between serial and state (adb pads serials).
-	// Matches known multi-word states (e.g. "no permissions") and any single-word state
-	// so unknown states (recovery, sideload, etc.) are not silently dropped.
+	// Matches known adb device states. Uses \s+ to handle both space and tab separators.
+	// Explicit state list prevents false positives from non-device lines.
 	static readonly Regex AdbDevicesRegex = new Regex (
-		@"^([^\s]+)\s{2,}(no permissions|\S+)\s*(.*)$", RegexOptions.Compiled);
+		@"^([^\s]+)\s+(device|offline|unauthorized|authorizing|no permissions|recovery|sideload|bootloader|connecting|host)\s*(.*)$",
+		RegexOptions.Compiled | RegexOptions.IgnoreCase);
 	static readonly Regex ApiRegex = new Regex (@"\bApi\b", RegexOptions.Compiled);
 
 	public AdbRunner (Func<string?> getSdkPath)

tests/Xamarin.Android.Tools.AndroidSdk-Tests/AdbRunnerTests.cs

@@ -215,6 +215,21 @@ public void ParseAdbDevicesOutput_WindowsNewlines ()
 		Assert.IsTrue (devices [0].IsEmulator);
 	}
 
+	[Test]
+	public void ParseAdbDevicesOutput_TabSeparator ()
+	{
+		var output =
+			"List of devices attached\n" +
+			"emulator-5554\tdevice\n" +
+			"R5CR10YZQPJ\tdevice\n";
+
+		var devices = AdbRunner.ParseAdbDevicesOutput (output);
+
+		Assert.AreEqual (2, devices.Count);
+		Assert.AreEqual ("emulator-5554", devices [0].Serial);
+		Assert.AreEqual ("R5CR10YZQPJ", devices [1].Serial);
+	}
+
 	[Test]
 	public void ParseAdbDevicesOutput_IpPortDevice ()
 	{