Skip to content

chore: Modernize CLI, refresh deps, and add regression tests#5

Open
alexbednarczyk wants to merge 1 commit intomainfrom
gpt-5.1-codex-max-high
Open

chore: Modernize CLI, refresh deps, and add regression tests#5
alexbednarczyk wants to merge 1 commit intomainfrom
gpt-5.1-codex-max-high

Conversation

@alexbednarczyk
Copy link
Copy Markdown
Owner

• Changes

  • Updated core deps to current releases (Cobra 1.10.1, Viper 1.20.1, Spinner
    go.sum.citeturn0search2turn0search1turn2search2turn0search0
  • Modernized CLI wiring: root now uses PersistentPreRunE with a --config flag and
    no auto-created config files; main.go handles errors; all commands converted
    to RunE, use command writers, and add validation/quality-of-life tweaks
    (date formatting flag, guidance for negative counts via --, safer password/
    env handling, spinner charset guard and config watching only when a config is
    loaded). Updated files: cmd/root.go, main.go, cmd/date.go, cmd/count.go, cmd/
    password.go, cmd/environment.go, cmd/spinner.go, cmd/config.go, cmd/version.go.
  • Added command regression tests and helpers in cmd/cmd_test.go; refreshed
    README with Go 1.22+ instructions and test command; new .gitignore entries for
    binaries/config/cache.

Tests

  • GOCACHE=$(pwd)/.gocache go test ./...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant