[alongd]

Summary

• reject network_id values that escape, collapse, or nest the round namespace
• preserve the legacy network_id=None layout
• add regression coverage for pre-fix path behavior and post-fix refusal

Verification

• Real-function probe passes the escape/collapse/nesting flips and retained run-tree path comparison.
• python -m pytest tests/test_pdep -q is not locally executable: both this branch and official/main stop during collection with the same pandas/pyarrow versus NumPy 2.2.6 ImportError (38 collection errors).
• No cluster submissions or quantum-chemistry runs were performed.

CI should provide the final full-suite result.

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.90%. Comparing base (1dae24c) to head (c0e97de).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #219      +/-   ##
==========================================
+ Coverage   81.89%   81.90%   +0.01%     
==========================================
  Files          78       78              
  Lines       12924    12943      +19     
  Branches     2723     2731       +8     
==========================================
+ Hits        10584    10601      +17     
- Misses       1694     1695       +1     
- Partials      646      647       +1     

Flag	Coverage Δ
unittests	81.90% <ø> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Copilot review overview

🟡 Changes recommended

The new validation error inaccurately describes nested and collapsing paths as outside the project.

Review effort: Balanced
Findings: 1

Open (1)

• Correct inaccurate error text for valid nested and project paths · New

What changed in this PR

Confines PDep round artifacts to safe, single-component network namespaces while preserving legacy paths.

Changes:

• Rejects escaping, nested, or collapsing network IDs.
• Adds regression coverage for valid, invalid, and legacy layouts.

File	Description
t3/​pdep/​pes_rounds.py	Validates network IDs before resolving round paths.
tests/​test_pdep/​test_pes_rounds.py	Tests confinement and legacy behavior.

---

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

[Copilot]

Copilot review overview

🟡 Changes recommended

Some rejected paths receive factually inaccurate diagnostics.

Review effort: Balanced
Findings: 1

Open (1)

• Correctly classify normalized and symlink-resolved nested paths · New

Resolved since last review (1)

• Correct inaccurate error text for valid nested and project paths

[Copilot]

Copilot review overview

🟡 Changes recommended

A direct-child symlink alias can still collapse two network IDs into the same round tree.

Review effort: Balanced
Findings: 1

Open (1)

• Reject direct-child symlink aliases that collapse network namespaces · New

Resolved since last review (1)

• Correctly classify normalized and symlink-resolved nested paths

[Copilot]

Copilot review overview

🟢 Approval recommended

The confinement logic is comprehensive, preserves compatibility, and has targeted regression coverage.

Review effort: Balanced
Findings: None

Resolved since last review (1)

• Reject direct-child symlink aliases that collapse network namespaces