Fredrik Kjellberg
02/14/2022, 5:20 PMsst-env
script to fail when trying to find the SST project. There is a PR that fixes this issue from @Ivo Evans-Storrie waiting for review. It would be great if this PR (or some other way of solving the issue) could be included in an upcoming release. https://github.com/serverless-stack/serverless-stack/pull/1224Jay
Frank
thdxr
02/15/2022, 2:41 PMsst-env
utility. I think it could be more into a more general tool and ideally not a seperate cli toolFrank
Frank
ideally not a seperate cli toolWould that require sst to be installed globally?
thdxr
02/15/2022, 2:45 PMthdxr
02/15/2022, 2:45 PMFrank
@serverless-stack/cli
as a dependency and do npx sst env -- …
insteadthdxr
02/15/2022, 2:50 PMthdxr
02/15/2022, 2:51 PMFredrik Kjellberg
02/15/2022, 3:38 PMsst-env
script in the @serverless-stack/cli
utility? The reason is that the `sst-env`tool is really lightweight with its 60 lines of code or something. I created a PR (https://github.com/serverless-stack/serverless-stack/pull/1347) to cleanup the current dependencies that carries over 450 packages into node_modules
when you want to use only the sst-env
tool.
The purpose of the tool is to just inject the necessary configuration from the related SST project into e.g. a React app that is otherwise completely standalone from the SST project. I don't see the benefit of including all those dependencies from SST into a frontend app if the only functionality needed is the `sst-env`script.Jay
Fredrik Kjellberg
02/15/2022, 4:31 PMnpm
and pnpm
. Pnpm will hide all those dependencies with its strict non-flat node_modules
structure but for smaller npm
based single repos I think that having sst-env
include hundreds of irrelevant dependencies is an issue.Fredrik Kjellberg
02/15/2022, 4:32 PMpnpm
, I have run into some issues with the SST dependency tree due to conflicting versions within the SST application itself. I think having the dependency tree as clean as possible would make it simpler for downstream consumers of SST.thdxr
02/15/2022, 4:36 PMFredrik Kjellberg
02/15/2022, 4:51 PMthdxr
02/15/2022, 4:54 PMFrank
Frank