8000 Use XPropertyElement.hasInaccessibleAnnotations() instead of parsing Kotlin metadata. by copybara-service[bot] · Pull Request #5337 · google/dagger · GitHub
[go: up one dir, main page]

Skip to content

Use XPropertyElement.hasInaccessibleAnnotations() instead of parsing Kotlin metadata. - #5337

Merged
copybara-service[bot] merged 1 commit into
masterfrom
test_995869306
Nov 5, 2026
Merged

copybara-service[bot] merged 1 commit into
masterfrom
test_995869306

Conversation

@copybara-service
Copy link
Copy Markdown

Use XPropertyElement.hasInaccessibleAnnotations() instead of parsing Kotlin metadata.

Dagger and Hilt still parse @kotlin.Metadata themselves for one thing: detecting when a Kotlin property's synthetic $annotations method is missing, which happens for properties of classes from other compilations (https://youtrack.jetbrains.com/issue/KT-34684). XProcessing now exposes this as XPropertyElement.hasInaccessibleAnnotations(), so use that and delete ClassMetadata, KotlinMetadata, and KotlinMetadataFactory from both compilers.

RELNOTES=N/A

@copybara-service
copybara-service Bot force-pushed the test_995869306 branch 4 times, most recently from 32f5e36 to c062d01 Compare November 5, 2026 08:11
…Kotlin metadata.

Dagger and Hilt still parse `@kotlin.Metadata` themselves for one thing: detecting when a Kotlin property's synthetic `$annotations` method is missing, which happens for properties of classes from other compilations (https://youtrack.jetbrains.com/issue/KT-34684). XProcessing now exposes this as `XPropertyElement.hasInaccessibleAnnotations()`, so use that and delete `ClassMetadata`, `KotlinMetadata`, and `KotlinMetadataFactory` from both compilers.

RELNOTES=N/A
PiperOrigin-RevId: 1014467854
@copybara-service
copybara-service Bot merged commit 9ff246c into master Nov 5, 2026
4 checks passed
@copybara-service
copybara-service Bot deleted the test_995869306 branch November 5, 2026 08:34
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

0