Merge pull request #16 from pepabo/refactor/version-management

refactor: move version management to bot package

Author: buty4649

.tagpr

@@ -45,4 +45,4 @@
 [tagpr]
 	vPrefix = true
 	releaseBranch = main
-	versionFile = version.go
+	versionFile = bot/version.go

bot/bot.go

@@ -30,11 +30,10 @@ type Bot struct {
 	verbose      bool
 	logger       *slog.Logger
 	repo         repository.UserPointsRepository
-	version      string
 }
 
 // New creates a new Slack bot instance
-func New(botToken, appToken string, repo repository.UserPointsRepository, verbose bool, logger *slog.Logger, version string) (*Bot, error) {
+func New(botToken, appToken string, repo repository.UserPointsRepository, verbose bool, logger *slog.Logger) (*Bot, error) {
 	if botToken == "" || appToken == "" {
 		return nil, fmt.Errorf("SLACK_BOT_TOKEN or SLACK_APP_TOKEN is not set")
 	}
@@ -60,13 +59,12 @@ func New(botToken, appToken string, repo repository.UserPointsRepository, verbos
 		verbose:      verbose,
 		logger:       logger,
 		repo:         repo,
-		version:      version,
 	}, nil
 }
 
 // Start starts the Slack bot
 func (b *Bot) Start() {
-	b.logger.Debug("Starting bot(version: " + b.version + ")...")
+	b.logger.Debug("Starting bot(version: " + Version + ")...")
 	go b.handleEvents()
 	b.logger.Debug("Starting socket mode client...")
 	if err := b.socketClient.Run(); err != nil {

bot/bot_test.go

@@ -33,7 +33,7 @@ func setupTestBot(t *testing.T) (*Bot, func()) {
 	}
 
 	// Create a test bot with dummy tokens
-	bot, err := New("dummy-bot-token", "dummy-app-token", repo, false, logger, "dummy-version")
+	bot, err := New("dummy-bot-token", "dummy-app-token", repo, false, logger)
 	if err != nil {
 		if err := os.Remove(tempFile.Name()); err != nil {
 			t.Logf("Failed to remove temp file: %v", err)
@@ -103,7 +103,7 @@ func TestNew(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			bot, err := New(tt.botToken, tt.appToken, repo, tt.verbose, logger, "dummy-version")
+			bot, err := New(tt.botToken, tt.appToken, repo, tt.verbose, logger)
 			if (err != nil) != tt.wantErr {
 				t.Errorf("New() error = %v, wantErr %v", err, tt.wantErr)
 			}

bot/version.go

@@ -0,0 +1,3 @@
+package bot
+
+var Version = "1.1.2"

go.mod

@@ -3,20 +3,21 @@ module plusplusbot
 go 1.24
 
 require (
+	github.com/aws/aws-sdk-go-v2 v1.36.3
+	github.com/aws/aws-sdk-go-v2/config v1.29.14
+	github.com/aws/aws-sdk-go-v2/credentials v1.17.67
+	github.com/aws/aws-sdk-go-v2/service/dynamodb v1.43.1
+	github.com/guregu/dynamo/v2 v2.3.0
 	github.com/ncruces/go-sqlite3 v0.25.2
 	github.com/slack-go/slack v0.16.0
 )
 
 require (
-	github.com/aws/aws-sdk-go-v2 v1.36.3 // indirect
-	github.com/aws/aws-sdk-go-v2/config v1.29.14 // indirect
-	github.com/aws/aws-sdk-go-v2/credentials v1.17.67 // indirect
 	github.com/aws/aws-sdk-go-v2/feature/dynamodb/attributevalue v1.14.11 // indirect
 	github.com/aws/aws-sdk-go-v2/feature/ec2/imds v1.16.30 // indirect
 	github.com/aws/aws-sdk-go-v2/internal/configsources v1.3.34 // indirect
 	github.com/aws/aws-sdk-go-v2/internal/endpoints/v2 v2.6.34 // indirect
 	github.com/aws/aws-sdk-go-v2/internal/ini v1.8.3 // indirect
-	github.com/aws/aws-sdk-go-v2/service/dynamodb v1.43.1 // indirect
 	github.com/aws/aws-sdk-go-v2/service/dynamodbstreams v1.22.4 // indirect
 	github.com/aws/aws-sdk-go-v2/service/internal/accept-encoding v1.12.3 // indirect
 	github.com/aws/aws-sdk-go-v2/service/internal/endpoint-discovery v1.10.15 // indirect
@@ -27,7 +28,6 @@ require (
 	github.com/aws/smithy-go v1.22.2 // indirect
 	github.com/cenkalti/backoff/v4 v4.3.0 // indirect
 	github.com/gorilla/websocket v1.4.2 // indirect
-	github.com/guregu/dynamo/v2 v2.3.0 // indirect
 	github.com/ncruces/julianday v1.0.0 // indirect
 	github.com/stretchr/testify v1.8.4 // indirect
 	github.com/tetratelabs/wazero v1.9.0 // indirect

main.go

@@ -41,7 +41,7 @@ func main() {
 
 	// Initialize bot
 	verbose := os.Getenv("DEBUG") != ""
-	bot, err := bot.New(botToken, appToken, repo, verbose, logger, version)
+	bot, err := bot.New(botToken, appToken, repo, verbose, logger)
 	if err != nil {
 		logger.Error("Failed to create bot", "error", err)
 		os.Exit(1)