Skip to content

refactor: Update message handling to use BytableMessage interface and…#2794

Open
drazisil wants to merge 10 commits into
devfrom
chore/staged-changes
Open

refactor: Update message handling to use BytableMessage interface and…#2794
drazisil wants to merge 10 commits into
devfrom
chore/staged-changes

Conversation

@drazisil

Copy link
Copy Markdown
Collaborator

… adjust logging verbosity.

Comment thread packages/transactions/src/_createStandardRace.ts Fixed
Comment thread packages/transactions/src/_createStandardRace.ts Fixed
Comment thread packages/transactions/src/_joinRace.ts Fixed
Comment thread packages/transactions/src/_joinRace.ts Fixed
@codecov

codecov Bot commented Nov 22, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 51.04167% with 423 lines in your changes missing coverage. Please review.
✅ Project coverage is 34.88%. Comparing base (153a56c) to head (46535cc).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
packages/shared/src/MessageNode.ts 48.93% 24 Missing ⚠️
packages/gateway/src/npsPortRouter.ts 0.00% 22 Missing ⚠️
packages/shared/src/LegacyMessage.ts 53.19% 22 Missing ⚠️
packages/transactions/src/_getFullCarInfo.ts 0.00% 22 Missing ⚠️
packages/shared/src/legacyHeader.ts 58.69% 19 Missing ⚠️
packages/transactions/src/internal.ts 36.66% 19 Missing ⚠️
packages/shared/src/NPSHeader.ts 69.49% 18 Missing ⚠️
packages/shared/src/SerializedBufferOld.ts 46.42% 15 Missing ⚠️
packages/nps/messageStructs/GameMessage.ts 27.77% 13 Missing ⚠️
packages/database/src/cache.ts 0.00% 12 Missing ⚠️
... and 74 more
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #2794      +/-   ##
==========================================
+ Coverage   34.01%   34.88%   +0.87%     
==========================================
  Files         292      294       +2     
  Lines       18144    18287     +143     
  Branches     1052     1087      +35     
==========================================
+ Hits         6171     6380     +209     
+ Misses      11971    11905      -66     
  Partials        2        2              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@deepsource-io

deepsource-io Bot commented Nov 22, 2025

Copy link
Copy Markdown

Here's the code health analysis summary for commits 153a56c..46535cc. View details on DeepSource ↗.

Analysis Summary

AnalyzerStatusSummaryLink
DeepSource JavaScript LogoJavaScript❌ Failure
❗ 123 occurences introduced
🎯 91 occurences resolved
View Check ↗

💡 If you’re a repository administrator, you can configure the quality gates from the settings.

…sensitive information

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@rustymotors rustymotors deleted a comment from coderabbitai Bot Nov 22, 2025
drazisil and others added 2 commits November 21, 2025 22:00
…sensitive information

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…sensitive information

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
@drazisil
drazisil enabled auto-merge November 22, 2025 03:03
…sensitive information

Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
Comment thread packages/transactions/src/_createStandardRace.ts Dismissed
@sonarqubecloud

Copy link
Copy Markdown

@drazisil
drazisil added this pull request to the merge queue Nov 22, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Nov 22, 2025
@drazisil
drazisil added this pull request to the merge queue Nov 22, 2025
@drazisil
drazisil removed this pull request from the merge queue due to the queue being cleared Nov 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants