Skip to content

[chat-perf] layoutDurationMs regressed ~20% in thinking-response scenario #337903

Description

@vs-code-engineering

Summary:
New regression detected: layoutDurationMs in the thinking-response scenario increased by +20.5% (from 130.21ms ±4ms to 156.95ms ±5.43ms).

Evidence:

  • Current: 156.95ms ±5.43ms
  • Baseline: 130.21ms ±4ms
  • Absolute delta: +26.74ms
  • Relative delta: +20.5%
  • p-value: 0 (statistically significant)
  • No prior baseline runs available (first run), but the change is large and outside typical noise for this metric.
  • No prior Slack threads or open issues cover this regression.

Suspect commits:

  • d92b683 (Merge branch 'main' into fix-281241): includes changes to chat widget, layout, and chatContentParts/chatThinkingContentPart.ts, which directly affect layout timing.

Suggested action:

  • Investigate recent changes in chat widget and layout logic, especially in chatContentParts/chatThinkingContentPart.ts and related files from d92b683.
  • Confirm if the regression is expected or needs mitigation.
  • Monitor for further layout regressions in chat scenarios.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions