What does this PR do?

Putting this POC into test here:

https://github.com/trpc/trpc/pull/5489

Fixes #13760

Requirement/Documentation

If there is a requirement document, please, share it here.
If there is ab UI/UX design document, please, share it here.

Type of change

Bug fix (non-breaking change which fixes an issue)
Chore (refactoring code, technical debt, workflow improvements)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Tests (Unit/Integration/E2E or any other test)
This change requires a documentation update

How should this be tested?

Are there environment variables that should be set?
What are the minimal test data to have?
What is expected (happy path) to have (input and output)?
Any other important info that could help to test that PR

Mandatory Tasks

[ ] Make sure you have self-reviewed the code. A decent size PR without self-review might be rejected.

Checklist

I haven't read the contributing guide
My code doesn't follow the style guidelines of this project
I haven't commented my code, particularly in hard-to-understand areas
I haven't checked if my PR needs changes to the documentation
I haven't checked if my changes generate no new warnings
I haven't added tests that prove my fix is effective or that my feature works
I haven't checked if new and existing unit tests pass locally with my changes

Mar 14 '24 00:03 zomars

CAL-3166 POC: tRPC lazy loaded routers

Mar 14 '24 00:03 linear[bot]

Thank you for following the naming conventions! 🙏 Feel free to join our discord and post your PR link.

Mar 14 '24 00:03 github-actions[bot]

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

4 Ignored Deployments

Name	Status	Preview	Updated (UTC)
ai	⬜️ Ignored (Inspect)	Visit Preview	Mar 14, 2024 2:29am
cal	⬜️ Ignored (Inspect)	Visit Preview	Mar 14, 2024 2:29am
calcom-web-canary	⬜️ Ignored (Inspect)	Visit Preview	Mar 14, 2024 2:29am
qa	⬜️ Ignored (Inspect)	Visit Preview	Mar 14, 2024 2:29am

Mar 14 '24 00:03 vercel[bot]

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/@types/[email protected]	None	`0`	1.63 MB	types
npm/@types/[email protected]	None	`0`	178 kB	types
npm/[email protected]	None	`0`	687 kB	davidgoli
npm/[email protected]	environment	`0`	212 kB	skn0tt

View full report↗︎

Mar 14 '24 00:03 socket-security[bot]

🚨 Potential security issues detected. Learn more about Socket for GitHub ↗︎

To accept the risk, merge this PR and you will not be notified again.

Alert	Package	Note	Source
Install scripts	npm/[email protected]	Install script: postinstall Source: `node -e "try{require('./postinstall')}catch(e){}"`	orphan: npm/[email protected]

View full report↗︎

Next steps

What is an install script?

Install scripts are run when the package is installed. The majority of malware in npm is hidden in install scripts.

Packages should not be running non-essential scripts during install and there are often solutions to problems people solve with install scripts that can be run at publish time instead.

Take a deeper look at the dependency

Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev.

Remove the package

If you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency.

Mark a package as acceptable risk

To ignore an alert, reply with a comment starting with @SocketSecurity ignore followed by a space separated list of ecosystem/package-name@version specifiers. e.g. @SocketSecurity ignore npm/[email protected] or ignore all packages with @SocketSecurity ignore-all

@SocketSecurity ignore npm/[email protected]

Mar 14 '24 00:03 socket-security[bot]

Current Playwright Test Results Summary

✅ 82 Passing - ❌ 13 Failing - ⚠️ 2 Flaky

Run may still be in progress, this comment will be updated as current testing workflow or job completes...

(Last updated on 03/14/2024 03:47:10pm UTC)

Run Details

Running Workflow PR Update on Github Actions

Commit: 2da898df162090c4301db4f30d36fa50479053b3

Started: 03/14/2024 03:43:27pm UTC

❌ Failures

📄 apps/web/playwright/booking-limits.e2e.ts • 3 Failures

Top 1 Common Error Messages

Timed out 10000ms waiting for expect(received).toBeVisible()

3 Test Cases Affected

Test Case Results

Test Case Last 7 days Failures Last 7 days Flakes

Test Case	Last 7 days Failures	Last 7 days Flakes
Duration limits day Retry 2 • Retry 1 • Initial Attempt Error: Timed out 10000ms waiting for expect(received).toBeVisible()... `Timed out 10000ms waiting for expect(received).toBeVisible() Call log: - expect.toBeVisible with timeout 10000ms - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true }) - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true })`	4.78% (12) 12 / 251 runs failed over last 7 days	0% (0) 0 / 251 runs flaked over last 7 days
Duration limits week Retry 2 • Retry 1 • Initial Attempt Error: Timed out 10000ms waiting for expect(received).toBeVisible()... `Timed out 10000ms waiting for expect(received).toBeVisible() Call log: - expect.toBeVisible with timeout 10000ms - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true }) - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true })`	5.16% (13) 13 / 252 runs failed over last 7 days	0% (0) 0 / 252 runs flaked over last 7 days
Duration limits month Retry 1 • Initial Attempt Error: Timed out 10000ms waiting for expect(received).toBeVisible()... `Timed out 10000ms waiting for expect(received).toBeVisible() Call log: - expect.toBeVisible with timeout 10000ms - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true }) - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true })`	4.80% (12) 12 / 250 runs failed over last 7 days	0.40% (1) 1 / 250 run flaked over last 7 days

