@cllns I think you're onto something here. It actually relates to some of the work I've been doing lately too, with #197 adding a hook for Zeitwerk to do its own eager-loading when we finalize the container in the production mode. So yes, I do wonder if we need to build some kind of more general-purpose "eager load" step into the finalization process. I think it's definitely in keeping with the spirit of finalization.

So I'm open to exploring this idea. In the meantime, however, I think #147 is still great to have since we'll want that for the cases when we're not eager loading, such as when the containers remain non-finalized.

I think it's worth getting this into 1.0.0. Getting confusing exceptions can be frustrating for the users, especially people who are trying out dry-system/hanami2 for the first time. Using acronyms in names is common, so this may bite many people.

@solnic FYI, we've already done the work to provide clear error messages via @cllns' work in #217.

This issue is about whether we should try and access the constant for every auto-registered component as part of the finalization process, instead of waiting for those components to be later resolved.

This would slow down finalization somewhat, but would have the benefit of bringing error messages forward in time.

@timriley yeah I know thanks! Verifying constants is a good thing to do, mismatches will happen. We could maybe either have it done by default or have a special verbose/verification mode that you can enable to verify your setup.

I was thinking we could just do this at finalization, but allow opt-out, eg.

Container.finalize(eager_load: false) # defaults to true

What do you think?

Makes sense!