Repository navigation
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
 |
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
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.
Adds the focused VeinMining slice from #76, stacked on #86. Opt-in connected-ore traversal stays bounded and never loads chunks. Additional native player breaks preserve protection events, tool wear and normal drops; deferred work rechecks the original event, exact player session, held slot and tool metadata. A shared in-flight guard suppresses recursive automation through listener replacement and reload.
Optional personal charges reserve the additional-block budget before gameplay. Proved-unused units are refunded; an uncertain native call remains spent and stops the batch. Existing perks do not gain the effect automatically. This PR leaves harvesting, PrecisionMode, queues, banking and statistics in their separate slices.
Validation: 56 focused cases and 421 clean-package tests passed, zero failures/errors/skips, with source stability confirmed. Existing P3 test assertions are retained. The earlier full P4 review is combined with a fresh independent focused correction/composition review returning No findings; this is not a fresh whole-PR verdict. The final regression surfaces callback failures outside the production callback, with a retained parent-source failure. ZIP integrity, all 40 unchanged resources, filtered descriptor, new Java 21 classes, test/provided-library exclusions and 75 actual dependency JARs were checked. JAR SHA-256: d3ecc81b1143c13ac4d1bb182fa2c59010b5d8782184f1d0f740458e79ce20de.
Remains a draft for normal dependency delivery, current-head hosted feedback and separately authorized native/older-server acceptance. Validation used isolated candidate dependencies, not released artifacts. MCPerks does not declare Folia support. Merge order is #78, #80, #86, then this slice. No merge, release or deployment is included.
Created with AI assistance.