Repository navigation

[JSC] removeEntry() and clearAll() keep the realm's [[LoadedModules]] in step with the registry - #748

Merged

dylan-conway merged 9 commits into

mainfrom

claude/module-loader-removed-entry

Sep 30, 2026

Commits on Sep 30, 2026

[JSC] A module load left in flight by removeEntry() or clearAll() writes only to its own registry entry
dylan-conway
committed
e8bfeef
A load by name has its registry entry from loadModule() on and no longer looks it up by key
dylan-conway
committed
71e808b
A module load asks the registry for its entry once: hostLoadImportedModule() opens, loadEntry() loads, and the realm's [[LoadedModules]] is gone
dylan-conway
committed
b64e865
provideFetch() is the fetch of an entry that is still waiting for one
dylan-conway
committed
4de861d
moduleLoadTopSettled names its entry once
dylan-conway
committed
b0ab9f6
An entry whose fetch failed leaves the registry when it fails, in setFetchError(), for every kind of load
dylan-conway
committed
eff6b5f
Put the loader back as it is on main and drop the realm's [[LoadedModules]]: hostLoadImportedModule() asks the registry
dylan-conway
committed
e0e02de
Keep the realm's [[LoadedModules]] as it is on main, and do not add a module to it that the registry no longer holds
dylan-conway
committed
c4c4663
removeEntry() takes a module out of the realm's [[LoadedModules]] under whatever specifier it was asked for
dylan-conway
committed
25dd940