marko 4.26.5 → 4.28.0

package/src/core-tags/migrate/all-tags/w-preserve.js

@@ -2,20 +2,16 @@ const newTags = {
   "w-preserve": "no-update",
   "w-preserve-if": "no-update-if",
   "w-preserve-body": "no-update-body",
-  "w-preserve-body-if": "no-update-body-if"
+  "w-preserve-body-if": "no-update-body-if",
 };
-module.exports = function migrate(el, context) {
-  el.forEachAttribute(attr => {
+module.exports = function migrate(el) {
+  el.forEachAttribute((attr) => {
     const name = attr.name;
     if (!name || !Object.keys(newTags).includes(name)) {
       return;
     }
 
     const newAttrName = newTags[name];
-    context.deprecate(
-      `The "${name}" attribute is deprecated. Please use "${newAttrName}" attribute instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes`
-    );
-
     attr.name = newAttrName;
   });
 };

package/src/core-tags/migrate/all-tags/widget-in-attrs.js

@@ -1,5 +1,5 @@
 module.exports = function migrate(el, context) {
-  el.forEachAttribute(attr => {
+  el.forEachAttribute((attr) => {
     const value = attr.value;
 
     if (
@@ -7,31 +7,18 @@ module.exports = function migrate(el, context) {
       el.tagName !== "var" &&
       el.tagName !== "assign"
     ) {
-      context.deprecate(
-        `The "*=widget.elId("someId")" is deprecated. Please use "*:scoped="someId"" modifier instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes`
-      );
-
       attr.name += ":scoped";
       attr.value = value.args[0];
       return;
     }
 
-    let found;
-    const walker = context.createWalker({
+    context.createWalker({
       enter(node) {
         if (isWidgetMemberExpression(node)) {
           node.object.name = "component";
-          found = true;
         }
-      }
-    });
-    walker.walk(value);
-
-    if (found) {
-      context.deprecate(
-        `The "widget" variable is deprecated. Please use "component" instead.`
-      );
-    }
+      },
+    }).walk(value);
   });
 };
 

package/src/core-tags/migrate/all-templates/index.js

@@ -2,10 +2,10 @@ const commonMigrators = [
   require("./non-standard-template-literals"),
   require("./render-calls"),
   require("./widget-data-is-state"),
-  require("./widget-get-template-data")
+  require("./widget-get-template-data"),
 ];
 
 module.exports = function (root, context) {
-  commonMigrators.forEach(migrator => migrator(root, context));
+  commonMigrators.forEach((migrator) => migrator(root, context));
   return true;
 };

package/src/core-tags/migrate/all-templates/non-standard-template-literals.js

@@ -16,7 +16,7 @@ module.exports = function migrator(el, context) {
         } else {
           // For others we keep track of them to run an optional migration below.
           node._originalExpressions = node.expressions;
-          node.expressions = node.expressions.map(expr =>
+          node.expressions = node.expressions.map((expr) =>
             builder.conditionalExpression(
               builder.binaryExpression(expr, "==", builder.literalNull()),
               builder.literal(""),
@@ -26,17 +26,11 @@ module.exports = function migrator(el, context) {
           literals.push(node);
         }
       }
-    }
+    },
   });
 
   walker.walk(el);
 
-  if (found) {
-    context.deprecate(
-      "Non standard template literals have been deprecated, please use javascript template literals instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-nonstandard-template-literals"
-    );
-  }
-
   if (literals.length) {
     context.addMigration({
       description: "Migrate template with non standard template literals",
@@ -46,16 +40,16 @@ module.exports = function migrator(el, context) {
             type: "confirm",
             message:
               "Non standard template literals convert null/undefined/false to empty strings, JavaScript template literals do not. If you are relying on this it may cause issues. Would you like to attempt to migrate?",
-            initial: true
+            initial: true,
           })
-          .then(shouldMigrate => {
+          .then((shouldMigrate) => {
             if (shouldMigrate) {
-              literals.forEach(node => {
+              literals.forEach((node) => {
                 node.expressions = node._originalExpressions;
               });
             }
           });
-      }
+      },
     });
   }
 };

package/src/core-tags/migrate/all-templates/render-calls.js

@@ -22,16 +22,12 @@ module.exports = function migrator(el, context) {
         return;
       }
 
-      context.deprecate(
-        "Directly rendering by passing `out` to a function is deprecated. Please use the dynamic tag instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-imperative-render-calls"
-      );
-
       if (hasErrors) {
         return;
       }
 
       if (!replacement.type) {
-        replacement.forEachChild(child => node.insertSiblingBefore(child));
+        replacement.forEachChild((child) => node.insertSiblingBefore(child));
         node.detach();
       } else {
         node.replaceWith(replacement);
@@ -41,14 +37,14 @@ module.exports = function migrator(el, context) {
         const builder = context.builder;
         if (!node.type) {
           if (node.replaceChild) {
-            node.forEach(child => {
+            node.forEach((child) => {
               const replacement = replaceScriptlets(child, context);
               if (child !== replacement) {
                 node.replaceChild(replacement, child);
               }
             });
           } else if (node.body) {
-            node.body.forEach(child => {
+            node.body.forEach((child) => {
               const replacement = replaceScriptlets(child, context);
               if (child !== replacement) {
                 node.body.replaceChild(replacement, child);
@@ -128,7 +124,7 @@ module.exports = function migrator(el, context) {
 
         return node;
       }
-    }
+    },
   });
   walker.walk(el);
 };

package/src/core-tags/migrate/all-templates/widget-data-is-state.js

@@ -8,9 +8,9 @@ module.exports = function migrator(el, context) {
     apply() {
       context.root.prependChild(
         context.builder.scriptlet({
-          value: "var data = state"
+          value: "var data = state",
         })
       );
-    }
+    },
   });
 };

package/src/core-tags/migrate/all-templates/widget-get-template-data.js

@@ -7,9 +7,9 @@ module.exports = function migrator(el, context) {
     apply() {
       context.root.prependChild(
         context.builder.scriptlet({
-          value: "var data = component.getTemplateData(state, input)"
+          value: "var data = component.getTemplateData(state, input)",
         })
       );
-    }
+    },
   });
 };

package/src/core-tags/migrate/assign-tag.js

@@ -7,10 +7,6 @@ module.exports = function migrator(elNode, context) {
   commonTagMigrator(elNode, context);
   elNode.setTransformerApplied(commonTagMigrator);
 
-  context.deprecate(
-    'The "<assign>" tag is deprecated. Please use "$ <js_code>" for JavaScript in the template. See: https://github.com/marko-js/marko/wiki/Deprecation:-var,-assign,-invoke-tags'
-  );
-
   if (!attributes) {
     context.addError(
       "Invalid <assign> tag. Argument is missing. Example; <assign x=123 />"
@@ -18,13 +14,13 @@ module.exports = function migrator(elNode, context) {
     return elNode;
   }
 
-  elNode.attributes.forEach(attr => {
+  elNode.attributes.forEach((attr) => {
     elNode.insertSiblingBefore(
       builder.scriptlet({
         value:
           attr.value == null
             ? attr.name
-            : `${attr.name} = ${printJS(attr.value, context, null, true)}`
+            : `${attr.name} = ${printJS(attr.value, context, null, true)}`,
       })
     );
   });

package/src/core-tags/migrate/async-fragment-tag.js

@@ -2,16 +2,12 @@ const awaitTagMigration = require("./await-tag");
 const newTags = {
   "async-fragment-placeholder": "await-placeholder",
   "async-fragment-timeout": "await-timeout",
-  "async-fragment-error": "await-error"
+  "async-fragment-error": "await-error",
 };
 module.exports = function migrator(elNode, context) {
   const builder = context.builder;
   const attributes = elNode.attributes;
 
-  context.deprecate(
-    `The "<async-fragment>" tag is deprecated. Please use "<await>" instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-async-fragment`
-  );
-
   if (!attributes || !attributes.length) {
     context.addError(
       'Invalid <async-fragment> tag. Argument is missing. Example; <async-fragment data-provider=data.userInfo var="userInfo" />'
@@ -34,7 +30,7 @@ module.exports = function migrator(elNode, context) {
     return;
   }
 
-  elNode.forEachChild(child => {
+  elNode.forEachChild((child) => {
     const newTag = newTags[child.tagName];
     if (newTag) {
       child.tagName = newTag;

package/src/core-tags/migrate/async-fragments-tag.js

@@ -5,10 +5,6 @@ module.exports = function migrator(elNode, context) {
   commonTagMigrator(elNode, context);
   elNode.setTransformerApplied(commonTagMigrator);
 
-  context.deprecate(
-    `The "<async-fragments>" tag is deprecated. Please use "<await-reorderer>" instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-async-fragment`
-  );
-
   // all this tag ever did was handling of client reordering
   // we'll remove the attribute as that's all this new tag does
   elNode.removeAttribute("client-reorder");

package/src/core-tags/migrate/await-tag.js

@@ -9,10 +9,6 @@ module.exports = function migrator(elNode, context) {
     return;
   }
 
-  context.deprecate(
-    'The "<await(result from promise)>" syntax has been deprecated, please use the modern syntax of "<await(promise)><@then|result|>". See: https://github.com/marko-js/marko/wiki/Deprecation:-legacy-await'
-  );
-
   elNode.argument = undefined;
   let provider = builder.expression(match[2]);
   const varName = match[1];
@@ -20,8 +16,8 @@ module.exports = function migrator(elNode, context) {
   const scopeExpression =
     elNode.getAttributeValue("scope") || (methodExpression ? provider : null);
   const argAssignments = elNode.attributes
-    .filter(attr => attr.name && attr.name.startsWith("arg-"))
-    .map(attr => {
+    .filter((attr) => attr.name && attr.name.startsWith("arg-"))
+    .map((attr) => {
       elNode.removeAttribute(attr.name);
       return [attr.name.slice(4), attr.value];
     });
@@ -50,14 +46,14 @@ module.exports = function migrator(elNode, context) {
         builder.scriptlet({
           value: printJS(
             builder.vars({
-              arg: argExpression
+              arg: argExpression,
             }),
             context
-          )
+          ),
         })
       );
 
-      argAssignments.forEach(assignment => {
+      argAssignments.forEach((assignment) => {
         elNode.insertSiblingBefore(
           builder.scriptlet({
             value: printJS(
@@ -71,7 +67,7 @@ module.exports = function migrator(elNode, context) {
                 "="
               ),
               context
-            )
+            ),
           })
         );
       });
@@ -102,34 +98,23 @@ module.exports = function migrator(elNode, context) {
 
   let placeholderBody;
   if (elNode.hasAttribute("placeholder")) {
-    context.deprecate(
-      'The "placeholder" attribute on the "<await>" tag is deprecated. Please use the "<@placeholder>" nested tag instead.'
-    );
-
     placeholderBody = [builder.text(elNode.getAttributeValue("placeholder"))];
     elNode.removeAttribute("placeholder");
   }
 
   let timeoutBody;
   if (elNode.hasAttribute("timeout-message")) {
-    context.deprecate(
-      'The "timeout-message" attribute on the "<await>" tag is deprecated. Please use the "<@catch|err|>" nested tag instead with a check for "err.name === "TimeoutError".'
-    );
-
     timeoutBody = [builder.text(elNode.getAttributeValue("timeout-message"))];
     elNode.removeAttribute("timeout-message");
   }
 
   let errorBody;
   if (elNode.hasAttribute("error-message")) {
-    context.deprecate(
-      'The "error-message" attribute on the "<await>" tag is deprecated. Please use the "<@catch>" nested tag instead.'
-    );
     errorBody = [builder.text(elNode.getAttributeValue("error-message"))];
     elNode.removeAttribute("error-message");
   }
 
-  elNode.forEachChild(childNode => {
+  elNode.forEachChild((childNode) => {
     if (childNode.type !== "HtmlElement") {
       return;
     }
@@ -202,7 +187,7 @@ module.exports = function migrator(elNode, context) {
         undefined,
         timeoutBody,
         'err.name === "TimeoutError"'
-      )
+      ),
     ];
 
     if (originalErrorBody) {

package/src/core-tags/migrate/class-tag.js

@@ -11,9 +11,5 @@ module.exports = function migrator(el, context) {
     );
   }
 
-  context.deprecate(
-    "Having a named class at the top level of a file is deprecated. Use `class {...}` without a name instead."
-  );
-
   el.tagString = el.tagString.replace(classNamedRegexp, "$1$2");
 };

package/src/core-tags/migrate/component-globals-tag.js

@@ -1,7 +1,3 @@
 module.exports = function render(elNode, context) {
-  context.deprecate(
-    "The <component-globals> tag is deprecated. This functionality has been added to the '<init-components>' tag (which is automatically inserted at the end of the <body> tag), you can safely remove the '<component-globals>' tag."
-  );
-
   elNode.detach();
 };

package/src/core-tags/migrate/for-tag.js

@@ -21,10 +21,6 @@ function migrateForLoop(elNode, context) {
     return;
   }
 
-  context.deprecate(
-    'The "<for(...)>" syntax has been deprecated, checkout the modern syntax here: https://github.com/marko-js/marko/wiki/Deprecation:-legacy-for'
-  );
-
   try {
     parsed = parseFor(elNode.argument);
     elNode.argument = undefined;
@@ -80,10 +76,10 @@ function migrateForLoop(elNode, context) {
                   builder.variableDeclarator(
                     iteratorResultName,
                     builder.arrayExpression([])
-                  )
+                  ),
                 ]),
                 context
-              )
+              ),
             })
           );
 
@@ -102,12 +98,12 @@ function migrateForLoop(elNode, context) {
                           builder.identifier("push")
                         ),
                         [itemName]
-                      )
+                      ),
                     ]
-                  )
+                  ),
                 ]),
                 context
-              )
+              ),
             })
           );
 
@@ -172,7 +168,7 @@ function migrateForLoop(elNode, context) {
                 builder.memberExpression(parsed.in, parsed.nameVarName, true)
               ),
               context
-            )
+            ),
           })
         );
 
