Remove the convert command

We only have one revision control provider now, so conversion is really
just changing the encryption.  This is relatively easy to implement as
a shell script, similar to the sample `passage` migration script shown
in <https://github.com/FiloSottile/passage/blob/main/README>.  The
`convert` command would probably be better implemented as a separate
utility using `gopass` as a library (similar to `gopass-hibp`).

docs/backends/age.md

@@ -19,8 +19,6 @@ gopass recipients add github:user
 
 This will automatically create a new age keypair and initilize the new store.
 
-Existing stores can be migrated using `gopass convert --crypto age`.
-
 ## Features
 
 * Encryption using `age` library, can be decrypted using the `age` CLI

docs/commands/convert.md

@@ -1,24 +0,0 @@
-# `convert` command
-
-The `convert` command exists to migrate stores between different backend
-implementations.
-
-Note: This command exists to enable a possible migration path. If we agree
-on a single set of backend implementations the multiple backend support
-might go away and this command as well.
-
-## Synopsis
-
-```
-$ gopass convert --store=foo --move=true --storage=gitfs --crypto=age
-$ gopass convert --store=bar --move=false --storage=fs --crypto=plain
-```
-
-## Flags
-
-Flag | Description
----- | -----------
-`--store` | Substore to convert.
-`--move` | Remove backup after converting? (default: `false`)
-`--storage` | Target storage backend.
-`--crypto` | Target crypto backend.

internal/action/commands.go

@@ -151,33 +151,6 @@ func (s *Action) GetCommands() []*cli.Command {
 			Action:       s.Config,
 			BashComplete: s.ConfigComplete,
 		},
-		{
-			Name:        "convert",
-			Usage:       "Convert a store to different backends",
-			Description: "Convert a store to a different set of backends",
-			Action:      s.Convert,
-			Before:      s.IsInitialized,
-			Hidden:      true,
-			Flags: []cli.Flag{
-				&cli.StringFlag{
-					Name:  "store",
-					Usage: "Specify which store to convert",
-				},
-				&cli.BoolFlag{
-					Name:  "move",
-					Value: true,
-					Usage: "Replace store?",
-				},
-				&cli.StringFlag{
-					Name:  "crypto",
-					Usage: fmt.Sprintf("Which crypto backend? %v", backend.CryptoRegistry.BackendNames()),
-				},
-				&cli.StringFlag{
-					Name:  "storage",
-					Usage: fmt.Sprintf("Which storage backend? %v", backend.StorageRegistry.BackendNames()),
-				},
-			},
-		},
 		{
 			Name:      "copy",
 			Aliases:   []string{"cp"},

main_test.go

@@ -121,7 +121,7 @@ func TestGetCommands(t *testing.T) { //nolint:paralleltest
 	c.Context = ctx
 
 	commands := getCommands(act, app)
-	assert.Equal(t, 39, len(commands))
+	assert.Equal(t, 38, len(commands))
 
 	prefix := ""
 	testCommands(t, c, commands, prefix)