From bbc1c211499ded47bec023d834ae84c7d58c2685 Mon Sep 17 00:00:00 2001
From: Jonathan Peppers <jonathan.peppers@microsoft.com>
Date: Sun, 16 Aug 2026 21:27:18 -0500
Subject: [PATCH 1/2] Document inspector API contracts

Refs #243

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 docs/xml/Android.Views.Inspectors/IntFlagMapping.xml  | 8 ++++----
 docs/xml/Android.Views.Inspectors/WindowInspector.xml | 4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/docs/xml/Android.Views.Inspectors/IntFlagMapping.xml b/docs/xml/Android.Views.Inspectors/IntFlagMapping.xml
index c8362c583..d3688f53d 100644
--- a/docs/xml/Android.Views.Inspectors/IntFlagMapping.xml
+++ b/docs/xml/Android.Views.Inspectors/IntFlagMapping.xml
@@ -106,8 +106,8 @@
       <Docs>
         <param name="mask">The bit mask to compare to and with a value</param>
         <param name="target">The target value to compare the masked value with</param>
-        <param name="name">The name of the flag to include if enabled</param>
-        <summary>Add a mutually exclusive flag to the map.</summary>
+        <param name="name">The non-null name of the flag to include if enabled.</param>
+        <summary>Adds a mutually exclusive flag to the map.</summary>
         <remarks>
           <para>Add a mutually exclusive flag to the map.</para>
           <para>
@@ -151,8 +151,8 @@
       </Parameters>
       <Docs>
         <param name="value">The value of the property</param>
-        <summary>Get a set of the names of enabled flags for a given property value.</summary>
-        <returns>The names of the enabled flags, empty if no flags enabled</returns>
+        <summary>Gets a set of the names of enabled flags for a given property value.</summary>
+        <returns>The non-null names of the enabled flags, or an empty set if no flags are enabled.</returns>
         <remarks>
           <para>Get a set of the names of enabled flags for a given property value.</para>
           <para>
diff --git a/docs/xml/Android.Views.Inspectors/WindowInspector.xml b/docs/xml/Android.Views.Inspectors/WindowInspector.xml
index 9ea7756e5..17095076e 100644
--- a/docs/xml/Android.Views.Inspectors/WindowInspector.xml
+++ b/docs/xml/Android.Views.Inspectors/WindowInspector.xml
@@ -66,8 +66,8 @@
         <ReturnType>System.Collections.Generic.IList&lt;Android.Views.View&gt;</ReturnType>
       </ReturnValue>
       <Docs>
-        <summary>Gets the window views attached to the current process.</summary>
-        <value>the list of all window views attached to the current process</value>
+        <summary>Gets the list of all window views attached to the current process.</summary>
+        <value>The non-null list of all window views attached to the current process.</value>
         <remarks>
           <para>
             <format type="text/html">

From 8271c38d009ed5917cd787125ba90f6262417217 Mon Sep 17 00:00:00 2001
From: Jonathan Peppers <jonathan.peppers@microsoft.com>
Date: Sun, 16 Aug 2026 21:38:05 -0500
Subject: [PATCH 2/2] Address inspector docs review

Refs #243

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
---
 docs/xml/Android.Views.Inspectors/IntFlagMapping.xml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/docs/xml/Android.Views.Inspectors/IntFlagMapping.xml b/docs/xml/Android.Views.Inspectors/IntFlagMapping.xml
index d3688f53d..259f0578d 100644
--- a/docs/xml/Android.Views.Inspectors/IntFlagMapping.xml
+++ b/docs/xml/Android.Views.Inspectors/IntFlagMapping.xml
@@ -106,10 +106,10 @@
       <Docs>
         <param name="mask">The bit mask to compare to and with a value</param>
         <param name="target">The target value to compare the masked value with</param>
-        <param name="name">The non-null name of the flag to include if enabled.</param>
+        <param name="name">The non-null name of the flag to include if enabled</param>
         <summary>Adds a mutually exclusive flag to the map.</summary>
         <remarks>
-          <para>Add a mutually exclusive flag to the map.</para>
+          <para>Adds a mutually exclusive flag to the map.</para>
           <para>
             <format type="text/html">
               <a href="https://developer.android.com/reference/android/view/inspector/IntFlagMapping#add(int,%20int,%20java.lang.String)" title="Reference documentation">Java documentation for <code>android.view.inspector.IntFlagMapping.add(int, int, java.lang.String)</code>.</a>
@@ -154,7 +154,7 @@
         <summary>Gets a set of the names of enabled flags for a given property value.</summary>
         <returns>The non-null names of the enabled flags, or an empty set if no flags are enabled.</returns>
         <remarks>
-          <para>Get a set of the names of enabled flags for a given property value.</para>
+          <para>Gets a set of the names of enabled flags for a given property value.</para>
           <para>
             <format type="text/html">
               <a href="https://developer.android.com/reference/android/view/inspector/IntFlagMapping#get(int)" title="Reference documentation">Java documentation for <code>android.view.inspector.IntFlagMapping.get(int)</code>.</a>