feat: ✨ Add ProfileLoader.required_module_names method

[remimd]

No description provided.

[Copilot]

Pull Request Overview

This PR introduces a new method to collect all required module names, including their nested subsets, and adds tests for both default and named scenarios.

• Implemented ProfileLoader.required_module_names with a helper __walk_subsets_for.
• Added two unit tests to verify the returned set of module names.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
tests/loaders/test_profile_loader.py	Added tests for default and named required modules
injection/loaders.py	Implemented required_module_names and subset walker

Comments suppressed due to low confidence (3)

injection/loaders.py:158

• [nitpick] The generator expression reuses name, shadowing the method parameter. Rename the inner loop variable (e.g. for module_name in names) to avoid confusion.

        subsets = (self.__walk_subsets_for(name) for name in names)

tests/loaders/test_profile_loader.py:14

• Calling mod() multiple times produces distinct modules with different names. Assign a single module instance to a variable (e.g. module = mod()), use module.name in the mapping, and then reuse that same variable in the expected set to ensure consistency.

                mod().name: ["a", "b", "c"],

tests/loaders/test_profile_loader.py:21

• Same issue as above: this call to mod().name creates a new instance. Reference the earlier module variable to compare the correct name.

            mod().name,

[Copilot]

[nitpick] Consider adding a docstring for required_module_names to explain its parameters (including name=None) and the structure of its return value, making its behavior clearer to future readers.