@sentry/junior-dashboard 0.77.0 → 0.78.0

package/dist/url.d.ts

@@ -5,7 +5,7 @@ export interface DashboardConversationURLConfig extends DashboardBaseURLConfig {
     basePath?: string;
     conversationId: string;
 }
-/** Normalize dashboard route prefixes so plugin and Nitro paths agree. */
+/** Normalize dashboard route prefixes for host routes and external links. */
 export declare function normalizeDashboardPath(path: string | undefined, fallback: string): string;
 /** Resolve the dashboard origin used for browser auth and external links. */
 export declare function resolveDashboardBaseURL(config?: DashboardBaseURLConfig): string;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry/junior-dashboard",
-  "version": "0.77.0",
+  "version": "0.78.0",
   "private": false,
   "publishConfig": {
     "access": "public"
@@ -15,14 +15,6 @@
     ".": {
       "types": "./dist/index.d.ts",
       "default": "./dist/index.js"
-    },
-    "./nitro": {
-      "types": "./dist/nitro.d.ts",
-      "default": "./dist/nitro.js"
-    },
-    "./handler": {
-      "types": "./dist/handler.d.ts",
-      "default": "./dist/handler.js"
     }
   },
   "files": [
@@ -41,8 +33,8 @@
     "react-router": "^7.16.0",
     "recharts": "^3.8.1",
     "shiki": "4.1.0",
-    "@sentry/junior-plugin-api": "0.77.0",
-    "@sentry/junior": "0.77.0"
+    "@sentry/junior-plugin-api": "0.78.0",
+    "@sentry/junior": "0.78.0"
   },
   "devDependencies": {
     "@tailwindcss/cli": "^4.3.0",

package/src/app.ts

@@ -38,6 +38,17 @@ export interface JuniorDashboardOptions {
   mockConversations?: boolean;
 }
 
+interface DashboardRuntimeOptions extends JuniorDashboardOptions {
+  pluginRoutes?: DashboardPluginRoute[];
+}
+
+interface DashboardPluginRoute {
+  app: {
+    fetch(request: Request): Promise<Response> | Response;
+  };
+  pluginName: string;
+}
+
 type Variables = {
   dashboardSession: DashboardSession;
 };
@@ -70,6 +81,66 @@ function normalizeValues(values: string[] | undefined): string[] {
   ];
 }
 
+/** Read dashboard list env vars as comma-separated strings or JSON arrays. */
+function readEnvList(name: string): string[] | undefined {
+  const value = process.env[name];
+  if (!value?.trim()) {
+    return undefined;
+  }
+
+  if (value.trim().startsWith("[")) {
+    let parsed: unknown;
+    try {
+      parsed = JSON.parse(value);
+    } catch (error) {
+      throw new Error(`${name} must be a JSON string array`, {
+        cause: error,
+      });
+    }
+    if (
+      !Array.isArray(parsed) ||
+      parsed.some((item) => typeof item !== "string")
+    ) {
+      throw new Error(`${name} must be a JSON string array`);
+    }
+    return parsed;
+  }
+
+  return value
+    .split(",")
+    .map((item) => item.trim())
+    .filter(Boolean);
+}
+
+/** Read dashboard boolean env vars; only explicit true/false values apply. */
+function readEnvFlag(name: string): boolean | undefined {
+  const value = process.env[name]?.trim();
+  if (!value) {
+    return undefined;
+  }
+  return value === "true" ? true : value === "false" ? false : undefined;
+}
+
+function resolveDashboardOptions(
+  options: DashboardRuntimeOptions,
+): DashboardRuntimeOptions {
+  return {
+    ...options,
+    authRequired:
+      options.authRequired ?? readEnvFlag("JUNIOR_DASHBOARD_AUTH_REQUIRED"),
+    allowedGoogleDomains:
+      options.allowedGoogleDomains ??
+      readEnvList("JUNIOR_DASHBOARD_GOOGLE_DOMAINS"),
+    allowedEmails:
+      options.allowedEmails ?? readEnvList("JUNIOR_DASHBOARD_ALLOWED_EMAILS"),
+    trustedOrigins:
+      options.trustedOrigins ?? readEnvList("JUNIOR_DASHBOARD_TRUSTED_ORIGINS"),
+    mockConversations:
+      options.mockConversations ??
+      readEnvFlag("JUNIOR_DASHBOARD_MOCK_CONVERSATIONS"),
+  };
+}
+
 function isJsonRoute(pathname: string): boolean {
   return pathname.startsWith("/api/");
 }
@@ -397,10 +468,26 @@ function renderFavicon(): Response {
   );
 }
 
