fix: unify extract-log-errors output behavior for empty manifests

[github-actions]

Summary

This PR resolves #812 by unifying output handling in scripts/extract-log-errors.py so both empty and non-empty log sets flow through emit_output(...).

What changed

• Removed emit_empty_output(...).
• Refactored main() to:
  • preserve No log files found. on stderr when no logs are present
  • always build summary
  • always call emit_output(summary, args.output)
• Kept run metadata attachment for non-empty log scans unchanged.

Tests

• Added assertions in tests/test_extract_log_errors.py to verify --output prints the output path for both:
  • non-empty manifest logs
  • empty manifest logs
• /tmp/gh-aw/agent/venv/bin/python -m pytest -q tests/test_extract_log_errors.py (4 passed)

Issue linkage

Closes #812.

---

The body of this PR is automatically managed by the Trigger Update PR Body workflow.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: d48da08c-a3ff-4d84-85aa-6f574cb03d73

📥 Commits

Reviewing files that changed from the base of the PR and between 3d0fa0f and 0f32c71.

📒 Files selected for processing (2)

• scripts/extract-log-errors.py
• tests/test_extract_log_errors.py

---

📝 Walkthrough

Walkthrough

The changes consolidate output handling in scripts/extract-log-errors.py by removing the emit_empty_output() function and its invocation. The script now constructs a summary object in all cases (including when no log files are found) and routes both empty and non-empty results through emit_output(). Tests in test_extract_log_errors.py were updated to assert that the output path is printed to stdout on successful execution.

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	PR fully addresses issue #812 requirements: removed emit_empty_output, unified output handling through single emit_output path, added test assertions for --output behavior in both empty/non-empty scenarios.
Out of Scope Changes check	✅ Passed	All changes are directly scoped to issue #812 objectives: function removal, main() refactoring, and test updates align entirely with complexity-reduction goals.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/812-unify-extract-output-0d6a703c3dcd5880

• 🛠️ Update Documentation: Commit on current branch
• 🛠️ Update Documentation: Create PR

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}