Skip to content

refactor: destroy SessionStatus facade#21968

Merged
kitlangton merged 1 commit intodevfrom
kit/destroy-more-facades-2
Apr 11, 2026
Merged

refactor: destroy SessionStatus facade#21968
kitlangton merged 1 commit intodevfrom
kit/destroy-more-facades-2

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

@kitlangton kitlangton commented Apr 11, 2026

Summary

  • remove the SessionStatus async facade and call the service directly through AppRuntime
  • update the last session route caller and the retry policy test to use SessionStatus.Service
  • record the migration in the Effect facade-removal spec

Verification

  • bun typecheck
  • bun run test test/session/retry.test.ts test/server/session-actions.test.ts test/server/session-list.test.ts

@kitlangton kitlangton merged commit f998124 into dev Apr 11, 2026
11 of 13 checks passed
@kitlangton kitlangton deleted the kit/destroy-more-facades-2 branch April 11, 2026 02:16
mrsimpson pushed a commit to mrsimpson/opencode that referenced this pull request Apr 14, 2026
@kitlangton @mrsimpson
refactor: destroy SessionStatus facade (anomalyco#21968)
08bf519
xywsxp pushed a commit to xywsxp/opencode that referenced this pull request Apr 24, 2026
@kitlangton
refactor: destroy SessionStatus facade (anomalyco#21968)
060e070
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

contributor Vouched

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@kitlangton