@sentry/wizard 6.6.1 → 6.7.0

package/dist/test/react-router/codemods/root.test.js

@@ -0,0 +1,178 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const vitest_1 = require("vitest");
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+const root_1 = require("../../../src/react-router/codemods/root");
+vitest_1.vi.mock('@clack/prompts', () => {
+    const mock = {
+        log: {
+            warn: vitest_1.vi.fn(),
+            info: vitest_1.vi.fn(),
+            success: vitest_1.vi.fn(),
+        },
+    };
+    return {
+        default: mock,
+        ...mock,
+    };
+});
+vitest_1.vi.mock('../../../src/utils/debug', () => ({
+    debug: vitest_1.vi.fn(),
+}));
+(0, vitest_1.describe)('instrumentRoot', () => {
+    const fixturesDir = path.join(__dirname, 'fixtures', 'root');
+    let tmpDir;
+    let appDir;
+    (0, vitest_1.beforeEach)(() => {
+        vitest_1.vi.clearAllMocks();
+        // Create unique tmp directory for each test
+        tmpDir = path.join(fixturesDir, 'tmp', `test-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`);
+        appDir = path.join(tmpDir, 'app');
+        // Ensure tmp and app directories exist
+        fs.mkdirSync(appDir, { recursive: true });
+        // Mock process.cwd() to return the tmp directory
+        vitest_1.vi.spyOn(process, 'cwd').mockReturnValue(tmpDir);
+    });
+    (0, vitest_1.afterEach)(() => {
+        // Clean up tmp directory
+        if (fs.existsSync(tmpDir)) {
+            fs.rmSync(tmpDir, { recursive: true });
+        }
+        vitest_1.vi.restoreAllMocks();
+    });
+    (0, vitest_1.it)('should add ErrorBoundary when no ErrorBoundary exists and no Sentry content', async () => {
+        // Copy fixture to tmp directory for testing
+        const srcFile = path.join(fixturesDir, 'no-error-boundary.tsx');
+        // Create app directory and copy file
+        fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));
+        // Mock process.cwd() to return tmpDir
+        await (0, root_1.instrumentRoot)('root.tsx');
+        // Check that the file was modified correctly
+        const modifiedContent = fs.readFileSync(path.join(appDir, 'root.tsx'), 'utf8');
+        (0, vitest_1.expect)(modifiedContent).toContain('import * as Sentry from "@sentry/react-router";');
+        (0, vitest_1.expect)(modifiedContent).toContain("import { Outlet, isRouteErrorResponse } from 'react-router';");
+        (0, vitest_1.expect)(modifiedContent).toContain('export function ErrorBoundary({ error })');
+        (0, vitest_1.expect)(modifiedContent).toContain('Sentry.captureException(error);');
+        (0, vitest_1.expect)(modifiedContent).toContain('if (isRouteErrorResponse(error))');
+    });
+    (0, vitest_1.it)('should add Sentry.captureException to existing function declaration ErrorBoundary', async () => {
+        const srcFile = path.join(fixturesDir, 'with-function-error-boundary.tsx');
+        fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));
+        await (0, root_1.instrumentRoot)('root.tsx');
+        const modifiedContent = fs.readFileSync(path.join(appDir, 'root.tsx'), 'utf8');
+        (0, vitest_1.expect)(modifiedContent).toContain('import * as Sentry from "@sentry/react-router";');
+        (0, vitest_1.expect)(modifiedContent).toContain('Sentry.captureException(error);');
+    });
+    (0, vitest_1.it)('should add Sentry.captureException to existing variable declaration ErrorBoundary', async () => {
+        const srcFile = path.join(fixturesDir, 'with-variable-error-boundary.tsx');
+        fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));
+        await (0, root_1.instrumentRoot)('root.tsx');
+        const modifiedContent = fs.readFileSync(path.join(appDir, 'root.tsx'), 'utf8');
+        (0, vitest_1.expect)(modifiedContent).toContain('import * as Sentry from "@sentry/react-router";');
+        // Now properly handles variable declaration ErrorBoundary
+        (0, vitest_1.expect)(modifiedContent).toContain('Sentry.captureException(error);');
+    });
+    (0, vitest_1.it)('should not modify file when ErrorBoundary already has Sentry.captureException', async () => {
+        const srcFile = path.join(fixturesDir, 'with-sentry-error-boundary.tsx');
+        fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));
+        await (0, root_1.instrumentRoot)('root.tsx');
+        const modifiedContent = fs.readFileSync(path.join(appDir, 'root.tsx'), 'utf8');
+        // Should not add duplicate Sentry.captureException
+        const captureExceptionOccurrences = (modifiedContent.match(/Sentry\.captureException/g) || []).length;
+        (0, vitest_1.expect)(captureExceptionOccurrences).toBe(1);
+    });
+    (0, vitest_1.it)('should not add Sentry import when Sentry content already exists', async () => {
+        const srcFile = path.join(fixturesDir, 'with-existing-sentry.tsx');
+        fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));
+        await (0, root_1.instrumentRoot)('root.tsx');
+        const modifiedContent = fs.readFileSync(path.join(appDir, 'root.tsx'), 'utf8');
+        // Should not duplicate Sentry imports
+        const sentryImportOccurrences = (modifiedContent.match(/import.*@sentry\/react-router/g) || []).length;
+        (0, vitest_1.expect)(sentryImportOccurrences).toBe(1);
+    });
+    (0, vitest_1.it)('should add isRouteErrorResponse import when not present and ErrorBoundary is added', async () => {
+        const srcFile = path.join(fixturesDir, 'no-isrouteerrorresponse.tsx');
+        fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));
+        await (0, root_1.instrumentRoot)('root.tsx');
+        const modifiedContent = fs.readFileSync(path.join(appDir, 'root.tsx'), 'utf8');
+        (0, vitest_1.expect)(modifiedContent).toContain("import { Outlet, isRouteErrorResponse } from 'react-router';");
+        (0, vitest_1.expect)(modifiedContent).toContain('export function ErrorBoundary({ error })');
+    });
+    (0, vitest_1.it)('should not add duplicate isRouteErrorResponse import when already present', async () => {
+        const srcFile = path.join(fixturesDir, 'with-isrouteerrorresponse.tsx');
+        fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));
+        await (0, root_1.instrumentRoot)('root.tsx');
+        const modifiedContent = fs.readFileSync(path.join(appDir, 'root.tsx'), 'utf8');
+        // Should not duplicate isRouteErrorResponse imports
+        const isRouteErrorResponseOccurrences = (modifiedContent.match(/isRouteErrorResponse/g) || []).length;
+        (0, vitest_1.expect)(isRouteErrorResponseOccurrences).toBe(2); // One import, one usage in template
+    });
+    (0, vitest_1.it)('should handle ErrorBoundary with alternative function declaration syntax', async () => {
+        const srcFile = path.join(fixturesDir, 'function-expression-error-boundary.tsx');
+        fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));
+        await (0, root_1.instrumentRoot)('root.tsx');
+        const modifiedContent = fs.readFileSync(path.join(appDir, 'root.tsx'), 'utf8');
+        (0, vitest_1.expect)(modifiedContent).toContain('import * as Sentry from "@sentry/react-router";');
+        (0, vitest_1.expect)(modifiedContent).toContain('Sentry.captureException(error);');
+    });
+    (0, vitest_1.it)('should handle function declaration with separate export', async () => {
+        const srcFile = path.join(fixturesDir, 'function-declaration-separate-export.tsx');
+        fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));
+        await (0, root_1.instrumentRoot)('root.tsx');
+        const modifiedContent = fs.readFileSync(path.join(appDir, 'root.tsx'), 'utf8');
+        (0, vitest_1.expect)(modifiedContent).toContain('import * as Sentry from "@sentry/react-router";');
+        (0, vitest_1.expect)(modifiedContent).toContain('Sentry.captureException(error);');
+        // Should preserve function declaration syntax
+        (0, vitest_1.expect)(modifiedContent).toMatch(/function ErrorBoundary\(/);
+        (0, vitest_1.expect)(modifiedContent).toContain('export { ErrorBoundary }');
+    });
+    (0, vitest_1.it)('should handle ErrorBoundary with captureException imported directly', async () => {
+        const srcFile = path.join(fixturesDir, 'with-direct-capture-exception.tsx');
+        fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));
+        await (0, root_1.instrumentRoot)('root.tsx');
+        const modifiedContent = fs.readFileSync(path.join(appDir, 'root.tsx'), 'utf8');
+        // Should not add duplicate captureException calls
+        const captureExceptionOccurrences = (modifiedContent.match(/captureException/g) || []).length;
+        (0, vitest_1.expect)(captureExceptionOccurrences).toBe(2); // One import, one usage
+    });
+    (0, vitest_1.it)('should not modify an already properly configured file', async () => {
+        const srcFile = path.join(fixturesDir, 'fully-configured.tsx');
+        fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));
+        await (0, root_1.instrumentRoot)('root.tsx');
+        const modifiedContent = fs.readFileSync(path.join(appDir, 'root.tsx'), 'utf8');
+        // Should not add duplicate imports or modify existing Sentry configuration
+        const sentryImportOccurrences = (modifiedContent.match(/import.*@sentry\/react-router/g) || []).length;
+        (0, vitest_1.expect)(sentryImportOccurrences).toBe(1);
+        const captureExceptionOccurrences = (modifiedContent.match(/Sentry\.captureException/g) || []).length;
+        (0, vitest_1.expect)(captureExceptionOccurrences).toBe(1);
+        const errorBoundaryOccurrences = (modifiedContent.match(/export function ErrorBoundary/g) || []).length;
+        (0, vitest_1.expect)(errorBoundaryOccurrences).toBe(1);
+        (0, vitest_1.expect)(modifiedContent).toContain("import * as Sentry from '@sentry/react-router';");
+        (0, vitest_1.expect)(modifiedContent).toContain("import { Outlet, isRouteErrorResponse } from 'react-router';");
+    });
+});
+//# sourceMappingURL=root.test.js.map

