-
Notifications
You must be signed in to change notification settings - Fork 166
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(torii-grpc): world class hash #2307
Conversation
WalkthroughOhayo, sensei! The recent changes streamline the data structures related to Changes
Sequence Diagram(s)sequenceDiagram
participant Client
participant Server
participant Database
Client->>Server: Request World Metadata
Server->>Database: Query for World Address
Database-->>Server: Return World Address
Server-->>Client: Respond with World Metadata (No world_class_hash)
This diagram illustrates the primary interaction for retrieving TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Outside diff range, codebase verification and nitpick comments (1)
crates/torii/grpc/src/types/mod.rs (1)
Ohayo, sensei! It looks like the
world_class_hash
field is still being used incrates/dojo-world/src/manifest/mod.rs
. This suggests that its removal from theTryFrom
implementation might affect functionality elsewhere. Please review the dependencies and ensure that this change does not introduce any issues.
- File:
crates/dojo-world/src/manifest/mod.rs
- Occurrences:
let world_class_hash =
class_hash: world_class_hash,
Analysis chain
Line range hint
115-122
:
Verify the impact of removingworld_class_hash
.The removal of
world_class_hash
from theTryFrom
implementation simplifies the data structure. Ensure that no other parts of the code rely on this field.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify that `world_class_hash` is not used in other parts of the codebase. # Test: Search for occurrences of `world_class_hash`. Expect: No results. rg --type rust 'world_class_hash'Length of output: 190
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #2307 +/- ##
=======================================
Coverage 69.55% 69.56%
=======================================
Files 352 352
Lines 46004 46003 -1
=======================================
Hits 32000 32000
+ Misses 14004 14003 -1 ☔ View full report in Codecov by Sentry. |
remove world class hash from grpc proto
Summary by CodeRabbit
New Features
WorldMetadata
andWorld
, improving readability and maintenance.Bug Fixes
Documentation