Skip to content

WIP: soundManager lazy import, requestId concurrency tests, pauseGuard tests, webhooks.md cross-check - #1576

Closed
Themancalledpg wants to merge 1 commit into
LabsCrypt:mainfrom
Themancalledpg:wip/soundmanager-requestid-pauseguard-webhooks-1520-1521-1522-1523
Closed

WIP: soundManager lazy import, requestId concurrency tests, pauseGuard tests, webhooks.md cross-check#1576
Themancalledpg wants to merge 1 commit into
LabsCrypt:mainfrom
Themancalledpg:wip/soundmanager-requestid-pauseguard-webhooks-1520-1521-1522-1523

Conversation

@Themancalledpg

Copy link
Copy Markdown
Contributor

Draft placeholder — claiming this batch of 4 assigned issues, real implementation to follow. No code changes yet.

closes #1520
closes #1521
closes #1522
closes #1523

Draft placeholder — implementation to follow.
Tracks:
- LabsCrypt#1520 webhooks.md event type list not cross-checked against webhookService.ts
- LabsCrypt#1521 pauseGuard.ts has no dedicated unit test
- LabsCrypt#1522 requestId.ts uniqueness under concurrency is untested
- LabsCrypt#1523 soundManager.ts is imported eagerly even when sound is disabled
@Themancalledpg

Copy link
Copy Markdown
Contributor Author

Closing as a duplicate — #1569 already covers this exact issue set (#1520, #1521, #1522, #1523) with real implementation work. This draft placeholder is redundant.

@Themancalledpg
Themancalledpg deleted the wip/soundmanager-requestid-pauseguard-webhooks-1520-1521-1522-1523 branch July 31, 2026 07:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment