feat: implement add, get, delete, and summarize commands

- Added `add`, `get`, `delete`, and `summarize` commands with corresponding handlers.
- Modified the `entry` model to include an auto-incrementing `id` field.
- Updated the database layer to support retrieving entries by id and deleting entries by id.
- Implemented validation for date ranges to ensure "from" date is not after "to" date.
- Enhanced the `get` command to allow retrieving a single entry by id or a list of entries with date range filters.
- Added flags and argument parsing for the new commands.
- Refactored option handling to include validation functions.
- Fixed a bug where max id was not being set properly.
- Removed homebrew cask from `.goreleaser.yml`.

Author: thdxg

.goreleaser.yml

@@ -24,16 +24,3 @@ changelog:
     exclude:
       - "^docs:"
       - "^test:"
-
-homebrew_casks:
-  - name: llog
-    repository:
-      owner: ethn1ee
-      name: homebrew-tap
-      token: "{{ .Env.HOMEBREW_TAP_TOKEN }}"
-    commit_author:
-      name: goreleaserbot
-      email: bot@goreleaser.com
-    homepage: "https://github.com/ethn1ee/llog"
-    description: "A simple life logger"
-    license: "MIT"

cmd/add.go

@@ -11,12 +11,13 @@ import (
 var addOpts = &handler.AddOpts{}
 
 var addCmd = &cobra.Command{
-	Use:     "add [body]",
-	Short:   "Add a log entry",
-	Long:    `Add a log entry.`,
-	Args:    cobra.ExactArgs(1),
-	PreRunE: handler.ValidateOptions(cfg, addOpts),
-	RunE:    handler.Add(cfg, db, addOpts),
+	Use:          "add [body]",
+	Short:        "Add a log entry",
+	Long:         `Add a log entry.`,
+	Args:         cobra.ExactArgs(1),
+	PreRunE:      handler.ValidateOptions(cfg, addOpts),
+	RunE:         handler.Add(cfg, db, addOpts),
+	SilenceUsage: true,
 }
 
 func init() {

cmd/delete.go

@@ -1,4 +1,3 @@
-
 /*
 Copyright © 2025 Ethan Lee <ethantlee21@gmail.com>
 */
@@ -12,11 +11,13 @@ import (
 var deleteOpts = &handler.DeleteOpts{}
 
 var deleteCmd = &cobra.Command{
-	Use:     "delete",
-	Short:   "Delete log entries",
-	Long:    `Delete log entries. You can delete with id or interactively`,
-	PreRunE: handler.ValidateOptions(cfg, deleteOpts),
-	RunE:    handler.Delete(cfg, db, deleteOpts),
+	Use:          "delete [id]",
+	Short:        "Delete log entries",
+	Long:         `Delete log entries. You can delete with entry id or interactively select them.`,
+	Args:         cobra.ExactArgs(1),
+	PreRunE:      handler.ValidateOptions(cfg, deleteOpts),
+	RunE:         handler.Delete(cfg, db, deleteOpts),
+	SilenceUsage: true,
 }
 
 func init() {

cmd/get.go

@@ -11,12 +11,18 @@ import (
 var getOpts = &handler.GetOpts{}
 
 var getCmd = &cobra.Command{
-	Use:     "get",
-	Short:   "Get log entries",
-	Long:    `Get log entries. You can specify date range with flags.`,
-	Args:    cobra.NoArgs,
-	PreRunE: handler.ValidateOptions(cfg, getOpts),
-	RunE:    handler.Get(cfg, db, getOpts),
+	Use:   "get [ID]",
+	Short: "Get log entries",
+	Long: `Get log entries.
+
+You can retrieve a single entry by providing its ID as an argument.
+Alternatively, you can retrieve a list of entries by using flags to filter by date range or limit the count.
+
+Note: Providing an ID is mutually exclusive with using any flags.`,
+	Args:         cobra.MaximumNArgs(1),
+	PreRunE:      handler.ValidateOptions(cfg, getOpts),
+	RunE:         handler.Get(cfg, db, getOpts),
+	SilenceUsage: true,
 }
 
 func init() {

cmd/root.go

@@ -4,7 +4,6 @@ Copyright © 2025 Ethan Lee <ethantlee21@gmail.com>
 package cmd
 
 import (
-	"log/slog"
 	"os"
 
 	"github.com/ethn1ee/llog/internal/config"
@@ -15,30 +14,23 @@ import (
 )
 
 var (
-	cfg     = &config.Config{}
-	db      = &_db.DB{}
-	lg      = &logger.Logger{}
-	cmdAttr slog.Attr
+	cfg = &config.Config{}
+	db  = &_db.DB{}
+	lg  = &logger.Logger{}
 )
 
 var rootCmd = &cobra.Command{
 	Use:               "llog",
 	Short:             "Life log",
 	Long:              `Record your fleeting moments with llog.`,
 	PersistentPreRunE: handler.Init(cfg, db, lg),
-	PersistentPostRunE: func(cmd *cobra.Command, args []string) error {
-		slog.Info("command completed", cmdAttr)
-		return nil
-	},
+	SilenceUsage:      true,
 }
 
 func Execute() {
 	err := rootCmd.Execute()
 	if err != nil {
-		slog.Error("command failed", cmdAttr, slog.Any("error", err))
 		os.Exit(1)
 	}
-	if err := lg.Close(); err != nil {
-		slog.Error("failed to close log file", cmdAttr, slog.Any("error", err))
-	}
+	_ = lg.Close()
 }

cmd/summarize.go

@@ -11,11 +11,12 @@ import (
 var summarizeOpts = &handler.SummarizeOpts{}
 
 var summarizeCmd = &cobra.Command{
-	Use:     "summarize",
-	Short:   "Summarize log entries",
-	Long:    `Summarize log entries. You can specify date range with flags.`,
-	PreRunE: handler.ValidateOptions(cfg, summarizeOpts),
-	RunE:    handler.Summarize(cfg, db, summarizeOpts),
+	Use:          "summarize",
+	Short:        "Summarize log entries",
+	Long:         `Summarize log entries. You can specify date range with flags.`,
+	PreRunE:      handler.ValidateOptions(cfg, summarizeOpts),
+	RunE:         handler.Summarize(cfg, db, summarizeOpts),
+	SilenceUsage: true,
 }
 
 func init() {

go.mod

@@ -5,6 +5,7 @@ go 1.25.0
 require (
 	github.com/fatih/color v1.18.0
 	github.com/spf13/cobra v1.10.1
+	github.com/spf13/pflag v1.0.10
 	github.com/spf13/viper v1.21.0
 	gorm.io/driver/sqlite v1.6.0
 	gorm.io/gorm v1.31.0
@@ -24,7 +25,6 @@ require (
 	github.com/sourcegraph/conc v0.3.1-0.20240121214520-5f936abd7ae8 // indirect
 	github.com/spf13/afero v1.15.0 // indirect
 	github.com/spf13/cast v1.10.0 // indirect
-	github.com/spf13/pflag v1.0.10 // indirect
 	github.com/subosito/gotenv v1.6.0 // indirect
 	go.yaml.in/yaml/v3 v3.0.4 // indirect
 	golang.org/x/sys v0.29.0 // indirect

internal/config/internal.go

@@ -1,6 +1,7 @@
 package config
 
 type internalConfig struct {
-	MaxId       uint
-	MaxIdDigits int
+	EntryCount       int64
+	MaxEntryId       uint64
+	MaxEntryIdDigits int
 }

internal/db/db.go

@@ -35,13 +35,22 @@ func Load(cfg *config.Config, ctx context.Context, db *DB) error {
 
 	db.Entry = &entryDB{gorm.G[model.Entry](gormdb)}
 
-	last, err := db.Entry.GetLast(ctx)
+	count, err := db.Entry.Count(ctx)
 	if err != nil {
 		return err
 	}
 
-	cfg.Internal.MaxId = last.ID
-	cfg.Internal.MaxIdDigits = len(strconv.FormatUint(uint64(last.ID), 10))
+	cfg.Internal.EntryCount = count
+
+	if count > 0 {
+		last, err := db.Entry.GetLast(ctx)
+		if err != nil {
+			return err
+		}
+
+		cfg.Internal.MaxEntryId = last.ID
+		cfg.Internal.MaxEntryIdDigits = len(strconv.FormatUint(uint64(last.ID), 10))
+	}
 
 	return nil
 }

internal/db/entry.go

@@ -2,6 +2,7 @@ package db
 
 import (
 	"context"
+	"errors"
 	"time"
 
 	"github.com/ethn1ee/llog/internal/model"
@@ -12,6 +13,10 @@ type entryDB struct {
 	i gorm.Interface[model.Entry]
 }
 
+func (db *entryDB) Count(ctx context.Context) (int64, error) {
+	return db.i.Count(ctx, "id")
+}
+
 func (db *entryDB) Add(ctx context.Context, entry *model.Entry) error {
 	return db.i.Create(ctx, entry)
 }
@@ -20,18 +25,37 @@ func (db *entryDB) GetAll(ctx context.Context) ([]model.Entry, error) {
 	return db.i.Find(ctx)
 }
 
-func (db *entryDB) GetFrom(ctx context.Context, from time.Time) ([]model.Entry, error) {
-	return db.i.Where("created_at >= ?", from).Find(ctx)
-}
-
-func (db *entryDB) GetTo(ctx context.Context, to time.Time) ([]model.Entry, error) {
-	return db.i.Where("created_at <= ?", to).Find(ctx)
-}
-
 func (db *entryDB) GetRange(ctx context.Context, from time.Time, to time.Time) ([]model.Entry, error) {
+	if from.IsZero() && to.IsZero() {
+		return nil, errors.New("range unspecified")
+	}
+	if from.IsZero() {
+		return db.i.Where("created_at <= ?", to).Find(ctx)
+	}
+	if to.IsZero() {
+		return db.i.Where("created_at >= ?", from).Find(ctx)
+	}
 	return db.i.Where("created_at >= ? AND created_at <= ?", from, to).Find(ctx)
 }
 
 func (db *entryDB) GetLast(ctx context.Context) (model.Entry, error) {
 	return db.i.Last(ctx)
 }
+
+func (db *entryDB) GetById(ctx context.Context, id uint64) (model.Entry, error) {
+	match, err := db.i.Where("id = ?", id).Find(ctx)
+	if err != nil {
+		return model.Entry{}, err
+	}
+
+	return match[0], nil
+}
+
+func (db *entryDB) DeleteById(ctx context.Context, id uint64) error {
+	_, err := db.i.Where("id = ?", id).Delete(ctx)
+	if err != nil {
+		return err
+	}
+
+	return nil
+}