Run cargo fmt across WTA - #658
Open
Kai Tao (vanzue) wants to merge 1 commit into
Open
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request applies the repository’s current rustfmt formatting across the WTA Rust workspace (tools/wta), aligning all WTA sources with cargo fmt --all output while keeping behavior unchanged.
Changes:
- Reformat WTA Rust sources with
cargo fmt --all(line wrapping, indentation, and argument/layout reflow). - Minor item/import ordering adjustments produced by formatting (no functional changes intended).
- Reformat associated WTA unit tests for consistency with workspace rustfmt settings.
Reviewed changes
Copilot reviewed 70 out of 75 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tools/wta/src/win32.rs | rustfmt-only reflow of clipboard path construction. |
| tools/wta/src/ui/setup.rs | rustfmt-only reflow of UI rendering logic and match formatting. |
| tools/wta/src/ui/recommendations.rs | rustfmt-only reflow of early-return and layout struct formatting. |
| tools/wta/src/ui/permission.rs | rustfmt-only reflow of line-length and local variable formatting. |
| tools/wta/src/ui/mod.rs | rustfmt-only reflow of module declarations ordering/formatting. |
| tools/wta/src/ui/layout.rs | rustfmt-only reflow of chained calls and function invocation formatting. |
| tools/wta/src/ui/input.rs | rustfmt-only reflow of caret span calculations and helper calls. |
| tools/wta/src/ui/command_popup.rs | rustfmt-only reflow of span building and function signatures. |
| tools/wta/src/ui/command_format.rs | rustfmt-only reflow of module docs, helpers, and tests. |
| tools/wta/src/ui/chat.rs | rustfmt-only reflow of helper formatting, conditional blocks, and tests. |
| tools/wta/src/ui/card.rs | rustfmt-only reflow of small helpers and function signature formatting. |
| tools/wta/src/ui/auth.rs | rustfmt-only reflow of layout constraints and localized string formatting. |
| tools/wta/src/ui/agents_view.rs | rustfmt-only reflow of rendering helpers and tests. |
| tools/wta/src/ui/agent_popup.rs | rustfmt-only reflow of popup rendering and state struct formatting. |
| tools/wta/src/ui/action_panel.rs | rustfmt-only reflow of layout computations and boolean expressions. |
| tools/wta/src/theme.rs | rustfmt-only reflow of Style constants formatting. |
| tools/wta/src/text_selection.rs | rustfmt-only reflow of test assertions formatting. |
| tools/wta/src/shell/shell_manager.rs | rustfmt-only reflow of args building, small helpers, and tests. |
| tools/wta/src/session_watcher/mod.rs | rustfmt-only reflow of record processing and test assertions. |
| tools/wta/src/session_watcher/discover.rs | rustfmt-only reflow of test string literal formatting. |
| tools/wta/src/session_watcher/classify_gemini.rs | rustfmt-only reflow of test setup formatting. |
| tools/wta/src/session_watcher/classify_copilot.rs | rustfmt-only reflow of test record formatting. |
| tools/wta/src/session_watcher/classify_claude.rs | rustfmt-only reflow of tool-name extraction formatting. |
| tools/wta/src/session_history.rs | rustfmt-only reflow of mapping helpers and tests. |
| tools/wta/src/rtl.rs | rustfmt-only reflow of RTL helpers and tests. |
| tools/wta/src/protocol/acp/spawn.rs | rustfmt-only reflow of map_err closure formatting. |
| tools/wta/src/protocol/acp/soft_stop.rs | rustfmt-only reflow of enum helpers and tests. |
| tools/wta/src/protocol/acp/session_list.rs | rustfmt-only reflow of client spawn call formatting. |
| tools/wta/src/protocol/acp/prompt.rs | rustfmt-only reflow of includes, atomic write helpers, and tests. |
| tools/wta/src/protocol/acp/prompt_builder.rs | rustfmt-only reflow of test assertions formatting. |
| tools/wta/src/protocol/acp/failure.rs | rustfmt-only reflow of enums, match arms, and tests. |
| tools/wta/src/pane_context.rs | rustfmt-only reflow of test assertions formatting. |
| tools/wta/src/osc52.rs | rustfmt-only reflow of constant declaration formatting. |
| tools/wta/src/main.rs | rustfmt-only reflow (including module list formatting). |
| tools/wta/src/logging.rs | rustfmt-only reflow of helper formatting and tests. |
| tools/wta/src/locale_parity_tests.rs | rustfmt-only reflow of locale parity tests formatting. |
| tools/wta/src/hook_contract_tests.rs | rustfmt-only reflow of panic formatting. |
| tools/wta/src/history_loader.rs | rustfmt-only reflow of parsing helpers, control flow, and tests. |
| tools/wta/src/helper/runtime.rs | rustfmt-only reflow of use grouping and imports. |
| tools/wta/src/cwd_util.rs | rustfmt-only reflow of helper functions and tests. |
| tools/wta/src/custom_model_provider.rs | rustfmt-only reflow of error message formatting and tests. |
| tools/wta/src/coordinator.rs | rustfmt-only reflow of match formatting and tests. |
| tools/wta/src/commands.rs | rustfmt-only reflow of closure indentation. |
| tools/wta/src/command_recall.rs | rustfmt-only reflow of helpers and tests. |
| tools/wta/src/cli/delegate.rs | rustfmt-only reflow of function signatures and tests. |
| tools/wta/src/cli_tests.rs | rustfmt-only reflow of test formatting and indentation. |
| tools/wta/src/autofix_tests.rs | rustfmt-only reflow of tests module formatting. |
| tools/wta/src/app/turn_state.rs | rustfmt-only reflow of match arms and tests. |
| tools/wta/src/app/autofix.rs | rustfmt-only reflow of enums and method signatures. |
| tools/wta/src/app/attachments.rs | rustfmt-only reflow of attachment state machine formatting. |
| tools/wta/src/agent_tools/action_proposal/pipe.rs | rustfmt-only reflow of tests module trailing whitespace. |
| tools/wta/src/agent_tools/action_proposal/invocation.rs | rustfmt-only reflow of test imports ordering/formatting. |
| tools/wta/src/agent_tools/action_proposal/channel.rs | rustfmt-only reflow of chained call formatting. |
| tools/wta/src/agent_registry.rs | rustfmt-only reflow of test assertions formatting. |
| tools/wta/src/agent_check.rs | rustfmt-only reflow of helpers, signatures, and tests. |
| tools/wta/src/action_links.rs | rustfmt-only reflow of nested iterator formatting. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
cargo fmt --allacross the WTA Rust workspacedoc/assetsworking-tree files untouchedValidation
cargo fmt --all --manifest-path tools/wta/Cargo.toml -- --checkcargo fmt --all --manifest-path installer/bootstrap/Cargo.toml -- --check