-
-
Notifications
You must be signed in to change notification settings - Fork 366
[DO NOT MERGE]: chore(deps): bump @sentry/* to 11.0.0-beta.0 #6665
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
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 |
|---|---|---|
|
|
@@ -8,10 +8,6 @@ | |
| 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'; | ||
|
|
@@ -24,10 +20,8 @@ | |
| 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'; | ||
|
|
@@ -47,33 +41,17 @@ | |
| 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'; | ||
|
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. Removing AI instrumentation exports breaks the existing React Native API and test suite
Evidence
Also found at 5 additional locations
Identified by Warden ยท code-review ยท W4Y-3U8
Contributor
Author
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. Intentional for this spike โ this PR is a DO NOT MERGE compatibility probe for
|
||
|
|
@@ -123,20 +101,12 @@ | |
| export { addConsoleInstrumentationFilter } | ||
|
|
||
| export { addEventProcessor } | ||
|
|
||
| export { addIntegration } | ||
|
|
||
| export { AnthropicAiClient } | ||
|
|
||
| export { AnthropicAiInstrumentedMethod } | ||
|
|
||
| export { AnthropicAiOptions } | ||
|
|
||
| export { AnthropicAiResponse } | ||
|
|
||
| // @public | ||
| export function appLoaded(): void; | ||
|
|
||
|
Check failure on line 109 in packages/core/etc/sentry-react-native.api.md
|
||
|
sentry-warden[bot] marked this conversation as resolved.
|
||
| // @public (undocumented) | ||
| export const appRegistryIntegration: () => Integration & { | ||
| onRunApplication: (callback: () => void) => void; | ||
|
|
@@ -179,19 +149,15 @@ | |
| // @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; | ||
|
Check failure on line 160 in packages/core/etc/sentry-react-native.api.md
|
||
|
|
||
| // @public | ||
| export function createReactNativeRewriteFrames(): Integration; | ||
|
|
@@ -427,59 +393,31 @@ | |
| // @public | ||
| export type GlobalErrorBoundaryProps = ErrorBoundaryProps & { | ||
| includeNonFatalGlobalErrors?: boolean; | ||
| 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 | ||
| export function graphqlIntegration(options: GraphQLReactNativeIntegrationOptions): Integration; | ||
|
|
||
| // Warning: (ae-forgotten-export) The symbol "HermesProfilingOptions" needs to be exported by the entry point index.d.ts | ||
| // | ||
| // @public | ||
| export const hermesProfilingIntegration: (initOptions?: HermesProfilingOptions) => Integration; | ||
|
|
||
| // @public @deprecated (undocumented) | ||
| export const hideFeedbackButton: () => void; | ||
|
|
||
| export { httpClientIntegration } | ||
|
|
||
| export { httpContextIntegration } | ||
|
|
||
| export { inboundFiltersIntegration } | ||
|
|
||
| // @public | ||
|
Check failure on line 418 in packages/core/etc/sentry-react-native.api.md
|
||
| 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 failure 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) | ||
|
|
||
|
|
||
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 type-check no longer validates published ts3.8 types
This suite now type-checks only with TypeScript 5.0.4, so it no longer exercises the still-published
typesVersions/ts3.8downlevel path for TS < 4.9; keep a TS < 4.9 check or drop that support path together.Evidence
sentry-react-native-ts-5-0-testand pinstypescriptto5.0.4(was3.8.3).dev-packages/type-check/run-type-check.shonly entersts5.0-testand runsyarn type-checkthere.packages/core/package.jsonstill shipstypesVersions["<4.9"] -> ts3.8/*and adownlevel-dts ... --to=3.8build.disttypings, so the downleveledts3.8types are never compiled in CI.Identified by Warden ยท code-review ยท 8CF-9M7