Add completion bash to alias roam to git completion

README.md

@@ -34,7 +34,7 @@ My one slight annoyance is the manual set up required for a new host/account whi
 ### Bash
 
 ```bash
-complete -F _git roam
+eval "$(roam completion bash)"
 ```
 
 Maybe somehow also use,

completion.go

@@ -46,6 +46,17 @@ func (a *app) completionCommand() *cobra.Command {
 	}
 	var force bool
 	out := a.defaultZshOut()
+	bash := &cobra.Command{
+		Use:           "bash",
+		Short:         "Generate the autocompletion script for bash",
+		Args:          cobra.NoArgs,
+		SilenceUsage:  true,
+		SilenceErrors: true,
+		RunE: func(cmd *cobra.Command, _ []string) error {
+			_, err := fmt.Fprintln(cmd.OutOrStdout(), "complete -F _git roam")
+			return err
+		},
+	}
 	zsh := &cobra.Command{
 		Use:           "zsh",
 		Short:         "Generate the autocompletion script for zsh",
@@ -58,7 +69,7 @@ func (a *app) completionCommand() *cobra.Command {
 	}
 	zsh.Flags().BoolVarP(&force, "force", "f", false, "Overwrite existing generated files")
 	zsh.Flags().StringVarP(&out, "out", "o", out, "Output directory for generated files (~/.local/share/zsh/site-functions)")
-	cmd.AddCommand(zsh)
+	cmd.AddCommand(bash, zsh)
 	return cmd
 }
 

completion_test.go

@@ -9,6 +9,14 @@ import (
 	"github.com/stretchr/testify/require"
 )
 
+func TestCompletionBash(t *testing.T) {
+	app, _ := testApp(t)
+	var stdout bytes.Buffer
+	app.stdout = &stdout
+	require.NoError(t, app.run(t.Context(), []string{"completion", "bash"}))
+	require.Equal(t, "complete -F _git roam\n", stdout.String())
+}
+
 func TestCompletionZsh(t *testing.T) {
 	app, _ := testApp(t)
 	out := filepath.Join(app.home, "completions")
@@ -67,7 +75,9 @@ func TestCompletionHelp(t *testing.T) {
 	var stdout bytes.Buffer
 	app.stdout = &stdout
 	require.NoError(t, app.run(t.Context(), []string{"completion", "-h"}))
-	require.Contains(t, stdout.String(), "zsh")
+	help := stdout.String()
+	require.Contains(t, help, "bash")
+	require.Contains(t, help, "zsh")
 }
 
 func TestCompletionZshTildeOut(t *testing.T) {