Skip to content

feat: pickpocket alerts rummage instead of flashing a title - #37

Merged
ryanbarlow97 merged 1 commit into
mainfrom
feat/pickpocket-rummage-alert
Oct 10, 2026
Merged

ryanbarlow97 merged 1 commit into
mainfrom
feat/pickpocket-rummage-alert

Conversation

@ryanbarlow97

@ryanbarlow97 ryanbarlow97 commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • A noticed or failed pickpocket plays the mark a bag rummaging sound (item.bundle.remove_one, pitch 0.8) instead of the red "Someone is pickpocketing you!" title and the iron trapdoor clank.
  • A critical clue still names the thief's character, now on the mark's action bar: pickpocket.alert-critical (default "You catch {character_name} going through your pockets").
  • pickpocket.alert-subtitle and alert-subtitle-critical are removed; live configs that still have them are ignored.

Docs: TF-Minecraft/Docs#121

Testing

  • mvn clean verify: 821 tests pass.

🤖 Generated with Claude Code

A noticed or failed pickpocket now plays the mark a bag rummaging sound
(item.bundle.remove_one) with no title. A critical clue still names the
thief's character, quietly on the action bar via pickpocket.alert-critical.
The alert-subtitle keys are gone.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 10, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 23 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b97c1811-de09-4345-acef-3a16331d900f

📥 Commits

Reviewing files that changed from the base of the PR and between a7c3b74 and 8c52d67.


📒 Files selected for processing (6)
  • src/main/java/net/tfminecraft/thievery/loader/PickpocketLoader.java
  • src/main/java/net/tfminecraft/thievery/player/PickpocketVictimAlerter.java
  • src/main/resources/config.yml
  • src/test/java/net/tfminecraft/thievery/door/PickpocketGameTest.java
  • src/test/java/net/tfminecraft/thievery/loader/ActivityLoadersTest.java
  • src/test/java/net/tfminecraft/thievery/player/PickpocketVictimAlerterTest.java

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ryanbarlow97
ryanbarlow97 merged commit 5e3a18d into main Oct 10, 2026
2 checks passed
@ryanbarlow97
ryanbarlow97 deleted the feat/pickpocket-rummage-alert branch October 10, 2026 18:44
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