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.

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

Layered APIs #276

Closed
domenic opened this issue Apr 26, 2018 · 6 comments
Closed

Layered APIs #276

domenic opened this issue Apr 26, 2018 · 6 comments
Assignees

Comments

@domenic
Copy link
Member

domenic commented Apr 26, 2018

Hello TAG!

I'm requesting a TAG review of:

Further details:

  • Relevant time constraints or deadlines: We are currently working to implement this behind a flag in Chrome, but it's still pretty early-stage and experimental.
  • I have read and filled out the Self-Review Questionnare on Security and Privacy. The assessment is as follows:
    • No to all questions, except:
    • Does this specification enable new script execution/loading mechanisms? Arguably yes, although it does so via a modification of the existing script type=module mechanisms. This is used for loading built-in browser features using the same syntax, or loading fallback scripts. The first of these does not have any security concerns; the second goes through the normal module script loading path and so is taken care of by existing mechanisms.
  • I have reviewed the TAG's API Design Principles

You should also know that...

The specific syntax, of using std:x|y module specifiers, is not set in stone. We are hoping to get wider feedback. Many alternate approaches were explored; another leading candidate is noted in the package name maps proposal. Thoughts welcome, although please do check the linked document to see if we've covered a specific suggestion before.

We'd prefer the TAG provide feedback as open issues in our Github repo for each point of feedback.

@travisleithead travisleithead self-assigned this May 22, 2018
@dbaron dbaron self-assigned this May 22, 2018
@torgo torgo added this to the 2018-07-24-f2f-seattle milestone May 22, 2018
@torgo
Copy link
Member

torgo commented May 22, 2018

Agreed to discuss in depth at July f2f - invite people working on this.

@kenchris
Copy link

Hi there, is this an urgent review? As far as we understood from TPAC, you (@domenic) might have changed some of your opinions around this?

@travisleithead
Copy link
Contributor

travisleithead commented Oct 31, 2018

Some thoughts from our Paris F2F:

  • In general, we love where we see this going, especially the latest updates we heard at TPAC.
  • In the current doc, there's a fallback syntax (|) which sounds like it could be trouble in unsupported browsers, but more recent thinking appears to solve this much better with the package name maps proposals.
  • Heard a suggestion at TPAC for the prefix to use for built-in modules: lib/ which really harmonized with me. Would prefer only this one prefix for now if possible--we can avoid name collisions through external processes.

Looking forward to seeing all these docs updated with your latest thinking. Let us know when we should take another detailed look!

@travisleithead travisleithead added Progress: pending external feedback The TAG is waiting on response to comments/questions asked by the TAG during the review and removed Paris2018f2f labels Oct 31, 2018
@travisleithead travisleithead removed their assignment Feb 5, 2019
@kenchris
Copy link

kenchris commented Feb 5, 2019

@travisleithead tc39/proposal-built-in-modules#12

@domenic
Copy link
Member Author

domenic commented Feb 5, 2019

It's probably best if this be closed in favor of #340.

@cynthia cynthia closed this as completed Feb 5, 2019
@cynthia cynthia removed the Progress: pending external feedback The TAG is waiting on response to comments/questions asked by the TAG during the review label Feb 5, 2019
@cynthia
Copy link
Member

cynthia commented Feb 5, 2019

Thanks for letting us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants