refactor: Centralize logging and Sentry error reporting (#871)

## Description

- **Centralized Logger**: Replace `app/utils/logger` (static class,
default export) with `app/shared/lib/logger` (singleton instance, named
export)
- **Sentry consolidation**: All direct `@sentry/nextjs` imports replaced
by `Logger.panic()`/`Logger.warn({ sentry: true })` or re-exports from
`app/shared/lib/sentry/`. ESLint `no-restricted-imports` rule enforces
this boundary
- **Global error boundaries**: New `app/error.tsx` and
`app/global-error.tsx` catch route-level and root-level errors,
replacing per-component ErrorBoundary wrappers
- **ESLint enforcement**: `no-console: error` globally to prevent
regression
- **Sentry SDK upgrade**: Pin `@sentry/core` and `@sentry/nextjs` to
`10.39.0`, migrate build config to
`webpack.treeshake.removeDebugLogging`
- **Test cleanup**: Global Logger mock in `test-setup.specs.ts` replaces
~8 per-file mock blocks

  ### Breaking changes

- `NEXT_PUBLIC_ENABLE_CATCH_EXCEPTIONS` env var removed — Sentry capture
is now always-on when DSN is configured
  - `SENTRY_SUPPRESS_GLOBAL_ERROR_HANDLER_FILE_WARNING` env var removed
- Import paths changed: `@utils/logger` → `@/app/shared/lib/logger`,
`SentryErrorBoundary` and `withTraceData` moved to
`@/app/shared/lib/sentry`
## Type of change

<!-- Check the appropriate options that apply to this PR -->

- [x] Other (please describe): Enable sentry for all pages, logger
refactoring

## Screenshots

<!-- For UI changes, especially protocol screens, include screenshots
showing the changes -->
<!-- This is REQUIRED for protocol integration PRs -->

## Testing

<!-- Describe how you tested your changes -->
<!-- For protocol integrations, explain how you verified the protocol
data is correctly displayed -->

## Related Issues

Closes
[HOO-246](https://linear.app/solana-fndn/issue/HOO-246/enable-sentry-for-all-the-pages)
## Checklist

<!-- Verify that you have completed the following before requesting
review -->

-   [x] My code follows the project's style guidelines
-   [x] I have added tests that prove my fix/feature works
-   [x] All tests pass locally and in CI
-   [x] I have updated documentation as needed
-   [x] I have run `build:info` script to update build information
-   [x] CI/CD checks pass

---------

Co-authored-by: Sergo <rogaldh@radsh.red>

Author: askov

.env.example

@@ -3,8 +3,10 @@
 NEXT_PUBLIC_MAINNET_RPC_URL=
 NEXT_PUBLIC_DEVNET_RPC_URL=
 NEXT_PUBLIC_TESTNET_RPC_URL=
-## Configuration for "Logger". Set max level of logs to see (0..5)
-NEXT_LOG_LEVEL=0
+## Configuration for "Logger". Set max level of logs to see (0..4)
+## 0=PANIC, 1=ERROR, 2=WARN, 3=INFO, 4=DEBUG
+## All logging is suppressed when this variable is not set.
+NEXT_LOG_LEVEL=4
 ## Configuration for IBRL Block Explorer
 NEXT_PUBLIC_IBRL_EXPLORER_URL=
 ## Configuration for "metadata" service. Set "**_ENABLED=true" to enable
@@ -52,11 +54,12 @@ RUGCHECK_API_KEY=
 COINGECKO_API_KEY=
 ### Configuration for Bluprynt API
 BLUPRYNT_CREDENTIAL_AUTHORITY=
-## Configuration for Sentry
-### Flag whether to catch exceptions with the boundary on the client or not
-NEXT_PUBLIC_ENABLE_CATCH_EXCEPTIONS=1
-### Flag that allows to not collect errors globally (allows to track for errors that we need at the moment
-SENTRY_SUPPRESS_GLOBAL_ERROR_HANDLER_FILE_WARNING=1
+## Configuration for Sentry feature enabled
+### Sentry is disabled by default. To enable error reporting, set SENTRY_DSN to
+### your project's DSN (found in Sentry under Settings > Projects > Client Keys).
+### Without a DSN all Sentry calls are silent no-ops.
+### SENTRY_ORG, SENTRY_PRJ, and SENTRY_AUTH_TOKEN are only needed for source-map
+### uploads during production builds.
 SENTRY_ORG=
 SENTRY_DSN=
 SENTRY_PRJ=

.eslintrc.json

@@ -22,7 +22,8 @@
                 "message": "RegExps are not recommended. If you sure regexp is needed - please use eslint-disable-next no-restricted-syntax -- %comment%  to explain why"
             }
         ],
-        "@eslint-community/eslint-comments/no-unlimited-disable": "error"
+        "@eslint-community/eslint-comments/no-unlimited-disable": "error",
+        "no-console": "error"
     },
     "overrides": [
         // Only uses Testing Library lint rules in test files
@@ -36,6 +37,31 @@
             "rules": {
                 "@eslint-community/eslint-comments/no-unlimited-disable": "off"
             }
+        },
+        // Allow console.* in the Logger implementation, scripts, and pnpm hooks
+        {
+            "files": ["app/shared/lib/logger.ts", "scripts/**", ".pnpmfile.cjs"],
+            "rules": {
+                "no-console": "off"
+            }
+        },
+        // Forbid direct @sentry/nextjs imports in app code — use @/app/shared/lib/sentry instead
+        {
+            "files": ["app/**/*.[jt]s?(x)"],
+            "excludedFiles": ["app/shared/lib/sentry/**", "app/shared/lib/logger.ts"],
+            "rules": {
+                "no-restricted-imports": [
+                    "error",
+                    {
+                        "paths": [
+                            {
+                                "name": "@sentry/nextjs",
+                                "message": "Import from '@/app/shared/lib/sentry' instead. For logging, use the Logger from '@/app/shared/lib/logger'."
+                            }
+                        ]
+                    }
+                ]
+            }
         }
     ]
 }

.github/workflows/ci.yaml

@@ -11,7 +11,6 @@ env:
   PNPM_CACHE_FOLDER: ~/.pnpm-store
   PW_CACHE_AFFIX: pw
   PW_CACHE_FOLDER: ~/.cache/ms-playwright
-  SENTRY_SUPPRESS_GLOBAL_ERROR_HANDLER_FILE_WARNING: 1
   SENTRY_TELEMETRY_DISABLE: true
   STORYBOOK_DISABLE_TELEMETRY: 1
 

__tests__/middleware.spec.ts

@@ -8,7 +8,7 @@ vi.mock('botid/server', () => ({
     checkBotId: vi.fn(),
 }));
 
-import Logger from '@utils/logger';
+import { Logger } from '@/app/shared/lib/logger';
 import { checkBotId } from 'botid/server';
 
 import { middleware } from '../middleware';
@@ -78,7 +78,8 @@ describe('middleware', () => {
                 expect(response.status).toBe(200);
                 expect(checkBotId).not.toHaveBeenCalled();
                 expect(loggerInfoSpy).toHaveBeenCalledWith(
-                    expect.stringContaining('[middleware] No x-is-human header')
+                    '[middleware] No x-is-human header, allowing',
+                    expect.objectContaining({ pathname: '/api/test' })
                 );
             });
         });
@@ -98,10 +99,13 @@ describe('middleware', () => {
                 expect(response.status).toBe(200);
                 expect(checkBotId).toHaveBeenCalled();
                 expect(loggerInfoSpy).toHaveBeenCalledWith(
-                    expect.stringContaining('[middleware] BotId verification'),
+                    '[middleware] BotId verification',
                     expect.objectContaining({ isHuman: true })
                 );
-                expect(loggerInfoSpy).toHaveBeenCalledWith(expect.stringContaining('[middleware] Human verified'));
+                expect(loggerInfoSpy).toHaveBeenCalledWith(
+                    '[middleware] Human verified',
+                    expect.objectContaining({ pathname: '/api/test' })
+                );
             });
 
             it('should allow bot requests when challenge mode is disabled and log warning', async () => {
@@ -116,7 +120,10 @@ describe('middleware', () => {
                 const response = await middleware(request);
 
                 expect(response.status).toBe(200);
-                expect(loggerWarnSpy).toHaveBeenCalledWith(expect.stringContaining('[middleware] Bot detected'));
+                expect(loggerWarnSpy).toHaveBeenCalledWith(
+                    '[middleware] Bot detected',
+                    expect.objectContaining({ pathname: '/api/test' })
+                );
             });
         });
 
