Skip to content

Commit be27c30

Browse files
committed
Changing commit naming to use sonnet not haiku
1 parent 6cc1d95 commit be27c30

File tree

6 files changed

+16
-15
lines changed

6 files changed

+16
-15
lines changed

src/lib/CommitManager.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -569,7 +569,7 @@ describe('CommitManager', () => {
569569
const claudeCall = vi.mocked(claude.launchClaude).mock.calls[0]
570570
expect(claudeCall[1]).toEqual(
571571
expect.objectContaining({
572-
model: 'claude-3-5-haiku-20241022',
572+
model: 'sonnet',
573573
})
574574
)
575575
})

src/lib/CommitManager.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -202,7 +202,7 @@ export class CommitManager {
202202
const claudeOptions = {
203203
headless: true,
204204
addDir: worktreePath,
205-
model: 'claude-3-5-haiku-20241022', // Fast, cost-effective model
205+
model: 'sonnet', // Fast, cost-effective model
206206
timeout: 120000, // 120 second timeout
207207
}
208208
logger.debug('Claude API call parameters:', {

src/utils/claude.test.ts

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,7 @@ describe('claude utils', () => {
117117
expect(result).toBe(output)
118118
expect(execa).toHaveBeenCalledWith(
119119
'claude',
120-
['-p', '--print'],
120+
['-p'],
121121
expect.objectContaining({
122122
input: prompt,
123123
timeout: 1200000, // 20 minutes
@@ -134,12 +134,12 @@ describe('claude utils', () => {
134134

135135
await launchClaude(prompt, {
136136
headless: true,
137-
model: 'claude-3-5-haiku-20241022',
137+
model: 'opus',
138138
})
139139

140140
expect(execa).toHaveBeenCalledWith(
141141
'claude',
142-
['-p', '--print', '--model', 'claude-3-5-haiku-20241022'],
142+
['-p', '--model', 'opus'],
143143
expect.any(Object)
144144
)
145145
})
@@ -158,7 +158,7 @@ describe('claude utils', () => {
158158

159159
expect(execa).toHaveBeenCalledWith(
160160
'claude',
161-
['-p', '--print', '--permission-mode', 'plan'],
161+
['-p', '--permission-mode', 'plan'],
162162
expect.any(Object)
163163
)
164164
})
@@ -177,7 +177,7 @@ describe('claude utils', () => {
177177

178178
expect(execa).toHaveBeenCalledWith(
179179
'claude',
180-
['-p', '--print'],
180+
['-p'],
181181
expect.any(Object)
182182
)
183183
})
@@ -197,7 +197,7 @@ describe('claude utils', () => {
197197

198198
expect(execa).toHaveBeenCalledWith(
199199
'claude',
200-
['-p', '--print', '--add-dir', workspacePath],
200+
['-p', '--add-dir', workspacePath],
201201
expect.any(Object)
202202
)
203203
})
@@ -217,7 +217,7 @@ describe('claude utils', () => {
217217

218218
expect(execa).toHaveBeenCalledWith(
219219
'claude',
220-
['-p', '--print', '--add-dir', workspacePath],
220+
['-p', '--add-dir', workspacePath],
221221
expect.objectContaining({
222222
input: prompt,
223223
timeout: 1200000,
@@ -239,7 +239,7 @@ describe('claude utils', () => {
239239

240240
expect(execa).toHaveBeenCalledWith(
241241
'claude',
242-
['-p', '--print'],
242+
['-p'],
243243
expect.objectContaining({
244244
input: prompt,
245245
timeout: 1200000,
@@ -524,7 +524,7 @@ describe('claude utils', () => {
524524
expect(result).toBe('feat/issue-123-user-authentication')
525525
expect(execa).toHaveBeenCalledWith(
526526
'claude',
527-
['-p', '--print', '--model', 'claude-3-5-haiku-20241022'],
527+
['-p', '--model', 'sonnet'],
528528
expect.objectContaining({
529529
input: expect.stringContaining(issueTitle),
530530
})

src/utils/claude.ts

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ export async function launchClaude(
5959
const args: string[] = []
6060

6161
if (headless) {
62-
args.push('-p', '--print')
62+
args.push('-p')
6363
}
6464

6565
if (model) {
@@ -81,6 +81,7 @@ export async function launchClaude(
8181
input: prompt,
8282
timeout,
8383
...(addDir && { cwd: addDir }), // Run Claude in the worktree directory
84+
verbose: true,
8485
})
8586
return result.stdout.trim()
8687
} else {
@@ -143,7 +144,7 @@ export async function launchClaude(
143144
export async function generateBranchName(
144145
issueTitle: string,
145146
issueNumber: number,
146-
model: string = 'claude-3-5-haiku-20241022'
147+
model: string = 'sonnet'
147148
): Promise<string> {
148149
try {
149150
// Check if Claude CLI is available

src/utils/github.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -369,7 +369,7 @@ describe('github utils', () => {
369369
const name = await strategy.generate(123, 'Add authentication')
370370

371371
expect(name).toBe('feat/issue-123-add-authentication')
372-
expect(mockGenerateBranchName).toHaveBeenCalledWith('Add authentication', 123, 'claude-3-5-haiku-20241022')
372+
expect(mockGenerateBranchName).toHaveBeenCalledWith('Add authentication', 123, 'sonnet')
373373
})
374374

375375
it('should use custom Claude model', async () => {

src/utils/github.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -164,7 +164,7 @@ export class SimpleBranchNameStrategy implements BranchNameStrategy {
164164
}
165165

166166
export class ClaudeBranchNameStrategy implements BranchNameStrategy {
167-
constructor(private claudeModel = 'claude-3-5-haiku-20241022') {}
167+
constructor(private claudeModel = 'sonnet') {}
168168

169169
async generate(issueNumber: number, title: string): Promise<string> {
170170
// Import dynamically to avoid circular dependency

0 commit comments

Comments
 (0)