Conversation
… to apps/web/modules This is part of the larger effort to move tRPC-dependent UI components from packages/features to apps/web/modules to eliminate circular dependencies. Changes: - Move BookingHistory.tsx and BookingHistoryPage.tsx to apps/web/modules/booking-audit/components/ - Update imports in apps/web to use the new location Co-Authored-By: benny@cal.com <sldisek783@gmail.com>
Contributor
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
…odules Co-Authored-By: benny@cal.com <sldisek783@gmail.com>
devin-ai-integration
bot
changed the title
…dules - Move useAppsData hook from packages/features/apps/hooks to apps/web/modules/apps/hooks - Move onboardingStore from packages/features/ee/organizations/lib to apps/web/modules/ee/organizations/lib - Move useWelcomeModal hook from packages/features/ee/organizations/hooks to apps/web/modules/ee/organizations/hooks - Move useAgentsData hook from packages/features/ee/workflows/hooks to apps/web/modules/ee/workflows/hooks - Update all import paths in consuming files Co-Authored-By: benny@cal.com <sldisek783@gmail.com>
devin-ai-integration
bot
changed the title
Co-Authored-By: benny@cal.com <sldisek783@gmail.com>
hbjORbj
commented
Jan 25, 2026
| @@ -29,12 +29,6 @@ export const initFormbricks = ({ | |||
| } | |||
| }; | |||
|
|
|||
| export const trackFormbricksAction = (eventName: string, properties: Record<string, string> = {}) => { | |||
Contributor
Author
There was a problem hiding this comment.
Extracted to apps/web/modules/formbricks/lib/trackFormbricksAction.ts
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
1 issue found across 30 files
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="apps/web/modules/formbricks/lib/trackFormbricksAction.ts">
<violation number="1" location="apps/web/modules/formbricks/lib/trackFormbricksAction.ts:4">
P3: Centralize Formbricks env values in a shared constants/config module and read from that instead of accessing process.env directly here.
(Based on your team's feedback about centralizing env-derived configuration.) [FEEDBACK_USED]</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
Contributor
Devin AI is addressing Cubic AI's review feedbackA Devin session has been created to address the issues identified by Cubic AI. |
Contributor
|
I reviewed the Cubic AI feedback on this PR. The only issue identified has a confidence score of 7/10, which is below the 9/10 threshold for automated fixes. Issue skipped:
Since this is a P3 suggestion with moderate confidence, it may be worth considering in a follow-up PR if the team agrees with the approach, but it doesn't meet the threshold for immediate automated changes. |
Contributor
E2E results are ready! |
Contributor
Devin AI is addressing Cubic AI's review feedbackA Devin session has been created to address the issues identified by Cubic AI. |
…-booking-audit-client-to-web
…-booking-audit-client-to-web
keithwillcode
approved these changes
Jan 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
This PR moves tRPC-dependent UI components from
packages/featurestoapps/web/modulesas part of the larger effort to relocate these components closer to where they're used.This is extracted from #26571 which was too large to review effectively.
Changes:
1. Booking-audit migration:
BookingHistory.tsxandBookingHistoryPage.tsxfrompackages/features/booking-audit/client/components/toapps/web/modules/booking-audit/components/2. Formbricks migration:
formbricks-client.tsfrompackages/features/formbricks/toapps/web/modules/formbricks/, split into:hooks/useFormbricks.ts- the React hook for initializing Formbrickslib/trackFormbricksAction.ts- the tracking utility function3. useAppsData hook migration:
Move
useAppsData.tsfrompackages/features/apps/hooks/toapps/web/modules/apps/hooks/Update imports in 3 files:
EventAppsTabEventTypeAppSettingsWrapperEventPaymentsTabPlatformWrapper4. onboardingStore migration:
onboardingStore.tsfrompackages/features/ee/organizations/lib/toapps/web/modules/ee/organizations/lib/onboardingStore.test.tsalongside the source file5. useWelcomeModal hook migration:
useWelcomeModal.tsfrompackages/features/ee/organizations/hooks/toapps/web/modules/ee/organizations/hooks/6. useAgentsData hook migration:
Move
useAgentsData.tsfrompackages/features/ee/workflows/hooks/toapps/web/modules/ee/workflows/hooks/Update imports in 1 file:
WorkflowDetailsPageMandatory Tasks (DO NOT REMOVE)