FIX: CI Pipeline Failures and Code Quality Issues#1
Merged
wicky-zipstack merged 7 commits intomainfrom Mar 19, 2026
Merged
Conversation
Update lock file to include missing dependencies that were causing npm ci to fail in GitHub Actions. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
tahierhussain
requested review from
Deepak-Gnanasekar,
abhizipstack and
wicky-zipstack
|
No reviewable files after applying ignore patterns. |
Add missing test dependency required by setupTests.js to fix test suite failure in GitHub Actions. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Configure transformIgnorePatterns to allow Jest to transform the axios package, which ships as an ES module and was causing test failures. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Include additional ES module packages in Jest transform config to fix test failures. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Remove unused React import from OnboardingProgress.jsx - Remove unused yamlContent variable from TransformApply.jsx 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
tahierhussain
changed the title
Remove Jest ES module transform config; will address in a separate PR. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Deepak-Gnanasekar
approved these changes
Mar 19, 2026
wicky-zipstack
approved these changes
Mar 19, 2026
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
3 participants
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.
What
package-lock.jsonto sync withpackage.json@testing-library/jest-domas a dev dependencyWhy
package.jsonandpackage-lock.jsonfiles were out of sync, causingnpm cito fail in GitHub Actions@testing-library/dom@10.4.1aria-query@5.3.0typescript@4.9.5@testing-library/jest-dompackage was missing, causing the test suite to fail with "Cannot find module '@testing-library/jest-dom'" error insetupTests.jsHow
npm installto regenerate the lock file with all required dependencies@testing-library/jest-dom@^6.9.1to devDependenciesReactimport fromOnboardingProgress.jsxyamlContentvariable fromTransformApply.jsxCan this PR break any existing features. If yes, please list possible items. If no, please explain why. (PS: Admins do not merge the PR without this section filled)
Database Migrations
Env Config
Relevant Docs
Related Issues or PRs
Dependencies Versions
@testing-library/jest-dom@^6.9.1as dev dependencyNotes on Testing
Screenshots
Checklist
I have read and understood the Contribution Guidelines.