hotfix: fix 405 error on Vercel login redirect (use 303 status)

alextnetto · cursoragent · alextnetto · commit b10ff96b2f66 · 2026-02-05T22:55:32.000-03:00
Co-authored-by: Cursor &lt;cursoragent@cursor.com&gt;
diff --git a/apps/dashboard/src/app/api/auth/route.ts b/apps/dashboard/src/app/api/auth/route.ts
@@ -15,7 +15,7 @@ export async function POST(request: NextRequest) {
   const password = formData.get('password');
 
   if (typeof password !== 'string' || password !== secret) {
-    return NextResponse.redirect(new URL('/login?error=1', request.url));
+    return NextResponse.redirect(new URL('/login?error=1', request.url), 303);
   }
 
   const cookieValue = await getAuthCookieValue();
@@ -26,7 +26,7 @@ export async function POST(request: NextRequest) {
     );
   }
 
-  const response = NextResponse.redirect(new URL('/', request.url));
+  const response = NextResponse.redirect(new URL('/', request.url), 303);
   response.cookies.set('dashboard_auth', cookieValue, {
     httpOnly: true,
     sameSite: 'lax',
diff --git a/apps/dashboard/src/app/api/metrics/top-addresses/route.ts b/apps/dashboard/src/app/api/metrics/top-addresses/route.ts
@@ -0,0 +1,27 @@
+import { NextRequest, NextResponse } from 'next/server';
+
+import { resolveEnsNames } from '../../../../lib/ens';
+import { getTopFollowedAddresses } from '../../../../lib/metrics';
+
+export const revalidate = 30;
+
+export async function GET(request: NextRequest) {
+  const limitParam = request.nextUrl.searchParams.get('limit');
+  const limit = limitParam ? Math.min(Math.max(Number(limitParam), 1), 50) : 10;
+
+  try {
+    const items = await getTopFollowedAddresses(limit);
+    const addresses = items.map((item) => item.address);
+    const { map: ensMap, available: ensAvailable } = await resolveEnsNames(addresses);
+
+    const itemsWithEns = items.map((item) => ({
+      ...item,
+      ens: ensMap[item.address] ?? null,
+    }));
+
+    return NextResponse.json({ items: itemsWithEns, ensAvailable });
+  } catch (error) {
+    console.error('Failed to load top followed addresses:', error);
+    return NextResponse.json({ error: 'Failed to load top followed addresses.' }, { status: 500 });
+  }
+}
diff --git a/apps/dashboard/src/app/api/metrics/users-by-dao-count/route.ts b/apps/dashboard/src/app/api/metrics/users-by-dao-count/route.ts
@@ -0,0 +1,15 @@
+import { NextResponse } from 'next/server';
+
+import { getUsersByDaoCount } from '../../../../lib/metrics';
+
+export const revalidate = 30;
+
+export async function GET() {
+  try {
+    const items = await getUsersByDaoCount();
+    return NextResponse.json({ items });
+  } catch (error) {
+    console.error('Failed to load users by DAO count:', error);
+    return NextResponse.json({ error: 'Failed to load users by DAO count.' }, { status: 500 });
+  }
+}
diff --git a/apps/dashboard/src/components/charts/top-followed-accounts-card.tsx b/apps/dashboard/src/components/charts/top-followed-accounts-card.tsx
@@ -0,0 +1,111 @@
+'use client';
+
+import { useState } from 'react';
+import {
+  BarChart,
+  Bar,
+  XAxis,
+  YAxis,
+  CartesianGrid,
+  Tooltip,
+  ResponsiveContainer,
+} from 'recharts';
+
+type TopFollowedAccount = {
+  address: string;
+  followerCount: number;
+  ens: string | null;
+};
+
+type TopFollowedAccountsCardProps = {
+  data: TopFollowedAccount[];
+};
+
+function truncateAddress(address: string) {
+  return `${address.slice(0, 6)}...${address.slice(-4)}`;
+}
+
+function CopyableAddress({ address, ens }: { address: string; ens: string | null }) {
+  const [copied, setCopied] = useState(false);
+
+  async function handleCopy() {
+    try {
+      await navigator.clipboard.writeText(address);
+      setCopied(true);
+      setTimeout(() => setCopied(false), 1500);
+    } catch {
+      // clipboard API may not be available
+    }
+  }
+
+  return (
+    <button
+      onClick={handleCopy}
+      title={`Copy ${address}`}
+      className="flex items-center gap-1.5 rounded px-1.5 py-0.5 text-xs text-muted hover:bg-white/5 hover:text-text transition-colors"
+    >
+      <span className="font-mono">{ens ?? truncateAddress(address)}</span>
+      <span className="text-[10px] opacity-60">
+        {copied ? '✓' : '⧉'}
+      </span>
+    </button>
+  );
+}
+
+export default function TopFollowedAccountsCard({ data }: TopFollowedAccountsCardProps) {
+  const chartData = data.map((item) => ({
+    ...item,
+    label: item.ens ?? truncateAddress(item.address),
+  }));
+
+  return (
+    <div className="rounded-xl border border-border bg-panel p-4 shadow-sm">
+      <h3 className="text-sm font-semibold text-text">Top followed accounts</h3>
+      <div className="mt-4 h-60">
+        <ResponsiveContainer width="100%" height="100%">
+          <BarChart data={chartData}>
+            <CartesianGrid strokeDasharray="3 3" stroke="#1f2937" />
+            <XAxis
+              dataKey="label"
+              tick={{ fill: '#cbd5f5', fontSize: 12 }}
+              axisLine={{ stroke: '#334155' }}
+              tickLine={{ stroke: '#334155' }}
+              interval={0}
+              angle={-45}
+              textAnchor="end"
+              height={60}
+            />
+            <YAxis
+              tick={{ fill: '#cbd5f5', fontSize: 12 }}
+              axisLine={{ stroke: '#334155' }}
+              tickLine={{ stroke: '#334155' }}
+            />
+            <Tooltip
+              contentStyle={{
+                backgroundColor: '#1e293b',
+                border: '1px solid #475569',
+                borderRadius: '6px',
+              }}
+              labelStyle={{ color: '#f1f5f9', fontWeight: 600 }}
+              itemStyle={{ color: '#94a3b8' }}
+              labelFormatter={(_label, payload) => {
+                if (!payload?.[0]) return _label;
+                const item = payload[0].payload as (typeof chartData)[number];
+                return item.ens ? `${item.ens} (${truncateAddress(item.address)})` : item.address;
+              }}
+              formatter={(value: number) => [value, 'Followers']}
+            />
+            <Bar dataKey="followerCount" fill="#38bdf8" radius={[4, 4, 0, 0]} />
+          </BarChart>
+        </ResponsiveContainer>
+      </div>
+      {data.length > 0 && (
+        <div className="mt-3 flex flex-wrap gap-1">
+          {data.map((item) => (
+            <CopyableAddress key={item.address} address={item.address} ens={item.ens} />
+          ))}
+        </div>
+      )}
+    </div>
+  );
+}
diff --git a/apps/dashboard/src/components/dashboard-client.tsx b/apps/dashboard/src/components/dashboard-client.tsx
@@ -6,6 +6,7 @@ import MetricCard from './metric-card';
 import BarChartCard from './charts/bar-chart-card';
 import LineChartCard from './charts/line-chart-card';
 import PieChartCard from './charts/pie-chart-card';
+import TopFollowedAccountsCard from './charts/top-followed-accounts-card';
 import UsersTable from './tables/users-table';
 
 type SummaryTotals = {
@@ -69,12 +70,34 @@ type NotificationActivityResponse = {
   points: NotificationPoint[];
 };
 
+type TopFollowedAddress = {
+  address: string;
+  followerCount: number;
+  ens: string | null;
+};
+
+type TopFollowedAddressResponse = {
+  items: TopFollowedAddress[];
+  ensAvailable: boolean;
+};
+
+type UsersByDaoCount = {
+  daoCount: number;
+  userCount: number;
+};
+
+type UsersByDaoCountResponse = {
+  items: UsersByDaoCount[];
+};
+
 export default function DashboardClient() {
   const [summary, setSummary] = useState<SummaryResponse | null>(null);
   const [growth, setGrowth] = useState<GrowthPoint[]>([]);
   const [daos, setDaos] = useState<DaoItem[]>([]);
   const [daoNotifications, setDaoNotifications] = useState<DaoNotificationItem[]>([]);
   const [notificationActivity, setNotificationActivity] = useState<NotificationPoint[]>([]);
+  const [topAddresses, setTopAddresses] = useState<TopFollowedAddress[]>([]);
+  const [usersByDaoCount, setUsersByDaoCount] = useState<UsersByDaoCount[]>([]);
   const [notificationDaoFilter, setNotificationDaoFilter] = useState('');
   const [loading, setLoading] = useState(true);
   const [error, setError] = useState<string | null>(null);
@@ -86,20 +109,24 @@ export default function DashboardClient() {
       setLoading(true);
       setError(null);
       try {
-        const [summaryRes, growthRes, daosRes, notificationsRes, activityRes] = await Promise.all([
+        const [summaryRes, growthRes, daosRes, notificationsRes, activityRes, topAddressesRes, usersByDaoCountRes] = await Promise.all([
           fetch('/api/metrics/summary'),
           fetch('/api/metrics/growth'),
           fetch('/api/metrics/daos'),
           fetch('/api/metrics/notifications-by-dao'),
           fetch('/api/metrics/notification-activity'),
+          fetch('/api/metrics/top-addresses'),
+          fetch('/api/metrics/users-by-dao-count'),
         ]);
 
         if (
           !summaryRes.ok ||
           !growthRes.ok ||
           !daosRes.ok ||
           !notificationsRes.ok ||
-          !activityRes.ok
+          !activityRes.ok ||
+          !topAddressesRes.ok ||
+          !usersByDaoCountRes.ok
         ) {
           throw new Error('Failed to load dashboard metrics.');
         }
@@ -109,13 +136,17 @@ export default function DashboardClient() {
         const daosJson = (await daosRes.json()) as DaoResponse;
         const notificationsJson = (await notificationsRes.json()) as DaoNotificationResponse;
         const activityJson = (await activityRes.json()) as NotificationActivityResponse;
+        const topAddressesJson = (await topAddressesRes.json()) as TopFollowedAddressResponse;
+        const usersByDaoCountJson = (await usersByDaoCountRes.json()) as UsersByDaoCountResponse;
 
         if (!cancelled) {
           setSummary(summaryJson);
           setGrowth(growthJson.points ?? []);
           setDaos(daosJson.items ?? []);
           setDaoNotifications(notificationsJson.items ?? []);
           setNotificationActivity(activityJson.points ?? summaryJson.notificationActivity ?? []);
+          setTopAddresses(topAddressesJson.items ?? []);
+          setUsersByDaoCount(usersByDaoCountJson.items ?? []);
         }
       } catch (err) {
         if (!cancelled) {
@@ -234,7 +265,7 @@ export default function DashboardClient() {
         </div>
       </section>
 
-      <section className="grid gap-4 lg:grid-cols-2 xl:grid-cols-4">
+      <section className="grid gap-4 lg:grid-cols-2 xl:grid-cols-3">
         <BarChartCard
           title="Top DAOs by subscribers"
           data={daoChart}
@@ -264,6 +295,17 @@ export default function DashboardClient() {
           barKey="userCount"
           valueLabel="Users"
         />
+        <TopFollowedAccountsCard data={topAddresses} />
+        <BarChartCard
+          title="DAOs tracked per user"
+          data={usersByDaoCount.map((item) => ({
+            ...item,
+            label: String(item.daoCount),
+          }))}
+          xKey="label"
+          barKey="userCount"
+          valueLabel="Users"
+        />
       </section>
 
       <section className="flex flex-col gap-3">
diff --git a/apps/dashboard/src/lib/metrics.test.ts b/apps/dashboard/src/lib/metrics.test.ts
@@ -5,6 +5,8 @@ import {
   buildEngagementDistributionQuery,
   buildGrowthSeries,
   buildNotificationActivityByDaoQuery,
+  buildTopFollowedAddressesQuery,
+  buildUsersByDaoCountQuery,
   buildUsersQueries,
 } from './metrics';
 
@@ -70,6 +72,27 @@ test('buildEngagementDistributionQuery groups by coalesced address count', () =>
   assert.match(text, /ORDER BY COALESCE\(addr\.address_count, 0\)/);
 });
 
+test('buildTopFollowedAddressesQuery groups by address and limits results', () => {
+  const { text, values } = buildTopFollowedAddressesQuery(5);
+
+  assert.match(text, /FROM user_addresses/);
+  assert.match(text, /COUNT\(DISTINCT ua\.user_id\)/);
+  assert.match(text, /LOWER\(ua\.address\)/);
+  assert.match(text, /ORDER BY follower_count DESC/);
+  assert.match(text, /LIMIT \$1/);
+  assert.equal(values[0], 5);
+});
+
+test('buildUsersByDaoCountQuery counts active preferences per user', () => {
+  const text = buildUsersByDaoCountQuery();
+
+  assert.match(text, /FROM users u/);
+  assert.match(text, /LEFT JOIN user_preferences p ON p\.user_id = u\.id AND p\.is_active = true/);
+  assert.match(text, /COUNT\(p\.id\) as dao_count/);
+  assert.match(text, /GROUP BY dao_count/);
+  assert.match(text, /ORDER BY dao_count/);
+});
+
 test('buildGrowthSeries fills missing days with zero counts', () => {
   const rows = [
     { day: '2024-01-01T00:00:00.000Z', count: '2' },
diff --git a/apps/dashboard/src/lib/metrics.ts b/apps/dashboard/src/lib/metrics.ts

Original file line number	Diff line number	Diff line change
`@@ -15,7 +15,7 @@ export async function POST(request: NextRequest) {`
`15`	`15`	`const password = formData.get('password');`
`16`	`16`
`17`	`17`	`if (typeof password !== 'string' \|\| password !== secret) {`
`18`		`- return NextResponse.redirect(new URL('/login?error=1', request.url));`
	`18`	`+ return NextResponse.redirect(new URL('/login?error=1', request.url), 303);`
`19`	`19`	`}`
`20`	`20`
`21`	`21`	`const cookieValue = await getAuthCookieValue();`
`@@ -26,7 +26,7 @@ export async function POST(request: NextRequest) {`
`26`	`26`	`);`
`27`	`27`	`}`
`28`	`28`
`29`		`- const response = NextResponse.redirect(new URL('/', request.url));`
	`29`	`+ const response = NextResponse.redirect(new URL('/', request.url), 303);`
`30`	`30`	`response.cookies.set('dashboard_auth', cookieValue, {`
`31`	`31`	`httpOnly: true,`
`32`	`32`	`sameSite: 'lax',`