Skip to content

Fix execution log download URL handling#339

Open
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/critical-bug-investigation-4b5d
Open

Fix execution log download URL handling#339
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/critical-bug-investigation-4b5d

Conversation

@cursor

@cursor cursor Bot commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes a critical breakage in the execution_log URL-only path:

  • Exposes return_download_url in the public harness_get input schema so schema-driven MCP clients can use the opt-in instead of falling back to buffered log retrieval.
  • Makes resolveLogDownloadUrl() fail fast for Harness log-service blob links that require MCP server auth/proxying, rather than returning a URL that callers cannot fetch directly.
  • Adds focused regressions for the public schema/handler path and non-exportable Harness blob links.

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Other

Checklist

  • Tests pass
  • Typecheck passes

Validation:

  • pnpm build
  • pnpm docs:generate
  • pnpm docs:check
  • pnpm typecheck
  • pnpm exec vitest run tests/utils/log-resolver.test.ts tests/tools/tool-handlers.test.ts -t 'resolveLogDownloadUrl|return_download_url'
  • pnpm test
Open in Web View Automation 

cursoragent and others added 2 commits June 13, 2026 12:21
Co-authored-by: Rohan Gupta <thisrohangupta@users.noreply.github.com>
Co-authored-by: Rohan Gupta <thisrohangupta@users.noreply.github.com>
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@thisrohangupta thisrohangupta marked this pull request as ready for review June 16, 2026 23:47
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.

2 participants