Refactor/project cleanup - #19
Merged
Merged
Conversation
Single source of truth for the localhost socket address and the typed reload command, plus a shared non-panicking logger. Common dependencies are centralized in the workspace manifest.
Injection sits behind a per-OS trait with a typed error type, and a UI-agnostic core runs the injection on a worker thread so the front-ends never block.
A new egui layout with process cards and clear status states; the terminal UI is rebuilt on the shared core.
Logging, JVM monitoring, the command server, command dispatch and the client-library lifecycle each move into their own module.
Drop the per-type instance() singletons and their dead Arcs; the mapping is reached through free accessors instead of being threaded through every constructor.
Player, world and game mode are fetched lazily, so initialization no longer fails when no world is loaded.
Sharded caches remove lock contention on the render path, and a real panic in method lookup is fixed.
A dedicated registry type replaces the ad-hoc locked map shared across the UI and the tick loop.
OpenGL resolution and the frame hook move behind a per-OS module with Linux, Windows and a macOS stub.
Cover the IPC protocol, process discovery, the mapping helpers and the ESP projection math.
Boots a real JVM with a small Java fixture to exercise the reflected mapping path and the menu/in-world transitions.
An xtask command builds the project and injects into a running game; the injector gains headless list and inject modes.
Resolve the outstanding lint warnings and update the README and project docs to the current architecture.
The build workflow caches the Cargo registry and the target directory so unchanged dependencies are not rebuilt.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.