chore: bump version to 0.1.16-beta and add changelog

- Add CHANGELOG entry for 0.1.16-beta and update Directory.Build.props version.
- Introduce SimConnectLogger.IsLevelEnabled (and internal helper) and guard hot-path Debug calls to avoid string allocations and GC churn when debug logging is disabled.
- Apply logging guards across SimObjectManager, SimConnectClient, and SimVarManager.
- Stabilize tests:
  - Use "CoffeeCup" model in AIObjectTests to avoid missing-asset timeouts.
  - Ensure PerformanceTests' short-timeout token fires before issuing request (explicit delay/cancel).
- Minor cleanup to pull request template formatting and checklist spacing.

Author: AussieScorcher

.github/PULL_REQUEST_TEMPLATE.md

@@ -1,24 +1,26 @@
 ## Summary
-<!-- Provide a brief description of what this PR does -->
 
+<!-- Provide a brief description of what this PR does -->
 
 ## Changes Made
+
 <!-- List the main changes in this pull request -->
-- 
-- 
-- 
+
+-
+-
+-
 
 ## Additional Information
-<!-- Any other relevant information, screenshots, or context -->
 
+<!-- Any other relevant information, screenshots, or context -->
 
 ## Author Information
+
 **Discord Username:** <!-- Your Discord username (if different from GitHub) -->
-**VATSIM CID:** <!-- Your VATSIM Controller ID -->
 
 ---
 
 ### Checklist:
 
-* [ ] Have you followed the guidelines in our Contributing document?
-* [ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
+-   [ ] Have you followed the guidelines in our Contributing document?
+-   [ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

CHANGELOG.md

@@ -7,6 +7,27 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.1.16-beta] - 2025-10-20
+
+### Added
+
+-   Set multiple SimVars in one shot by passing a struct to `SimVars.SetAsync<T>()`, mirroring the existing struct-based reader pipeline.
+-   Internal field-writer pipeline that packs annotated struct fields into the unmanaged buffer used for SimConnect writes.
+-   Test coverage that exercises struct-based sets, validates altitude adjustments, and restores the original values afterward.
+
+### Performance
+
+-   Wrapped verbose debug logging in hot paths with `SimConnectLogger.IsLevelEnabled` checks to avoid string allocations and GC churn when debug logging is disabled.
+
+### Fixed
+
+-   Updated the AI object lifecycle test to spawn the `CoffeeCup` model so it no longer times out on missing assets.
+-   Stabilized the performance timeout test by ensuring the cancellation token fires before issuing the request.
+
+### Notes
+
+-   Thanks to @bstudtma for the contribution in PR #11.
+
 ## [0.1.15-beta] - 2025-09-15
 
 ### Added

Directory.Build.props

@@ -1,6 +1,6 @@
 <Project>
   <PropertyGroup>
-    <Version>0.1.15-beta</Version>
+    <Version>0.1.16-beta</Version>
     <Authors>BARS</Authors>
     <Company>BARS</Company>
     <Product>SimConnect.NET</Product>

src/SimConnect.NET/AI/SimObjectManager.cs

@@ -83,7 +83,10 @@ public async Task<SimObject> CreateObjectAsync(
                         (SimConnectError)result);
                 }
 
-                SimConnectLogger.Debug($"SimObjectManager: Requested creation of '{containerTitle}' with requestId {requestId}");
+                if (SimConnectLogger.IsLevelEnabled(SimConnectLogger.LogLevel.Debug))
+                {
+                    SimConnectLogger.Debug($"SimObjectManager: Requested creation of '{containerTitle}' with requestId {requestId}");
+                }
 
                 // Wait for the object creation to complete with shorter timeout
                 using var timeoutCts = CancellationTokenSource.CreateLinkedTokenSource(cancellationToken);
