bobbai00 opened a new pull request, #5928: URL: https://github.com/apache/texera/pull/5928
### What changes were proposed in this PR? Centralizes the agent-service type/config layer (no runtime behavior change). **Stacked on #5751** (the WS-types PR). - `types/dto.ts` — backend request/response DTOs (workflow persistence, compiling-service responses, `WorkflowFatalError`). - `types/metadata.ts` — operator-metadata types extracted from `api/backend-api`. - `config/endpoints.ts` — `getServiceEndpoints()` centralizing backend base URLs. - `auth/jwt.ts` — relocated from `api/auth-api`; import paths updated. - Replace `any` with precise types across `src/types/*`; type `ReActStep` tool/message fields. - Consume the centralized types at the call sites (clients, server, agent utils); expand test coverage (jwt, `workflow-system-metadata`, compile/workflow clients) and rename `*.test.ts` → `*.spec.ts`. `types/execution.ts` is intentionally **untouched** here — its redesign is #5927. ### Any related issues, documentation, discussions? Part of #5747. **Depends on #5751** (WS types) — please review/merge that first. This PR targets `main`, so until #5751 lands the diff below also includes the WS changes; it narrows to just the DTO/centralizing files once #5751 merges. #5927 (execution-result redesign) stacks on top of this. ### How was this PR tested? `bunx tsc --noEmit`, `bun test` (126 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]
