Skip to content

refactor(session): destroy SessionRunState facade#22064

Open
kitlangton wants to merge 10 commits intodevfrom
worktree-agent-a7cb5136
Open

refactor(session): destroy SessionRunState facade#22064
kitlangton wants to merge 10 commits intodevfrom
worktree-agent-a7cb5136

Conversation

@kitlangton
Copy link
Copy Markdown
Contributor

@kitlangton kitlangton commented Apr 11, 2026

Summary

  • Removed makeRuntime and assertNotBusy facade from SessionRunState
  • Converted the delete-message handler in server/routes/session.ts to wrap the entire body in Effect.gen, yielding both SessionRunState.Service and Session.Service from context
  • Updated session-actions.test.ts to make the session actually busy via ensureRunning instead of spying on the deleted facade

Verification

  • bun run typecheck — clean
  • bun run test test/server/session-actions.test.ts — 2/2

@kitlangton kitlangton force-pushed the worktree-agent-a7cb5136 branch from 5830aef to 5cda6cc Compare April 11, 2026 23:35
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