chore(typescript): agent-server to v1.49.2 - #5162
all-hands-bot wants to merge 1 commit into
Conversation
|
👋 This PR needs a couple of things fixed before OpenHands can review it:
Push an update once this is addressed and this check re-runs automatically. This is an automated check - no AI was used to generate this comment. |
REST API breakage checks (OpenAPI) — ✅ PASSEDResult: ✅ PASSED |
all-hands-bot
left a comment
There was a problem hiding this comment.
This review was posted by an AI agent (OpenHands).
Review: chore(typescript): agent-server to v1.49.2
Scope: In-repo. clients/typescript/ owns the browser-compatible TypeScript client and mirrors the Agent Server OpenAPI contract; version-bump-prs.yml is the intended producer of bump-typescript-agent-server-* PRs against main. This change belongs here.
What the diff does (verified against the merge base d128a786):
clients/typescript/package.json:config.agentServerImagebumpedghcr.io/openhands/agent-server:1.47.0-python→1.49.2-python. Theversionfield is untouched by this PR's three-dot diff.clients/typescript/src/generated/agent-server-schema.ts: regenerated from the v1.49.2 release artifact (header now readsSource: ghcr.io/openhands/agent-server:1.49.2-python).
Verified locally in the workspace (head d3adaa0):
npm run check:agent-server-api— regenerated cleanly fromreleases/download/v1.49.2/openapi.json; "Checked-in Agent Server contract is current."npm run test:agent-server-api-tooling— passed.npm test— 20 files / 328 tests passed.npm run lint— 0 errors (pre-existing warnings only, none in changed files);npm run format:check— clean.npm run audit:endpoints— passed.
Contract-removal check: The regenerated schema drops DesktopUrlResponse and the /api/desktop/url operation (along with the pre-existing /api/file/* forms that now coexist with the /api/conversations/{runtime_conversation_id}/file/* variants). I grepped the handwritten client (src/client, src/conversation, src/workspace, examples, scripts, endpoint-audit.config.json): nothing references /api/desktop/url or DesktopUrlResponse outside the generated file, so the removal introduces no dangling references. No code-level defect found.
Blocking signal — failing PR validation:
The required check PR Description Check → Validate PR description is failing on this head (mergeStateStatus: BLOCKED). The annotations are:
- The first visible line of the PR description must be
HUMAN: - Add a short human-written note between
HUMAN:andAGENT: - Keep the
AGENT:marker from the PR template - Keep/fill the
## Summary,## Why,## How to Testsections - Link an issue in the
## Issue Numbersection
This is the automation-generated release-bump body, which omits the standard PR template. The HUMAN: section and its note are reserved for a human author — an AI agent must not write them. This is not a code defect, but it is a red gate on the head, so I am not approving.
Ask: a human maintainer should add the required HUMAN: note and template sections so Validate PR description re-runs green, then this PR is safe to merge on the code side. There are no inline code findings to fix.
🔄 CHANGES REQUESTED
|
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 |
|
Confirmed — no action taken. I verified the closure rationale:
Leaving this PR closed; not reopening or pushing further changes. This comment was created by an AI agent (OpenHands) on behalf of the user. |
|
I'm on it! VascoSch92 can track my progress at all-hands.dev |
|
Verified — no action taken, and this PR stays closed. I independently confirmed the closure rationale:
No code changes or pushes are warranted. Closing rationale stands. This comment was created by an AI agent (OpenHands) on behalf of the user. |
|
The comment on PR #5162 was a closure notice from the maintainer (@VascoSch92), not a request for code changes. I verified its rationale before acting: Verification (all confirmed via GitHub API):
Action taken: No code changes, no commit, no push, no reopen — the PR is intentionally obsolete. I posted a short confirmation on the PR so the bot's open "I'm on it!" loop is closed: #5162 (comment) The separate |
|
I'm on it! VascoSch92 can track my progress at all-hands.dev |
|
Done. The comment required no code changes — it was a confirmation that this bump PR is obsolete and should stay closed. I independently verified each claim and posted a confirmation to the PR. What I verified
Action taken
The PR remains closed, which is the correct end state. |
|
I've completed the review. No action is warranted — here's my independent verification of the closure rationale: Verification resultsPR #5162 status — Claim 1 — #5476 moved the pin into the release PR: ✅ Confirmed
Claim 2 —
Claim 3 — published
ConclusionThis branch's I made no code changes and no pushes — the working tree is clean. There is nothing to fix, and re-opening or rebasing this obsolete bump would only reintroduce a stale pin. |
Bump tracked Agent Server version
Updates from Agent Server v1.47.0 to v1.49.2 and regenerates its checked-in OpenAPI contract.
Agent Server API change summary
ArchiveDirectoryApiConversationsRuntimeConversationIdFileArchiveGetData,ArchiveDirectoryApiConversationsRuntimeConversationIdFileArchiveGetError,ArchiveDirectoryApiConversationsRuntimeConversationIdFileArchiveGetErrors,ArchiveDirectoryApiConversationsRuntimeConversationIdFileArchiveGetResponse,ArchiveDirectoryApiConversationsRuntimeConversationIdFileArchiveGetResponses,BatchGetBashEventsApiConversationsRuntimeConversationIdBashBashEventsGetData,BatchGetBashEventsApiConversationsRuntimeConversationIdBashBashEventsGetError,BatchGetBashEventsApiConversationsRuntimeConversationIdBashBashEventsGetErrors,BatchGetBashEventsApiConversationsRuntimeConversationIdBashBashEventsGetResponse,BatchGetBashEventsApiConversationsRuntimeConversationIdBashBashEventsGetResponses,BodyUploadFileQueryApiConversationsRuntimeConversationIdFileUploadPost,ClearAllBashEventsApiConversationsRuntimeConversationIdBashBashEventsDeleteData,ClearAllBashEventsApiConversationsRuntimeConversationIdBashBashEventsDeleteError,ClearAllBashEventsApiConversationsRuntimeConversationIdBashBashEventsDeleteErrors,ClearAllBashEventsApiConversationsRuntimeConversationIdBashBashEventsDeleteResponse,ClearAllBashEventsApiConversationsRuntimeConversationIdBashBashEventsDeleteResponses,ConversationRuntimeError,ConversationRuntimeInfo,ConversationRuntimeStatus,CreateDirectoryApiConversationsRuntimeConversationIdFileCreateDirectoryPostData,CreateDirectoryApiConversationsRuntimeConversationIdFileCreateDirectoryPostError,CreateDirectoryApiConversationsRuntimeConversationIdFileCreateDirectoryPostErrors,CreateDirectoryApiConversationsRuntimeConversationIdFileCreateDirectoryPostResponse,CreateDirectoryApiConversationsRuntimeConversationIdFileCreateDirectoryPostResponses,DownloadFileQueryApiConversationsRuntimeConversationIdFileDownloadGetData, and 76 moreDesktopUrlResponse,GetDesktopUrlApiDesktopUrlGetData,GetDesktopUrlApiDesktopUrlGetError,GetDesktopUrlApiDesktopUrlGetErrors,GetDesktopUrlApiDesktopUrlGetResponse,GetDesktopUrlApiDesktopUrlGetResponsesTriggered by https://github.com/OpenHands/software-agent-sdk/releases/tag/v1.49.2.
🐳 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:d3adaa0-pythonRun
All tags pushed for this build
About Multi-Architecture Support
d3adaa0-python) is a multi-arch manifest supporting both amd64 and arm64d3adaa0-python-amd64) are also available if needed