@@ -123,7 +126,11 @@ public Task RemoveObjectAsync(SimObject simObject, CancellationToken cancellatio
 
             if (!simObject.IsActive)
             {
-                SimConnectLogger.Debug($"SimObjectManager: Object {simObject.ObjectId} is already inactive");
+                if (SimConnectLogger.IsLevelEnabled(SimConnectLogger.LogLevel.Debug))
+                {
+                    SimConnectLogger.Debug($"SimObjectManager: Object {simObject.ObjectId} is already inactive");
+                }
+
                 return Task.CompletedTask;
             }
 
@@ -332,7 +339,10 @@ public void Dispose()
 
                 this.managedObjects.Clear();
 
-                SimConnectLogger.Debug("SimObjectManager: Disposed");
+                if (SimConnectLogger.IsLevelEnabled(SimConnectLogger.LogLevel.Debug))
+                {
+                    SimConnectLogger.Debug("SimObjectManager: Disposed");
+                }
             }
             finally
             {

src/SimConnect.NET/SimConnectClient.cs

@@ -362,7 +362,10 @@ public async Task<bool> ProcessNextMessageAsync(CancellationToken cancellationTo
                         // Filter out the common "no messages available" error to reduce log spam
                         if (result != -2147467259)
                         {
-                            SimConnectLogger.Debug($"SimConnect_GetNextDispatch returned: {(SimConnectError)result}");
+                            if (SimConnectLogger.IsLevelEnabled(SimConnectLogger.LogLevel.Debug))
+                            {
+                                SimConnectLogger.Debug($"SimConnect_GetNextDispatch returned: {(SimConnectError)result}");
+                            }
                         }
 
                         return false;
@@ -373,7 +376,10 @@ public async Task<bool> ProcessNextMessageAsync(CancellationToken cancellationTo
                         var recv = Marshal.PtrToStructure<SimConnectRecv>(ppData);
                         var recvId = (SimConnectRecvId)recv.Id;
 
-                        SimConnectLogger.Debug($"Received SimConnect message: Id={recv.Id}, Size={recv.Size}");
+                        if (SimConnectLogger.IsLevelEnabled(SimConnectLogger.LogLevel.Debug))
+                        {
+                            SimConnectLogger.Debug($"Received SimConnect message: Id={recv.Id}, Size={recv.Size}");
+                        }
 
                         try
                         {

src/SimConnect.NET/SimConnectLogger.cs

@@ -129,6 +129,14 @@ public static void Configure(LogLevel minimumLevel = LogLevel.Debug, string? log
             }
         }
 
+        /// <summary>
+        /// Determines whether logging is currently enabled for the specified level.
+        /// Helps avoid expensive string formatting on disabled levels.
+        /// </summary>
+        /// <param name="level">The log level to check.</param>
+        /// <returns><c>true</c> if the level is enabled; otherwise, <c>false</c>.</returns>
+        public static bool IsLevelEnabled(LogLevel level) => Instance.IsLevelEnabledInternal(level);
+
         /// <summary>
         /// Logs a debug message.
         /// </summary>
@@ -220,7 +228,7 @@ private void Enqueue(LogLevel level, string message)
                 return;
             }
 
-            if (level < this.MinimumLevel)
+            if (!this.IsLevelEnabledInternal(level))
             {
                 return;
             }
@@ -235,6 +243,8 @@ private void Enqueue(LogLevel level, string message)
             }
         }
 
+        private bool IsLevelEnabledInternal(LogLevel level) => level >= this.MinimumLevel;
+
         private void ProcessQueue()
         {
             try

src/SimConnect.NET/SimVar/SimVarManager.cs

@@ -355,7 +355,10 @@ public void Dispose()
                     }
                     catch (Exception ex)
                     {
-                        SimConnectLogger.Debug($"Suppressing error disposing subscription {kvp.Key}: {ex.Message}");
+                        if (SimConnectLogger.IsLevelEnabled(SimConnectLogger.LogLevel.Debug))
+                        {
+                            SimConnectLogger.Debug($"Suppressing error disposing subscription {kvp.Key}: {ex.Message}");
+                        }
                     }
                 }
 
@@ -451,12 +454,19 @@ internal void CancelRequest(ISimVarRequest request)
             {
                 try
                 {
-                    SimConnectLogger.Debug($"Canceling recurring request {request.RequestId} (Def={request.DefinitionId}, Obj={request.ObjectId})");
+                    if (SimConnectLogger.IsLevelEnabled(SimConnectLogger.LogLevel.Debug))
+                    {
+                        SimConnectLogger.Debug($"Canceling recurring request {request.RequestId} (Def={request.DefinitionId}, Obj={request.ObjectId})");
+                    }
+
                     this.RequestDataOnSimObject(request.RequestId, request.DefinitionId, request.ObjectId, SimConnectPeriod.Never);
                 }
                 catch (Exception ex)
                 {
-                    SimConnectLogger.Debug($"Suppressing error during CancelRequest for {request.RequestId}: {ex.Message}");
+                    if (SimConnectLogger.IsLevelEnabled(SimConnectLogger.LogLevel.Debug))
+                    {
+                        SimConnectLogger.Debug($"Suppressing error during CancelRequest for {request.RequestId}: {ex.Message}");
+                    }
                 }
             }
 
@@ -593,15 +603,15 @@ private static string ParseString(IntPtr dataPtr, SimConnectDataType dataType)
             return SimVarMemoryReader.ReadFixedString(dataPtr, maxLength);
         }
 
-    /// <summary>
-    /// Wrapper to call SimConnect_RequestDataOnSimObject with consistent error handling.
-    /// A local context string is generated from the parameters for logging and exception messages.
-    /// </summary>
+        /// <summary>
+        /// Wrapper to call SimConnect_RequestDataOnSimObject with consistent error handling.
+        /// A local context string is generated from the parameters for logging and exception messages.
+        /// </summary>
         /// <param name="requestId">The SimConnect request identifier.</param>
         /// <param name="definitionId">The data definition identifier.</param>
         /// <param name="objectId">The target object identifier.</param>
