Skip to content

Improve analyze-logs signal extraction#32

Merged
stephen-tatari merged 3 commits into
mainfrom
worktree-pr-status-log-analysis
Mar 4, 2026
Merged

Improve analyze-logs signal extraction#32
stephen-tatari merged 3 commits into
mainfrom
worktree-pr-status-log-analysis

Conversation

@stephen-tatari

Copy link
Copy Markdown
Owner

Summary

  • Add noise filtering to remove GitHub Actions boilerplate (group markers, debug lines, action downloads, cache/progress lines)
  • Extract ##[error] annotations with surrounding context as a focused summary
  • Extract test failure summaries (pytest FAILURES sections, FAILED/ERROR lines)
  • Reduce truncation window from 500 to 250 lines (signal-dense after filtering)
  • Fix script invocation: add explicit instructions to SKILL.md to invoke directly, not via python

Test plan

  • All 98 tests pass (uv run --script test_pr_status.py)
  • Script runs directly without python prefix (scripts/pr_status.py check-cli)
  • Test against real PR with large CI logs

Add noise filtering, error annotation extraction,
and test summary extraction to reduce log output
from ~500 lines of noise to focused signal. Fix
script invocation instructions in SKILL.md.
Extract only the "short test summary info" section
instead of the full FAILURES section with tracebacks.
Reduces real-world output from 45K to 343 lines.
@stephen-tatari stephen-tatari marked this pull request as ready for review March 4, 2026 19:22
@stephen-tatari stephen-tatari merged commit 016b217 into main Mar 4, 2026
1 check passed
@stephen-tatari stephen-tatari deleted the worktree-pr-status-log-analysis branch March 4, 2026 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant