fix: show git hash as clickable link above stacktrace (#201) #211

Merged
guettlibot merged 3 commits from issue-201-fix into main 2026-05-24 10:56:29 +00:00
guettlibot commented 2026-05-24 08:56:38 +00:00 (Migrated from codeberg.org)

Why the link was not visible yet

Previous PRs (#150, #179) both attempted to add a git hash link to the crash screen, but a rebase conflict resolution left duplicate implementations:

  1. Above the stacktrace (lines 59–66 before this fix): plain grey text 'Git Commit: abc1234' — shown, but not a link.
  2. Below the stacktrace (lines 109–134 before this fix): a clickable GestureDetector link — a link, but in the wrong position and redundant.

The commit 6e22683 only removed a duplicate variable declaration from the conflict, not the duplicate widget sections.

What this PR does

  • Replaces the plain text display above the stacktrace with a proper clickable link (blue, underlined, opens the Codeberg commit page).
  • Removes the now-redundant clickable section that was below the stacktrace.
  • Makes gitHash an injectable constructor parameter (defaults to String.fromEnvironment('GIT_HASH')) so widget tests can exercise the link without a release build.
  • Adds a widget test that verifies the link is present, positioned above the stacktrace, and opens the correct Codeberg URL when tapped.

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com

## Why the link was not visible yet Previous PRs (#150, #179) both attempted to add a git hash link to the crash screen, but a rebase conflict resolution left duplicate implementations: 1. **Above the stacktrace** (lines 59–66 before this fix): plain grey text `'Git Commit: abc1234'` — shown, but not a link. 2. **Below the stacktrace** (lines 109–134 before this fix): a clickable `GestureDetector` link — a link, but in the wrong position and redundant. The commit `6e22683` only removed a duplicate *variable* declaration from the conflict, not the duplicate *widget* sections. ## What this PR does - Replaces the plain text display above the stacktrace with a proper clickable link (blue, underlined, opens the Codeberg commit page). - Removes the now-redundant clickable section that was below the stacktrace. - Makes `gitHash` an injectable constructor parameter (defaults to `String.fromEnvironment('GIT_HASH')`) so widget tests can exercise the link without a release build. - Adds a widget test that verifies the link is present, positioned above the stacktrace, and opens the correct Codeberg URL when tapped. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sign in to join this conversation.