garrett/wafb
3
3
Fork 0
Code Issues 2 Pull requests Projects 1 Releases 4 Packages Wiki Activity Actions 2

feat: dumpsys_acquirer #40

Merged
n merged 3 commits from feat/dumpsys_acquirer into develop 2026-02-11 16:06:33 +00:00
Conversation 0 Commits 3 Files changed 20 +1737 -1432
garrett commented 2026-02-11 13:38:04 +00:00
Owner
Copy link

🛠 Type of Change

  • 🐛 Bug fix
  • ✨ New Feature (androidqf module)
  • ⚡️ Performance / Refactoring
  • 📝 Documentation

📖 Description

Implementa l'acquirer dumpsys

🧪 Testing

  • Manually tested with device: (Pixel 7)
  • Unit tests passed (pnpm test)
  • Lint verified (pnpm lint)
### 🛠 Type of Change - [ ] 🐛 Bug fix - [x] ✨ New Feature (androidqf module) - [ ] ⚡️ Performance / Refactoring - [ ] 📝 Documentation ### 📖 Description Implementa l'acquirer dumpsys ### 🧪 Testing - [x] Manually tested with device: (Pixel 7) - [x] Unit tests passed (`pnpm test`) - [x] Lint verified (`pnpm lint`)
Consolidate three separate dumpsys command classes into DumpsysCommand with:
- Optional service parameter and dynamic command building
- Support for packageName and args
- Sensor information collection

Remove redundant GetBatteryInfoCommand, GetMemoryInfoCommand, GetPackageInfoCommand.
Rewrite tests with proper Adb mocking and realistic dumpsys outputs.

BREAKING CHANGE: Use DumpsysCommand.exec({ service }) instead of removed command classes
feat: implement dumpsys acquirer
All checks were successful
WAFB CI + Deploy / quality (pull_request) Successful in 1m58s
Details
WAFB CI + Deploy / deploy-dev (pull_request) Has been skipped
Details
WAFB CI + Deploy / release (pull_request) Has been skipped
Details
811f9545d2
garrett force-pushed feat/dumpsys_acquirer from 811f9545d2
All checks were successful
WAFB CI + Deploy / quality (pull_request) Successful in 1m58s
Details
WAFB CI + Deploy / deploy-dev (pull_request) Has been skipped
Details
WAFB CI + Deploy / release (pull_request) Has been skipped
Details
to 0c4f2f679b
All checks were successful
WAFB CI + Deploy / quality (pull_request) Successful in 2m27s
Details
WAFB CI + Deploy / deploy-dev (pull_request) Has been skipped
Details
WAFB CI + Deploy / release (pull_request) Has been skipped
Details
2026-02-11 13:44:23 +00:00 Compare
garrett force-pushed feat/dumpsys_acquirer from 0c4f2f679b
All checks were successful
WAFB CI + Deploy / quality (pull_request) Successful in 2m27s
Details
WAFB CI + Deploy / deploy-dev (pull_request) Has been skipped
Details
WAFB CI + Deploy / release (pull_request) Has been skipped
Details
to a14cb6dfa0
All checks were successful
WAFB CI + Deploy / quality (pull_request) Successful in 2m26s
Details
WAFB CI + Deploy / deploy-dev (pull_request) Has been skipped
Details
WAFB CI + Deploy / release (pull_request) Has been skipped
Details
2026-02-11 14:25:08 +00:00 Compare
fix(core/adb): implement idle timeout for ShellCommand to prevent hanging
All checks were successful
WAFB CI + Deploy / quality (pull_request) Successful in 1m58s
Details
WAFB CI + Deploy / deploy-dev (pull_request) Has been skipped
Details
WAFB CI + Deploy / release (pull_request) Has been skipped
Details
7ac2d4890e 
ShellCommand previously relied on the stream's `done` signal to finish reading.
Some ADB daemons (especially with dumpsys) never close the stream, causing
commands to hang indefinitely until the global timeout. This replaces the
read loop with an idle‑aware mechanism: if no data is received for
`idleTimeout` ms (default 5s), reading stops and the accumulated output is
returned. This ensures commands complete promptly after the last byte of
output, while preserving backward compatibility.

- Add `idleTimeout` option to CommandOptions (default 5000ms)
- Replace `readStreamAsText` with manual chunk reading + idle timeout
- Keep global timeout as safety net for commands that produce no output
n merged commit 7e81512e9e into develop 2026-02-11 16:06:33 +00:00
n referenced this pull request from a commit 2026-02-11 16:06:35 +00:00
feat: dumpsys_acquirer (#40)
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
Compat/Breaking

Breaking change that won't be backward compatible

  
Kind/Bug

Something is not working

  
Kind/Documentation

Documentation changes

  
Kind/Enhancement

Improve existing functionality

  
Kind/Feature

New functionality

  
Kind/Security

This is security issue

  
Kind/Testing

Issue or pull request related to testing

  
Priority
Critical
The priority is critical

  
Priority
High
The priority is high

  
Priority
Low
The priority is low

  
Priority
Medium
The priority is medium

  
Reviewed
Confirmed
Issue has been confirmed

  
Reviewed
Duplicate
This issue or pull request already exists

  
Reviewed
Invalid
Invalid issue

  
Reviewed
Won't Fix
This issue won't be fixed

  
Status
Abandoned
Somebody has started to work on this but abandoned work

  
Status
Blocked
Something is blocking this issue or pull request

  
Status
Need More Info
Feedback is required to reproduce issue or to continue work

No labels
Compat/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Blocked
Status
Need More Info
Milestone
Clear milestone
No items
Projects
Clear projects
No items
No project
Parita' funzionale androidqf
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
garrett/wafb!40
No description provided.