Skip to content

feat: Show when each test execution ran in the pull request panel - #17

Merged
olensmar merged 2 commits into
mainfrom
feat/pr-panel-run-times
Sep 30, 2026
Merged

olensmar merged 2 commits into
mainfrom
feat/pr-panel-run-times

Conversation

@olensmar

@olensmar olensmar commented Sep 30, 2026 •

Copy link
Copy Markdown
Member

Summary

Brings the run times from the repository panel to the pull request panel.

  • Each test execution row in the PR panel now shows how long ago it ran ("10 minutes ago") instead of the status word; the status icon already shows the status, and the exact time and status are on hover. This matches the workflow rows in the repository panel's status popovers.
  • The status stays readable without hovering: screen readers get it before the time on every row, and it is shown visibly ("running · 2 minutes ago") for statuses whose icon is not unique. The workflow rows in the status popovers use the same helper.
  • The same applies to the test popover of the Recent pull requests list on the repository page, which lists the same executions.
  • Times come from the event data the extension already fetches (finishedAt, else scheduledAt for runs still in progress): no extra requests.
  • README and STORE_LISTING mention the times.

Independent of #16. Both edit nearby README and STORE_LISTING lines, so whichever merges second may need a small rebase.

Test plan

  • npm run lint, npm run format:check, npm run typecheck, npm run build
  • PR page of a connected repo: each test row shows "… ago", with status and exact time on hover
  • Repository page: hovering a recent PR's pass count shows the same times

🤖 Generated with Claude Code

@olensmar
olensmar force-pushed the feat/pr-panel-run-times branch from 704a4c5 to 8ffa677 Compare September 30, 2026 12:19
@olensmar
olensmar changed the base branch from docs/since-1.0.0 to main September 30, 2026 12:19
@greptile-apps

greptile-apps Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[Medium risk] Adds execution timestamps to test results in pull request panels.

The PR appears safe to merge; no outstanding or new actionable finding remains.

Summary

The PR adds relative execution times to pull-request test rows and recent-PR popovers, while retaining status text for screen readers and for states with ambiguous icons. The current head preserves the store listing’s existing disclosures. No new actionable issue was found.

Reviews (3) · Last reviewed commit: "fix: Keep each execution's status readab..."

Comment thread src/content/widget.ts Outdated
@olensmar

Copy link
Copy Markdown
Member Author

@greptile

olensmar and others added 2 commits September 30, 2026 20:21
Each test execution row in the PR panel, and in the recent pull request
test popover on the repository page, now shows how long ago it ran instead
of the status word (the icon shows the status), with the status and exact
time on hover, like the workflow rows in the status popovers. The times
come from the event data the extension already fetches.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The status moved to a hover-only tooltip when run times replaced the status
word, which screen readers and keyboard users cannot reach, and several
statuses share the same icon. Execution rows now always include the status
as screen-reader text before the time, and show it visibly ("running · 2
minutes ago") for statuses whose icon is not unique; passed, failed and
aborted keep the time only. The workflow rows in the status popovers use
the same helper.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@olensmar
olensmar force-pushed the feat/pr-panel-run-times branch from 19d3e37 to e08c7db Compare September 30, 2026 18:21
@olensmar

Copy link
Copy Markdown
Member Author

The latest Greptile summary reviewed 19d3e37, the head before this branch was rebased onto main after #16 merged. That commit predates #16's PRIVACY.md and STORE_LISTING.md disclosure updates, so compared with today's main it looked like it removed them. The current head (e08c7db) doesn't touch PRIVACY.md, and STORE_LISTING.md keeps the metadata and github.com disclosures; its only change there is the "with when it ran" wording.

@greptile please review the current head.

@olensmar
olensmar merged commit 3c029cb into main Sep 30, 2026
1 check passed
@olensmar
olensmar deleted the feat/pr-panel-run-times branch September 30, 2026 18:42
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.

1 participant