Skip to content

Allow crash reports to be shown by short ID - #376

Merged
jongio merged 2 commits into
mainfrom
idea-report-short-id
Jul 31, 2026
Merged

Allow crash reports to be shown by short ID#376
jongio merged 2 commits into
mainfrom
idea-report-short-id

Conversation

@jongio

@jongio jongio commented Jul 25, 2026

Copy link
Copy Markdown
Owner

Lets grut report --show accept an unambiguous crash report ID prefix, matching the shortened IDs users see in grut report --list.

Changes:

  • Keeps exact ID lookup working.
  • Resolves one matching prefix to the full report.
  • Returns a clear error when a prefix matches more than one report.

Validation:

  • go build ./...
  • go test ./...
  • mage preflight failed on existing lint findings in internal\panels\gitinfo\gitinfo_submodules.go and internal\panels\gitdiff\gitdiff.go.

Closes #373

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: e02217cd-ffed-496e-b8ea-848a781cda64
@jongio jongio self-assigned this Jul 25, 2026
@jongio jongio added the idea Feature idea from the idea pipeline label Jul 25, 2026
runReportShow now resolves reports through List plus findCrashReport, so Read had no remaining callers and the deadcode gate failed on it. Removes the function and its two orphaned tests, and reworks TestWriteAndRead to verify the written report through List.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 66a101e5-8e3c-41f0-bcf4-e26e7a44f1ed
@jongio
jongio merged commit 1ef070e into main Jul 31, 2026
4 checks passed
@jongio
jongio deleted the idea-report-short-id branch July 31, 2026 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

idea Feature idea from the idea pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow crash reports to be shown by short ID

1 participant