chore: switching to the same pattern with connect

apps/web/app/connect/callback/CallbackEffect.tsx

@@ -2,8 +2,7 @@
 
 import {Loader2} from 'lucide-react'
 import React from 'react'
-
-import type {FrameMessage} from '@usevenice/connect/common'
+import type {FrameMessage} from '@usevenice/connect'
 
 export function CallbackEffect(props: {
   msg: FrameMessage | null

apps/web/tsconfig.json

@@ -15,12 +15,7 @@
     "jsx": "preserve",
     "strictNullChecks": true,
     "plugins": [{"name": "next"}],
-    "paths": {
-      "@/*": ["./*"],
-      // Override because we don't want to use ./dist unnecessarily
-      "@usevenice/connect": ["../../kits/connect/src/index.ts"],
-      "@usevenice/connect/*": ["../../kits/connect/src/*"]
-    }
+    "paths": {"@/*": ["./*"]}
   },
   "include": ["next-env.d.ts", "**/*.ts", "**/*.tsx", ".next/types/**/*.ts"],
   "exclude": ["node_modules"]

kits/connect/index.ts

@@ -0,0 +1,3 @@
+// Do not but any other logic in this file, which is only used to re-export from
+// src/index so that vscode jump to definition works inside the monorepo
+export * from './src/index'

tsconfig.json

@@ -19,12 +19,7 @@
     // In practice no module outside apps/web should use the `@/` alias ever
     // In addition to opening the door to invalid imports this also prevents us
     // from two separate next.js apps in the same monorepo which might be relevant for future
-    "paths": {
-      "@/*": ["./apps/web/*"],
-      // Override because we don't want to use ./dist unnecessarily
-      "@usevenice/connect": ["./kits/connect/src/index.ts"],
-      "@usevenice/connect/*": ["./kits/connect/src/*"]
-    }
+    "paths": {"@/*": ["./apps/web/*"]}
   },
   "include": ["env.d.ts", "**/*.ts", "**/*.tsx"],
   "exclude": ["node_modules"]