Skip to content

Fix failed log misattribution in pipeline summaries#673

Draft
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/critical-bug-investigation-6d9b
Draft

Fix failed log misattribution in pipeline summaries#673
cursor[bot] wants to merge 2 commits into
mainfrom
cursor/critical-bug-investigation-6d9b

Conversation

@cursor

@cursor cursor Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes a correctness bug in the new bulk pipeline-log summarization path. When multiple failed steps shared an identifier across stages, all_step_logs could attach the first matching stage's log to another step, producing a false execution summary.

The failed-log fetch path now retains a map keyed by the canonical logBaseKey, and bulk result assembly merges each fetched value using that exact key. A regression test covers duplicate deploy identifiers in separate stages with distinct logs.

Type of Change

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

Checklist

  • pnpm test passes (117 files, 2565 tests)
  • pnpm typecheck passes
  • pnpm build passes
  • pnpm standards:check passes (80 tests)
  • pnpm docs:check passes

Coding Standards (registry-driven MCP model)

Not applicable — this change does not add or modify Harness API coverage or tool registration.

Open in Web View Automation 

cursoragent and others added 2 commits July 21, 2026 11:14
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.

@Saranya-jena

Copy link
Copy Markdown
Contributor

@cursor wouldn't it cause an issue when logbasekey is very long?

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