chore: remove unused Path::filled/stroked/unpainted constructors

[detail-app]

Removed the three Path constructors (Path::filled, Path::stroked,
Path::unpainted) from geom/store.rs, along with their sole test consumer
(stroked_paths_preserve_line_pattern) and its now-unused imports in
dialects/artwork/mod.rs.

These constructors emitted degenerate empty-shell paths (Span::EMPTY +
BBox::empty()) and had no production callers anywhere in the workspace —
path construction goes through PathArena::push_path plus struct literals.
Orphaned by the geom-substrate restructure that introduced them, and the
file does not exist in the published crate/pcb-ir/v0.3.82 tag, so they
were never released API.

History

• Last materially changed July 2026 (Preserve arcs + reduce hairline slivers in ipc -> gerber converter #920 @akhilles)

---

Dead Code PRs can be configured here.

---

Note

Low Risk
Dead-code removal with no production callers; behavior of path construction is unchanged.

Overview
Removes dead Path factory helpers filled, stroked, and unpainted from geom/store.rs. Those helpers only built empty placeholder paths (Span::EMPTY, empty bbox) and were not used in production; real paths are created via PathArena::push_path and struct literals.

The artwork dialect test stroked_paths_preserve_line_pattern (the only caller of Path::stroked) is deleted, with related test imports trimmed in dialects/artwork/mod.rs.

^{Reviewed by Cursor Bugbot for commit eb79502. Bugbot is set up for automated code reviews on this repo. Configure here.}

[devin-ai-integration]

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

[cursor]

Approved: this is a small unused-constructor cleanup, and Cursor Bugbot plus Cursor Security Agent completed with no findings that need human review. No reviewers were assigned.

  

_{Sent by Cursor Approval Agent: Pull Request Router and Approver}