Skip to content

refactor(compaction): migrate replay validation to Effect - #170

Merged
robinbraemer merged 1 commit into
mainfrom
feat/effect-compaction-remote
Aug 1, 2026
Merged

refactor(compaction): migrate replay validation to Effect#170
robinbraemer merged 1 commit into
mainfrom
feat/effect-compaction-remote

Conversation

@robinbraemer

Copy link
Copy Markdown
Member

Summary

  • decode persisted JSON signatures through Effect Schema instead of raw JSON.parse/try-catch
  • remove assertion-based response-item mutation by validating reconstructed provider items
  • move message and session replay coverage onto @effect/vitest
  • remove the ambient environment/dynamic-require/throw Zod loader; normal package resolution remains until the follow-up Effect Schema definition conversion
  • delete four satisfied Effect-migration baseline entries

Verification

  • bun run check
    • 502 legacy tests passed, 4 skipped
    • 284 Node Effect tests passed
    • 6 Bun-hosted Effect tests passed
    • 114 docs tests passed
    • build, typecheck, docs lint/typecheck passed
  • bun tooling/effect-migration/check.ts (410 paths, zero violations)
  • git diff --check

Progresses #98 and #102. Does not close either issue: remote transport, extension tests, and the full Zod-to-Effect-Schema definition migration remain.

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 1, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
agentos-site 29503d8 Aug 01 2026, 10:21 PM

@robinbraemer
robinbraemer merged commit e87bfa7 into main Aug 1, 2026
1 of 3 checks passed
@robinbraemer
robinbraemer deleted the feat/effect-compaction-remote branch August 1, 2026 22:27
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.

1 participant