Merge pull request #46 from pyscript/drop-core-interpreters

Allow embedders to avoid default interpreters

Author: WebReflection

core.js.map

@@ -12,9 +12,8 @@ export { env } from './listeners.js';
 export * from './errors.js';
 export const XWorker = xworker();
 
-const INTERPRETER_SELECTORS = selectors.join(',');
-
 const mo = new MutationObserver((records) => {
+    const selector = selectors.join(',');
     for (const { type, target, attributeName, addedNodes } of records) {
         // attributes are tested via integration / e2e
         /* c8 ignore start */
@@ -40,9 +39,9 @@ const mo = new MutationObserver((records) => {
         for (const node of addedNodes) {
             if (node.nodeType === 1) {
                 addAllListeners(node);
-                if (node.matches(INTERPRETER_SELECTORS)) handle(node);
+                if (selector && node.matches(selector)) handle(node);
                 else {
-                    $$(INTERPRETER_SELECTORS, node).forEach(handle);
+                    if (selector) $$(selector, node).forEach(handle);
                     if (!CUSTOM_SELECTORS.length) continue;
                     handleCustomType(node);
                     $$(CUSTOM_SELECTORS.join(','), node).forEach(
@@ -67,5 +66,9 @@ assign(Element.prototype, {
     },
 });
 
-addAllListeners(observe(document));
-$$(INTERPRETER_SELECTORS, document).forEach(handle);
+// give 3rd party a chance to apply changes before this happens
+queueMicrotask(() => {
+    addAllListeners(observe(document));
+    const selector = selectors.join(',');
+    if (selector) $$(selector, document).forEach(handle);
+});

docs/core.js

@@ -5,7 +5,9 @@ import { all, create } from './utils.js';
 import { registry, prefixes } from './interpreters.js';
 
 export const env = new Proxy(create(null), {
-    get: (_, name) => awaitInterpreter(name),
+    get: (_, name) => new Promise(queueMicrotask).then(
+        () => awaitInterpreter(name)
+    ),
 });
 
 /* c8 ignore start */ // attributes are tested via integration / e2e
@@ -27,6 +29,7 @@ const awaitInterpreter = async (key) => {
 
 export const listener = async (event) => {
     const { type, currentTarget } = event;
+    if (!prefixes.length) return;
     for (let { name, value, ownerElement: el } of $x(
         `./@*[${prefixes.map((p) => `name()="${p}${type}"`).join(' or ')}]`,
         currentTarget,
@@ -45,6 +48,7 @@ export const listener = async (event) => {
  * @param {Document | Element} root
  */
 export const addAllListeners = (root) => {
+    if (!prefixes.length) return;
     for (let { name, ownerElement: el } of $x(
         `.//@*[${prefixes
             .map((p) => `starts-with(name(),"${p}")`)
@@ -59,8 +63,7 @@ export const addAllListeners = (root) => {
             if ('disabled' in el && !el.disabled) {
                 el.disabled = true;
                 // set these to enable once the interpreter is known (registered + loaded)
-                queueMicrotask(async () => {
-                    await awaitInterpreter(name.slice(0, i));
+                env[name.slice(0, i)].then(() => {
                     el.disabled = false;
                 });
             }