Skip to content

Consolidate TreeCRDT thoughtspace ownership - #106

Merged
Antonov548 merged 11 commits into
mainfrom
refactor/pr4325-review-cleanup
Aug 5, 2026
Merged

Consolidate TreeCRDT thoughtspace ownership#106
Antonov548 merged 11 commits into
mainfrom
refactor/pr4325-review-cleanup

Conversation

@marcus-pousette

@marcus-pousette marcus-pousette commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator
  • replace the ambient TreeCRDT singleton with one createTreecrdtThoughtspace owner for the client, provider, lifecycle, and WebSocket sync
  • keep the provider readiness gate factory-local so pre-init writes still work, while failed or dropped startup writes reject cleanly
  • bind materialization reads and writes permanently to the exact client session that emitted them

@marcus-pousette
marcus-pousette marked this pull request as draft July 23, 2026 20:49
@marcus-pousette
marcus-pousette force-pushed the refactor/pr4325-review-cleanup branch from befb01d to d928934 Compare July 23, 2026 21:20
@marcus-pousette marcus-pousette changed the title Refactor TreeCRDT runtime lifecycle Address TreeCRDT review feedback Jul 23, 2026
@marcus-pousette
marcus-pousette marked this pull request as ready for review July 23, 2026 21:32
@marcus-pousette
marcus-pousette force-pushed the refactor/pr4325-review-cleanup branch from d928934 to 90dbbe4 Compare July 24, 2026 09:11
@marcus-pousette marcus-pousette changed the title Address TreeCRDT review feedback Consolidate TreeCRDT thoughtspace ownership Jul 24, 2026
@marcus-pousette
marcus-pousette marked this pull request as draft July 24, 2026 09:40
@marcus-pousette
marcus-pousette force-pushed the refactor/pr4325-review-cleanup branch from 90dbbe4 to f4721e0 Compare July 24, 2026 12:04
@marcus-pousette
marcus-pousette changed the base branch from main to refactor/window-em-test-api July 24, 2026 12:05
@marcus-pousette
marcus-pousette force-pushed the refactor/pr4325-review-cleanup branch from f4721e0 to a88ff14 Compare July 24, 2026 12:11
Base automatically changed from refactor/window-em-test-api to main August 4, 2026 13:37
Comment thread src/data-providers/treecrdt/runtime.ts Outdated
@marcus-pousette
marcus-pousette marked this pull request as ready for review August 4, 2026 17:18
@Antonov548
Antonov548 merged commit f32545a into main Aug 5, 2026
9 of 11 checks passed
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.

2 participants