@@ -208,7 +204,7 @@ function migrateForLoop(elNode, context) {
           elNode.insertSiblingBefore(
             builder.scriptlet({
               value: printJS(parsed.init, context),
-              block: true
+              block: true,
             })
           );
         }
@@ -217,7 +213,7 @@ function migrateForLoop(elNode, context) {
           elNode.appendChild(
             builder.scriptlet({
               value: printJS(parsed.update, context),
-              block: true
+              block: true,
             })
           );
         }
@@ -329,7 +325,7 @@ function normalizeParts(parsed, builder) {
     varName: varName,
     from: from,
     to: to,
-    step: step
+    step: step,
   };
 }
 
@@ -354,7 +350,7 @@ function replaceStatusVarInBody(el, context, parsed) {
       indexIdentifier,
       "===",
       builder.binaryExpression(lengthIdentifier, "-", builder.literal(1))
-    )
+    ),
   };
 
   const remapKeys = Object.keys(remaps);
@@ -373,7 +369,7 @@ function replaceStatusVarInBody(el, context, parsed) {
     `${statusVarName}\\.(${remapKeys.join("|")})\\(\\)`,
     "g"
   );
-  const remapReplacer = str =>
+  const remapReplacer = (str) =>
     str.replace(remapRegexp, (_, key) => remapStrings[key]);
   const visitor = {
     enter(node) {
@@ -402,7 +398,7 @@ function replaceStatusVarInBody(el, context, parsed) {
           this.skip();
         }
       }
-    }
+    },
   };
 
   context.createWalker(visitor).walk(el.body);

package/src/core-tags/migrate/include-tag.js

@@ -5,9 +5,6 @@ const commonTagMigrator = require("./all-tags");
 module.exports = function migrator(elNode, context) {
   commonTagMigrator(elNode, context);
   elNode.setTransformerApplied(commonTagMigrator);
-  context.deprecate(
-    'The "<include>" tag is deprecated. Please use the "<${dynamic}/>" tag or regular text replacement instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-include-tag'
-  );
 
   const builder = context.builder;
   const rawArg = elNode.argument;
@@ -53,8 +50,8 @@ module.exports = function migrator(elNode, context) {
       elNode.insertSiblingBefore(
         builder.scriptlet({
           value: builder.vars({
-            [identifierName]: target
-          })
+            [identifierName]: target,
+          }),
         })
       );
 

package/src/core-tags/migrate/invoke-tag.js

@@ -8,10 +8,6 @@ module.exports = function migrator(elNode, context) {
   const functionAttr = elNode.attributes[0];
   const functionArgs = functionAttr.argument;
 
-  context.deprecate(
-    'The "<invoke>" tag is deprecated. Please use "$ <js_code>" for JavaScript in the template. See: https://github.com/marko-js/marko/wiki/Deprecation:-var,-assign,-invoke-tags'
-  );
-
   if (!functionAttr) {
     context.addError(
       'Invalid <invoke> tag. Missing function attribute. Expected: <invoke console.log("Hello World")'
@@ -31,14 +27,14 @@ module.exports = function migrator(elNode, context) {
   let replacement = renderCallToDynamicTag(functionAst, context);
 
   if (replacement) {
-    elNode.forEachAttribute(attr => {
+    elNode.forEachAttribute((attr) => {
       if (attr !== functionAttr) {
         replacement.addAttribute(attr);
       }
     });
   } else {
     replacement = builder.scriptlet({
-      value: functionCallExpression
+      value: functionCallExpression,
     });
   }
 

package/src/core-tags/migrate/layout-placeholder-tag.js

@@ -5,9 +5,6 @@ const commonTagMigrator = require("./all-tags");
 module.exports = function render(elNode, context) {
   commonTagMigrator(elNode, context);
   elNode.setTransformerApplied(commonTagMigrator);
-  context.deprecate(
-    "The <layout-placeholder> tag is deprecated. Please use the <${dynamic}> tag instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-layout-tag"
-  );
 
   const builder = context.builder;
   const name = elNode.getAttributeValue("name");

package/src/core-tags/migrate/layout-put-tag.js

@@ -3,9 +3,6 @@ const commonTagMigrator = require("./all-tags");
 module.exports = function migrator(oldNode, context) {
   commonTagMigrator(oldNode, context);
   oldNode.setTransformerApplied(commonTagMigrator);
-  context.deprecate(
-    'The "<layout-put>" tag is deprecated and replaced with first class language support. See: https://github.com/marko-js/marko/wiki/Deprecation:-layout-tag'
-  );
 
   const attributes = oldNode.attributes;
   if (!attributes) {

package/src/core-tags/migrate/layout-use-tag.js

@@ -5,9 +5,6 @@ const commonTagMigrator = require("./all-tags");
 module.exports = function render(elNode, context) {
   commonTagMigrator(elNode, context);
   elNode.setTransformerApplied(commonTagMigrator);
-  context.deprecate(
-    "The <layout-use> tag is deprecated. Please use a combination of the <${dynamic}> tag and imports instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-layout-tag"
-  );
 
   const rawArg = elNode.argument;
   const attributes = elNode.attributes;

package/src/core-tags/migrate/macro-body-tag.js

@@ -1,9 +1,4 @@
 module.exports = function codeGenerator(elNode, context) {
-  context.deprecate(
-    'The "<macro-body>" tag has been deprecated. Please use the <${dynamic}/> tag instead, eg: "<macro|input| ...><${input.renderBody}/></macro>". See: https://github.com/marko-js/marko/wiki/Deprecation:-legacy-macro',
-    elNode
-  );
-
   const builder = context.builder;
   const dynamicTag = builder.htmlElement(undefined, elNode.attributes);
   dynamicTag.rawTagNameExpression = "macroInput";

package/src/core-tags/migrate/macro-tag.js

@@ -8,7 +8,7 @@ module.exports = function migrator(elNode, context) {
   if (
     !attributes.length ||
     attributes.some(
-      attr => attr.name === "name" && attr.value.type === "Literal"
+      (attr) => attr.name === "name" && attr.value.type === "Literal"
     )
   ) {
     return;
@@ -22,11 +22,6 @@ module.exports = function migrator(elNode, context) {
     return elNode;
   }
 
-  context.deprecate(
-    'The "<macro my-macro(input)>" syntax has been deprecated. Please use the new tag param syntax, eg: "<macro|input| name="my-macro">. See: https://github.com/marko-js/marko/wiki/Deprecation:-legacy-macro',
-    elNode
-  );
-
   const name = defAttr.name;
   const params = defAttr.argument
     ? builder.parseJavaScriptParams(defAttr.argument)
@@ -48,7 +43,7 @@ module.exports = function migrator(elNode, context) {
         params.push(builder.identifier("renderBody"));
         macroWalker.stop();
       }
-    }
+    },
   });
 
   macroWalker.walk(elNode);
@@ -56,7 +51,7 @@ module.exports = function migrator(elNode, context) {
   params
     .slice()
     .reverse()
-    .forEach(param => {
+    .forEach((param) => {
       const name = builder.identifier(param.name);
       const value = builder.memberExpression(
         builder.identifier("macroInput"),
@@ -67,7 +62,7 @@ module.exports = function migrator(elNode, context) {
           value: printJS(
             builder.vars([builder.variableDeclarator(name, value)]),
             context
-          )
+          ),
         })
       );
     });
@@ -90,7 +85,7 @@ module.exports = function migrator(elNode, context) {
           child.addAttribute({ name: name, value: value });
         }
       }
-    }
+    },
   });
 
   templateWalker.walk(context.root);

package/src/core-tags/migrate/unless-tag.js

@@ -4,9 +4,6 @@ module.exports = function migrator(oldNode, context) {
   const attributes = oldNode.attributes;
   const argument = oldNode.argument;
 
-  context.deprecate(
-    'The "<unless(x)>" tag is deprecated. Please use "<if(!x)>" instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-unless-tag'
-  );
   if (!argument) {
     context.addError(
       "Invalid <unless> tag. Argument is missing. Example; <unless(foo === true)>"

package/src/core-tags/migrate/util/addIdScopedAttr.js

@@ -15,7 +15,7 @@ module.exports = function addIdScopedAttr(context, el, attrValue) {
           );
         }
       }
-    }
+    },
   });
   walker.walk(templateRoot);
 };