kotlin-inject generates dead code

[vRallev]

This is one field being generated:

private val meteredCommunication: MeteredCommunication
  get() = _scoped.get("com.amazon.bivis.service.MeteredCommunication") {
    run<MeteredCommunication> {
      lateinit var meteredCommunication: MeteredCommunication
      provideMeteredCommunicationImpl(
        ...
      ).also {
        meteredCommunication = it
      }
    }
  }

Another example:

private val infoService: InfoService
  get() = run<InfoService> {
    lateinit var infoService: InfoService
    provideInfoServiceImplInfoService(
      infoServiceImpl = infoServiceImpl
    ).also {
      infoService = it
    }
  }

In both samples the lateinit var is redundant and not used. kotlin-inject should not generate dead code.

[evant]

Likely related to #449 because both contain lateinit which is used for breaking cycles. I might have mentioned this on another ticket but the cycle handling code is not great and needs to be re-written. You are also seeing more duplicate code using them as it was a quick fix against invalid code generation caused by trying to de-dup the common parts.