?
PPLX-68
feature,frontend,deep-research
Created: 2025-12-21 Updated: 2025-12-21
Relationships Loading...
Attachments
Loading...
Comments (3)
agent · 2025-12-21
## Integration Fix Complete ### Issue Identified QA Agent correctly identified that HomePage.tsx and HomePageContainer.tsx were not wiring up the reasoning effort state management, causing the selector to appear but be non-functional. ### Changes Made 1. **HomePage.tsx** - Added reasoningEffort state, handleReasoningEffortChange callback, pass props to HomeSearchBar 2. **HomePageContainer.tsx** - Handle reasoning effort in search handler, include in navigation URL ### Verification - All 23 HomeSearchBar tests pass - All 19 ReasoningEffortSelector tests pass - TypeScript compiles without errors - Visual verification: selector hidden in search mode, visible in research mode with animation ### Commit 50cca0f PPLX-68: Wire reasoning effort selector to HomePage and HomePageContainer
QA Agent · 2025-12-21
[QA] Critical integration bug found - reasoning effort selector is non-functional ## Issue The HomeSearchBar component correctly implements the reasoning effort selector, but HomePage.tsx does not wire up the required state management. The selector appears visually but clicking radio buttons has no effect. ## Root Cause HomePage.tsx (lines 131-144) renders HomeSearchBar without passing: - reasoningEffort prop - onReasoningEffortChange callback ## Impact - All 23 unit tests pass ✅ - Component works in isolation ✅ - Integration is broken ❌ - Users see a non-functional control (bad UX) ## Required Fixes 1. Add reasoningEffort state to HomePage 2. Add handleReasoningEffortChange callback 3. Pass both props to HomeSearchBar 4. Update HomePageContainer to handle reasoning effort in search 5. Re-test in browser to verify functionality ## Acceptance Criteria Status - ❌ User can change effort level (clicks don't work) - ❌ Selection is passed to parent (no callback wired) - ✅ All other criteria met See detailed report: /tmp/qa-PPLX-68.md Status changed: in-review → todo
agent · 2025-12-21
## Implementation Complete ### Changes Made - **HomeSearchBar.tsx**: Added ReasoningEffortSelector with smooth fade-in/fade-out animation when mode === 'research'. Added reasoningEffort prop (default: 'medium') and onReasoningEffortChange callback. - **HomeSearchBar.stories.tsx**: Added ResearchMode stories and Interactive story demonstrating mode switching. - **HomeSearchBar.test.tsx** (new): 23 comprehensive tests covering all acceptance criteria. ### Verification - All 23 HomeSearchBar tests pass - All 19 ReasoningEffortSelector tests pass - TypeScript compiles without errors ### Commit 4d76d33 PPLX-68: Add reasoning effort selector to HomeSearchBar