fix: inject progressToken when resetTimeoutOnProgress is set

Previously, _meta.progressToken was only injected into the outgoing
JSON-RPC request when options.onprogress was provided. Setting
resetTimeoutOnProgress: true alone had no effect because the SDK's
_onprogress handler matches incoming notifications by
params.progressToken → messageId — without a token in the request,
server progress notifications can never be matched, and _resetTimeout
is never called.

This change expands the injection condition to also fire when
resetTimeoutOnProgress is true, making the flag behave as its name
implies: if you want the timeout to reset on progress, the server must
be able to send progress notifications that match the request.

The onprogress handler registration is unchanged — a no-op progressToken
injection when only resetTimeoutOnProgress is set does not register a
progress handler (avoiding unnecessary memory use).

Fixes: #245

Author: techtoboggan

packages/core/src/shared/protocol.ts

@@ -832,8 +832,10 @@ export abstract class Protocol<ContextT extends BaseContext> {
                 id: messageId
             };
 
-            if (options?.onprogress) {
-                this._progressHandlers.set(messageId, options.onprogress);
+            if (options?.onprogress || options?.resetTimeoutOnProgress) {
+                if (options.onprogress) {
+                    this._progressHandlers.set(messageId, options.onprogress);
+                }
                 jsonrpcRequest.params = {
                     ...request.params,
                     _meta: {