Skip to content
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

Meta: Update Explainer for Isolated Realms #292

Merged
merged 11 commits into from
Apr 12, 2021
Merged

Meta: Update Explainer for Isolated Realms #292

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
88dd197
Meta: Update Explainer for Isolated Realms
leobalter Mar 27, 2021
c9718a8
Update explainer.md
leobalter Mar 29, 2021
380b797
Update explainer.md
leobalter Mar 29, 2021
a9dcc69
fixup, remove unclear comment
leobalter Mar 29, 2021
c9bb8d3
Address review feedback
leobalter Apr 1, 2021
6220f8e
Add Link to discussions
leobalter Apr 5, 2021
a83b493
Add link to compartments
leobalter Apr 5, 2021
0f671db
Update explainer.md
leobalter Apr 6, 2021
7b88d54
Remove sentence about fingerprints
leobalter Apr 6, 2021
246e41b
Slightly expand comparison with Node vm
leobalter Apr 6, 2021
20535f0
Remove templates from use cases
leobalter Apr 6, 2021
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
1 change: 1 addition & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@ See some examples [in the Explainer file](explainer.md).

## <a name='History'></a>History

* we moved on from the exposed globalThis model to a lean isolated realms API (see #289 and #291)
Copy link
Collaborator

Choose a reason for hiding this comment

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

isolated realms => Realm with Callable Boundary or something like that...

we chatted about this with Shu, and I believe we should avoid using the term isolated realm all together.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Another alternative is also "Callable Realms"

Copy link
Member

Choose a reason for hiding this comment

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

I find the names that @caridy is suggesting confusing, though avoiding the word "isolated" could be useful to avoid other kinds of confusion. Maybe we could avoid giving this version a name for now, until we're really confident in one, since it's awkward to change names later.

Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
* we moved on from the exposed globalThis model to a lean isolated realms API (see #289 and #291)
* we moved on from the exposed globalThis model to a revisited format of Realms with a Callable Boundary API (see #289 and #291)

How about this? We can't explain what this is in a single line, but we can further explain the function auto wrapping creates this callable boundary.

* we worked on this during ES2015 time frame, so never went through stages process ([ES6 Realm Objects proto-spec.pdf](https://github.com/tc39/proposal-realms/files/717415/ES6.Realm.Objects.proto-spec.pdf))
* got punted to later (rightly so!)
* goal of this proposal: resume work on this, reassert committee interest via advancing to stage 2
Expand Down
Loading