add axios and default timeout to api

[goastler]

No description provided.

[Copilot]

Pull Request Overview

This PR replaces the existing fetch-based HTTP client with Axios (including a default timeout) and adjusts class formatting in one API file.

• Swapped out fetch calls for axios.get/axios.post with a 5 s default timeout in HttpClientBase.ts
• Removed the old JSON responseHandler and streamlined error handling
• Updated brace placement for the ProviderApi class declaration

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
packages/api/src/api/ProviderApi.ts	Moved the class’s opening { inline with the implements clause for consistency
packages/api/src/api/HttpClientBase.ts	Replaced fetch with Axios, added default timeout and JSON header, removed handler

Comments suppressed due to low confidence (1)

packages/api/src/api/HttpClientBase.ts:24

• [nitpick] The new Axios-based fetch method and its default timeout behavior aren't covered by existing tests. Adding unit tests for successful responses, timeout scenarios, and error handling will help prevent regressions.

	protected async fetch<T>(input: RequestInfo, config: AxiosRequestConfig = {}): Promise<T> {

[goastler]

@HughParry is wary of axios, apparently past portal issues?

[HughParry]

Needs testing! This adds helpers to base lvl http client so should be more tested across all services before it's ready for review