Found a bug in Collectors (#50). All collectors redundantly copied a bug where the Promise was not awaited.

https://github.com/sam-goodwin/punchcard/blob/d476533a032a9dfee9fecfffd7a852744f63d288/packages/punchcard/lib/kinesis/collector.ts#L63-L68

Bug is fixed, but we should make a generic SinkCollector for all types that implement Sink<T>.

There's a warning note in README.md detailing:

Warning - the AnalyzeDocument process from AWS Textract costs $50 per 1,000 PDF pages. Be careful when deploying this CDK stack as you could unintentionally rack up an expensive AWS bill quickly if you're not paying attention.

This might not be enough - if a user finds this project and doesn't read the documentation, they could inadvertently

It'd be helpful to be able to receive notifications when snapshot exports complete, which customers can use to trigger other workflows or simply become aware that the exports are done.

See docs