Skip to content

[client][Android] Prevent some debug only artifacts from being included in release builds#41378

Merged
lukmccall merged 3 commits intomainfrom
@lukmccall/client/remove-from-release
Dec 4, 2025
Merged

[client][Android] Prevent some debug only artifacts from being included in release builds#41378
lukmccall merged 3 commits intomainfrom
@lukmccall/client/remove-from-release

Conversation

@lukmccall
Copy link
Copy Markdown
Contributor

Why

Fixes #41263.

How

Went through the dev-menu and dev-launcher code and ensured that all unwanted code, assets and dependencies will be removed in the release.

Test Plan

  • build bare-expo in
    • debug ✅
    • debugOptimized ✅
    • release ✅

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Dec 3, 2025

Subscribed to pull request

File Patterns Mentions
packages/expo-dev-launcher/** @douglowder, @gabrieldonadel
packages/expo-dev-menu/** @douglowder, @gabrieldonadel

Generated by CodeMention

@expo-bot expo-bot added the bot: suggestions ExpoBot has some suggestions label Dec 3, 2025
@alanjhughes
Copy link
Copy Markdown
Collaborator

@lukmccall it will still work in expo go in release?

@lukmccall
Copy link
Copy Markdown
Contributor Author

@lukmccall it will still work in expo go in release?

Not yet, it'll be added in the next PR.

@lukmccall lukmccall force-pushed the @lukmccall/client/remove-from-release branch from 6611337 to d48af82 Compare December 4, 2025 10:49
@expo-bot expo-bot added bot: passed checks ExpoBot has nothing to complain about and removed bot: suggestions ExpoBot has some suggestions labels Dec 4, 2025
@expo-bot
Copy link
Copy Markdown
Collaborator

expo-bot commented Dec 4, 2025

The Pull Request introduced fingerprint changes against the base commit: ed5035a

Fingerprint diff
[
  {
    "op": "changed",
    "beforeSource": {
      "type": "dir",
      "filePath": "../../packages/expo-dev-launcher",
      "reasons": [
        "expoAutolinkingIos",
        "expoAutolinkingAndroid"
      ],
      "hash": "4afe0249e38331c00990d1bebc4ac54740b0ff97"
    },
    "afterSource": {
      "type": "dir",
      "filePath": "../../packages/expo-dev-launcher",
      "reasons": [
        "expoAutolinkingIos",
        "expoAutolinkingAndroid"
      ],
      "hash": "a2516dd2906d1e6b034b6b36d2a79ec0b6471754"
    }
  },
  {
    "op": "changed",
    "beforeSource": {
      "type": "dir",
      "filePath": "../../packages/expo-dev-menu",
      "reasons": [
        "expoAutolinkingIos",
        "expoAutolinkingAndroid"
      ],
      "hash": "d45e48f64be9f8fa5c89b2a9c313293c63757eb3"
    },
    "afterSource": {
      "type": "dir",
      "filePath": "../../packages/expo-dev-menu",
      "reasons": [
        "expoAutolinkingIos",
        "expoAutolinkingAndroid"
      ],
      "hash": "212deeed099d254387b2c13f61d175527e1ec92f"
    }
  }
]

Generated by PR labeler 🤖

@lukmccall lukmccall merged commit b165906 into main Dec 4, 2025
14 checks passed
@lukmccall lukmccall deleted the @lukmccall/client/remove-from-release branch December 4, 2025 14:04
aleqsio pushed a commit that referenced this pull request Dec 5, 2025
@hraschan
Copy link
Copy Markdown

hraschan commented Mar 6, 2026

@lukmccall This is a major security issue. Is there a possibility to have the fix in expo 54? We can't upgrade at the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot: fingerprint changed bot: passed checks ExpoBot has nothing to complain about

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Android][expo-dev-client]Release Build contains expo-dev-client specific artefacts

5 participants