Skip to content

fix(mind): clippy on macOS battery probe split - #1908

Merged
ucguy4u merged 2 commits into
mainfrom
cursor/mind-live-probes-clippy-9ceb
Aug 24, 2026
Merged

fix(mind): clippy on macOS battery probe split#1908
ucguy4u merged 2 commits into
mainfrom
cursor/mind-live-probes-clippy-9ceb

Conversation

@ucguy4u

@ucguy4u ucguy4u commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Summary

#1903 landed on main but Rust Core CI failed on clippy -D warnings:

clippy::manual_pattern_char_comparison
airo_mind_audio/src/probes.rs:165

parse_pmset_battery_percent now splits on [' ', '\t', ';', '\\n', '\\r'] instead of a manual matches! predicate. Behavior is unchanged.

Fixes the red Rust test + clippy (host) on main after #1903 (d6e32e22 / ac819ab1).

Test Plan

  • cargo clippy -p airo_mind_audio --no-deps -- -D warnings
  • cargo test -p airo_mind_audio probes --lib — 9 passed
  • Rust Core CI on this PR

Verification environment: Host-only

Agent Policy

User-Facing Documentation

  • No wiki update — clippy-only.

Release Notes

  • Not user-facing.
Open in Web Open in Cursor 

Rust Core CI denies warnings. Use a char array instead of a manual
pattern so airo_mind_audio compiles under -D warnings.

Co-authored-by: Uday Chauhan <chauhan.s.uday26@gmail.com>
@github-actions

Copy link
Copy Markdown
Contributor

🚀 PR Quick Check Summary

Check Status Description
PR Validation ✅ success Title format, docs, bundled model guardrail
Code Quality ✅ success Analyze, formatting
Core Tests ✅ success Core package unit tests
Airo Mind Shell ✅ success Mind flavour analyze, tests, entrypoint compile

💡 Note: Full app tests, coverage reports, and security scans run on merge to main.

View Details

@ucguy4u
ucguy4u marked this pull request as ready for review August 24, 2026 04:41
@ucguy4u
ucguy4u merged commit 348e288 into main Aug 24, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🚀 PR Quick Check Summary

Check Status Description
PR Validation ✅ success Title format, docs, bundled model guardrail
Code Quality ✅ success Analyze, formatting
Core Tests ✅ success Core package unit tests
Airo Mind Shell ✅ success Mind flavour analyze, tests, entrypoint compile

💡 Note: Full app tests, coverage reports, and security scans run on merge to main.

View Details

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