[Bugfix] Use the correct taskData when retrieving descriptors

[h2zero]

No description provided.

[coderabbitai]

Warning

Rate limit exceeded

@h2zero has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 9 minutes and 52 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 9 minutes and 52 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 2104af13-8506-484a-a8d5-82e87abd253e

📥 Commits

Reviewing files that changed from the base of the PR and between 6b69cb1 and abfe6c8.

📒 Files selected for processing (1)

• src/NimBLERemoteCharacteristic.cpp

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch bugfix/retrieve-descriptor-taskdata

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

Fixes descriptor discovery to consistently use the correct TaskData instance when waiting for and reading discovery results, preventing mismatched state when a custom filter is supplied.

Changes:

• Ensures retrieveDescriptors() always wires the active stack taskData into the filter used by the discovery callback.
• Reads completion status from the local taskData rather than indirectly through pFilter->taskData.
• Simplifies getDescriptor() filter construction while relying on retrieveDescriptors() to bind taskData.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.