Fix security, complexity, and maintainability issues

[Copilot]

Addresses 8 code quality issues flagged by CodeFactor: 4 security vulnerabilities, 2 complex methods, and 2 ambiguous variable names.

Security Fixes

• Shell injection in visualization.py:392 - Replaced os.system(f"open {f.name}") with subprocess.run(["open", f.name], check=False) to prevent command injection
• Platform logic in visualization.py:393 - Fixed inverted conditional that called os.startfile() on non-Windows platforms
• Pickle deserialization in computeengine.py:1524-1526 - Added security warning to read_dill() docstring about arbitrary code execution risks

Complexity Reduction

• Refactored add_node() (computeengine.py:359-498) - Extracted parameter processing into _process_function_args(), _build_param_map(), and _process_function_kwds() helper methods
• Refactored to_pydot() (visualization.py:452-507) - Decomposed graph construction into _group_nodes_and_edges(), _create_pydot_nodes(), _ensure_parent_subgraphs(), _link_subgraphs(), and _add_edges_to_subgraphs() helper functions

Maintainability

• Renamed ambiguous variable l to node_list in test_computeengine.py (lines 365, 415)

All changes preserve existing behavior and pass 91 existing tests.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• astral.sh
  • Triggering command: /usr/bin/curl curl -LsSf REDACTED (dns block)
• esm.ubuntu.com
  • Triggering command: /usr/lib/apt/methods/https /usr/lib/apt/methods/https (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Fix 4 Security, 2 Complexity, 2 Maintainability issues in multiple files</issue_title>
<issue_description>CodeFactor found multiple issues:

Pickle and modules that wrap it can be unsafe when used to deserialize untrusted data, possible security issue.

• src\loman\computeengine.py:1526
• src\loman\computeengine.py:1524

Complex Method

• src\loman\computeengine.py:359-498
• src\loman\visualization.py:452-507

Ambiguous variable name 'l'.

• tests\test_computeengine.py:415
• tests\test_computeengine.py:365

Starting a process with a shell, possible injection detected, security issue.

• src\loman\visualization.py:392

Starting a process without a shell.

• src\loman\visualization.py:393</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Fix 4 Security, 2 Complexity, 2 Maintainability issues in multiple files #1

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}