refactor: disable isEmptyContent method for debug in AI insights

[Artuomka]

No description provided.

[Copilot]

Pull Request Overview

This PR disables the isEmptyContent method for debugging purposes in AI insights by making it always return false and commenting out the original logic. The method is used to filter empty content chunks in streaming AI responses.

• Changed isEmptyContent method to always return false instead of checking if content is empty
• Commented out the original implementation that checked for empty or whitespace-only strings
• Renamed the parameter from content to _content to indicate it's unused

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The commented code after the return statement is unreachable and serves no purpose. If this is intended as temporary debug code, consider using a feature flag or configuration parameter instead of commenting out the implementation. For production code, remove the commented lines entirely.

Suggested change

	// if (content === ' ') {
	// return false;
	// }
	// return !content || content.trim() === '';

[Copilot]

This method now always returns false, which means the check at line 262 (if (this.isEmptyContent(chunk.delta))) will never filter out any content chunks. This causes all chunks, including potentially empty ones, to be written to the response stream. If this is temporary debug code, ensure it's not merged to production or use a configuration-based approach to toggle this behavior.

Suggested change

	private isEmptyContent(_content: string): boolean {
	return false;
	// if (content === ' ') {
	// return false;
	// }
	// return !content || content.trim() === '';
	private isEmptyContent(content: string): boolean {
	if (content === ' ') {
	return false;
	}
	return !content || content.trim() === '';