-
-
Notifications
You must be signed in to change notification settings - Fork 367
[DO NOT MERGE] chore(deps): Probe @sentry/* v11.0.0-beta.2 compatibility #6688
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
b206b45
5acaf85
4bfae4c
dff13af
dfabda5
7c61999
6f07544
0d72292
c3dd1bf
1d966e7
b3e1d7c
121c043
e4c369d
3bd9954
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -5,78 +5,56 @@ | |
| ```ts | ||
|
|
||
| import { addBreadcrumb } from '@sentry/core'; | ||
| import { addConsoleInstrumentationFilter } from '@sentry/core'; | ||
| import { addEventProcessor } from '@sentry/core'; | ||
| import { addIntegration } from '@sentry/core'; | ||
| import { AnthropicAiClient } from '@sentry/core'; | ||
| import { AnthropicAiInstrumentedMethod } from '@sentry/core'; | ||
| import { AnthropicAiOptions } from '@sentry/core'; | ||
| import { AnthropicAiResponse } from '@sentry/core'; | ||
| import { Breadcrumb } from '@sentry/core'; | ||
| import { browserApiErrorsIntegration } from '@sentry/react'; | ||
| import { globalHandlersIntegration as browserGlobalHandlersIntegration } from '@sentry/react'; | ||
| import { linkedErrorsIntegration as browserLinkedErrorsIntegration } from '@sentry/react'; | ||
| import type { BrowserOptions } from '@sentry/react'; | ||
| import type { CaptureContext } from '@sentry/core'; | ||
| import { captureEvent } from '@sentry/core'; | ||
| import { captureException } from '@sentry/core'; | ||
| import { captureFeedback } from '@sentry/core'; | ||
| import { captureMessage } from '@sentry/core'; | ||
| import { Client } from '@sentry/core'; | ||
| import type { ClientOptions } from '@sentry/core'; | ||
| import { CompiledGraph } from '@sentry/core'; | ||
| import { consoleLoggingIntegration } from '@sentry/browser'; | ||
| import { consoleSandbox } from '@sentry/core'; | ||
| import { createLangChainCallbackHandler } from '@sentry/core'; | ||
| import { createReduxEnhancer } from '@sentry/react'; | ||
| import { dedupeIntegration } from '@sentry/react'; | ||
| import type { Envelope } from '@sentry/core'; | ||
| import { ErrorBoundary } from '@sentry/react'; | ||
| import type { ErrorBoundaryProps } from '@sentry/react'; | ||
| import { ErrorEvent } from '@sentry/core'; | ||
| import { Event as Event_2 } from '@sentry/core'; | ||
| import type { EventHint } from '@sentry/core'; | ||
| import { Exception } from '@sentry/core'; | ||
| import { extraErrorDataIntegration } from '@sentry/react'; | ||
| import { FeatureFlagsIntegration } from '@sentry/browser'; | ||
| import type { FeedbackFormData } from '@sentry/core'; | ||
| import { functionToStringIntegration } from '@sentry/react'; | ||
| import { getActiveSpan } from '@sentry/core'; | ||
| import { getClient } from '@sentry/core'; | ||
| import { getCurrentScope } from '@sentry/core'; | ||
| import { getGlobalScope } from '@sentry/core'; | ||
| import { getIsolationScope } from '@sentry/core'; | ||
| import { getRootSpan } from '@sentry/core'; | ||
| import { GoogleGenAIChat } from '@sentry/core'; | ||
| import { GoogleGenAIClient } from '@sentry/core'; | ||
| import { GoogleGenAIOptions } from '@sentry/core'; | ||
| import type { HostComponent } from 'react-native'; | ||
| import { httpClientIntegration } from '@sentry/react'; | ||
| import { httpContextIntegration } from '@sentry/react'; | ||
| import type { ImageStyle } from 'react-native'; | ||
| import { inboundFiltersIntegration } from '@sentry/react'; | ||
| import { instrumentAnthropicAiClient } from '@sentry/core'; | ||
| import { InstrumentedMethod } from '@sentry/core'; | ||
| import { instrumentGoogleGenAIClient } from '@sentry/core'; | ||
| import { instrumentLangGraph } from '@sentry/core'; | ||
| import { instrumentOpenAiClient } from '@sentry/core'; | ||
| import { instrumentStateGraph } from '@sentry/core'; | ||
| import { instrumentStateGraphCompile } from '@sentry/core'; | ||
| import { eventFiltersIntegration as inboundFiltersIntegration } from '@sentry/react'; | ||
| import { Integration } from '@sentry/core'; | ||
| import { LangChainIntegration } from '@sentry/core'; | ||
| import { LangChainOptions } from '@sentry/core'; | ||
| import { LangGraphIntegration } from '@sentry/core'; | ||
| import { LangGraphOptions } from '@sentry/core'; | ||
| import { lastEventId } from '@sentry/core'; | ||
| import { logger } from '@sentry/browser'; | ||
| import type { makeFetchTransport } from '@sentry/browser'; | ||
| import { Metric } from '@sentry/core'; | ||
| import { metrics } from '@sentry/browser'; | ||
| import { OpenAiClient } from '@sentry/core'; | ||
| import { OpenAiOptions } from '@sentry/core'; | ||
| import type { Options } from '@sentry/core'; | ||
| import { Profiler } from '@sentry/react'; | ||
| import * as React_2 from 'react'; | ||
|
Check warning on line 57 in packages/core/etc/sentry-react-native.api.md
|
||
|
Comment on lines
8
to
57
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Previously public AI instrumentation exports are removed without a React Native migration path The v11 compatibility change removes the previously public AI helpers and types from Evidence
Also found at 1 additional location
Identified by Warden ยท code-review ยท ZPV-RBD |
||
| import { replayIntegration } from '@sentry/react'; | ||
| import type { ReplayRecordingMode } from '@sentry/core'; | ||
| import { rewriteFramesIntegration } from '@sentry/react'; | ||
|
|
@@ -126,14 +104,6 @@ | |
|
|
||
| export { addIntegration } | ||
|
|
||
| export { AnthropicAiClient } | ||
|
|
||
| export { AnthropicAiInstrumentedMethod } | ||
|
|
||
| export { AnthropicAiOptions } | ||
|
|
||
| export { AnthropicAiResponse } | ||
|
|
||
| // @public | ||
| export function appLoaded(): void; | ||
|
|
||
|
|
@@ -179,17 +149,13 @@ | |
| // @public | ||
| export function close(): Promise<void>; | ||
|
|
||
| export { CompiledGraph } | ||
|
|
||
| export { consoleLoggingIntegration } | ||
|
|
||
| export { consoleSandbox } | ||
|
|
||
| // @public | ||
| export function crashedLastRun(): Promise<boolean | null>; | ||
|
|
||
| export { createLangChainCallbackHandler } | ||
|
|
||
| // @public (undocumented) | ||
| export const createNativeFramesIntegrations: (enable: boolean | undefined) => Integration | undefined; | ||
|
|
||
|
|
@@ -430,12 +396,6 @@ | |
| includeUnhandledRejections?: boolean; | ||
| }; | ||
|
|
||
| export { GoogleGenAIChat } | ||
|
|
||
| export { GoogleGenAIClient } | ||
|
|
||
| export { GoogleGenAIOptions } | ||
|
|
||
| // Warning: (ae-forgotten-export) The symbol "GraphQLReactNativeIntegrationOptions" needs to be exported by the entry point index.d.ts | ||
| // | ||
| // @public | ||
|
|
@@ -458,28 +418,6 @@ | |
| // @public | ||
| export function init(passedOptions: ReactNativeOptions): void; | ||
|
|
||
| export { instrumentAnthropicAiClient } | ||
|
|
||
| export { InstrumentedMethod } | ||
|
|
||
| export { instrumentGoogleGenAIClient } | ||
|
|
||
| export { instrumentLangGraph } | ||
|
|
||
| export { instrumentOpenAiClient } | ||
|
|
||
| export { instrumentStateGraph } | ||
|
|
||
| export { instrumentStateGraphCompile } | ||
|
|
||
| export { LangChainIntegration } | ||
|
|
||
| export { LangChainOptions } | ||
|
|
||
| export { LangGraphIntegration } | ||
|
|
||
| export { LangGraphOptions } | ||
|
|
||
| export { lastEventId } | ||
|
|
||
| // @public (undocumented) | ||
|
|
@@ -549,16 +487,12 @@ | |
| medium: FontStyle; | ||
| bold: FontStyle; | ||
| heavy: FontStyle; | ||
| }; | ||
| } | ||
|
|
||
| export { OpenAiClient } | ||
|
|
||
| export { OpenAiOptions } | ||
|
|
||
| // @public | ||
| export function pauseAppHangTracking(): void; | ||
|
|
||
|
Check warning on line 495 in packages/core/etc/sentry-react-native.api.md
|
||
| // @public | ||
| export function popTurboModuleCall(callId: number): void; | ||
|
|
||
|
|
@@ -949,7 +883,7 @@ | |
| // src/js/feedback/integration.ts:21:5 - (ae-forgotten-export) The symbol "ScreenshotButtonProps" needs to be exported by the entry point index.d.ts | ||
| // src/js/feedback/integration.ts:23:5 - (ae-forgotten-export) The symbol "FeedbackFormTheme" needs to be exported by the entry point index.d.ts | ||
| // src/js/tracing/reactnativetracing.ts:90:3 - (ae-forgotten-export) The symbol "ReactNativeTracingState" needs to be exported by the entry point index.d.ts | ||
| // src/js/tracing/reactnavigation.ts:228:3 - (ae-forgotten-export) The symbol "RouteOverrideProvider" needs to be exported by the entry point index.d.ts | ||
| // src/js/tracing/reactnavigation.ts:229:3 - (ae-forgotten-export) The symbol "RouteOverrideProvider" needs to be exported by the entry point index.d.ts | ||
|
|
||
| // (No @packageDocumentation comment for this package) | ||
|
|
||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -39,11 +39,13 @@ | |
| browserApiErrorsIntegration, | ||
| dedupeIntegration, | ||
| functionToStringIntegration, | ||
| globalHandlersIntegration as browserGlobalHandlersIntegration, | ||
| httpClientIntegration, | ||
| httpContextIntegration, | ||
| inboundFiltersIntegration, | ||
| // `inboundFiltersIntegration` was renamed to `eventFiltersIntegration` in JS v11. | ||
| // Keep the existing public name to avoid breaking the SDK's own API surface. | ||
| eventFiltersIntegration as inboundFiltersIntegration, | ||
| linkedErrorsIntegration as browserLinkedErrorsIntegration, | ||
| rewriteFramesIntegration, | ||
| extraErrorDataIntegration, | ||
|
Check warning on line 50 in packages/core/src/js/integrations/exports.ts
|
||
|
Comment on lines
42
to
50
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Aliased inbound filter factory no longer preserves the integration name The React Native export aliases Evidence
Identified by Warden ยท code-review ยท D3A-GCF |
||
| } from '@sentry/react'; | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TS 3.8 downlevel types no longer type-checked
Bumping this suite to TypeScript 5.0.4 means CI only validates modern
disttypes, while@sentry/react-nativestill shipsts3.8viatypesVersions(<4.9) with no remaining consumer that exercises those downlevel declarations.Evidence
typescriptfrom3.8.3to5.0.4and renames the package tosentry-react-native-ts-5-0-test.run-type-check.shonlycds intots5.0-testand runsyarn type-checkthere.packages/core/package.jsonstill definestypesVersions["<4.9"] -> ts3.8/*and thedownlevelscript still emitsts3.8/dist.<4.9, so this job resolvesdisttypes and never loads the shippedts3.8declarations.Identified by Warden ยท code-review ยท A8Z-SC9