Skip to content

fix: LiteralPath for remaining exporter writes - #86

Merged
jacobyoby merged 1 commit into
mainfrom
cursor/literalpath-exports-510f
Sep 17, 2026
Merged

jacobyoby merged 1 commit into
mainfrom
cursor/literalpath-exports-510f

Conversation

@jacobyoby

Copy link
Copy Markdown
Owner

Summary

Completes the literal-path hardening started in #85: exporters that already used -LiteralPath for existence checks now also write with Out-File -LiteralPath, matching Export-DrawIO.

Changes

  • Export-Metadata, Export-Topology, Export-Mermaid, Export-NodeInventoryCsv, Export-NetBox — Out-File -LiteralPath (row export via Export-Csv was already literal).
  • Pester coverage for bracketed paths on metadata/Mermaid/topology/CSV/NetBox outputs, Compare-NetworkScans end-to-end with bracketed inputs/output, and SNMP credential-map reads.
  • CHANGELOG (Unreleased).

Testing

  • ./tests/run-tests.sh → CANNOT-RUN in cloud agent (pwsh not on PATH).
  • CI expected to run full Pester matrix on this PR.
Open in Web Open in Cursor 

Extend bracket-path safety to Export-Metadata, Export-Topology,
Export-Mermaid, Export-NodeInventoryCsv, and Export-NetBox. Add Pester
coverage for bracketed output paths and SNMP credential maps.

Co-authored-by: Jacobrakai <jacobyoby@users.noreply.github.com>
@jacobyoby
jacobyoby marked this pull request as ready for review September 17, 2026 18:52
@jacobyoby
jacobyoby merged commit 221c437 into main Sep 17, 2026
7 checks passed
@cursor
cursor Bot deleted the cursor/literalpath-exports-510f branch September 19, 2026 18:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants