Temporary folder cleanup error handling

[tino097]

Description

This PR is adding error handling in the clean mindsdb tmp dir if errors are occurring.
Additionally i have added tests to cover the change

Type of change

(Please delete options that are not relevant)

• 🐛 Bug fix (non-breaking change which fixes an issue)

Verification Process

To ensure the changes are working as expected:

• Test Location: tests/unit/various/test_main.py
• Verification Steps: Outline the steps or queries needed to validate the change. Include any data, configurations, or actions required to reproduce or see the new functionality.

Additional Media:

• I have attached a brief loom video or screenshots showcasing the new functionality or change.

Checklist:

• My code follows the style guidelines(PEP 8) of MindsDB.
• I have appropriately commented on my code, especially in complex areas.
• Necessary documentation updates are either made or tracked in issues.
• Relevant unit and integration tests are updated or added.

[entelligence-ai-pr-reviews]

🔒 Entelligence AI Vulnerability Scanner

❌ Security analysis failed: Security analysis failed: context deadline exceeded: This error is likely due to exceeding 'timeoutMs' — the total time a long running request (like process or directory watch) can be active. It can be modified by passing 'timeoutMs' when making the request. Use '0' to disable the timeout.

[entelligence-ai-pr-reviews]

correctness: shutil.rmtree(file, ignore_errors=True) in clean_mindsdb_tmp_dir suppresses all errors, including critical ones like permission errors, which can leave the temp directory uncleared and mask real failures.

🤖 AI Agent Prompt for Cursor/Windsurf

📋 Copy this prompt to your AI coding assistant (Cursor, Windsurf, etc.) to get help fixing this issue

In mindsdb/__main__.py, lines 177-179, the use of `shutil.rmtree(file, ignore_errors=True)` in `clean_mindsdb_tmp_dir` suppresses all errors, including critical ones (e.g., permission errors), which can leave the temp directory uncleared and mask real failures. Replace this with a try/except block that calls `shutil.rmtree(file)` and logs any exceptions, so that real errors are not silently ignored.

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change

	if file.is_dir():
	# https://docs.python.org/3/library/shutil.html#shutil.rmtree
	shutil.rmtree(file, ignore_errors=True)
	if file.is_dir():
	# https://docs.python.org/3/library/shutil.html#shutil.rmtree
	try:
	shutil.rmtree(file)
	except Exception as e:
	logger.error(f"Failed to remove directory {file}: {e}")

[ZoranPandovski]

Thanks, I've added two minor comments

[ZoranPandovski]

Please check the ruff format warnings

[hamishfagg]

Same as other PRs, comment in check_requirements.py

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}