Skip to content

fix(cli): recover compaction from payload limits#10012

Merged
marius-kilocode merged 1 commit into
mainfrom
mint-stealer
May 7, 2026
Merged

fix(cli): recover compaction from payload limits#10012
marius-kilocode merged 1 commit into
mainfrom
mint-stealer

Conversation

@marius-kilocode

Copy link
Copy Markdown
Collaborator

Summary

  • Recover failed compactions when provider payload-size limits reject large request bodies.
  • Move the destructive stripping/retry behavior into Kilo-specific compaction recovery code while keeping shared opencode hooks minimal.
  • Cover payload-limit recovery for media attachments and old tool outputs.

Test plan

  • bun test test/session/compaction.test.ts
  • bun test test/kilocode/compaction-payload-recovery.test.ts
  • bun test test/session/processor-effect.test.ts
  • bun test test/session/message-v2.test.ts
  • bun run typecheck
  • bun run script/check-opencode-annotations.ts
  • git diff --check

Comment thread packages/opencode/src/session/compaction.ts Outdated
@kilo-code-bot

kilo-code-bot Bot commented May 7, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (3 files)
  • packages/opencode/src/kilocode/session/compaction-payload-recovery.ts
  • packages/opencode/src/session/compaction.ts
  • packages/opencode/src/session/processor.ts

Reviewed by gpt-5.5-2026-04-23 · 741,121 tokens

@marius-kilocode marius-kilocode force-pushed the mint-stealer branch 2 times, most recently from 6815b36 to c20868a Compare May 7, 2026 13:04
@marius-kilocode

Copy link
Copy Markdown
Collaborator Author

Moved more logic to kilo files to reduce conflicts

@marius-kilocode marius-kilocode merged commit 0363006 into main May 7, 2026
13 checks passed
@marius-kilocode marius-kilocode deleted the mint-stealer branch May 7, 2026 14:01
jliounis pushed a commit to jliounis/kilocode that referenced this pull request May 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants