-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature request: Support KSP #2349
Comments
Just updating this to give a sense of where we are at. This is a priority for us that we are looking into. There are some complications though mostly due to the SPI plugins which expose the JSR-269 APIs through our public interface. There’s also the fact that our annotation processor code base is huge. Because we still need to support our javac users, we cannot switch entirely to the KSP APIs which leaves us in a bit of a pickle since the places these are exposed are deep in our core model classes. I don’t think there is a possible way to get around this being a breaking API change, but we’ll need to figure out the right API to switch to that can support both KSP and javac SPI users. Anyway, all this is to say, this is a high priority and we are working on this, but it is also a complicated thing to rollout for a codebase our size. |
Can you use compiler-processing from Room to simultaneously support both? SPI users can, for now at least, be told they're unsupported in KSP (which seems fair considering the SPI API is javac-based anyway). |
Thanks for the pointer, we are already in discussion with that team about that as a possibility =) |
Hi any update on this? Thanks! |
Still working on it. We're preparing a new model that will wrap both javac and KSP types like in this recent change: 30fa0f3 |
I've opened a separate issue #2802 for that to track status and discuss possible solutions as my team is concerned about KSP support in SPI. |
@Jeffset, once we're ready to support KSP in Dagger, we will likewise support KSP in the SPI plugins as well. I think we can probably close #2802 , as this issue should track both. Just to give you more details on our plans for the SPI plugin: The new SPI model currently exists, and we are beginning to use it internally in Dagger. However, it is not yet available for users to write their own plugins. The new model will be a thin wrapper around the javac/ksp types. For example, Feel free to let us know if you have any concerns. |
KSP is officially 1.0! My last dep that uses kapt is dagger + hilt! Do we have any estimated timeline? Just curious if this is a "likely by the end of 2021" type of feature request, or a "likely by the end of 2022" type of feature request. 😅 |
Dagger maintainers are pretty good about posting updates, let's trust a push system rather than nagging about something they are obviously aware of and working on. |
Updating here since it has been some time. We've been making progress on the KSP migration, which includes making changes to Room's xprocessing libraries which we are using. Things were recently delayed for a while as the migration uncovered and exacerbated an existing issue which is the main change of the 2.41 release we just did. This issue caused a large delay not only in pulling people off the migration but also because we were afraid of making the problem worse by continuing the migration. For that reason, it is still hard to give any sort of estimate as it is really the unknowns that we're going to hit that will take the bulk of the time. Just that one issue discovered set us back a couple months. With that said, now that the 2.41 release is finally out, we are now unblocked to continue the migration work. |
any news here? |
any update on the progress? |
KSP support is something my team urgently needs. To the point that we are considering finding a solution other than dagger. |
@byates-fleetian, please take a look at @ZacSweers’s blog post to help keep the discussion about the actual content of the feature rather than trying to put pressure on the maintainers 🙏 |
Considering the number of votes on this, which I would put around 4-500, a better question might be: |
@TWiStErRob As usual in the OSS community – PRs are welcome ;) |
Quick Update: We’ve recently wrapped up the bulk of our migration to XProcessing -- a library that abstracts away the Javac APIs to allow Dagger’s processor to run in both Javac and KSP environments (see commits here). However, along the way, there was a number of places in our codebase where we had to fall back to using Javac when there was no XProcessing equivalent. Thus, we’re now working with the XProcessing team to add support for these missing features. Unfortunately, it’s unlikely that contributing PRs will help much in this phase of the project. The bulk of the work will be dedicated to designing the APIs, so any PRs would just shift our time from designing APIs to reviewing designs for APIs. In addition, this will require some level of coordination between the XProcessing and Dagger codebases, which will be easier to coordinate from our side. However, once the bulk of that work is done, we will need some help testing with real projects using KSP, so we’ll definitely let you all know once we get there. |
it's been a long time since last update. Any planned ETA? |
This is quite a blocker for compose. To use 1.3.0 I need Kotlin 1.7, but that removes kapt. I'm sure I can't be the only one. Is a resolution of this issue close? |
Hoping some good news this IO 🤞 |
I/O mentioned this progress, hope we could see the progress updates and ETA soon! |
ive been playing around with dagger snapshot since #3857 piqued my curiosity. looks like snapshot has ksp support?! |
Sorry to bother all subscribers. KAPT to KSP is one thing, but can we expect KMM soon after that or it is not planned yet ? |
If a community member wants to help with KSP migration, will his pull requests be accepted? |
@IlyaGulya there's only 3 issues there, the rest are PRs. See the "merge" vs circle icons on the left. Anything opened by copybara is an externalised changeset from their internal systems. |
@IlyaGulya Thanks for the offer, we really appreciate it. Unfortunately, it probably isn't so easy for community members to contribute to the KSP migration. We're tracking a lot of the issues we're discovering internally because they have stacktraces/information from internal codebases. Also, a lot of issues are actually across different parts of the stack, not just in Dagger, and so a lot of the work comes from coordinating fixes across those other projects. Additionally, there are extra complications from the process to push code from our internal systems out here that just makes it difficult for us to take large code contributions from the community in general. @bcorso is planning to update this thread soon with more details on where we're at and some of the issues we're currently working on. We know this has been going on for a long time, so we really appreciate all of your patience and support here. |
KSP Update:Hey all, it’s been a while since the last update, so thanks for being patient. In the past few months we’ve been collaborating with the KSP team to enable testing of Dagger’s KSP processor within Google so that we can test the processor on the many applications and libraries that are already using Dagger within Google's codebase. As a result of that work, we found a number of bugs that we’re still working through (Note: these aren’t just bugs in Dagger, but at multiple layers including kotlinc, KSP, XProcessing, and even the testing infra pipeline itself). I'll likely have another update once we've finished debugging and triaging those issues and have a better sense of which ones will block the official release. However, there is one breaking change to Dagger's API that will require changes from users to safely enable KSP, so I’ve added more details on that below. We're hoping that these changes will cause minimal churn for users and will provide an overall better experience when using Dagger in Kotlin, but please let us know if you have concerns. Breaking changes required to use Dagger KSP:With KSP, Dagger’s processor inspects the Kotlin source itself (rather than intermediate java stubs as when using KAPT). This makes it difficult to ensure correct behavior when switching from KAPT to KSP since the Kotlin types can elide use-site variance that were previously encoded explicitly in KAPT’s Java stubs. This is especially important in Dagger because Dagger currently relies on the exact type (wildcards included) to match a provided binding type to its requested types. To fix this, our initial approach was to try to create a mapping function that would allow us to transform a KSP type to its equivalent KAPT type (basically trying to infer the correct variance/wildcards of each type argument). While this mostly works, it’s also error-prone since there’s no good way to ensure we’ve correctly inferred all variances. In addition, getting the mapping wrong can lead to subtle (even silent) functional changes when migrating from KAPT to KSP (e.g. with Dagger's multibindings), which can prevent production applications from safely making the switch to KSP. In fact, even once a user has switched to KSP, future changes to the mapping (e.g applying a fix) has a liability of accidentally regressing other untested cases, making it risky to change. While eventually, we could add enough tests to get this mostly right, it’s a risk that we’d rather not subject our clients to. For these reasons, we have decided to change the way Dagger matches it's provided binding types to its request types, which should make it safe for clients to migrate to Dagger KSP:
As an additional benefit, once the changes above have been applied, Kotlin users will no longer need to add We’re hoping to roll out these flags (with further instructions on how to enable them) out within the next few months. At least initially, these flags will be optional for users who don't plan to migrate to KSP. |
Waiting for the good news, our team will move to KSP once Dagger is supported. |
Any news here? |
* Hilt is making slow progress in adapting to KSP.(google/dagger#2349) * Potential plans to support KMP in the future.
Hi all, Dagger 2.48 includes the alpha release of the Dagger and Hilt KSP processors. Please see the release notes for more details. Also, as an update on https://github.com/google/dagger/issues/2349#issuecomment-1549890357, the |
Thanks for the alpha release, great job. Does it mean that the "2nd part" is gonna generate Kotlin instead of Java code? |
@FunkyMuse, for the "2nd part" I mean this:
Ideally, the implementation of that feature would not require generating Kotlin source, but admittedly that is part of what makes it difficult. Fwiw, I know generating Kotlin source is of some interest due to KMP; however, just to be upfront, it's not something we are currently working on at the moment. |
…se Compiler 1.5.3, Koin 3.5.0, Dagger 2.48 with KSP, kmp-viewmodel 0.5.0, FlowExt 0.7.1 and refactor. (#90) * deps * Update build.gradle.kts kapt { correctErrorTypes = true } * Update deps.kt * a * deps * deps: hilt ksp google/dagger#2349 * fix ios * refactor android * jdk 17 * fix tests * gradle * gradle * gradle
For Kotlin projects, Dagger currently makes use of KAPT. This is generally quite slow since it requires generating Java stubs before the annotation processing step.
KSP offers an alternative to this by making it easier for annotation processors like Dagger to more directly use Kotlin's compiler plugin infrastructure. Based on initial benchmarks, using KSP can speed up build times by as much as 2x.
The text was updated successfully, but these errors were encountered: