Fix UsesLibrary 2-arg ctor losing Required flag

UsesLibraryAttribute(string, bool) puts Required in FixedArguments[1].
ParseNameAndProperties now stores additional bool ctor args in props
so CreateUsesLibraryInfo can read them.

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

Author: simonrozsival

external/Java.Interop

@@ -406,6 +406,12 @@ internal void ScanAssemblyAttributes (AssemblyManifestInfo info)
 		if (value.FixedArguments.Length > 0 && value.FixedArguments [0].Value is string ctorName) {
 			name = ctorName;
 		}
+		// Handle 2-arg ctors like UsesLibrary(string, bool) — store extra ctor args in props
+		for (int i = 1; i < value.FixedArguments.Length; i++) {
+			if (value.FixedArguments [i].Value is bool boolVal) {
+				props ["Required"] = boolVal;
+			}
+		}
 		foreach (var named in value.NamedArguments) {
 			if (named.Name == "Name" && named.Value is string n) {
 				name = n;

src/Microsoft.Android.Sdk.TrimmableTypeMap/Scanner/AssemblyIndex.cs

@@ -46,6 +46,16 @@ public void UsesLibrary_ConstructorArg ()
 		var info = ScanAssemblyManifestInfo ();
 		var lib = info.UsesLibraries.FirstOrDefault (l => l.Name == "org.apache.http.legacy");
 		Assert.NotNull (lib);
+		Assert.True (lib.Required);
+	}
+
+	[Fact]
+	public void UsesLibrary_TwoArgConstructor ()
+	{
+		var info = ScanAssemblyManifestInfo ();
+		var lib = info.UsesLibraries.FirstOrDefault (l => l.Name == "com.example.optional");
+		Assert.NotNull (lib);
+		Assert.False (lib.Required);
 	}
 
 	[Fact]

tests/Microsoft.Android.Sdk.TrimmableTypeMap.Tests/Scanner/AssemblyAttributeScanningTests.cs

@@ -5,4 +5,5 @@
 [assembly: UsesFeature (GLESVersion = 0x00020000)]
 [assembly: UsesPermission ("android.permission.INTERNET")]
 [assembly: UsesLibrary ("org.apache.http.legacy")]
+[assembly: UsesLibrary ("com.example.optional", false)]
 [assembly: MetaData ("com.example.key", Value = "test-value")]