👌 IMPROVE: Address code review feedback - add comments for resolve() and test semantics

Agent-Logs-Url: https://github.com/useblocks/sphinx-codelinks/sessions/e9503678-3d44-4fee-9eb5-0d065cf4b5af

Co-authored-by: chrisjsewell <2997570+chrisjsewell@users.noreply.github.com>

Author: Copilot

src/sphinx_codelinks/source_discover/source_discover.py

@@ -70,6 +70,8 @@ def _discover(self) -> list[Path]:
                 continue
             if self.file_types and filepath.suffix.lower() not in self.file_types:
                 continue
+            # resolve() produces canonical absolute paths; follow_links only
+            # controls whether the walker descends into symlinked directories
             discovered_files.append(filepath.resolve())
 
         sorted_filepaths = sorted(

tests/test_source_discover.py

@@ -130,6 +130,9 @@ def test_schema_positive(config):
                 "exclude": ["charge/*.cpp"],
                 "include": ["**/*.cpp"],
             },
+            # With ignore-python, include patterns whitelist files (overriding
+            # gitignore) and exclude patterns are applied after, so both
+            # charge/*.cpp files are excluded resulting in 2 instead of 4.
             2,
             "",
         ),