Add missing XML documentation for Microsoft.Extensions.FileSystemGlobbing APIs

[Copilot]

Fixes#43908.

• Add <summary> XML documentation to MatcherExtensions class
• Fix FilePatternMatch.GetHashCode()<returns> tag
• Fix constructor summaries to follow guidelines in FilePatternMatch, Matcher, InMemoryDirectoryInfo
• Add protected constructor with doc to FileSystemInfoBase
• Add protected constructor with doc to FileInfoBase
• Add protected constructor with doc to DirectoryInfoBase
• Fix property summaries in FileSystemInfoBase to follow "Gets..." pattern
• Fix constructor summaries in DirectoryInfoWrapper, FileInfoWrapper, PatternMatchingResult to follow guidelines
• Fix FileInfoWrapper.FullName remarks (incorrectly referenced FileSystemInfo.Name, now FileSystemInfo.FullName)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>API documentation debt - Microsoft.Extensions.FileSystemGlobbing</issue_title>
<issue_description>Area owners: @maryamariyan@carlossanlop@jozkee

The APIs in the list below are missing some or all of their documentation. Please add the missing documentation directly in triple slash comments in source. We will make sure it gets ported to dotnet-api-docs after it's merged.

Make sure to follow the documentation guidelines defined in the dotnet-api-docs wiki:
https://github.com/dotnet/dotnet-api-docs/wiki

Also please add me as a PR reviewer.

Community contributions are welcome.

Microsoft.Extensions.FileSystemGlobbing

DocId	Summary	Parameters	TypeParameters	ReturnValue	Source File Path	Docs URL
Microsoft.Extensions.FileSystemGlobbing namespace	Missing	NA	NA	NA	https://github.com/dotnet/dotnet-api-docs/blob/master/xml/ns-Microsoft.Extensions.FileSystemGlobbing.xml	https://review.docs.microsoft.com/en-us/dotnet/api/Microsoft.Extensions.FileSystemGlobbing?branch=master
Microsoft.Extensions.FileSystemGlobbing.Abstractions namespace	Missing	NA	NA	NA	https://github.com/dotnet/dotnet-api-docs/blob/master/xml/ns-Microsoft.Extensions.FileSystemGlobbing.Abstractions.xml	https://review.docs.microsoft.com/en-us/dotnet/api/Microsoft.Extensions.FileSystemGlobbing.Abstractions?branch=master
M:Microsoft.Extensions.FileSystemGlobbing.Abstractions.DirectoryInfoBase.#ctor	Missing	NA	NA	NA	https://github.com/dotnet/dotnet-api-docs/blob/master/xml/Microsoft.Extensions.FileSystemGlobbing.Abstractions/DirectoryInfoBase.xml	https://review.docs.microsoft.com/en-us/dotnet/api/Microsoft.Extensions.FileSystemGlobbing.Abstractions.DirectoryInfoBase.-ctor?branch=master
M:Microsoft.Extensions.FileSystemGlobbing.Abstractions.DirectoryInfoWrapper.EnumerateFileSystemInfos	Missing	NA	NA	Missing	https://github.com/dotnet/dotnet-api-docs/blob/master/xml/Microsoft.Extensions.FileSystemGlobbing.Abstractions/DirectoryInfoWrapper.xml	https://review.docs.microsoft.com/en-us/dotnet/api/Microsoft.Extensions.FileSystemGlobbing.Abstractions.DirectoryInfoWrapper.EnumerateFileSystemInfos?branch=master
M:Microsoft.Extensions.FileSystemGlobbing.Abstractions.DirectoryInfoWrapper.GetFile(System.String)	Missing	Missing	NA	Missing	https://github.com/dotnet/dotnet-api-docs/blob/master/xml/Microsoft.Extensions.FileSystemGlobbing.Abstractions/DirectoryInfoWrapper.xml	https://review.docs.microsoft.com/en-us/dotnet/api/Microsoft.Extensions.FileSystemGlobbing.Abstractions.DirectoryInfoWrapper.GetFile?branch=master
M:Microsoft.Extensions.FileSystemGlobbing.Abstractions.FileInfoBase.#ctor	Missing	NA	NA	NA	https://github.com/dotnet/dotnet-api-docs/blob/master/xml/Microsoft.Extensions.FileSystemGlobbing.Abstractions/FileInfoBase.xml	https://review.docs.microsoft.com/en-us/dotnet/api/Microsoft.Extensions.FileSystemGlobbing.Abstractions.FileInfoBase.-ctor?branch=master
M:Microsoft.Extensions.FileSystemGlobbing.Abstractions.FileSystemInfoBase.#ctor	Missing	NA	NA	NA	https://github.com/dotnet/dotnet-api-docs/blob/master/xml/Microsoft.Extensions.FileSystemGlobbing.Abstractions/FileSystemInfoBase.xml	https://review.docs.microsoft.com/en-us/dotnet/api/Microsoft.Extensions.FileSystemGlobbing.Abstractions.FileSystemInfoBase.-ctor?branch=master
M:Microsoft.Extensions.FileSystemGlobbing.InMemoryDirectoryInfo.EnumerateFileSystemInfos	Missing	NA	NA	Missing	https://github.com/dotnet/dotnet-api-docs/blob/master/xml/Microsoft.Extensions.FileSystemGlobbing/InMemoryDirectoryInfo.xml	https://review.docs.microsoft.com/en-us/dotnet/api/Microsoft.Extensions.FileSystemGlobbing.InMemoryDirectoryInfo.EnumerateFileSystemInfos?branch=master
M:Microsoft.Extensions.FileSystemGlobbing.InMemoryDirectoryInfo.GetDirectory(System.String)	Missing	Missing	NA	Missing	https://github.com/dotnet/dotnet-api-docs/blob/master/xml/Microsoft.Extensions.FileSystemGlobbing/InMemoryDirectoryInfo.xml	https://review.docs.microsoft.com/en-us/dotnet/api/Microsoft.Extensions.FileSystemGlobbing.InMemoryDirectoryInfo.GetDirectory?branch=master
P:Microsoft.Extensions.FileSystemGlobbing.Abstractions.DirectoryInfoWrapper.Name	Missing	NA	NA	NA	https://github.com/dotnet/dotnet-api-docs/blob/master/xml/Microsoft.Extensions.FileSystemGlobbing.Abstractions/DirectoryInfoWrapper.xml	https://review.docs.microsoft.com/en-us/dotnet/api/Microsoft.Extensions.FileSystemGlobbing.Abstractions.DirectoryInfoWrapper.Name?branch=master
P:Microsoft.Extensions.FileSystemGlobbing.InMemoryDirectoryInfo.FullName	Missing	NA	NA	NA	https://github.com/dotnet/dotnet-api-docs/blob/master/xml/Microsoft.Extensions.FileSystemGlobbing/InMemoryDirectoryInfo.xml	https://review.docs.microsoft.com/en-us/dotnet/api/Micro...

