Skip to content

fix(web): show pull request refresh feedback - #6384

Open
t3-code[bot] wants to merge 1 commit into
mainfrom
fix/pull-request-refresh-feedback
Open

fix(web): show pull request refresh feedback#6384
t3-code[bot] wants to merge 1 commit into
mainfrom
fix/pull-request-refresh-feedback

Conversation

@t3-code

@t3-code t3-code Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

problem

refreshing a pull request from its action menu gave no visible confirmation, making it easy to wonder whether the click worked and press it again.

fix

  • show an immediate loading toast when refresh is selected
  • replace it with a success toast once the host cache has been invalidated and the pull request refresh has started
  • keep automatic and page-level refreshes silent

verification

  • vp run --filter @t3tools/web typecheck
  • vp lint apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx --report-unused-disable-directives
  • vp fmt --check apps/web/src/components/pullRequest/PullRequestDetailPanel.tsx
  • git diff --check

screenshots

before

clicking refresh leaves the panel unchanged, with no confirmation.

before

after

clicking refresh confirms completion with a visible success toast.

after

implemented with gpt-5.6-sol via hermes agent.

request provenance

Co-authored-by: Bilal Bakr <62337003+Bil0000@users.noreply.github.com>
@github-actions github-actions Bot added size:M 30-99 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. labels Aug 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 11.3 KiB 11.3 KiB +14 B (+0.1%) 15.1 KiB
Codex Thread snapshot wire 5.5 KiB 5.5 KiB +1 B (+0.0%) 7.3 KiB
Codex Live turn WebSocket wire 5.9 KiB 5.9 KiB +13 B (+0.2%) 7.8 KiB
Codex Live turn WebSocket decoded 49.7 KiB 49.7 KiB 0 B (0.0%) 66.4 KiB
Codex Live turn messages 16 16 0 (0.0%) 21
Claude Total thread wire 11.4 KiB 11.3 KiB −20 B (−0.2%) 15.1 KiB
Claude Thread snapshot wire 5.5 KiB 5.5 KiB +4 B (+0.1%) 7.3 KiB
Claude Live turn WebSocket wire 5.9 KiB 5.9 KiB −24 B (−0.4%) 7.8 KiB
Claude Live turn WebSocket decoded 50.6 KiB 50.6 KiB 0 B (0.0%) 66.4 KiB
Claude Live turn messages 16 16 0 (0.0%) 21

Baseline: ac1264e · PR result: f0d0b80 · Source CI: failure

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 94.6 KiB
  • Claude decoded thread snapshot: 95.4 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@macroscopeapp

macroscopeapp Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved f0d0b80

This PR adds toast notification feedback when refreshing a pull request. It's a self-contained UI enhancement using existing patterns, with no changes to core behavior or data flow.

You can customize Macroscope's approvability policy. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants