Skip to content

Commit 642eea4

Browse files
committed
refactor(core): remove duplicate claude provider aliases
1 parent f5863b7 commit 642eea4

2 files changed

Lines changed: 8 additions & 34 deletions

File tree

main.go

Lines changed: 7 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@ func createApp() *cli.App {
4646
Flags: []cli.Flag{
4747
&cli.StringFlag{
4848
Name: "provider",
49-
Usage: "AI provider to use (claude*, claude, claude, gemini, copilot)",
50-
Value: "claude", // Default to claude like gitcommit function
49+
Usage: "AI provider to use (claude*, gemini, copilot)",
50+
Value: "claude",
5151
},
5252
},
5353
Commands: []*cli.Command{
@@ -86,7 +86,7 @@ func createStagedCommand() *cli.Command {
8686
Flags: []cli.Flag{
8787
&cli.StringFlag{
8888
Name: "provider",
89-
Usage: "AI provider to use (claude*, claude, claude, gemini, copilot)",
89+
Usage: "AI provider to use (claude*, gemini, copilot)",
9090
Value: "claude",
9191
},
9292
},
@@ -102,7 +102,7 @@ func createAllCommand() *cli.Command {
102102
Flags: []cli.Flag{
103103
&cli.StringFlag{
104104
Name: "provider",
105-
Usage: "AI provider to use (claude*, claude, claude, gemini, copilot)",
105+
Usage: "AI provider to use (claude*, gemini, copilot)",
106106
Value: "claude",
107107
},
108108
},
@@ -118,7 +118,7 @@ func createUntrackedCommand() *cli.Command {
118118
Flags: []cli.Flag{
119119
&cli.StringFlag{
120120
Name: "provider",
121-
Usage: "AI provider to use (claude*, claude, claude, gemini, copilot)",
121+
Usage: "AI provider to use (claude*, gemini, copilot)",
122122
Value: "claude",
123123
},
124124
},
@@ -414,30 +414,8 @@ func analyzeUntrackedFiles() (string, error) {
414414
func callAIAPI(analysisInput, provider string) (string, error) {
415415
prompt := commitrules.GetPrompt(analysisInput)
416416

417-
var cmd *exec.Cmd
418-
419-
// Handle special providers that are typically defined as aliases
420-
switch provider {
421-
case "claude":
422-
// Expand the claude alias with actual environment variables
423-
cmd = exec.Command("claude")
424-
cmd.Env = append(os.Environ(),
425-
"ANTHROPIC_BASE_URL=https://open.bigmodel.cn/api/anthropic",
426-
"ANTHROPIC_API_KEY=REDACTED_API_KEY",
427-
"ANTHROPIC_MODEL=glm-4.6",
428-
)
429-
case "claude":
430-
// Expand the claude alias with actual environment variables
431-
cmd = exec.Command("claude")
432-
cmd.Env = append(os.Environ(),
433-
"ANTHROPIC_BASE_URL=https://open.bigmodel.cn/api/anthropic",
434-
"ANTHROPIC_API_KEY=REDACTED_API_KEY",
435-
"ANTHROPIC_MODEL=glm-4.6",
436-
)
437-
default:
438-
// For other providers, try direct execution first
439-
cmd = exec.Command(provider)
440-
}
417+
// Execute the provider command directly
418+
cmd := exec.Command(provider)
441419

442420
cmd.Stdin = strings.NewReader(prompt)
443421

utils_test.go

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -159,13 +159,9 @@ func TestProviderValidation(t *testing.T) {
159159
}{
160160
// Claude variants
161161
{providerClaude, true, providerClaude},
162-
{"claudex", true, providerClaude},
163-
{"claude", true, providerClaude},
164-
{"claude", true, providerClaude},
165162
{"claude-external", true, providerClaude},
166163
{"claude-custom", true, providerClaude},
167164
{"claude-2", true, providerClaude},
168-
{"claudex-external", true, providerClaude},
169165

170166
// Other providers
171167
{providerGemini, true, providerGemini},
@@ -351,7 +347,7 @@ func BenchmarkCleanCommitMessageComplex(b *testing.B) {
351347
}
352348

353349
func BenchmarkProviderValidation(b *testing.B) {
354-
providers := []string{providerClaude, "claude", "claude-external", providerGemini, providerCopilot, "unknown"}
350+
providers := []string{providerClaude, "claude-external", providerGemini, providerCopilot, "unknown"}
355351
for i := 0; i < b.N; i++ {
356352
provider := providers[i%len(providers)]
357353
switch {

0 commit comments

Comments
 (0)