Skip to content

Commit

Permalink
chore(electron): temporary remove @sentry/electron due to version mis…
Browse files Browse the repository at this point in the history
…match (#7001)
  • Loading branch information
Brooooooklyn committed May 21, 2024
1 parent d898dae commit 41c7215
Show file tree
Hide file tree
Showing 5 changed files with 27 additions and 860 deletions.
1 change: 0 additions & 1 deletion packages/frontend/electron/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,6 @@
"@electron-forge/shared-types": "^7.3.0",
"@emotion/react": "^11.11.4",
"@pengx17/electron-forge-maker-appimage": "^1.2.0",
"@sentry/electron": "^5.0.0-beta.0",
"@sentry/esbuild-plugin": "^2.16.1",
"@sentry/react": "^8.0.0",
"@toeverything/infra": "workspace:*",
Expand Down
33 changes: 15 additions & 18 deletions packages/frontend/electron/renderer/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,10 @@ import '@affine/core/bootstrap/preload';
import { appConfigProxy } from '@affine/core/hooks/use-app-config-storage';
import { performanceLogger } from '@affine/core/shared';
import { apis, events } from '@affine/electron-api';
import { init, setTags } from '@sentry/electron/renderer';
import {
init as reactInit,
init,
reactRouterV6BrowserTracingIntegration,
setTags,
} from '@sentry/react';
import { debounce } from 'lodash-es';
import { StrictMode, useEffect } from 'react';
Expand Down Expand Up @@ -38,22 +38,19 @@ function main() {
performanceMainLogger.info('setup start');
if (window.SENTRY_RELEASE || environment.isDebug) {
// https://docs.sentry.io/platforms/javascript/guides/electron/
init(
{
dsn: process.env.SENTRY_DSN,
environment: process.env.BUILD_TYPE ?? 'development',
integrations: [
reactRouterV6BrowserTracingIntegration({
useEffect,
useLocation,
useNavigationType,
createRoutesFromChildren,
matchRoutes,
}),
],
},
reactInit
);
init({
dsn: process.env.SENTRY_DSN,
environment: process.env.BUILD_TYPE ?? 'development',
integrations: [
reactRouterV6BrowserTracingIntegration({
useEffect,
useLocation,
useNavigationType,
createRoutesFromChildren,
matchRoutes,
}),
],
});
setTags({
appVersion: runtimeConfig.appVersion,
editorVersion: runtimeConfig.editorVersion,
Expand Down
13 changes: 0 additions & 13 deletions packages/frontend/electron/src/main/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@ import './security-restrictions';

import path from 'node:path';

import { init, IPCMode } from '@sentry/electron/main';
import { app } from 'electron';

import { createApplicationMenu } from './application-menu/create';
Expand All @@ -13,7 +12,6 @@ import { registerEvents } from './events';
import { registerHandlers } from './handlers';
import { logger } from './logger';
import { registerProtocol } from './protocol';
import { isOnline } from './ui/handlers';
import { registerUpdater } from './updater';
import { launch } from './windows-manager/launcher';
import { launchStage } from './windows-manager/stage';
Expand Down Expand Up @@ -65,17 +63,6 @@ app.on('activate', () => {

setupDeepLink(app);

// https://docs.sentry.io/platforms/javascript/guides/electron/
if (process.env.SENTRY_DSN) {
init({
dsn: process.env.SENTRY_DSN,
ipcMode: IPCMode.Protocol,
transportOptions: {
shouldSend: () => isOnline,
},
});
}

/**
* Create app window when background process will be ready
*/
Expand Down
1 change: 0 additions & 1 deletion packages/frontend/electron/src/preload/index.ts
Original file line number Diff line number Diff line change
@@ -1,2 +1 @@
import '@sentry/electron/preload';
import './bootstrap';

0 comments on commit 41c7215

Please sign in to comment.