docs: add in-game test plan for v2.2.0 validation #5

Open

agent wants to merge 0 commits from agent/shadow-forge-testplan into multiplatform/1.16.5 AGit

agent commented

2026-06-14 00:20:52 +00:00

First-time contributor

No description provided.

agent added 7 commits

2026-06-14 00:20:52 +00:00

v2.2.0: Sound notification docs, full rule field reference, version bump 9c66007bfe

- Comprehensive Sound Notifications section with alias table, examples
- Documented all rule fields: notifyStyle, colorAware, matchJson, respond, server, valuestack
- Legacy playSound/soundName backward compat documented
- Version bump 2.1.2 → 2.2.0
- CHANGELOG updated

fix: resolve Shannon's critical sound bugs b98982f6b5

- wasModified() now includes playSound — sound-only rules no longer silently ignored
- ForgeTextHelper.playSound() resolves soundName param instead of hardcoding UI_BUTTON_CLICK
- FabricChatHelper.resolveSound() delegates to SoundResolver (adds note/bell/click aliases)
- Both client platforms support arbitrary MC sound IDs via dynamic SoundEvent

fix: Forge playSound() now delegates to SoundResolver for alias parity with Fabric cf88516313

Aliases like 'bell', 'click', 'note' now resolve correctly on Forge.
Also handles 'none'/'silent' to skip sound playback.

fix: shadowJar ASM failure — upgrade shadow 8.3.5→8.3.11, set Java 17 target on server modules 13bbe44638

Shadow 8.3.5's ASM couldn't handle Java 21 bytecode StringConcatFactory
invokedynamic handles during relocation. Two changes:
- Upgrade com.gradleup.shadow 8.3.5 → 8.3.11 (fixed ASM)
- Add explicit Java 17 source/target on bungee, spigot, velocity,
  packetevents-common (common stays Java 8 for Fabric 1.16.5 compat)

fix: Forge SoundEvents field names for official mappings, bump to 2.2.0 1bbe11f980

ForgeGradle 5.1 with official mappings uses shorter field names
(e.g. EXPERIENCE_ORB_PICKUP not ENTITY_EXPERIENCE_ORB_PICKUP).

chore: gitignore stale neoforge/build artifacts 883e96b972

docs: add in-game test plan for v2.2.0 validation 04c023e184

This branch is already included in the target branch. There is nothing to merge.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin +refs/pull/5/head:agent/shadow-forge-testplan

git switch agent/shadow-forge-testplan

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch multiplatform/1.16.5

git merge --no-ff agent/shadow-forge-testplan

git switch agent/shadow-forge-testplan

git rebase multiplatform/1.16.5

git switch multiplatform/1.16.5

git merge --ff-only agent/shadow-forge-testplan

git switch agent/shadow-forge-testplan

git rebase multiplatform/1.16.5

git switch multiplatform/1.16.5

git merge --no-ff agent/shadow-forge-testplan

git switch multiplatform/1.16.5

git merge --squash agent/shadow-forge-testplan