`os.Exit` inside `Run` bypasses deferred database closures

[DylanDevelops]

The problem

Throughout the CLI commands (e.g., start.go, stop.go), the code defers db.Close() and then calls os.Exit(1) on error paths. os.Exit immediately terminates the program, completely bypassing all defer statements. While SQLite is generally forgiving of unclosed connections, this is a dangerous pattern that can cause locks to persist or interrupt active operations like VACUUM.

Release version

0.8.6

Operating system

macOS

Steps to reproduce the behavior

1. Trigger any error state in a command (e.g., trying to start a timer when one is already running).
2. The program exits via os.Exit(1), skipping the deferred db.Close().

Screenshots

No response

Additional context

The idiomatic Cobra fix is to use RunE instead of Run, return the errors up the chain, and allow cmd.Execute() to handle the exit codes, ensuring all defers execute normally.