Skip to content

fix(examples): emit EXAMPLE_COST for marketplace demo#2379

Merged
xingyaoww merged 1 commit intomainfrom
fix/example-cost-marketplace-demo
Mar 10, 2026
Merged

fix(examples): emit EXAMPLE_COST for marketplace demo#2379
xingyaoww merged 1 commit intomainfrom
fix/example-cost-marketplace-demo

Conversation

@enyst
Copy link
Collaborator

@enyst enyst commented Mar 10, 2026

Summary

  • move the marketplace demo EXAMPLE_COST fix onto its own branch
  • keep release PR Release v1.13.0 #2378 limited to the release commit
  • apply the previously removed change from 21704dc on top of main

Testing

  • not run (branch split / PR management only)

Agent Server images for this PR

GHCR package: https://github.com/OpenHands/agent-sdk/pkgs/container/agent-server

Variants & Base Images

Variant Architectures Base Image Docs / Tags
java amd64, arm64 eclipse-temurin:17-jdk Link
python amd64, arm64 nikolaik/python-nodejs:python3.13-nodejs22 Link
golang amd64, arm64 golang:1.21-bookworm Link

Pull (multi-arch manifest)

# Each variant is a multi-arch manifest supporting both amd64 and arm64
docker pull ghcr.io/openhands/agent-server:b737441-python

Run

docker run -it --rm \
  -p 8000:8000 \
  --name agent-server-b737441-python \
  ghcr.io/openhands/agent-server:b737441-python

All tags pushed for this build

ghcr.io/openhands/agent-server:b737441-golang-amd64
ghcr.io/openhands/agent-server:b737441-golang_tag_1.21-bookworm-amd64
ghcr.io/openhands/agent-server:b737441-golang-arm64
ghcr.io/openhands/agent-server:b737441-golang_tag_1.21-bookworm-arm64
ghcr.io/openhands/agent-server:b737441-java-amd64
ghcr.io/openhands/agent-server:b737441-eclipse-temurin_tag_17-jdk-amd64
ghcr.io/openhands/agent-server:b737441-java-arm64
ghcr.io/openhands/agent-server:b737441-eclipse-temurin_tag_17-jdk-arm64
ghcr.io/openhands/agent-server:b737441-python-amd64
ghcr.io/openhands/agent-server:b737441-nikolaik_s_python-nodejs_tag_python3.13-nodejs22-amd64
ghcr.io/openhands/agent-server:b737441-python-arm64
ghcr.io/openhands/agent-server:b737441-nikolaik_s_python-nodejs_tag_python3.13-nodejs22-arm64
ghcr.io/openhands/agent-server:b737441-golang
ghcr.io/openhands/agent-server:b737441-java
ghcr.io/openhands/agent-server:b737441-python

About Multi-Architecture Support

  • Each variant tag (e.g., b737441-python) is a multi-arch manifest supporting both amd64 and arm64
  • Docker automatically pulls the correct architecture for your platform
  • Individual architecture tags (e.g., b737441-python-amd64) are also available if needed

Co-authored-by: openhands <openhands@all-hands.dev>
@enyst enyst marked this pull request as ready for review March 10, 2026 18:30
@github-actions
Copy link
Contributor

API breakage checks (Griffe)

Result: Passed

Action log

@github-actions
Copy link
Contributor

Agent server REST API breakage checks (OpenAPI)

Result: Failed

Log excerpt (first 1000 characters)
{"asctime": "2026-03-10 18:31:16,913", "levelname": "WARNING", "name": "openhands.agent_server.config", "filename": "config.py", "lineno": 173, "message": "\u26a0\ufe0f OH_SECRET_KEY was not defined. Secrets will not be persisted between restarts."}
::error title=openhands-agent-server REST API::Breaking REST API change detected without MINOR version bump (1.12.0 -> 1.12.0).

Breaking REST API changes detected compared to baseline release:
- added '#/components/schemas/HookExecutionEvent' to the '/items/anyOf[#/components/schemas/Event]/' response property 'oneOf' list for the response status '200'
- added '#/components/schemas/HookExecutionEvent' to the 'items/items/' response property 'oneOf' list for the response status '200'
- added '#/components/schemas/HookExecutionEvent' to the response body 'oneOf' list for the response status '200'
- the 'file' request property type/format changed from 'string'/'' to 'string'/'binary'
- added the new 'hook' enum value to the '/items/anyOf[#/co

Action log

@enyst enyst mentioned this pull request Mar 10, 2026
11 tasks
Copy link
Collaborator

@all-hands-bot all-hands-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Good taste - Elegant, simple solution

Taste Rating: 🟢

This is a straightforward CI infrastructure fix. The example was missing the required EXAMPLE_COST output that the test workflow expects. The fix:

  1. Documents the requirement in AGENTS.md (good - makes it discoverable)
  2. Adds the required print statement for a non-LLM example

No complexity, no special cases, just solving the real problem. LGTM!

@xingyaoww xingyaoww merged commit 8dd71b3 into main Mar 10, 2026
28 checks passed
@xingyaoww xingyaoww deleted the fix/example-cost-marketplace-demo branch March 10, 2026 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants