Skip to content

Migrate compaction LLM test#26733

Merged
kitlangton merged 1 commit into
devfrom
cleanup/compaction-llm-effect-tests
May 10, 2026
Merged

Migrate compaction LLM test#26733
kitlangton merged 1 commit into
devfrom
cleanup/compaction-llm-effect-tests

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

@kitlangton kitlangton commented May 10, 2026

Summary

  • Extend the compaction process test layer helper to accept an LLM layer.
  • Convert the first LLM-backed compaction summary test to it.instance and yielded services.
  • Keep per-test variation local with Effect.provide(compactionProcessLayer(...)).

Testing

  • bun test --timeout 5000 test/session/compaction.test.ts -t "marks summary message as errored|persists tail_start_id|shrinks retained tail|falls back to full summary when even one recent turn|replays the prior user turn|falls back to overflow guidance|publishes compacted event|adds synthetic continue"
  • bun typecheck from packages/opencode
  • bunx prettier --check packages/opencode/test/session/compaction.test.ts

Stack

@kitlangton kitlangton force-pushed the cleanup/compaction-llm-effect-tests branch from ce1d8da to d35c264 Compare May 10, 2026 23:18
@kitlangton kitlangton merged commit 8f5f75d into dev May 10, 2026
10 checks passed
@kitlangton kitlangton deleted the cleanup/compaction-llm-effect-tests branch May 10, 2026 23:20
joshue031 pushed a commit to joshue031/opencode that referenced this pull request May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant