refactor: always preserve marketplace search state in URL

[tomerqodo]

Benchmark PR from qodo-benchmark#415

[augmentcode]

🤖 Augment PR Summary

Summary: Refactors the plugins marketplace to always reflect search state in the URL.

Changes:

• Removes the Jotai-based “preserve search state” toggle and related hydration wrapper
• Switches marketplace search inputs (query/category/tags) to use nuqs useQueryState consistently
• Simplifies the Marketplace component tree by dropping client-side atom hydration
• Adjusts marketplace search-mode determination logic tied to category/collections
• Loosens the typing of usePluginPageContext’s selector argument

Technical Notes: Marketplace still uses server-side prefetch/hydration via TanStack Query based on request searchParams.

_{🤖 Was this summary useful? React with 👍 or 👎}

[augmentcode]

Review completed. 2 suggestions posted.

Comment augment review to trigger a new review at any time.

[augmentcode]

useActivePluginType() now binds the plugin type to the tab query param, but server-side parsing/prefetch (via createLoader(marketplaceSearchParamsParsers)) still expects category, which can cause SSR prefetch/state to default to all and diverge from the client URL. Consider aligning on a single query key (or supporting both for backward compatibility).

Other Locations

• web/app/components/plugins/marketplace/search-params.ts:6
• web/app/components/plugins/marketplace/hydration-server.tsx:16

_{🤖 Was this useful? React with 👍 or 👎}

[augmentcode]

In useMarketplaceSearchMode(), the fallback uses PLUGIN_CATEGORY_WITH_COLLECTIONS.has(activePluginType), which seems inverted relative to the comment (and would force search mode for categories with collections). If this is unintended, it will prevent showing the collections view for all/tool by default.

_{🤖 Was this useful? React with 👍 or 👎}