@extrahorizon/exh-cli 1.12.0-dev-153-e5a1b1d → 1.12.0-dev-155-01e75cd

package/build/services/templates/util/buildTemplates.js

@@ -35,8 +35,8 @@ async function buildTemplate(name, templateFilesByName, callChain = []) {
     else {
         templateConfig = { ...templateFile, name };
     }
-    if (!(0, utils_1.isV1Template)(templateConfig) && Object.keys(templateConfig.outputs || {}).length < 1) {
-        throw new Error(`Template '${name}' must have at least one output defined!`);
+    if (!(0, utils_1.isV1Template)(templateConfig)) {
+        validateV2Template(templateConfig);
     }
     return templateConfig;
 }
@@ -132,8 +132,21 @@ function resolveVariableValue(value) {
     }
     return resolved;
 }
+function validateV2Template(template) {
+    if (!/^[A-Za-z][A-Za-z0-9_-]{0,49}$/.test(template.name)) {
+        throw new Error(`Template name '${template.name}' is invalid. Template names must start with a letter and can only contain letters, numbers, underscores and hyphens, and be at most 50 characters long.`);
+    }
+    if (Object.keys(template.outputs || {}).length < 1) {
+        throw new Error(`Template '${template.name}' must have at least one output defined.`);
+    }
+    for (const outputName of Object.keys(template.outputs)) {
+        if (!/^[A-Za-z][A-Za-z0-9_-]{0,49}$/.test(outputName)) {
+            throw new Error(`Output name '${outputName}' is invalid. Output names must start with a letter and can only contain letters, numbers, underscores and hyphens, and be at most 50 characters long.`);
+        }
+    }
+}
 function v1ExtendingV2Error(extendsTemplate, callChain) {
-    return new Error(`You cannot extend a v2 template ('${extendsTemplate}') in a v1 template!` +
+    return new Error(`You cannot extend a v2 template ('${extendsTemplate}') in a v1 template.` +
         `In ${renderCallChain(callChain)}`);
 }
 function v1VariableNotFoundError(variableName, extendsTemplate, callChain) {
@@ -142,7 +155,7 @@ function v1VariableNotFoundError(variableName, extendsTemplate, callChain) {
         `in ${renderCallChain(callChain)}`);
 }
 function v2ExtendingV1Error(extendsTemplate, callChain) {
-    return new Error(`You cannot extend a v1 template ('${extendsTemplate}') in a v2 template!` +
+    return new Error(`You cannot extend a v1 template ('${extendsTemplate}') in a v2 template.` +
         `In ${renderCallChain(callChain)}`);
 }
 function v2VariableNotFoundError(variableName, extendsTemplate, callChain) {

package/build/services/templates/util/uploadTemplate.js

@@ -38,7 +38,8 @@ async function uploadV2Template(template) {
         }
         else {
             console.log(`Updating existing template '${template.name}'`);
-            await templateV2Repository.update(existingTemplate.id, template);
+            const update = { description: null, inputs: null, ...template };
+            await templateV2Repository.update(existingTemplate.id, update);
         }
     }
     catch (err) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@extrahorizon/exh-cli",
-  "version": "1.12.0-dev-153-e5a1b1d",
+  "version": "1.12.0-dev-155-01e75cd",
   "main": "build/index.js",
   "exports": "./build/index.js",
   "license": "MIT",