Skip to content

chore: remove axios from overrides#288

Merged
Daryna-del merged 1 commit intomainfrom
fix/rremove-axios-from-override
Apr 16, 2026
Merged

chore: remove axios from overrides#288
Daryna-del merged 1 commit intomainfrom
fix/rremove-axios-from-override

Conversation

@Daryna-del
Copy link
Copy Markdown
Contributor

What/Why/How?

The axios package is provided via the realmtypesense dependency chain. If there are any vulnerabilities or issues, they should be addressed on the realm side rather than overridden here.

I tested the changes, and everything works as expected. Therefore, there is no need to keep the override in this repository.

There are no changes to package-lock.json, as it was already updated to version 1.15.0 in this commit.

Reference

Testing

Screenshots (optional)

Check yourself

  • Code is linted
  • Tested
  • All new/updated code is covered with tests

Security

  • Security impact of change has been considered
  • Code follows company security practices and guidelines

@Daryna-del Daryna-del requested a review from a team as a code owner April 16, 2026 11:35
Copy link
Copy Markdown

@redocly redocly bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

marketing-site AI Review: 🟢 Completed

Note

Low Risk

Removing the override for axios simply defers to the dependency chain (realmtypesense) for version resolution. Given that the package-lock.json is already up to date and correctly resolves the required version, the risk of breaking functionality is minimal.

Overview

Removes axios from the overrides section in package.json. Relying on the natural dependency graph shifts the responsibility of managing axios versions (and related vulnerabilities) back to the upstream realm package, eliminating the need for a manual override in this repository.

@Daryna-del Daryna-del self-assigned this Apr 16, 2026
@Daryna-del Daryna-del merged commit 3d14c13 into main Apr 16, 2026
7 checks passed
@Daryna-del Daryna-del deleted the fix/rremove-axios-from-override branch April 16, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants