Revert sentry (#222)

* Revert "adding log because no idea (#221)"

This reverts commit 2ac342e.

* Revert "trying workflow stuff (#220)"

This reverts commit ce897ac.

* Revert "add release to main_push workflow (#219)"

This reverts commit 8b10ac7.

* Revert "Zoe/poc sentry (#218)"

This reverts commit 7fa5323.

Author: gh-zoe-cade

.github/workflows/build_and_deploy.yaml

@@ -59,15 +59,6 @@ jobs:
         with:
           install_components: beta
 
-      - name: Create Sentry release
-        uses: getsentry/action-release@v1
-        env:
-          SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
-          SENTRY_ORG: 'checkmarble'
-          SENTRY_PROJECT: 'marble-frontend'
-        with:
-          environment: staging
-
       - name: Deploy server
         run: |
           gcloud run deploy marble-frontend \

.gitignore

@@ -44,6 +44,4 @@ Thumbs.db
 
 # Firebase
 firebase-debug.log
-ui-debug.log
-# Sentry Config File
-.sentryclirc
+ui-debug.log

Dockerfile

@@ -12,7 +12,7 @@ RUN pnpm install --prod --frozen-lockfile
 FROM base AS build
 # RUN --mount=type=cache,id=pnpm,target=$PNPM_HOME/store pnpm install --frozen-lockfile
 RUN pnpm install --frozen-lockfile
-RUN pnpm run -r build-with-sourcemaps
+RUN pnpm run -r build
 
 FROM base AS app-builder
 ENV NODE_ENV=production
@@ -27,3 +27,4 @@ COPY --from=build /app/packages/app-builder/public /app/packages/app-builder/pub
 WORKDIR /app/packages/app-builder
 EXPOSE 8080
 CMD [ "pnpm", "exec", "remix-serve", "build"]
+

package.json

@@ -73,8 +73,6 @@
     "@remix-run/node": "^1.19.3",
     "@remix-run/react": "^1.19.3",
     "@remix-run/serve": "^1.19.3",
-    "@sentry/cli": "^2.21.2",
-    "@sentry/remix": "^7.74.0",
     "@tanstack/react-table": "^8.10.6",
     "@tanstack/react-virtual": "3.0.0-beta.61",
     "clsx": "^1.2.1",

packages/app-builder/.env.local

@@ -5,7 +5,6 @@ SESSION_SECRET=SESSION_SECRET
 SESSION_MAX_AGE=43200
 
 MARBLE_API_DOMAIN=http://127.0.0.1:8080
-MARBLE_APP_DOMAIN=http://127.0.0.1:3000
 
 FIREBASE_AUTH_EMULATOR=true
 FIREBASE_AUTH_EMULATOR_HOST=http://localhost:9099
@@ -15,8 +14,4 @@ FIREBASE_AUTH_DOMAIN=tokyo-country-381508.firebaseapp.com
 FIREBASE_PROJECT_ID=tokyo-country-381508
 FIREBASE_STORAGE_BUCKET=tokyo-country-381508.appspot.com
 FIREBASE_MESSAGING_SENDER_ID=1047691849054
-FIREBASE_APP_ID=1:1047691849054:web:a5b69dd2ac584c1160b3cf
-
-SENTRY_AUTH_TOKEN=dummy
-SENTRY_DSN=dummy
-SENTRY_ENVIRONMENT=development
+FIREBASE_APP_ID=1:1047691849054:web:a5b69dd2ac584c1160b3cf

packages/app-builder/package.json

@@ -15,7 +15,6 @@
     "type-check": "npx tsc --noEmit",
     "dev": "dotenv -e .env.local -- remix dev",
     "build": "remix build",
-    "build-with-sourcemaps": "remix build --sourcemap && sentry-upload-sourcemaps --org checkmarble --project marble-frontend --log-level=debug",
     "start": "remix start",
     "generate-routes": "source scripts/generateRoutes.sh"
   },

packages/app-builder/src/entry.client.tsx

@@ -1,11 +1,9 @@
-import { RemixBrowser, useLocation, useMatches } from '@remix-run/react';
-import * as Sentry from '@sentry/remix';
-import { startTransition, StrictMode, useEffect } from 'react';
+import { RemixBrowser } from '@remix-run/react';
+import { startTransition, StrictMode } from 'react';
 import { hydrateRoot } from 'react-dom/client';
 import { I18nextProvider } from 'react-i18next';
 
 import { clientServices } from './services/init.client';
