Skip to content

Commit 9bbd279

Browse files
committed
address PR review comments
- Remove AGENTS.md reference to migration guide - Add note that OpenAITransport is not yet available (PR #172) - Remove unnecessary note about return type ordering
1 parent da0c91c commit 9bbd279

File tree

2 files changed

+4
-9
lines changed

2 files changed

+4
-9
lines changed

AGENTS.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -85,7 +85,3 @@ Uses npm workspaces. Examples in `examples/` are separate packages:
8585
- `basic-server-*` - Starter templates (vanillajs, react, vue, svelte, preact, solid). Use these as the basis for new examples.
8686
- `basic-host` - Reference host implementation
8787
- Other examples showcase specific features (charts, 3D, video, etc.)
88-
89-
## Migrating code from OpenAI Apps SDK to MCP Apps
90-
91-
See [Migrating from OpenAI Apps SDK to MCP Apps SDK](./docs/migrate_from_openai_apps.md)

docs/migrate_from_openai_apps.md

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ This guide helps you migrate from the OpenAI Apps SDK (`window.openai.*`) to the
1616
| -------------------------------- | -------------------------------------------------- | ------------------------------------------------------ |
1717
| `window.openai` (auto-available) | `const app = new App({name, version}, {})` | MCP requires explicit instantiation |
1818
| (implicit) | `await app.connect()` | MCP requires async connection; auto-detects OpenAI env |
19-
|| `await app.connect(new OpenAITransport())` | Force OpenAI mode explicitly |
19+
|| `await app.connect(new OpenAITransport())` | Force OpenAI mode (not yet available, see [PR #172](https://github.com/modelcontextprotocol/ext-apps/pull/172)) |
2020
|| `await app.connect(new PostMessageTransport(...))` | Force MCP mode explicitly |
2121

2222
## Host Context Properties
@@ -44,10 +44,9 @@ This guide helps you migrate from the OpenAI Apps SDK (`window.openai.*`) to the
4444

4545
## Calling Tools
4646

47-
| OpenAI | MCP Apps | Notes |
48-
| ---------------------------------------------------- | ----------------------------------------------------- | --------------------------------------- |
49-
| `await window.openai.callTool(name, args)` | `await app.callServerTool({ name, arguments: args })` | Call another MCP server tool |
50-
| Returns `{ structuredContent?, content?, isError? }` | Returns `{ content, structuredContent?, isError? }` | Same shape, slightly different ordering |
47+
| OpenAI | MCP Apps | Notes |
48+
| ------------------------------------------ | ----------------------------------------------------- | ---------------------------- |
49+
| `await window.openai.callTool(name, args)` | `await app.callServerTool({ name, arguments: args })` | Call another MCP server tool |
5150

5251
## Sending Messages
5352

0 commit comments

Comments
 (0)