Duration limits day
Retry 2 • Retry 1 • Initial Attempt

Error: Timed out 10000ms waiting for expect(received).toBeVisible()...

Timed out 10000ms waiting for expect(received).toBeVisible()
Call log:
  - expect.toBeVisible with timeout 10000ms
  - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true })
  - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true })

4.78% (12)

12 / 251 runs
failed over last 7 days

0% (0)

0 / 251 runs
flaked over last 7 days

Duration limits week
Retry 2 • Retry 1 • Initial Attempt

Error: Timed out 10000ms waiting for expect(received).toBeVisible()...

Timed out 10000ms waiting for expect(received).toBeVisible()
Call log:
  - expect.toBeVisible with timeout 10000ms
  - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true })
  - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true })

5.16% (13)

13 / 252 runs
failed over last 7 days

0% (0)

0 / 252 runs
flaked over last 7 days

Duration limits month
Retry 1 • Initial Attempt

Error: Timed out 10000ms waiting for expect(received).toBeVisible()...

Timed out 10000ms waiting for expect(received).toBeVisible()
Call log:
  - expect.toBeVisible with timeout 10000ms
  - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true })
  - waiting for locator('[data-testid="day"][data-disabled="false"]').getByText('8', { exact: true })

4.80% (12)

12 / 250 runs
failed over last 7 days

0.40% (1)

1 / 250 run
flaked over last 7 days

📄 apps/web/playwright/organization/booking.e2e.ts • 2 Failures

Top 1 Common Error Messages

Test timeout of 60000ms exceeded.

2 Test Cases Affected

Test Case Results

Test Case	Last 7 days Failures	Last 7 days Flakes
Bookings Team Event Can create a booking for Collective EventType Retry 2 • Retry 1 • Initial Attempt Error: Test timeout of 60000ms exceeded. `Test timeout of 60000ms exceeded.`	5.45% (14) 14 / 257 runs failed over last 7 days	0% (0) 0 / 257 runs flaked over last 7 days
Bookings Team Event Can create a booking for Round Robin EventType Retry 2 • Retry 1 • Initial Attempt Error: Test timeout of 60000ms exceeded. `Test timeout of 60000ms exceeded.`	5.08% (13) 13 / 256 runs failed over last 7 days	0% (0) 0 / 256 runs flaked over last 7 days

📄 apps/web/playwright/integrations-stripe.e2e.ts • 2 Failures

Top 1 Common Error Messages

Test timeout of 60000ms exceeded.

2 Test Cases Affected

Test Case Results

Test Case	Last 7 days Failures	Last 7 days Flakes
Stripe integration when enabling Stripe, credentialId is included Retry 2 • Retry 1 • Initial Attempt Error: Test timeout of 60000ms exceeded. `Test timeout of 60000ms exceeded.`	1.56% (4) 4 / 257 runs failed over last 7 days	0.39% (1) 1 / 257 run flaked over last 7 days
Stripe integration when enabling Stripe, team credentialId is included Retry 2 • Retry 1 • Initial Attempt Error: Test timeout of 60000ms exceeded. `Test timeout of 60000ms exceeded.`	1.95% (5) 5 / 257 runs failed over last 7 days	0.39% (1) 1 / 257 run flaked over last 7 days

📄 apps/web/playwright/dynamic-booking-pages.e2e.ts • 2 Failures

Top 1 Common Error Messages

Test timeout of 60000ms exceeded.

2 Test Cases Affected

Test Case Results

Test Case	Last 7 days Failures	Last 7 days Flakes
dynamic booking Retry 2 • Retry 1 • Initial Attempt Error: Test timeout of 60000ms exceeded. `Test timeout of 60000ms exceeded.`	1.97% (5) 5 / 254 runs failed over last 7 days	0.39% (1) 1 / 254 run flaked over last 7 days
Organization: Can book a time slot for an organization Retry 2 • Retry 1 • Initial Attempt Error: Test timeout of 60000ms exceeded. `Test timeout of 60000ms exceeded.`	0.78% (2) 2 / 255 runs failed over last 7 days	0% (0) 0 / 255 runs flaked over last 7 days

📄 apps/web/playwright/teams.e2e.ts • 1 Failure

Test Case Results

Test Case	Last 7 days Failures	Last 7 days Flakes
Teams - NonOrg -- legacy Can create a booking for Round Robin EventType Retry 1 • Initial Attempt Error: Test timeout of 60000ms exceeded. `Test timeout of 60000ms exceeded.`	3.83% (10) 10 / 261 runs failed over last 7 days	0.38% (1) 1 / 261 run flaked over last 7 days

