bobbai00 opened a new pull request, #5753: URL: https://github.com/apache/texera/pull/5753
### What changes were proposed in this PR? Second slice of the agent-service reorganization (no runtime behavior change): - Rename the per-service HTTP clients to `*-client.ts` (`compile-api`→`compile-client`, `workflow-api`→`workflow-client`), split operator-metadata fetching into `operator-metadata-client.ts`, and add a dedicated `execution-client.ts`; all clients now read base URLs via `getServiceEndpoints()`. - Rewire `api/index.ts`; add unit tests for all four clients. - Add the additive types the clients/tests need: `ExecutionRequestParams` (`types/agent.ts`) and the `ExecutionMode` enum + optional `WorkflowSettings.executionMode` (`types/workflow.ts`). The follow-up server/state PR completes the `workflow.ts` reshaping and makes `executionMode` required. - Trim `api/backend-api.ts` to a transitional `getBackendConfig`; the remaining server/tools consumers switch to `getServiceEndpoints()` in the follow-up PR. Consumer changes are import-path-only; no logic was modified. > **Stacked on #5751.** Until that merges, the "Files changed" tab here also includes its commit — review the `isolate HTTP clients` commit for this PR's changes. Please merge after #5751. ### Any related issues, documentation, discussions? Closes #5749 Part of #5747 ### How was this PR tested? `bunx tsc --noEmit`, `bun test` (116 pass / 0 fail, including 4 new client test suites), and `prettier --check` all pass in `agent-service`. ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Opus 4.8 (1M context) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
