feat: add secrets generator to the Python SDK (#1344)

## 🎟️ Tracking

https://bitwarden.atlassian.net/browse/SM-1603

## 📔 Objective

Add a secret generator to the Python SDK. 

## ⏰ Reminders before review

- Contributor guidelines followed
- All formatters and local linters executed and passed
- Written new unit and / or integration tests where applicable
- Protected functional changes with optionality (feature flags)
- Used internationalization (i18n) for all UI strings
- CI builds passed
- Communicated to DevOps any deployment requirements
- Updated any necessary documentation (Confluence, contributing docs) or
informed the documentation
  team

## 🦮 Reviewer guidelines

<!-- Suggested interactions but feel free to use (or not) as you desire!
-->

- 👍 (`:+1:`) or similar for great changes
- 📝 (`:memo:`) or ℹ️ (`:information_source:`) for notes or general info
- ❓ (`:question:`) for questions
- 🤔 (`:thinking:`) or 💭 (`:thought_balloon:`) for more open inquiry
that's not quite a confirmed
  issue and could potentially benefit from discussion
- 🎨 (`:art:`) for suggestions / improvements
- ❌ (`:x:`) or ⚠️ (`:warning:`) for more significant problems or
concerns needing attention
- 🌱 (`:seedling:`) or ♻️ (`:recycle:`) for future improvements or
indications of technical debt
- ⛏ (`:pick:`) for minor or nitpick changes

Author: tangowithfoxtrot

Cargo.toml

@@ -7,7 +7,7 @@ members = ["crates/*"]
 # Update using `cargo set-version -p bitwarden <new-version>`
 version = "1.0.0"
 authors = ["Bitwarden Inc"]
-edition = "2021"
+edition = "2024"
 # Note: Changing rust-version should be considered a breaking change
 rust-version = "1.85"
 homepage = "https://bitwarden.com"

crates/bitwarden-c/src/c.rs

@@ -17,7 +17,7 @@ pub struct CClient {
     client: Client,
 }
 
-#[no_mangle]
+#[unsafe(no_mangle)]
 pub extern "C" fn run_command(c_str_ptr: *const c_char, client_ptr: *const CClient) -> *mut c_char {
     let client = unsafe { ffi_ref!(client_ptr) };
     let input_str = str::from_utf8(unsafe { CStr::from_ptr(c_str_ptr) }.to_bytes())
@@ -35,7 +35,7 @@ pub extern "C" fn run_command(c_str_ptr: *const c_char, client_ptr: *const CClie
 
 type OnCompletedCallback = unsafe extern "C" fn(result: *mut c_char) -> ();
 
-#[no_mangle]
+#[unsafe(no_mangle)]
 pub extern "C" fn run_command_async(
     c_str_ptr: *const c_char,
     client_ptr: *const CClient,
@@ -74,7 +74,7 @@ pub extern "C" fn run_command_async(
 }
 
 // Init client, potential leak! You need to call free_mem after this!
-#[no_mangle]
+#[unsafe(no_mangle)]
 pub extern "C" fn init(c_str_ptr: *const c_char) -> *mut CClient {
     // This will only fail if another logger was already initialized, so we can ignore the result
     let _ = env_logger::try_init();
@@ -97,19 +97,19 @@ pub extern "C" fn init(c_str_ptr: *const c_char) -> *mut CClient {
 }
 
 // Free mem
-#[no_mangle]
+#[unsafe(no_mangle)]
 pub extern "C" fn free_mem(client_ptr: *mut CClient) {
     std::mem::drop(unsafe { Box::from_raw(client_ptr) });
 }
 
-#[no_mangle]
+#[unsafe(no_mangle)]
 pub extern "C" fn abort_and_free_handle(join_handle_ptr: *mut tokio::task::JoinHandle<()>) {
     let join_handle = unsafe { Box::from_raw(join_handle_ptr) };
     join_handle.abort();
     std::mem::drop(join_handle);
 }
 
-#[no_mangle]
+#[unsafe(no_mangle)]
 pub extern "C" fn free_handle(join_handle_ptr: *mut tokio::task::JoinHandle<()>) {
     std::mem::drop(unsafe { Box::from_raw(join_handle_ptr) });
 }

crates/bitwarden-json/src/client.rs

@@ -25,7 +25,7 @@ impl Client {
         let mut cmd_value: serde_json::Value = match serde_json::from_str(input_str) {
             Ok(cmd) => cmd,
             Err(e) => {
-                return Response::error(format!("Invalid command string: {}", e)).into_string()
+                return Response::error(format!("Invalid command string: {}", e)).into_string();
             }
         };
 
@@ -45,7 +45,7 @@ impl Client {
         let cmd: Command = match serde_json::from_value(cmd_value) {
             Ok(cmd) => cmd,
             Err(e) => {
-                return Response::error(format!("Invalid command value: {}", e)).into_string()
+                return Response::error(format!("Invalid command value: {}", e)).into_string();
             }
         };
 

crates/bitwarden-wasm/src/client.rs

@@ -4,7 +4,7 @@ use std::rc::Rc;
 use argon2::{Algorithm, Argon2, Params, Version};
 use bitwarden_json::client::Client as JsonClient;
 use js_sys::Promise;
-use log::{set_max_level, Level};
+use log::{Level, set_max_level};
 use wasm_bindgen::prelude::*;
 use wasm_bindgen_futures::future_to_promise;
 

crates/bws/src/cli.rs

@@ -1,7 +1,7 @@
 use std::path::PathBuf;
 
 use bitwarden_cli::Color;
-use clap::{builder::ValueParser, ArgGroup, Parser, Subcommand, ValueEnum};
+use clap::{ArgGroup, Parser, Subcommand, ValueEnum, builder::ValueParser};
 use clap_complete::Shell;
 use uuid::Uuid;
 

crates/bws/src/command/mod.rs

@@ -7,9 +7,9 @@ use std::{path::PathBuf, str::FromStr};
 use bitwarden::auth::AccessToken;
 use clap::CommandFactory;
 use clap_complete::Shell;
-use color_eyre::eyre::{bail, Result};
+use color_eyre::eyre::{Result, bail};
 
-use crate::{config, util, Cli, ProfileKey};
+use crate::{Cli, ProfileKey, config, util};
 
 pub(crate) fn completions(shell: Option<Shell>) -> Result<()> {
     let Some(shell) = shell.or_else(Shell::from_env) else {

crates/bws/src/command/project.rs

@@ -1,19 +1,19 @@
 use bitwarden::{
+    Client,
     secrets_manager::{
+        ClientProjectsExt,
         projects::{
             ProjectCreateRequest, ProjectGetRequest, ProjectPutRequest, ProjectsDeleteRequest,
             ProjectsListRequest,
         },
-        ClientProjectsExt,
     },
-    Client,
 };
-use color_eyre::eyre::{bail, Result};
+use color_eyre::eyre::{Result, bail};
 use uuid::Uuid;
 
 use crate::{
-    render::{serialize_response, OutputSettings},
     ProjectCommand,
+    render::{OutputSettings, serialize_response},
 };
 
 pub(crate) async fn process_command(

crates/bws/src/command/run.rs

@@ -5,20 +5,20 @@ use std::{
 };
 
 use bitwarden::{
+    Client,
     secrets_manager::{
-        secrets::{SecretIdentifiersByProjectRequest, SecretIdentifiersRequest, SecretsGetRequest},
         ClientSecretsExt,
+        secrets::{SecretIdentifiersByProjectRequest, SecretIdentifiersRequest, SecretsGetRequest},
     },
-    Client,
 };
-use color_eyre::eyre::{bail, Result};
+use color_eyre::eyre::{Result, bail};
 use itertools::Itertools;
 use uuid::Uuid;
 use which::which;
 
 use crate::{
-    util::{is_valid_posix_name, uuid_to_posix},
     ACCESS_TOKEN_KEY_VAR_NAME,
+    util::{is_valid_posix_name, uuid_to_posix},
 };
 
 // Essential environment variables that should be preserved even when `--no-inherit-env` is used
@@ -80,7 +80,10 @@ pub(crate) async fn run(
 
     if !uuids_as_keynames {
         if let Some(duplicate) = secrets.iter().map(|s| &s.key).duplicates().next() {
-            bail!("Multiple secrets with name: '{}'. Use --uuids-as-keynames or use unique names for secrets", duplicate);
+            bail!(
+                "Multiple secrets with name: '{}'. Use --uuids-as-keynames or use unique names for secrets",
+                duplicate
+            );
         }
     }
 

crates/bws/src/command/secret.rs

@@ -1,20 +1,20 @@
 use bitwarden::{
+    Client,
     secrets_manager::{
+        ClientSecretsExt,
         secrets::{
             SecretCreateRequest, SecretGetRequest, SecretIdentifiersByProjectRequest,
             SecretIdentifiersRequest, SecretPutRequest, SecretResponse, SecretsDeleteRequest,
             SecretsGetRequest,
         },
-        ClientSecretsExt,
     },
-    Client,
 };
-use color_eyre::eyre::{bail, Result};
+use color_eyre::eyre::{Result, bail};
 use uuid::Uuid;
 
 use crate::{
-    render::{serialize_response, OutputSettings},
     SecretCommand,
+    render::{OutputSettings, serialize_response},
 };
 
 #[derive(Debug)]

crates/bws/src/config.rs

@@ -4,11 +4,11 @@ use std::{
     path::{Path, PathBuf},
 };
 
-use color_eyre::eyre::{bail, Result};
+use color_eyre::eyre::{Result, bail};
 use directories::BaseDirs;
 use serde::{Deserialize, Serialize};
 
-use crate::cli::{ProfileKey, DEFAULT_CONFIG_DIRECTORY, DEFAULT_CONFIG_FILENAME};
+use crate::cli::{DEFAULT_CONFIG_DIRECTORY, DEFAULT_CONFIG_FILENAME, ProfileKey};
 
 #[derive(Debug, Serialize, Deserialize, Default)]
 pub(crate) struct Config {