Remove deprecated releases service and related tests #30
Conversation
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
The legacy `/releases/v1` service and all associated code, tests, and mocks have been removed. Documentation and type definitions were updated to reflect that only the versions and central-alerts services remain. This simplifies the codebase and test suite, as the releases service is no longer supported.
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
api-worker | 8b15ac9 | Commit Preview URL Branch Preview URL |
Dec 24 2025, 12:08 AM |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the deprecated /releases/v1 service and all related code, tests, and documentation. The legacy service has been superseded by the more comprehensive /versions/v1 service. As a bonus, CORS middleware is added to the central-alerts service for consistency with the versions service.
- Complete removal of the
/releases/v1service implementation and route registration - Deletion of all associated test files and mock data for the releases service
- Updates to test files to remove all references to the deprecated service
- Documentation updates reflecting that only two services (versions and central-alerts) remain active
Reviewed changes
Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/app/index.ts | Removed import and route registration for the deprecated releases service |
| src/services/releases/v1/index.ts | Deleted entire releases service implementation |
| src/services/releases/v1/interfaces.ts | Deleted releases service type definitions |
| src/services/releases/v1/README.md | Deleted releases service documentation |
| src/services/central-alerts/v1/index.ts | Added CORS middleware for consistency with versions service |
| test/services/releases/v1/index.test.ts | Deleted comprehensive test suite for releases service |
| test/mocks/releases.ts | Deleted mock data used for releases service testing |
| test/mocks/README.md | Updated to remove reference to deleted releases mock file |
| test/utils/test-types.ts | Removed ReleasesResponse and ReleaseVersion type definitions |
| test/integration/app.test.ts | Updated integration tests to remove all releases service test cases and references |
| test/app/index.test.ts | Updated app tests to remove releases service routing and communication tests |
| README.md | Updated main documentation to reflect two services instead of three |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The legacy
/releases/v1service and all associated code, tests, and mocks have been removed. Documentation and type definitions were updated to reflect that only the versions and central-alerts services remain. This simplifies the codebase and test suite, as the releases service is no longer supported.