Skip to content

fix: update Sessions link selector for workspace UI changes#643

Open
empiricalrun[bot] wants to merge 1 commit intomainfrom
chat-session_BKKup7qy6wQ2
Open

fix: update Sessions link selector for workspace UI changes#643
empiricalrun[bot] wants to merge 1 commit intomainfrom
chat-session_BKKup7qy6wQ2

Conversation

@empiricalrun
Copy link
Copy Markdown
Contributor

@empiricalrun empiricalrun Bot commented Jan 20, 2026

Summary

Fixed the failing test "Verify session creation and basic chat interaction from My Sessions" in tests/sessions.spec.ts (line 1016).

Root Cause

The UI was updated to support multiple workspaces/organizations. The "My Sessions" link no longer exists in the sidebar. Instead, there are now:

  • A root-level "Sessions" link (URL: /sessions)
  • A workspace-level "Sessions" link under each workspace (e.g., /lorem-ipsum/sessions)

Fix

Updated the selector on line 1016 from:

await page.getByRole('link', { name: 'My Sessions', exact: true }).click();

To:

await page.getByRole('link', { name: 'Sessions', exact: true }).first().click();

Using .first() ensures we target the root-level Sessions link (which appears first in the sidebar) rather than the workspace-level Sessions link.

Test Results

Test passed: Report URL

  • Navigation step completed in 49ms (previously timing out after 15000ms)
  • All assertions passed
  • Session creation, message sending, and minimap verification all working correctly

Evidence

Before fix: 100% failure rate across 3 retries in test run #45292
After fix: Test passes consistently

PR created from session #71120

Updated at 2026-01-20 12:39:26.696 UTC

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.

0 participants