Skip to content

Fix grammar errors in apps.mdx #300

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
glen-84 wants to merge 1 commit into
base: main
Choose a base branch
from
+7 −7
Open

Fix grammar errors in apps.mdx #300

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions specification/draft/apps.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -463,7 +463,7 @@ If the Host is a web page, it MUST wrap the Guest UI and communicate with it thr

1. The Host and the Sandbox MUST have different origins.
2. The Sandbox MUST have the following permissions: `allow-scripts`, `allow-same-origin`.
3. The Sandbox MUST send a `ui/notifications/sandbox-proxy-ready` notification to the host when it's ready to process an `ui/notifications/sandbox-resource-ready` notification.
3. The Sandbox MUST send a `ui/notifications/sandbox-proxy-ready` notification to the host when it's ready to process a `ui/notifications/sandbox-resource-ready` notification.
4. Once the Sandbox is ready, the Host MUST send the raw HTML resource to load in a `ui/notifications/sandbox-resource-ready` notification.
5. The Sandbox MUST load the raw HTML of the Guest UI with CSP settings that:
- Enforce the domains declared in `ui.csp` metadata
Expand Down Expand Up @@ -499,7 +499,7 @@ UI iframes can use the following subset of standard MCP protocol messages:

### Host Context in `McpUiInitializeResult`

When the Guest UI sends an `ui/initialize` request to the Host, the Host SHOULD include UI-specific context in the `McpUiInitializeResult`'s `hostContext` field:
When the Guest UI sends a `ui/initialize` request to the Host, the Host SHOULD include UI-specific context in the `McpUiInitializeResult`'s `hostContext` field:

```typescript
interface HostContext {
Expand Down Expand Up @@ -1075,19 +1075,19 @@ Guest UI behavior (optional):

Host MUST send this notification when tool execution completes (if UI is displayed during tool execution).

`ui/notifications/tool-cancelled` - Tool execution was cancelled
`ui/notifications/tool-canceled` - Tool execution was canceled

```typescript
{
jsonrpc: "2.0",
method: "ui/notifications/tool-cancelled",
method: "ui/notifications/tool-canceled",
Copy link
Contributor

@ochafik ochafik Jan 21, 2026 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately this would be a breaking change / needs type update (probably not happening before first non-draft version at this point)

Copy link
Collaborator

@idosal idosal Jan 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both are valid, so I'm not sure this must be changed.

Copy link
Author

@glen-84 glen-84 Jan 22, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not American myself, but most technical literature is written in US English.

The specification is using words like color and behavior, which is US English, so it might be best not to mix US and non-US English.

I noted the breaking change in the OP. If you decide to make the change, it would surely be easier to do that now with the draft version?

params: {
reason: string
}
}
```

Host MUST send this notification if the tool execution was cancelled, for any reason (which can optionally be specified), including user action, sampling error, classifier intervention, etc.
Host MUST send this notification if the tool execution was canceled, for any reason (which can optionally be specified), including user action, sampling error, classifier intervention, etc.

`ui/resource-teardown` - Host notifies UI before teardown

Expand Down Expand Up @@ -1238,8 +1238,8 @@ sequenceDiagram
end
alt Tool complete
H -->> UI: ui/notifications/tool-result
else Tool cancelled
H -->> UI: ui/notifications/tool-cancelled
else Tool canceled
H -->> UI: ui/notifications/tool-canceled
end
```

Expand Down