feat(os): refactor and improvemnts

.changes/os-plugin-refactor.md

@@ -0,0 +1,11 @@
+---
+"os": minor
+"os-js": minor
+---
+
+The os plugin is recieving a few changes to improve consistency and add new features:
+
+- Renamed `Kind` enum to `OsType` and `kind()` function to `os_type()`.
+- Added `family()`,`exe_extension()`, and `hostname()` functions and their equivalents for JS.
+- Removed `tempdir()` function and its equivalent on JS, use `std::env::temp_dir` instead of `temp_dir` from `tauri::path::PathResolver::temp_dir` and `path.tempDir` on JS.
+- Modified `platform()` implementation to return `windows` instead of `win32` and `macos` instead of `darwin` to align with Rust's `std::env::consts::OS`

Cargo.lock

@@ -206,7 +206,7 @@ checksum = "7de8ce5e0f9f8d88245311066a578d72b7af3e7088f32783804676302df237e4"
 
 [[package]]
 name = "api"
-version = "2.0.0-alpha.0"
+version = "2.0.0-alpha.1"
 dependencies = [
  "log",
  "serde",
@@ -1854,6 +1854,16 @@ dependencies = [
  "winapi",
 ]
 
+[[package]]
+name = "gethostname"
+version = "0.4.3"
+source = "registry+https://github.com/rust-lang/crates.io-index"
+checksum = "0176e0459c2e4a1fe232f984bca6890e681076abb9934f6cea7c326f3fc47818"
+dependencies = [
+ "libc",
+ "windows-targets 0.48.0",
+]
+
 [[package]]
 name = "getrandom"
 version = "0.1.16"
@@ -5263,7 +5273,7 @@ dependencies = [
 
 [[package]]
 name = "tauri-plugin-notification"
-version = "2.0.0-alpha.0"
+version = "2.0.0-alpha.1"
 dependencies = [
  "log",
  "notify-rust",
@@ -5283,6 +5293,7 @@ dependencies = [
 name = "tauri-plugin-os"
 version = "2.0.0-alpha.0"
 dependencies = [
+ "gethostname 0.4.3",
  "log",
  "os_info",
  "serde",
@@ -6886,7 +6897,7 @@ version = "0.10.1"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "592b4883219f345e712b3209c62654ebda0bb50887f330cbd018d0f654bfd507"
 dependencies = [
- "gethostname",
+ "gethostname 0.2.3",
  "nix 0.24.3",
  "winapi",
  "winapi-wsapoll",

plugins/os/Cargo.toml

@@ -14,3 +14,4 @@ log = { workspace = true }
 thiserror = { workspace = true }
 os_info = "3"
 sys-locale = "0.3"
+gethostname = "0.4"

plugins/os/guest-js/index.ts

@@ -16,7 +16,7 @@ declare global {
 
 type Platform =
   | "linux"
-  | "darwin"
+  | "macos"
   | "ios"
   | "freebsd"
   | "dragonfly"
@@ -24,9 +24,9 @@ type Platform =
   | "openbsd"
   | "solaris"
   | "android"
-  | "win32";
+  | "windows";
 
-type OsType = "Linux" | "Darwin" | "Windows_NT";
+type OsType = "linux" | "windows" | "macss" | "ios" | "android";
 
 type Arch =
   | "x86"
@@ -55,8 +55,9 @@ function isWindows(): boolean {
 const EOL = isWindows() ? "\r\n" : "\n";
 
 /**
- * Returns a string identifying the operating system platform.
- * The value is set at compile time. Possible values are `'linux'`, `'darwin'`, `'ios'`, `'freebsd'`, `'dragonfly'`, `'netbsd'`, `'openbsd'`, `'solaris'`, `'android'`, `'win32'`
+ * Returns a string describing the specific operating system in use.
+ * The value is set at compile time. Possible values are `'linux'`, `'macos'`, `'ios'`, `'freebsd'`, `'dragonfly'`, `'netbsd'`, `'openbsd'`, `'solaris'`, `'android'`, `'windows'`
+ *
  * @example
  * ```typescript
  * import { platform } from '@tauri-apps/plugin-os';
@@ -71,7 +72,7 @@ async function platform(): Promise<Platform> {
 }
 
 /**
- * Returns a string identifying the kernel version.
+ * Returns the current operating system version.
  * @example
  * ```typescript
  * import { version } from '@tauri-apps/plugin-os';
@@ -85,7 +86,7 @@ async function version(): Promise<string> {
 }
 
 /**
- * Returns `'Linux'` on Linux, `'Darwin'` on macOS, and `'Windows_NT'` on Windows.
+ * Returns the current operating system type. Returns `'linux'` on Linux, `'macos'` on macOS, `'windows'` on Windows, `'ios'` on iOS and `'android'` on Android.
  * @example
  * ```typescript
  * import { type } from '@tauri-apps/plugin-os';
@@ -95,11 +96,11 @@ async function version(): Promise<string> {
  * @since 2.0.0
  */
 async function type(): Promise<OsType> {
-  return window.__TAURI_INVOKE__("plugin:os|kind");
+  return window.__TAURI_INVOKE__("plugin:os|os_type");
 }
 
 /**
- * Returns the operating system CPU architecture for which the tauri app was compiled.
+ * Returns the current operating system architecture.
  * Possible values are `'x86'`, `'x86_64'`, `'arm'`, `'aarch64'`, `'mips'`, `'mips64'`, `'powerpc'`, `'powerpc64'`, `'riscv64'`, `'s390x'`, `'sparc64'`.
  * @example
  * ```typescript
@@ -114,35 +115,47 @@ async function arch(): Promise<Arch> {
 }
 
 /**
- * Returns the operating system's default directory for temporary files as a string.
+ * Returns a String with a `BCP-47` language tag inside. If the locale couldn’t be obtained, `null` is returned instead.
  * @example
  * ```typescript
- * import { tempdir } from '@tauri-apps/plugin-os';
- * const tempdirPath = await tempdir();
+ * import { locale } from '@tauri-apps/plugin-os';
+ * const locale = await locale();
+ * if (locale) {
+ *    // use the locale string here
+ * }
  * ```
  *
  * @since 2.0.0
  */
-async function tempdir(): Promise<string> {
-  return window.__TAURI_INVOKE__("plugin:os|tempdir");
+async function locale(): Promise<string | null> {
+  return window.__TAURI_INVOKE__("plugin:os|locale");
 }
 
 /**
- * Returns a String with a `BCP-47` language tag inside. If the locale couldn’t be obtained, `null` is returned instead.
+ * Returns the file extension, if any, used for executable binaries on this platform. Possible values are `'exe'` and `''` (empty string).
  * @example
  * ```typescript
- * import { locale } from '@tauri-apps/plugin-os';
- * const locale = await locale();
- * if (locale) {
- *    // use the locale string here
- * }
+ * import { exeExtension } from '@tauri-apps/plugin-os';
+ * const exeExt = await exeExtension();
  * ```
  *
  * @since 2.0.0
  */
-async function locale(): Promise<string | null> {
-  return window.__TAURI_INVOKE__("plugin:os|locale");
+async function exeExtension(): Promise<string | null> {
+  return window.__TAURI_INVOKE__("plugin:os|exe_extension");
+}
+
+/**
+ * Returns the host name of the operating system.
+ * @example
+ * ```typescript
+ * import { hostname } from '@tauri-apps/api/os';
+ * const hostname = await hostname();
+ * ```
+ */
+async function hostname(): Promise<string | null> {
+  return window.__TAURI_INVOKE__("plugin:os|hostname");
 }
 
-export { EOL, platform, version, type, arch, tempdir, locale };
+export { EOL, platform, version, type, arch, locale, exeExtension, hostname };
 export type { Platform, OsType, Arch };

plugins/os/src/commands.rs

@@ -2,8 +2,6 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-License-Identifier: MIT
 
-use std::path::PathBuf;
-
 #[tauri::command]
 pub fn platform() -> &'static str {
     crate::platform()
@@ -15,8 +13,13 @@ pub fn version() -> String {
 }
 
 #[tauri::command]
-pub fn kind() -> String {
-    crate::kind().to_string()
+pub fn os_type() -> String {
+    crate::type_().to_string()
+}
+
+#[tauri::command]
+pub fn family() -> &'static str {
+    crate::family()
 }
 
 #[tauri::command]
@@ -25,11 +28,16 @@ pub fn arch() -> &'static str {
 }
 
 #[tauri::command]
-pub fn tempdir() -> PathBuf {
-    crate::tempdir()
+pub fn exe_extension() -> &'static str {
+    crate::exe_extension()
 }
 
 #[tauri::command]
 pub fn locale() -> Option<String> {
     crate::locale()
 }
+
+#[tauri::command]
+pub fn hostname() -> String {
+    crate::hostname()
+}

plugins/os/src/lib.rs

@@ -2,7 +2,7 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-License-Identifier: MIT
 
-use std::{fmt::Display, path::PathBuf};
+use std::fmt::Display;
 
 pub use os_info::Version;
 use tauri::{
@@ -15,74 +15,93 @@ mod error;
 
 pub use error::Error;
 
-pub enum Kind {
+pub enum OsType {
     Linux,
     Windows,
-    Darwin,
+    Macos,
     IOS,
     Android,
 }
 
-impl Display for Kind {
+impl Display for OsType {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
         match self {
-            Self::Linux => write!(f, "Linux"),
-            Self::Windows => write!(f, "Linux_NT"),
-            Self::Darwin => write!(f, "Darwin"),
-            Self::IOS => write!(f, "iOS"),
-            Self::Android => write!(f, "Android"),
+            Self::Linux => write!(f, "linux"),
+            Self::Windows => write!(f, "windows"),
+            Self::Macos => write!(f, "macos"),
+            Self::IOS => write!(f, "ios"),
+            Self::Android => write!(f, "android"),
         }
     }
 }
 
+/// Returns a string describing the specific operating system in use, see [std::env::consts::OS].
 pub fn platform() -> &'static str {
-    match std::env::consts::OS {
-        "windows" => "win32",
-        "macos" => "darwin",
-        _ => std::env::consts::OS,
-    }
+    std::env::consts::OS
 }
 
+/// Returns the current operating system version.
 pub fn version() -> Version {
     os_info::get().version().clone()
 }
 
-pub fn kind() -> Kind {
-    #[cfg(target_os = "linux")]
-    return Kind::Linux;
+/// Returns the current operating system type.
+pub fn type_() -> OsType {
+    #[cfg(any(
+        target_os = "linux",
+        target_os = "dragonfly",
+        target_os = "freebsd",
+        target_os = "netbsd",
+        target_os = "openbsd"
+    ))]
+    return OsType::Linux;
     #[cfg(target_os = "windows")]
-    return Kind::Windows;
+    return OsType::Windows;
     #[cfg(target_os = "macos")]
-    return Kind::Darwin;
+    return OsType::Macos;
     #[cfg(target_os = "ios")]
-    return Kind::IOS;
+    return OsType::IOS;
     #[cfg(target_os = "android")]
-    return Kind::Android;
+    return OsType::Android;
+}
+
+/// Returns the current operating system family, see [std::env::consts::FAMILY].
+pub fn family() -> &'static str {
+    std::env::consts::FAMILY
 }
 
+/// Returns the current operating system architecture, see [std::env::consts::ARCH].
 pub fn arch() -> &'static str {
     std::env::consts::ARCH
 }
 
-pub fn tempdir() -> PathBuf {
-    std::env::temp_dir()
+/// Returns the file extension, if any, used for executable binaries on this platform. Example value is `exe`, see [std::env::consts::EXE_EXTENSION].
+pub fn exe_extension() -> &'static str {
+    std::env::consts::EXE_EXTENSION
 }
 
-/// Returns the locale with the `BCP-47` language tag. If the locale couldn’t be obtained, `None` is returned instead.
+/// Returns the current operating system locale with the `BCP-47` language tag. If the locale couldn’t be obtained, `None` is returned instead.
 pub fn locale() -> Option<String> {
     sys_locale::get_locale()
 }
 
+/// Returns the current operating system hostname.
+pub fn hostname() -> String {
+    gethostname::gethostname().to_string_lossy().to_string()
+}
+
 pub fn init<R: Runtime>() -> TauriPlugin<R> {
     Builder::new("os")
         .js_init_script(include_str!("api-iife.js").to_string())
         .invoke_handler(tauri::generate_handler![
             commands::platform,
             commands::version,
-            commands::kind,
+            commands::os_type,
+            commands::family,
             commands::arch,
-            commands::tempdir,
-            commands::locale
+            commands::exe_extension,
+            commands::locale,
+            commands::hostname
         ])
         .build()
 }