Use findContext for looking up app context

[skydoves]

Use findContext for looking up app context (#953)

Summary by CodeRabbit

Release Notes

• Refactor
  • Updated internal Activity resolution mechanism across balloon components, improving how the library locates and interacts with parent activity context.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 573b8b63-482e-4717-aba3-fd289139ee65

📥 Commits

Reviewing files that changed from the base of the PR and between d8a308b and 9c2e1ad.

📒 Files selected for processing (6)

• balloon-compose/src/main/kotlin/com/skydoves/balloon/compose/BalloonModifier.kt
• balloon/api/balloon.api
• balloon/src/main/kotlin/com/skydoves/balloon/Balloon.kt
• balloon/src/main/kotlin/com/skydoves/balloon/extensions/ContextExtension.kt
• balloon/src/main/kotlin/com/skydoves/balloon/extensions/ViewExtension.kt
• balloon/src/main/kotlin/com/skydoves/balloon/overlay/BalloonAnchorOverlayView.kt

---

Walkthrough

The changes refactor how the codebase retrieves Activity instances from Context. A new Context.findActivity(): Activity? extension function is added to safely traverse ContextWrapper chains and locate the underlying Activity, replacing direct Activity casts throughout the codebase.

Changes

Cohort / File(s)	Summary
New Extension Utility balloon/src/main/kotlin/com/skydoves/balloon/extensions/ContextExtension.kt	Added Context.findActivity(): Activity? extension function with @InternalBalloonApi and @JvmSynthetic annotations. Traverses ContextWrapper chain via baseContext to locate and return underlying Activity, returning null if not found.
API Surface Update balloon/api/balloon.api	Exposed new ContextExtensionKt class with synthetic findActivity(Context): Activity? method to public API.
Compose Modifier Update balloon-compose/src/main/kotlin/com/skydoves/balloon/compose/BalloonModifier.kt	Updated Modifier.balloon() to opt into @OptIn(InternalBalloonApi::class). Changed anchor parent resolution from (context as? Activity)?.window?.decorView to context.findActivity()?.window?.decorView.
Activity Cast Replacements balloon/src/main/kotlin/com/skydoves/balloon/Balloon.kt, balloon/src/main/kotlin/com/skydoves/balloon/extensions/ViewExtension.kt, balloon/src/main/kotlin/com/skydoves/balloon/overlay/BalloonAnchorOverlayView.kt	Replaced direct Activity casts with context.findActivity() extension calls. Updated null-safety checks and decorView access patterns to use the retrieved activity instance.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• Introduce Modifier.balloon() and BalloonState, deprecate Balloon() composable #904: Modifies the same Modifier.balloon implementation in BalloonModifier.kt, so changes are directly related at the code level.
• Add anchorParent and prevent measure for Constraints.Infinity #921: Updates anchor parent resolution and decor view determination in the Modifier.balloon implementation, affecting the same code regions.

Poem

🐰 A hop through contexts, we gently refine,
No casting about—just extensions divine!
ContextWrapper chains, we traverse with care,
Finding Activities nested with flair! ✨

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix/context-regression

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}