-
Notifications
You must be signed in to change notification settings - Fork 343
Open
Labels
Description
Describe the bug
The codebase still contains references, imports, and configurations for Axios (axios). This is observable in files like src/http-client/axios-client.ts and references throughout the build, test, and documentation files. This persists despite previous discussion around migrating to fetch (see #241, which was closed after suggesting a migration to fetch-based HTTP instead of Axios).
What version are you on?
Latest main branch as of Jan 2026.
To Reproduce
- Search the codebase for the string "axios"
- Observe
src/http-client/axios-client.ts, multiple build/test configs, and documentation sections that referenceaxios. - Review the package.json exports, which include no-axios builds, indicating partial migration but not complete removal.
Expected behavior
Axios should be fully removed from the repository if the intent is to use fetch/undici or another solution exclusively. Documentation and test/build configs should not mention Axios as a dependency.
Additional context
- The presence of "no-axios" build artifacts implies Axios support is optional, but the default build and code paths still depend on it.
- Replace Axios with Fetch #241 originally tracked a migration request from Axios to fetch, but Axios is still required for most build/test contexts.
- See also: https://github.com/stellar/js-stellar-sdk/search?q=axios&type=code
Reactions are currently unavailable