Map multibindings

[matejdro]

Any chance a support for contributing to a map multibinding could be added?

Would adding a key property to the contributes annotation work?

[vRallev]

I don't see why not.

[bddckr]

This is the last reason I can't move my projects away from Dagger+Anvil yet.

Is anything required to get the linked PR over the finish line and get a release out with it? I can see there's a question in the PR to add support for LazyClassKey, but IMO that can be added another time. The PR seems to resolve the original request of getting maps working for multibindings.

[vRallev]

I'm sorry for the delay, but this gave me a lot of time to think about this request and #86.

While Map multibindings as a feature makes sense, the the proposed changes would significantly increase the library's API surface, which I'm hesitant to do for long-term maintainability. We don't use this feature and I'm concerned about the potential for future complexity and the ongoing maintenance burden.

But the good thing is, this could be easily implemented as an extension and doesn't need to be part of the core library: https://github.com/amzn/kotlin-inject-anvil?tab=readme-ov-file#custom-symbol-processors For now, I suggest going this route.

[matejdro]

This would be kinda funny and complex situation. Having an addon for an addon for an addon, a 3-tiered dependency injection solution. But I understand your point, thanks for clarifying.