Skip to content

Comments

feat: iterating on summarize oneshot prompt#4113

Merged
katzdave merged 1 commit intomainfrom
alexhancock/summarize-prompt-iteration
Aug 15, 2025
Merged

feat: iterating on summarize oneshot prompt#4113
katzdave merged 1 commit intomainfrom
alexhancock/summarize-prompt-iteration

Conversation

@alexhancock
Copy link
Collaborator

@alexhancock alexhancock commented Aug 15, 2025

Trying to address some issues we're seeing with overly aggressive/short summaries during auto-compaction

Open to feedback on any of this, and also good methods to test

@alexhancock alexhancock requested a review from katzdave August 15, 2025 14:48
katzdave
katzdave reviewed Aug 15, 2025
View reviewed changes
crates/goose/src/prompts/summarize_oneshot.md
4. **Errors + Fixes** – Bugs, resolutions, user-driven changes
5. **Problem Solving** – Issues solved or in progress
6. **User Messages** – All user messages, exclude tool output
6. **User Messages** – All user messages including tool calls, but truncate long tool call arguments or results
Copy link
Collaborator

@katzdave katzdave Aug 15, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Notice any difference from this? I do think this has potential to help; seems like losing where you put a one-off script was one thing people were complaining about.

@alexhancock alexhancock force-pushed the alexhancock/summarize-prompt-iteration branch 3 times, most recently from ee6d33f to cf221d2 Compare August 15, 2025 18:17
DOsinga
DOsinga reviewed Aug 15, 2025
View reviewed changes
crates/goose/src/prompts/summarize_oneshot.md Outdated
Generate detailed summary of conversation to date.
Include user requests, your responses, and all technical content.
An llm context limit was reached when a user was in a working session with an agent (you)
Generate a version of the below messagees with only the most verbose parts removed
Copy link
Collaborator

@DOsinga DOsinga Aug 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo

