Skip to content

Conversation

@tomerqodo
Copy link

@tomerqodo tomerqodo commented Jan 21, 2026

Benchmark PR from qodo-benchmark#708


Note

Refactors Web UI to move various WebWrapper components from packages/platform/atoms into apps/web and updates all references.

  • Migrates EventTypeWebWrapper, booking (BookerWebWrapper), conferencing, and calendars wrappers into apps/web/modules/*; updates app pages and dynamic import paths accordingly
  • Introduces local DestinationCalendarSettingsWebWrapper and uses SelectedCalendarsSettingsWebWrapper from apps/web/modules/calendars; adjusts calendar settings usage in advanced tab
  • Centralizes host sorting into @calcom/lib/bookings/hostGroupUtils and updates consumers; removes inline/duplicate implementation
  • Cleans up atoms package exports (removes booker and wrapper exports; adds event-types hooks exports) and fixes import paths throughout
  • Minor internal cleanups (unused imports, router usage) without changing features

Written by Cursor Bugbot for commit 44582ee. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.


if (isRRWeightsEnabled) {
if (weightA === weightB) {
return priorityA - priorityB;
Copy link

Choose a reason for hiding this comment

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

Host sorting order reversed when weights equal

Medium Severity

The sortHosts function has inconsistent priority sorting. The old code in HostEditDialogs.tsx used priorityB - priorityA when weights are equal, sorting higher priority hosts first. The new implementation uses priorityA - priorityB, which reverses the sort order to put lower priority hosts first. This is also inconsistent with the else branch on line 58 which still uses priorityB - priorityA.

Fix in Cursor Fix in Web

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