feat: put circular navigation behind a flag

Open URIs directly without invoking `tea.ExecProcess`.

Author: dhth

cmd/assets/guide/config-a-sample-toml-config.md

@@ -9,4 +9,5 @@ list_density            = "spacious"
 show_context            = false
 editor                  = "vi -u NONE"
 confirm_before_deletion = false
+circular_nav            = true
 ```

cmd/assets/updates.txt

@@ -1,3 +1,9 @@
+👉 upcoming
+• 🔄 URIs with custom schemes are considered as task bookmarks. For example:
+    • spotify:track:4fVBFyglBhMf0erfF7pBJp
+    • obsidian://open?vault=VAULT&file=FILE
+• 🆕 Circular navigation for lists via "--circular-nav=true"
+
 👉 0.4.3
 • 🔄 omm asks for confirmation before deleting a task. Change this behavior with
     "--confirm-before-deletion=false"

cmd/root.go

@@ -142,6 +142,7 @@ func NewRootCommand() (*cobra.Command, error) {
 		editorCmd             string
 		showContextFlagInp    bool
 		confirmBeforeDeletion bool
+		circularNav           bool
 	)
 
 	rootCmd := &cobra.Command{
@@ -247,6 +248,7 @@ Tip: Quickly add a task using 'omm "task summary goes here"'.
 				TextEditorCmd:         strings.Fields(editorCmd),
 				ShowContext:           showContextFlagInp,
 				ConfirmBeforeDeletion: confirmBeforeDeletion,
+				CircularNav:           circularNav,
 			}
 
 			ui.RenderUI(db, config)
@@ -397,6 +399,7 @@ Error: %s`, author, repoIssuesUrl, err)
 	rootCmd.Flags().StringVar(&editorFlagInp, "editor", "vi", "editor command to run when adding/editing context to a task")
 	rootCmd.Flags().BoolVar(&showContextFlagInp, "show-context", false, "whether to start omm with a visible task context pane or not; this can later be toggled on/off in the TUI")
 	rootCmd.Flags().BoolVar(&confirmBeforeDeletion, "confirm-before-deletion", true, "whether to ask for confirmation before deleting a task")
+	rootCmd.Flags().BoolVar(&circularNav, "circular-nav", false, "whether to enable circular navigation for lists (cycle back to the first entry from the last, and vice versa)")
 
 	tasksCmd.Flags().Uint8VarP(&printTasksNum, "num", "n", printTasksDefault, "number of tasks to print")
 	tasksCmd.Flags().StringVarP(&configPath, "config-path", "c", defaultConfigPath, fmt.Sprintf("location of omm's TOML config file%s", configPathAdditionalCxt))

internal/ui/cmds.go

@@ -105,16 +105,18 @@ func openURI(uri string) tea.Cmd {
 		cmd = "xdg-open"
 	}
 	c := exec.Command(cmd, append(args, uri)...)
-	return tea.ExecProcess(c, func(err error) tea.Msg {
+	err := c.Run()
+	return func() tea.Msg {
 		return uriOpenedMsg{uri, err}
-	})
+	}
 }
 
 func openURIsDarwin(uris []string) tea.Cmd {
 	c := exec.Command("open", uris...)
-	return tea.ExecProcess(c, func(err error) tea.Msg {
+	err := c.Run()
+	return func() tea.Msg {
 		return urisOpenedDarwinMsg{uris, err}
-	})
+	}
 }
 
 func copyContextToClipboard(context string) tea.Cmd {

internal/ui/config.go

@@ -23,4 +23,5 @@ type Config struct {
 	DBPath                string
 	ShowContext           bool
 	ConfirmBeforeDeletion bool
+	CircularNav           bool
 }

internal/ui/update.go

@@ -428,6 +428,10 @@ func (m model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		case "down", "j":
 			switch m.activeView {
 			case taskListView, archivedTaskListView, contextBookmarksView, prefixSelectionView:
+				if !m.cfg.CircularNav {
+					break
+				}
+
 				// cycle back to top
 				var list *list.Model
 				switch m.activeView {
@@ -473,6 +477,10 @@ func (m model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		case "up", "k":
 			switch m.activeView {
 			case taskListView, archivedTaskListView, contextBookmarksView, prefixSelectionView:
+				if !m.cfg.CircularNav {
+					break
+				}
+
 				// cycle to the end
 				var list *list.Model
 				switch m.activeView {