Skip to content

feat: reduce cargo-insta-update-snapshots tool verbosity#110

Merged
Vaiz merged 5 commits intomasterfrom
roots2
Apr 5, 2026
Merged

feat: reduce cargo-insta-update-snapshots tool verbosity#110
Vaiz merged 5 commits intomasterfrom
roots2

Conversation

@Vaiz
Copy link
Copy Markdown
Owner

@Vaiz Vaiz commented Apr 5, 2026

No description provided.

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.

Pull request overview

This PR reduces verbosity for the cargo-insta-update-snapshots MCP tool by switching it to run cargo test with INSTA_UPDATE and adding explicit output verbosity controls, while also slightly tightening workspace auto-detection behavior/logging.

Changes:

  • Rework cargo-insta-update-snapshots to use cargo test + INSTA_UPDATE (adds force + output_verbosity, removes several prior cargo-insta-specific inputs).
  • Update tool documentation (tools.md) to reflect the new inputs/behavior.
  • Refresh insta snapshots for the updated command-line args.

Reviewed changes

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

Show a summary per file
File Description
tools.md Updates the documented schema/inputs for cargo-insta-update-snapshots (adds force, output_verbosity, removes prior fields).
src/workspace.rs Adds an early return + warning when the client lacks roots capability; minor cleanup in URI-to-path conversion.
src/tools/cargo_insta.rs Implements the new cargo test + INSTA_UPDATE command building and adds verbosity/force options; updates unit tests accordingly.
src/tools/snapshots/rust_mcp_server__tools__cargo_insta__tests__cargo_insta_update_snapshots_nextest_runner_args.snap Deletes snapshot for removed nextest/unreferenced args behavior.
src/tools/snapshots/rust_mcp_server__tools__cargo_insta__tests__cargo_insta_update_snapshots_manifest_path_args.snap Updates expected args for --manifest-path flow (now includes --quiet, removes old cargo-insta args).
src/tools/snapshots/rust_mcp_server__tools__cargo_insta__tests__cargo_insta_update_snapshots_features_and_targets_args.snap Updates expected args to match new default quiet behavior and removal of cargo-insta subcommand.
src/tools/snapshots/rust_mcp_server__tools__cargo_insta__tests__cargo_insta_update_snapshots_default_args.snap Updates expected default args (now cargo test --workspace --quiet).
src/tools/snapshots/rust_mcp_server__tools__cargo_insta__tests__cargo_insta_update_snapshots_all_features_args.snap Updates expected args (now includes --quiet, removes cargo-insta args).

Comment thread src/tools/cargo_insta.rs
Comment thread src/tools/cargo_insta.rs
Comment thread src/tools/cargo_insta.rs
Comment thread src/tools/cargo_insta.rs
@Vaiz Vaiz linked an issue Apr 5, 2026 that may be closed by this pull request
@Vaiz Vaiz merged commit e937ab9 into master Apr 5, 2026
12 checks passed
@Vaiz Vaiz deleted the roots2 branch April 5, 2026 20:13
@github-actions github-actions Bot mentioned this pull request Apr 5, 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.

updating insta snapshots is too verbose

2 participants