feat: support live-reload tui when file is changed

[fioncat]

No description provided.

[Copilot]

Pull Request Overview

This PR refactors how the application manages configuration by replacing lifetime parameters with shared ownership via Rc to support live-reload functionality. Key changes include:

• Removing explicit lifetime annotations in UI and tree-related modules in favor of Rc-based configuration.
• Updating constructors and method calls across multiple files to clone Rc as needed.
• Adjusting configuration access in data rendering and parser invocations to accommodate the new Rc usage.

Reviewed Changes

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

Show a summary per file

File	Description
src/ui/tree_overview.rs	Removed lifetime parameters and adjusted constructors to accept Rc.
src/ui/popup.rs	Updated Popup to use Rc instead of borrowed references.
src/ui/header.rs	Changed Header to operate with Rc; consider revisiting unnecessary cloning.
src/ui/footer.rs	Refactored Footer to accept Rc.
src/ui/data_block.rs	Updated DataBlock to work with Rc and modified data rendering accordingly.
src/ui/app.rs	Updated App to utilize Rc across components and adjusted Edit initialization.
src/tree.rs	Modified tree parsing and conversion to support Rc usage.
src/main.rs	Initialized configuration as Rc and propagated it to the application components.

Files not reviewed (1)

• src/live_reload.rs: Language not supported

Comments suppressed due to low confidence (1)

src/ui/header.rs:55

• [nitpick] The explicit cfg.clone() may be redundant since cfg is already an Rc; consider using cfg.header.format directly to simplify the code.

let header_format = &cfg.clone().header.format;

[Copilot]

Pull Request Overview

This PR adds support for live-reloading the TUI when the underlying file is changed. It refactors the UI modules to use shared configuration via Rc, integrates a new live_reload module with a FileWatcher, and updates the application logic to reload the tree upon file updates.

Reviewed Changes

Copilot reviewed 13 out of 13 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/ui/tree_overview.rs	Replaces lifetime parameters with Rc and updates tree reload logic.
src/ui/popup.rs	Converts configuration reference to Rc.
src/ui/header.rs	Updates header construction to use a cloned Rc.
src/ui/footer.rs	Refactors to use Rc instead of lifetimes.
src/ui/data_block.rs	Updates configuration handling and rendering to adopt Rc.
src/ui/app.rs	Integrates FileWatcher into the app, adjusts refresh logic, and renames copy message to foot_message.
src/tree.rs	Removes lifetime annotations and propagates Rc usage in tree parsing.
src/main.rs	Adjusts config usage with Rc, integrates live_reload, and passes FileWatcher to App.
src/live_reload.rs	Introduces file watching functionality using the notify crate and updates live-reload behavior.
src/debug.rs	Minor update to ensure logs include a newline.
src/cmd.rs	Adds command-line support for live reloading.
Cargo.toml	Adds the notify crate as a dependency.

[Copilot]

Handling file removal by bailing out of the watcher thread may interrupt live-reload functionality unexpectedly. Consider handling file removal gracefully (for example, by notifying the user and optionally attempting to re-establish the watcher) rather than exiting the watcher thread.

Suggested change

	EventKind::Remove(_) => bail!("the file {} was removed by user", path.display()),
	EventKind::Remove(_) => {
	debug!("the file {} was removed by user, attempting to re-watch", path.display());
	watcher.watch(&path, RecursiveMode::NonRecursive)?;
	},