Hook thinking status

[DOsinga]

Summary

continues the work from:

#6702

[Copilot]

Pull request overview

This PR adds status hook functionality to notify external scripts when Goose transitions between "waiting" (ready for input) and "thinking" (processing) states, continuing work from PR #6702. This enables use cases like updating tmux window indicators to show session status.

Changes:

• Adds run_status_hook() function that spawns a thread to execute a user-configured hook script
• Calls the hook with "waiting" status before prompting for user input
• Calls the hook with "thinking" status before processing agent responses

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
crates/goose-cli/src/session/output.rs	Adds run_status_hook() function that executes GOOSE_STATUS_HOOK script with status argument via shell command
crates/goose-cli/src/session/mod.rs	Integrates status hook calls into interactive loop at appropriate state transitions

[Copilot]

Execute the hook directly without a shell to avoid potential injection. Use Command::new(&hook).arg(&status) instead of passing through sh -c. This is safer and simpler.

Suggested change

	#[cfg(target_os = "windows")]
	let result = std::process::Command::new("cmd")
	.arg("/C")
	.arg(format!("{} {}", hook, status))
	.stdin(std::process::Stdio::null())
	.stdout(std::process::Stdio::null())
	.stderr(std::process::Stdio::null())
	.status();
	#[cfg(not(target_os = "windows"))]
	let result = std::process::Command::new("sh")
	.arg("-c")
	.arg(format!("{} {}", hook, status))
	.stdin(std::process::Stdio::null())
	.stdout(std::process::Stdio::null())
	.stderr(std::process::Stdio::null())
	.status();
	let result = std::process::Command::new(&hook)
	.arg(&status)
	.stdin(std::process::Stdio::null())
	.stdout(std::process::Stdio::null())
	.stderr(std::process::Stdio::null())
	.status();

[Copilot]

Execute the hook directly without a shell to avoid potential injection. Use Command::new(&hook).arg(&status) instead of passing through cmd /C. This is safer and simpler.

Suggested change

	#[cfg(target_os = "windows")]
	let result = std::process::Command::new("cmd")
	.arg("/C")
	.arg(format!("{} {}", hook, status))
	.stdin(std::process::Stdio::null())
	.stdout(std::process::Stdio::null())
	.stderr(std::process::Stdio::null())
	.status();
	#[cfg(not(target_os = "windows"))]
	let result = std::process::Command::new("sh")
	.arg("-c")
	.arg(format!("{} {}", hook, status))
	.stdin(std::process::Stdio::null())
	.stdout(std::process::Stdio::null())
	.stderr(std::process::Stdio::null())
	.status();
	let result = std::process::Command::new(&hook)
	.arg(&status)
	.stdin(std::process::Stdio::null())
	.stdout(std::process::Stdio::null())
	.stderr(std::process::Stdio::null())
	.status();