Skip to content

perf(diagnostics): bound RPC latency aggregation - #567

Open
beruro wants to merge 1 commit into
developfrom
junyu/rpc-diagnostics-lifecycle
Open

perf(diagnostics): bound RPC latency aggregation#567
beruro wants to merge 1 commit into
developfrom
junyu/rpc-diagnostics-lifecycle

Conversation

@beruro

@beruro beruro commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • bound RPC duration aggregation cardinality and preserve stable buckets
  • sanitize runtime counters before persistence/submission
  • retain only the diagnostics detail needed for performance investigation
  • cover bucket edges and queue/sanitization behavior

Independence

This 8-file PR is independent of both the #512 session stack and the #562 async stack.

Validation

  • diagnostics Vitest: 4 passed
  • pnpm typecheck and changed-file ESLint
  • cargo test -p org2 usage_diagnostics: 7 passed
  • cargo check -p org2
  • rustfmt check on changed Rust files
  • git diff --check origin/develop...HEAD

Performance guard

Aggregation has explicit cardinality bounds; raw/path-like values remain sanitized; queue rows transition from unsent to sent without retained duplicate payloads.

Pre-commit hook ran. Total eslint: 0, total circular: 0
@beruro
beruro force-pushed the junyu/rpc-diagnostics-lifecycle branch from 9d9ed11 to 94863db Compare July 30, 2026 08:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant