feat: @goose in terminal (native terminal support) #5887

DOsinga · 2025-11-26T11:47:50Z

Summary

Douwe's take on this saga :)

Term command is a new way to use goose without the builtin REPL - instead the session is tied right into your terminal

- Fix bash/zsh: properly quote goose binary path to handle spaces - Extract session creation logic to ensure_terminal_session helper - Reduces code duplication between handle_term_log and handle_term_run

DOsinga · 2025-11-26T11:50:29Z

crates/goose-cli/src/commands/term.rs

+        chrono::Utc::now().timestamp_millis(),
+        vec![MessageContent::text(command)],
+    )
+    .with_metadata(MessageMetadata::user_only());


we could leave this out since we truncate again

do messages have an LLM only state? because these would be weird/spammy if you saw them in a GUI i think

if they are like what I saw, isn't end of the world.

github-actions · 2025-11-26T11:50:54Z

PR Preview Action v1.6.0
Preview removed because the pull request was closed.
2025-12-01 06:42 UTC

Copilot

Pull request overview

This PR adds terminal integration features to Goose, enabling users to interact with Goose directly from their shell prompt without switching to a separate REPL session. The integration tracks shell command history and provides it as context when users ask questions.

Key changes:

New SessionType::Terminal enum variant for terminal-specific sessions
New goose term CLI commands for shell initialization, command logging, and running prompts
Comprehensive documentation explaining setup and usage across multiple shells (bash, zsh, fish, PowerShell)

Reviewed changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
`documentation/docs/guides/terminal-integration.md`	Complete user guide for terminal integration feature including setup, usage examples, and technical details
`crates/goose/src/session/session_manager.rs`	Adds `SessionType::Terminal` variant for terminal-integrated sessions
`crates/goose-cli/src/commands/term.rs`	Core implementation: shell initialization script generation, command logging, and prompt execution with shell history context
`crates/goose-cli/src/commands/mod.rs`	Exports new `term` command module
`crates/goose-cli/src/cli.rs`	CLI interface for `goose term` commands with subcommands for init, log, run, and info
`Cargo.lock`	Dependency version update (windows-sys indirect dependency)

documentation/docs/guides/terminal-integration.md

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 4 comments.

crates/goose-cli/src/commands/term.rs

documentation/docs/guides/terminal-integration.md

crates/goose-cli/src/commands/term.rs

Copilot · 2025-11-26T13:03:01Z

crates/goose/src/session/session_manager.rs

+                   COUNT(m.id) as message_count
+            FROM sessions s
+            INNER JOIN messages m ON s.id = m.session_id


The INNER JOIN with messages excludes sessions with no messages. This breaks named session lookup - a newly created terminal session won't be found on the next term init call until it has messages. Use LEFT JOIN instead and COALESCE(COUNT(m.id), 0) for the message count.

Suggested change

COUNT(m.id) as message_count

FROM sessions s

INNER JOIN messages m ON s.id = m.session_id

COALESCE(COUNT(m.id), 0) as message_count

FROM sessions s

LEFT JOIN messages m ON s.id = m.session_id

