Add AvdManagerRunner for avdmanager CLI operations by rmarinho · Pull Request #282 · dotnet/android-tools

rmarinho · 2026-02-23T17:39:13Z

Summary

Wraps avdmanager CLI operations for programmatic AVD management. Addresses #277.

Changes

New constructor overload with JDK path

public AvdManagerRunner (Func<string?> getSdkPath, Func<string?>? getJdkPath)

Sets JAVA_HOME and ANDROID_HOME on all child process ProcessStartInfo via ConfigureEnvironment().
The existing 1-arg constructor remains backward compatible.

New: CreateAvdAsync

Duplicate detection: checks ListAvdsAsync first, returns existing AVD if name matches
Orphaned AVD handling: detects .avd directory without .ini registration and auto-applies --force
stdin handling: answers "no" to avdmanager hardware profile customization prompt
Error reporting: captures stdout/stderr and throws InvalidOperationException on failure

New: ParseAvdListOutput (internal static)

Extracted from ListAvdsAsync for testability.

Updated: ListAvdsAsync / DeleteAvdAsync

Now call ConfigureEnvironment(psi) before ProcessUtils.StartProcess to pass JAVA_HOME.

Why

avdmanager.bat requires JAVA_HOME to locate the JVM. Without it, operations fail silently on Windows when the system JAVA_HOME is stale or missing.

Copilot

Pull request overview

Adds a new “runner” layer to Xamarin.Android.Tools.AndroidSdk to execute Android SDK command-line tools programmatically, with an initial AvdManagerRunner implementation for creating/listing/deleting AVDs.

Changes:

Introduces AndroidToolRunner for running SDK tools with timeout/stdout/stderr capture (sync + async) and a background-start helper.
Adds AvdManagerRunner to locate and invoke avdmanager, parse list avd output, and enrich results from config.ini.
Adds supporting models (ToolRunnerResult, AvdInfo) and environment setup utilities (AndroidEnvironmentHelper).

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
src/Xamarin.Android.Tools.AndroidSdk/Runners/AvdManagerRunner.cs	Adds an `avdmanager` wrapper with list/create/delete and output parsing/enrichment.
src/Xamarin.Android.Tools.AndroidSdk/Runners/AndroidToolRunner.cs	Adds process execution utilities (sync/async + background start).
src/Xamarin.Android.Tools.AndroidSdk/Runners/AndroidEnvironmentHelper.cs	Provides JAVA_HOME / ANDROID_HOME setup plus ABI/API/tag mapping helpers.
src/Xamarin.Android.Tools.AndroidSdk/Models/ToolRunnerResult.cs	Introduces a result type for tool execution (typed + untyped).
src/Xamarin.Android.Tools.AndroidSdk/Models/AvdInfo.cs	Adds an AVD info model used by `AvdManagerRunner`.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

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

src/Xamarin.Android.Tools.AndroidSdk/Models/AvdInfo.cs

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

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

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

jonathanpeppers · 2026-02-23T18:08:51Z

I'd like to get the System.Diagnostics.Process code unified like mentioned here:

Add base Android tool runner infrastructure #281 (comment)

Addresses PR #282 feedback to use existing ProcessUtils instead of the removed AndroidToolRunner. Simplifies API: - Methods now throw InvalidOperationException on failure instead of returning result types with error states - Uses ProcessUtils.RunToolAsync() for all tool invocations - Simplified AvdInfo model - Removed complex ToolRunnerResult<T> wrapper types Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 12 comments.

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

src/Xamarin.Android.Tools.AndroidSdk/Models/AvdInfo.cs

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

Copilot

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

Copilot · 2026-03-03T16:27:46Z

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

+		// Detect orphaned AVD directory (folder exists without .ini registration).
+		var avdDir = Path.Combine (
+			Environment.GetFolderPath (Environment.SpecialFolder.UserProfile),
+			".android", "avd", $"{name}.avd");
+		if (Directory.Exists (avdDir))
+			force = true;


CreateAvdAsync hard-codes the AVD directory as $HOME/.android/avd/<name>.avd for orphan detection and for the returned AvdInfo.Path. avdmanager can place AVDs elsewhere when ANDROID_USER_HOME/ANDROID_AVD_HOME are set, so this can incorrectly force --force and/or return the wrong path. Consider deriving the AVD location from the tool output (or re-list after creation and return the matching entry) and basing orphan detection on the same resolved location/env vars.

Copilot · 2026-03-03T16:27:47Z

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