+function pluginRoutePrefix(pluginName: string): string {
+  return `/api/dashboard/plugins/${pluginName}`;
+}
+
+/** Strip the core-owned plugin prefix before dispatching to a plugin app. */
+function pluginRouteRequest(request: Request, prefix: string): Request {
+  const url = new URL(request.url);
+  const pathname = url.pathname;
+  const nextPath =
+    pathname === prefix ? "/" : pathname.slice(prefix.length) || "/";
+  url.pathname = nextPath.startsWith("/") ? nextPath : `/${nextPath}`;
+  return new Request(url, request);
+}
+
 /** Create the authenticated dashboard Hono app mounted by Nitro. */
 export function createDashboardApp(
-  options: JuniorDashboardOptions,
+  rawOptions: DashboardRuntimeOptions,
 ): Hono<{ Variables: Variables }> {
+  const options = resolveDashboardOptions(rawOptions);
+
   if (process.env.SENTRY_DSN?.trim()) {
     initSentry();
   }
@@ -518,6 +605,13 @@ export function createDashboardApp(
   app.get("/api/dashboard/sessions", async () => {
     return Response.json(await reporting.getSessions());
   });
+  for (const route of options.pluginRoutes ?? []) {
+    const prefix = pluginRoutePrefix(route.pluginName);
+    const handler = (c: Context<{ Variables: Variables }>) =>
+      route.app.fetch(pluginRouteRequest(c.req.raw, prefix));
+    app.all(prefix, handler);
+    app.all(`${prefix}/*`, handler);
+  }
   app.get("/api/dashboard/conversation-stats", async () => {
     try {
       return Response.json(await readConversationStats(reporting));

package/src/index.ts

@@ -1,83 +1 @@
-import {
-  type PluginRoute,
-  defineJuniorPlugin,
-  type PluginRegistration,
-} from "@sentry/junior-plugin-api";
-import { buildDashboardConversationURL, normalizeDashboardPath } from "./url";
-import { createDashboardApp, type JuniorDashboardOptions } from "./app";
-
 export { createDashboardApp, type JuniorDashboardOptions } from "./app";
-
-export interface JuniorDashboardPluginOptions extends JuniorDashboardOptions {
-  disabled?: boolean;
-}
-
-function dashboardRoutePaths(options: JuniorDashboardPluginOptions): string[] {
-  const basePath = normalizeDashboardPath(options.basePath, "/");
-  const authPath = normalizeDashboardPath(options.authPath, "/api/auth");
-  const pagePaths =
-    basePath === "/"
-      ? [
-          "/",
-          "/conversations",
-          "/conversations/*",
-          "/plugins",
-          "/sessions",
-          "/sessions/*",
-        ]
-      : [basePath, `${basePath}/*`];
-
-  return [
-    ...pagePaths,
-    "/favicon.ico",
-    "/api/dashboard/*",
-    authPath,
-    `${authPath}/*`,
-  ];
-}
-
-function dashboardRoutes(options: JuniorDashboardPluginOptions): PluginRoute[] {
-  let app: ReturnType<typeof createDashboardApp> | undefined;
-  const fetch = (request: Request) => {
-    app ??= createDashboardApp(options);
-    return app.fetch(request);
-  };
-
-  return dashboardRoutePaths(options).map((path) => ({
-    handler: fetch,
-    path,
-  }));
-}
-
-/** Register dashboard routes and Slack footer links through plugin hooks. */
-export function juniorDashboardPlugin(
-  options: JuniorDashboardPluginOptions = {},
-): PluginRegistration {
-  return defineJuniorPlugin({
-    manifest: {
-      name: "dashboard",
-      displayName: "Dashboard",
-      description: "Junior dashboard routes and Slack footer links",
-    },
-    hooks: {
-      routes() {
-        if (options.disabled) {
-          return [];
-        }
-        return dashboardRoutes(options);
-      },
-      slackConversationLink(ctx) {
-        if (options.disabled) {
-          return undefined;
-        }
-        return {
-          url: buildDashboardConversationURL({
-            basePath: options.basePath,
-            baseURL: options.baseURL,
-            conversationId: ctx.conversationId,
-          }),
-        };
-      },
-    },
-  });
-}

package/src/url.ts

@@ -19,7 +19,7 @@ function stripTrailingSlashes(value: string): string {
   return end === value.length ? value : value.slice(0, end);
 }
 
-/** Normalize dashboard route prefixes so plugin and Nitro paths agree. */
+/** Normalize dashboard route prefixes for host routes and external links. */
 export function normalizeDashboardPath(
   path: string | undefined,
   fallback: string,

package/dist/config.d.ts

@@ -1,4 +0,0 @@
-import type { JuniorDashboardOptions } from "./app";
-export type JuniorDashboardRuntimeConfig = Omit<JuniorDashboardOptions, "auth" | "reporting">;
-/** Read dashboard runtime config injected by the Nitro module. */
-export declare function resolveDashboardConfig(): Promise<JuniorDashboardRuntimeConfig>;

package/dist/handler.d.ts

@@ -1,2 +0,0 @@
-declare const handler: unknown;
-export default handler;