Add refHash support for transaction hash replay in SimulationResultsPage and TransactionSummary

[Timidan]

No description provided.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
web3-toolkit	Ready	Preview, Comment	Apr 23, 2026 2:00pm

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[Timidan]

@copilot review

[Copilot]

@copilot review

Here's a review of the three changes in this PR:

SimulationResultsPage.tsx

• The refHash derivation is well-guarded — it's only set when the simulation origin is "tx-hash-replay" and the replayed hash differs from the simulation ID (avoiding a duplicate row). ✅
• simulationOrigin and replayTxHash are already part of the base SimulationContractContext interface, so no unsafe type-cast is needed — cleaner than the existing contextWithExtras pattern. ✅

TransactionSummary.tsx

• refHash is correctly optional with a clear JSDoc comment. ✅
• The "Ref Hash" row renders directly below the "Hash" row with a copy button — sensible placement and consistent with the rest of the summary grid. ✅

TopBar.tsx

• Replacing <div> with <Link to="/"> is minimal and correct. react-router-dom is already a project dependency. ✅
• aria-label="HexKit home" and focus-visible:ring classes are good accessibility additions. ✅

No issues found — the implementation is correct and consistent with existing patterns in the codebase.