Skip to content

fix: resolve crashes, color calculations, recursion, and rendering errors - #139

Closed
Spagles wants to merge 10 commits into
WhatYouThing:mainfrom
Spagles:main
Closed

fix: resolve crashes, color calculations, recursion, and rendering errors#139
Spagles wants to merge 10 commits into
WhatYouThing:mainfrom
Spagles:main

Conversation

@Spagles

@Spagles Spagles commented Jul 28, 2026

Copy link
Copy Markdown
  • Dynamic Map Patch Dimensions: Render map patches in DungeonMap using startX, startY, width, and height from ClientboundMapItemDataPacket to prevent ArrayIndexOutOfBoundsException on partial map updates.
  • Stack Mutation & Count Guards: Prevent IllegalStateException on empty stack modifications in SlotOptions and enforce minimum stack counts (Math.max(1, ...)) in BeaconTuningSolver.
  • Float Cast Precedence in Color Calculation: Fix operator precedence in RenderColor where float-to-int casting truncated RGB channels, resulting in black hex colors.
  • Matrix Solver Error Handling: Handle RuntimeException (singular matrix) in CurveSolver polynomial fitting and add null checks for trajectory calculation.
  • Recursion Guard in Streamer Mode: Add a ThreadLocal flag in StringDecomposerMixin to prevent StackOverflowError during string decomposition.
  • Empty Lore Checks: Prevent NoSuchElementException when checking chest items with empty lore in CroesusSolver and VesuviusSolver.
  • Regex & Name Matching Refinements: Update NoDamageSplash regex to require one or more digits ([0-9]+) and refine slot name prefix matching in WardrobeKeybinds.

@WhatYouThing

Copy link
Copy Markdown
Owner

AI generated patches are not welcome. A large portion of these fixes are aimed towards issues that have never occurred to begin with.

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