@sentry/junior 0.15.2 → 0.17.0

package/dist/{chunk-KTBQH6L5.js → chunk-4XWTSMRF.js}

@@ -5,7 +5,7 @@ import {
   getPluginSkillRoots,
   logInfo,
   logWarn
-} from "./chunk-5JHLDXBN.js";
+} from "./chunk-DTOS5CG4.js";
 import {
   skillRoots
 } from "./chunk-RBB2MZAN.js";

package/dist/{chunk-5JHLDXBN.js → chunk-DTOS5CG4.js}

@@ -47,7 +47,7 @@ var LEGACY_KEY_MAP = {
   steps: "app.ai.steps",
   toolCalls: "app.ai.tool_calls",
   toolResults: "app.ai.tool_results",
-  finishReason: "app.ai.finish_reason",
+  finishReason: "gen_ai.response.finish_reasons",
   sources: "app.ai.sources",
   generatedFiles: "app.ai.generated_files",
   resultFiles: "app.ai.result_files",
@@ -301,7 +301,9 @@ function mergeAttributes(...maps) {
     if (!map) continue;
     for (const [rawKey, rawValue] of Object.entries(map)) {
       const key = normalizeAttributeKey(rawKey);
-      const value = sanitizeValue(rawValue);
+      const value = sanitizeValue(
+        key === "gen_ai.response.finish_reasons" && typeof rawValue === "string" && rawValue.trim() ? [rawValue] : rawValue
+      );
       if (value !== void 0) {
         merged[key] = value;
       }
@@ -1623,6 +1625,12 @@ function createGitHubAppBroker(manifest, credentials) {
   function leaseDomainsFor(capability) {
     return GIT_CAPABILITIES.has(capability) ? [...apiDomains, GIT_DOMAIN] : apiDomains;
   }
+  function authorizationFor(domain, token) {
+    if (domain === GIT_DOMAIN) {
+      return `Basic ${Buffer.from(`x-access-token:${token}`).toString("base64")}`;
+    }
+    return `Bearer ${token}`;
+  }
   const supportedCapabilities = new Set(manifest.capabilities);
   return {
     async issue(input) {
@@ -1659,7 +1667,7 @@ function createGitHubAppBroker(manifest, credentials) {
             domain,
             headers: {
               ...apiHeaders ?? {},
-              Authorization: `Bearer ${cached.token}`
+              Authorization: authorizationFor(domain, cached.token)
             }
           })),
           expiresAt: new Date(cached.expiresAt).toISOString(),
@@ -1703,7 +1711,7 @@ function createGitHubAppBroker(manifest, credentials) {
           domain,
           headers: {
             ...apiHeaders ?? {},
-            Authorization: `Bearer ${accessTokenResponse.token}`
+            Authorization: authorizationFor(domain, accessTokenResponse.token)
           }
         })),
         expiresAt: new Date(expiresAtMs).toISOString(),
@@ -2288,6 +2296,7 @@ export {
   withSpan,
   setSpanAttributes,
   setSpanStatus,
+  getActiveTraceId,
   resolveErrorReference,
   serializeGenAiAttribute,
   extractGenAiUsageAttributes,

package/dist/{chunk-ESPIOJPM.js → chunk-XYOKYK6U.js}

@@ -2,7 +2,7 @@ import {
   getPluginRuntimeDependencies,
   getPluginRuntimePostinstall,
   withSpan
-} from "./chunk-5JHLDXBN.js";
+} from "./chunk-DTOS5CG4.js";
 
 // src/chat/state/adapter.ts
 import { createMemoryState } from "@chat-adapter/state-memory";

package/dist/cli/check.js

@@ -1,9 +1,9 @@
 import {
   parseSkillFile
-} from "../chunk-KTBQH6L5.js";
+} from "../chunk-4XWTSMRF.js";
 import {
   parsePluginManifest
-} from "../chunk-5JHLDXBN.js";
+} from "../chunk-DTOS5CG4.js";
 import "../chunk-Z3YD6NHK.js";
 import "../chunk-RBB2MZAN.js";
 import "../chunk-2KG3PWR4.js";

package/dist/cli/init.js

