fix(data-consumption): prevent hook usage counter from going negative

[Arthurk12]

What does this PR do?

The usage counter that tracks how many plugins are listening to a given subscription can reach negative values, which is invalid and likely indicates a bug in the unsubscribe logic.

Extract the hook usage update logic into a dedicated function that guards against negative values and logs a warning when such an update is attempted.

Closes Issue(s)

Closes N/A

More

Closely related to:

• useCustomSubscription stuck in loading: true state bigbluebutton-html-plugin-sdk#275
• fix(data-consumption): unsubscribe with current args on unmount & test: initial unit test setup bigbluebutton-html-plugin-sdk#278

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: a55422a1-6ac5-4e1b-b077-ddb6a2279d9a

📥 Commits

Reviewing files that changed from the base of the PR and between c1506bc and cf3f238.

⛔ Files ignored due to path filters (1)

• bigbluebutton-html5/package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (2)

• bigbluebutton-html5/package.json
• bigbluebutton-web/grails-app/conf/bigbluebutton.properties

---

Walkthrough

Hook usage count updates now use applyHookUsageCountDelta, clamping underflowing decrements to zero and logging a warning across both update paths. The HTML5 plugin SDK dependency and web configuration property are updated from version 0.1.20 to 0.1.24.

Estimated code review effort: 2 (Simple) | ~10 minutes

Suggested reviewers: guileme, antobinary, gustavotrott

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main change: preventing the hook usage counter from going negative.
Description check	✅ Passed	The description directly matches the changeset by explaining the negative-count guard and warning behavior.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 Checkov (3.3.8)

bigbluebutton-html5/package.json

Traceback (most recent call last):
File "/usr/local/bin/checkov", line 2, in
from checkov.main import Checkov
ModuleNotFoundError: No module named 'checkov'

---

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.}

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[GuiLeme]

Just this comment that would imply changes here: bigbluebutton/bigbluebutton-html-plugin-sdk#278 (review)

[github-actions]

✅ Automated tests have passed!

[GuiLeme]

LGTM

Tested locally and reviewed the code.