marko 4.26.3 → 4.27.0

package/src/core-tags/core/core-transformer.js

@@ -5,8 +5,8 @@ var coreAttrHandlers = [
     "marko-preserve-whitespace",
     function (attr, node, el) {
       el.setPreserveWhitespace(true);
-    }
-  ]
+    },
+  ],
 ];
 
 class AttributeTransformer {
@@ -19,7 +19,7 @@ class AttributeTransformer {
   addError(message) {
     this.context.addError({
       node: this.el,
-      message: message
+      message: message,
     });
   }
 }
@@ -47,7 +47,7 @@ module.exports = function transform(el, context) {
         params: el.params,
         openTagOnly: el.openTagOnly,
         selfClosed: el.selfClosed,
-        body: el.body
+        body: el.body,
       }))
     );
   }
@@ -55,7 +55,7 @@ module.exports = function transform(el, context) {
   var attributeTransfomer;
   var node = el;
 
-  el.forEachAttribute(attr => {
+  el.forEachAttribute((attr) => {
     let attrName = attr.name;
     var attrTransformerFunc = attributeTransformers[attrName];
     if (attrTransformerFunc) {

package/src/core-tags/core/html-comment-tag.js

@@ -12,7 +12,7 @@ module.exports = function codeGenerator(elNode, codegen) {
       Array.from(body.slice(1)).reduce((expr, current) => {
         return builder.binaryExpression(expr, "+", current.argument);
       }, body[0].argument),
-      builder.identifier("component")
+      builder.identifier("component"),
     ]
   );
 };

package/src/core-tags/core/import-tag.js

@@ -9,7 +9,7 @@ module.exports = function codeGenerator(el, codegen) {
   var args = parseImport(el.tagString);
   var vars = {};
 
-  args.forEach(arg => {
+  args.forEach((arg) => {
     var varName = arg.name;
 
     if (arg.value[0] === "<") {

package/src/core-tags/core/util/parseImport.js

@@ -4,13 +4,13 @@ function getSpecifiers(importDeclaration) {
   var match = /^([\s\S]+)\bfrom\s*(("|')(.*?)("|'))$/.exec(importDeclaration);
   if (!match) {
     return {
-      moduleSpecifier: importDeclaration.replace(/"|'/g, "").trim()
+      moduleSpecifier: importDeclaration.replace(/"|'/g, "").trim(),
     };
   }
 
   return {
     importSpecifierSet: match[1].trim(),
-    moduleSpecifier: match[4].trim()
+    moduleSpecifier: match[4].trim(),
   };
 }
 
@@ -23,12 +23,12 @@ function getImportSpecifierGroups(importSpecifierSet) {
     defaultImport = defaultImport.replace(decomposedImports[0], "");
     decomposedImports = decomposedImports[1]
       .split(",")
-      .map(specifier => specifier.trim());
+      .map((specifier) => specifier.trim());
   }
 
   return {
     defaultImport: defaultImport,
-    decomposedImports: decomposedImports
+    decomposedImports: decomposedImports,
   };
 }
 
@@ -51,7 +51,7 @@ function getNames(importSpecifier) {
 
   return {
     exported: names[0].trim(),
-    local: names[1].trim()
+    local: names[1].trim(),
   };
 }
 
@@ -71,13 +71,13 @@ module.exports = function importToAssignments(tagString) {
     getVariableName(moduleSpecifier);
   var specifierList = importGroups.decomposedImports
     .map(getNames)
-    .map(names => {
+    .map((names) => {
       return {
         name: names.local,
         value: {
           object: rootVariable,
-          property: names.exported
-        }
+          property: names.exported,
+        },
       };
     });
 

package/src/core-tags/html/normalize-script-text.js

@@ -1,6 +1,6 @@
 module.exports = function codeGenerator(elNode, codegen) {
   const builder = codegen.builder;
-  const body = elNode.body.array.map(text => text.argument);
+  const body = elNode.body.array.map((text) => text.argument);
   if (body.length > 1) {
     const quasis = [];
     const expressions = [];

package/src/core-tags/migrate/all-tags/body-only-if.js

@@ -4,15 +4,14 @@ module.exports = function migrator(node, context) {
   const arg = node.getAttribute("body-only-if");
 
   if (arg) {
-    context.deprecate(
-      'The "body-only-if(x)" tag is deprecated. Please use "<${test ? null : tag>" instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-body‐only‐if'
-    );
     // Check if body-only-if(true) is set and automatically unwrap element
     if (arg.argument === "true") {
       // w-body will take care of the unwrapping
       if (!node.getAttribute("w-body")) {
         // Unwrap if there is no w-body tag
-        node.forEachChild(currentNode => node.insertSiblingBefore(currentNode));
+        node.forEachChild((currentNode) =>
+          node.insertSiblingBefore(currentNode)
+        );
         node.detach();
       }
       return;
@@ -34,7 +33,7 @@ module.exports = function migrator(node, context) {
 
     if (!tagDef.html && tagDef.renderer) {
       tagName = builder.objectExpression({
-        renderer: builder.identifier(tagName)
+        renderer: builder.identifier(tagName),
       });
     }
 

package/src/core-tags/migrate/all-tags/control-flow-directives.js

@@ -4,22 +4,19 @@ const CONTROL_FLOW_ATTRIBUTES = [
   "if",
   "unless",
   "else-if",
-  "else"
+  "else",
 ];
 
 module.exports = function migrate(el, context) {
   const builder = context.builder;
 
-  el.forEachAttribute(attr => {
+  el.forEachAttribute((attr) => {
     const name = attr.name;
     if (
       CONTROL_FLOW_ATTRIBUTES.includes(name) &&
       (name === "else" || attr.argument) &&
       !(el.tagName === "else" && name === "if") // <else if(x)> gets passed through
     ) {
-      context.deprecate(
-        `The "${name}" attribute is deprecated. Please use the <${name}> tag instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-control-flow-attributes`
-      );
       el.removeAttribute(name);
       el.wrapWith(
         builder.htmlElement(name, undefined, undefined, attr.argument)

package/src/core-tags/migrate/all-tags/dynamic-attributes.js

@@ -1,9 +1,6 @@
-module.exports = function migrate(el, context) {
+module.exports = function migrate(el) {
   el.forEachAttribute((attr, index) => {
     if (!attr.name && !attr.spread) {
-      context.deprecate(
-        'The "${attributes}" is deprecated. Please use "...attributes" modifier instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes'
-      );
       const attribute = el.attributes.splice(index, 1)[0];
       attribute.spread = true;
       el.attributes.unshift(attribute);

package/src/core-tags/migrate/all-tags/include-directive.js

@@ -6,10 +6,6 @@ module.exports = function migrate(el, context) {
     return;
   }
 
-  context.deprecate(
-    'The "include" attribute is deprecated. Please use the <${dynamic}> tag instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-include-tag'
-  );
-
   el.removeAttribute("include");
   el.appendChild(
     builder.htmlElement("include", undefined, undefined, attr.argument)

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

@@ -17,13 +17,13 @@ const commonMigrators = [
   require("./w-on"),
   require("./w-preserve"),
   require("./w-preserve-attrs"),
-  require("./widget-in-attrs")
+  require("./widget-in-attrs"),
 ];
 
 module.exports = function (el, context) {
   if (el.detachNode) {
     return false;
   }
-  commonMigrators.forEach(migrator => migrator(el, context));
+  commonMigrators.forEach((migrator) => migrator(el, context));
   return true;
 };

package/src/core-tags/migrate/all-tags/legacy-nested-tag.js

@@ -12,9 +12,4 @@ module.exports = function migrate(el, context) {
 
   const nestedTagName = match[1];
   el.tagName = `@${nestedTagName}`;
-
-  context.deprecate(
-    'The "<my-tag:nested>" tagName syntax is deprecated. Please use the "<@nested>" tagName syntax instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-legacy-nested-tags',
-    el
-  );
 };

package/src/core-tags/migrate/all-tags/marko-init.js

@@ -5,7 +5,7 @@ const SAFE_TO_PRINT_ON_ONE_LINE = [
   "ElseIf",
   "ForStatement",
   "WhileStatement",
-  "FunctionDeclaration"
+  "FunctionDeclaration",
 ];
 
 module.exports = function migrate(el, context) {
@@ -17,9 +17,6 @@ module.exports = function migrate(el, context) {
       return;
     }
 
-    context.deprecate(
-      'The "marko-init" attribute is deprecated.  Use the static tag instead. See https://github.com/marko-js/marko/wiki/Deprecation:-script-template-helpers'
-    );
     el.removeAttribute("marko-init");
   } else if (el.hasAttribute("template-helpers")) {
     if (el.tagName !== "script") {
@@ -29,9 +26,6 @@ module.exports = function migrate(el, context) {
       return;
     }
 
-    context.deprecate(
-      'The "template-helpers" attribute is deprecated and will be removed in the next release candidate. Use the static tag instead. See https://github.com/marko-js/marko/wiki/Deprecation:-script-template-helpers'
-    );
     el.removeAttribute("template-helpers");
   } else {
     return;
@@ -50,7 +44,7 @@ module.exports = function migrate(el, context) {
     if (!child.type) {
       child.forEachChild(insertStaticOrImport);
     } else if (child.type === "Vars") {
-      child.declarations.forEach(declaration => {
+      child.declarations.forEach((declaration) => {
         const id = declaration.id;
         const init = declaration.init;
         el.insertSiblingBefore(

package/src/core-tags/migrate/all-tags/modifier-key.js

@@ -2,23 +2,17 @@ const addIdScopedAttr = require("../util/addIdScopedAttr");
 const componentElId = /^component\.(?:getE|e)lId\(.*\)$/;
 
 module.exports = function migrate(el, context) {
-  el.forEachAttribute(attr => {
+  el.forEachAttribute((attr) => {
     let name = attr.name;
     let value = attr.value;
 
     if (name && name.endsWith(":key")) {
-      context.deprecate(
-        `The ":key" modifier is deprecated. Please use ":scoped" modifier instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes`
-      );
       name = name.slice(0, 0 - ":key".length) + ":scoped";
     } else if (
       value &&
       value.type === "FunctionCall" &&
       componentElId.test(attr.rawValue)
     ) {
-      context.deprecate(
-        `Using component.${value.callee.property.name} as an attribute is deprecated. Please use ":scoped" modifier instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes`
-      );
       name += ":scoped";
       value = value.args[0];
     } else {

package/src/core-tags/migrate/all-tags/params.js

@@ -17,7 +17,7 @@ function enableTagParams(el, context) {
     el.params = context.builder.parseJavaScriptParams(el.argument);
     delete el.argument;
   }
-  el.forEachChild(childNode => {
+  el.forEachChild((childNode) => {
     if (isNestedTag(childNode)) {
       enableTagParams(childNode, context);
     }

package/src/core-tags/migrate/all-tags/ref.js

@@ -1,13 +1,9 @@
-module.exports = function migrate(el, context) {
+module.exports = function migrate(el) {
   const attr = el.getAttribute("ref");
   if (!attr) {
     return;
   }
 
-  context.deprecate(
-    `The "ref" attribute is deprecated. Please use "key" attribute instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-ref-attribute`
-  );
-
   el.setAttributeValue("key", attr.value);
   el.removeAttribute(attr.name);
 };

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

@@ -14,10 +14,6 @@ module.exports = function migrate(el, context) {
 
   context.setMeta("legacy", true);
 
-  context.deprecate(
-    "Legacy components using w-bind and defineRenderer/defineComponent or defineComponent are deprecated. See: https://github.com/marko-js/marko/issues/421"
-  );
-
   if (value == null) {
     componentModule = getDefaultWidgetFile(context.dirname);
 
@@ -54,7 +50,7 @@ module.exports = function migrate(el, context) {
     componentModule = {
       legacy: true,
       filename,
-      requirePath: literalValue
+      requirePath: literalValue,
     };
   }
 
@@ -80,18 +76,18 @@ module.exports = function migrate(el, context) {
           message:
             "A widget file was discovered, would you like to migrate that as well?\n" +
             "Note: widget migrations are not 100% safe and should be tested after migration.",
-          initial: true
+          initial: true,
         })
-        .then(shouldMigrate => {
+        .then((shouldMigrate) => {
           if (shouldMigrate) {
             el.removeAttribute("w-bind");
             return helpers.run("componentFile", {
               templateFile: context.filename,
-              componentFile: componentModule.filename
+              componentFile: componentModule.filename,
             });
           }
         });
-    }
+    },
   });
 };
 
@@ -103,19 +99,19 @@ function getDefaultWidgetFile(dirname) {
     return {
       filename,
       requirePath: "./widget",
-      legacy
+      legacy,
     };
   } else if ((filename = resolveFrom(dirname, "./component"))) {
     return {
       filename,
       requirePath: "./component",
-      legacy
+      legacy,
     };
   } else if ((filename = resolveFrom(dirname, "./"))) {
     return {
       filename,
       requirePath: "./",
-      legacy
+      legacy,
     };
   } else {
     return null;

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

@@ -5,10 +5,6 @@ module.exports = function migrate(el, context) {
     return;
   }
 
-  context.deprecate(
-    'The "w-body" attribute is deprecated. Please use the "<${dynamicTag}/>" tag instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-Widget-body-(w‐body)'
-  );
-
   const builder = context.builder;
   const defaultValue = builder.identifier("input");
   const bodyValue = el.getAttributeValue("w-body") || defaultValue;
@@ -44,15 +40,15 @@ module.exports = function migrate(el, context) {
       )
     ),
     builder.htmlElement("else", undefined, [
-      builder.text(isDefault ? renderBodyValue : bodyValue)
-    ])
+      builder.text(isDefault ? renderBodyValue : bodyValue),
+    ]),
   ]);
 
   if (
     el.hasAttribute("body-only-if") &&
     el.getAttribute("body-only-if").argument === "true"
   ) {
-    el.forEachChild(node => el.insertSiblingBefore(node));
+    el.forEachChild((node) => el.insertSiblingBefore(node));
     el.detach();
   }
 };

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

@@ -7,10 +7,6 @@ module.exports = function migrate(el, context) {
     return;
   }
 
-  context.deprecate(
-    'The "w-config" attribute is deprecated. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐config'
-  );
-
   if (attr.value) {
     el.insertSiblingBefore(
       builder.scriptlet({
@@ -24,7 +20,7 @@ module.exports = function migrate(el, context) {
             "="
           ),
           context
-        )
+        ),
       })
     );
   }

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

@@ -7,7 +7,7 @@ module.exports = function migrate(el, context) {
     el.hasAttribute("for-key") ||
     el.hasAttribute("for-ref")
   ) {
-    el.forEachAttribute(attr => {
+    el.forEachAttribute((attr) => {
       const name = attr.name;
       if (
         !name ||
@@ -17,14 +17,7 @@ module.exports = function migrate(el, context) {
       }
 
       if (name === "w-for" && !findBoundParent(el)) {
-        context.deprecate(
-          `Using "w-for" in a template without a "w-bind" is deprecated. The "${name}" attribute is also deprecated. Please use "for:scoped" instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes`
-        );
         context.setMigrationFlag("legacyWidgetAttrsWithoutBind");
-      } else {
-        context.deprecate(
-          `The "${name}" attribute is deprecated. Please use "for:scoped" instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes`
-        );
       }
 
       if (el.hasAttribute("for:scoped")) {

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

@@ -6,14 +6,7 @@ module.exports = function migrate(el, context) {
     return;
   }
 
-  if (findBoundParent(el)) {
-    context.deprecate(
-      `The "w-id" attribute is deprecated. Please use "key" attribute instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes`
-    );
-  } else {
-    context.deprecate(
-      `Using "w-id" in a template without a "w-bind" is deprecated. The "w-id" attribute is also deprecated. Please use "key" attribute instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes`
-    );
+  if (!findBoundParent(el)) {
     context.setMigrationFlag("legacyWidgetAttrsWithoutBind");
   }
 

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

@@ -2,20 +2,13 @@ const printJS = require("../util/printJS");
 const findBoundParent = require("../util/findBoundParent");
 
 module.exports = function migrate(el, context) {
-  el.forEachAttribute(attr => {
+  el.forEachAttribute((attr) => {
     let name = attr.name;
     if (!name || !name.startsWith("w-on")) {
       return;
     }
 
-    if (findBoundParent(el)) {
-      context.deprecate(
-        `The "w-on*" attributes are deprecated. Please use "on*()" instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes`
-      );
-    } else {
-      context.deprecate(
-        `Using "w-on*" in a template without a "w-bind" is deprecated. The "w-on*" attributes are also deprecated. Please use "on*()" instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes`
-      );
+    if (!findBoundParent(el)) {
       context.setMigrationFlag("legacyWidgetAttrsWithoutBind");
     }
 

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

@@ -5,11 +5,7 @@ module.exports = function migrate(el, context) {
   }
 
   const values = attr.value.value;
-  context.deprecate(
-    `The "w-preserve-attrs" attribute is deprecated. Please use ":no-update" modifier instead. See: https://github.com/marko-js/marko/wiki/Deprecation:-w‐*-Attributes`
-  );
-
-  values.split(",").forEach(val => {
+  values.split(",").forEach((val) => {
     const existingAttr = el.getAttribute(val);
 
     if (existingAttr) {

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;
               });
             }
           });
-      }
+      },
     });
   }
 };