feat: migrate from axios to native fetch API #377
Merged
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
Replace axios with native fetch while maintaining full API compatibility. This migration reduces bundle size by ~500KB and modernizes the HTTP client. ## Major Changes - **Core HTTP Client**: Migrated `rest-client.ts` from axios to fetch - **File Uploads**: Updated multipart upload utility to use fetch - **Case Conversion**: Implemented custom snake_case/camelCase utilities with emoji preservation - **Retry Logic**: Built fetch wrapper with same retry behavior (3 retries, network errors only) - **Error Handling**: Preserved stack traces and error structure - **Timeout Support**: Added configurable request timeouts using AbortController - **Request Cancellation**: Added AbortController support for cancelling requests ## New Files - `src/utils/fetch-with-retry.ts`: Fetch wrapper with retry logic and timeout support - `src/utils/case-conversion.ts`: Snake/camel case conversion with emoji preservation - `src/test-utils/msw-setup.ts`: MSW setup for testing (future use) ## Type System Updates - **HTTP Types**: Created custom `HttpResponse`, `HttpError` types replacing axios types - **Named Parameters**: Updated multi-parameter functions to use named parameters - **Type Safety**: Used `type` instead of `interface`, eliminated `any` usage ## Dependencies Removed - `axios` (1.12.2) - `axios-case-converter` (1.1.1) - `axios-retry` (4.5.0) ## Bundle Size Impact - **Reduced**: ~500KB smaller bundle - **Dependencies**: Removed 3 HTTP-related packages - **Performance**: Native fetch API with better performance characteristics ## Compatibility - ✅ **Zero breaking changes**: All public APIs remain identical - ✅ **CommonJS support**: Maintained for backward compatibility - ✅ **Node.js 20+**: Leverages native fetch support - ✅ **All tests passing**: 262/262 tests pass with updated mocks 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
- Fix case declaration blocks in rest-client.ts - Remove unused imports and constants in test files - Use proper typing for test headers with RequestInit cast and toHaveProperty - Add explicit type annotations for case conversion functions All tests passing (262/262) with clean linting and formatting. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
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.
Summary
Key Changes
HttpResponse,HttpError,NetworkErrortypes replacing axios typesfetchWithRetryutility with retry logic matching original axios-retry behaviorDependencies Removed
axios- HTTP client libraryaxios-case-converter- Case conversion middlewareaxios-retry- Retry logic for axiosDependencies Added
camelcase- For consistent camelCase conversionTest Plan
This completes the migration from axios to native fetch API, building on PR #376 which established Node.js 20+ requirement.
🤖 Generated with Claude Code