Update launchClaudeInNewTerminalWindow to use 'hb ignite' command instead of building complex Claude command

Author: acreeger

src/utils/claude.test.ts

@@ -412,7 +412,7 @@ describe('claude utils', () => {
 	})
 
 	describe('launchClaudeInNewTerminalWindow', () => {
-		it('should open new terminal window with Claude command', async () => {
+		it('should open new terminal window with hb ignite command', async () => {
 			const prompt = 'Work on this issue'
 			const workspacePath = '/path/to/workspace'
 
@@ -423,7 +423,9 @@ describe('claude utils', () => {
 
 			await launchClaudeInNewTerminalWindow(prompt, { workspacePath })
 
-			// Verify osascript was called for terminal window
+			// Verify osascript was called for terminal window with hb ignite command
+			const applescript = vi.mocked(execa).mock.calls[0][1]?.[1] as string
+			expect(applescript).toContain('hb ignite')
 			expect(execa).toHaveBeenCalledWith(
 				'osascript',
 				['-e', expect.stringContaining('tell application "Terminal"')]
@@ -450,7 +452,9 @@ describe('claude utils', () => {
 
 			await launchClaudeInNewTerminalWindow(prompt, { workspacePath, branchName })
 
-			// Verify terminal window was opened
+			// Verify terminal window was opened with hb ignite
+			const applescript = vi.mocked(execa).mock.calls[0][1]?.[1] as string
+			expect(applescript).toContain('hb ignite')
 			expect(execa).toHaveBeenCalledWith(
 				'osascript',
 				['-e', expect.stringContaining('tell application "Terminal"')]
@@ -470,9 +474,10 @@ describe('claude utils', () => {
 
 			await launchClaudeInNewTerminalWindow(prompt, { workspacePath })
 
-			// Verify .env sourcing is included
+			// Verify .env sourcing is included and hb ignite is used
 			const applescript = vi.mocked(execa).mock.calls[0][1]?.[1] as string
 			expect(applescript).toContain('source .env')
+			expect(applescript).toContain('hb ignite')
 			expect(existsSync).toHaveBeenCalledWith('/path/to/workspace/.env')
 		})
 
@@ -489,12 +494,13 @@ describe('claude utils', () => {
 
 			await launchClaudeInNewTerminalWindow(prompt, { workspacePath })
 
-			// Verify .env sourcing is NOT included
+			// Verify .env sourcing is NOT included but hb ignite is used
 			const applescript = vi.mocked(execa).mock.calls[0][1]?.[1] as string
 			expect(applescript).not.toContain('source .env')
+			expect(applescript).toContain('hb ignite')
 		})
 
-		it('should properly escape prompt for AppleScript', async () => {
+		it('should not build complex claude command with prompt', async () => {
 			const prompt = "Fix the user's \"authentication\" issue"
 			const workspacePath = '/path/to/workspace'
 
@@ -505,14 +511,14 @@ describe('claude utils', () => {
 
 			await launchClaudeInNewTerminalWindow(prompt, { workspacePath })
 
-			// Verify command was constructed
-			expect(execa).toHaveBeenCalledWith(
-				'osascript',
-				['-e', expect.stringContaining('tell application "Terminal"')]
-			)
+			// Verify simple hb ignite command is used, not complex claude command with prompt
+			const applescript = vi.mocked(execa).mock.calls[0][1]?.[1] as string
+			expect(applescript).toContain('hb ignite')
+			expect(applescript).not.toContain('--append-system-prompt')
+			expect(applescript).not.toContain(prompt)
 		})
 
-		it('should use --append-system-prompt flag for terminal window mode', async () => {
+		it('should use hb ignite instead of building claude command with args', async () => {
 			const prompt = 'Work on this issue'
 			const workspacePath = '/path/to/workspace'
 
@@ -523,9 +529,12 @@ describe('claude utils', () => {
 
 			await launchClaudeInNewTerminalWindow(prompt, { workspacePath })
 
-			// Verify --append-system-prompt is used in the command
+			// Verify hb ignite is used, not claude with model/permission args
 			const applescript = vi.mocked(execa).mock.calls[0][1]?.[1] as string
-			expect(applescript).toContain('--append-system-prompt')
+			expect(applescript).toContain('hb ignite')
+			expect(applescript).not.toContain('--model')
+			expect(applescript).not.toContain('--permission-mode')
+			expect(applescript).not.toContain('--add-dir')
 		})
 	})
 

src/utils/claude.ts

@@ -120,40 +120,22 @@ export async function launchClaude(
  * Ports the terminal window opening, coloring, and .env sourcing behavior
  */
 export async function launchClaudeInNewTerminalWindow(
-	prompt: string,
+	_prompt: string,
 	options: ClaudeCliOptions & {
 		workspacePath: string // Required for terminal window launch
 	}
 ): Promise<void> {
-	const { model, permissionMode, workspacePath, branchName } = options
+	const { workspacePath, branchName } = options
 
 	// Verify required parameter
 	if (!workspacePath) {
 		throw new Error('workspacePath is required for terminal window launch')
 	}
 
-	// Build command arguments (same as launchClaude)
-	const args: string[] = []
-
-	if (model) {
-		args.push('--model', model)
-	}
-
-	if (permissionMode && permissionMode !== 'default') {
-		args.push('--permission-mode', permissionMode)
-	}
-
-	args.push('--add-dir', workspacePath)
-	args.push('--add-dir', '/tmp') //TODO: Won't work on Windows
-
 	// Import terminal launcher for new terminal window creation
 	const { openTerminalWindow } = await import('./terminal.js')
 
-	// Build Claude command using --append-system-prompt flag
-	// This is semantically correct for system instructions in terminal window mode
-	const baseCommand = ['claude', ...args].join(' ')
-	const quotedSystemPrompt = `'${prompt.replace(/'/g, "'\\''")}'`
-	const claudeCommand = `${baseCommand} --append-system-prompt ${quotedSystemPrompt} -- 'Go!'`
+	const launchCommand = "hb ignite"
 
 	// Apply terminal background color if branch name available
 	let backgroundColor: { r: number; g: number; b: number } | undefined
@@ -175,7 +157,7 @@ export async function launchClaudeInNewTerminalWindow(
 	// Open new terminal window with Claude
 	await openTerminalWindow({
 		workspacePath,
-		command: claudeCommand,
+		command: launchCommand,
 		...(backgroundColor && { backgroundColor }),
 		includeEnvSetup: hasEnvFile, // source .env only if it exists
 	})