docs(core): clarify resetTimeoutOnProgress requires onprogress

packages/client/src/client/auth.ts

@@ -763,28 +763,31 @@ async function authInternal(
 
     // Handle token refresh or new authorization
     if (tokens?.refresh_token) {
+        let newTokens;
         try {
             // Attempt to refresh the token
-            const newTokens = await refreshAuthorization(authorizationServerUrl, {
+            newTokens = await refreshAuthorization(authorizationServerUrl, {
                 metadata,
                 clientInformation,
                 refreshToken: tokens.refresh_token,
                 resource,
                 addClientAuthentication: provider.addClientAuthentication,
                 fetchFn
             });
-
-            await provider.saveTokens(newTokens);
-            return 'AUTHORIZED';
         } catch (error) {
             // If this is a ServerError, or an unknown type, log it out and try to continue. Otherwise, escalate so we can fix things and retry.
             if (!(error instanceof OAuthError) || error.code === OAuthErrorCode.ServerError) {
-                // Could not refresh OAuth tokens
+                // Could not refresh OAuth tokens — fall through to re-auth
             } else {
                 // Refresh failed for another reason, re-throw
                 throw error;
             }
         }
+
+        if (newTokens) {
+            await provider.saveTokens(newTokens); // Let I/O errors propagate
+            return 'AUTHORIZED';
+        }
     }
 
     const state = provider.state ? await provider.state() : undefined;

packages/core/src/shared/protocol.ts

@@ -124,7 +124,11 @@ export type RequestOptions = {
 
     /**
      * If `true`, receiving a progress notification will reset the request timeout.
-     * This is useful for long-running operations that send periodic progress updates.
+     * Note: a progress token is only included in the request when `onprogress` is
+     * explicitly provided (to avoid unnecessary server round-trips). Therefore
+     * `resetTimeoutOnProgress` only takes effect when `onprogress` is also set.
+     * If progress is consumed via lower-level transport features, provide a no-op
+     * `onprogress` callback to enable timeout reset on progress.
      * Default: `false`
      */
     resetTimeoutOnProgress?: boolean;

packages/server/src/server/mcp.ts

@@ -8,6 +8,7 @@ import type {
     CreateTaskResult,
     CreateTaskServerContext,
     GetPromptResult,
+    Icon,
     Implementation,
     ListPromptsResult,
     ListResourcesResult,
@@ -533,6 +534,7 @@ export class McpServer {
                             title: prompt.title,
                             description: prompt.description,
                             arguments: prompt.argsSchema ? promptArgumentsFromStandardSchema(prompt.argsSchema) : undefined,
+                            icons: prompt.icons,
                             _meta: prompt._meta
                         };
                     })
@@ -702,6 +704,7 @@ export class McpServer {
         title: string | undefined,
         description: string | undefined,
         argsSchema: StandardSchemaWithJSON | undefined,
+        icons: Icon[] | undefined,
         callback: PromptCallback<StandardSchemaWithJSON | undefined>,
         _meta: Record<string, unknown> | undefined
     ): RegisteredPrompt {
@@ -713,6 +716,7 @@ export class McpServer {
             title,
             description,
             argsSchema,
+            icons,
             _meta,
             handler: createPromptHandler(name, argsSchema, callback),
             enabled: true,
@@ -726,6 +730,7 @@ export class McpServer {
                 }
                 if (updates.title !== undefined) registeredPrompt.title = updates.title;
                 if (updates.description !== undefined) registeredPrompt.description = updates.description;
+                if (updates.icons !== undefined) registeredPrompt.icons = updates.icons;
                 if (updates._meta !== undefined) registeredPrompt._meta = updates._meta;
 
                 // Track if we need to regenerate the handler
@@ -952,6 +957,7 @@ export class McpServer {
             title?: string;
             description?: string;
             argsSchema?: Args;
+            icons?: Icon[];
             _meta?: Record<string, unknown>;
         },
         cb: PromptCallback<Args>
@@ -963,6 +969,7 @@ export class McpServer {
             title?: string;
             description?: string;
             argsSchema?: Args;
+            icons?: Icon[];
             _meta?: Record<string, unknown>;
         },
         cb: LegacyPromptCallback<Args>
@@ -973,6 +980,7 @@ export class McpServer {
             title?: string;
             description?: string;
             argsSchema?: StandardSchemaWithJSON | ZodRawShape;
+            icons?: Icon[];
             _meta?: Record<string, unknown>;
         },
         cb: PromptCallback<StandardSchemaWithJSON> | LegacyPromptCallback<ZodRawShape>
@@ -981,13 +989,14 @@ export class McpServer {
             throw new Error(`Prompt ${name} is already registered`);
         }
 
-        const { title, description, argsSchema, _meta } = config;
+        const { title, description, argsSchema, icons, _meta } = config;
 
         const registeredPrompt = this._createRegisteredPrompt(
             name,
             title,
             description,
             normalizeRawShapeSchema(argsSchema),
+            icons,
             cb as PromptCallback<StandardSchemaWithJSON | undefined>,
             _meta
         );
@@ -1308,6 +1317,7 @@ export type RegisteredPrompt = {
     title?: string;
     description?: string;
     argsSchema?: StandardSchemaWithJSON;
+    icons?: Icon[];
     _meta?: Record<string, unknown>;
     /** @hidden */
     handler: PromptHandler;
@@ -1319,6 +1329,7 @@ export type RegisteredPrompt = {
         title?: string;
         description?: string;
         argsSchema?: Args;
+        icons?: Icon[];
         _meta?: Record<string, unknown>;
         callback?: PromptCallback<Args>;
         enabled?: boolean;