Update BuildSearchDirectories documentation to reflect removal of sibling check

[Copilot]

The XML documentation for BuildSearchDirectories referenced a sibling check fallback that no longer exists. The AddSiblingEntityDirectories method was removed in commit 07ffa2f, but the documentation wasn't updated.

Changes:

• Removed outdated reference to "The sibling check is kept as a fallback for non-nested structures"
• Removed reference to removed entityNamespaces parameter in summary
• Clarified that only context directory and parent directory are searched, with parent scan recursively including siblings

/// <remarks>
/// This method starts with the context directory and then its parent directory.
/// Since the parent directory scan is recursive, it will naturally include the context directory
/// and all siblings through the recursive search.
/// </remarks>

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[Copilot]

Pull request overview

This pull request updates the XML documentation for the BuildSearchDirectories method to reflect the removal of the AddSiblingEntityDirectories method and associated sibling check logic. The documentation had become outdated after commit 07ffa2f removed the sibling check functionality.

Changes:

• Removed outdated reference to "sibling check fallback" that no longer exists
• Removed reference to non-existent entityNamespaces parameter in the summary
• Clarified that the method searches the context directory and parent directory, with recursive parent scanning naturally including siblings

Comments suppressed due to low confidence (2)

src/ProjGraph.Lib/Services/EfAnalysis/EntityFileDiscovery.cs:205

• This foreach loop immediately maps its iteration variable to another variable - consider mapping the sequence explicitly using '.Select(...)'.

            foreach (var csFile in Directory.EnumerateFiles(searchDir, EfAnalysisConstants.FilePatterns.CSharpFiles,
                         options))
            {
                var fullPath = Path.GetFullPath(csFile);
                if (fullPath.Equals(normalizedContextPath, StringComparison.OrdinalIgnoreCase))
                {
                    continue;
                }

                // Skip common non-source directories that can be large
                var pathSegments = fullPath.Split(Path.DirectorySeparatorChar, Path.AltDirectorySeparatorChar);
                if (pathSegments.Any(s => s is "bin" or "obj" or ".git" or "node_modules"))
                {
                    continue;
                }

                await ProcessSourceFileAsync(fullPath, entityTypeNames, entityFiles);
            }

src/ProjGraph.Lib/Services/EfAnalysis/EntityFileDiscovery.cs:276

• This foreach loop immediately maps its iteration variable to another variable - consider mapping the sequence explicitly using '.Select(...)'.

        foreach (var entityFile in entityFiles.Values.Distinct())
        {
            var entityCode = await File.ReadAllTextAsync(entityFile);
            var entityTree = CSharpSyntaxTree.ParseText(entityCode);
            var entityRoot = await entityTree.GetRootAsync();

            ExtractBaseClassNamesFromSyntax(entityRoot, baseClassNames);
        }

---

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