+	public async Task DeleteAvdAsync (string name, CancellationToken cancellationToken = default)
+	{
+		var avdManagerPath = RequireAvdManagerPath ();
+
+		using var stderr = new StringWriter ();
+		var psi = ProcessUtils.CreateProcessStartInfo (avdManagerPath, "delete", "avd", "--name", name);
+		ConfigureEnvironment (psi);


DeleteAvdAsync doesn't validate name (null/empty/whitespace). Passing an empty name will invoke avdmanager with an invalid --name value and can lead to confusing errors; please add the same argument validation used in CreateAvdAsync.

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

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

- Fix argument validation: CreateAvdAsync/DeleteAvdAsync now throw ArgumentException for empty strings, ArgumentNullException for null, and validate before RequireAvdManagerPath() - Make AndroidEnvironmentHelper internal, remove unused GetToolEnvironment - Add ANDROID_USER_HOME to ConfigureEnvironment for consistent AVD location across tools (matches SdkManager behavior) - Add AndroidUserHome constant to EnvironmentVariableNames - Fix hard-coded AVD directory: use GetAvdRootDirectory() that respects ANDROID_AVD_HOME and ANDROID_USER_HOME env vars - Re-list after CreateAvdAsync to get actual path from avdmanager - Add 5 new validation tests for Create/DeleteAvdAsync Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

jonathanpeppers

🤖 AI Review Summary

Found 5 issues: 1 security concern, 1 error handling gap, 1 consistency gap, 1 missing constant, 1 convention violation.

Security / API design: additionalArgs as single string risks injection and breaks multi-arg callers (EmulatorRunner.cs:57)
Error handling: Bare catch block silently swallows exceptions (AdbRunner.cs:106)
Consistency: Exit code not checked in ListDevicesAsync / ListAvdNamesAsync while other methods do (AdbRunner.cs:71)
Naming: Raw "ANDROID_AVD_HOME" string instead of EnvironmentVariableNames constant (AvdManagerRunner.cs:212)
Code organization: Three types in one file (AdbDeviceInfo.cs)

👍 Excellent test coverage with thorough parsing tests. Good CancellationToken propagation throughout, correct OperationCanceledException rethrow. AndroidEnvironmentHelper is a clean factoring of shared env setup. Orphaned AVD detection in CreateAvdAsync is a smart touch.

Review generated by android-tools-reviewer from review guidelines by @jonathanpeppers.

jonathanpeppers · 2026-03-04T20:32:01Z

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

+		AndroidEnvironmentHelper.ConfigureEnvironment (psi, getSdkPath (), getJdkPath?.Invoke ());
+	}
+
+	public Process StartAvd (string avdName, bool coldBoot = false, string? additionalArgs = null)


🤖 Security / API design — string? additionalArgs is a command injection risk and a correctness bug. If a caller passes "-no-window -no-audio", it becomes a single argument to the emulator (not two), silently breaking. Change to IEnumerable<string>? additionalArgs and use args.AddRange(additionalArgs) so each flag is a separate argument handled safely by ProcessUtils.CreateProcessStartInfo's ArgumentList.

