Skip to content

Conversation

@AC19970
Copy link
Contributor

@AC19970 AC19970 commented Dec 14, 2025

  • Use CROESUS_PATTERN instead of DROP_PATTERN cuz DROP_PATTERN requires player picking items and that's slow
  • Remove player name detection
  • Add m3-m5 star and shadow fury
  • Add recomb to see more effects cuz it is the most profitable/rare item in low floors and also nice in high floors
  • Separate scrolls cuz they may have different looks if player is using resource pack

 - Use CROESUS_PATTERN instead of DROP_PATTERN cuz DROP_PATTERN requires player picking items and that's slow
 - Remove player name detection
 - Add m3-m5 star and shadow fury
 - Add recomb <del>to see more effects</del> cuz it is the most profitable/rare item in low floors and also nice in high floors
 - Separate scrolls cuz they may have different looks if player is using resource pack
@LifeIsAParadox LifeIsAParadox added the reviews needed This PR needs reviews label Dec 14, 2025
@kevinthegreat1
Copy link
Collaborator

Has this been tested? (Have you dropped something while using these changes?)

@AC19970
Copy link
Contributor Author

AC19970 commented Dec 15, 2025

Has this been tested? (Have you dropped something while using these changes?)

ofc i did 40 runs

@kevinthegreat1
Copy link
Collaborator

This still works outside of croesus right? Like opening the chests directly at dungeon end.

@AC19970
Copy link
Contributor Author

AC19970 commented Dec 15, 2025

This still works outside of croesus right? Like opening the chests directly at dungeon end.

Yes. Although I call it CROESUS_PATTERN, the text for opening the chest is the same both inside and outside the dungeon.

 - Rename rareDungeonDropEffects to rareDropEffects
 - Use DUNGEON_CHEST_PATTERN and MAGIC_FIND_PATTERN instead of CROESUS_PATTERN. now we can add more rare items.
 - Fix regex
 - Add more items
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

reviews needed This PR needs reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants