Skip to content

KT-84413: Add support for per-target configuration of sourceInformation and traceMarkers in Compose Compiler Gradle plugin#5695

Open
eymar wants to merge 1 commit intoJetBrains:masterfrom
eymar:rr/okarpovich/per_target_sourceinformation
Open

KT-84413: Add support for per-target configuration of sourceInformation and traceMarkers in Compose Compiler Gradle plugin#5695
eymar wants to merge 1 commit intoJetBrains:masterfrom
eymar:rr/okarpovich/per_target_sourceinformation

Conversation

@eymar
Copy link
Member

@eymar eymar commented Feb 19, 2026

By default, disable the sourceInformation and traceMarkers generation for non Android/JVM platforms to reduce the binaries size.

Fixes https://youtrack.jetbrains.com/issue/KT-84413/Add-per-target-configuration-for-Compose-Compiler-source-information-and-trace-markers

…ceMarkers in Compose Compiler Gradle plugin

By default, disable the sourceInformation and traceMarkers generation for non Android/JVM platforms to reduce the binaries size.
@eymar eymar requested a review from a team as a code owner February 19, 2026 14:28
@eymar eymar requested review from Tapchicoma and removed request for a team February 19, 2026 14:28
@kotlin-safe-merge
Copy link

Code Owners

Rule Owners Approval
/​libraries/​tools/​kotlin-​compose-​compiler/​ @JetBrains/kotlin-build-tools

@eymar
Copy link
Member Author

eymar commented Feb 19, 2026

cc: @ShikaSD @Tapchicoma

/**
* Include composition trace markers in the generated code for specific targets.
*
* This property takes effect only when [includeTraceMarkers] is true.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I wonder if we can detect if includeTraceMarkers is not set (value provided by convention) and only apply it then?

In general, I find this logic to be slightly confusing, but it is better than nothing.

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.

2 participants