feat: add Flutter and Dart command filters with compact output#2457
Open
thomasdhulst wants to merge 1 commit into
Open
feat: add Flutter and Dart command filters with compact output#2457thomasdhulst wants to merge 1 commit into
thomasdhulst wants to merge 1 commit into
Conversation
- Introduced `flutter_cmd.rs` for handling Flutter commands: analyze, pub get, and test. - Added `dart/mod.rs` for Dart command filtering. - Implemented output filtering for Flutter analyze, pub get, and test commands. - Created new fixtures for testing Flutter command outputs. - Updated command routing in `main.rs` to include Flutter and Dart commands. - Enhanced TOML filter to recognize Flutter and Dart commands. - Added classification rules for Flutter and Dart commands in the registry. - Implemented tests to ensure proper functionality of new command filters.
|
|
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
2 participants
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.
flutter_cmd.rsfor handling Flutter commands: analyze, pub get, and test.dart/mod.rsfor Dart command filtering.main.rsto include Flutter and Dart commands.Summary
Adds first-class RTK support for Flutter and Dart commands, focused on shrinking noisy CLI output for flutter analyze, flutter pub get, flutter test, and dart analyze. It introduces new command routing, output filters, updated discovery rules, and fixtures/tests that validate grouped diagnostics, trimmed package-install logs, and compact test failure summaries.
Test plan
Verified with the Rust test suite and formatting/lint checks where applicable, then manually exercised the new rtk flutter and rtk dart paths to inspect the rewritten CLI output. I also checked the sample Flutter fixtures to confirm the filters keep failures, summarize diagnostics by file, and remove the noisy pass-through logs.
cargo fmt --all && cargo clippy --all-targets && cargo testrtk <command>output inspected