[Calebux]

Closes #1246

Summary

Replaced the silent redirect in handleCollectFees with a state-driven authenticate prompt that reuses the existing handleAuthenticate flow.

Changes to apps/web/app/portfolio/page.tsx:

• Added pendingCollectPositionId state to track which position triggered the collect action before authentication.
• Extracted the post-auth collect logic into a doCollectFees callback so both handleCollectFees and handleAuthenticate can invoke it without circular dependencies.
• handleCollectFees: Instead of router.replace('/') when !authToken, it sets pendingCollectPositionId and returns, keeping the user on the page.
• handleAuthenticate: After a successful authentication, it checks for a pending collect and calls doCollectFees to proceed automatically.
• Added a collect-specific authenticate CTA (with message "Authenticate your wallet to collect fees") that appears when pendingCollectPositionId is set. The general authenticate CTA is hidden while the collect-specific one is visible.

Changes to apps/web/__tests__/PortfolioPage.test.tsx:

• Made the localStorage mock configurable so tests can override it.
• Added a test that verifies clicking collect without a token shows the authenticate prompt and does not redirect to /.

---

Opened with PRmaster after human review.