fix(js): adjust minimal publish script to display better error for ve…

…rsion validation
nrwl · Apr 12, 2022 · cbfe573 · cbfe573
1 parent 2193c1f
commit cbfe573
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/packages/js/src/utils/minimal-publish-script.ts b/packages/js/src/utils/minimal-publish-script.ts
@@ -21,21 +21,26 @@ const [, , name, version, tag = 'next'] = process.argv;
 // A simple SemVer validation to validate the version 
 const validVersion = /^\\d+\\.\\d+\\.\\d(-\\w+\\.\\d+)?/;
 if (!version || !validVersion.test(version)) {
-console.error(chalk.red(\`Invalid version\`));
-process.exit(1);
+  console.error(chalk.bold.red(\`No version provided or version did not match Semantic Versioning, expected: #.#.#-tag, got \${version}\`));
+  process.exit(1);
 }
 
 // Updating the version in "package.json" before publishing
-const json = JSON.parse(readFileSync(\`package.json\`).toString());
-json.version = version;
-writeFileSync(\`package.json\`, JSON.stringify(json, null, 2));
+try {
+  const json = JSON.parse(readFileSync(\`package.json\`).toString());
+  json.version = version;
+  writeFileSync(\`package.json\`, JSON.stringify(json, null, 2));
+} catch (e) {
+  console.error(chalk.bold.red(\`Error reading package.json file from library build output.\`))
+}
 
 // Execute "npm publish" to publish
 execSync(\`npm publish --access public\`);
 `;
 
 export function addMinimalPublishScript(tree: Tree) {
   const publishScriptPath = 'tools/scripts/publish.mjs';
+
   if (!tree.exists(publishScriptPath)) {
     tree.write(publishScriptPath, publishScriptContent);
   }