Type afterCreate in factories

[Istanful]

When creating a factory when using typescript, the afterCreate hook
could not be defined as it is not a key of the model. This commit adds
typing for adding the afterCreate hook.

Unfortunately the exact type of the model is not inferred in this
commit. In order to make the factory fully infer the model, we would
need to create a dependency between the factory type and the model type,
which is something that seems to have been avoided purposefully.

IMHO the any model instance works good enough for now.

Fixes #791

[IanVS]

Thanks for this! I tested it out in my own project by copying the type file into my node_modules, and while it does allow me to remove my // @ts-expect-error comments (\o/), I start to get some other errors unfortunately.

• I now get a type error on my overall factories definition:
  'factories' implicitly has type 'any' because it does not have a type annotation and is referenced directly or indirectly in its own initializer.

• I'm no longer able to access attributes on the factory as I used to.
  
  vs previously:
  

• I seem to have lost the types when I use schema.all():
  
  vs previously:
  

I'm not sure what needs to be done to address these, but if you're able to take another crack at it, I'd love to give it another review. Just send me a ping to be sure I see it. Thanks again for your efforts!

[Nol-go]

Try this. I did not test it but I guess it should work

[Nol-go]

Suggested change

	type WithFactoryMethods<T> =
	| { afterCreate?: (model: ModelInstance<any>, server: Server) => void }
	| { [K in keyof Omit<T, "afterCreate">]: T[K] | ((n: number) => T[K]) };
	type WithFactoryMethods<T> = {
	afterCreate?: (model: ModelInstance<T>, server: Server) => void
	[K in keyof T]: T[K] | ((n: number) => T[K])
	};

[Nol-go]

Suggested change

	[K in keyof Omit<T, "afterCreate">]: T[K] extends (n: number) => infer V
	? V
	: T[K];
	[K in keyof T]: T[K] extends (n: number) => infer V ? V : T[K];