Skip to content

fix(quilt): serialize recipe load snapshots - #13

Open
Exohayvan wants to merge 2 commits into
minecraft/quilt-1.21.xfrom
minecraft-quilt-1.21.x-fix-recipe_snapshot_concurrency
Open

fix(quilt): serialize recipe load snapshots#13
Exohayvan wants to merge 2 commits into
minecraft/quilt-1.21.xfrom
minecraft-quilt-1.21.x-fix-recipe_snapshot_concurrency

Conversation

@Exohayvan

Copy link
Copy Markdown
Owner

Summary

  • Snapshots recipe input before the delayed parser runs.
  • Serializes shared collector access through publication.
  • Adds an exclusive-load regression test.

Fixes #4

Validation

  • ./gradlew test --tests net.exohayvan.dissolver_enhanced.data.RecipeLoadCoordinatorTest

@Exohayvan Exohayvan added the type:bug A defect or incorrect behavior label Jul 20, 2026
Repository owner deleted a comment from sonarqubecloud Bot Jul 20, 2026
@Exohayvan Exohayvan added P1 Critical: makes the mod unusable or fully breaks core functionality area:emc EMC values, calculation, storage, or overrides area:recipes Recipe loading, querying, snapshots, or derived EMC loader:quilt Quilt-specific status:in-review A linked pull request is awaiting review or checks labels Jul 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:emc EMC values, calculation, storage, or overrides area:recipes Recipe loading, querying, snapshots, or derived EMC loader:quilt Quilt-specific P1 Critical: makes the mod unusable or fully breaks core functionality status:in-review A linked pull request is awaiting review or checks type:bug A defect or incorrect behavior

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Fabric recipe snapshot can throw ConcurrentModificationException

1 participant