Skip to content

Refactor ITEM_PROXY usage and update version to 3.2.2#315

Merged
twisti-dev merged 2 commits intoversion/26.1from
fix/set-max-stack-size
Apr 19, 2026
Merged

Refactor ITEM_PROXY usage and update version to 3.2.2#315
twisti-dev merged 2 commits intoversion/26.1from
fix/set-max-stack-size

Conversation

@twisti-dev
Copy link
Copy Markdown
Contributor

This pull request updates the version to 3.2.2 and refactors how item component updates are performed for Minecraft 1.20.6 (v26_1) support. The main change is the removal of the custom reflection proxy for setting item components, switching instead to direct usage of the built-in API. This simplifies the codebase and reduces reliance on reflection.

Key changes include:

Version Update:

  • Bumped the project version from 3.2.1 to 3.2.2 in gradle.properties.

NMS (Net Minecraft Server) Item Component Handling:

  • Replaced the use of the custom V26_1ItemProxy reflection proxy for setting item components with a direct call to nmsItem.builtInRegistryHolder().bindComponents(updatedComponents), using the official API and suppressing deprecation warnings.
  • Removed the import and usage of V26_1Reflection.ITEM_PROXY in V26_1SurfPaperNmsItemBridgeImpl.kt, reflecting the move away from the reflection proxy.

Reflection Proxy Cleanup:

  • Deleted the V26_1ItemProxy interface, removing unnecessary reflection-based code for item component updates.
  • Removed the ITEM_PROXY property and its initialization from V26_1Reflection, cleaning up unused reflection proxies. [1] [2]

…tem component binding

- eliminate unused ITEM_PROXY property and initialization in V26_1Reflection
- replace ITEM_PROXY usage with direct call to bindComponents in V26_1SurfPaperNmsItemBridgeImpl
@twisti-dev twisti-dev self-assigned this Apr 19, 2026
@twisti-dev twisti-dev merged commit a9d733c into version/26.1 Apr 19, 2026
3 of 4 checks passed
@twisti-dev twisti-dev deleted the fix/set-max-stack-size branch April 19, 2026 20:01
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.

1 participant