dd search state management for about page details

[ComputelessComputer]

No description provided.

[netlify]

✅ Deploy Preview for hyprnote ready!

Name	Link
🔨 Latest commit	e9a7655
🔍 Latest deploy log	https://app.netlify.com/projects/hyprnote/deploys/69248842a4681d00084f2340
😎 Deploy Preview	https://deploy-preview-1857--hyprnote.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The About page now synchronizes its selected item state with URL search parameters. Route validation filters allowed type values ("story", "founder", "photo") and coerces search parameters into a typed AboutSearch object. Component lifecycle derives initial state from URL parameters, and selection changes update both local state and query parameters via navigation.

Changes

Cohort / File(s)

Summary

URL-synced selection state apps/web/src/routes/_view/about.tsx

Added AboutSearch type for validated search parameters (type and id). Added Route.validateSearch() function to coerce and filter incoming search params. Imported useNavigate to enable URL updates. Added handleSetSelectedItem() handler to sync selection state with URL query params. Updated useEffect to derive initial state from Route search parameters. Modified AboutContentSection to receive handleSetSelectedItem instead of raw state setter.

Sequence Diagram

sequenceDiagram
    participant User
    participant Component as About Component
    participant Route as Route/Search
    participant Nav as useNavigate

    User->>Component: Click item
    Component->>Component: handleSetSelectedItem(item)
    
    rect rgb(200, 220, 255)
    Note over Component,Nav: State + URL Sync
    Component->>Component: setSelectedItem(item)
    Component->>Nav: navigate with updated search params
    Nav->>Route: Update URL query params
    Route->>Component: Route.useSearch updates
    Component->>Component: Derive selectedItem from search
    end
    
    Component-->>User: UI reflects selection & URL updated

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• Validation logic: Review the validateSearch function's filtering and coercion logic for search parameters to ensure all edge cases are handled.
• State-URL synchronization: Verify that handleSetSelectedItem correctly updates both local state and URL parameters, and that the useEffect properly derives UI state from URL search without infinite loops.
• Route configuration: Confirm the Route.validateSearch integration is correct and search parameter types match the component's expectations.

Possibly related PRs

• Refactored About page #1753: Refactors the About page into a modular SelectedItem-driven layout, providing the component structure that this PR's URL synchronization enhances.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch c-branch-5

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 330202e and e9a7655.

📒 Files selected for processing (1)

• apps/web/src/routes/_view/about.tsx (4 hunks)

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}