Skip to content

feat: add Activity Log runs export endpoint - #297

Closed
ShashwatXD wants to merge 5 commits into
OpenHands:mainfrom
ShashwatXD:feat/activity-log-export
Closed

feat: add Activity Log runs export endpoint#297
ShashwatXD wants to merge 5 commits into
OpenHands:mainfrom
ShashwatXD:feat/activity-log-export

Conversation

@ShashwatXD

Copy link
Copy Markdown

Summary

  • Add GET /v1/{id}/runs/export (JSON/CSV) with pagination
  • Share run-fetch helpers between list and export
  • List + export are pagination-only

Fixes

OpenHands/OpenHands#16157

Tests

  • Export JSON/CSV across multiple pages
  • List runs with limit/offset only
  • Export unit tests pass

Keep started_after/started_before on export only; list stays status + pagination.
Simplify Activity Log list/export to pagination only (plus format and
conversation_base_url on export) so clients can page the full run history.
@ShashwatXD ShashwatXD changed the title Feat/activity log export feat: add Activity Log runs export endpoint Aug 1, 2026
@github-actions github-actions Bot added the type: feat A new feature label Aug 1, 2026

@neubig neubig left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for the contribution and for adding thorough coverage for Activity Log export. I think we should avoid introducing a separate export endpoint here.

GET /v1/{automation_id}/runs already provides the data needed for this feature (id, status, timestamps, conversation_id, and error_detail) with limit/offset pagination and a total. The client already has the automation name and trigger, can derive the conversation URL from its own origin, calculate duration from timestamps, and generate CSV locally. It can page the existing endpoint to retrieve the complete history (and we can raise the list limit if reducing request count is important).

Please rework this to reuse the existing runs endpoint and remove the new /runs/export API and export-specific response types/helpers. A dedicated export API would make sense later for server-side streaming/asynchronous exports or a separately supported external export contract, but neither is needed for the current UI.

@ShashwatXD

Copy link
Copy Markdown
Author

we dont need this pr anymore, closing this

@ShashwatXD ShashwatXD closed this Aug 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: feat A new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants