From 5d4b70721483de0a480b107302a3629e5e4440b8 Mon Sep 17 00:00:00 2001
From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com>
Date: Sat, 19 Sep 2026 15:36:47 +0000
Subject: [PATCH] fix review feedback for OTRACK projection and OtrackHit API

Co-authored-by: dichhead <330471626+dichhead@users.noreply.github.com>
(cherry picked from commit 97f94a2bfe6fe58354d6713b78a00de688f12745)
---
 src/app/view/mod.rs | 6 +++++-
 src/snap.rs         | 1 +
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/src/app/view/mod.rs b/src/app/view/mod.rs
index 65a74c0df..dc6679743 100644
--- a/src/app/view/mod.rs
+++ b/src/app/view/mod.rs
@@ -780,7 +780,11 @@ bg={bg_ms:.1}ms n={view_count}"
                         .chain(self.otrack_cross)
                         .filter_map(|(base, _dir)| {
                             let b = ost_project(base, view_rot, eye, ob);
-                            (b.x.is_finite() && a.x.is_finite()).then_some((b, a))
+                            (b.x.is_finite()
+                                && b.y.is_finite()
+                                && a.x.is_finite()
+                                && a.y.is_finite())
+                                .then_some((b, a))
                         })
                         .collect()
                 }
diff --git a/src/snap.rs b/src/snap.rs
index 9afe2ad9d..7b13f8ab1 100644
--- a/src/snap.rs
+++ b/src/snap.rs
@@ -170,6 +170,7 @@ pub(crate) fn wire_source(wire: &WireModel) -> Option<DimensionAssociationSource
 /// Object-snap-tracking alignment: the cursor projected onto a ray from an
 /// acquired tracking point.
 #[derive(Debug, Clone, Copy)]
+#[non_exhaustive]
 pub struct OtrackHit {
     /// Cursor projected onto the tracking ray.
     pub aligned: DVec3,