Skip to content

Fix Darwin writeWithoutResponse queue completion - #272

Open
fotiDim wants to merge 2 commits into
mainfrom
fix/darwin-write-without-response-271
Open

Fix Darwin writeWithoutResponse queue completion#272
fotiDim wants to merge 2 commits into
mainfrom
fix/darwin-write-without-response-271

Conversation

@fotiDim

@fotiDim fotiDim commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • fix iOS/macOS withoutResponse write flow to complete when CoreBluetooth accepts writes (canSendWriteWithoutResponse) instead of waiting on peripheralIsReady for every write
  • queue and flush pending Darwin without-response writes only when the peripheral is ready, and fail pending writes on disconnect cleanup
  • bump release metadata to 2.1.2 in pubspec.yaml and CHANGELOG.md

Test plan

  • flutter test test/ble_command_queue_test.dart test/queue_test.dart
  • Validate on macOS/iOS with QueueType.global and QueueType.perDevice using withoutResponse: true writes
  • Verify no progressive write latency growth and no queue timeouts under sustained writes

Made with Cursor

fotiDim and others added 2 commits July 28, 2026 09:22
Align iOS/macOS write-without-response completion with other platforms by completing when CoreBluetooth accepts writes and only deferring when the buffer is full. This resolves queue stalls and throughput degradation seen with QueueType.global/perDevice on Apple (#271).

Co-authored-by: Cursor <cursoragent@cursor.com>
Align package metadata and changelog header with the Darwin write-without-response queue fix release.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cursor

cursor Bot commented Jul 28, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@fotiDim fotiDim changed the title Fix Darwin writeWithoutResponse queue completion and bump 2.1.2 Fix Darwin writeWithoutResponse queue completion Jul 28, 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.

1 participant