Skip to content

Migrate compaction overflow tests#26731

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

Migrate compaction overflow tests#26731
kitlangton merged 1 commit into
devfrom
cleanup/compaction-overflow-effect-tests

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

@kitlangton kitlangton commented May 10, 2026

Summary

  • Convert two overflow compaction process tests to it.instance and yielded services.
  • Use SessionCompaction.use.process directly instead of per-test ManagedRuntime wrappers.
  • Remove an unused import from the compaction test file.

Testing

  • bun test --timeout 5000 test/session/compaction.test.ts -t "replays the prior user turn|falls back to overflow guidance"
  • bun typecheck from packages/opencode
  • bunx prettier --check packages/opencode/test/session/compaction.test.ts
  • bunx oxlint packages/opencode/test/session/compaction.test.ts (existing warnings only, no errors)

Stack

@kitlangton kitlangton force-pushed the cleanup/compaction-overflow-effect-tests branch from 6bf64ad to b9dfda8 Compare May 10, 2026 23:08
@kitlangton kitlangton enabled auto-merge (squash) May 10, 2026 23:09
@kitlangton kitlangton merged commit 2a571b3 into dev May 10, 2026
11 checks passed
@kitlangton kitlangton deleted the cleanup/compaction-overflow-effect-tests branch May 10, 2026 23:12
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