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