Repository navigation
fix: make Explore filters accessible and recoverable - #144
Conversation
Co-authored-by: OpenAI Codex <noreply@openai.com>
|
Continuation verification passed on merged main commit 0a598cf. |
Co-authored-by: OpenAI Codex <noreply@openai.com>
|
Continuation verification passed on merged main commit 0a598cf. |
Explore now shows one empty result with Clear filters, which restores all tools and returns focus to search. The search field has a visible label and fits narrow screens, category buttons expose their selected state, and the tool count is announced as a status.
Part of #85. Closes #143. The parent demo-readiness epic remains open.
Verified locally with
MISE_TRUSTED_CONFIG_PATHS="$PWD"after reviewing the worktree configuration.mise run installpassed with existing peer dependency warnings.mise run lintpassed, including TypeScript and ESLint across all workspaces.mise run test --runInBand --verbosepassed 61 tests in ten suites. One existing test and suite remain skipped.mise run test --runInBand --verbose ExplorePagepassed all six new cases again after the final conditional-rendering simplification.BROWSER=none mise run devcompiled successfully and served the app on localhost:3000 and backend on localhost:7007.mise run proof-browser -- open http://localhost:3000/explore --browser=chrome, followed by Guest > Enter, rendered the real portal. Before the fix, browser inspection confirmed two empty messages for the same search and category combination.mise run proof-browser -- resize 1440 1000andmise run proof-browser -- run-code '<browser assertions>'verified Space-key category selection, exactly one empty message, Enter-key reset, all 28 tools returning and search regaining focus.mise run proof-browser -- resize 375 900andmise run proof-browser -- run-code '<browser assertions>'verified the search input fits the viewport and keyboard reset still restores all tools and focus. Screenshots usedpage.screenshotunder.tmp/proof/, with transitions disabled for stable capture. All three images were visually inspected.git diff --checkpassed. The temporary localproof-browsertask wrapped the installed Playwright CLI and was removed after capture.This is a frontend-only slice. No backend or packaging changed, so a Docker image build was not required locally. The existing PR workflow also runs the Docker gate.
The new tests use the actual Explore page, category controls and tool cards.
Browser capture succeeded on desktop and mobile. No capture path was blocked and no fallback was needed. Images are committed and pinned to the reviewed commit.
Tool definitions, destinations and matching rules are unchanged. Existing React deprecation and jsdom CSS warnings remain in tests. Runtime logs still show the previously recorded Signals websocket/events 404s and unavailable TechDocs search indexes; these did not block Explore and remain follow-up work. No manual screen reader session was performed.