package/dist/test/react-router/codemods/root.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"root.test.js","sourceRoot":"","sources":["../../../../test/react-router/codemods/root.test.ts"],"names":[],"mappings":";;;;;;;;;;;;;;;;;;;;;;;;;AAAA,mCAAyE;AACzE,uCAAyB;AACzB,2CAA6B;AAC7B,kEAAyE;AAEzE,WAAE,CAAC,IAAI,CAAC,gBAAgB,EAAE,GAAG,EAAE;IAC7B,MAAM,IAAI,GAAG;QACX,GAAG,EAAE;YACH,IAAI,EAAE,WAAE,CAAC,EAAE,EAAE;YACb,IAAI,EAAE,WAAE,CAAC,EAAE,EAAE;YACb,OAAO,EAAE,WAAE,CAAC,EAAE,EAAE;SACjB;KACF,CAAC;IACF,OAAO;QACL,OAAO,EAAE,IAAI;QACb,GAAG,IAAI;KACR,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,WAAE,CAAC,IAAI,CAAC,0BAA0B,EAAE,GAAG,EAAE,CAAC,CAAC;IACzC,KAAK,EAAE,WAAE,CAAC,EAAE,EAAE;CACf,CAAC,CAAC,CAAC;AAEJ,IAAA,iBAAQ,EAAC,gBAAgB,EAAE,GAAG,EAAE;IAC9B,MAAM,WAAW,GAAG,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,UAAU,EAAE,MAAM,CAAC,CAAC;IAC7D,IAAI,MAAc,CAAC;IACnB,IAAI,MAAc,CAAC;IAEnB,IAAA,mBAAU,EAAC,GAAG,EAAE;QACd,WAAE,CAAC,aAAa,EAAE,CAAC;QAEnB,4CAA4C;QAC5C,MAAM,GAAG,IAAI,CAAC,IAAI,CAChB,WAAW,EACX,KAAK,EACL,QAAQ,IAAI,CAAC,GAAG,EAAE,IAAI,IAAI,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,CAChE,CAAC;QACF,MAAM,GAAG,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,KAAK,CAAC,CAAC;QAElC,uCAAuC;QACvC,EAAE,CAAC,SAAS,CAAC,MAAM,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAE1C,iDAAiD;QACjD,WAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,eAAe,CAAC,MAAM,CAAC,CAAC;IACnD,CAAC,CAAC,CAAC;IAEH,IAAA,kBAAS,EAAC,GAAG,EAAE;QACb,yBAAyB;QACzB,IAAI,EAAE,CAAC,UAAU,CAAC,MAAM,CAAC,EAAE;YACzB,EAAE,CAAC,MAAM,CAAC,MAAM,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;SACxC;QACD,WAAE,CAAC,eAAe,EAAE,CAAC;IACvB,CAAC,CAAC,CAAC;IAEH,IAAA,WAAE,EAAC,6EAA6E,EAAE,KAAK,IAAI,EAAE;QAC3F,4CAA4C;QAC5C,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,uBAAuB,CAAC,CAAC;QAEhE,qCAAqC;QACrC,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC;QAExD,sCAAsC;QAEtC,MAAM,IAAA,qBAAc,EAAC,UAAU,CAAC,CAAC;QAEjC,6CAA6C;QAC7C,MAAM,eAAe,GAAG,EAAE,CAAC,YAAY,CACrC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,MAAM,CACP,CAAC;QAEF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAC/B,iDAAiD,CAClD,CAAC;QACF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAC/B,8DAA8D,CAC/D,CAAC;QACF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAC/B,0CAA0C,CAC3C,CAAC;QACF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAAC,iCAAiC,CAAC,CAAC;QACrE,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAAC,kCAAkC,CAAC,CAAC;IACxE,CAAC,CAAC,CAAC;IAEH,IAAA,WAAE,EAAC,mFAAmF,EAAE,KAAK,IAAI,EAAE;QACjG,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,kCAAkC,CAAC,CAAC;QAE3E,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC;QAExD,MAAM,IAAA,qBAAc,EAAC,UAAU,CAAC,CAAC;QAEjC,MAAM,eAAe,GAAG,EAAE,CAAC,YAAY,CACrC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,MAAM,CACP,CAAC;QAEF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAC/B,iDAAiD,CAClD,CAAC;QACF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAAC,iCAAiC,CAAC,CAAC;IACvE,CAAC,CAAC,CAAC;IAEH,IAAA,WAAE,EAAC,mFAAmF,EAAE,KAAK,IAAI,EAAE;QACjG,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,kCAAkC,CAAC,CAAC;QAE3E,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC;QAExD,MAAM,IAAA,qBAAc,EAAC,UAAU,CAAC,CAAC;QAEjC,MAAM,eAAe,GAAG,EAAE,CAAC,YAAY,CACrC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,MAAM,CACP,CAAC;QAEF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAC/B,iDAAiD,CAClD,CAAC;QACF,0DAA0D;QAC1D,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAAC,iCAAiC,CAAC,CAAC;IACvE,CAAC,CAAC,CAAC;IAEH,IAAA,WAAE,EAAC,+EAA+E,EAAE,KAAK,IAAI,EAAE;QAC7F,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,gCAAgC,CAAC,CAAC;QAEzE,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC;QAExD,MAAM,IAAA,qBAAc,EAAC,UAAU,CAAC,CAAC;QAEjC,MAAM,eAAe,GAAG,EAAE,CAAC,YAAY,CACrC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,MAAM,CACP,CAAC;QAEF,mDAAmD;QACnD,MAAM,2BAA2B,GAAG,CAClC,eAAe,CAAC,KAAK,CAAC,2BAA2B,CAAC,IAAI,EAAE,CACzD,CAAC,MAAM,CAAC;QACT,IAAA,eAAM,EAAC,2BAA2B,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAC9C,CAAC,CAAC,CAAC;IAEH,IAAA,WAAE,EAAC,iEAAiE,EAAE,KAAK,IAAI,EAAE;QAC/E,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,0BAA0B,CAAC,CAAC;QAEnE,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC;QAExD,MAAM,IAAA,qBAAc,EAAC,UAAU,CAAC,CAAC;QAEjC,MAAM,eAAe,GAAG,EAAE,CAAC,YAAY,CACrC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,MAAM,CACP,CAAC;QAEF,sCAAsC;QACtC,MAAM,uBAAuB,GAAG,CAC9B,eAAe,CAAC,KAAK,CAAC,gCAAgC,CAAC,IAAI,EAAE,CAC9D,CAAC,MAAM,CAAC;QACT,IAAA,eAAM,EAAC,uBAAuB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,IAAA,WAAE,EAAC,oFAAoF,EAAE,KAAK,IAAI,EAAE;QAClG,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,6BAA6B,CAAC,CAAC;QAEtE,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC;QAExD,MAAM,IAAA,qBAAc,EAAC,UAAU,CAAC,CAAC;QAEjC,MAAM,eAAe,GAAG,EAAE,CAAC,YAAY,CACrC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,MAAM,CACP,CAAC;QAEF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAC/B,8DAA8D,CAC/D,CAAC;QACF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAC/B,0CAA0C,CAC3C,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,IAAA,WAAE,EAAC,2EAA2E,EAAE,KAAK,IAAI,EAAE;QACzF,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,+BAA+B,CAAC,CAAC;QAExE,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC;QAExD,MAAM,IAAA,qBAAc,EAAC,UAAU,CAAC,CAAC;QAEjC,MAAM,eAAe,GAAG,EAAE,CAAC,YAAY,CACrC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,MAAM,CACP,CAAC;QAEF,oDAAoD;QACpD,MAAM,+BAA+B,GAAG,CACtC,eAAe,CAAC,KAAK,CAAC,uBAAuB,CAAC,IAAI,EAAE,CACrD,CAAC,MAAM,CAAC;QACT,IAAA,eAAM,EAAC,+BAA+B,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,oCAAoC;IACvF,CAAC,CAAC,CAAC;IAEH,IAAA,WAAE,EAAC,0EAA0E,EAAE,KAAK,IAAI,EAAE;QACxF,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CACvB,WAAW,EACX,wCAAwC,CACzC,CAAC;QAEF,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC;QAExD,MAAM,IAAA,qBAAc,EAAC,UAAU,CAAC,CAAC;QAEjC,MAAM,eAAe,GAAG,EAAE,CAAC,YAAY,CACrC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,MAAM,CACP,CAAC;QAEF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAC/B,iDAAiD,CAClD,CAAC;QACF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAAC,iCAAiC,CAAC,CAAC;IACvE,CAAC,CAAC,CAAC;IAEH,IAAA,WAAE,EAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;QACvE,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CACvB,WAAW,EACX,0CAA0C,CAC3C,CAAC;QAEF,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC;QAExD,MAAM,IAAA,qBAAc,EAAC,UAAU,CAAC,CAAC;QAEjC,MAAM,eAAe,GAAG,EAAE,CAAC,YAAY,CACrC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,MAAM,CACP,CAAC;QAEF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAC/B,iDAAiD,CAClD,CAAC;QACF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAAC,iCAAiC,CAAC,CAAC;QAErE,8CAA8C;QAC9C,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,OAAO,CAAC,0BAA0B,CAAC,CAAC;QAC5D,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAAC,0BAA0B,CAAC,CAAC;IAChE,CAAC,CAAC,CAAC;IAEH,IAAA,WAAE,EAAC,qEAAqE,EAAE,KAAK,IAAI,EAAE;QACnF,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,mCAAmC,CAAC,CAAC;QAE5E,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC;QAExD,MAAM,IAAA,qBAAc,EAAC,UAAU,CAAC,CAAC;QAEjC,MAAM,eAAe,GAAG,EAAE,CAAC,YAAY,CACrC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,MAAM,CACP,CAAC;QAEF,kDAAkD;QAClD,MAAM,2BAA2B,GAAG,CAClC,eAAe,CAAC,KAAK,CAAC,mBAAmB,CAAC,IAAI,EAAE,CACjD,CAAC,MAAM,CAAC;QACT,IAAA,eAAM,EAAC,2BAA2B,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,wBAAwB;IACvE,CAAC,CAAC,CAAC;IAEH,IAAA,WAAE,EAAC,uDAAuD,EAAE,KAAK,IAAI,EAAE;QACrE,MAAM,OAAO,GAAG,IAAI,CAAC,IAAI,CAAC,WAAW,EAAE,sBAAsB,CAAC,CAAC;QAE/D,EAAE,CAAC,YAAY,CAAC,OAAO,EAAE,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,CAAC,CAAC;QAExD,MAAM,IAAA,qBAAc,EAAC,UAAU,CAAC,CAAC;QAEjC,MAAM,eAAe,GAAG,EAAE,CAAC,YAAY,CACrC,IAAI,CAAC,IAAI,CAAC,MAAM,EAAE,UAAU,CAAC,EAC7B,MAAM,CACP,CAAC;QAEF,2EAA2E;QAC3E,MAAM,uBAAuB,GAAG,CAC9B,eAAe,CAAC,KAAK,CAAC,gCAAgC,CAAC,IAAI,EAAE,CAC9D,CAAC,MAAM,CAAC;QACT,IAAA,eAAM,EAAC,uBAAuB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAExC,MAAM,2BAA2B,GAAG,CAClC,eAAe,CAAC,KAAK,CAAC,2BAA2B,CAAC,IAAI,EAAE,CACzD,CAAC,MAAM,CAAC;QACT,IAAA,eAAM,EAAC,2BAA2B,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAE5C,MAAM,wBAAwB,GAAG,CAC/B,eAAe,CAAC,KAAK,CAAC,gCAAgC,CAAC,IAAI,EAAE,CAC9D,CAAC,MAAM,CAAC;QACT,IAAA,eAAM,EAAC,wBAAwB,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;QAEzC,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAC/B,iDAAiD,CAClD,CAAC;QACF,IAAA,eAAM,EAAC,eAAe,CAAC,CAAC,SAAS,CAC/B,8DAA8D,CAC/D,CAAC;IACJ,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC","sourcesContent":["import { describe, expect, it, vi, beforeEach, afterEach } from 'vitest';\nimport * as fs from 'fs';\nimport * as path from 'path';\nimport { instrumentRoot } from '../../../src/react-router/codemods/root';\n\nvi.mock('@clack/prompts', () => {\n  const mock = {\n    log: {\n      warn: vi.fn(),\n      info: vi.fn(),\n      success: vi.fn(),\n    },\n  };\n  return {\n    default: mock,\n    ...mock,\n  };\n});\n\nvi.mock('../../../src/utils/debug', () => ({\n  debug: vi.fn(),\n}));\n\ndescribe('instrumentRoot', () => {\n  const fixturesDir = path.join(__dirname, 'fixtures', 'root');\n  let tmpDir: string;\n  let appDir: string;\n\n  beforeEach(() => {\n    vi.clearAllMocks();\n\n    // Create unique tmp directory for each test\n    tmpDir = path.join(\n      fixturesDir,\n      'tmp',\n      `test-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`,\n    );\n    appDir = path.join(tmpDir, 'app');\n\n    // Ensure tmp and app directories exist\n    fs.mkdirSync(appDir, { recursive: true });\n\n    // Mock process.cwd() to return the tmp directory\n    vi.spyOn(process, 'cwd').mockReturnValue(tmpDir);\n  });\n\n  afterEach(() => {\n    // Clean up tmp directory\n    if (fs.existsSync(tmpDir)) {\n      fs.rmSync(tmpDir, { recursive: true });\n    }\n    vi.restoreAllMocks();\n  });\n\n  it('should add ErrorBoundary when no ErrorBoundary exists and no Sentry content', async () => {\n    // Copy fixture to tmp directory for testing\n    const srcFile = path.join(fixturesDir, 'no-error-boundary.tsx');\n\n    // Create app directory and copy file\n    fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));\n\n    // Mock process.cwd() to return tmpDir\n\n    await instrumentRoot('root.tsx');\n\n    // Check that the file was modified correctly\n    const modifiedContent = fs.readFileSync(\n      path.join(appDir, 'root.tsx'),\n      'utf8',\n    );\n\n    expect(modifiedContent).toContain(\n      'import * as Sentry from \"@sentry/react-router\";',\n    );\n    expect(modifiedContent).toContain(\n      \"import { Outlet, isRouteErrorResponse } from 'react-router';\",\n    );\n    expect(modifiedContent).toContain(\n      'export function ErrorBoundary({ error })',\n    );\n    expect(modifiedContent).toContain('Sentry.captureException(error);');\n    expect(modifiedContent).toContain('if (isRouteErrorResponse(error))');\n  });\n\n  it('should add Sentry.captureException to existing function declaration ErrorBoundary', async () => {\n    const srcFile = path.join(fixturesDir, 'with-function-error-boundary.tsx');\n\n    fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));\n\n    await instrumentRoot('root.tsx');\n\n    const modifiedContent = fs.readFileSync(\n      path.join(appDir, 'root.tsx'),\n      'utf8',\n    );\n\n    expect(modifiedContent).toContain(\n      'import * as Sentry from \"@sentry/react-router\";',\n    );\n    expect(modifiedContent).toContain('Sentry.captureException(error);');\n  });\n\n  it('should add Sentry.captureException to existing variable declaration ErrorBoundary', async () => {\n    const srcFile = path.join(fixturesDir, 'with-variable-error-boundary.tsx');\n\n    fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));\n\n    await instrumentRoot('root.tsx');\n\n    const modifiedContent = fs.readFileSync(\n      path.join(appDir, 'root.tsx'),\n      'utf8',\n    );\n\n    expect(modifiedContent).toContain(\n      'import * as Sentry from \"@sentry/react-router\";',\n    );\n    // Now properly handles variable declaration ErrorBoundary\n    expect(modifiedContent).toContain('Sentry.captureException(error);');\n  });\n\n  it('should not modify file when ErrorBoundary already has Sentry.captureException', async () => {\n    const srcFile = path.join(fixturesDir, 'with-sentry-error-boundary.tsx');\n\n    fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));\n\n    await instrumentRoot('root.tsx');\n\n    const modifiedContent = fs.readFileSync(\n      path.join(appDir, 'root.tsx'),\n      'utf8',\n    );\n\n    // Should not add duplicate Sentry.captureException\n    const captureExceptionOccurrences = (\n      modifiedContent.match(/Sentry\\.captureException/g) || []\n    ).length;\n    expect(captureExceptionOccurrences).toBe(1);\n  });\n\n  it('should not add Sentry import when Sentry content already exists', async () => {\n    const srcFile = path.join(fixturesDir, 'with-existing-sentry.tsx');\n\n    fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));\n\n    await instrumentRoot('root.tsx');\n\n    const modifiedContent = fs.readFileSync(\n      path.join(appDir, 'root.tsx'),\n      'utf8',\n    );\n\n    // Should not duplicate Sentry imports\n    const sentryImportOccurrences = (\n      modifiedContent.match(/import.*@sentry\\/react-router/g) || []\n    ).length;\n    expect(sentryImportOccurrences).toBe(1);\n  });\n\n  it('should add isRouteErrorResponse import when not present and ErrorBoundary is added', async () => {\n    const srcFile = path.join(fixturesDir, 'no-isrouteerrorresponse.tsx');\n\n    fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));\n\n    await instrumentRoot('root.tsx');\n\n    const modifiedContent = fs.readFileSync(\n      path.join(appDir, 'root.tsx'),\n      'utf8',\n    );\n\n    expect(modifiedContent).toContain(\n      \"import { Outlet, isRouteErrorResponse } from 'react-router';\",\n    );\n    expect(modifiedContent).toContain(\n      'export function ErrorBoundary({ error })',\n    );\n  });\n\n  it('should not add duplicate isRouteErrorResponse import when already present', async () => {\n    const srcFile = path.join(fixturesDir, 'with-isrouteerrorresponse.tsx');\n\n    fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));\n\n    await instrumentRoot('root.tsx');\n\n    const modifiedContent = fs.readFileSync(\n      path.join(appDir, 'root.tsx'),\n      'utf8',\n    );\n\n    // Should not duplicate isRouteErrorResponse imports\n    const isRouteErrorResponseOccurrences = (\n      modifiedContent.match(/isRouteErrorResponse/g) || []\n    ).length;\n    expect(isRouteErrorResponseOccurrences).toBe(2); // One import, one usage in template\n  });\n\n  it('should handle ErrorBoundary with alternative function declaration syntax', async () => {\n    const srcFile = path.join(\n      fixturesDir,\n      'function-expression-error-boundary.tsx',\n    );\n\n    fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));\n\n    await instrumentRoot('root.tsx');\n\n    const modifiedContent = fs.readFileSync(\n      path.join(appDir, 'root.tsx'),\n      'utf8',\n    );\n\n    expect(modifiedContent).toContain(\n      'import * as Sentry from \"@sentry/react-router\";',\n    );\n    expect(modifiedContent).toContain('Sentry.captureException(error);');\n  });\n\n  it('should handle function declaration with separate export', async () => {\n    const srcFile = path.join(\n      fixturesDir,\n      'function-declaration-separate-export.tsx',\n    );\n\n    fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));\n\n    await instrumentRoot('root.tsx');\n\n    const modifiedContent = fs.readFileSync(\n      path.join(appDir, 'root.tsx'),\n      'utf8',\n    );\n\n    expect(modifiedContent).toContain(\n      'import * as Sentry from \"@sentry/react-router\";',\n    );\n    expect(modifiedContent).toContain('Sentry.captureException(error);');\n\n    // Should preserve function declaration syntax\n    expect(modifiedContent).toMatch(/function ErrorBoundary\\(/);\n    expect(modifiedContent).toContain('export { ErrorBoundary }');\n  });\n\n  it('should handle ErrorBoundary with captureException imported directly', async () => {\n    const srcFile = path.join(fixturesDir, 'with-direct-capture-exception.tsx');\n\n    fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));\n\n    await instrumentRoot('root.tsx');\n\n    const modifiedContent = fs.readFileSync(\n      path.join(appDir, 'root.tsx'),\n      'utf8',\n    );\n\n    // Should not add duplicate captureException calls\n    const captureExceptionOccurrences = (\n      modifiedContent.match(/captureException/g) || []\n    ).length;\n    expect(captureExceptionOccurrences).toBe(2); // One import, one usage\n  });\n\n  it('should not modify an already properly configured file', async () => {\n    const srcFile = path.join(fixturesDir, 'fully-configured.tsx');\n\n    fs.copyFileSync(srcFile, path.join(appDir, 'root.tsx'));\n\n    await instrumentRoot('root.tsx');\n\n    const modifiedContent = fs.readFileSync(\n      path.join(appDir, 'root.tsx'),\n      'utf8',\n    );\n\n    // Should not add duplicate imports or modify existing Sentry configuration\n    const sentryImportOccurrences = (\n      modifiedContent.match(/import.*@sentry\\/react-router/g) || []\n    ).length;\n    expect(sentryImportOccurrences).toBe(1);\n\n    const captureExceptionOccurrences = (\n      modifiedContent.match(/Sentry\\.captureException/g) || []\n    ).length;\n    expect(captureExceptionOccurrences).toBe(1);\n\n    const errorBoundaryOccurrences = (\n      modifiedContent.match(/export function ErrorBoundary/g) || []\n    ).length;\n    expect(errorBoundaryOccurrences).toBe(1);\n\n    expect(modifiedContent).toContain(\n      \"import * as Sentry from '@sentry/react-router';\",\n    );\n    expect(modifiedContent).toContain(\n      \"import { Outlet, isRouteErrorResponse } from 'react-router';\",\n    );\n  });\n});\n"]}

