| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 1 | package claudetool |
| 2 | |
| 3 | import ( |
| 4 | "context" |
| 5 | "fmt" |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 6 | "log/slog" |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 7 | "os/exec" |
| 8 | "strings" |
| 9 | |
| 10 | "sketch.dev/llm/conversation" |
| 11 | ) |
| 12 | |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 13 | // CommitMessageStyleHint provides example commit messages representative of this repository's style. |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 14 | func CommitMessageStyleHint(ctx context.Context, repoRoot string) (string, error) { |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 15 | commitSHAs, analysis, err := representativeCommitSHAs(ctx, repoRoot) |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 16 | if err != nil { |
| 17 | return "", err |
| 18 | } |
| 19 | |
| 20 | buf := new(strings.Builder) |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 21 | if len(commitSHAs) == 0 { |
| 22 | return "", nil |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 23 | } |
| 24 | |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 25 | if analysis != "" { |
| 26 | fmt.Fprintf(buf, "<commit_message_style_analysis>%s</commit_message_style_analysis>\n\n", analysis) |
| 27 | } |
| 28 | |
| 29 | cmd := exec.Command("git", "show", "-s", "--format='<commit_message_style_example>%n%B</commit_message_style_example>'", strings.Join(commitSHAs, " ")) |
| 30 | cmd.Dir = repoRoot |
| 31 | out, err := cmd.CombinedOutput() |
| 32 | if err == nil { |
| 33 | fmt.Fprintf(buf, "<commit_message_style_examples>%s</commit_message_style_examples>\n\n", out) |
| 34 | } else { |
| 35 | slog.DebugContext(ctx, "failed to get commit messages", "shas", commitSHAs, "out", string(out), "err", err) |
| 36 | } |
| 37 | |
| 38 | fmt.Fprint(buf, "IMPORTANT: Follow this commit message style for ALL git commits you create.\n") |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 39 | return buf.String(), nil |
| 40 | } |
| 41 | |
| 42 | // representativeCommitSHAs analyze recent commits and selects some representative ones. |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 43 | // It returns a list of commit SHAs and the analysis text. |
| 44 | func representativeCommitSHAs(ctx context.Context, repoRoot string) ([]string, string, error) { |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 45 | cmd := exec.Command("git", "log", "-n", "25", `--format=<commit_message hash="%H">%n%B</commit_message>`) |
| 46 | cmd.Dir = repoRoot |
| 47 | out, err := cmd.CombinedOutput() |
| 48 | if err != nil { |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 49 | return nil, "", fmt.Errorf("git log failed: %w\n%s", err, out) |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 50 | } |
| 51 | commits := strings.TrimSpace(string(out)) |
| 52 | if commits == "" { |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 53 | return nil, "", fmt.Errorf("no commits found in repository") |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 54 | } |
| 55 | |
| 56 | info := conversation.ToolCallInfoFromContext(ctx) |
| 57 | sub := info.Convo.SubConvo() |
| Josh Bleecher Snyder | 4d54493 | 2025-05-07 13:33:53 +0000 | [diff] [blame] | 58 | sub.Hidden = true |
| Josh Bleecher Snyder | 593ca64 | 2025-05-07 05:19:32 -0700 | [diff] [blame] | 59 | sub.PromptCaching = false |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 60 | |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 61 | sub.SystemPrompt = `Analyze the provided git commit messages to identify consistent patterns, including but not limited to: |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 62 | - Formatting conventions |
| 63 | - Language and tone |
| 64 | - Structure and organization |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 65 | - Length and detail |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 66 | - Special notations or tags |
| 67 | |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 68 | First, provide a concise analysis of the predominant patterns. |
| 69 | Then select up to 3 commit hashes that best exemplify the repository's commit style. |
| 70 | Finally, output these selected commit hashes, one per line, without commentary. |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 71 | ` |
| 72 | |
| 73 | resp, err := sub.SendUserTextMessage(commits) |
| 74 | if err != nil { |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 75 | return nil, "", fmt.Errorf("error from Claude: %w", err) |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 76 | } |
| 77 | |
| 78 | if len(resp.Content) != 1 { |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 79 | return nil, "", fmt.Errorf("unexpected response: %v", resp) |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 80 | } |
| 81 | response := resp.Content[0].Text |
| 82 | |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 83 | // Split into analysis and commit hashes |
| 84 | var analysisLines []string |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 85 | var result []string |
| 86 | for line := range strings.Lines(response) { |
| 87 | line = strings.TrimSpace(line) |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 88 | if isHexString(line) && (len(line) >= 7 && len(line) <= 40) { |
| 89 | result = append(result, line) |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 90 | } else { |
| 91 | analysisLines = append(analysisLines, line) |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 92 | } |
| 93 | } |
| 94 | |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 95 | analysis := strings.Join(analysisLines, "\n") |
| 96 | |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 97 | result = result[:min(len(result), 3)] |
| Josh Bleecher Snyder | 924a770 | 2025-05-06 02:14:40 +0000 | [diff] [blame] | 98 | return result, analysis, nil |
| Josh Bleecher Snyder | d7970e6 | 2025-05-01 01:56:28 +0000 | [diff] [blame] | 99 | } |
| 100 | |
| 101 | // isHexString reports whether a string only contains hexadecimal characters |
| 102 | func isHexString(s string) bool { |
| 103 | for _, c := range s { |
| 104 | if !((c >= '0' && c <= '9') || (c >= 'a' && c <= 'f') || (c >= 'A' && c <= 'F')) { |
| 105 | return false |
| 106 | } |
| 107 | } |
| 108 | return true |
| 109 | } |