components/ui/button.tsx (1)

39-58: Forward refs through Button

Right now Button is a plain function component, so any consumer passing a ref (for focus, measurement, etc.) gets null, unlike a native <button>. Wrap it in React.forwardRef and forward the ref through to preserve accessibility ergonomics.

-function Button({
-  className,
-  variant,
-  size,
-  asChild = false,
-  ...props
-}: React.ComponentProps<"button"> &
-  VariantProps<typeof buttonVariants> & {
-    asChild?: boolean;
-  }) {
-  const Comp = asChild ? Slot : "button";
-
-  return (
-    <Comp
-      data-slot="button"
-      className={cn(buttonVariants({ variant, size, className }))}
-      {...props}
-    />
-  );
-}
-
-export { Button, buttonVariants };
+const Button = React.forwardRef<
+  HTMLButtonElement,
+  React.ComponentPropsWithoutRef<"button"> &
+    VariantProps<typeof buttonVariants> & {
+      asChild?: boolean;
+    }
+>(({ className, variant, size, asChild = false, ...props }, ref) => {
+  const Comp = asChild ? Slot : "button";
+
+  return (
+    <Comp
+      ref={ref}
+      data-slot="button"
+      className={cn(buttonVariants({ variant, size, className }))}
+      {...props}
+    />
+  );
+});
+Button.displayName = "Button";
+
+export { Button, buttonVariants };

components/globe.tsx (1)

23-23: Clean up redundant Tailwind classes.

The className has a double space and includes both dark:invert and invert-0, which is redundant since the default is no invert.

Apply this diff:

-    <div className="relative w-[400px] max-w-5xl mx-auto aspect-square  dark:invert invert-0 saturate-100 brightness-100">
+    <div className="relative w-[400px] max-w-5xl mx-auto aspect-square dark:invert saturate-100 brightness-100">

app/(portfolio)/page.tsx (3)

2-2: Remove unused import.

TextEffect is imported but never used in this component.

Apply this diff:

 import { Globe } from "@/components/globe";
-import { TextEffect } from "@/components/ui/text-effect";

---

4-4: Remove unnecessary async keyword.

The function is marked async but contains no await statements or asynchronous operations.

Apply this diff:

-export default async function IndexPage() {
+export default function IndexPage() {

---

9-12: Replace placeholder content.

The lorem ipsum text should be replaced with actual portfolio content.

Would you like me to open an issue to track replacing this placeholder content?

components/ui/globe.tsx (2)

147-150: Optimize duplicate point filtering.

The current implementation uses findIndex inside filter, resulting in O(n²) complexity. For the current dataset (~80 points), this is acceptable, but a Set-based approach would be more efficient and scale better.

Apply this diff for O(n) complexity:

-    const uniquePoints = points.filter(
-      (v, i, a) =>
-        a.findIndex((v2) => v2.lat === v.lat && v2.lng === v.lng) === i
-    );
+    const seen = new Set<string>();
+    const uniquePoints = points.filter((v) => {
+      const key = `${v.lat},${v.lng}`;
+      if (seen.has(key)) return false;
+      seen.add(key);
+      return true;
+    });

---

227-227: Consider removing non-null assertion.

While the guard on Line 210 ensures globeRef.current exists, using the non-null assertion operator is slightly risky. A more defensive approach would check explicitly.

Apply this diff:

-      globeRef.current!.ringsData(ringsData);
+      if (globeRef.current) {
+        globeRef.current.ringsData(ringsData);
+      }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro