Skip to content

chore(react-native-plugin): drop posthog-android's screenshot option - #5479

Open
turnipdabeets wants to merge 1 commit into
chore/rn-plugin-posthog-android-4-pinfrom
chore/rn-plugin-drop-android-screenshot
Open

turnipdabeets wants to merge 1 commit into
chore/rn-plugin-posthog-android-4-pinfrom
chore/rn-plugin-drop-android-screenshot

Conversation

@turnipdabeets

Copy link
Copy Markdown
Contributor

Important

Depends on PostHog/posthog-android#908 (remove wireframe replay) shipping in posthog-android 4.0. Stacked on #5476 (posthog-android 4.0 pin). If PostHog/posthog-android#908 is dropped from 4.0, close this PR. Without it, removing this line makes Android replay fall back to wireframe.

Problem

posthog-android 4.0 removes wireframe session replay (PostHog/posthog-android#908). Replay always records masked screenshots, and PostHogSessionReplayConfig.screenshot is removed. The plugin still sets sessionReplayConfig.screenshot = true, which won't compile against 4.0.

This is the Android counterpart of #5236 (iOS screenshotMode).

Changes

  • Remove sessionReplayConfig.screenshot = true from the Android plugin module.
  • Changeset (patch). Android replay already records screenshots, so replay doesn't change.

⚠️ Merge order

Merge into #5476 (or after it, into #5194). #5476 is already waiting on posthog-android 4.0 being published.

Release info Sub-libraries affected

Libraries affected

  • All of them
  • posthog-js (web)
  • posthog-js-lite (web lite)
  • posthog-node
  • posthog-react-native
  • @posthog/react-native-plugin
  • @posthog/react
  • @posthog/ai
  • @posthog/convex
  • @posthog/next
  • @posthog/nextjs-config
  • @posthog/nuxt
  • @posthog/openfeature-node-provider
  • @posthog/openfeature-web-provider
  • @posthog/rollup-plugin
  • @posthog/webpack-plugin
  • @posthog/types
  • @posthog/browser-common

Checklist

  • All commits are signed (see Commit signing)
  • Tests for new code
  • Accounted for the impact of any changes across different platforms
  • Accounted for backwards compatibility of any changes (no breaking changes!)
  • Took care not to unnecessarily increase the bundle size

If releasing new changes

  • Ran pnpm changeset to generate a changeset file

How did you test it?

Not built locally. The change is one deleted line, and the plugin doesn't reference screenshot anywhere else. CI can't build it until posthog-android 4.0.0 is on Maven Central, which is the same blocker as #5476.

🤖 Agent context

Autonomy: Human-driven (agent-assisted)

Written with Claude Code (Opus 5.5) at the maintainer's request, alongside PostHog/posthog-android#908 and the KMP follow-up.

🤖 Generated with Claude Code

posthog-android 4.0 removes wireframe replay and the `screenshot` flag;
replay always records masked screenshots.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

Coherence

0 spec problems · 0 failing chokepoints · 2 spec gaps · 0 guard failures

44 components, 7 bullets (5 invariants, 2 requirements), 0 problems

Chokepoints: none declared.

Spec gaps: 2 entrances with outside or unknown trust and no traced control, and 10 more held by the adoption baseline

  • packages/web/Web.spec.md entrance public api (trust host-app): packages/web
  • packages/webpack-plugin/Webpack-plugin.spec.md entrance webpack plugin (trust build): packages/webpack-plugin

Close one with guard: <chokepoint>, an invariant that names it on its entrances: line, or control: none — <reason>; scaffold control proposes it.

@turnipdabeets
turnipdabeets marked this pull request as ready for review October 29, 2026 14:51
@turnipdabeets
turnipdabeets requested a review from a team as a code owner October 29, 2026 14:51

This branch has not been deployed

No deployments
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