feat: add msw API handlers for development#183
Merged
iamitprakash merged 8 commits intochore/msw-mock-datafrom Aug 19, 2025
Merged
feat: add msw API handlers for development#183iamitprakash merged 8 commits intochore/msw-mock-datafrom
iamitprakash merged 8 commits intochore/msw-mock-datafrom
Conversation
|
Based on your review schedule, I'll hold off on reviewing this PR until it's marked as ready for review. If you'd like me to take a look now, comment
|
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
![korbit-ai[bot]](https://avatars.githubusercontent.com/in/322216?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review by Korbit AI
Korbit automatically attempts to detect when you fix issues in new commits.
| Category | Issue | Status |
|---|---|---|
 |
Untyped Error Parameter ▹ view | |
 |
Missing Mock Response Debug Logs ▹ view | |
|
Magic number in default parameter ▹ view | |
 |
Invalid MSW URL Pattern ▹ view | |
 |
Invalid Error Response Structure ▹ view | ✅ Fix detected |
 |
Unsanitized URL Path Parameter ▹ view | ✅ Fix detected |
|
Loss of Error Context in Response ▹ view | ✅ Fix detected |
|
Direct Console Usage Instead of Logger ▹ view | |
|
Missing Pagination Parameter Validation ▹ view | ✅ Fix detected |
 |
Inadequate Error Handling Strategy ▹ view | ✅ Fix detected |
Files scanned
| File Path | Reviewed |
|---|---|
| mocks/utils/common.ts | ✅ |
| mocks/handlers/index.ts | ✅ |
| mocks/handlers/labels.handler.ts | ✅ |
| mocks/handlers/users.handler.ts | ✅ |
| mocks/data/labels.mock.ts | ✅ |
| mocks/data/users.mock.ts | ✅ |
| mocks/handlers/teams.handler.ts | ✅ |
| mocks/data/watchlist.mock.ts | ✅ |
| mocks/handlers/tasks.handler.ts | ✅ |
| mocks/data/teams.mock.ts | ✅ |
| mocks/data/tasks.mock.ts | ✅ |
Explore our documentation to understand the languages and file types we support and the files we ignore.
Check out our docs on how you can make Korbit work best for you and your team.
![graphite-app[bot]](https://avatars.githubusercontent.com/in/158384?s=60&v=4){kind=small}
Hariom01010
approved these changes
Aug 19, 2025
iamitprakash
pushed a commit
that referenced
this pull request
Aug 19, 2025
* chore: removed unused mock files and add mock data for msw * refactor(mocks): export mock data for labels, tasks, teams, users, and watchlist * feat: add msw API handlers for development (#183) * refactor(mocks): remove unused Task and Team mock files * fix(mocks): start mocking before API calls to prevent race conditions * refactor(mocks): export handlers * refactor(mocks): enhance task labels handling in mock data * refactor(mocks): integrate user assignment in task creation for mock data * refactor(mocks): improve error handling and response structure in mock API handlers
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.
Date: 25 Aug 2025
Developer Name: @AnujChhikara
Issue Ticket Number
Description
Documentation Updated?
Under Feature Flag
Database Changes
Breaking Changes
Development Tested?
Screenshots
Screenshot 1
screen-recording-2025-08-18-at-10852-pm_fvOZ5bHR.mp4
Test Coverage
Screenshot 1
Additional Notes
Description by Korbit AI
What change is being made?
Add mock API handlers using MSW (Mock Service Worker) for the development environment, covering labels, tasks, teams, users, and watchlist functionality.
Why are these changes being made?
These changes are being introduced to enable efficient development and testing workflows by providing a way to simulate real API behavior without making actual network requests. This setup supports the frontend application by allowing it to interact with mock data for different scenarios, facilitating quicker and more reliable integration testing.