Skip to content

feat(session): add summary-diff opt-out config - #26036

Closed
ualtinok wants to merge 2 commits into
anomalyco:devfrom
ualtinok:fix/session-summarize-config
Closed

feat(session): add summary-diff opt-out config#26036
ualtinok wants to merge 2 commits into
anomalyco:devfrom
ualtinok:fix/session-summarize-config

Conversation

@ualtinok

Copy link
Copy Markdown
Contributor

Issue for this PR

Closes#26035

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

Adds session.summarize config so users can opt out of automatic session diff summarization during prompt processing.

By default nothing changes. When set to false, opencode skips the two automatic SessionSummary.summarize(...) callers used during prompt runs. This avoids broad session-history hydration for diff metadata when responsiveness is more important than automatic per-turn/session diff summaries.

It does not disable prompt execution, title generation, compaction, tool calls, or revert/manual diff behavior.

Example:

{
"session": {
"summarize": false
}
}

How did you verify your code works?

From packages/opencode:

  • bun typecheck
  • bun test test/config/config.test.ts
  • bun test test/session/prompt.test.ts

The push hook also ran root bun turbo typecheck successfully.

Screenshots / recordings

Not a UI change.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@ualtinokualtinok changed the title feat(session): add summary opt-out configfeat(session): add summary-diff opt-out configMay 6, 2026
@ualtinok

Copy link
Copy Markdown
ContributorAuthor

@Hona Could you recheck now? Thanks

@elmagow

Copy link
Copy Markdown

Can we have this reviewed? thanks

@ualtinok
ualtinokforce-pushed the fix/session-summarize-config branch from 13f1423 to 7399a44CompareJune 1, 2026 05:14
@ualtinok

Copy link
Copy Markdown
ContributorAuthor

#30127 landed, making this PR obsolete. Thanks @thdxr

@ualtinokualtinok closed this Jun 1, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE]: Allow disabling automatic session diff summarization

2 participants

@ualtinok@elmagow