From 9cc6cfa1b1cb7cbc8895ca503145e2b0df96be84 Mon Sep 17 00:00:00 2001
From: AJ Isaacs <ajisaacs27@gmail.com>
Date: Sat, 14 Mar 2026 20:33:07 -0400
Subject: [PATCH] fix(engine): add volatile to AnglePredictor lock field and
 Models content copy

- Mark _loadAttempted as volatile for correct double-checked locking
- Add Content item to copy Models/ directory to output for ONNX inference

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 OpenNest.Engine/ML/AnglePredictor.cs   | 2 +-
 OpenNest.Engine/OpenNest.Engine.csproj | 3 +++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/OpenNest.Engine/ML/AnglePredictor.cs b/OpenNest.Engine/ML/AnglePredictor.cs
index 3f2386c..da864a8 100644
--- a/OpenNest.Engine/ML/AnglePredictor.cs
+++ b/OpenNest.Engine/ML/AnglePredictor.cs
@@ -12,7 +12,7 @@ namespace OpenNest.Engine.ML
     public static class AnglePredictor
     {
         private static InferenceSession _session;
-        private static bool _loadAttempted;
+        private static volatile bool _loadAttempted;
         private static readonly object _lock = new();
 
         public static List<double> PredictAngles(
diff --git a/OpenNest.Engine/OpenNest.Engine.csproj b/OpenNest.Engine/OpenNest.Engine.csproj
index a4b3e98..bcf6d9e 100644
--- a/OpenNest.Engine/OpenNest.Engine.csproj
+++ b/OpenNest.Engine/OpenNest.Engine.csproj
@@ -10,4 +10,7 @@
   <ItemGroup>
     <PackageReference Include="Microsoft.ML.OnnxRuntime" Version="1.17.3" />
   </ItemGroup>
+  <ItemGroup>
+    <Content Include="Models\**" CopyToOutputDirectory="PreserveNewest" Condition="Exists('Models')" />
+  </ItemGroup>
 </Project>