diff --git a/src/__tests__/openapi-route-coverage-guard.test.ts b/src/__tests__/openapi-route-coverage-guard.test.ts index bafc8d560..6df4b68ff 100644 --- a/src/__tests__/openapi-route-coverage-guard.test.ts +++ b/src/__tests__/openapi-route-coverage-guard.test.ts @@ -339,7 +339,6 @@ const UNPUBLISHED: Readonly> = { "/api/internal/web-vitals": { kind: "internalOps", methods: ["POST"] }, "/api/monitoring/csp-report": { kind: "internalOps", methods: ["POST"] }, "/api/monitoring/glitchtip": { kind: "internalOps", methods: ["POST"] }, - "/api/monitoring/settings": { kind: "internalOps", methods: ["GET"] }, "/api/monitoring/umami-script": { kind: "internalOps", methods: ["GET"] }, "/api/monitoring/umami/{path}": { kind: "internalOps", methods: ["POST"] }, "/api/mcp/oauth/authorize": { diff --git a/src/app/api/monitoring/settings/route.ts b/src/app/api/monitoring/settings/route.ts deleted file mode 100644 index bf03de6ef..000000000 --- a/src/app/api/monitoring/settings/route.ts +++ /dev/null @@ -1,17 +0,0 @@ -import { apiHandler } from "@/lib/api-handler"; -import { annotate } from "@/lib/logging/context"; -import { apiSuccess } from "@/lib/api-response"; -import { getPublicMonitoringSettings } from "@/lib/monitoring-settings"; - -export const dynamic = "force-dynamic"; - -export const GET = apiHandler(async () => { - annotate({ action: { name: "monitoring.settings.get" } }); - - const settings = await getPublicMonitoringSettings(); - return apiSuccess({ - umamiEnabled: settings.umamiEnabled, - umamiWebsiteId: settings.umamiWebsiteId, - glitchtipEnabled: settings.glitchtipEnabled, - }); -}); diff --git a/src/components/monitoring/__tests__/bootstrap.test.tsx b/src/components/monitoring/__tests__/bootstrap.test.tsx index e0a8c8fdb..d62d4f23e 100644 --- a/src/components/monitoring/__tests__/bootstrap.test.tsx +++ b/src/components/monitoring/__tests__/bootstrap.test.tsx @@ -3,8 +3,8 @@ import { renderToStaticMarkup } from "react-dom/server"; /** * The monitoring switches come from the root layout's render, not from a - * request every page load sent to `/api/monitoring/settings` for three - * instance-wide values. The first render already mounts what they turn on. + * request on every page load for three instance-wide values. The first + * render already mounts what they turn on. */ const apiGet = vi.fn(); diff --git a/src/components/monitoring/bootstrap.tsx b/src/components/monitoring/bootstrap.tsx index 1f6c790ae..4f38aa5fc 100644 --- a/src/components/monitoring/bootstrap.tsx +++ b/src/components/monitoring/bootstrap.tsx @@ -5,8 +5,8 @@ import type { PublicMonitoringSettings } from "@/lib/monitoring-settings"; /** * Mounts the operator's analytics script and error reporter. The root layout * reads the instance-wide switches on the server and passes them in, so no - * page load asks `/api/monitoring/settings` for three values the document - * render already had. `null` (the key-mismatch page) mounts nothing. + * page load asks for three values the document render already has. `null` + * (the key-mismatch page) mounts nothing. */ export function MonitoringBootstrap({ settings, diff --git a/src/lib/query-keys/admin.ts b/src/lib/query-keys/admin.ts index c5157762e..691df7dff 100644 --- a/src/lib/query-keys/admin.ts +++ b/src/lib/query-keys/admin.ts @@ -65,7 +65,6 @@ export const adminKeys = { adminRecoveryKit: () => ["admin", "recovery-kit"] as const, /** Every backup signal's verdict (admin overview + Backups). */ adminRecoveryReadiness: () => ["admin", "recovery-readiness"] as const, - adminCoachFeedback: () => ["admin", "coach-feedback"] as const, adminFeedback: (status: string) => ["admin", "feedback", status] as const, adminFeedbackRoot: () => ["admin", "feedback"] as const, adminHostMetrics: (window: string) =>