feat(viz): expose lifetime stats on LosQueryManager

[WomB0ComB0]

Summary

• The audit routine flags "queries dropping under load?" as a known sensor-stack risk. The manager itself never drops — it queues per-slot via .then() chains — so the real signal is slot depth.
• Add a public stats getter returning { totalQueries, totalRays, peakSlotDepth }. peakSlotDepth > 1 means callers are pushing harder than GPU + readback can settle on a single slot; an audit script or overlay can read this directly.
• Pure additive observability. No behaviour change to mesh-link / LiDAR consumers.

Test plan

• npm run build passes; main bundle unchanged at 808.9 KB
• tsc --noEmit clean
• LosQueryStats exported alongside LosRay for typed downstream consumers
• CI: client-budget, .NET CI, security gates

🤖 Generated with Claude Code

Summary by CodeRabbit

• New Features

  • Added performance metrics tracking for query operations, including total queries processed, rays computed, and peak resource utilization.

• Bug Fixes

  • Improved query failure handling to ensure consistent cleanup regardless of success or failure outcomes.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: cdadf3c6-e29f-4f3c-9056-ec372455d118

📥 Commits

Reviewing files that changed from the base of the PR and between 4f117fe and 9125b6f.

📒 Files selected for processing (1)

• src/ResQ.Viz.Web/client/webgpu/los.ts

---

📝 Walkthrough

Walkthrough

The LosQueryManager class now tracks lifetime query metrics through a new exported LosQueryStats type and a stats getter. Slots are enhanced with depth counters that increment on query assignment and decrement when promises settle. Promise chain handling is refactored to ensure proper cleanup on both success and failure outcomes.

Changes

Cohort / File(s)	Summary
Query Metrics Tracking src/ResQ.Viz.Web/client/webgpu/los.ts	Added LosQueryStats export type with totalQueries, totalRays, and peakSlotDepth counters. Introduced per-slot depth tracking that increments on assignment and decrements on promise settlement. Refactored promise tail from .catch() to .then(decrement, decrement) to ensure cleanup on both success and failure paths. Manager counters updated on each accepted query.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• refactor(viz): ring-buffered LosQueryManager (foundation for LiDAR) #71: Directly modifies the same LosQueryManager implementation in los.ts, specifically the per-slot promise chaining and slot depth management logic.

Poem

🐰 A rabbit hops through metrics bright,
Counting queries, depths in flight,
When promises settle, left or right,
Counters dance, the ledgers tight! ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'feat(viz): expose lifetime stats on LosQueryManager' accurately and concisely summarizes the main change: exposing a new stats getter on LosQueryManager for lifetime query metrics observability.
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/webgpu-los-stats

---

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

_{Review rate limit: 0/1 reviews remaining, refill in 60 minutes.}

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

[gemini-code-assist]

Code Review

This pull request adds performance and usage tracking to the LosQueryManager by introducing a LosQueryStats type and a depth counter for each Slot. The query method was updated to maintain these statistics, including total queries, total rays, and peak slot depth, while ensuring the depth counter is decremented via a refined promise chain. I have no feedback to provide.