ai-summary: treat killed processes and pytest collection errors as crashes

[ppetrovicTT]

Both were silently reported as SUCCESS. Detect SIGKILL/SIGTERM/" Killed" and "ERROR collecting" so the LLM is invoked, and surface the kill line in the error sections it reads.

[Copilot]

Pull request overview

Treats SIGKILL/SIGTERM-killed processes and pytest collection errors as crashes so the AI summary path runs (instead of silently reporting SUCCESS), and ensures the killer line is surfaced into the LLM's error sections.

Changes:

• Add SIGKILL, SIGTERM, and \b\d+\s+Killed\b (bash reaper format) to HIGH_PRIORITY_PATTERNS so the kill line is extracted into error sections.
• Extend has_crash detection in extract_log to fire on ERROR collecting (pytest collection failures) and on the same kill signals, anchored to avoid matching the English word "killed".
• Add tests covering the new positive matches and anti–false-positive cases, plus a test that the kill line reaches error_sections.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
.github/actions/ai_summary/tool/ai_job_summary/extract.py	Adds kill-signal patterns to HIGH_PRIORITY_PATTERNS and extends has_crash regex for pytest collection errors and signal-killed processes.
.github/actions/ai_summary/tool/ai_job_summary/tests/test_extraction.py	Renames/updates the has_crash test class and adds positive/negative tests for collection errors and kill detection, plus a section-extraction test.

---

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