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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 5, 2026 2:05am

PR Summary

Low Risk
Low risk, localized client-side URL update that only affects browser history state when changing activeResourceId. Potential impact is limited to navigation/deep-link behavior.

Overview
Clears any existing URL fragment (#...) when updating the resource query param on active resource changes in home.tsx, preventing markdown header hashes from persisting across resource switches.

^{Reviewed by Cursor Bugbot for commit eab6aed. Bugbot is set up for automated code reviews on this repo. Configure here.}

Greptile Summary

This PR fixes a bug where the URL hash fragment (e.g. #some-heading set during markdown header navigation) was persisted in the browser URL when switching between resources in the Mothership view. The one-line fix (url.hash = '') is added to the existing useEffect that already synchronizes the ?resource= URL search param whenever activeResourceId changes, ensuring the URL fragment is cleaned up on every resource switch.

• Adds url.hash = '' before window.history.replaceState in home.tsx so any #heading fragment accumulated from markdown navigation is cleared when the user switches to a different resource
• No other files are affected; the change is minimal, well-targeted, and correctly placed in the existing URL-sync effect
• No issues found during review

Confidence Score: 5/5

Safe to merge — single-line fix with no side effects

The change is a one-line addition that clears the URL hash fragment before replaceState. It is correctly placed in the useEffect responsible for URL synchronization, uses the standard URL API idiom (url.hash = '' fully removes the fragment from the serialized URL), and has no observable side effects beyond the intended fix. No custom rules are violated.

No files require special attention

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant User
    participant MothershipView
    participant Home
    participant BrowserHistory

    User->>MothershipView: Click resource tab
    MothershipView->>Home: setActiveResourceId(newId)
    Home->>Home: useEffect fires (activeResourceId changed)
    Note over Home: url.hash = '' clears #heading fragment
    Home->>BrowserHistory: replaceState(?resource=newId, no fragment)
    Note over BrowserHistory: URL is clean — no stale #heading

_{Reviews (1): Last reviewed commit: "fix(mothership): fix url keeping markdow..." | Re-trigger Greptile}