Skip to content

Remove fuzzy string searching for scrolls of genocide#1480

Open
NateChoe1 wants to merge 2 commits intoNetHack:NetHack-3.7from
NateChoe1:geno-nofuzz
Open

Remove fuzzy string searching for scrolls of genocide#1480
NateChoe1 wants to merge 2 commits intoNetHack:NetHack-3.7from
NateChoe1:geno-nofuzz

Conversation

@NateChoe1
Copy link

@NateChoe1 NateChoe1 commented Jan 31, 2026

Addresses issue #1474. I also created PR #1475 last month which handled this by adding a new paranoid_confirm option, although copperwater pointed out some concerns which have been implemented here.

This PR disables fuzzy string searching for scrolls of genocide. For uncursed/cursed scrolls, strings like master mnd flayer (which would eliminate monks) are no longer accepted because the monster name isn't the entire string. For blessed scrolls, strings like master mind flayer are no longer accepted because the monster name isn't a single letter or full class description.

I changed the message "This [symbol/response] does not represent any monster" to "This [symbol/response] does not represent any monster class" because after this change it felt strange to have a response like master mind flayer, which directly represents a monster, not represent a monster.

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