Add 13 new plan analysis rules to PlanAnalyzer

[erikdarlingdata]

Summary

• Expands PlanAnalyzer from 11 to 24 rules with domain-specific detection for common SQL Server performance anti-patterns
• New rules: non-SARGable predicates (5 sub-patterns), data type mismatch, lazy spool ratio, join OR clause, NL high executions, many-to-many merge, large memory grant with sort/hash guidance, compile memory exceeded, high compile CPU, local variables, CTE multi-reference, table variables, TVFs, top above scan
• Fixes existing scan rules to exclude columnstore indexes (designed to be scanned)
• Extracts IsRowstoreScan() helper for consistent rowstore scan detection

Test plan

• Open a plan with implicit conversions — verify "Non-SARGable Predicate" warning fires
• Open a plan with YEAR() or DATEPART() in WHERE — verify function call detection
• Open a plan with table variables — verify "Table Variable" warning
• Open a plan with a TVF — verify "Table-Valued Function" warning
• Open a plan with a many-to-many merge join — verify warning fires
• Open a columnstore scan plan — verify NO scan-with-predicate warning fires
• Verify existing warnings still fire (filter, eager spool, UDF, spills, etc.)

🤖 Generated with Claude Code