• Explore repository structure and understand current RemoveDirFixed implementation
• Identify current retry logic using Files.GetFileWriteRetryAttempts() (10 retries) and Files.GetFileWriteRetryDelay() (1000ms)
• Locate usage patterns in Xamarin.Android.Common.targets
• Review existing test structure for RemoveDirFixed
• Add RetryAttempts and RetryDelayMs properties to RemoveDirFixed task
• Modify retry logic to use new properties with backward compatibility
• Update temp directory cleanup calls to use RetryAttempts="3"
• Add tests for new retry configuration functionality
• Build and test changes
• Verify implementation addresses the performance issue

✅ Implementation Complete

Problem Solved: The RemoveDirFixed MSBuild task was taking 8+ seconds due to excessive retry attempts (10 retries × 1000ms = up to 10 seconds) when cleaning temporary directories.

Solution Implemented:

• Added configurable RetryAttempts and RetryDelayMs properties to RemoveDirFixed task
• Reduced retry attempts to 3 for temp directory cleanup (max ~3 seconds vs 10+ seconds)
• Maintained full backward compatibility for all other usage scenarios

Files Modified:

1. RemoveDirFixed.cs: Added retry configuration properties with backward compatibility
2. Xamarin.Android.Common.targets: Updated 2 temp cleanup calls to use RetryAttempts="3"
3. RemoveDirTests.cs: Added comprehensive tests for new functionality

Performance Impact: Reduces temp directory cleanup time from 8+ seconds to ~3 seconds maximum, directly addressing the reported issue while preserving robust retry behavior where actually needed.

Testing: All changes compile successfully and maintain existing functionality. New tests verify both custom retry behavior and backward compatibility.

Fixes #10338.

Commit message:

Fixes: https://github.com/dotnet/android/issues/10338

The `RemoveDirFixed` MSBuild task could take 8+ seconds due to
excessive retry attempts (10 retries × 1000ms = up to 10 seconds) when
cleaning temporary directories.

**Solution Implemented**:

- Added configurable `RetryAttempts` and `RetryDelayMs` properties to `RemoveDirFixed` task
- Reduced retry attempts to 3 for temp directory cleanup (max ~3 seconds vs 10+ seconds)
- Maintained full backward compatibility for all other usage scenarios

**Files Modified**:

1. **RemoveDirFixed.cs**: Added retry configuration properties with backward compatibility
2. **Xamarin.Android.Common.targets**: Updated 2 temp cleanup calls to use `RetryAttempts="3"`
3. **RemoveDirTests.cs**: Added comprehensive tests for new functionality

**Performance Impact**: Reduces temp directory cleanup time from 8+
seconds to ~3 seconds maximum, directly addressing the reported issue
while preserving robust retry behavior where actually needed.

**Testing**: All changes compile successfully and maintain existing
functionality. New tests verify both custom retry behavior and
backward compatibility.

Co-authored-by: Jonathan Peppers <[email protected]>

💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.