Skip to content

Version Packages#105

Merged
Vadman97 merged 2 commits intomainfrom
changeset-release/main
Jul 9, 2025
Merged

Version Packages#105
Vadman97 merged 2 commits intomainfrom
changeset-release/main

Conversation

@github-actions
Copy link
Contributor

@github-actions github-actions bot commented Jul 9, 2025

RELEASING: Releasing 11 package(s)

Releases:
  @launchdarkly/observability-react-native@0.2.0
  highlight.run@9.18.20
  @launchdarkly/observability@0.3.4
  @launchdarkly/session-replay@0.3.4
  @highlight-run/next@7.9.32
  @highlight-run/node@3.12.16
  @highlight-run/react@19.0.20
  @highlight-run/remix@2.0.115
  @highlight-run/apollo@3.4.43
  @highlight-run/hono@1.0.19
  @highlight-run/nest@3.6.28

@github-actions github-actions bot requested a review from a team as a code owner July 9, 2025 04:49
@Vadman97 Vadman97 force-pushed the changeset-release/main branch from 727daaf to 591381e Compare July 9, 2025 22:28
Vadman97 added 2 commits July 9, 2025 15:50
Releases:
  @launchdarkly/observability-react-native@0.2.0
  highlight.run@9.18.20
  @launchdarkly/observability@0.3.4
  @launchdarkly/session-replay@0.3.4
  @highlight-run/next@7.9.32
  @highlight-run/node@3.12.16
  @highlight-run/react@19.0.20
  @highlight-run/remix@2.0.115
  @highlight-run/apollo@3.4.43
  @highlight-run/hono@1.0.19
  @highlight-run/nest@3.6.28
@Vadman97 Vadman97 force-pushed the changeset-release/main branch from 591381e to 050c56c Compare July 9, 2025 22:51
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Incorrect Dependency Placement Causes Runtime Failures

The @launchdarkly/observability-shared dependency was incorrectly moved from dependencies to devDependencies. If this package is used at runtime, it will cause runtime errors for consumers because devDependencies are not installed in production environments.

sdk/@launchdarkly/observability-react-native/package.json#L29-L45

},
"dependencies": {
"@opentelemetry/api": "^1.9.0",
"@opentelemetry/exporter-trace-otlp-http": "^0.56.0",
"@opentelemetry/instrumentation": "^0.57.2",
"@opentelemetry/instrumentation-fetch": "^0.57.2",
"@opentelemetry/instrumentation-xml-http-request": "^0.57.2",
"@opentelemetry/resources": "^1.30.1",
"@opentelemetry/sdk-trace-web": "^1.30.1",
"@opentelemetry/semantic-conventions": "^1.30.0"
},
"peerDependencies": {
"@launchdarkly/react-native-client-sdk": "^10.0.0",
"react-native": ">=0.70.0"
},
"devDependencies": {
"@launchdarkly/observability-shared": "workspace:*",

Fix in CursorFix in Web


Was this report helpful? Give feedback by reacting with 👍 or 👎

@Vadman97
Copy link
Contributor

Vadman97 commented Jul 9, 2025

Bug: Incorrect Dependency Placement Causes Runtime Failures

The @launchdarkly/observability-shared dependency was incorrectly moved from dependencies to devDependencies. If this package is used at runtime, it will cause runtime errors for consumers because devDependencies are not installed in production environments.

quite intentional :) since we bundle the observability-shared into the react native package

image

confirmed the shared lib is bundled in

@Vadman97 Vadman97 merged commit 18170a5 into main Jul 9, 2025
17 checks passed
@Vadman97 Vadman97 deleted the changeset-release/main branch July 9, 2025 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant