Skip to content

Describe issue with using compiler plugin in lambdas with DslMarker receivers#1669

Merged
koperagen merged 4 commits intomasterfrom
plugin-workaround-android
Feb 4, 2026
Merged

Describe issue with using compiler plugin in lambdas with DslMarker receivers#1669
koperagen merged 4 commits intomasterfrom
plugin-workaround-android

Conversation

@koperagen
Copy link
Collaborator

Related to #1604, but i'd like to add a warning to resolve it. For now here's a description with a workaround.
While investigating i found that we don't use remember where we should to avoid unnecessary re compositions

And small UI problem. I think it's only applicable to Preview, but worth to fix anyway
image

image

Lastly i couldn't import project in Android studio, so i updated AGP version according to its recommendation

@koperagen koperagen added this to the 1.0.0-Beta5 milestone Jan 26, 2026
@koperagen koperagen self-assigned this Jan 26, 2026
@koperagen koperagen added documentation Improvements or additions to documentation (not KDocs) examples Something related to the examples labels Jan 26, 2026
@Jolanrensen
Copy link
Collaborator

Also related to #1610 a bit :)

@Jolanrensen
Copy link
Collaborator

When writing pages for the website, please check you write full sentences with punctuation. Otherwise it looks good :)
Clever to use remember() {} in Compose on Android! It's a good practice indeed :)

@koperagen koperagen force-pushed the plugin-workaround-android branch from 2955428 to 24f50a8 Compare January 27, 2026 12:31
@DataSchema
data class Person(val age: Int, val name: String)

@Composable
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be clearer if you split this example because it now contains the bug and the solution interwoven with each other.

I'd first write the reproducer containing the bug, then separately write the workaround, so it's easy to see the difference.

@koperagen koperagen force-pushed the plugin-workaround-android branch 2 times, most recently from 97bd47a to 17bcaab Compare January 28, 2026 15:44
@koperagen koperagen force-pushed the plugin-workaround-android branch from 17bcaab to b5848a0 Compare February 4, 2026 22:59
@koperagen koperagen merged commit 2666ba9 into master Feb 4, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation (not KDocs) examples Something related to the examples

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants