chore(root): Add LGTM observability suite#169
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
@cursoragent review |
API Preview 🐙Status: ❌ Deployment Failed |
UI Preview 🎈Status: ❌ Deployment Failed |
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
|
View your CI Pipeline Execution ↗ for commit b5b25ae
☁️ Nx Cloud last updated this comment at |
… export with metalness and roughness
…for Node.js ESM compatibility
Note
Medium Risk
Touches core chat/WebSocket execution paths (auth middleware ordering, room join semantics, RPC request/response handling) and introduces new external compaction calls + telemetry metrics, which could affect reliability/latency if misconfigured. Changes are mostly additive with expanded test coverage, lowering regression risk but still warranting careful review in production.
Overview
Observability: Adds a new
TelemetryIngestModulewithPOST /v1/telemetry/ingestto accept batched client metrics, wiresHealthModuleinto the API, and updates the API Docker image to auto-bootstrap OTEL and expose the Prometheus port (9464). CI is extended withknipdead-code analysis and a PR-only bundle size report, with.size-limit.jsonignored.Chat execution hardening + metrics: Refactors chat RPC to use Socket.IO
emitWithAck(removing the server-sidependingRequests/rpc_responseevent flow), enforces per-chatIdownership byuserId, and adds connection/RPC message-size/duration metrics + SSE event/connection counting. Chat streaming adds TTFT callbacks, newline/LaTeX normalization transforms, higherrecursionLimit, and supports acontextPayloadpassed into the agent via a newcreateClientContextMiddleware(skills/memory injected without extra RPC).Geometry testing/analysis: Moves GLB analysis and requirement evaluation logic out of the Nest service into
@taucad/testing/geometry, and expands measurement tests with a newconnectedComponentscheck coverage.Reviewed by Cursor Bugbot for commit 796e401. Bugbot is set up for automated code reviews on this repo. Configure here.