Fix overlay showing stale glucose value instead of latest

Keith Klein · Keith Klein · commit 191dbd0002d2 · 2026-02-20T17:22:16.000-08:00
GetCurrentReadingAsync() used FirstOrDefault() on the ascending-sorted
cached readings list, returning the oldest reading in the 10-minute
window instead of the newest. Changed to LastOrDefault() so the overlay
value matches the graph window.
diff --git a/DexcomOverlay/Services/DexcomShareClient.cs b/DexcomOverlay/Services/DexcomShareClient.cs
@@ -72,7 +72,7 @@ public DexcomShareClient(string username, string password, string region = "us")
     public async Task<GlucoseReading?> GetCurrentReadingAsync(CancellationToken ct = default)
     {
         var readings = await GetGlucoseReadingsAsync(minutes: 10, maxCount: 1, ct).ConfigureAwait(false);
-        return readings.FirstOrDefault();
+        return readings.LastOrDefault();
     }
 
     /// <summary>Get glucose readings within the specified time window. Uses cache when possible.</summary>

Original file line number	Diff line number	Diff line change
`@@ -72,7 +72,7 @@ public DexcomShareClient(string username, string password, string region = "us")`
`72`	`72`	`public async Task<GlucoseReading?> GetCurrentReadingAsync(CancellationToken ct = default)`
`73`	`73`	`{`
`74`	`74`	`var readings = await GetGlucoseReadingsAsync(minutes: 10, maxCount: 1, ct).ConfigureAwait(false);`
`75`		`- return readings.FirstOrDefault();`
	`75`	`+ return readings.LastOrDefault();`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`/// <summary>Get glucose readings within the specified time window. Uses cache when possible.</summary>`