Merge origin/main into ochafik/openai-compatibility

src/react/useApp.tsx

@@ -30,7 +30,7 @@ export interface UseAppOptions {
    */
   experimentalOAICompatibility?: boolean;
   /**
-   * Called after App is created but before connection.
+   * Called after {@link App} is created but before connection.
    *
    * Use this to register request/notification handlers that need to be in place
    * before the initialization handshake completes.
@@ -79,6 +79,9 @@ export interface AppState {
  *
  * **Important**: The hook intentionally does NOT re-run when options change
  * to avoid reconnection loops. Options are only used during the initial mount.
+ * Furthermore, the `App` instance is NOT closed on unmount. This avoids cleanup
+ * issues during React Strict Mode's double-mount cycle. If you need to
+ * explicitly close the `App`, call {@link App.close} manually.
  *
  * **Note**: This is part of the optional React integration. The core SDK
  * (App, PostMessageTransport, OpenAITransport) is framework-agnostic and can be

src/server/index.ts

@@ -68,7 +68,8 @@ export const OPENAI_RESOURCE_SUFFIX = "+skybridge";
 export const OPENAI_MIME_TYPE = "text/html+skybridge";
 
 /**
- * Tool configuration (same as McpServer.registerTool).
+ * Base tool configuration matching the standard MCP server tool options.
+ * Extended by {@link McpUiAppToolConfig} to add UI metadata requirements.
  */
 export interface ToolConfig {
   title?: string;
@@ -247,7 +248,7 @@ export function registerAppTool<
  * Register an app resource with dual MCP/OpenAI support.
  *
  * This is a convenience wrapper around `server.registerResource` that:
- * - Defaults the MIME type to "text/html;profile=mcp-app"
+ * - Defaults the MIME type to {@link RESOURCE_MIME_TYPE} (`"text/html;profile=mcp-app"`)
  * - Registers both MCP and OpenAI variants for cross-platform compatibility
  *
  * Registers two resources: