Simplify Header URL query synchronization

[coderabbitai]

Summary

Evaluate replacing the guarded effect that synchronizes the URL query to Header local input state with a simpler representation.

Required changes

• Consider key={urlQuery} remounting or a fully derived input value.
• Keep behavior unchanged if the existing implementation remains necessary.

Affected areas

• Header.tsx

Acceptance criteria

• The search input remains synchronized with the URL query.
• The implementation does not need a ref-guarded synchronization effect, unless a documented constraint requires it.

Source

• Review comment: Search and filter tasks through url search params #37 (comment)
• Pull request: Search and filter tasks through url search params #37
• Requested by: @Alejandroq12

[coderabbitai]

Technical note

Choose one source of truth for the search query. URL state should normally be canonical because navigation can change it externally. Keep local state only for a deliberate draft or debounce boundary. Test back, forward, direct links, clearing the query, and browser navigation.

Reference: React: You might not need an Effect

[coderabbitai]

Engineering note

Avoid copying URL state into local state unless the input needs an independent editing phase. A controlled value derived from the URL is simpler when it meets the interaction requirements. If local state remains necessary, define one explicit synchronization boundary and test browser navigation.

Read: React: You might not need an Effect