Pythonrc: Don't pollute PyREPL with variables

[Eduardo Villalpando Mello (edvilme)]

Fixes #26147

Before

After

This pull request refactors the initialization logic in python_files/pythonrc.py to avoid polluting the module's global namespace, and adds a corresponding test to ensure this behavior. The main change is wrapping the script's setup code in a private function and then deleting it after execution.

Refactoring and namespace cleanliness:

• Moved all initialization code in pythonrc.py into a private _initialize() function, and called and deleted it at the end of the file to prevent leftover names in the module's global namespace. [1] [2]
• Removed unnecessary blank lines and improved code organization for readability. [1] [2] [3]

Testing:

• Added test_does_not_pollute_namespace in test_shell_integration.py to verify that pythonrc.py does not leave any non-dunder (non-__) names in its global namespace after import.

[Copilot]

🟢 Approval recommended

The focused refactor preserves behavior, adds appropriate regression coverage, and has no identified issues.

Pull request overview

Refactors Python REPL startup initialization to prevent internal variables leaking into the interactive namespace.

Changes:

• Encapsulates setup logic in a temporary initializer.
• Updates integration tests and adds namespace-cleanliness coverage.

File summaries

File	Description
python_files/pythonrc.py	Isolates and removes initialization symbols.
python_files/tests/test_shell_integration.py	Tests installed hooks and namespace cleanliness.

Review details

• Files reviewed: 2/2 changed files
• Comments generated: 0
• Review effort level: Balanced

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[Bill Schnurr (bschnurr)]

🔒 Automated review in progress — Bill Schnurr (@bschnurr) is auto-reviewing this PR.

[Bill Schnurr (bschnurr)]

Approved via Review Center.

[Rich Chiodo (rchiodo)]

Approved via Review Center.