Improve comment clarity in StreamableHttpSession

Copilot · stephentoub · Copilot · commit ccfeaa46a12f · 2025-12-15T19:14:53.000Z
Updated comment to be more precise about when LastActivityTicks is updated.

Co-authored-by: stephentoub &lt;2642209+stephentoub@users.noreply.github.com&gt;
diff --git a/src/ModelContextProtocol.AspNetCore/StreamableHttpSession.cs b/src/ModelContextProtocol.AspNetCore/StreamableHttpSession.cs
@@ -58,7 +58,7 @@ public async ValueTask<IAsyncDisposable> AcquireReferenceAsync(CancellationToken
                     {
                         sessionManager.DecrementIdleSessionCount();
                     }
-                    // Update LastActivityTicks whenever a request is made to keep the session alive
+                    // Update LastActivityTicks when acquiring reference in Started state to prevent timeout during active usage
                     LastActivityTicks = sessionManager.TimeProvider.GetTimestamp();
                     break;
                 case SessionState.Disposed:

Original file line number	Diff line number	Diff line change
`@@ -58,7 +58,7 @@ public async ValueTask<IAsyncDisposable> AcquireReferenceAsync(CancellationToken`
`58`	`58`	`{`
`59`	`59`	`sessionManager.DecrementIdleSessionCount();`
`60`	`60`	`}`
`61`		`- // Update LastActivityTicks whenever a request is made to keep the session alive`
	`61`	`+ // Update LastActivityTicks when acquiring reference in Started state to prevent timeout during active usage`
`62`	`62`	`LastActivityTicks = sessionManager.TimeProvider.GetTimestamp();`
`63`	`63`	`break;`
`64`	`64`	`case SessionState.Disposed:`