Errors out when calling create_temporary_module with functions that

[elijahbenizzy]

start with "_"

See #510

This is not backwards compatible, but this is a specific "ad-hoc" util and has an easy fix, so I'm OK not bumping versions accordingly. I would consider this previously undefined behavior, and this will make debugging easier.

• PR has an informative and human-readable title (this will be pulled into the release notes)
• Changes are limited to a single goal (no scope creep)
• Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
• Any change in functionality is tested
• New functions are documented (with a description, list of inputs, and expected output)
• Placeholder code is flagged / future TODOs are captured in comments
• Project documentation has been updated if adding/changing functionality.

[sweep-ai]

Apply Sweep Rules to your PR?

• Apply: All new business logic should have corresponding unit tests.
• Apply: Refactor large functions to be more modular.
• Apply: Add docstrings to all functions and file headers.

[elijahbenizzy]

See #510