Move /admin under /api and prefix everything in backend with /api via API_BASE (#7)

Author: ambv

backend/app/routers/admin.py

@@ -59,7 +59,7 @@ class TokenAnalytics(BaseModel):
     never_used_tokens: int
     recent_active_tokens: int
 
-router = APIRouter(prefix="/admin", tags=["admin"])
+router = APIRouter(prefix="/api/admin", tags=["admin"])
 
 
 @router.get("/auth/github")

frontend/Dockerfile

@@ -15,7 +15,7 @@ COPY --from=deps /app/node_modules ./node_modules
 COPY . .
 
 # Set the API base URL as a build argument
-ARG NEXT_PUBLIC_API_BASE=http://localhost:8000
+ARG NEXT_PUBLIC_API_BASE=http://localhost:8000/api
 ENV NEXT_PUBLIC_API_BASE=$NEXT_PUBLIC_API_BASE
 
 # Build the application

frontend/src/app/admin/components/AdminUsersManager.tsx

@@ -60,7 +60,7 @@ export default function AdminUsersManager() {
   });
   const { toast } = useToast();
 
-  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000';
+  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000/api';
 
   useEffect(() => {
     fetchUsers();

frontend/src/app/admin/components/BinariesManager.tsx

@@ -42,7 +42,7 @@ export default function BinariesManager() {
   const [isDialogOpen, setIsDialogOpen] = useState(false);
   const { toast } = useToast();
 
-  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000';
+  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000/api';
 
   const [formData, setFormData] = useState({
     id: '',

frontend/src/app/admin/components/EnvironmentsManager.tsx

@@ -39,7 +39,7 @@ export default function EnvironmentsManager() {
   const [isDialogOpen, setIsDialogOpen] = useState(false);
   const { toast } = useToast();
 
-  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000';
+  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000/api';
 
   const [formData, setFormData] = useState({
     id: '',

frontend/src/app/admin/components/RunsManager.tsx

@@ -79,7 +79,7 @@ export default function RunsManager() {
   });
   const { toast } = useToast();
 
-  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000';
+  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000/api';
 
   const [filters, setFilters] = useState({
     commit_sha: '',

frontend/src/app/admin/page.tsx

@@ -30,7 +30,7 @@ export default function AdminPage() {
   const [authenticating, setAuthenticating] = useState(false);
   const { toast } = useToast();
 
-  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000';
+  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000/api';
 
   useEffect(() => {
     checkAuth();

frontend/src/app/auth/callback/page.tsx

@@ -20,7 +20,7 @@ function AuthCallbackContent() {
   );
   const [errorMessage, setErrorMessage] = useState('');
 
-  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000';
+  const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000/api';
 
   useEffect(() => {
     const handleCallback = async () => {

frontend/src/lib/api.ts

@@ -12,7 +12,7 @@ import type {
   TokenAnalytics,
 } from './types';
 
-const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000';
+const API_BASE = process.env.NEXT_PUBLIC_API_BASE || 'http://localhost:8000/api';
 
 class ApiError extends Error {
   constructor(
@@ -98,12 +98,12 @@ async function fetchApi<T>(
 export const api = {
   // Commit endpoints
   getCommits: (skip: number = 0, limit: number = 100) =>
-    fetchApi<Commit[]>(`/api/commits?skip=${skip}&limit=${limit}`),
-  getCommit: (sha: string) => fetchApi<Commit>(`/api/commits/${sha}`),
+    fetchApi<Commit[]>(`/commits?skip=${skip}&limit=${limit}`),
+  getCommit: (sha: string) => fetchApi<Commit>(`/commits/${sha}`),
 
   // Binary endpoints
-  getBinaries: () => fetchApi<Binary[]>(`/api/binaries?_t=${Date.now()}`),
-  getBinary: (id: string) => fetchApi<Binary>(`/api/binaries/${id}`),
+  getBinaries: () => fetchApi<Binary[]>(`/binaries?_t=${Date.now()}`),
+  getBinary: (id: string) => fetchApi<Binary>(`/binaries/${id}`),
   getEnvironmentsForBinary: (binaryId: string) =>
     fetchApi<
       Array<{
@@ -113,7 +113,7 @@ export const api = {
         run_count: number;
         commit_count: number;
       }>
-    >(`/api/binaries/${binaryId}/environments`),
+    >(`/binaries/${binaryId}/environments`),
   getCommitsForBinaryAndEnvironment: (
     binaryId: string,
     environmentId: string
@@ -127,19 +127,19 @@ export const api = {
         python_version: { major: number; minor: number; patch: number };
         run_timestamp: string;
       }>
-    >(`/api/binaries/${binaryId}/environments/${environmentId}/commits`),
+    >(`/binaries/${binaryId}/environments/${environmentId}/commits`),
 
   // Environment endpoints
-  getEnvironments: () => fetchApi<Environment[]>('/api/environments'),
+  getEnvironments: () => fetchApi<Environment[]>('/environments'),
   getEnvironment: (id: string) =>
-    fetchApi<Environment>(`/api/environments/${id}`),
+    fetchApi<Environment>(`/environments/${id}`),
 
   // Python version endpoints
   getPythonVersions: () =>
-    fetchApi<PythonVersionFilterOption[]>('/api/python-versions'),
+    fetchApi<PythonVersionFilterOption[]>('/python-versions'),
 
   // Benchmark endpoints
-  getAllBenchmarks: () => fetchApi<string[]>('/api/benchmarks'),
+  getAllBenchmarks: () => fetchApi<string[]>('/benchmarks'),
   getBenchmarkNames: (params: {
     environment_id: string;
     binary_id: string;
@@ -152,7 +152,7 @@ export const api = {
     queryParams.append('python_major', params.python_major.toString());
     queryParams.append('python_minor', params.python_minor.toString());
 
-    return fetchApi<string[]>(`/api/benchmark-names?${queryParams.toString()}`);
+    return fetchApi<string[]>(`/benchmark-names?${queryParams.toString()}`);
   },
 
   // Diff endpoint
@@ -168,7 +168,7 @@ export const api = {
     queryParams.append('environment_id', params.environment_id);
     queryParams.append('metric_key', params.metric_key);
 
-    return fetchApi<DiffTableRow[]>(`/api/diff?${queryParams.toString()}`);
+    return fetchApi<DiffTableRow[]>(`/diff?${queryParams.toString()}`);
   },
 
   // Upload endpoint
@@ -183,7 +183,7 @@ export const api = {
     };
     benchmark_results: BenchmarkResultJson[];
   }) =>
-    fetchApi<{ success: boolean }>('/api/upload', {
+    fetchApi<{ success: boolean }>('/upload', {
       method: 'POST',
       body: JSON.stringify(data),
     }),
@@ -209,7 +209,7 @@ export const api = {
         high_watermark_bytes: number;
         total_allocated_bytes: number;
       }>
-    >(`/api/trends?${queryParams.toString()}`);
+    >(`/trends?${queryParams.toString()}`);
   },
 
   // Batch trends endpoint
@@ -232,7 +232,7 @@ export const api = {
           total_allocated_bytes: number;
         }>
       >;
-    }>('/api/trends-batch', {
+    }>('/trends-batch', {
       method: 'POST',
       body: JSON.stringify({
         trend_queries: trendQueries.map((query) => ({
@@ -247,7 +247,7 @@ export const api = {
 
   // Flamegraph endpoint
   getFlamegraph: (id: string) =>
-    fetchApi<{ flamegraph_html: string }>(`/api/flamegraph/${id}`),
+    fetchApi<{ flamegraph_html: string }>(`/flamegraph/${id}`),
 
   // Token management endpoints
   getTokens: () =>