Introduce Defer to module loader

[radeksimko]

@Nepomuceno Do you think this would help in #551 ?

I guess the Enqueue method could return the operation and make the caller do the actual queueing and potentially setting Defer function instead of passing it as argument, but that's mostly a stylistic concern that can be refactored later.

If this helps solving the problem I'd ship it. 🤷🏻‍♂️

[radeksimko]

The idea is that you'd modify the following line(s) in the watcher and walker:

w.modMgr.EnqueueModuleOp(mod.Path, op.OpTypeParseModuleManifest, nil)

to async enqueue all the other relevant/dependent operations, e.g.

w.modMgr.EnqueueModuleOp(mod.Path, op.OpTypeParseModuleManifest, func(opErr error) {
    if opErr != nil {
        return
    }
    // TODO: range over module calls to get modPath for each
    w.modMgr.EnqueueModuleOp(modPath, op.OpTypeParseModuleConfiguration, nil)
    w.modMgr.EnqueueModuleOp(modPath, op.OpTypeParseVariables, nil)
    w.modMgr.EnqueueModuleOp(modPath, op.OpTypeLoadModuleMetadata, nil)
    w.modMgr.EnqueueModuleOp(modPath, op.OpTypeDecodeReferenceTargets, nil)
    w.modMgr.EnqueueModuleOp(modPath, op.OpTypeDecodeReferenceOrigins, nil)
})

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.