Skip to content

Migrate configurable compaction tests#26732

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

Migrate configurable compaction tests#26732
kitlangton merged 1 commit into
devfrom
cleanup/compaction-layer-effect-tests

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

@kitlangton kitlangton commented May 10, 2026

Summary

  • Add a per-test compaction process layer helper for variant processor/config cases.
  • Convert three config/processor-heavy compaction process tests to it.instance and yielded services.
  • Keep per-test variation local with .pipe(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|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
  • bunx oxlint packages/opencode/test/session/compaction.test.ts (existing warnings only, no errors)

Stack

@kitlangton kitlangton force-pushed the cleanup/compaction-layer-effect-tests branch from dd7874b to 17d7090 Compare May 10, 2026 23:12
@kitlangton kitlangton merged commit 4ff0d07 into dev May 10, 2026
11 checks passed
@kitlangton kitlangton deleted the cleanup/compaction-layer-effect-tests branch May 10, 2026 23:18
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