@@ -27,7 +27,7 @@ export default defineConfig({
   preset: "vercel",
   modules: [juniorNitro()],
   routes: {
-    "/api/**": { handler: "./server.ts" },
+    "/**": { handler: "./server.ts" },
   },
 });
 `

package/dist/cli/snapshot-warmup.js

@@ -1,12 +1,12 @@
 import {
   disconnectStateAdapter,
   resolveRuntimeDependencySnapshot
-} from "../chunk-ESPIOJPM.js";
+} from "../chunk-XYOKYK6U.js";
 import {
   getPluginProviders,
   getPluginRuntimeDependencies,
   getPluginRuntimePostinstall
-} from "../chunk-5JHLDXBN.js";
+} from "../chunk-DTOS5CG4.js";
 import "../chunk-Z3YD6NHK.js";
 import "../chunk-RBB2MZAN.js";
 import "../chunk-2KG3PWR4.js";

package/dist/nitro.js

@@ -4,56 +4,62 @@ import {
 import "./chunk-2KG3PWR4.js";
 
 // src/nitro.ts
-import { cpSync, existsSync, mkdirSync, readdirSync } from "fs";
+import path3 from "path";
+
+// src/build/rolldown-workarounds.ts
+function applyRolldownTreeshakeWorkaround(nitro) {
+  nitro.options.rolldownConfig = {
+    ...nitro.options.rolldownConfig,
+    treeshake: {
+      ...nitro.options.rolldownConfig?.treeshake,
+      moduleSideEffects: true
+    }
+  };
+}
+
+// src/build/copy-build-content.ts
+import { cpSync, existsSync as existsSync2, mkdirSync, readdirSync } from "fs";
+import path2 from "path";
+
+// src/build/glob-to-regex.ts
+function globToRegex(pattern) {
+  const escaped = pattern.replace(/[\\^$+?.()|[\]{}]/g, "\\$&").replace(/\*/g, ".*");
+  return new RegExp(`^${escaped}$`);
+}
+
+// src/build/resolve-package.ts
+import { existsSync } from "fs";
 import path from "path";
 import { fileURLToPath } from "url";
-function juniorNitro(options = {}) {
-  return {
-    nitro: {
-      setup(nitro) {
-        const cwd = path.resolve(
-          options.cwd ?? nitro.options.rootDir ?? process.cwd()
-        );
-        nitro.options.vercel ??= {};
-        nitro.options.vercel.functions ??= {};
-        nitro.options.vercel.functions.maxDuration ??= options.maxDuration ?? 800;
-        nitro.options.rolldownConfig = {
-          ...nitro.options.rolldownConfig,
-          treeshake: {
-            ...nitro.options.rolldownConfig?.treeshake,
-            moduleSideEffects: true
-          }
-        };
-        nitro.options.virtual["#junior/config"] = `export const pluginPackages = ${JSON.stringify(options.pluginPackages ?? [])};`;
-        nitro.hooks.hook("compiled", () => {
-          copyAppAndPluginContent(
-            cwd,
-            nitro.options.output.serverDir,
-            options.pluginPackages
-          );
-          copyIncludedFiles(
-            nitro.options.output.serverDir,
-            options.includeFiles
-          );
-        });
-      }
+function resolvePackageDir(pkgName) {
+  try {
+    const resolved = import.meta.resolve(pkgName);
+    const entry = resolved.startsWith("file://") ? fileURLToPath(resolved) : resolved;
+    let dir = path.dirname(entry);
+    while (dir !== path.dirname(dir)) {
+      if (existsSync(path.join(dir, "package.json"))) return dir;
+      dir = path.dirname(dir);
     }
-  };
+  } catch {
+  }
+  return void 0;
 }
+
+// src/build/copy-build-content.ts
 function copyAppAndPluginContent(cwd, serverRoot, pluginPackages) {
-  copyIfExists(path.join(cwd, "app"), path.join(serverRoot, "app"));
+  copyIfExists(path2.join(cwd, "app"), path2.join(serverRoot, "app"));
   const packagedContent = discoverInstalledPluginPackageContent(cwd, {
     packageNames: pluginPackages
   });
   for (const root of packagedContent.manifestRoots) {
-    if (existsSync(path.join(root, "plugin.yaml"))) {
-      const relative = path.relative(cwd, root);
-      if (!relative || path.isAbsolute(relative) || relative.startsWith("..")) {
+    if (existsSync2(path2.join(root, "plugin.yaml"))) {
+      const relative = path2.relative(cwd, root);
+      if (!relative || path2.isAbsolute(relative) || relative.startsWith("..")) {
         continue;
       }
       copyIfExists(
-        path.join(root, "plugin.yaml"),
-        path.join(serverRoot, relative, "plugin.yaml")
+        path2.join(root, "plugin.yaml"),
+        path2.join(serverRoot, relative, "plugin.yaml")
       );
       continue;
     }
@@ -63,19 +69,6 @@ function copyAppAndPluginContent(cwd, serverRoot, pluginPackages) {
     copyRootIntoServerOutput(cwd, serverRoot, root);
   }
 }
-function resolvePackageDir(pkgName) {
-  try {
-    const resolved = import.meta.resolve(pkgName);
-    const entry = resolved.startsWith("file://") ? fileURLToPath(resolved) : resolved;
-    let dir = path.dirname(entry);
-    while (dir !== path.dirname(dir)) {
-      if (existsSync(path.join(dir, "package.json"))) return dir;
-      dir = path.dirname(dir);
-    }
-  } catch {
-  }
-  return void 0;
-}
 function copyIncludedFiles(serverRoot, patterns) {
   if (!patterns?.length) return;
   for (const pattern of patterns) {
@@ -83,38 +76,70 @@ function copyIncludedFiles(serverRoot, patterns) {
     const parts = normalized.split("/");
     const pkgName = parts[0].startsWith("@") ? `${parts[0]}/${parts[1]}` : parts[0];
     const subpath = parts.slice(pkgName.includes("/") ? 2 : 1).join("/");
-    const fileGlob = path.basename(subpath);
-    const subDir = path.dirname(subpath);
+    const fileGlob = path2.basename(subpath);
+    const subDir = path2.dirname(subpath);
     const pkgDir = resolvePackageDir(pkgName);
     if (!pkgDir) continue;
-    const sourceDir = path.join(pkgDir, subDir);
-    if (!existsSync(sourceDir)) continue;
+    const sourceDir = path2.join(pkgDir, subDir);
+    if (!existsSync2(sourceDir)) continue;
     const entries = readdirSync(sourceDir);
-    const re = fileGlob.includes("*") ? new RegExp(
-      `^${fileGlob.replace(/[\\^$+?.()|[\]{}]/g, "\\$&").replace(/\*/g, ".*")}$`
-    ) : null;
+    const re = fileGlob.includes("*") ? globToRegex(fileGlob) : null;
     for (const entry of entries) {
       if (re ? !re.test(entry) : entry !== fileGlob) continue;
       copyIfExists(
-        path.join(sourceDir, entry),
-        path.join(serverRoot, "node_modules", pkgName, subDir, entry)
+        path2.join(sourceDir, entry),
+        path2.join(serverRoot, "node_modules", pkgName, subDir, entry)
       );
     }
   }
 }
 function copyIfExists(source, target) {
-  if (!existsSync(source)) {
+  if (!existsSync2(source)) {
     return;
   }
-  mkdirSync(path.dirname(target), { recursive: true });
+  mkdirSync(path2.dirname(target), { recursive: true });
   cpSync(source, target, { recursive: true });
 }
 function copyRootIntoServerOutput(cwd, serverRoot, root) {
-  const relative = path.relative(cwd, root);
-  if (!relative || path.isAbsolute(relative) || relative.startsWith("..")) {
+  const relative = path2.relative(cwd, root);
+  if (!relative || path2.isAbsolute(relative) || relative.startsWith("..")) {
     return;
   }
-  copyIfExists(root, path.join(serverRoot, relative));
+  copyIfExists(root, path2.join(serverRoot, relative));
+}
+
+// src/build/virtual-config.ts
+function injectVirtualConfig(nitro, pluginPackages) {
+  nitro.options.virtual["#junior/config"] = `export const pluginPackages = ${JSON.stringify(pluginPackages)};`;
+}
+
+// src/nitro.ts
+function juniorNitro(options = {}) {
+  return {
+    nitro: {
+      setup(nitro) {
+        const cwd = path3.resolve(
+          options.cwd ?? nitro.options.rootDir ?? process.cwd()
+        );
+        nitro.options.vercel ??= {};
+        nitro.options.vercel.functions ??= {};
+        nitro.options.vercel.functions.maxDuration ??= options.maxDuration ?? 800;
+        applyRolldownTreeshakeWorkaround(nitro);
+        injectVirtualConfig(nitro, options.pluginPackages ?? []);
+        nitro.hooks.hook("compiled", () => {
+          copyAppAndPluginContent(
+            cwd,
+            nitro.options.output.serverDir,
+            options.pluginPackages
+          );
+          copyIncludedFiles(
+            nitro.options.output.serverDir,
+            options.includeFiles
+          );
+        });
+      }
+    }
+  };
 }
 export {
   juniorNitro

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sentry/junior",
-  "version": "0.15.2",
+  "version": "0.17.0",
   "private": false,
   "publishConfig": {
     "access": "public"
@@ -46,8 +46,10 @@
   "devDependencies": {
     "@sentry/node": "^10.46.0",
     "@types/node": "^25.5.0",
+    "dependency-cruiser": "^17.3.10",
     "msw": "^2.12.14",
     "nitro": "3.0.260311-beta",
+    "oxlint": "^1.58.0",
     "tsup": "^8.5.1",
     "typescript": "^5.9.3",
     "vercel": "^50.37.3",
@@ -55,9 +57,12 @@
   },
   "scripts": {
     "build": "tsup",
-    "test": "pnpm run test:slack-boundary && vitest run",
+    "lint": "oxlint --config .oxlintrc.json --quiet --import-plugin --node-plugin --vitest-plugin src tests scripts bin tsup.config.ts",
+    "lint:fix": "oxlint --config .oxlintrc.json --quiet --import-plugin --node-plugin --vitest-plugin --fix src tests scripts bin tsup.config.ts",
+    "test": "pnpm run test:slack-boundary && pnpm run test:arch-boundary && vitest run",
     "test:watch": "vitest",
     "test:slack-boundary": "node scripts/check-slack-test-boundary.mjs",
+    "test:arch-boundary": "depcruise --config .dependency-cruiser.mjs src/chat",
     "typecheck": "tsc --noEmit",
     "skills:check": "node scripts/check-skills.mjs"
   }