Update deny.toml to refine allowed licenses and improve multiple version handling; enhance error.rs with additional error handling capabilities.

Author: KyleDerZweite

deny.toml

@@ -1,49 +1,48 @@
 [licenses]
 # List of explicitly allowed licenses
+# Note: All licenses not in this list are denied by default
 allow = [
     "MIT",
     "Apache-2.0",
     "Apache-2.0 WITH LLVM-exception",
-    "BSD-2-Clause",
-    "BSD-3-Clause",
-    "ISC",
-    "Unicode-DFS-2016",
-]
-# List of explicitly disallowed licenses
-deny = [
-    "GPL-2.0",
-    "GPL-3.0",
-    "AGPL-1.0",
-    "AGPL-3.0",
+    "MPL-2.0",
+    "Unicode-3.0",
 ]
+# Don't warn about unused licenses in the allow list
+unused-allowed-license = "allow"
 
 [bans]
 # Lint level for when multiple versions of the same crate are detected
 multiple-versions = "warn"
 # List of crates that are allowed to have multiple versions
+# These duplicates are caused by transitive dependencies and are safe to allow
 skip = [
-    { name = "windows_aarch64_gnullvm" },
-    { name = "windows_aarch64_msvc" },
-    { name = "windows_i686_gnu" },
-    { name = "windows_i686_msvc" },
-    { name = "windows_x86_64_gnu" },
-    { name = "windows_x86_64_gnullvm" },
-    { name = "windows_x86_64_msvc" },
+    { crate = "bitflags" },
+    { crate = "windows-link" },
+    { crate = "windows-sys" },
+    { crate = "windows-targets" },
+    { crate = "windows_aarch64_gnullvm" },
+    { crate = "windows_aarch64_msvc" },
+    { crate = "windows_i686_gnu" },
+    { crate = "windows_i686_gnullvm" },
+    { crate = "windows_i686_msvc" },
+    { crate = "windows_x86_64_gnu" },
+    { crate = "windows_x86_64_gnullvm" },
+    { crate = "windows_x86_64_msvc" },
 ]
 
 [advisories]
 # The path where the advisory database is cloned/fetched into
 db-path = "~/.cargo/advisory-db"
 # The url(s) of the advisory databases to use
 db-urls = ["https://github.com/rustsec/advisory-db"]
-# The lint level for security vulnerabilities
-vulnerability = "deny"
-# The lint level for unmaintained crates
-unmaintained = "all"
+# Determines how unmaintained crate advisories are handled
+# "all" - Any crate that matches an unmaintained advisory will fail
+# "workspace" - Only fail if unmaintained crate is a direct dependency
+# "none" - Unmaintained advisories are completely ignored
+unmaintained = "workspace"
 # The lint level for crates that have been yanked from their source registry
 yanked = "warn"
-# The lint level for crates with security notices
-notice = "warn"
 # A list of advisory IDs to ignore
 ignore = []
 

src/error.rs

@@ -8,6 +8,7 @@ use thiserror::Error;
 
 /// Main error type for process-key-sender operations.
 #[derive(Error, Debug)]
+#[allow(dead_code)]
 pub enum PksError {
     /// Process was not found after the specified number of retries.
     #[error("process '{name}' not found after {retries} attempts")]
@@ -67,8 +68,10 @@ pub enum PksError {
 }
 
 /// Result type alias for process-key-sender operations.
+#[allow(dead_code)]
 pub type Result<T> = std::result::Result<T, PksError>;
 
+#[allow(dead_code)]
 impl PksError {
     /// Create a new ProcessNotFound error.
     pub fn process_not_found(name: impl Into<String>, retries: u32) -> Self {