Skip to content

Commit c1614e4

Browse files
committed
fix links
1 parent c7c7787 commit c1614e4

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

slides/2026-05-update.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -130,21 +130,21 @@ would depend on the provider of `ScopeCeiling` adding them.
130130

131131
1. Is `NewModuleEnvironment` a spec fiction? Can we verify that `ModuleEnvironmentRecord` points to its `OuterEnv` explicitly as opposed to it being looked up by default in the end?
132132

133-
- This [area in Firefox](https://searchfox.org/firefox-main/source/js/src/builtin/ModuleObject.cpp#1598) seems related and [this line](https://searchfox.org/firefox-main/source/js/src/vm/EnvironmentObject.cpp#408) looks like a match for spec setting `OuterEnv`
134-
- This [line in V8](https://github.com/v8/v8/blob/main/src/heap/factory.cc#L1709) mentions `outer` in the context of `ModuleContext` - might be related
133+
- This [area in Firefox](https://searchfox.org/firefox-main/rev/ada2ab816a8c8cdf48c1ccf2111ec74eaff4c7cb/js/src/builtin/ModuleObject.cpp#1576) seems related and [this line](https://searchfox.org/firefox-main/rev/0a95d145f781bc57ef4183545cbb88e35d27b66d/js/src/vm/EnvironmentObject.cpp#388) looks like a match for spec setting `OuterEnv`
134+
- This [line in V8](https://github.com/v8/v8/blob/c3e9684c91b3d706fc22f2fb5b7ede5d62ad6da9/src/heap/factory.cc#L1715) mentions `outer` in the context of `ModuleContext` - might be related
135135

136136
---
137137

138138
## Open questions
139139

140140
2. `ScopeCeiling` needs means to be associated with a module.
141141

142+
- The Compartment proposal would define a common `ScopeCeiling` for all modules loaded in a `Compartment`
143+
142144
- We are open to exploring exposing it independently of Compartment, as some of the motivating usec-ases (e.g. DSLs) do not require an importHook to also be present.
143145
- Is `import('specifier', { scope: {} })` an option?
144146
- An entity encapsulating a pair: `ModuleSource`, `ScopeCeiling` and also be acceptable as input whereever ModuleSource is?
145-
- Open to fedback and suggestions.
146147

147-
- Regardless, the Compartment proposal would define a common `ScopeCeiling` for all modules loaded in a `Compartment`
148148

149149
- If we don't find a useful way to surface `ScopeCeiling` on its own, we will fold this proposal into refreshed Compartment proposal.
150150

0 commit comments

Comments
 (0)