fix: return diagnose log download URLs on request

[cursor]

Description

harness_diagnose now honors options.return_download_url for pipeline logs, returning signed logs.zip URLs for failed-step logs and requested-step logs instead of downloading and buffering inline log content. The default inline-log behavior is unchanged.

Type of Change

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

Checklist

• Tests pass
• Typecheck passes

Verification:

• pnpm exec vitest run "tests/tools/diagnose/pipeline.test.ts" -t "return.*download URL"
• pnpm exec vitest run "tests/tools/diagnose/pipeline.test.ts"
• pnpm exec vitest run "tests/tools/tool-handlers.test.ts" -t "harness_get — execution_log|harness_diagnose"
• pnpm build
• pnpm docs:generate
• pnpm typecheck && pnpm docs:check && pnpm test

  

[CLAassistant]

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.}