Skip to content

Conversation

@edmocosta
Copy link
Owner

  • Reduced the maximum number of Logstash threads states in memory from 500 to 300 to avoid high memory usage when running for long periods.
  • Improved TUI backend error handling.
  • Bumped dependencies versions.

@edmocosta edmocosta changed the title Improve error handling and bump versions Improve error handling and bump dependencies versions Jan 13, 2026
@edmocosta edmocosta requested a review from Copilot January 13, 2026 17:53
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR enhances error handling in the TUI backend and updates project dependencies to their latest versions. The changes focus on improving terminal cleanup reliability and reducing memory consumption during long-running operations.

Changes:

  • Reduced maximum thread states from 500 to 300 to prevent excessive memory usage
  • Refactored TUI backend initialization and cleanup with improved error handling
  • Updated dependency versions (clap, serde, ratatui, crossterm)

Reviewed changes

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

Show a summary per file
File Description
src/cli/output.rs Added explicit lifetime annotation to Output::new return type
src/cli/commands/tui/threads/state.rs Reduced MAX_THREAD_STATES constant from 500 to 300
src/cli/commands/tui/pipelines/ui.rs Added explicit lifetime annotations to Row return types
src/cli/commands/tui/data_decorator.rs Added early return when pipeline stats are empty
src/cli/commands/tui/backend.rs Refactored terminal initialization/cleanup with better error handling and extracted helper functions
Cargo.toml Bumped versions for clap, serde, serde_json, ratatui, and crossterm dependencies
CHANGELOG.md Added version 0.7.2 release notes

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

@edmocosta edmocosta merged commit 76d997e into main Jan 13, 2026
5 checks passed
@edmocosta edmocosta deleted the improve-error-handling-bump-versions branch January 13, 2026 17:55
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.

2 participants