perf: do not import from @calcom/lib/server barrel file

[hbjORbj]

What does this PR do?

Context: I loaded a route that had import { getServerErrorFromUnknown } from "@calcom/lib/server". This loaded getLuckyUser (which is exported from "@calcom/lib/server"), which was pulling in Calendar Manager, etc:

../../packages/lib/server/index.ts
../../packages/lib/server/getLuckyUser.ts
../../packages/core/CalendarManager.ts
../../packages/app-store/_utils/getCalendar.ts
../../packages/app-store/index.ts
../../packages/app-store/exchange***endar/index.ts
../../packages/app-store/exchange***endar/lib/index.ts
../../packages/app-store/exchange***endar/lib/CalendarService.ts
../../node_modules/@ewsjs/xhr/dist/index.js
../../node_modules/@ewsjs/ntlm-client/lib/index.js

This poses 2 problems: (1) it prevents us from using Edge config for running routes or apis because these unnecessarily pulled dependencies usually depend on other deps that run on Node only (2) unnecessary load of irrelevant packages

Mandatory Tasks (DO NOT REMOVE)

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

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments

Name	Status	Preview	Comments	Updated (UTC)
cal	⬜️ Ignored (Inspect)			Feb 26, 2025 3:58pm
calcom-web-canary	⬜️ Ignored (Inspect)			Feb 26, 2025 3:58pm

[graphite-app]

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (02/26/25)

1 reviewer was added to this PR based on Keith Williams's automation.