Add ability to pass in additional-properties in vc create

crates/web5_cli/src/vcs.rs

@@ -6,7 +6,7 @@ use web5::{
         CredentialSubject, Issuer, VerifiableCredential, VerifiableCredentialCreateOptions,
     },
     dids::{bearer_did::BearerDid, portable_did::PortableDid},
-    json::{FromJson, ToJson},
+    json::{FromJson, JsonObject, ToJson},
 };
 
 #[derive(Subcommand, Debug)]
@@ -19,16 +19,19 @@ pub enum Commands {
     ///
     /// Examples:
     ///
-    /// Create a VC with an issuer and return the minified form
+    /// Create a VC with an issuer and two additional properties and return the minified form
     ///
     /// $ web5 vc create did:dht:36xw3konj1pdd93axsn9p3a58a83uatcgx1nsjud97d91dtr56ry \
     ///     --issuer did:dht:36xw3konj1pdd93axsn9p3a58a83uatcgx1nsjud97d91dtr56ry \
+    ///     --additional-property "role:engineer" \
+    //      --additional-property "clearance:level5" \
     ///     --no-indent
     ///
-    /// Create a VC with a portable DID and return escaped JSON
+    /// Create a VC with a portable DID and one additional property and return escaped JSON
     ///
     /// $ web5 vc create did:dht:36xw3konj1pdd93axsn9p3a58a83uatcgx1nsjud97d91dtr56ry \
     ///     --portable-did '{"uri": ... }' \
+    ///     --additional-property "role:engineer" \
     ///     --json-escape
     #[command(verbatim_doc_comment)]
     Create {
@@ -52,6 +55,10 @@ pub enum Commands {
         /// If true, output JSON will be escaped
         #[arg(long)]
         json_escape: bool,
+        /// An additional property to add to the credentials subject of the VC. The
+        /// key and value are separated by a colon.
+        #[arg(long = "additional-property")]
+        additional_properties: Option<Vec<String>>,
     },
     Verify {
         vc_jwt: String,
@@ -63,13 +70,14 @@ pub enum Commands {
 }
 
 impl Commands {
-    pub async fn command(&self) {
+    pub async fn command(self) {
         match self {
             Commands::Create {
                 credential_subject_id,
                 portable_did,
                 issuer,
                 expiration_date,
+                additional_properties,
                 no_indent,
                 json_escape,
             } => {
@@ -92,12 +100,22 @@ impl Commands {
                         }
                     },
                 };
+                let additional_properties = additional_properties.map(|vs| {
+                    vs.into_iter().fold(JsonObject::new(), |mut acc, cur| {
+                        let split = cur.split(':').map(String::from).collect::<Vec<_>>();
+                        // clap will make sure we are getting a string. since we will always split a string
+                        // and get two strings, the insert should never fail since the passed in value will
+                        // always be a string and therefore always be able to successfully become a JSON value
+                        acc.insert(&split[0], &split[1]).unwrap();
+                        acc
+                    })
+                });
 
                 let vc = VerifiableCredential::create(
                     issuer,
                     CredentialSubject {
                         id: credential_subject_id.to_string(),
-                        ..Default::default()
+                        additional_properties,
                     },
                     Some(VerifiableCredentialCreateOptions {
                         expiration_date,
@@ -112,7 +130,7 @@ impl Commands {
                     false => serde_json::to_string_pretty(&vc).unwrap(),
                 };
 
-                if *json_escape {
+                if json_escape {
                     output_str = output_str.replace('"', "\\\"");
                 }
 
@@ -128,7 +146,7 @@ impl Commands {
                 vc_jwt,
                 no_indent,
                 json_escape,
-            } => match VerifiableCredential::from_vc_jwt(vc_jwt, true).await {
+            } => match VerifiableCredential::from_vc_jwt(&vc_jwt, true).await {
                 Err(e) => {
                     println!("\n❌ Verfication failed\n");
                     println!("{:?} {}", e, e);
@@ -141,7 +159,7 @@ impl Commands {
                         false => serde_json::to_string_pretty(&vc).unwrap(),
                     };
 
-                    if *json_escape {
+                    if json_escape {
                         output_str = output_str.replace('"', "\\\"");
                     }