SM-807: BWS CLI & SDK Updates

crates/bitwarden-napi/src-ts/bitwarden_client/schemas.ts

@@ -351,6 +351,7 @@ export interface SecretPutRequest {
      * Organization ID of the secret to modify
      */
     organizationId: string;
+    projectIds?:    string[] | null;
     value:          string;
 }
 
@@ -923,6 +924,7 @@ const typeMap: any = {
         { json: "key", js: "key", typ: "" },
         { json: "note", js: "note", typ: "" },
         { json: "organizationId", js: "organizationId", typ: "" },
+        { json: "projectIds", js: "projectIds", typ: u(undefined, u(a(""), null)) },
         { json: "value", js: "value", typ: "" },
     ], false),
     "SyncRequest": o([

crates/bitwarden/src/secrets_manager/secrets/update.rs

@@ -20,6 +20,7 @@ pub struct SecretPutRequest {
     pub key: String,
     pub value: String,
     pub note: String,
+    pub project_ids: Option<Vec<Uuid>>,
 }
 
 pub(crate) async fn update_secret(
@@ -37,7 +38,7 @@ pub(crate) async fn update_secret(
         key: enc.encrypt(input.key.as_bytes(), org_id)?.to_string(),
         value: enc.encrypt(input.value.as_bytes(), org_id)?.to_string(),
         note: enc.encrypt(input.note.as_bytes(), org_id)?.to_string(),
-        project_ids: None,
+        project_ids: input.project_ids.clone(),
     });
 
     let config = client.get_api_configurations().await;

crates/bws/src/main.rs

@@ -109,15 +109,16 @@ enum SecretCommand {
         key: String,
         value: String,
 
+        #[arg(help = "The ID of the project this secret will be added to")]
+        project_id: Uuid,
+
         #[arg(long, help = "An optional note to add to the secret")]
         note: Option<String>,
-
-        #[arg(long, help = "The ID of the project this secret will be added to")]
-        project_id: Option<Uuid>,
     },
     Delete {
         secret_ids: Vec<Uuid>,
     },
+    #[clap(group = ArgGroup::new("edit_field").required(true).multiple(true))]
     Edit {
         secret_id: Uuid,
         #[arg(long, group = "edit_field")]
@@ -126,6 +127,8 @@ enum SecretCommand {
         value: Option<String>,
         #[arg(long, group = "edit_field")]
         note: Option<String>,
+        #[arg(long, group = "edit_field")]
+        project_id: Option<Uuid>,
     },
     Get {
         secret_id: Uuid,
@@ -179,7 +182,7 @@ enum CreateCommand {
         note: Option<String>,
 
         #[arg(long, help = "The ID of the project this secret will be added to")]
-        project_id: Option<Uuid>,
+        project_id: Uuid,
     },
 }
 
@@ -200,6 +203,8 @@ enum EditCommand {
         value: Option<String>,
         #[arg(long, group = "edit_field")]
         note: Option<String>,
+        #[arg(long, group = "edit_field")]
+        project_id: Option<Uuid>,
     },
 }
 
@@ -383,17 +388,34 @@ async fn process_commands() -> Result<()> {
         | Commands::Delete {
             cmd: DeleteCommand::Project { project_ids },
         } => {
-            let project_count = project_ids.len();
+            let count = project_ids.len();
 
-            client
+            let result = client
                 .projects()
                 .delete(ProjectsDeleteRequest { ids: project_ids })
                 .await?;
 
-            if project_count > 1 {
-                println!("Projects deleted successfully.");
-            } else {
-                println!("Project deleted successfully.");
+            let projects_failed: Vec<(Uuid, String)> = result
+                .data
+                .into_iter()
+                .filter_map(|r| r.error.map(|e| (r.id, e)))
+                .collect();
+            let deleted_projects = count - projects_failed.len();
+
+            if deleted_projects > 1 {
+                println!("{} projects deleted successfully.", deleted_projects);
+            } else if deleted_projects == 1 {
+                println!("{} project deleted successfully.", deleted_projects);
+            }
+
+            if projects_failed.len() > 1 {
+                println!("{} projects had errors:", projects_failed.len());
+            } else if projects_failed.len() == 1 {
+                println!("{} project had an error:", projects_failed.len());
+            }
+
+            for project in projects_failed {
+                println!("{}: {}", project.0, project.1);
             }
         }
 
@@ -466,7 +488,7 @@ async fn process_commands() -> Result<()> {
                     key,
                     value,
                     note: note.unwrap_or_default(),
-                    project_ids: project_id.map(|p| vec![p]),
+                    project_ids: Some(vec![project_id]),
                 })
                 .await?;
             serialize_response(secret, cli.output, color);
@@ -479,6 +501,7 @@ async fn process_commands() -> Result<()> {
                     key,
                     value,
                     note,
+                    project_id,
                 },
         }
         | Commands::Edit {
@@ -488,6 +511,7 @@ async fn process_commands() -> Result<()> {
                     key,
                     value,
                     note,
+                    project_id,
                 },
         } => {
             let old_secret = client
@@ -505,6 +529,13 @@ async fn process_commands() -> Result<()> {
                     key: key.unwrap_or(old_secret.key),
                     value: value.unwrap_or(old_secret.value),
                     note: note.unwrap_or(old_secret.note),
+                    project_ids: match project_id {
+                        Some(id) => Some(vec![id]),
+                        None => match old_secret.project_id {
+                            Some(id) => Some(vec![id]),
+                            None => bail!("Editing a secret requires a project_id."),
+                        },
+                    },
                 })
                 .await?;
             serialize_response(secret, cli.output, color);
@@ -516,12 +547,35 @@ async fn process_commands() -> Result<()> {
         | Commands::Delete {
             cmd: DeleteCommand::Secret { secret_ids },
         } => {
-            client
+            let count = secret_ids.len();
+
+            let result = client
                 .secrets()
                 .delete(SecretsDeleteRequest { ids: secret_ids })
                 .await?;
 
-            println!("Secret deleted correctly");
+            let secrets_failed: Vec<(Uuid, String)> = result
+                .data
+                .into_iter()
+                .filter_map(|r| r.error.map(|e| (r.id, e)))
+                .collect();
+            let deleted_secrets = count - secrets_failed.len();
+
+            if deleted_secrets > 1 {
+                println!("{} secrets deleted successfully.", deleted_secrets);
+            } else if deleted_secrets == 1 {
+                println!("{} secret deleted successfully.", deleted_secrets);
+            }
+
+            if secrets_failed.len() > 1 {
+                println!("{} secrets had errors:", secrets_failed.len());
+            } else if secrets_failed.len() == 1 {
+                println!("{} secret had an error:", secrets_failed.len());
+            }
+
+            for secret in secrets_failed {
+                println!("{}: {}", secret.0, secret.1);
+            }
         }
 
         Commands::Config { .. } => {

languages/csharp/schemas.cs

@@ -436,6 +436,9 @@ public partial class SecretPutRequest
         [JsonProperty("organizationId")]
         public Guid OrganizationId { get; set; }
 
+        [JsonProperty("projectIds")]
+        public Guid[] ProjectIds { get; set; }
+
         [JsonProperty("value")]
         public string Value { get; set; }
     }

languages/js_webassembly/bitwarden_client/schemas.ts

@@ -351,6 +351,7 @@ export interface SecretPutRequest {
      * Organization ID of the secret to modify
      */
     organizationId: string;
+    projectIds?:    string[] | null;
     value:          string;
 }
 
@@ -923,6 +924,7 @@ const typeMap: any = {
         { json: "key", js: "key", typ: "" },
         { json: "note", js: "note", typ: "" },
         { json: "organizationId", js: "organizationId", typ: "" },
+        { json: "projectIds", js: "projectIds", typ: u(undefined, u(a(""), null)) },
         { json: "value", js: "value", typ: "" },
     ], false),
     "SyncRequest": o([

languages/python/BitwardenClient/schemas.py

@@ -479,6 +479,7 @@ class SecretPutRequest:
     """Organization ID of the secret to modify"""
     organization_id: UUID
     value: str
+    project_ids: Optional[List[UUID]] = None
 
     @staticmethod
     def from_dict(obj: Any) -> 'SecretPutRequest':
@@ -488,7 +489,8 @@ def from_dict(obj: Any) -> 'SecretPutRequest':
         note = from_str(obj.get("note"))
         organization_id = UUID(obj.get("organizationId"))
         value = from_str(obj.get("value"))
-        return SecretPutRequest(id, key, note, organization_id, value)
+        project_ids = from_union([from_none, lambda x: from_list(lambda x: UUID(x), x)], obj.get("projectIds"))
+        return SecretPutRequest(id, key, note, organization_id, value, project_ids)
 
     def to_dict(self) -> dict:
         result: dict = {}
@@ -497,6 +499,8 @@ def to_dict(self) -> dict:
         result["note"] = from_str(self.note)
         result["organizationId"] = str(self.organization_id)
         result["value"] = from_str(self.value)
+        if self.project_ids is not None:
+            result["projectIds"] = from_union([from_none, lambda x: from_list(lambda x: str(x), x)], self.project_ids)
         return result
 
 

support/schemas/bitwarden_json/Command.json

@@ -428,6 +428,16 @@
           "type": "string",
           "format": "uuid"
         },
+        "projectIds": {
+          "type": [
+            "array",
+            "null"
+          ],
+          "items": {
+            "type": "string",
+            "format": "uuid"
+          }
+        },
         "value": {
           "type": "string"
         }