@@ -139,11 +146,13 @@ describe('middleware', () => {
                 expect(response.status).toBe(401);
                 const body = await response.json();
                 expect(body).toEqual({ error: 'Access denied: request identified as automated bot' });
-                expect(loggerWarnSpy).toHaveBeenCalledWith(expect.stringContaining('[middleware] Bot detected'));
+                expect(loggerWarnSpy).toHaveBeenCalledWith(
+                    '[middleware] Bot detected',
+                    expect.objectContaining({ pathname: '/api/test' })
+                );
                 expect(loggerErrorSpy).toHaveBeenCalledWith(
-                    expect.objectContaining({
-                        message: expect.stringContaining('[middleware] Challenge mode enabled, blocking'),
-                    })
+                    new Error('[middleware] Challenge mode enabled, blocking'),
+                    expect.objectContaining({ pathname: '/api/test' })
                 );
             });
 
@@ -160,9 +169,8 @@ describe('middleware', () => {
 
                 expect(response.status).toBe(401);
                 expect(loggerErrorSpy).toHaveBeenCalledWith(
-                    expect.objectContaining({
-                        message: expect.stringContaining('[middleware] Challenge mode enabled, blocking'),
-                    })
+                    new Error('[middleware] Challenge mode enabled, blocking'),
+                    expect.objectContaining({ pathname: '/api/test' })
                 );
             });
 
@@ -178,7 +186,10 @@ describe('middleware', () => {
                 const response = await middleware(request);
 
                 expect(response.status).toBe(200);
-                expect(loggerInfoSpy).toHaveBeenCalledWith(expect.stringContaining('[middleware] Human verified'));
+                expect(loggerInfoSpy).toHaveBeenCalledWith(
+                    '[middleware] Human verified',
+                    expect.objectContaining({ pathname: '/api/test' })
+                );
             });
         });
     });
@@ -199,7 +210,10 @@ describe('middleware', () => {
             const response = await middleware(request);
 
             expect(response.status).toBe(200);
-            expect(loggerWarnSpy).toHaveBeenCalledWith(expect.stringContaining('[middleware] Bot detected'));
+            expect(loggerWarnSpy).toHaveBeenCalledWith(
+                '[middleware] Bot detected',
+                expect.objectContaining({ pathname: '/api/test' })
+            );
         });
 
         it('should block request when both simulate bot mode and challenge mode are enabled', async () => {
@@ -219,9 +233,8 @@ describe('middleware', () => {
 
             expect(response.status).toBe(401);
             expect(loggerErrorSpy).toHaveBeenCalledWith(
-                expect.objectContaining({
-                    message: expect.stringContaining('[middleware] Challenge mode enabled, blocking'),
-                })
+                new Error('[middleware] Challenge mode enabled, blocking'),
+                expect.objectContaining({ pathname: '/api/test' })
             );
         });
     });

app/address/[address]/idl/page-renderer.tsx

@@ -1,10 +1,4 @@
-import { captureException } from '@sentry/nextjs';
 import { ComponentType } from 'react';