-import { getClientEnv } from './utils/environment.client';
 
 async function hydrate() {
   const { i18nextClientService } = clientServices;
@@ -32,35 +30,3 @@ if (window.requestIdleCallback) {
   // eslint-disable-next-line @typescript-eslint/no-misused-promises
   window.setTimeout(hydrate, 1);
 }
-
-Sentry.init({
-  dsn: getClientEnv('SENTRY_DSN'),
-  environment: getClientEnv('SENTRY_ENVIRONMENT'),
-  integrations: [
-    new Sentry.BrowserTracing({
-      routingInstrumentation: Sentry.remixRouterInstrumentation(
-        useEffect,
-        useLocation,
-        useMatches
-      ),
-    }),
-    // Replay is only available in the client
-    new Sentry.Replay(),
-  ],
-
-  // Set tracesSampleRate to 1.0 to capture 100%
-  // of transactions for performance monitoring.
-  // We recommend adjusting this value in production
-  tracesSampleRate: 1.0,
-
-  // Set `tracePropagationTargets` to control for which URLs distributed tracing should be enabled
-  tracePropagationTargets: [
-    getClientEnv('MARBLE_APP_DOMAIN'),
-    getClientEnv('MARBLE_API_DOMAIN'),
-  ],
-
-  // Capture Replay for 10% of all sessions,
-  // plus for 100% of sessions with an error
-  replaysSessionSampleRate: 0.1,
-  replaysOnErrorSampleRate: 1.0,
-});

packages/app-builder/src/entry.server.tsx

@@ -1,17 +1,11 @@
-import {
-  type DataFunctionArgs,
-  type EntryContext,
-  Response,
-} from '@remix-run/node';
+import { type EntryContext, Response } from '@remix-run/node';
 import { RemixServer } from '@remix-run/react';
-import * as Sentry from '@sentry/remix';
 import isbot from 'isbot';
 import { renderToPipeableStream } from 'react-dom/server';
 import { I18nextProvider } from 'react-i18next';
 import { PassThrough } from 'stream';
 
 import { serverServices } from './services/init.server';
-import { getServerEnv } from './utils/environment.server';
 
 const ABORT_DELAY = 5000;
 
@@ -111,24 +105,3 @@ function handleBrowserRequest(
     setTimeout(abort, ABORT_DELAY);
   });
 }
-
-Sentry.init({
-  dsn: getServerEnv('SENTRY_DSN'),
-  environment: getServerEnv('SENTRY_ENVIRONMENT'),
-  // Set tracesSampleRate to 1.0 to capture 100%
-  // of transactions for performance monitoring.
-  // We recommend adjusting this value in production
-  tracesSampleRate: 1.0,
-});
-
-export async function handleError(
-  error: unknown,
-  { request }: DataFunctionArgs
-) {
-  if (error instanceof Error) {
-    await Sentry.captureRemixServerException(error, 'remix.server', request);
-  } else {
-    // Optionally capture non-Error objects
-    Sentry.captureException(error);
-  }
-}

packages/app-builder/src/root.tsx

@@ -14,7 +14,6 @@ import {
   useLoaderData,
   useRouteError,
 } from '@remix-run/react';
-import { captureRemixErrorBoundaryError, withSentry } from '@sentry/remix';
 import { Tooltip } from '@ui-design-system';
 import { LogoStandard } from '@ui-icons';
 import { type Namespace } from 'i18next';
@@ -102,7 +101,6 @@ export const meta: V2_MetaFunction = () => [
 
 export function ErrorBoundary() {
   const error = useRouteError();
-  captureRemixErrorBoundaryError(error);
 
   return (
     <html lang="en">
@@ -137,7 +135,7 @@ export function ErrorBoundary() {
   );
 }
 
-function App() {
+export default function App() {
   const { locale, ENV, toastMessage, csrf } = useLoaderData<typeof loader>();
 
   const { i18n } = useTranslation(handle.i18n);
@@ -171,5 +169,3 @@ function App() {
     </html>
   );
 }
-
-export default withSentry(App);

packages/app-builder/src/routes/__builder/decisions.tsx

@@ -6,7 +6,6 @@ import {
 import { DecisionsPage } from '@app-builder/components/Decisions';
 import { getRoute } from '@app-builder/utils/routes';
 import { Outlet, useRouteError } from '@remix-run/react';
-import { captureRemixErrorBoundaryError } from '@sentry/remix';
 import { Decision } from '@ui-icons';
 import { type Namespace } from 'i18next';
 import { useTranslation } from 'react-i18next';
@@ -53,8 +52,5 @@ export default function DecisionsLayout() {
 }
 
 export function ErrorBoundary() {
-  const error = useRouteError();
-  captureRemixErrorBoundaryError(error);
-
-  return <ErrorComponent error={error} />;
+  return <ErrorComponent error={useRouteError()} />;
 }