🔧 chore(project): upgrade Go to 1.24, add tidy target, update docs

Signed-off-by: samzong <samzong.lu@gmail.com>

Author: samzong

.github/workflows/pr-review.yml

@@ -21,7 +21,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v5
         with:
-          go-version: '1.21'
+          go-version: '1.24.x'
           cache: true
 
       - name: Format check
@@ -39,4 +39,4 @@ jobs:
         uses: actions/upload-artifact@v4
         with:
           name: gmc
-          path: bin/gmc 
+          path: bin/gmc 

.github/workflows/release.yml

@@ -34,7 +34,7 @@ jobs:
       - name: Set up Go
         uses: actions/setup-go@v4
         with:
-          go-version: ">=1.21.0"
+          go-version: "1.24.x"
           cache: true
 
       - name: Install git-chglog

Makefile

@@ -59,12 +59,17 @@ test-coverage-ci: ## Run tests with coverage for CI (no HTML)
 	@go tool cover -func=$(BUILD_DIR)/coverage.out | tail -1
 
 .PHONY: fmt
-fmt: ## Format code and tidy modules
+fmt: ## Format code
 	@echo "Formatting code..."
 	@go fmt ./...
-	@go mod tidy
 	@echo "Format Command Done"
 
+.PHONY: tidy
+tidy: ## Tidy go.mod and go.sum
+	@echo "Tidying modules..."
+	@go mod tidy
+	@echo "Tidy Command Done"
+
 .PHONY: lint
 lint: ## Run code analysis
 	@echo "$(BLUE)Running code analysis...$(NC)"

README.md

@@ -25,7 +25,7 @@
 
 ### Quick start (required config)
 
-`gmc` reads configuration from `~/.gmc.yaml` by default (override with `--config`). On macOS/Linux, the config file is forced to permission `0600`. You can also place a `.gmc.yaml` in your project directory to override global settings on a per-repo basis.
+`gmc` reads configuration from `~/.config/gmc/config.yaml` by default (override with `--config`). If a legacy `~/.gmc.yaml` exists, it is used as a fallback. On macOS/Linux, the config file is forced to permission `0600`. You can also place a `.gmc.yaml` in your project directory to override global settings on a per-repo basis.
 
 Recommended: run the guided setup (or accept the prompt shown on first use):
 
