You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@texera.apache.org by GitBox <gi...@apache.org> on 2026/07/09 16:27:14 UTC
[GH] (texera/gh-readonly-queue/main/pr-5995-29233cc86ed8687f5eafd6b5b7dd4e51b7000c84): Workflow run "Required Checks" failed!
The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-5995-29233cc86ed8687f5eafd6b5b7dd4e51b7000c84 has failed.
Run started by GitHub user aicam (triggered by aicam).
Head commit for run:
f07d243d69be3a9210bf46b99a5ca2c6a4db9398 / Matthew B. <mg...@uci.edu>
refactor(frontend): extract agent payload and ReActStep mapping helpers (#5995)
### What changes were proposed in this PR?
- Added a private `mapApiAgentInfo(api)` helper in `AgentService` and
routed `createAgent`, `getAgent`, and `getAllAgents` through it,
removing three identical inline `ApiAgentInfo -> AgentInfo` mappings
(including the duplicated `delegate` sub-mapping).
- Added a private `convertApiReActSteps(steps)` helper and routed the
three `steps.map(s => convertApiReActStep(s))` sites through it.
- No behavior change: the produced `AgentInfo` and `ReActStep[]` values
are identical to before.
### Any related issues, documentation, discussions?
Closes: #5990
### How was this PR tested?
- Run `cd frontend && yarn test --include='**/agent.service.spec.ts'`,
expect the existing `AgentService` suite (including the `createAgent`
cases that flush an API payload and assert the mapped `AgentInfo`) to
pass.
- Typecheck with `cd frontend && npx tsc --noEmit -p tsconfig.json`,
expect no errors in `agent.service.ts` (the only error is a pre-existing
unrelated missing-module in `migration-llm.ts`).
### Was this PR authored or co-authored using generative AI tooling?
Co-authored with Claude Opus 4.8 in compliance with ASF
Report URL: https://github.com/apache/texera/actions/runs/29032837265
With regards,
GitHub Actions via GitBox