package/dist/test/react-router/codemods/server-entry.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/test/react-router/codemods/server-entry.test.js

@@ -0,0 +1,415 @@
+"use strict";
+/* eslint-disable @typescript-eslint/no-unsafe-assignment */
+/* eslint-disable @typescript-eslint/no-unsafe-member-access */
+/* eslint-disable @typescript-eslint/no-explicit-any */
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || function (mod) {
+    if (mod && mod.__esModule) return mod;
+    var result = {};
+    if (mod != null) for (var k in mod) if (k !== "default" && Object.prototype.hasOwnProperty.call(mod, k)) __createBinding(result, mod, k);
+    __setModuleDefault(result, mod);
+    return result;
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+const vitest_1 = require("vitest");
+const fs = __importStar(require("fs"));
+const path = __importStar(require("path"));
+const recast = __importStar(require("recast"));
+const server_entry_1 = require("../../../src/react-router/codemods/server-entry");
+// @ts-expect-error - magicast is ESM and TS complains about that. It works though
+const magicast_1 = require("magicast");
+vitest_1.vi.mock('@clack/prompts', () => {
+    const mock = {
+        log: {
+            warn: vitest_1.vi.fn(),
+            info: vitest_1.vi.fn(),
+            success: vitest_1.vi.fn(),
+        },
+    };
+    return {
+        default: mock,
+        ...mock,
+    };
+});
+vitest_1.vi.mock('../../../src/utils/debug', () => ({
+    debug: vitest_1.vi.fn(),
+}));
+(0, vitest_1.describe)('instrumentServerEntry', () => {
+    const fixturesDir = path.join(__dirname, 'fixtures', 'server-entry');
+    let tmpDir;
+    let tmpFile;
+    (0, vitest_1.beforeEach)(() => {
+        vitest_1.vi.clearAllMocks();
+        // Create unique tmp directory for each test
+        tmpDir = path.join(__dirname, 'fixtures', 'tmp', `test-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`);
+        tmpFile = path.join(tmpDir, 'entry.server.tsx');
+        // Ensure tmp directory exists
+        fs.mkdirSync(tmpDir, { recursive: true });
+    });
+    (0, vitest_1.afterEach)(() => {
+        // Clean up tmp directory
+        if (fs.existsSync(tmpDir)) {
+            fs.rmSync(tmpDir, { recursive: true });
+        }
+    });
+    (0, vitest_1.it)('should add Sentry import and wrap handleRequest function', async () => {
+        const basicContent = fs.readFileSync(path.join(fixturesDir, 'basic.tsx'), 'utf8');
+        fs.writeFileSync(tmpFile, basicContent);
+        await (0, server_entry_1.instrumentServerEntry)(tmpFile);
+        const modifiedContent = fs.readFileSync(tmpFile, 'utf8');
+        // Should add Sentry import
+        (0, vitest_1.expect)(modifiedContent).toContain('import * as Sentry from "@sentry/react-router";');
+        // Should wrap the existing handleRequest function
+        (0, vitest_1.expect)(modifiedContent).toContain('export default Sentry.wrapSentryHandleRequest(handleRequest);');
+        // Should add the Sentry import at the top of the file (after existing imports)
+        const lines = modifiedContent.split('\n');
+        const sentryImportLine = lines.findIndex((line) => line.includes('import * as Sentry from "@sentry/react-router";'));
+        (0, vitest_1.expect)(sentryImportLine).toBeGreaterThanOrEqual(0);
+        // Should create default handleError since none exists
+        (0, vitest_1.expect)(modifiedContent).toContain('export const handleError = Sentry.createSentryHandleError({');
+        (0, vitest_1.expect)(modifiedContent).toContain('logErrors: false');
+    });
+    (0, vitest_1.it)('should handle already instrumented server entry without duplication', async () => {
+        const alreadyInstrumentedContent = fs.readFileSync(path.join(fixturesDir, 'already-instrumented.tsx'), 'utf8');
+        fs.writeFileSync(tmpFile, alreadyInstrumentedContent);
+        await (0, server_entry_1.instrumentServerEntry)(tmpFile);
+        const modifiedContent = fs.readFileSync(tmpFile, 'utf8');
+        // Should not add duplicate imports or wrapping since already instrumented
+        (0, vitest_1.expect)(modifiedContent).toContain("import * as Sentry from '@sentry/react-router';");
+        (0, vitest_1.expect)(modifiedContent).toContain('export default Sentry.wrapSentryHandleRequest(handleRequest);');
+        // Should NOT add a new createSentryHandleError export since handleError already has captureException
+        (0, vitest_1.expect)(modifiedContent).not.toContain('export const handleError = Sentry.createSentryHandleError({');
+        // Should preserve the existing handleError function with captureException
+        (0, vitest_1.expect)(modifiedContent).toContain('Sentry.captureException(error);');
+        (0, vitest_1.expect)(modifiedContent).toContain('export async function handleError');
+    });
+    (0, vitest_1.it)('should handle variable export pattern with existing export', async () => {
+        const variableExportContent = fs.readFileSync(path.join(fixturesDir, 'variable-export.tsx'), 'utf8');
+        fs.writeFileSync(tmpFile, variableExportContent);
+        await (0, server_entry_1.instrumentServerEntry)(tmpFile);
+        const modifiedContent = fs.readFileSync(tmpFile, 'utf8');
+        // Should add Sentry import and wrap handleRequest
+        (0, vitest_1.expect)(modifiedContent).toContain('import * as Sentry from "@sentry/react-router";');
+        (0, vitest_1.expect)(modifiedContent).toContain('export default Sentry.wrapSentryHandleRequest(handleRequest);');
+        // Should instrument the existing handleError variable with captureException
+        (0, vitest_1.expect)(modifiedContent).toContain('Sentry.captureException(error);');
+        // Should preserve the variable export pattern
+        (0, vitest_1.expect)(modifiedContent).toContain('export const handleError');
+    });
+});
+(0, vitest_1.describe)('instrumentHandleRequest', () => {
+    let tmpDir;
+    (0, vitest_1.beforeEach)(() => {
+        tmpDir = path.join(__dirname, 'fixtures', 'tmp', `handle-request-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`);
+        fs.mkdirSync(tmpDir, { recursive: true });
+    });
+    (0, vitest_1.afterEach)(() => {
+        if (fs.existsSync(tmpDir)) {
+            fs.rmSync(tmpDir, { recursive: true });
+        }
+    });
+    (0, vitest_1.it)('should add required imports when creating new handleRequest', async () => {
+        const content = `// Empty server entry file`;
+        const tempFile = path.join(tmpDir, 'entry.server.tsx');
+        fs.writeFileSync(tempFile, content);
+        const mod = await (0, magicast_1.loadFile)(tempFile);
+        (0, server_entry_1.instrumentHandleRequest)(mod);
+        // Check if required imports were added
+        const imports = mod.imports.$items;
+        const hasServerRouter = imports.some((item) => item.imported === 'ServerRouter' && item.from === 'react-router');
+        const hasRenderToPipeableStream = imports.some((item) => item.imported === 'renderToPipeableStream' &&
+            item.from === 'react-dom/server');
+        (0, vitest_1.expect)(hasServerRouter).toBe(true);
+        (0, vitest_1.expect)(hasRenderToPipeableStream).toBe(true);
+    });
+    (0, vitest_1.it)('should not duplicate imports if they already exist', async () => {
+        const content = `
+import { ServerRouter } from 'react-router';
+import { renderToPipeableStream } from 'react-dom/server';
+import { createReadableStreamFromReadable } from '@react-router/node';
+`;
+        const tempFile = path.join(tmpDir, 'entry.server.tsx');
+        fs.writeFileSync(tempFile, content);
+        const mod = await (0, magicast_1.loadFile)(tempFile);
+        const originalImportsCount = mod.imports.$items.length;
+        (0, server_entry_1.instrumentHandleRequest)(mod);
+        // Should not add duplicate imports
+        (0, vitest_1.expect)(mod.imports.$items.length).toBe(originalImportsCount);
+    });
+});
+(0, vitest_1.describe)('instrumentHandleError', () => {
+    let tmpDir;
+    (0, vitest_1.beforeEach)(() => {
+        tmpDir = path.join(__dirname, 'fixtures', 'tmp', `handle-error-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`);
+        fs.mkdirSync(tmpDir, { recursive: true });
+    });
+    (0, vitest_1.afterEach)(() => {
+        if (fs.existsSync(tmpDir)) {
+            fs.rmSync(tmpDir, { recursive: true });
+        }
+    });
+    (0, vitest_1.it)('should not modify existing handleError with captureException', async () => {
+        const content = `
+export function handleError(error: unknown) {
+  Sentry.captureException(error);
+  console.error(error);
+}
+`;
+        const tempFile = path.join(tmpDir, 'entry.server.tsx');
+        fs.writeFileSync(tempFile, content);
+        const mod = await (0, magicast_1.loadFile)(tempFile);
+        const originalBodyLength = mod.$ast.body.length;
+        (0, server_entry_1.instrumentHandleError)(mod);
+        // Should not modify since captureException already exists
+        (0, vitest_1.expect)(mod.$ast.body.length).toBe(originalBodyLength);
+    });
+    (0, vitest_1.it)('should not modify existing handleError with createSentryHandleError', async () => {
+        const content = `
+export const handleError = Sentry.createSentryHandleError({
+  logErrors: false
+});
+`;
+        const tempFile = path.join(tmpDir, 'entry.server.tsx');
+        fs.writeFileSync(tempFile, content);
+        const mod = await (0, magicast_1.loadFile)(tempFile);
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+        const originalBodyLength = mod.$ast.body.length;
+        (0, server_entry_1.instrumentHandleError)(mod);
+        // Should not modify since createSentryHandleError already exists
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any
+        (0, vitest_1.expect)(mod.$ast.body.length).toBe(originalBodyLength);
+    });
+    (0, vitest_1.it)('should add captureException to existing handleError function declaration without breaking AST', async () => {
+        const content = `
+export function handleError(error: unknown) {
+  console.error('Custom error handling:', error);
+  // some other logic here
+}
+`;
+        const tempFile = path.join(tmpDir, 'entry.server.tsx');
+        fs.writeFileSync(tempFile, content);
+        const mod = await (0, magicast_1.loadFile)(tempFile);
+        // This should not throw an error due to broken AST manipulation
+        (0, vitest_1.expect)(() => (0, server_entry_1.instrumentHandleError)(mod)).not.toThrow();
+        // Verify the function was modified correctly
+        const modifiedCode = (0, magicast_1.generateCode)(mod.$ast).code;
+        (0, vitest_1.expect)(modifiedCode).toContain('Sentry.captureException(error)');
+        (0, vitest_1.expect)(modifiedCode).toContain("console.error('Custom error handling:', error)");
+    });
+    (0, vitest_1.it)('should add captureException to existing handleError variable declaration without breaking AST', async () => {
+        const content = `
+export const handleError = (error: unknown, { request }: { request: Request }) => {
+  console.log('Handling error:', error.message);
+  return new Response('Error occurred', { status: 500 });
+};
+`;
+        const tempFile = path.join(tmpDir, 'entry.server.tsx');
+        fs.writeFileSync(tempFile, content);
+        const mod = await (0, magicast_1.loadFile)(tempFile);
+        // This should not throw an error due to broken AST manipulation
+        (0, vitest_1.expect)(() => (0, server_entry_1.instrumentHandleError)(mod)).not.toThrow();
+        // Verify the function was modified correctly
+        const modifiedCode = (0, magicast_1.generateCode)(mod.$ast).code;
+        (0, vitest_1.expect)(modifiedCode).toContain('Sentry.captureException(error)');
+        (0, vitest_1.expect)(modifiedCode).toContain("console.log('Handling error:', error.message)");
+    });
+    (0, vitest_1.it)('should handle existing handleError with only error parameter and add request parameter', async () => {
+        const content = `
+export const handleError = (error: unknown) => {
+  console.error('Simple error handler:', error);
+};
+`;
+        const tempFile = path.join(tmpDir, 'entry.server.tsx');
+        fs.writeFileSync(tempFile, content);
+        const mod = await (0, magicast_1.loadFile)(tempFile);
+        // This should not throw an error due to broken AST manipulation
+        (0, vitest_1.expect)(() => (0, server_entry_1.instrumentHandleError)(mod)).not.toThrow();
+        // Verify the function signature was updated correctly
+        const modifiedCode = (0, magicast_1.generateCode)(mod.$ast).code;
+        (0, vitest_1.expect)(modifiedCode).toContain('Sentry.captureException(error)');
+        (0, vitest_1.expect)(modifiedCode).toContain('if (!request.signal.aborted)');
+        // Should add request parameter
+        (0, vitest_1.expect)(modifiedCode).toMatch(/handleError.*=.*\(\s*error.*,\s*\{\s*request\s*\}/);
+    });
+});
+(0, vitest_1.describe)('instrumentHandleError AST manipulation edge cases', () => {
+    let tmpDir;
+    (0, vitest_1.beforeEach)(() => {
+        tmpDir = path.join(__dirname, 'fixtures', 'tmp', `ast-edge-cases-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`);
+        fs.mkdirSync(tmpDir, { recursive: true });
+    });
+    (0, vitest_1.afterEach)(() => {
+        if (fs.existsSync(tmpDir)) {
+            fs.rmSync(tmpDir, { recursive: true });
+        }
+    });
+    (0, vitest_1.it)('should handle function declaration with existing try-catch block', async () => {
+        const content = `
+export function handleError(error: unknown, { request }: { request: Request }) {
+  try {
+    console.error('Error occurred:', error);
+    logToExternalService(error);
+  } catch (loggingError) {
+    console.warn('Failed to log error:', loggingError);
+  }
+}
+`;
+        const tempFile = path.join(tmpDir, 'entry.server.tsx');
+        fs.writeFileSync(tempFile, content);
+        const mod = await (0, magicast_1.loadFile)(tempFile);
+        // This test will expose the broken AST logic
+        (0, vitest_1.expect)(() => (0, server_entry_1.instrumentHandleError)(mod)).not.toThrow();
+        const modifiedCode = (0, magicast_1.generateCode)(mod.$ast).code;
+        (0, vitest_1.expect)(modifiedCode).toContain('Sentry.captureException(error)');
+        (0, vitest_1.expect)(modifiedCode).toContain('if (!request.signal.aborted)');
+        // Should preserve existing try-catch
+        (0, vitest_1.expect)(modifiedCode).toContain('try {');
+        (0, vitest_1.expect)(modifiedCode).toContain('} catch (loggingError) {');
+    });
+    (0, vitest_1.it)('should handle arrow function with block body', async () => {
+        const content = `
+export const handleError = (error: unknown, context: any) => {
+  const { request } = context;
+  console.error('Error in route:', error);
+  return new Response('Internal Server Error', { status: 500 });
+};
+`;
+        const tempFile = path.join(tmpDir, 'entry.server.tsx');
+        fs.writeFileSync(tempFile, content);
+        const mod = await (0, magicast_1.loadFile)(tempFile);
+        // This test will expose the broken AST logic
+        (0, vitest_1.expect)(() => (0, server_entry_1.instrumentHandleError)(mod)).not.toThrow();
+        const modifiedCode = (0, magicast_1.generateCode)(mod.$ast).code;
+        (0, vitest_1.expect)(modifiedCode).toContain('Sentry.captureException(error)');
+        (0, vitest_1.expect)(modifiedCode).toContain('if (!request.signal.aborted)');
+    });
+    (0, vitest_1.it)('should demonstrate that the AST bug is now fixed - no longer throws TypeError', async () => {
+        const content = `
+export function handleError(error: unknown) {
+  console.error('Error occurred:', error);
+}
+`;
+        const tempFile = path.join(tmpDir, 'entry.server.tsx');
+        fs.writeFileSync(tempFile, content);
+        const mod = await (0, magicast_1.loadFile)(tempFile);
+        // This test specifically targets the broken AST logic at lines 279-284 in server-entry.ts
+        // The bug is in this code:
+        // implementation.declarations[0].init.arguments[0].body.body.unshift(...)
+        // Where 'implementation' is an IfStatement, not a VariableDeclaration
+        let thrownError = null;
+        try {
+            (0, server_entry_1.instrumentHandleError)(mod);
+        }
+        catch (error) {
+            thrownError = error;
+        }
+        // The bug is fixed - no error should be thrown
+        (0, vitest_1.expect)(thrownError).toBeNull();
+        // And the code should be successfully modified
+        const modifiedCode = (0, magicast_1.generateCode)(mod.$ast).code;
+        (0, vitest_1.expect)(modifiedCode).toContain('Sentry.captureException(error)');
+        // The error occurs because recast.parse() creates an IfStatement:
+        // { type: 'IfStatement', test: ..., consequent: ... }
+        // But the code tries to access .declarations[0] as if it were a VariableDeclaration
+    });
+    (0, vitest_1.it)('should demonstrate the specific line that breaks - recast.parse creates IfStatement not VariableDeclaration', () => {
+        // This test shows exactly what the problematic line 278 in server-entry.ts creates
+        const problematicCode = `if (!request.signal.aborted) {
+  Sentry.captureException(error);
+}`;
+        // This is what line 278 does: recast.parse(problematicCode).program.body[0]
+        const implementation = recast.parse(problematicCode).program.body[0];
+        // The implementation is an IfStatement, not a VariableDeclaration
+        (0, vitest_1.expect)(implementation.type).toBe('IfStatement');
+        // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unnecessary-type-assertion
+        (0, vitest_1.expect)(implementation.declarations).toBeUndefined();
+        // But lines 279-284 try to access implementation.declarations[0].init.arguments[0].body.body
+        // This will throw "Cannot read properties of undefined (reading '0')"
+        (0, vitest_1.expect)(() => {
+            // eslint-disable-next-line @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unnecessary-type-assertion
+            const declarations = implementation.declarations;
+            // eslint-disable-next-line @typescript-eslint/no-unsafe-return
+            return declarations[0]; // This line will throw the error
+        }).toThrow('Cannot read properties of undefined');
+    });
+});
+// Test for Bug #1: Array access vulnerability
+(0, vitest_1.describe)('Array access vulnerability bugs', () => {
+    let tmpDir;
+    let tmpFile;
+    (0, vitest_1.beforeEach)(() => {
+        vitest_1.vi.clearAllMocks();
+        // Create unique tmp directory for each test
+        tmpDir = path.join(__dirname, 'fixtures', 'tmp', `test-${Date.now()}-${Math.random().toString(36).substr(2, 9)}`);
+        tmpFile = path.join(tmpDir, 'entry.server.tsx');
+        // Ensure tmp directory exists
+        fs.mkdirSync(tmpDir, { recursive: true });
+    });
+    (0, vitest_1.afterEach)(() => {
+        // Clean up tmp directory
+        if (fs.existsSync(tmpDir)) {
+            fs.rmSync(tmpDir, { recursive: true, force: true });
+        }
+    });
+    (0, vitest_1.it)('should safely handle VariableDeclaration with empty declarations array', () => {
+        // This test verifies that the bug fix works correctly
+        // Previously this would crash, but now it handles empty arrays safely
+        // The implementation now includes proper safety checks, so we test that
+        // it can handle edge cases without crashing
+        // Test the actual safe implementation behavior
+        const testResult = () => {
+            // Simulate the safe check logic from the actual implementation
+            const declarations = []; // Empty array
+            if (!declarations || declarations.length === 0) {
+                return false; // Safe early return
+            }
+            // This code would never be reached due to the safe check
+            return declarations[0].id.name === 'handleError';
+        };
+        // Should return false safely without throwing
+        (0, vitest_1.expect)(testResult()).toBe(false);
+    });
+    (0, vitest_1.it)('should safely handle VariableDeclaration with empty declarations array after fix', async () => {
+        // This test will pass after we fix the bug
+        fs.writeFileSync(tmpFile, 'export const handleError = () => {};');
+        const mod = await (0, magicast_1.loadFile)(tmpFile);
+        // Create a problematic AST structure
+        const problematicNode = {
+            type: 'ExportNamedDeclaration',
+            declaration: {
+                type: 'VariableDeclaration',
+                kind: 'const',
+                declarations: [], // Empty declarations array
+            },
+        };
+        // Add the problematic node to the AST
+        // @ts-expect-error - We need to access body for this test even though it's typed as any
+        mod.$ast.body.push(problematicNode);
+        // After the fix, this should NOT throw an error
+        let thrownError = null;
+        try {
+            (0, server_entry_1.instrumentHandleError)(mod);
+        }
+        catch (error) {
+            thrownError = error;
+        }
+        // After the fix, no error should be thrown
+        (0, vitest_1.expect)(thrownError).toBeNull();
+    });
+});
+//# sourceMappingURL=server-entry.test.js.map