Skip to content

improve specificity for chat selector to fix smoketest#292922

Merged
meganrogge merged 1 commit intomainfrom
partial-chickadee
Feb 4, 2026
Merged

improve specificity for chat selector to fix smoketest#292922
meganrogge merged 1 commit intomainfrom
partial-chickadee

Conversation

@meganrogge
Copy link
Contributor

The smoketest was failing because Playwright found 2 Monaco editors with role="code" in the chat panel:

  • The chat input editor
  • A code block in the terminal tool response

The fix narrows the selector, ensuring only the chat input editor is targeted.

Copilot AI review requested due to automatic review settings February 4, 2026 20:02
@meganrogge meganrogge self-assigned this Feb 4, 2026
@meganrogge meganrogge added this to the February 2026 milestone Feb 4, 2026
@meganrogge meganrogge enabled auto-merge (squash) February 4, 2026 20:02
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a smoketest failure caused by Playwright finding multiple Monaco editors with role="code" in the chat panel. The fix narrows the CSS selector to specifically target only the chat input editor by adding the .interactive-input-part class to the selector chain.

Changes:

  • Renamed CHAT_EDITOR to CHAT_INPUT_EDITOR and updated selector to include .interactive-input-part class
  • Renamed CHAT_EDITOR_FOCUSED to CHAT_INPUT_EDITOR_FOCUSED with the same specificity improvement
  • Updated all references to use the new constant names throughout the Chat class

@meganrogge meganrogge merged commit b40b1db into main Feb 4, 2026
27 of 28 checks passed
@meganrogge meganrogge deleted the partial-chickadee branch February 4, 2026 20:36
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.

2 participants