fix: raise TypeError for non-string TextCleaner inputs

[MohammadHijjawi97]

Related Issues

No existing issue. This PR includes a regression test for the crash.

Proposed Changes:

Proposed Changes:

TextCleaner.run applied regex, lowercasing, and str.translate to every item. A None entry raised AttributeError. Coerce None and non-string items to empty strings so the output list keeps the same length.

How did you test it?

Added a unit test covering None, an integer, and mixed strings.

Notes for the reviewer

Checklist

• I have read the contributors guidelines and the code of conduct.
• I have updated the related issue with new insights and changes.
• I have added unit tests and updated the docstrings.
• I've used one of the conventional commit types for my PR title: fix:, feat:, build:, chore:, ci:, docs:, style:, refactor:, perf:, test: and added ! in case the PR includes breaking changes.
• I have documented my code.
• I have added a release note file, following the contributors guidelines.
• I have run pre-commit hooks and fixed any issue.

[vercel]

@MohammadHijjawi97 is attempting to deploy a commit to the deepset Team on Vercel.

A member of the Team first needs to authorize it.

[CLAassistant]

All committers have signed the CLA.

[github-actions]

Hi @MohammadHijjawi97, thanks for your interest in contributing to Haystack! 🙏

⚠️ You currently have 5 open pull requests in this repository (#12794, #12793, #12791, #12790 and this one). Our review capacity is limited, so please hold off opening more PRs until we've had a chance to review your first 2 open PRs. This helps us give each contribution the attention it deserves. Thank you!

This is an automated message to help us keep the review queue healthy.

[anakin87]

Hello!

This component accepts list[str], so I would not silently accept and replace a None value.

What we can do instead is check whether every element in texts is a str and raise a clear TypeError if not.

[MohammadHijjawi97]

Thanks. Updated this to keep the list[str] contract: every element must be a str, otherwise it raises a clear TypeError. Tests updated accordingly.

[anakin87]

I pushed some little refinements.
Ready to be merged. Thanks!

[github-actions]

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
haystack/components/preprocessors
text_cleaner.py
haystack/core/pipeline
pipeline.py
Project Total

This report was generated by python-coverage-comment-action