cleanup: remove orphaned SearchBar + hide OCR on Linux - #250
Merged
Conversation
- src/components/layout/SearchBar.tsx was dead code — the live SearchBar is exported from DragBar.tsx and nothing imported the standalone file. tsc confirms nothing referenced it. Deleted. - "Extract Text" (OCR) was offered for every image on every platform, but the ocr_image command only has a real backend on macOS (Vision) and Windows (Windows.Media.Ocr) — on Linux it just returns "not yet available". Added isLinux() and gated the menu item off Linux so we don't advertise an action that always errors there. Tests: isLinux platform cases added. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two small hygiene fixes.
Dead file
src/components/layout/SearchBar.tsxwas orphaned — the live SearchBar is exported fromDragBar.tsx, and nothing imported the standalone file (tsc confirms). Deleted.OCR on Linux
"Extract Text" was offered for every image on every platform, but
ocr_imageonly has a real backend on macOS (Vision) and Windows (Windows.Media.Ocr) — on Linux it returns"OCR is not yet available on Linux". AddedisLinux()and gated the menu item off Linux, so we don't advertise an action that always errors.Tests:
isLinuxplatform cases.tscclean · 565 tests green.🤖 Generated with Claude Code