style: fix two clippy warnings

[hongqitai]

Summary

Run "cargo clippy" find two warnings, use "cargo clippy --fix" fix them.

Testing

• [ x ] cargo fmt --all -- --check
• [ x ] cargo clippy --workspace --all-targets --all-features
• [ x ] cargo test --workspace --all-features

Checklist

• [ x ] Updated docs or comments as needed
• [ x ] Added or updated tests where relevant
• [ x ] Verified TUI behavior manually if UI changes

Greptile Summary

This PR applies two mechanical clippy lint fixes generated by cargo clippy --fix, with no behavioral changes.

• config.rs: Removes a redundant format!() call wrapping a plain string literal, replacing it with .to_string() directly on the literal (clippy::useless_format).
• runtime_log.rs: Eta-reduces a closure |h| resolve(h) to pass the local closure resolve directly into and_then, eliminating a needless wrapper (clippy::redundant_closure).

Confidence Score: 5/5

Both changes are purely cosmetic clippy fixes with identical runtime behavior — safe to merge.

The two edits are exact semantic equivalents of the code they replace: removing a no-op format! wrapper and inlining a single-argument closure pass-through. Neither touches logic, data flow, or any externally visible behavior.

No files require special attention.

Important Files Changed

Filename	Overview
crates/tui/src/commands/config.rs	Replaces unnecessary format!("literal") with "literal".to_string() — a correct one-line clippy lint fix with no semantic change.
crates/tui/src/runtime_log.rs	Eta-reduces `

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[set_config_value: base_url branch] --> B{persist?}
    B -- yes --> C[persist_root_string_key]
    C -- Ok --> D[CommandResult::message]
    C -- Err --> E["CommandResult::error(format!(...))"]
    B -- no --> F["CommandResult::error('base_url must be saved...'.to_string())"]

    G[log_directory] --> H{HOME env set?}
    H -- yes --> I[resolve HOME]
    H -- no --> J{USERPROFILE env set?}
    J -- yes --> K[resolve USERPROFILE]
    J -- no --> L["dirs::home_dir().and_then(resolve)"]

Loading

_{Reviews (1): Last reviewed commit: "style: fix two clippy warnings" | Re-trigger Greptile}

[gemini-code-assist]

Code Review

This pull request simplifies code in the TUI crate by refactoring a closure to a direct function reference in runtime_log.rs and replacing a format! macro with a string literal in config.rs. The reviewer noted that the .to_string() call on the string literal in config.rs is redundant and can be omitted to avoid an unnecessary allocation.

[gemini-code-assist]

The .to_string() call is redundant here because CommandResult::error accepts a &str (or impl Into<String>) directly, as seen in other parts of this file (e.g., lines 34, 438, and 463). Passing the string literal directly is more idiomatic and avoids an unnecessary allocation.

            return CommandResult::error("base_url must be saved with --save; client base URL is loaded from config on startup. Restart and re-open your session after saving.");

[Hmbown]

Both clippy fixes are behavior-preserving: removing a useless on a string literal and eta-reducing a redundant closure. Clean.

APPROVE.

[Hmbown]

Both clippy fixes are behavior-preserving: removing a useless format!() on a string literal and eta-reducing a redundant closure. APPROVE.