-
Notifications
You must be signed in to change notification settings - Fork 71
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.
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
feat(endo): Search for index.js and implied package exports #424
Closed
Commits on Aug 24, 2020
-
Configuration menu - View commit details
-
Copy full SHA for c04592d - Browse repository at this point
Copy the full SHA c04592dView commit details -
Configuration menu - View commit details
-
Copy full SHA for 208afd5 - Browse repository at this point
Copy the full SHA 208afd5View commit details -
Configuration menu - View commit details
-
Copy full SHA for 22eb6c4 - Browse repository at this point
Copy the full SHA 22eb6c4View commit details -
Configuration menu - View commit details
-
Copy full SHA for c24bc99 - Browse repository at this point
Copy the full SHA c24bc99View commit details -
Configuration menu - View commit details
-
Copy full SHA for 54bfebf - Browse repository at this point
Copy the full SHA 54bfebfView commit details -
Configuration menu - View commit details
-
Copy full SHA for 3c8dc53 - Browse repository at this point
Copy the full SHA 3c8dc53View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4cee940 - Browse repository at this point
Copy the full SHA 4cee940View commit details -
Configuration menu - View commit details
-
Copy full SHA for d5bc072 - Browse repository at this point
Copy the full SHA d5bc072View commit details -
Configuration menu - View commit details
-
Copy full SHA for aff7916 - Browse repository at this point
Copy the full SHA aff7916View commit details -
Configuration menu - View commit details
-
Copy full SHA for c7ad8db - Browse repository at this point
Copy the full SHA c7ad8dbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9153014 - Browse repository at this point
Copy the full SHA 9153014View commit details -
feat(endo): Search for index.js and implied package exports
When asked for a module named "x", Node.js will search for "x.js" then "x/index.js". For packages that do not supply an "exports" property in their "package.json", any module contained by that package is a valid exported module. To achieve parity with these two features, Endo uses different techniques when loading from the file system and loading from an archive. When reading from the file system, Endo will search for a satsifactory candidate in each compartment's asynchronous importHook. Endo also uses the compartment's new moduleMapHook to search for dependency compartments in the "scope" of a module identifier prefix. These allow Endo to operate from an incomplete compartment map for the initial load. The Endo archiver instead creates a more complete compartment map, with every discovered module. This introduces a new kind of module to the compartment map module descriptor type union: modules with known locations and corresponding parsers. The archiver erases the "scopes", "types", and "parsers" on each compartment description since they are no longer necessary. When reading from an archive, Endo uses an importHook that consults the compartment map directly for the locations of all contained modules, and creates a complete moduleMap up front.
Configuration menu - View commit details
-
Copy full SHA for 115e791 - Browse repository at this point
Copy the full SHA 115e791View commit details -
Configuration menu - View commit details
-
Copy full SHA for 444b063 - Browse repository at this point
Copy the full SHA 444b063View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.