@alexhancock alexhancock force-pushed the alexhancock/summarize-prompt-iteration branch from cf221d2 to 882f1f9 Compare August 15, 2025 18:37
@katzdave katzdave merged commit b000911 into main Aug 15, 2025
10 checks passed
@katzdave katzdave deleted the alexhancock/summarize-prompt-iteration branch August 15, 2025 18:57
katzdave pushed a commit that referenced this pull request Aug 15, 2025
@alexhancock @katzdave
feat: iterating on summarize oneshot prompt (#4113)
e99296c
zanesq added a commit that referenced this pull request Aug 15, 2025
@zanesq
Merge branch 'main' of github.com:block/goose into zane/upgrade-npm-p…
62ce9b4 
…ackages

* 'main' of github.com:block/goose:
  feat: iterating on summarize oneshot prompt (#4113)
  feat(mcp): Persist OAuth credentials to keyring (#4007)
  Sanitize Tags Unicode Block at prompt level (#4047)
  Fixing typos (#4114)
alexhancock added a commit that referenced this pull request Aug 16, 2025
@alexhancock
feat: iterating on summarize oneshot prompt (#4113)
3da6098
alexhancock added a commit that referenced this pull request Aug 16, 2025
@alexhancock
feat: iterating on summarize oneshot prompt (#4113)
583b976
jsibbison-square added a commit that referenced this pull request Aug 18, 2025
@jsibbison-square
Merge remote-tracking branch 'origin/main' into jsibbison-20250811-to…
18b2d76 
…ol-visibility

* origin/main: (26 commits)
  feat: adds cursor-agent as a cli provider (#4101)
  chore: remove vector search tool selection strategy (#3933)
  docs: add streamable_http install links (#4130)
  feat: iterating on summarize oneshot prompt (#4113)
  feat(mcp): Persist OAuth credentials to keyring (#4007)
  Sanitize Tags Unicode Block at prompt level (#4047)
  Fixing typos (#4114)
  chore(release): release version 1.4.0 (#4069)
  E2E tests working again (#4103)
  chore: Delete ARCHITECTURE.md (#4108)
  Add Youtube short to Kiwi MCP tutorial (#4107)
  docs: fix experimental warning (#4102)
  fix not being able to add spaces to activity message area (#4055)
  remove and cleanup unused code (#4074)
  docs: Add missing references to smart_approve mode. (#4094)
  quick typo fix pulse blog (#4095)
  fix: update dictation settings handling and improve user feedback (#4093)
  chore: add info tooltip to share session button (#4038)
  Reroute routes (#4088)
  fix(cli): fix compiling issue on windows system by adding a missing param (#4019)
  ...
This was referenced Aug 18, 2025
Closed
Merged
katzdave added a commit that referenced this pull request Aug 18, 2025
@katzdave
Merge branch 'main' of github.com:block/goose into dkatz/auto-compact…
e261482 
…-messaging

* 'main' of github.com:block/goose: (26 commits)
  fix: consistent font sizing in ToolCallWithResponse (#4167)
  Temporarily disable TODO Tool (#4158)
  docs: add integrated MCP server config to jetbrains tutorial  (#4120)
  docs: remove figma MCP from suggested servers (#4123)
  Blog: The AI Skeptic’s Guide to Context Windows (#4152)
  Docs: Auto-compact context (#4116)
  chore(deps): bump brace-expansion from 1.1.11 to 1.1.12 in /documentation (#4149)
  Recipe config to limit tool availability (#4020)
  docs: fix warning message (#4148)
  feat: adds cursor-agent as a cli provider (#4101)
  chore: remove vector search tool selection strategy (#3933)
  docs: add streamable_http install links (#4130)
  feat: iterating on summarize oneshot prompt (#4113)
  feat(mcp): Persist OAuth credentials to keyring (#4007)
  Sanitize Tags Unicode Block at prompt level (#4047)
  Fixing typos (#4114)
  chore(release): release version 1.4.0 (#4069)
  E2E tests working again (#4103)
  chore: Delete ARCHITECTURE.md (#4108)
  Add Youtube short to Kiwi MCP tutorial (#4107)
  ...
michaelneale added a commit that referenced this pull request Aug 19, 2025
@michaelneale
Merge branch 'main' into micn/hi-viz-autovisualiser
eb9d1f9 
* main: (67 commits)
  blog: Transforming AI Assistance with Goose Mentor Mode (#4151)
  upgraded all npm packages and fixed related issues (#4072)
  Docs: @-mentions in goosehints (#4171)
  fix: consistent font sizing in ToolCallWithResponse (#4167)
  Temporarily disable TODO Tool (#4158)
  docs: add integrated MCP server config to jetbrains tutorial  (#4120)
  docs: remove figma MCP from suggested servers (#4123)
  Blog: The AI Skeptic’s Guide to Context Windows (#4152)
  Docs: Auto-compact context (#4116)
  chore(deps): bump brace-expansion from 1.1.11 to 1.1.12 in /documentation (#4149)
  Recipe config to limit tool availability (#4020)
  docs: fix warning message (#4148)
  feat: adds cursor-agent as a cli provider (#4101)
  chore: remove vector search tool selection strategy (#3933)
  docs: add streamable_http install links (#4130)
  feat: iterating on summarize oneshot prompt (#4113)
  feat(mcp): Persist OAuth credentials to keyring (#4007)
  Sanitize Tags Unicode Block at prompt level (#4047)
  Fixing typos (#4114)
  chore(release): release version 1.4.0 (#4069)
  ...
michaelneale added a commit that referenced this pull request Aug 19, 2025
@michaelneale
Merge branch 'main' into micn/multi-model-multi-provider-autopilot
a74ec94 
* main: (67 commits)
  blog: Transforming AI Assistance with Goose Mentor Mode (#4151)
  upgraded all npm packages and fixed related issues (#4072)
  Docs: @-mentions in goosehints (#4171)
  fix: consistent font sizing in ToolCallWithResponse (#4167)
  Temporarily disable TODO Tool (#4158)
  docs: add integrated MCP server config to jetbrains tutorial  (#4120)
  docs: remove figma MCP from suggested servers (#4123)
  Blog: The AI Skeptic’s Guide to Context Windows (#4152)
  Docs: Auto-compact context (#4116)
  chore(deps): bump brace-expansion from 1.1.11 to 1.1.12 in /documentation (#4149)
  Recipe config to limit tool availability (#4020)
  docs: fix warning message (#4148)
  feat: adds cursor-agent as a cli provider (#4101)
  chore: remove vector search tool selection strategy (#3933)
  docs: add streamable_http install links (#4130)
  feat: iterating on summarize oneshot prompt (#4113)
  feat(mcp): Persist OAuth credentials to keyring (#4007)
  Sanitize Tags Unicode Block at prompt level (#4047)
  Fixing typos (#4114)
  chore(release): release version 1.4.0 (#4069)
  ...
zanesq added a commit that referenced this pull request Aug 19, 2025
@zanesq
Merge branch 'main' of github.com:block/goose into zane/extensions-tests
a1c63b5 
* 'main' of github.com:block/goose: (51 commits)
  Fix tests from upstream changes and add testing to lint staged and ci (#4127)
  Unlist figma tutorial (#4186)
  feat(ui): Implement in-place message editing with re-response (#3798)
  Retry all 500 codes (#4160)
  blog: Transforming AI Assistance with Goose Mentor Mode (#4151)
  upgraded all npm packages and fixed related issues (#4072)
  Docs: @-mentions in goosehints (#4171)
  fix: consistent font sizing in ToolCallWithResponse (#4167)
  Temporarily disable TODO Tool (#4158)
  docs: add integrated MCP server config to jetbrains tutorial  (#4120)
  docs: remove figma MCP from suggested servers (#4123)
  Blog: The AI Skeptic’s Guide to Context Windows (#4152)
  Docs: Auto-compact context (#4116)
  chore(deps): bump brace-expansion from 1.1.11 to 1.1.12 in /documentation (#4149)
  Recipe config to limit tool availability (#4020)
  docs: fix warning message (#4148)
  feat: adds cursor-agent as a cli provider (#4101)
  chore: remove vector search tool selection strategy (#3933)
  docs: add streamable_http install links (#4130)
  feat: iterating on summarize oneshot prompt (#4113)
  ...

# Conflicts:
#	ui/desktop/src/App.test.tsx
ayax79 pushed a commit to ayax79/goose that referenced this pull request Aug 21, 2025
@alexhancock
feat: iterating on summarize oneshot prompt (block#4113)
ed26876 
Signed-off-by: Jack Wright <jack.wright@nike.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DOsinga DOsinga DOsinga left review comments

@katzdave katzdave katzdave approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@alexhancock @DOsinga @katzdave