[电大老baby (HhH666-ux)]

Description

DenylistConverter.convert_async short-circuited with a case-sensitive substring check, so a prompt containing a case-mismatched denylisted term (e.g. Molotov cocktail against molotov cocktail) was returned unchanged without ever calling the converter target. The system prompt template documents case-insensitive matching.

• Compare denylist terms and the prompt case-insensitively in the pre-flight check.
• Add a regression test covering a case-mismatched phrase.

Fixes #4061

[Copilot]

🟡 Changes recommended

The lower()-based pre-flight check misses valid Unicode denylist matches, such as Greek sigma, returning the prompt unchanged without calling the target.

1 open issue

• Use casefold to preserve Unicode denylist matches · New

What changed in this PR

This PR addresses #4061 by making DenylistConverter’s pre-flight check case-insensitive, aligning it with the converter’s system prompt instructions.

Changes:

• Lowercase denylist terms and prompts before checking for matches.
• Add a regression test confirming that a case-mismatched phrase reaches the converter target.

File	Description
tests/​unit/​converter/​test_denylist_converter.py	Adds coverage for a case-mismatched denylisted phrase.
pyrit/​converter/​denylist_converter.py	Updates the pre-flight substring check to ignore casing.

_{Review effort: Balanced}

[Roman Lutz (romanlutz)]

Thanks for the contribution. I'm closing this because #4062 is the earlier active implementation for #4061 and covers the same fix.

Please claim an issue and wait for ownership confirmation before starting work so maintainers can coordinate ownership and avoid duplicated effort.