Repository navigation
chore(typescript): agent-server to v1.50.1 - #5413
all-hands-bot wants to merge 1 commit into
Conversation
REST API breakage checks (OpenAPI) — ✅ PASSEDResult: ✅ PASSED |
all-hands-bot
left a comment
all-hands-bot
left a comment
There was a problem hiding this comment.
This review was posted by an AI agent (OpenHands).
Scope
In scope. clients/typescript/ is the owned browser-compatible mirror of the Agent Server REST contract, and the flow SDK/Agent Server → OpenAPI → clients/typescript → Agent Canvas is exactly what this change follows. This is the routine post-release pin bump (config.agentServerImage 1.47.0-python → 1.50.1-python) plus regeneration of the checked-in transport contract. No product/architecture decision is required, so the technical review proceeded.
What I verified on the current head (9a51ebf)
- Diff is exactly the two intended files.
package.json(+1/-1, onlyconfig.agentServerImage) andsrc/generated/agent-server-schema.ts(+1941/-159). No distributable packageversionchange — this is not a release PR. - The contract is genuinely regenerated from the pinned release. I re-ran
npm run check:agent-server-api; it fetchedreleases/download/v1.50.1/openapi.jsonand reported the checked-in contract is current. The regenerated file is byte-identical to the committed one, and the header pins the same image. - The removals are upstream contract changes, not stale-client artifacts. The six
DesktopUrl*declarations (nodesktop-urlpath in v1.50.1) and themodify_paramsfield are absent from the pinned release. No file underclients/typescript/srcoutsidegenerated/, norexample/,docs/,config/, orscripts/, references any removed symbol, so nothing handwritten breaks. - Checks I ran locally on this head pass:
npm run build(tsc),npm run lint(0 errors),npm run format:check,npm run test(347 tests / 23 files),npm run check:public-type-budget(budget unchanged),npm run test:agent-server-api-tooling, andnpm run test:endpoint-audit-tooling. No tracked file was modified; the regeneration left the schema unchanged. - GitHub Actions on this head is green for the client-relevant jobs (
agent-server-api,public-type-budget,test (22.12),security,build,coverage-report,agent-server-tests, plus the REST API breakage report PASSED). The only non-success conclusions are the expectedcleanup-*skips.
Non-blocking note
The Validate PR description check is red on this head. The cause is not this diff: pr-description-check.yml requires a HUMAN:/AGENT: template body with a linked issue, while version-bump-prs.yml generates this PR's body from ## Bump tracked Agent Server version/## Agent Server API change summary with no linked issue. The same failure appears on the sibling bump PRs (#5388, #5323, #5271, #5221, #5214), confirming it is a repo-level automation gap rather than a defect in this change. I flag it only because it is a red check on the current head that a maintainer may need to account for at merge time; the repository cleanup workflow owns .pr/ artifacts and this description is workflow-owned.
No material correctness, security, compatibility, or acceptance-criterion finding on the current head.
✅ APPROVED
|
Closing as obsolete: since #5476 the release PR pins the TypeScript client to its own Agent Server version, and |
|
I'm on it! VascoSch92 can track my progress at all-hands.dev |
|
OpenHands encountered an error: **Failed to fetch final response from agent server: HTTP 503 error: no available server See the conversation for more information. |
Bump tracked Agent Server version
Updates from Agent Server v1.47.0 to v1.50.1 and regenerates its checked-in OpenAPI contract.
Agent Server API change summary
ActivateMetaProfileApiMetaProfilesNameActivatePostData,ActivateMetaProfileApiMetaProfilesNameActivatePostError,ActivateMetaProfileApiMetaProfilesNameActivatePostErrors,ActivateMetaProfileApiMetaProfilesNameActivatePostResponse,ActivateMetaProfileApiMetaProfilesNameActivatePostResponses,ActivateMetaProfileResponse,ArchiveDirectoryApiConversationsRuntimeConversationIdFileArchiveGetData,ArchiveDirectoryApiConversationsRuntimeConversationIdFileArchiveGetError,ArchiveDirectoryApiConversationsRuntimeConversationIdFileArchiveGetErrors,ArchiveDirectoryApiConversationsRuntimeConversationIdFileArchiveGetResponse,ArchiveDirectoryApiConversationsRuntimeConversationIdFileArchiveGetResponses,BackendLogs,BackendRevisionRequest,BackendStatus,BatchGetBashEventsApiConversationsRuntimeConversationIdBashBashEventsGetData,BatchGetBashEventsApiConversationsRuntimeConversationIdBashBashEventsGetError,BatchGetBashEventsApiConversationsRuntimeConversationIdBashBashEventsGetErrors,BatchGetBashEventsApiConversationsRuntimeConversationIdBashBashEventsGetResponse,BatchGetBashEventsApiConversationsRuntimeConversationIdBashBashEventsGetResponses,BodyUploadFileQueryApiConversationsRuntimeConversationIdFileUploadPost,CanvasExtensionBackend,CanvasExtensionBackendArtifact,CanvasExtensionBackendHealth,ClassifyAndSwitchLlmAction,ClassifyAndSwitchLlmObservation, and 154 moreDesktopUrlResponse,GetDesktopUrlApiDesktopUrlGetData,GetDesktopUrlApiDesktopUrlGetError,GetDesktopUrlApiDesktopUrlGetErrors,GetDesktopUrlApiDesktopUrlGetResponse,GetDesktopUrlApiDesktopUrlGetResponsesTriggered by https://github.com/OpenHands/software-agent-sdk/releases/tag/v1.50.1.
🐳 Agent Server images for this PR — GHCR package, pull/run commands, and all pushed tags (click to expand)
• GHCR package: https://github.com/OpenHands/agent-sdk/pkgs/container/agent-server
Variants & Base Images
eclipse-temurin:17-jdkpython-node-runtimepython-node-runtimepython-node-runtimegolang:1.21-bookwormPull (multi-arch manifest)
# Each variant is a multi-arch manifest supporting both amd64 and arm64 docker pull ghcr.io/openhands/agent-server:9a51ebf-pythonRun
All tags pushed for this build
About Multi-Architecture Support
9a51ebf-python) is a multi-arch manifest supporting both amd64 and arm649a51ebf-python-amd64) are also available if needed