Remove SST runtime integration#61
Conversation
There was a problem hiding this comment.
Code Review
This pull request removes the dependency on SST (Serverless Stack) across the monorepo, replacing auto-generated SST resources with local environment-variable-based configurations and updating references from SST to EmberHarmony. The code review identified a critical authentication bypass vulnerability in the API's admin secret verification when the environment variable is undefined. Additionally, several medium-severity issues were found in the new environment variable helper functions in drizzle.config.ts and resource.node.ts, where empty strings (such as empty database passwords or ports) are incorrectly treated as falsy, potentially causing configuration failures or defaulting the database port to 0.
Summary
Verification
Notes