fix(cli/info): fix crash when checking node version (#9411)

closes #9396
tauri-apps · Apr 15, 2024 · aeddc40 · aeddc40
1 parent fe6f81f
commit aeddc40
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 10 deletions.
diff --git a/.changes/tauri-info-no-node.md b/.changes/tauri-info-no-node.md
@@ -0,0 +1,7 @@
+---
+"tauri-cli": "patch:bug"
+"@tauri-apps/cli": "patch:bug"
+---
+
+Fix `tauri info` crashing when Node.js is not installed.
+
diff --git a/tooling/cli/src/info/env_nodejs.rs b/tooling/cli/src/info/env_nodejs.rs
@@ -40,16 +40,17 @@ pub fn items(metadata: &VersionMetadata) -> Vec<SectionItem> {
               .unwrap_or_default()
               .trim();
             ActionResult::Description(format!("node: {}{}", v, {
-              let version = semver::Version::parse(v).unwrap();
-              let target_version = semver::Version::parse(node_target_ver.as_str()).unwrap();
-              if version < target_version {
-                format!(
-                  " ({}, latest: {})",
-                  "outdated".red(),
-                  target_version.to_string().green()
-                )
-              } else {
-                "".into()
+              let version = semver::Version::parse(v);
+              let target_version = semver::Version::parse(node_target_ver.as_str());
+              match (version, target_version) {
+                (Ok(version), Ok(target_version)) if version < target_version => {
+                  format!(
+                    " ({}, latest: {})",
+                    "outdated".red(),
+                    target_version.to_string().green()
+                  )
+                }
+                _ => "".into(),
               }
             }))
           } else {