-    /// <param name="period">The request period.</param>
-    /// <remarks>Throws a SimConnectException on error (except when period == Never which is used internally for cancellation).</remarks>
+        /// <param name="period">The request period.</param>
+        /// <remarks>Throws a SimConnectException on error (except when period == Never which is used internally for cancellation).</remarks>
         private void RequestDataOnSimObject(
             uint requestId,
             uint definitionId,
@@ -737,7 +747,11 @@ private uint EnsureDataDefinition(SimVarDefinition definition, CancellationToken
 
             if (this.dataDefinitions.TryGetValue(key, out var existingId))
             {
-                SimConnectLogger.Debug($"Reusing existing definition ID {existingId} for {key.Name}|{key.Unit}");
+                if (SimConnectLogger.IsLevelEnabled(SimConnectLogger.LogLevel.Debug))
+                {
+                    SimConnectLogger.Debug($"Reusing existing definition ID {existingId} for {key.Name}|{key.Unit}");
+                }
+
                 return existingId;
             }
 
@@ -775,7 +789,11 @@ private uint EnsureScalarDefinition(string name, string? unit = null, SimConnect
 
             if (this.dataDefinitions.TryGetValue(key, out var existingId))
             {
-                SimConnectLogger.Debug($"Reusing existing definition ID {existingId} for {key.Name}|{key.Unit}");
+                if (SimConnectLogger.IsLevelEnabled(SimConnectLogger.LogLevel.Debug))
+                {
+                    SimConnectLogger.Debug($"Reusing existing definition ID {existingId} for {key.Name}|{key.Unit}");
+                }
+
                 return existingId;
             }
 
@@ -788,7 +806,10 @@ private uint EnsureScalarDefinition(string name, string? unit = null, SimConnect
             }
 
             var definitionId = Interlocked.Increment(ref this.nextDefinitionId);
-            SimConnectLogger.Debug($"Creating new definition ID {definitionId} for {name}|{unit}");
+            if (SimConnectLogger.IsLevelEnabled(SimConnectLogger.LogLevel.Debug))
+            {
+                SimConnectLogger.Debug($"Creating new definition ID {definitionId} for {name}|{unit}");
+            }
 
             var result = SimConnectNative.SimConnect_AddToDataDefinition(
                 this.simConnectHandle,

tests/SimConnect.NET.Tests.Net8/Tests/AIObjectTests.cs

@@ -12,6 +12,8 @@ namespace SimConnect.NET.Tests.Net8.Tests
     /// </summary>
     public class AIObjectTests : ISimConnectTest
     {
+        private const string TestSimObjectModel = "CoffeeCup";
+
         /// <inheritdoc/>
         public string Name => "AI Object Management";
 
@@ -83,7 +85,7 @@ private static async Task<bool> TestSingleObjectLifecycle(SimConnectClient clien
 
             Console.WriteLine($"      🎯 Creating AI object at {position.Latitude:F6}, {position.Longitude:F6}");
 
-            var aiObject = await client.AIObjects.CreateObjectAsync("BARS_Stopbar_On", position, "Test Object", cancellationToken);
+            var aiObject = await client.AIObjects.CreateObjectAsync(TestSimObjectModel, position, "Test Object", cancellationToken);
             Console.WriteLine($"      ✅ AI Object created with ID: {aiObject.ObjectId}");
 
             if (!aiObject.IsActive)
@@ -130,7 +132,7 @@ private static async Task<bool> TestMultipleObjects(SimConnectClient client, Can
                         Airspeed = 0,
                     };
 
-                    var obj = await client.AIObjects.CreateObjectAsync("BARS_Stopbar_On", position, $"Test Object {i}", cancellationToken);
+                    var obj = await client.AIObjects.CreateObjectAsync(TestSimObjectModel, position, $"Test Object {i}", cancellationToken);
                     objects.Add(obj);
                     Console.WriteLine($"      ✅ Created object {i + 1} with ID: {obj.ObjectId}");
                 }
@@ -190,7 +192,7 @@ private static async Task<bool> TestObjectTracking(SimConnectClient client, Canc
                 Airspeed = 0,
             };
 
-            var aiObject = await client.AIObjects.CreateObjectAsync("BARS_Stopbar_On", position, "Tracking Test", cancellationToken);
+            var aiObject = await client.AIObjects.CreateObjectAsync(TestSimObjectModel, position, "Tracking Test", cancellationToken);
 
             try
             {

tests/SimConnect.NET.Tests.Net8/Tests/PerformanceTests.cs

@@ -172,6 +172,15 @@ private static async Task<bool> TestTimeoutHandling(SimConnectClient client, Can
                 using var shortTimeout = new CancellationTokenSource(TimeSpan.FromMilliseconds(1));
                 using var combined = CancellationTokenSource.CreateLinkedTokenSource(cancellationToken, shortTimeout.Token);
 
+                // Allow the timeout token to fire before issuing the request so cancellation is guaranteed.
+                await Task.Delay(TimeSpan.FromMilliseconds(15), cancellationToken).ConfigureAwait(false);
+
+                if (!shortTimeout.IsCancellationRequested)
+                {
+                    // In the unlikely event the timer hasn't fired yet (timer resolution quirks), cancel explicitly.
+                    shortTimeout.Cancel();
+                }
+
                 // This should timeout quickly
                 await client.SimVars.GetAsync<double>("PLANE LATITUDE", "degrees", cancellationToken: combined.Token);