@DOsinga this sounds so convincing but I am not so sure (also if it is true, I don't think this is a bad thing... I was wondering how do we get a fresh session)?

baxen

LGTM!

Feel free to retarget to my outstanding branch/PR and i will address the nits, or if you have time can merge directly

baxen · 2025-11-26T19:38:26Z

crates/goose-cli/src/commands/term.rs

+
+static BASH_CONFIG: ShellConfig = ShellConfig {
+    script_template: r#"export GOOSE_SESSION_ID="{session_id}"
+alias gt='{goose_bin} term run'


nit: i'd remove this now, but i like having both @goose and @g

baxen · 2025-11-26T19:42:07Z

crates/goose-cli/src/commands/term.rs

+        chrono::Utc::now().timestamp_millis(),
+        vec![MessageContent::text(command)],
+    )
+    .with_metadata(MessageMetadata::user_only());


do messages have an LLM only state? because these would be weird/spammy if you saw them in a GUI i think

baxen · 2025-11-26T19:45:22Z

documentation/docs/guides/terminal-integration.md

+---
+unlisted: true
+---
+# Terminal Integration


a todo for future, would be nice to show how to set this up with
https://starship.rs/
and
https://github.com/ianthehenry/zsh-autoquoter

michaelneale · 2025-11-27T01:27:56Z

ok, needs blessing from @blackgirlbytes (as this is a new PR with doc change) and I think we can get this in

Copilot

Pull request overview

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

documentation/docs/guides/terminal-integration.md

crates/goose-cli/src/commands/term.rs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 1 comment.

crates/goose-cli/src/commands/term.rs

@goose

* origin/main: Feat/automatic update installation (#5345) fix: Added "Merged consecutive assistant messages" to the acceptable issues for moim injection check (#5933) fix: anthropic provider model fetching (#5932) [MCP-UI] add CSP for images to proxy HTML (#5931) fix: correct typo in blog post (AIMDOEL -> AIMODEL) (#5902) feat: @goose in terminal (native terminal support) (#5887) docs: adding AI-friendly features (#5918) Blog/advent of ai announcement (#5917) Extension selector behind ALPHA flag (#5892)

@goose

* 'main' of github.com:block/goose: fix: correct typo in blog post (AIMDOEL -> AIMODEL) (#5902) feat: @goose in terminal (native terminal support) (#5887) docs: adding AI-friendly features (#5918) Blog/advent of ai announcement (#5917)

@goose

…nses-streaming * 'main' of github.com:block/goose: Feat/automatic update installation (#5345) fix: Added "Merged consecutive assistant messages" to the acceptable issues for moim injection check (#5933) fix: anthropic provider model fetching (#5932) [MCP-UI] add CSP for images to proxy HTML (#5931) fix: correct typo in blog post (AIMDOEL -> AIMODEL) (#5902) feat: @goose in terminal (native terminal support) (#5887) docs: adding AI-friendly features (#5918) Blog/advent of ai announcement (#5917) Extension selector behind ALPHA flag (#5892) blog: typo fixes (#5896) blog: fixing img url (#5895) blog: MCPs for Developers (#5884) docs: Extension Manager MCP (#5883) Update cleanup marker logic for Fedora users. (#5868) Improve AWS credential loading and configuration handling in BedrockProvider (#5699)

@goose

…0-5147 * 'main' of github.com:block/goose: (243 commits) chore: upgrade npm packages (#5951) feat: ActionRequired (#5897) feat(acp): support loading sessions in acp (#5942) docs: add videos to multi-model page (#5938) docs: promote planning guide (#5934) fix: use a lock to ensure only need to run tunnel just in case multiple go… (#5885) Feat: Added custom headers and toggle keyring CLI options (#5017) Feat/automatic update installation (#5345) fix: Added "Merged consecutive assistant messages" to the acceptable issues for moim injection check (#5933) fix: anthropic provider model fetching (#5932) [MCP-UI] add CSP for images to proxy HTML (#5931) fix: correct typo in blog post (AIMDOEL -> AIMODEL) (#5902) feat: @goose in terminal (native terminal support) (#5887) docs: adding AI-friendly features (#5918) Blog/advent of ai announcement (#5917) Extension selector behind ALPHA flag (#5892) blog: typo fixes (#5896) blog: fixing img url (#5895) blog: MCPs for Developers (#5884) docs: Extension Manager MCP (#5883) ... # Conflicts: # crates/goose-server/src/routes/config_management.rs # crates/goose/src/providers/mod.rs # ui/desktop/openapi.json # ui/desktop/src/api/sdk.gen.ts # ui/desktop/src/api/types.gen.ts # ui/desktop/src/components/ProgressiveMessageList.tsx

baxen and others added 17 commits November 22, 2025 12:18

[feat] add term command

c345c82

Term command is a new way to use goose without the builtin REPL - instead the session is tied right into your terminal

add info command for prompt integration

6cb829f

break up migration function for clippy

f36f678

comments

3b9d53b

Address remaining PR comments

5b768e5

- Fix bash/zsh: properly quote goose binary path to handle spaces - Extract session creation logic to ensure_terminal_session helper - Reduces code duplication between handle_term_log and handle_term_run

unlist

155e086

tweaks

e1be322

allow gt to run without quotes or without even using gt prefix

67a6074

simplifying out of session manager

fe90b74

session not hidden and used paths

ee32410

session naming

c6c62a0

using new alias and docs

fe6616c

Merge branch 'main' into term-mic

a56b265

dependency tidy

255d863

not async

2668eb5

remove silly comments

9942fda

Use messages

ae2a6e9

Copilot AI review requested due to automatic review settings November 26, 2025 11:47

DOsinga requested a review from a team as a code owner November 26, 2025 11:47

Copilot started reviewing on behalf of DOsinga November 26, 2025 11:48 View session

DOsinga commented Nov 26, 2025

View reviewed changes

Copilot finished reviewing on behalf of DOsinga November 26, 2025 11:52

Copilot AI reviewed Nov 26, 2025

View reviewed changes

documentation/docs/guides/terminal-integration.md Outdated Show resolved Hide resolved

Douwe Osinga added 2 commits November 26, 2025 13:48

Named sessions

9460b33

Update documentation - thank you goose

5016de9

Copilot AI review requested due to automatic review settings November 26, 2025 12:58

Copilot started reviewing on behalf of DOsinga November 26, 2025 12:59 View session

Copilot finished reviewing on behalf of DOsinga November 26, 2025 13:02

Copilot AI reviewed Nov 26, 2025

View reviewed changes

Update instructions

fd46023

baxen approved these changes Nov 26, 2025

View reviewed changes

michaelneale changed the title ~~Term dmo~~ feat: @goose in terminal (native terminal support) Nov 27, 2025

michaelneale self-assigned this Nov 27, 2025

michaelneale mentioned this pull request Nov 27, 2025

feat: enhancing new terminal feature #5852

Closed

drop the old gt alias to save conflict

e16c061

Copilot AI review requested due to automatic review settings November 27, 2025 01:27

michaelneale approved these changes Nov 27, 2025

View reviewed changes

Copilot started reviewing on behalf of michaelneale November 27, 2025 01:27 View session

michaelneale mentioned this pull request Nov 27, 2025

[draft] [feat] add term command #5847

Closed

1 task

Copilot finished reviewing on behalf of michaelneale November 27, 2025 01:30

Copilot AI reviewed Nov 27, 2025

View reviewed changes

documentation/docs/guides/terminal-integration.md Outdated Show resolved Hide resolved

crates/goose-cli/src/commands/term.rs Show resolved Hide resolved

blackgirlbytes approved these changes Dec 1, 2025

View reviewed changes

Update documentation/docs/guides/terminal-integration.md

c6df680

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings December 1, 2025 04:00

Copilot started reviewing on behalf of michaelneale December 1, 2025 04:01 View session

Copilot finished reviewing on behalf of michaelneale December 1, 2025 04:04

Copilot AI reviewed Dec 1, 2025

View reviewed changes

crates/goose-cli/src/commands/term.rs Show resolved Hide resolved

michaelneale merged commit 5f50198 into main Dec 1, 2025
24 checks passed

michaelneale deleted the term-dmo branch December 1, 2025 06:40

github-actions bot mentioned this pull request Dec 4, 2025

chore(release): release version 1.16.0 (minor) #5985

Merged

emma-squared mentioned this pull request Dec 10, 2025

Docs for new terminal integration feature #6042

Merged

10 tasks

feat: @goose in terminal (native terminal support) #5887

feat: @goose in terminal (native terminal support) #5887

Uh oh!

Conversation

DOsinga commented Nov 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

DOsinga Nov 26, 2025

Choose a reason for hiding this comment

Uh oh!

baxen Nov 26, 2025

Choose a reason for hiding this comment

Uh oh!

michaelneale Nov 27, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Nov 26, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Nov 26, 2025

Choose a reason for hiding this comment

Uh oh!

michaelneale Nov 27, 2025

Choose a reason for hiding this comment

Uh oh!

baxen left a comment

Choose a reason for hiding this comment

Uh oh!

baxen Nov 26, 2025

Choose a reason for hiding this comment

Uh oh!

baxen Nov 26, 2025

Choose a reason for hiding this comment

Uh oh!

baxen Nov 26, 2025

Choose a reason for hiding this comment

Uh oh!

michaelneale commented Nov 27, 2025

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

DOsinga commented Nov 26, 2025 •

edited

Loading

github-actions bot commented Nov 26, 2025 •

edited

Loading