bobbai00 opened a new pull request, #5754: URL: https://github.com/apache/texera/pull/5754
### What changes were proposed in this PR? Final slice of the agent-service reorganization (no runtime behavior change): - Extract shared tool helpers into `agent/tools/tools-utility.ts`; slim `result-formatting.ts` and `workflow-execution-tools.ts`. - Tidy `texera-agent.ts` and `agent/util/*` (`workflow-system-metadata`, `workflow-utils`). - Simplify `server.ts`/`index.ts` wiring and startup-banner logging; switch the remaining `getBackendConfig` consumers to `getServiceEndpoints()` and remove the now-unused `api/backend-api.ts`. - Complete the type reshaping deferred from the earlier slices: `agent.ts` DTO renames (`AgentDelegateConfig`→`AgentDelegation`, `AgentSettingsApi`→`AgentSettingsDto`) and `workflow.ts` `Comment`/`CommentBox` plus the now-required `WorkflowSettings.executionMode` (updating `workflow-state.ts`). - Add tests: `texera-agent.test.ts`, `tools/workflow-execution-tools.test.ts`, and expanded `server.test.ts`/`result-formatting.test.ts`. > **Stacked on #5753** (which is stacked on #5751). Until those merge, the "Files changed" tab here includes their commits — review the `extract tool utilities and simplify server wiring` commit for this PR's changes. Please merge after #5753. ### Any related issues, documentation, discussions? Closes #5750 Part of #5747 ### How was this PR tested? `bunx tsc --noEmit`, `bun test` (134 pass / 0 fail), 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]
