Skip to content

Commit 50cf6a4

Browse files
ochafikclaude
andauthored
Fix sandbox-ready notification name in spec to match implementation (#160)
The spec used `ui/notifications/sandbox-ready` but the implementation uses `ui/notifications/sandbox-proxy-ready`. Updated the spec to match. Also fixed a typo: `ui/sandbox-resource-ready` → `ui/notifications/sandbox-resource-ready` Fixes #156 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude <[email protected]>
1 parent 31e05d3 commit 50cf6a4

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

specification/draft/apps.mdx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -358,7 +358,7 @@ If the Host is a web page, it MUST wrap the Guest UI and communicate with it thr
358358

359359
1. The Host and the Sandbox MUST have different origins.
360360
2. The Sandbox MUST have the following permissions: `allow-scripts`, `allow-same-origin`.
361-
3. The Sandbox MUST send a `ui/notifications/sandbox-ready` notification to the host when its ready to process an `ui/sandbox-resource-ready` notification.
361+
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.
362362
4. Once the Sandbox is ready, the Host MUST send the raw HTML resource to load in a `ui/notifications/sandbox-resource-ready` notification.
363363
5. The Sandbox MUST load the raw HTML of the Guest UI with CSP settings that:
364364
- Enforce the domains declared in `ui.csp` metadata
@@ -841,12 +841,12 @@ Host MAY send this notification when any context field changes (e.g., theme togg
841841

842842
These messages are reserved for web-based hosts that implement the recommended double-iframe sandbox architecture:
843843

844-
`ui/notifications/sandbox-ready` (Sandbox Proxy → Host) - Sandbox proxy is ready
844+
`ui/notifications/sandbox-proxy-ready` (Sandbox Proxy → Host) - Sandbox proxy is ready
845845

846846
```typescript
847847
{
848848
jsonrpc: "2.0",
849-
method: "ui/notifications/sandbox-ready",
849+
method: "ui/notifications/sandbox-proxy-ready",
850850
params: {}
851851
}
852852
```
@@ -899,7 +899,7 @@ sequenceDiagram
899899
H ->> H: Render Guest UI in an iframe (HTML from the ui:// resource)
900900
else Web hosts
901901
H ->> H: Render Sandbox Proxy in an iframe (different origin)
902-
P ->> H: ui/notifications/sandbox-ready
902+
P ->> H: ui/notifications/sandbox-proxy-ready
903903
H -->> P: ui/notifications/sandbox-resource-ready (HTML content)
904904
P -> P: Render inner iframe with HTML
905905

0 commit comments

Comments
 (0)