Skip to content

BC-11049 - Improve logging in tldraw#89

Merged
SevenWaysDP merged 18 commits intomainfrom
BC-11049
Dec 3, 2025
Merged

BC-11049 - Improve logging in tldraw#89
SevenWaysDP merged 18 commits intomainfrom
BC-11049

Conversation

@SevenWaysDP
Copy link
Contributor

@SevenWaysDP SevenWaysDP commented Dec 2, 2025 •
edited by Loki-Afro
Loading

Description

includes fix from yjs/yhub#36

Links to Tickets or other pull requests

Screenshots of UI changes

Approval for review

  • DEV: If the API or client code was changed, all necessary code generation or synchronization steps were completed and tested.
  • QA: In addition to review, the code has been manually tested (if manual testing is possible)
  • All points were discussed with the ticket creator, support-team or product owner. The code upholds all quality guidelines from the PR-template.

CeEv and others added 17 commits November 24, 2025 16:50
@CeEv
Extend log
0a035b3
@CeEv
increase logging
f139d77
@CeEv
Kommentar
c4c8e69
@CeEv
Push for testing
43d47d5
@CeEv
try again
7b9cb2e
@CeEv
remove duplicate
6a4050d
@CeEv
Add todo
4442a7d
@CeEv
Add counter
1dcd09a
@SevenWaysDP
Refactor YRedisClient: remove unused state vector encoding and improv…
3438895 
…e pending structs analysis
@SevenWaysDP
Enhance YRedisClient: log existing pending structs after transaction …
2abc62b 
…completion
@SevenWaysDP
Add CLI module and service for tldraw-server with clearPendingDocumen…
595a3b7 
…tStructs command
@SevenWaysDP
Change log level from debug to info for missing structs message in Cl…
de74a59 
…iService
@SevenWaysDP
Refactor CliService: streamline clearPendingDocumentStructs method an…
5adcca3 
…d improve logging
@SevenWaysDP
Add tests for logExistingPendingStructs and analyzePendingStructs met…
d25604f 
…hods in YRedisClient
@SevenWaysDP
Add test for handling null response in redis.readStreams in YRedisClient
7fb552c
@SevenWaysDP
Implement onModuleDestroy method in CliService to handle Redis client…
15c84d6 
… disconnection
@SevenWaysDP
Update README.md: Enhance documentation for clearPendingDocumentStruc…
c73da21 
…ts command with detailed usage, parameters, returns, and warnings
@SevenWaysDP SevenWaysDP mentioned this pull request Dec 2, 2025
Merged
3 tasks
@Loki-Afro Loki-Afro force-pushed the BC-11049 branch 2 times, most recently from b320b40 to c73da21 Compare December 2, 2025 18:20
@Loki-Afro
removed scripting attempt
5e945a9 
script destroyed ydocs, those were not openable afterwards, even in
simplified forms it made things ratehr worse
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 2, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Loki-Afro Loki-Afro changed the title BC-11049 - Improve logging in tldraw and clean pending structs comand BC-11049 - Improve logging in tldraw Dec 3, 2025
@SevenWaysDP SevenWaysDP merged commit 8a28dbf into main Dec 3, 2025
72 checks passed
@SevenWaysDP SevenWaysDP deleted the BC-11049 branch December 3, 2025 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Loki-Afro Loki-Afro Loki-Afro approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@SevenWaysDP @Loki-Afro @CeEv