[Feat] Make Sessions the primary workspace for Roomote work - #1708
[Feat] Make Sessions the primary workspace for Roomote work#1708roomote-roomote[bot] wants to merge 35 commits into
Conversation
|
No new code issues found. See task
Reviewed f08c61a |
|
Fixed the failed Test check in The branch is merged with current |
…ssions-2tuskyfsof5bf # Conflicts: # apps/docs/fast-sessions.mdx # apps/web/src/app/(authenticated)/home/Home.tsx # apps/web/src/app/(sandbox)/sessions/[sessionId]/SessionWorkspace.tsx # packages/communication/src/fast-session-footer.ts # packages/db/drizzle/meta/0061_snapshot.json # packages/db/drizzle/meta/_journal.json
|
Resolved the failed Roomote review and merge conflicts through Exact checks: Cloud Agents 109 files / 993 tests with Redis unreachable; DB Session helpers 22 tests; focused web server 8 tests; focused web client 57 tests; responsive Session workspace 10 tests; web, Cloud Agents, and DB fast type checks; pre-push oxlint, residual lint, monorepo fast typecheck, and knip. A fresh database applied migrations through |
This reverts commit 6035b16.
…ssions-2tuskyfsof5bf
…sof5bf' into feature/unified-sessions-2tuskyfsof5bf
|
Fixed the failing Test check through Exact checks: focused navigation tests 24/24; complete web client suite 247 files / 1,869 tests; complete DB suite 59 files / 638 tests; web and DB fast type checks; targeted oxlint; pre-push oxlint, residual lint, monorepo fast typecheck, and knip. The replacement GitHub Test job 98814409791 passed in 2m37s. Production behavior is unchanged. See task. |
What changed
/task/:idcontractssessions_data,sessions_ui, andsessions_commsflags, with Session-aware communication links/copy and additive coarse telemetrysessions_uiWhy this change was made
Users currently move between conversation and execution as separate top-level objects. This change keeps conversation, delegated work, review context, unread state, artifacts, pull requests, and cost under one durable Session without changing operational Task identity or runtime contracts.
The rollout gates and migration behavior were tightened during review so enabling Session persistence alone does not expose unavailable UI, deleted tasks cannot keep Sessions blocked, deployments without environments can still start conversational Sessions, and existing Fast-mode opt-outs remain intact.
Impact
The schema change is additive and N-1 safe. After
developmigration0063_organic_garia.sql, migration0064_deep_vengeance.sqlcreates the Session tables and indexes and adds a nullablesession_idusage dimension. Migration0065_enable_fast_mode_for_existing_users.sqlenables the new default only for users with no saved preference, preserving explicitfalseandtruevalues.Rollout order remains
sessions_datafirst, thensessions_ui, thensessions_comms. Each flag can be reversed independently. Settings changes explicitly invalidate both Redis and process-local deployment metadata caches; otherwise local metadata refreshes within 30 seconds.Current
developis merged into the branch. The final checks include 109 Cloud Agents files / 993 tests with Redis deliberately unreachable; 8 focused web server tests; 57 focused Session client tests; 10 responsive Session workspace tests; the full web client suite at 247 files / 1,869 tests; and the full DB suite at 59 files / 638 tests. The original navigation failures pass 24/24 focused tests, and the migration-journal consistency regression is covered by the complete DB run. Web, Cloud Agents, and DB fast type checks pass, as do the pre-push oxlint, residual lint, monorepo fast typecheck, and knip gates. A fresh database applied all migrations through0065, and migration fixtures producedfalse:false,missing:true, andtrue:true.One scope item remains deliberately deferred: durable multi-task Session-level notification coalescing from M12. Existing per-run Fast-parent delivery claims/dedup remain active; the new read/notified cursors and Session projection are in place for a follow-up implementation.
Screenshots