fix #4

[lorenzofox3]

Late bound factories will receive defined injectables.

Technically they could define new dependencies which would be resolved on the late as well, but this get contrived.

[sybers]

Exceptional 👏

[sybers]

I think arg here should not be optional

[lorenzofox3]

I think it should:

const createModule = createProvider({
    injectables:{
        someThing({random}: {random:number}) { return random * 42} 
    },
})

const myModule = createModule({
    random() {
        return Math.random();
    }
})

should be ok

[sybers]

Yes, if you make it mandatory your example above is still valid as

() => number

is compatible with

(arg: InjectableMap<Registry>) => ExternalDeps<Registry>['random']

On the other hand, this is not possible:

const createModule = createProvider({
    injectables:{
        someThing: ({ random }: { random: number }) => random * 42
        randomMin: () => 100
    },
})

const myModule = createModule({
    random(injectables) => injectables?.randomMin +Math.random()
})

above, the injectables type is InjectableMap<Registry> | undefined, so we have to use the ?. operator to access the value even if we know it will be defined (also because of that we can't destructure it).