Add line limit filtering to workflow log display

[t0mdavid-m]

Summary

Enhanced the workflow log display in the execution section to support filtering log output based on a configurable line limit, while maintaining the full content for status checks.

Key Changes

• Refactored log file reading from read() to readlines() for line-by-line processing
• Separated content parsing (for status checks) from display rendering
• Added conditional logic to apply line limit filtering:
  • When log_lines_count == "all": display entire log
  • Otherwise: display only the last N lines based on st.session_state.log_lines_count
• Preserved original status detection logic that checks full content for "WORKFLOW FINISHED"

Implementation Details

• The full log content is still read and checked for workflow completion status
• Only the display output is filtered, preventing loss of important status information
• Uses list slicing (lines[-count:]) for efficient tail-like behavior on large logs
• Maintains backward compatibility with existing "all" option for displaying complete logs

Summary by CodeRabbit

• New Features

  • Added line-limiting display option for static logs, allowing you to view either all lines or only the last N lines based on your configuration.

• Bug Fixes

  • Improved log file reading and display handling for better reliability.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The log display mechanism in StreamlitUI's execution section was refactored to read log files line-by-line using readlines() instead of read(), and now supports line-limited display showing either all lines or only the last N lines as configured in session state.

Changes

Cohort / File(s)	Summary
Streamlit UI Log Display src/workflow/StreamlitUI.py	Modified static log display in execution_section to use readlines() for reading logs and reconstruct content from individual lines. Added conditional logic to display all lines or limit output to the last N lines based on st.session_state.log_lines_count configuration. Real-time spinner path remains unchanged.

Poem

📜 Logs line by line, now neatly trimmed,
No longer reading all at once, so grim,
The last N lines shine bright and clear,
Or all at once, without a fear! 🐰✨

✨ Finishing touches

• 📝 Generate docstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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