Skip to content

Docs updates and minor Rust formatting fixes#39

Merged
BunsDev merged 2 commits into
mainfrom
val/perf-command-integration-audit
Jun 6, 2026
Merged

Docs updates and minor Rust formatting fixes#39
BunsDev merged 2 commits into
mainfrom
val/perf-command-integration-audit

Conversation

@BunsDev
Copy link
Copy Markdown
Member

@BunsDev BunsDev commented Jun 3, 2026

Clarify visible slash commands, plugin and MCP behavior in docs: updates to docs/commands.md, docs/mcp.md, docs/plugins.md, and docs/tools.md to list current command usages, MCP subcommands, plugin reload semantics, and tool trait details. Apply numerous minor Rust edits across crates (acp, api, core, plugins, tools, tui, etc.) mostly reflowing long lines, small formatting/unwrap/serde error formatting changes, import reorders, and small cleanups in JSON parsing and logging initialization. These changes improve documentation accuracy for plugin-provided MCP servers (availability at startup vs after reload) and tidy code for readability without functional API changes.

Clarify visible slash commands, plugin and MCP behavior in docs: updates to docs/commands.md, docs/mcp.md, docs/plugins.md, and docs/tools.md to list current command usages, MCP subcommands, plugin reload semantics, and tool trait details. Apply numerous minor Rust edits across crates (acp, api, core, plugins, tools, tui, etc.) mostly reflowing long lines, small formatting/unwrap/serde error formatting changes, import reorders, and small cleanups in JSON parsing and logging initialization. These changes improve documentation accuracy for plugin-provided MCP servers (availability at startup vs after reload) and tidy code for readability without functional API changes.
Copilot AI review requested due to automatic review settings June 3, 2026 22:23
Copy link
Copy Markdown

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.

Copilot wasn't able to review this pull request because it exceeds the maximum number of lines (20,000). Try reducing the number of changed lines and requesting a review from Copilot again.

…nflicts

- TUI files: took main's trusted_command_path helpers over HEAD's hardcoded cmds
- app.rs: kept HistorySearch::default impl, took main's clipboard delegation
- cli/main.rs: took main (already has #27 + #38 fixes); fixed EffortLevel::from_str → ::parse
- All other Rust files: took main (formatting/reformatting only)

cargo check clean.
@BunsDev BunsDev merged commit 399f8e9 into main Jun 6, 2026
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