amazon-kinesis-video-streams-webrtc-sdk-js icon indicating copy to clipboard operation
amazon-kinesis-video-streams-webrtc-sdk-js copied to clipboard

Hidden peer dependency / process.env reference

Open fochlac opened this issue 7 months ago • 0 comments

Hi,

I'm trying to use your library as a npm package, bundle it for the browser and I'm importing from it as follows: import { Role, SignalingClient } from 'amazon-kinesis-video-streams-webrtc' If I want to compile my library I now need to do two things that are not documented, I need to add the dependency 'events' and add an alias for process.env.PACKAGE_VERSION. While this works, its not a nice DX and it would be better to ship the lib in a way that the entry point has all process.env.variables replaced and events is at least listed as peer dependency. If this is not possible it would be good to add a note on those requirements if bundling for the browser. See this code-sandbox and the following screenshots: image image

fochlac avatar Jul 10 '24 21:07 fochlac