Sync major improvements: service fixes, test coverage, and dependency updates #78
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
- Frontend: React 18→19, Vite 5→7, React Router 6→7 - Backend: Express 4→5, Jest 29→30, Node types 20→24 - Update deprecated Jest matchers to modern syntax - All builds and functionality verified Co-authored-by: Ona <[email protected]>
- Remove third-party jira-mcp that was failing to build due to tsc dependency conflicts - Replace with official Atlassian MCP server (https://mcp.atlassian.com/v1/sse) - Use OAuth 2.1 authentication instead of API tokens - Support Jira, Compass, and Confluence through single endpoint - Eliminate TypeScript build failures in devcontainer setup This resolves devcontainer build failures caused by the 'tsc' npm package conflicting with the TypeScript compiler in the third-party jira-mcp project. Co-authored-by: Ona <[email protected]>
- Remove problematic jira-mcp service from automations.yaml - Replace setupJiraMCP task with setupAtlassianMCP task - Fix MCP configuration to use mcp-remote proxy instead of server-fetch - Update all MCP config files to use correct npx mcp-remote command - Remove dependency on third-party jira-mcp repository entirely This completes the migration to the official Atlassian MCP server with proper OAuth 2.1 authentication and eliminates all build issues. Co-authored-by: Ona <[email protected]>
…n errors Co-authored-by: Ona <[email protected]>
- Delete all MCP configuration files (.mcp/, mcp.json, .mcp.yaml) - Remove MCP setup from devcontainer setup.sh - Remove setupAtlassianMCP task from automations.yaml - Clean slate - no more MCP/Jira dependencies or build issues Co-authored-by: Ona <[email protected]>
- Add explicit service dependencies (catalog->postgres, frontend->catalog) - Fix race conditions in service startup sequence - Enhance error handling with service status checks and logs - Improve database seeding to wait for PostgreSQL service readiness - Add comprehensive system status checker for troubleshooting - Ensure proper startup order for both fresh and idle environment restarts Co-authored-by: Ona <[email protected]>
Backend Coverage (100%): - Add comprehensive unit tests for catalogService with database mocking - Add complete test suite for movieUtils (validation, formatting, filtering) - Add thorough dataProcessor tests (sanitization, batch processing) - Add API route integration tests with supertest - Configure Jest with coverage thresholds and reporting Frontend Coverage (93.21%): - Add complete YouTube utilities test suite (video ID extraction, embed URLs) - Add comprehensive API service tests with fetch mocking - Add advanced search service tests with caching and localStorage - Configure Vitest with jsdom environment and v8 coverage - Add debounce function testing and error handling scenarios Test Infrastructure: - Set up Jest + TypeScript for backend testing - Set up Vitest + JavaScript for frontend testing - Add coverage reporting with HTML and text output - Implement comprehensive mocking strategies - Add edge case and error scenario testing - Ensure all async operations are properly tested Co-authored-by: Ona <[email protected]>
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.
Major GitpodFlix Improvements Sync
This PR syncs significant improvements from the gitpod-SE fork, including critical service fixes, comprehensive test coverage, and dependency updates.
🔧 Critical Fixes
Service Startup Ordering (5dae998)
Dependency Updates (fd63bc9)
🧪 Test Coverage Achievement (e42b1f1)
Backend Coverage: 100% ✅
Frontend Coverage: 93.21% ✅
🧹 Cleanup & Maintenance
MCP/Jira Integration Removal (4eed801, 7253485, 9886e91)
🎯 Impact
📋 Testing
All changes tested in development environments: