Skip to content

Conversation

@tomerqodo
Copy link

Benchmark PR calcom#26629

Type: Corrupted (contains bugs)

Original PR Title: refactor: move WebWrapper files from packages/platform to apps/web/modules
Original PR Description: ## What does this PR do?

Moves WebWrapper files from packages/platform/atoms to apps/web/modules.

Why?

As part of moving components/hooks from packages/features to apps/web, the WebWrapper components in packages/platform/atoms were importing those moved files via @calcom/web. Moving the WebWrapper files to apps/web allows them to import from sibling locations instead of cross-package, unblocking the larger refactoring effort.

Files moved:

  • EventTypeWebWrapper and tab wrappers → apps/web/modules/event-types/components/
  • BookerWebWrapperapps/web/modules/bookings/components/
  • ConferencingAppsViewWebWrapperapps/web/modules/apps/components/
  • SelectedCalendarsSettingsWebWrapperapps/web/modules/calendars/components/
  • AddMembersWithSwitchWebWrapperapps/web/modules/event-types/components/
  • DestinationCalendarSettingsWebWrapperapps/web/components/apps/

Additional changes:

  • Moved sortHosts utility to @calcom/lib/bookings/hostGroupUtils.ts for shared access
  • Added hook exports to @calcom/atoms package.json (useEventTypeForm, useHandleRouteChange, useTabsNavigations)
  • Removed WebWrapper exports from @calcom/atoms package.json

Updates since last revision:

  • Fixed deep relative imports in EventTypeWebWrapper.tsx to use @calcom/atoms package alias instead of traversing outside apps/web (addresses Cubic AI review feedback)

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. N/A
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

Run yarn type-check:ci --force and verify event types, booking pages, and settings pages load correctly.

Human Review Checklist

  • Verify sortHosts function move to @calcom/lib/bookings/hostGroupUtils.ts doesn't break existing functionality
  • Confirm no external consumers depend on the deleted @calcom/atoms/booker export
  • Check that @calcom/atoms hook exports resolve correctly

Link to Devin run: https://app.devin.ai/sessions/ffdc374f531f46a399a052a715abe50b
Requested by: @hbjORbj
Original PR URL: calcom#26629

@github-actions
Copy link

Hey there and thank you for opening this pull request! 👋🏼

We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.

Details:

No release type found in pull request title "[CORRUPTED] Synthetic Benchmark PR #26629 - refactor: move WebWrapper files from packages/platform to apps/web/modules". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

@github-actions
Copy link

This PR has been marked as stale due to inactivity. If you're still working on it or need any help, please let us know or update the PR to keep it active.

@github-actions github-actions bot added the Stale label Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants