Enhance LocalStack integration and improve test configurations#1
Merged
Enhance LocalStack integration and improve test configurations#1
Conversation
…lobal setup and teardown for tests
…status in the UI Parse the LocalStack /_localstack/health response to extract which services are actually running. The health endpoint now returns a `services` array containing only the services that are both enabled in the backend config and reported as "running" or "available" by LocalStack. The frontend Sidebar and Dashboard consume this data to visually disable services that are configured but not currently active on the LocalStack instance (reduced opacity, non-clickable, tooltip on collapsed sidebar). Additional changes included in this commit: - Add app icons (favicon, apple-touch-icon, PWA manifest) and a Vite plugin to copy them into the public directory at build time - Copy desktop icon in the Electron build script - Fix IAM route response schemas (DeleteResponseSchema → MessageResponseSchema) - Add isDefaultVersion field to IAM policy version response - Add v8 ignore hints for unreachable fallback expressions in S3, SNS, SQS - Update .gitignore for generated icon assets - Update health unit, integration, and index tests to match the new response shape
Separate the Fastify app factory (buildApp in index.ts) from the server bootstrap logic (main in server.ts) so that index.ts is a pure library module with no side effects, making it easier to import in tests and other consumers. - Move main() from index.ts to new server.ts with top-level error handling - Update package.json dev/start scripts to use server.ts/server.js - Update Dockerfile ENTRYPOINT to node server.js - Add .nvmrc pinned to Node 24
|
🎉 This PR is included in version 1.1.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request makes several significant improvements to the backend package, focusing on refactoring the server startup process, enhancing health check logic, improving configuration handling, and updating tests for better coverage and reliability. The changes also include some minor dependency and Dockerfile adjustments.
Key changes include:
Backend Server Refactor and Startup
index.tsto a newserver.tsfile, and updated the Dockerfile and scripts to useserver.jsas the entrypoint. ThebuildAppfunction inindex.tsnow returns a Fastify instance for easier testing and modularity. [1] [2] [3] [4] [5] [6]Health Check Logic Improvements
Configuration and Testing Enhancements
Dependency and Code Quality Updates
package.json, including new dev dependencies for testing and coverage, and reordered some dependencies for consistency. Addedv8 ignorecomments in several places to improve code coverage reporting. [1] [2] [3] [4] [5]Minor Fixes and Cleanups
isDefaultVersionto IAM policy version responses for completeness.aws/clients.tsfile..nvmrcto specify Node.js version 24.iconsdirectory.These changes collectively improve the maintainability, testability, and reliability of the backend service.