Skip to content

Support StabilityInferred annotation for Gradle plugin#112

Merged
skydoves merged 1 commit intomainfrom
gradle/StabilityInferred
Feb 10, 2026
Merged

Support StabilityInferred annotation for Gradle plugin#112
skydoves merged 1 commit intomainfrom
gradle/StabilityInferred

Conversation

@skydoves
Copy link
Owner

@skydoves skydoves commented Feb 10, 2026

Support StabilityInferred annotation for Gradle plugin (#102)

Summary by CodeRabbit

  • Bug Fixes

    • Improved stability analysis in the compiler to more accurately evaluate cross-module type stability by properly considering StabilityInferred annotations on types.
  • Improvements

    • Enhanced parameter handling for stability inference annotations, providing more precise stability determination during compilation.

@skydoves skydoves self-assigned this Feb 10, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 10, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

The change implements full support for reading the @StabilityInferred annotation's parameters field to determine cross-module stability. A TODO placeholder is replaced with logic that maps parameter names to indices, extracts the bitmask value, and uses it to derive stability information.

Changes

Cohort / File(s) Summary
Stability Inference Enhancement
stability-compiler/src/main/kotlin/com/skydoves/compose/stability/compiler/lower/StabilityAnalyzerTransformer.kt
Implements reading of @StabilityInferred annotation and its parameters field to extract a bitmask for determining cross-module parameter stability. Replaces placeholder TODO with parameter name-to-index mapping and stability derivation logic.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Poem

🐰 Hopping through annotations with care,
Reading parameters floating in air,
Bitmasks of stability now align,
Cross-module checks divine! ✨
A TODO becomes real, crisp and fair.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch gradle/StabilityInferred

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

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

@skydoves skydoves merged commit 8926403 into main Feb 10, 2026
8 of 9 checks passed
@skydoves skydoves deleted the gradle/StabilityInferred branch February 10, 2026 09:50
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