Skip to content

fix: reject ambiguous show run ids - #51

Merged
royalpinto007 merged 2 commits into
AgentPostmortem:mainfrom
Rayan-and-beyond:fix-ambiguous-show-9
Sep 16, 2026
Merged

royalpinto007 merged 2 commits into
AgentPostmortem:mainfrom
Rayan-and-beyond:fix-ambiguous-show-9

Conversation

@Rayan-and-beyond

Copy link
Copy Markdown
Contributor

Summary

  • reject ambiguous short IDs in agentrace show instead of silently selecting the first run
  • prefer an exact full-ID match when another run shares the same suffix
  • list matching full IDs and safe descriptions so the user can retry precisely

Closes #9

Validation

  • .venv/bin/pytest -q — 57 passed
  • .venv/bin/ruff check . — passed
  • focused show regressions — 2 passed
  • mutation proof: restoring the old match[0] behavior makes the ambiguity regression fail
  • git diff --check — passed

@royalpinto007
royalpinto007 merged commit 1bdc372 into AgentPostmortem:main Sep 16, 2026
2 checks passed
@github-actions

Copy link
Copy Markdown

This is in, @Rayan-and-beyond. Really appreciate it.

Agentrace gets better every time someone outside the repo digs in like this.

If you find Agentrace useful, consider dropping a star. It helps more people find it.

Happy to see more from you whenever you have the time.

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.

show picks the first match when a short id is ambiguous

2 participants