fix

Author: iignatevich

README.md

@@ -27,6 +27,32 @@ launchr login \
   --keyring-passphrase-file=/path/to/your/secret
 ```
 
+To add a new key-value pair with an interactive shell:
+```shell
+launchr keyring:set your-key
+```
+
+It's possible to parse a user value and store it as a struct in a keyring. Supported formats are [string, yaml, json]:
+```shell
+launchr keyring:set your-key value
+```
+
+It's possible to parse a user value and store it as a struct in a keyring. Possible formats are [string, yaml, json]:
+```shell
+launchr keyring:set key --format yaml -- "- name: test-1
+- name: test-2"
+launchr keyring:set key --format yaml -- "$(cat file.yaml)"
+launchr keyring:set key --format json -- '[
+  {
+    "name": "test-1"
+  },
+  {
+    "name": "test-2"
+  }
+]'
+launchr keyring:set key --format json -- "$(cat file.json)"
+```
+
 Flags `--keyring-passphrase` and `--keyring-passphrase-file` are available for all launchr commands, for example:
 ```shell
 launchr compose --keyring-passphrase=YOURPASSHRPASE

action.set.yaml

@@ -17,5 +17,5 @@ action:
     - name: format
       title: Value Format
       description: Format to parse the input value
-      enum: ["string", "yaml"]
+      enum: ["string", "yaml", "json"]
       default: string

plugin.go

@@ -177,16 +177,13 @@ func (p *Plugin) DiscoverActions(_ context.Context) ([]*action.Action, error) {
 		}
 
 		userValue := input.Arg("value").(string)
-		format := input.Opt("format")
-		if format == "yaml" && userValue != "" {
-			yamlValue, err := parseYamlString(userValue)
-			if err != nil {
-				return fmt.Errorf("failed to parse YAML: %w", err)
-			}
-
-			key.Value = yamlValue
-		} else {
-			key.Value = userValue
+		format := input.Opt("format").(string)
+		var err error
+
+		// @TODO cover with tests
+		key.Value, err = parseFromString(format, userValue)
+		if err != nil {
+			return fmt.Errorf("failed to parse %s: %w", format, err)
 		}
 
 		return saveKey(p.k, key)

yaml.go

@@ -1,6 +1,7 @@
 package keyring
 
 import (
+	"encoding/json"
 	"errors"
 	"os"
 	"strings"
@@ -236,8 +237,21 @@ func (s *dataStoreYaml) Destroy() error {
 	return s.file.Remove()
 }
 
-func parseYamlString(value string) (interface{}, error) {
-	var yamlValue interface{}
-	err := yaml.Unmarshal([]byte(value), &yamlValue)
-	return yamlValue, err
+func parseFromString(format, value string) (interface{}, error) {
+	if format == "string" || value == "" {
+		return value, nil
+	}
+
+	var parsed interface{}
+	var err error
+	switch format {
+	case "json":
+		err = json.Unmarshal([]byte(value), &parsed)
+	case "yaml":
+		err = yaml.Unmarshal([]byte(value), &parsed)
+	default:
+		panic("unsupported format: " + format)
+	}
+
+	return parsed, err
 }