Rule: Structured args, not string interpolation (Postmortem #50)

jonathanpeppers · 2026-03-04T20:32:01Z

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

+			}
+		} catch (OperationCanceledException) {
+			throw;
+		} catch {


🤖 Error handling — This bare catch block swallows all exceptions without logging. Even if failures are expected (emulator may not support the command), the exception should be captured and logged for diagnostics. At minimum: catch (Exception ex) { /* log ex */ }. Silent swallowing makes debugging null AVD names nearly impossible.

Rule: No empty catch blocks (Postmortem #11)

jonathanpeppers · 2026-03-04T20:32:01Z

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

+		var adb = RequireAdb ();
+		using var stdout = new StringWriter ();
+		var psi = CreateAdbProcess (adb, "devices", "-l");
+		await ProcessUtils.StartProcess (psi, stdout, null, cancellationToken).ConfigureAwait (false);


🤖 Consistency — ListDevicesAsync discards the exit code from ProcessUtils.StartProcess, but AvdManagerRunner.ListAvdsAsync and DeleteAvdAsync both check it and throw on failure. Same gap in EmulatorRunner.ListAvdNamesAsync. If adb devices fails (e.g., daemon not running), this silently returns an empty list instead of surfacing the error.

Rule: Check exit codes consistently (Postmortem #48)

jonathanpeppers · 2026-03-04T20:32:01Z

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

+	{
+		// ANDROID_AVD_HOME takes highest priority
+		var avdHome = Environment.GetEnvironmentVariable ("ANDROID_AVD_HOME");
+		if (!string.IsNullOrEmpty (avdHome))


🤖 Naming — Raw "ANDROID_AVD_HOME" string literal. The PR already adds EnvironmentVariableNames.AndroidUserHome — add an AndroidAvdHome constant too and use it here. Typos in env var names cause silent, hard-to-debug failures.

Rule: Environment variable constants (Postmortem #18)

jonathanpeppers · 2026-03-04T20:32:01Z

src/Xamarin.Android.Tools.AndroidSdk/Models/AdbDeviceInfo.cs

+/// <summary>
+/// Represents the type of an Android device.
+/// </summary>
+public enum AdbDeviceType


🤖 Code organization — Three public types (AdbDeviceType, AdbDeviceStatus, AdbDeviceInfo) in one file. Per repo conventions, each public type should be in its own .cs file named after the type: AdbDeviceType.cs, AdbDeviceStatus.cs, AdbDeviceInfo.cs.

Rule: One type per file (Postmortem #9)

- Fix argument validation: CreateAvdAsync/DeleteAvdAsync now throw ArgumentException for empty strings, ArgumentNullException for null, and validate before RequireAvdManagerPath() - Make AndroidEnvironmentHelper internal, remove unused GetToolEnvironment - Add ANDROID_USER_HOME to ConfigureEnvironment for consistent AVD location across tools (matches SdkManager behavior) - Add AndroidUserHome constant to EnvironmentVariableNames - Fix hard-coded AVD directory: use GetAvdRootDirectory() that respects ANDROID_AVD_HOME and ANDROID_USER_HOME env vars - Re-list after CreateAvdAsync to get actual path from avdmanager - Add 5 new validation tests for Create/DeleteAvdAsync Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…ed args - AdbRunner: check exit codes consistently in ListDevicesAsync, WaitForDeviceAsync, StopEmulatorAsync - AdbRunner: bare catch {} → catch (Exception) in GetEmulatorAvdNameAsync - AdbDeviceInfo.cs: split 3 public types into separate files (AdbDeviceType.cs, AdbDeviceStatus.cs) - EnvironmentVariableNames: add AndroidAvdHome constant, use in AvdManagerRunner - EmulatorRunner.StartAvd: additionalArgs string? → IEnumerable<string>? for proper arg separation - EmulatorRunner.StartAvd: set RedirectStandardOutput/Error=false to prevent pipe deadlock Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- GetToolEnvironment(): Set ANDROID_HOME, JAVA_HOME, and PATH Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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

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

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

… add tests - Replace manual ProcessStartInfo with ProcessUtils.CreateProcessStartInfo - Use AndroidEnvironmentHelper.ConfigureEnvironment instead of inline env setup - Extract ParseDeviceListOutput as internal static for testability - Return IReadOnlyList<AdbDeviceInfo> from ListDevicesAsync - Add AdbRunnerTests (7 tests): parsing, path discovery, null/missing sdk Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Port the adb device parsing, display name formatting, status mapping, and emulator merging logic from dotnet/android's GetAvailableAndroidDevices MSBuild task into the shared android-tools library. Changes: - AdbDeviceInfo: Add Type, Status, Description, AvdName, Product, TransportId fields using AdbDeviceType/AdbDeviceStatus enums - AdbRunner.ParseAdbDevicesOutput: Regex-based parser matching dotnet/android (handles device/offline/unauthorized/no permissions states) - AdbRunner.BuildDeviceDescription: Priority order AVD name > model > product > device > serial, with underscore-to-space cleanup - AdbRunner.FormatDisplayName: Title case + API capitalization for AVD names - AdbRunner.MapAdbStateToStatus: Maps adb states to AdbDeviceStatus enum - AdbRunner.MergeDevicesAndEmulators: Merges running + non-running emulators, deduplicates by AVD name (case-insensitive), sorts online first - AdbRunner.GetEmulatorAvdNameAsync: Queries AVD name via adb emu avd name - AdbRunnerTests: 33 tests ported from dotnet/android test cases Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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

The bare catch block was swallowing cancellation tokens, causing cancelled operations to silently continue spawning adb processes instead of propagating the cancellation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- StartAvd(): Start an emulator for an AVD - ListAvdNamesAsync(): List installed AVD names Minimal implementation without external dependencies. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Add XML documentation to all public members - Use 'is not null' instead of '!= null' - Improve code formatting - Remove unused variable assignment Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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

- Add optional Func<string?> getJdkPath constructor parameter - ConfigureEnvironment() sets JAVA_HOME and ANDROID_HOME on all ProcessStartInfo - Applied to StartAvd and ListAvdNamesAsync - Backward compatible: existing 1-arg constructor still works Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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

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

… add tests - Replace manual ProcessStartInfo with ProcessUtils.CreateProcessStartInfo - Use AndroidEnvironmentHelper.ConfigureEnvironment instead of duplicated method - Extract ParseListAvdsOutput as internal static for testability - Return IReadOnlyList<string> from ListAvdNamesAsync - Add RequireEmulatorPath helper - Add EmulatorRunnerTests (7 tests): parsing, path discovery, null/missing sdk Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- ListAvdsAsync(): List configured AVDs - DeleteAvdAsync(): Delete an AVD Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Add XML documentation to all public members - Split AvdInfo into its own file (one type per file) - Use 'is not null' instead of '!= null' - Improve code formatting Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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

- Add optional Func<string?> getJdkPath constructor parameter - ConfigureEnvironment() sets JAVA_HOME and ANDROID_HOME on all ProcessStartInfo - Add CreateAvdAsync with duplicate detection, orphaned AVD directory handling, stdin 'no' for hardware profile prompt - Extract ParseAvdListOutput as internal static for testability - Backward compatible: existing 1-arg constructor still works Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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

- Add 'using' to StringWriter instances in ListAvdsAsync and CreateAvdAsync to ensure proper disposal - Add AvdManagerRunnerTests with 5 focused tests for ParseAvdListOutput: multiple AVDs, Windows newlines, empty output, no AVDs, missing device Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Convert AvdManagerRunner.cs and AvdInfo.cs to file-scoped namespaces per repo conventions for new files - Remove accidentally tracked nupkg (already in .gitignore) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

…ironmentVariableNames, exit code checks - AvdManagerPath: enumerate versioned cmdline-tools dirs descending (mirrors SdkManager.FindSdkManagerPath pattern), then latest, then legacy tools/bin - Use ProcessUtils.CreateProcessStartInfo with separate args instead of building Arguments string manually (all 3 methods) - Use EnvironmentVariableNames.AndroidHome/.JavaHome constants instead of hard-coded strings - Check exit codes in ListAvdsAsync and DeleteAvdAsync, throw on failure - Return IReadOnlyList<AvdInfo> from ListAvdsAsync - Set Path on AvdInfo returned from CreateAvdAsync for consistency - Extract RequireAvdManagerPath helper Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Delegate to AndroidEnvironmentHelper.ConfigureEnvironment instead of inline setup - Add 5 path discovery tests: versioned dir, higher version priority, latest fallback, null/missing sdk Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Fix argument validation: CreateAvdAsync/DeleteAvdAsync now throw ArgumentException for empty strings, ArgumentNullException for null, and validate before RequireAvdManagerPath() - Make AndroidEnvironmentHelper internal, remove unused GetToolEnvironment - Add ANDROID_USER_HOME to ConfigureEnvironment for consistent AVD location across tools (matches SdkManager behavior) - Add AndroidUserHome constant to EnvironmentVariableNames - Fix hard-coded AVD directory: use GetAvdRootDirectory() that respects ANDROID_AVD_HOME and ANDROID_USER_HOME env vars - Re-list after CreateAvdAsync to get actual path from avdmanager - Add 5 new validation tests for Create/DeleteAvdAsync Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

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

…eTool - ProcessUtils.ThrowIfFailed: consistent exit code checking with stderr/stdout context - ProcessUtils.ValidateNotNullOrEmpty: reusable null/empty argument validation - ProcessUtils.FindCmdlineTool: version-aware cmdline-tools directory search with legacy fallback Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Replace cmdline-tools path lookup with ProcessUtils.FindCmdlineTool (~20 lines removed) - Replace duplicated null/empty validation with ProcessUtils.ValidateNotNullOrEmpty - Replace inline exit code checks with ProcessUtils.ThrowIfFailed Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

- Fix argument validation: CreateAvdAsync/DeleteAvdAsync now throw ArgumentException for empty strings, ArgumentNullException for null, and validate before RequireAvdManagerPath() - Make AndroidEnvironmentHelper internal, remove unused GetToolEnvironment - Add ANDROID_USER_HOME to ConfigureEnvironment for consistent AVD location across tools (matches SdkManager behavior) - Add AndroidUserHome constant to EnvironmentVariableNames - Fix hard-coded AVD directory: use GetAvdRootDirectory() that respects ANDROID_AVD_HOME and ANDROID_USER_HOME env vars - Re-list after CreateAvdAsync to get actual path from avdmanager - Add 5 new validation tests for Create/DeleteAvdAsync Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings February 23, 2026 17:39

Copilot started reviewing on behalf of rmarinho February 23, 2026 17:39 View session

Copilot AI reviewed Feb 23, 2026

View reviewed changes

rmarinho added the copilot `copilot-cli` or other AIs were used to author this label Feb 23, 2026

rmarinho force-pushed the feature/avdmanager-runner branch 6 times, most recently from b286a7a to 6e09e61 Compare March 3, 2026 13:16

rmarinho requested review from Copilot and jonathanpeppers March 3, 2026 13:56

Copilot started reviewing on behalf of rmarinho March 3, 2026 13:57 View session

Copilot AI reviewed Mar 3, 2026

View reviewed changes

rmarinho force-pushed the feature/avdmanager-runner branch from be6eb7f to eb4b66a Compare March 3, 2026 14:36

rmarinho requested a review from Copilot March 3, 2026 16:23

Copilot started reviewing on behalf of rmarinho March 3, 2026 16:23 View session

Copilot AI reviewed Mar 3, 2026

View reviewed changes

rmarinho force-pushed the feature/avdmanager-runner branch from 4c8ce13 to 3a788bb Compare March 3, 2026 18:23

jonathanpeppers reviewed Mar 4, 2026

View reviewed changes

rmarinho force-pushed the feature/avdmanager-runner branch from 7a84734 to 37c4ffe Compare March 5, 2026 12:53

rmarinho and others added 2 commits March 5, 2026 15:37

Add AndroidEnvironmentHelper for SDK tool environment setup

10da27d

- GetToolEnvironment(): Set ANDROID_HOME, JAVA_HOME, and PATH Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Convert AndroidEnvironmentHelper to file-scoped namespace, fix indent…

d1e0381

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

rmarinho and others added 28 commits March 5, 2026 15:37

Make AdbPath public for consumer access

8b0c25d

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

Convert AdbRunner and AdbDeviceInfo to file-scoped namespaces, add us…

1cab277

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

Make parsing/formatting methods public for dotnet/android consumption

2c8d29c

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

Add EmulatorRunner for emulator CLI operations

a1df9f9

- StartAvd(): Start an emulator for an AVD - ListAvdNamesAsync(): List installed AVD names Minimal implementation without external dependencies. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Refactor EmulatorRunner per copilot instructions

852b1dc

- Add XML documentation to all public members - Use 'is not null' instead of '!= null' - Improve code formatting - Remove unused variable assignment Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fix indentation: add tab indentation per Mono coding guidelines

f6c49de

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

Trim verbose XML docs, make EmulatorPath internal

fe493f1

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

Make EmulatorPath public for consumer access

3285c0a

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

Convert EmulatorRunner to file-scoped namespace, add using on StringW…

19e5f46

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

Add AvdManagerRunner for avdmanager CLI operations

5ecd99b

- ListAvdsAsync(): List configured AVDs - DeleteAvdAsync(): Delete an AVD Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Refactor AvdManagerRunner per copilot instructions

62a4858

- Add XML documentation to all public members - Split AvdInfo into its own file (one type per file) - Use 'is not null' instead of '!= null' - Improve code formatting Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Fix indentation: add tab indentation per Mono coding guidelines

77e8101

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

Trim verbose XML docs, make AvdManagerPath internal

a7f2865

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

Make AvdManagerPath public for consumer access

def84fd

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

Remove unused using System.Collections.Generic

e818313

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

rmarinho force-pushed the feature/avdmanager-runner branch from 797964e to 271b5c1 Compare March 5, 2026 15:38

Conversation

rmarinho commented Feb 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

New constructor overload with JDK path

New: CreateAvdAsync

New: ParseAvdListOutput (internal static)

Updated: ListAvdsAsync / DeleteAvdAsync

Why

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jonathanpeppers commented Feb 23, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jonathanpeppers left a comment

Choose a reason for hiding this comment

🤖 AI Review Summary

Uh oh!

jonathanpeppers Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

jonathanpeppers Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

jonathanpeppers Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

jonathanpeppers Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

jonathanpeppers Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

rmarinho commented Feb 23, 2026 •

edited

Loading