Skip to content

Conversation

@Anemy
Copy link
Member

@Anemy Anemy commented Nov 7, 2025

Hmmm a few of the recent CI flakes on mac have been for the AI query not being displayed:
element ("[data-testid="open-ai-query-entry-button"]") still not displayed after 120000ms
Example 1, Example 2
Looks like one of the mac hosts has a smaller screen size or isn't opening Compass at the default width/height and is instead using the min width. 1024x656

This log will help us know for sure what's happening (likely I'll message devprod). I think it'll help us in the future if something like this happens again.

Example log:

{"t":{"$date":"2025-11-07T21:38:17.975Z"},"s":"I","c":"COMPASS-WINDOW-MANAGER","id":1001000380,"ctx":"Window Manager","msg":"Creating new browser window","attr":{"options":{"x":293,"y":207,"width":1157,"height":702,"minWidth":1025,"minHeight":640,"icon":null,"show":false,"backgroundColor":"#ffffff","webPreferences":{"subpixel-font-scaling":true,"direct-write":true,"nodeIntegration":true,"contextIsolation":false,"enableRemoteModule":true,"nodeIntegrationInWorker":true,"webSecurity":false}},"screenSize":{"width":1728,"height":1079}}}

@Anemy Anemy requested a review from a team as a code owner November 7, 2025 21:41
Copilot AI review requested due to automatic review settings November 7, 2025 21:41
@Anemy Anemy added the no release notes Fix or feature not for release notes label Nov 7, 2025
@Anemy Anemy requested a review from lerouxb November 7, 2025 21:41
@Anemy Anemy added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Nov 7, 2025
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 adds logging to capture screen size and browser window options during window creation to help diagnose CI flakes on macOS where the AI query button fails to display due to potential window sizing issues.

Key Changes:

  • Import screen from electron to access display information
  • Add logging before window creation to record window options and primary display work area size

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codeowners-service-app
Copy link

Assigned nbbeeken for team compass-developers because lerouxb is out of office.


const primaryDisplay = electronScreen.getPrimaryDisplay();
log.info(
mongoLogId(1_001_000_380),
Copy link
Collaborator

Choose a reason for hiding this comment

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

Now I have to bump my log ids on my PR! 😅

Copy link
Member Author

@Anemy Anemy Nov 7, 2025

Choose a reason for hiding this comment

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

🙈 I am sososorry. Merge it first !

  • too slow!

@Anemy Anemy merged commit 1a86a3b into main Nov 8, 2025
64 of 65 checks passed
@Anemy Anemy deleted the log-screensize-compass branch November 8, 2025 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no release notes Fix or feature not for release notes no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants