Skip to content

refactor(session): effectify SessionSummary service#20142

Merged
kitlangton merged 1 commit intodevfrom
worktree-effectify-summary
Mar 31, 2026
Merged

refactor(session): effectify SessionSummary service#20142
kitlangton merged 1 commit intodevfrom
worktree-effectify-summary

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

Summary

Convert SessionSummary to Effect service pattern.

  • Interface / Service / Layer / makeRuntime with async facades
  • Use Session.Service and Bus.Service directly instead of facades
  • Export DiffInput schema for server route validators (replaces fn().schema)
  • Simplify processor call — summarize is fire-and-forget, no Effect.promise wrapper needed

Test plan

  • 195 session/server tests pass, 0 fail

@kitlangton kitlangton force-pushed the worktree-effectify-summary branch 4 times, most recently from 625fda4 to 53b8ae7 Compare March 31, 2026 01:33
Convert SessionSummary to Effect service pattern with Interface,
Service, Layer, and async facades. Use Session.Service and Bus.Service
directly instead of facades.
@kitlangton kitlangton force-pushed the worktree-effectify-summary branch 2 times, most recently from e532dbd to 9b8dbed Compare March 31, 2026 01:36
@kitlangton kitlangton marked this pull request as ready for review March 31, 2026 01:40
@kitlangton kitlangton merged commit 954a6ca into dev Mar 31, 2026
8 checks passed
@kitlangton kitlangton deleted the worktree-effectify-summary branch March 31, 2026 23:14
jeromelau pushed a commit to jeromelau/opencode that referenced this pull request Apr 2, 2026
vicary pushed a commit to vicary/opencode that referenced this pull request Apr 6, 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