Merge pull request #17807 from getsentry/prepare-release/10.16.0

meta(changelog): Update changelog for 10.16.0

Author: s1gr1d

.cursor/commands/publish_release.md

@@ -0,0 +1,5 @@
+# Release Command
+
+Execute the standard Sentry JavaScript SDK release process.
+
+Find the "publishing_release" rule in `.cursor/rules/publishing_release` and follow those complete instructions step by step.

CHANGELOG.md

@@ -4,6 +4,31 @@
 
 - "You miss 100 percent of the chances you don't take. — Wayne Gretzky" — Michael Scott
 
+## 10.16.0
+
+- feat(logs): Add internal `replay_is_buffering` flag ([#17752](https://github.com/getsentry/sentry-javascript/pull/17752))
+- feat(react-router): Update loadContext type to be compatible with middleware ([#17758](https://github.com/getsentry/sentry-javascript/pull/17758))
+- feat(replay/logs): Only attach sampled replay Ids to logs ([#17750](https://github.com/getsentry/sentry-javascript/pull/17750))
+- fix(browser): Use current start timestamp for CLS span when CLS is 0 ([#17800](https://github.com/getsentry/sentry-javascript/pull/17800))
+- fix(core): Prevent `instrumentAnthropicAiClient` breaking MessageStream api ([#17754](https://github.com/getsentry/sentry-javascript/pull/17754))
+- fix(nextjs): Don't use chalk in turbopack config file ([#17806](https://github.com/getsentry/sentry-javascript/pull/17806))
+- fix(react): Do not send additional navigation span on pageload ([#17799](https://github.com/getsentry/sentry-javascript/pull/17799))
+
+<details>
+  <summary> <strong>Internal Changes</strong> </summary>
+
+- build(aws): Ensure AWS build cache does not keep old files ([#17776](https://github.com/getsentry/sentry-javascript/pull/17776))
+- chore: Add `publish_release` command ([#17797](https://github.com/getsentry/sentry-javascript/pull/17797))
+- ref(aws-serverless): Add resolution for `import-in-the-middle` when building the Lambda layer ([#17780](https://github.com/getsentry/sentry-javascript/pull/17780))
+- ref(aws-serverless): Improve README with better examples ([#17787](https://github.com/getsentry/sentry-javascript/pull/17787))
+- ref(core): Improve promise buffer ([#17788](https://github.com/getsentry/sentry-javascript/pull/17788))
+- Revert "test(e2e): Pin `import-in-the-middle@1.14.2` due to `@vercel/nft` incompatibility ([#17777](https://github.com/getsentry/sentry-javascript/pull/17777))" (#17784)
+- test(e2e): Pin `import-in-the-middle@1.14.2` due to `@vercel/nft` incompatibility ([#17777](https://github.com/getsentry/sentry-javascript/pull/17777))
+- test(nextjs): Add route handler tests for turbopack ([#17515](https://github.com/getsentry/sentry-javascript/pull/17515))
+- test(react-router): Test v8 middleware ([#17783](https://github.com/getsentry/sentry-javascript/pull/17783))
+
+</details>
+
 ## 10.15.0
 
 ### Important Changes

dev-packages/e2e-tests/test-applications/nextjs-turbo/app/route-handlers/[param]/error/route.ts

@@ -0,0 +1,3 @@
+export async function GET(request: Request) {
+  throw new Error('Dynamic route handler error');
+}

dev-packages/e2e-tests/test-applications/nextjs-turbo/app/route-handlers/[param]/route.ts

@@ -0,0 +1,9 @@
+import { NextResponse } from 'next/server';
+
+export async function GET() {
+  return NextResponse.json({ name: 'Beep' });
+}
+
+export async function POST() {
+  return NextResponse.json({ name: 'Boop' }, { status: 400 });
+}

dev-packages/e2e-tests/test-applications/nextjs-turbo/app/route-handlers/static/route.ts

@@ -0,0 +1,5 @@
+import { NextResponse } from 'next/server';
+
+export async function GET(request: Request) {
+  return NextResponse.json({ name: 'Static' });
+}

dev-packages/e2e-tests/test-applications/nextjs-turbo/package.json

@@ -17,7 +17,7 @@
     "@types/node": "^18.19.1",
     "@types/react": "^19",
     "@types/react-dom": "^19",
-    "next": "^15.3.5",
+    "next": "^15.5.4",
     "react": "^19",
     "react-dom": "^19",
     "typescript": "~5.0.0"

dev-packages/e2e-tests/test-applications/nextjs-turbo/tests/app-router/route-handlers.test.ts

@@ -0,0 +1,73 @@
+import { expect, test } from '@playwright/test';
+import { waitForError, waitForTransaction } from '@sentry-internal/test-utils';
+
+test('Should create a transaction for dynamic route handlers', async ({ request }) => {
+  const routehandlerTransactionPromise = waitForTransaction('nextjs-turbo', async transactionEvent => {
+    return transactionEvent?.transaction === 'GET /route-handlers/[param]';
+  });
+
+  const response = await request.get('/route-handlers/foo');
+  expect(await response.json()).toStrictEqual({ name: 'Beep' });
+
+  const routehandlerTransaction = await routehandlerTransactionPromise;
+
+  expect(routehandlerTransaction.contexts?.trace?.status).toBe('ok');
+  expect(routehandlerTransaction.contexts?.trace?.op).toBe('http.server');
+});
+
+test('Should create a transaction for static route handlers', async ({ request }) => {
+  const routehandlerTransactionPromise = waitForTransaction('nextjs-turbo', async transactionEvent => {
+    return transactionEvent?.transaction === 'GET /route-handlers/static';
+  });
+
+  const response = await request.get('/route-handlers/static');
+  expect(await response.json()).toStrictEqual({ name: 'Static' });
+
+  const routehandlerTransaction = await routehandlerTransactionPromise;
+
+  expect(routehandlerTransaction.contexts?.trace?.status).toBe('ok');
+  expect(routehandlerTransaction.contexts?.trace?.op).toBe('http.server');
+});
+
+test('Should create a transaction for route handlers and correctly set span status depending on http status', async ({
+  request,
+}) => {
+  const routehandlerTransactionPromise = waitForTransaction('nextjs-turbo', async transactionEvent => {
+    return transactionEvent?.transaction === 'POST /route-handlers/[param]';
+  });
+
+  const response = await request.post('/route-handlers/bar');
+  expect(await response.json()).toStrictEqual({ name: 'Boop' });
+
+  const routehandlerTransaction = await routehandlerTransactionPromise;
+
+  expect(routehandlerTransaction.contexts?.trace?.status).toBe('invalid_argument');
+  expect(routehandlerTransaction.contexts?.trace?.op).toBe('http.server');
+});
+
+test('Should record exceptions and transactions for faulty route handlers', async ({ request }) => {
+  const errorEventPromise = waitForError('nextjs-turbo', errorEvent => {
+    return errorEvent?.exception?.values?.[0]?.value === 'Dynamic route handler error';
+  });
+
+  const routehandlerTransactionPromise = waitForTransaction('nextjs-turbo', async transactionEvent => {
+    return transactionEvent?.transaction === 'GET /route-handlers/[param]/error';
+  });
+
+  await request.get('/route-handlers/boop/error').catch(() => {});
+
+  const routehandlerTransaction = await routehandlerTransactionPromise;
+  const routehandlerError = await errorEventPromise;
+
+  expect(routehandlerTransaction.contexts?.trace?.status).toBe('internal_error');
+  expect(routehandlerTransaction.contexts?.trace?.op).toBe('http.server');
+  expect(routehandlerTransaction.contexts?.trace?.origin).toContain('auto');
+
+  expect(routehandlerError.exception?.values?.[0].value).toBe('Dynamic route handler error');
+
+  expect(routehandlerError.request?.method).toBe('GET');
+  // todo: make sure url is attached to request object
+  // expect(routehandlerError.request?.url).toContain('/route-handlers/boop/error');
+
+  expect(routehandlerError.transaction).toBe('/route-handlers/[param]/error');
+});

dev-packages/e2e-tests/test-applications/react-router-7-framework/app/context.ts

@@ -0,0 +1,8 @@
+import { createContext } from 'react-router';
+
+export type User = {
+  id: string;
+  name: string;
+};
+
+export const userContext = createContext<User | null>(null);

dev-packages/e2e-tests/test-applications/react-router-7-framework/app/routes.ts

@@ -17,5 +17,6 @@ export default [
     route('static', 'routes/performance/static.tsx'),
     route('server-loader', 'routes/performance/server-loader.tsx'),
     route('server-action', 'routes/performance/server-action.tsx'),
+    route('with-middleware', 'routes/performance/with-middleware.tsx'),
   ]),
 ] satisfies RouteConfig;

dev-packages/e2e-tests/test-applications/react-router-7-framework/app/routes/performance/with-middleware.tsx

@@ -0,0 +1,38 @@
+import type { Route } from './+types/with-middleware';
+import type { User } from '../../context';
+import { userContext } from '../../context';
+import * as Sentry from '@sentry/react-router';
+
+async function getUser() {
+  await new Promise(resolve => setTimeout(resolve, 500));
+  return {
+    id: '1',
+    name: 'Carlos Gomez',
+  };
+}
+
+const authMiddleware: Route.MiddlewareFunction = async ({ request, context }, next) => {
+  Sentry.startSpan({ name: 'authMiddleware', op: 'middleware.auth' }, async () => {
+    const user: User = await getUser();
+    context.set(userContext, user);
+    await next();
+  });
+};
+
+export const middleware: Route.MiddlewareFunction[] = [authMiddleware];
+
+export const loader = async ({ context }: Route.LoaderArgs) => {
+  const user = context.get(userContext);
+  return { user };
+};
+
+export default function WithMiddlewarePage({ loaderData }: Route.ComponentProps) {
+  const { user } = loaderData;
+
+  return (
+    <div>
+      <h1>With Middleware Page</h1>
+      <p>User: {user?.name}</p>
+    </div>
+  );
+}