@@ -175,6 +175,10 @@ fpath=(~/.zsh/completions $fpath)
 autoload -Uz compinit && compinit
 ```
 
+### Man pages
+
+Man pages under `docs/man/` are generated via `make man` (from `cmd/gendoc/main.go`). Do not edit them manually.
+
 ## Prompt templates
 
 `gmc` supports a single prompt template override. If `prompt_template` is empty or set to `default`, the built-in template is used.

action.yml

@@ -25,9 +25,6 @@ inputs:
   prompt_template:
     description: "Prompt template name (in prompts_dir)"
     required: false
-  prompts_dir:
-    description: "Directory containing prompt templates"
-    required: false
   working-directory:
     description: "Working directory to run commands in"
     required: false
@@ -51,7 +48,7 @@ runs:
     - name: Set up Go
       uses: actions/setup-go@v5
       with:
-        go-version: '1.21'
+        go-version: '1.24.x'
 
     - name: Build gmc
       shell: bash
@@ -71,7 +68,6 @@ runs:
         if [ -n "${{ inputs.model }}" ]; then ./gmc config set model "${{ inputs.model }}"; fi
         if [ -n "${{ inputs.role }}" ]; then ./gmc config set role "${{ inputs.role }}"; fi
         if [ -n "${{ inputs.prompt_template }}" ]; then ./gmc config set prompt_template "${{ inputs.prompt_template }}"; fi
-        if [ -n "${{ inputs.prompts_dir }}" ]; then ./gmc config set prompts_dir "${{ inputs.prompts_dir }}"; fi
 
     - name: Run GMC
       id: run

cmd/root.go

@@ -4,6 +4,7 @@ import (
 	"bufio"
 	"errors"
 	"fmt"
+	"io"
 	"os"
 	"os/exec"
 	"strings"
@@ -15,6 +16,7 @@ import (
 	"github.com/samzong/gmc/internal/formatter"
 	"github.com/samzong/gmc/internal/git"
 	"github.com/samzong/gmc/internal/llm"
+	"github.com/samzong/gmc/internal/stringsutil"
 	"github.com/samzong/gmc/internal/ui"
 	"github.com/spf13/cobra"
 )
@@ -227,15 +229,13 @@ func handleStdinDiff() error {
 	// Extract file names from diff
 	changedFiles := extractFilesFromDiff(diff)
 
-	cfg := config.GetConfig()
-	proceed, err := ensureLLMConfigured(cfg, os.Stdin, os.Stderr, runInitWizard)
+	cfg, proceed, err := ensureConfiguredAndGetConfig(config.GetConfig(), os.Stdin, os.Stderr, runInitWizard)
 	if err != nil {
 		return err
 	}
 	if !proceed {
 		return nil
 	}
-	cfg = config.GetConfig()
 
 	// Generate commit message
 	message, err := generateCommitMessage(cfg, changedFiles, diff, userPrompt)
@@ -249,41 +249,44 @@ func handleStdinDiff() error {
 	return nil
 }
 
+func ensureConfiguredAndGetConfig(
+	cfg *config.Config,
+	in io.Reader,
+	out io.Writer,
+	initRunner func(io.Reader, io.Writer, *config.Config) error,
+) (*config.Config, bool, error) {
+	proceed, err := ensureLLMConfigured(cfg, in, out, initRunner)
+	if err != nil || !proceed {
+		return nil, proceed, err
+	}
+	return config.GetConfig(), true, nil
+}
+
 // extractFilesFromDiff parses file names from unified diff format
 func extractFilesFromDiff(diff string) []string {
 	var files []string
-	seen := make(map[string]bool)
 
 	for _, line := range strings.Split(diff, "\n") {
 		if strings.HasPrefix(line, "+++ b/") {
 			file := strings.TrimPrefix(line, "+++ b/")
-			if !seen[file] {
-				files = append(files, file)
-				seen[file] = true
-			}
+			files = append(files, file)
 		} else if strings.HasPrefix(line, "--- a/") {
 			file := strings.TrimPrefix(line, "--- a/")
-			if !seen[file] {
-				files = append(files, file)
-				seen[file] = true
-			}
+			files = append(files, file)
 		}
 	}
 
-	return files
+	return stringsutil.UniqueStrings(files)
 }
 
 func handleCommitFlow(diff string, changedFiles []string) error {
-	cfg := config.GetConfig()
-
-	proceed, err := ensureLLMConfigured(cfg, os.Stdin, os.Stderr, runInitWizard)
+	cfg, proceed, err := ensureConfiguredAndGetConfig(config.GetConfig(), os.Stdin, os.Stderr, runInitWizard)
 	if err != nil {
 		return err
 	}
 	if !proceed {
 		return nil
 	}
-	cfg = config.GetConfig()
 
 	return runCommitFlow(cfg, changedFiles, diff, performCommit)
 }
@@ -493,16 +496,13 @@ func commitStagedFiles(files []string) error {
 }
 
 func handleSelectiveCommitFlow(diff string, files []string) error {
-	cfg := config.GetConfig()
-
-	proceed, err := ensureLLMConfigured(cfg, os.Stdin, os.Stderr, runInitWizard)
+	cfg, proceed, err := ensureConfiguredAndGetConfig(config.GetConfig(), os.Stdin, os.Stderr, runInitWizard)
 	if err != nil {
 		return err
 	}
 	if !proceed {
 		return nil
 	}
-	cfg = config.GetConfig()
 
 	return runCommitFlow(cfg, files, diff, func(message string) error {
 		return performSelectiveCommit(message, files)

internal/emoji/emoji.go

@@ -5,6 +5,7 @@ import (
 	"regexp"
 	"sort"
 	"strings"
+	"sync"
 )
 
 // emojiMap maps commit types to their corresponding emojis
@@ -31,6 +32,11 @@ var emojiMap = map[string]string{
 // Pre-compiled regex pattern for extracting commit type
 var commitTypeRegex = regexp.MustCompile(`^([a-zA-Z]+)(?:\([^)]+\))?:`)
 
+var (
+	commitTypesOnce   sync.Once
+	cachedCommitTypes []string
+)
+
 // GetEmojiForType returns the emoji for a given commit type.
 // Returns empty string if type is not recognized.
 func GetEmojiForType(commitType string) string {
@@ -42,12 +48,15 @@ func GetEmojiForType(commitType string) string {
 
 // GetAllCommitTypes returns all supported commit types in alphabetical order.
 func GetAllCommitTypes() []string {
-	types := make([]string, 0, len(emojiMap))
-	for t := range emojiMap {
-		types = append(types, t)
-	}
-	sort.Strings(types)
-	return types
+	commitTypesOnce.Do(func() {
+		types := make([]string, 0, len(emojiMap))
+		for t := range emojiMap {
+			types = append(types, t)
+		}
+		sort.Strings(types)
+		cachedCommitTypes = types
+	})
+	return append([]string(nil), cachedCommitTypes...)
 }
 
 // GetEmojiDescription returns a formatted string listing all commit types with their emojis.