Skip to content

init: add the INTERNET permission Flutter/Android release builds need (0.3.3) - #15

Merged
bazingga08 merged 1 commit into
mainfrom
fix/flutter-internet-1109
Nov 9, 2026
Merged

bazingga08 merged 1 commit into
mainfrom
fix/flutter-internet-1109

Conversation

@bazingga08

Copy link
Copy Markdown
Contributor

strait init for Flutter left out android.permission.INTERNET: Flutter's template only declares it for debug and profile builds, so a release build can't reach Strait (QA 9 Nov 2026: reason=network, 0 rows). Android Studio templates have none at all, so Kotlin apps get it too. React Native and Expo templates already have it.

  • Adds <uses-permission android:name="android.permission.INTERNET" /> right after <manifest> when missing; idempotent; keeps indent and CRLF; ignores a commented-out copy.
  • Listed under "Edited:"; --dry-run shows it and writes nothing; a refused run (existing lib/links.dart) writes nothing.
  • No manifest found: the printed manifest step includes the line.
  • README ("It never edits your existing files" corrected), CHANGELOG, version 0.3.3.

Tests: +6 (red first: missing export), 96 pass / 2 skipped.
Real thing: fresh flutter create (Flutter 3.47.9) → strait init → main manifest 0 → 1 INTERNET (second run unchanged) → flutter build apk --release → aapt2 dump permissions lists INTERNET (ops/run-1109/cli-web-followups/cli-flutter-release-proof.txt).

🤖 Generated with Claude Code

…lds; 0.3.3

Flutter's template declares android.permission.INTERNET only in the debug and profile
manifests (an Android Studio template has none), so a release build set up with
`strait init` could not reach Strait: no deferred match, no install row (QA 9 Nov 2026).

init now adds the one line after <manifest> when it is missing: once (a second run
leaves the file alone), matching the file's indent and line endings, ignoring a
commented-out copy. Listed under "Edited:"; --dry-run shows it without writing; a run
refused for an existing integration file writes nothing. With no manifest found, the
printed manifest step carries the line. React Native and Expo are unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@bazingga08
bazingga08 merged commit 36a51a0 into main Nov 9, 2026
4 checks passed
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