• FixesAPI documentation debt - Microsoft.Extensions.FileSystemGlobbing #43908

---

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

[svick]

cc: @carlossanlop, in case you're still interested

[Copilot]

Pull request overview

This PR addresses API documentation debt in Microsoft.Extensions.FileSystemGlobbing by adding/fixing XML documentation comments so the public surface is correctly described in generated docs.

Changes:

• Add missing type/ctor XML docs and standardize constructor summaries across key globbing and abstraction types.
• Fix incorrect/low-quality XML docs (e.g., GetHashCode()<returns>, FileInfoWrapper.FullName remarks, property “Gets…” wording).
• Add documented protected constructors to the abstraction base types to ensure constructors are properly documented.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
src/libraries/Microsoft.Extensions.FileSystemGlobbing/src/PatternMatchingResult.cs	Updates constructor XML summaries (but exposes an existing null-handling bug in the 1-arg ctor).
src/libraries/Microsoft.Extensions.FileSystemGlobbing/src/MatcherExtensions.cs	Adds missing <summary> for the extensions class.
src/libraries/Microsoft.Extensions.FileSystemGlobbing/src/Matcher.cs	Standardizes constructor summaries to “Initializes a new instance…”.
src/libraries/Microsoft.Extensions.FileSystemGlobbing/src/InMemoryDirectoryInfo.cs	Standardizes constructor summaries (incl. case-sensitivity wording).
src/libraries/Microsoft.Extensions.FileSystemGlobbing/src/FilePatternMatch.cs	Fixes ctor summary and improves GetHashCode() documentation.
src/libraries/Microsoft.Extensions.FileSystemGlobbing/src/Abstractions/FileSystemInfoBase.cs	Adds documented protected ctor; updates property summaries to “Gets…”.
src/libraries/Microsoft.Extensions.FileSystemGlobbing/src/Abstractions/FileInfoWrapper.cs	Fixes ctor summary and corrects FullName remarks to reference FileSystemInfo.FullName.
src/libraries/Microsoft.Extensions.FileSystemGlobbing/src/Abstractions/FileInfoBase.cs	Adds documented protected ctor.
src/libraries/Microsoft.Extensions.FileSystemGlobbing/src/Abstractions/DirectoryInfoWrapper.cs	Standardizes ctor summary.
src/libraries/Microsoft.Extensions.FileSystemGlobbing/src/Abstractions/DirectoryInfoBase.cs	Adds documented protected ctor.

Comments suppressed due to low confidence (1)

src/libraries/Microsoft.Extensions.FileSystemGlobbing/src/PatternMatchingResult.cs:22

• PatternMatchingResult(IEnumerable<FilePatternMatch> files) calls files.Any() before any null validation, so passing null will throw a NullReferenceException rather than the ArgumentNullException enforced by the other constructor. Consider validating files before computing hasMatches, and you can also drop the redundant Files = files; assignment since the chained constructor already sets it.

 public PatternMatchingResult(IEnumerable<FilePatternMatch> files)
: this(files, hasMatches: files.Any())
{
Files = files;

[mrek-msft]

Is there any real value? I think we do not need comment that constructor on X creates instance of X and similar.

[svick]

@mrek-msft I think the value is mostly consistency. Empty documentation looks like we forgot to document it.

[svick]

/ba-g This PR is only changing documentation, the iOS test failures are unrelated.