History

Kayos 10154c380b vc=42: loop round 4/5 — surgical cleanup, round 7 hit diminishing returns Round-7 Opus audits explicitly flagged the audit loop as past diminishing returns. Landing the few real items + one race-order fix; leaving the comment-hygiene tail for the next time someone reads the files. MED R7-1 SearchViewModel cache-preview race: vc=41 wrote the cached preview to UI before cancelling the prior inFlight, so the prior coroutine (already past its ensureActive() gate) could reach its terminal _ui.update and clobber the cache preview. Moved inFlight?.cancel() above the preview write. R7-2 StrawActivity.YT_HOSTS duplicated util.YtUrl.ALLOWED_YT_HOSTS — drift risk where one gets a new host and the other doesn't. Collapsed StrawActivity.looksLikeYouTube to call util.isAllowedYtUrl, picking up the scheme + trailing-dot defenses for free. R7-3 Dropped dead once_cell dep from rust/strawcore/Cargo.toml. Round-4's runtime rewrite uses AtomicBool + Mutex; nothing consumes once_cell anymore. Audit explicitly called out (and we agree) these as past the value threshold for further rounds: - Verified-clean: try_lock no deadlock, ensureActive fence, bad-URL early-return cancel, duplicate-zip-entry rejection, LIMIT clauses, SponsorBlock 50ms exclusion drop, applied++ count. - False positive: H1 "Related videos always empty" — the section already gates on `d.related.isNotEmpty()`; UI doesn't paint when extractor stub returns empty. - Deferred: R8 enable, Nav rememberSaveable, LazyColumn keys, collectAsStateWithLifecycle, DASH/HLS max-resolution cap, Gradle cargo-build input/output declarations (CI cost, not runtime), legacy :app module trim, stale comment cleanup.		2026-05-25 15:36:00 -07:00
..
strawcore	vc=42: loop round 4/5 — surgical cleanup, round 7 hit diminishing returns	2026-05-25 15:36:00 -07:00
Cargo.toml	v0.1.0-V (vc=9): U-3 — streamInfo via rustypipe drives VideoDetail+Player	2026-05-24 08:52:43 -07:00
README.md	v0.1.0-U (vc=8): Phase U-1 + U-2 — Rust core + rustypipe search	2026-05-24 08:36:50 -07:00

README.md

`rust/` — strawcore: Rust YouTube core for Straw

Phase U- of the Straw build. Goal: replace the Java NewPipeExtractor dependency with a Rust core (rustypipe + tokio + reqwest), exposed to the Kotlin/Compose UI via UniFFI. Compose UI stays in Kotlin — only the YouTube/Innertube fetching layer moves to Rust.

Phases

Phase	What
U-1	Toolchain + UniFFI smoke test (`hello_from_rust`) round-tripping through JNA. No real APIs yet.
U-2	rustypipe search. `SearchViewModel` calls the Rust core.
U-3	rustypipe streamInfo + streams. `VideoDetailViewModel` + `PlayerViewModel` use it.
U-4	rustypipe channel + tabs. `ChannelViewModel` + `SubscriptionFeedViewModel`.
U-5	Rip NewPipeExtractor Java dep. Measure APK + cold-fetch latency before/after.
U-6 (stretch)	SponsorBlock + RYD HTTP through reqwest + tokio in the same lib.

Build chain

crafting-table
├── rustup stable (target add: aarch64-linux-android, armv7-linux-androideabi,
│                  x86_64-linux-android, i686-linux-android)
├── cargo-ndk      (cross-compile helper)
├── android-sdk    (ANDROID_HOME, sdkmanager, build-tools, platforms)
└── android-ndk    (ANDROID_NDK_HOME, r27c LTS at /caches/android-sdk/ndk/...)

Gradle (strawApp/build.gradle.kts)
├── cargoBuild         Exec task → cargo ndk -t <abi>... -o jniLibs/ build --release
├── uniffiBindgen      Exec task → cargo run --bin uniffi-bindgen ... --library libstrawcore.so
└── source-set wiring  generated Kotlin lands in strawApp/src/main/java/uniffi/strawcore/

Runtime (StrawApp.onCreate)
├── System.loadLibrary("strawcore")
└── uniffi.strawcore.initLogging()

Why UniFFI (and not raw JNI / JNA bindings)

Hand-written JNI: tedious, error-prone, every type change is two files (Kotlin + Rust) that must stay in sync.
Raw JNA: better, but you still hand-write the Kotlin side and worry about string ownership.
UniFFI: write Rust, annotate with #[uniffi::export], get a Kotlin shim generated. Strings, structs, enums, Result types, async functions all cross the boundary transparently. The runtime is JNA under the hood.

When in doubt

cargo check -p strawcore --target aarch64-linux-android — fast iteration.
cargo run --bin uniffi-bindgen -- generate ... — regenerate Kotlin bindings.
adb logcat -s strawcore — Rust log::info!() lands here.
aapt dump badging strawApp/build/outputs/apk/debug/strawApp-debug.apk — inspect what ABIs/native-libs the APK carries.