-import { ErrorBoundary } from 'react-error-boundary';
-
-import { ErrorCard } from '@/app/components/common/ErrorCard';
-
-const isSentryEnabled = process.env.NEXT_PUBLIC_ENABLE_CATCH_EXCEPTIONS === '1';
 
 export function PageRenderer({
     address,
@@ -13,16 +7,5 @@ export function PageRenderer({
     address: string;
     renderComponent: ComponentType<{ address: string }>;
 }) {
-    return (
-        <ErrorBoundary
-            onError={(error: Error) => {
-                if (isSentryEnabled) {
-                    captureException(error);
-                }
-            }}
-            fallbackRender={({ error }) => <ErrorCard text={`Failed to load: ${error.message}`} />}
-        >
-            <RenderComponent address={address} />
-        </ErrorBoundary>
-    );
+    return <RenderComponent address={address} />;
 }

app/address/[address]/idl/page.tsx

@@ -1,7 +1,7 @@
 import getReadableTitleFromAddress, { AddressPageMetadataProps } from '@utils/get-readable-title-from-address';
 import { Metadata } from 'next/types';
 
-import { withSentryTraceData } from '@/app/utils/with-sentry-trace-data';
+import { withTraceData } from '@/app/shared/lib/sentry';
 
 import IdlPageClient from './page-client';
 
@@ -12,7 +12,7 @@ type Props = Readonly<{
 }>;
 
 export async function generateMetadata(props: AddressPageMetadataProps): Promise<Metadata> {
-    return withSentryTraceData({
+    return withTraceData({
         description: `The Interface Definition Language (IDL) file for the program at address ${props.params.address} on Solana`,
         title: `Program IDL | ${await getReadableTitleFromAddress(props)} | Solana`,
     });

app/address/[address]/security/page-client.tsx

@@ -1,15 +1,10 @@
 'use client';
 
-import { captureException } from '@sentry/nextjs';
 import React from 'react';
-import { ErrorBoundary } from 'react-error-boundary';
 
 import { ParsedAccountRenderer } from '@/app/components/account/ParsedAccountRenderer';
-import { ErrorCard } from '@/app/components/common/ErrorCard';
 import { SecurityCard } from '@/app/features/security-txt/ui/SecurityCard';
 
-const isSentryEnabled = process.env.NEXT_PUBLIC_ENABLE_CATCH_EXCEPTIONS === '1';
-
 type Props = Readonly<{
     params: {
         address: string;
@@ -28,16 +23,5 @@ function SecurityCardRenderer({
 }
 
 export default function SecurityPageClient({ params: { address } }: Props) {
-    return (
-        <ErrorBoundary
-            onError={(error: Error) => {
-                if (isSentryEnabled) {
-                    captureException(error);
-                }
-            }}
-            fallbackRender={({ error }) => <ErrorCard text={`Failed to load security data: ${error.message}`} />}
-        >
-            <ParsedAccountRenderer address={address} renderComponent={SecurityCardRenderer} />
-        </ErrorBoundary>
-    );
+    return <ParsedAccountRenderer address={address} renderComponent={SecurityCardRenderer} />;
 }

app/address/[address]/security/page.tsx

@@ -1,12 +1,12 @@
 import getReadableTitleFromAddress, { AddressPageMetadataProps } from '@utils/get-readable-title-from-address';
 import { Metadata } from 'next/types';
 
-import { withSentryTraceData } from '@/app/utils/with-sentry-trace-data';
+import { withTraceData } from '@/app/shared/lib/sentry';
 
 import SecurityPageClient from './page-client';
 
 export async function generateMetadata(props: AddressPageMetadataProps): Promise<Metadata> {
-    return withSentryTraceData({
+    return withTraceData({
         description: `Contents of the security.txt for the program with address ${props.params.address} on Solana`,
         title: `Security | ${await getReadableTitleFromAddress(props)} | Solana`,
     });

app/api/ans-domains/[address]/__tests__/route.spec.ts

@@ -1,19 +1,14 @@
 import { fetchAnsDomains } from '@entities/domain/api/fetch-ans-domains';
-import Logger from '@utils/logger';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 
+import { Logger } from '@/app/shared/lib/logger';
+
 import { GET } from '../route';
 
 vi.mock('@entities/domain/api/fetch-ans-domains', () => ({
     fetchAnsDomains: vi.fn(),
 }));
 
-vi.mock('@utils/logger', () => ({
-    default: {
-        error: vi.fn(),
-    },
-}));
-
 const VALID_ADDRESS = 'TokenkegQfeZyiNwAJbNbGKPFXCWuBvf9Ss623VQ5DA';
 const mockRequest = new Request('http://localhost:3000/api/ans-domains/' + VALID_ADDRESS);
 
@@ -83,7 +78,7 @@ describe('GET /api/ans-domains/[address]', () => {
 
             await GET(mockRequest, { params: { address: VALID_ADDRESS } });
 
-            expect(Logger.error).toHaveBeenCalledWith(error, `Failed to fetch ANS domains for ${VALID_ADDRESS}`);
+            expect(Logger.error).toHaveBeenCalledWith(error, { address: VALID_ADDRESS });
         });
 
         it('should not cache error responses', async () => {

app/api/ans-domains/[address]/route.ts

@@ -1,8 +1,9 @@
 import { fetchAnsDomains } from '@entities/domain/api/fetch-ans-domains';
 import { PublicKey } from '@solana/web3.js';
-import Logger from '@utils/logger';
 import { NextResponse } from 'next/server';
 
+import { Logger } from '@/app/shared/lib/logger';
+
 const CACHE_HEADERS = { 'Cache-Control': 'public, s-maxage=86400, stale-while-revalidate=3600' };
 
 type Params = {
@@ -22,7 +23,7 @@ export async function GET(_request: Request, { params: { address } }: Params) {
         const domains = await fetchAnsDomains(address);
         return NextResponse.json({ domains }, { headers: CACHE_HEADERS });
     } catch (error) {
-        Logger.error(error, `Failed to fetch ANS domains for ${address}`);
+        Logger.error(error, { address });
         return NextResponse.json({ domains: [] }, { headers: { 'Cache-Control': 'no-store' }, status: 500 });
     }
 }