📄 packages/embeds/embed-core/playwright/tests/action-based.e2e.ts • 1 Failure

Test Case Results

Test Case	Last 7 days Failures	Last 7 days Flakes
Popup Tests should open embed iframe on click - Configured with light theme Retry 2 • Retry 1 • Initial Attempt Error: Test timeout of 60000ms exceeded. `Test timeout of 60000ms exceeded.`	5.49% (14) 14 / 255 runs failed over last 7 days	44.31% (113) 113 / 255 runs flaked over last 7 days

📄 packages/embeds/embed-core/playwright/tests/embed-pages.e2e.ts • 1 Failure

Test Case Results

Test Case	Last 7 days Failures	Last 7 days Flakes
Embed Pages Event Type Page: should not have margin top on embed page Retry 2 • Retry 1 • Initial Attempt Error: Test timeout of 60000ms exceeded. `Test timeout of 60000ms exceeded.`	4.71% (12) 12 / 255 runs failed over last 7 days	0% (0) 0 / 255 runs flaked over last 7 days

📄 packages/embeds/embed-core/playwright/tests/inline.e2e.ts • 1 Failure

Test Case Results

Test Case	Last 7 days Failures	Last 7 days Flakes
Inline Iframe Inline Iframe - Configured with Dark Theme Retry 2 • Retry 1 • Initial Attempt Error: Test timeout of 60000ms exceeded. `Test timeout of 60000ms exceeded.`	5.47% (14) 14 / 256 runs failed over last 7 days	47.27% (121) 121 / 256 runs flaked over last 7 days

⚠️ Flakes

📄 packages/app-store/routing-forms/playwright/tests/basic.e2e.ts • 1 Flake

Test Case Results

Test Case	Last 7 days Failures	Last 7 days Flakes
Routing Forms Seeded Routing Form Test preview should return correct route Retry 1 • Initial Attempt	0% (0) 0 / 252 runs failed over last 7 days	7.54% (19) 19 / 252 runs flaked over last 7 days

📄 apps/web/playwright/signup.e2e.ts • 1 Flake

Test Case Results

Test Case	Last 7 days Failures	Last 7 days Flakes
Signup Flow Test Signup with valid (non premium) username Retry 2 • Retry 1 • Initial Attempt	1.87% (5) 5 / 268 runs failed over last 7 days	10.07% (27) 27 / 268 runs flaked over last 7 days

View Detailed Build Results

Mar 14 '24 00:03 deploysentinel[bot]

📦 Next.js Bundle Analysis for @calcom/web

This analysis was generated by the Next.js Bundle Analysis action. 🤖

🎉 Global Bundle Size Decreased

Page	Size (compressed)
`global`	`166.48 KB` (🟢 -21.91 KB)

Details

The global bundle is the javascript bundle that loads alongside every page. It is in its own category because its impact is much higher - an increase to its size means that every page on your website loads slower, and a decrease means every page loads faster.

Any third party scripts you have added directly to your app using the <script> tag are not accounted for in this analysis

If you want further insight into what is behind the changes, give @next/bundle-analyzer a try!

Mar 14 '24 00:03 github-actions[bot]

This PR is being marked as stale due to inactivity.

Apr 11 '24 00:04 github-actions[bot]

This PR is being marked as stale due to inactivity.

Apr 30 '24 00:04 github-actions[bot]

This PR is being marked as stale due to inactivity.

Jun 19 '24 00:06 github-actions[bot]

Closing due to staleness

Jul 17 '24 07:07 keithwillcode

cal.com
cal.com copied to clipboard

feat: WIP: POC: TRPC lazy routers

What does this PR do?

Requirement/Documentation

Type of change

How should this be tested?

Mandatory Tasks

Checklist

Next steps

Current Playwright Test Results Summary

❌ Failures

Top 1 Common Error Messages

Test Case Results

Top 1 Common Error Messages

Test Case Results

Top 1 Common Error Messages

Test Case Results

Top 1 Common Error Messages

Test Case Results

Test Case Results

Test Case Results

Test Case Results

Test Case Results

⚠️ Flakes

Test Case Results

Test Case Results

📦 Next.js Bundle Analysis for @calcom/web

🎉 Global Bundle Size Decreased

cal.com cal.com copied to clipboard

feat: WIP: POC: TRPC lazy routers

What does this PR do?

Requirement/Documentation

Type of change

How should this be tested?

Mandatory Tasks

Checklist

Next steps

Current Playwright Test Results Summary

❌ Failures

Top 1 Common Error Messages

Test Case Results

Top 1 Common Error Messages

Test Case Results

Top 1 Common Error Messages

Test Case Results

Top 1 Common Error Messages

Test Case Results

Test Case Results

Test Case Results

Test Case Results

Test Case Results

⚠️ Flakes

Test Case Results

Test Case Results

📦 Next.js Bundle Analysis for @calcom/web

🎉 Global Bundle Size Decreased

cal.com
cal.com copied to clipboard