Skip to content

feat: add full-text search results#149

Merged
k1LoW merged 8 commits intomainfrom
fulltext
Mar 28, 2026
Merged

feat: add full-text search results#149
k1LoW merged 8 commits intomainfrom
fulltext

Conversation

@k1LoW
Copy link
Copy Markdown
Owner

@k1LoW k1LoW commented Mar 27, 2026

Summary

  • Add a full-text search API and surface content matches in the sidebar
  • Add file name match grouping, collapsible search sections, and richer result snippets
  • Add in-document search hit markers and refine their placement and theme-aware styling

Closes #146

@k1LoW k1LoW requested a review from Copilot March 27, 2026 16:05
@k1LoW k1LoW self-assigned this Mar 27, 2026
@k1LoW k1LoW added enhancement New feature or request tagpr:minor labels Mar 27, 2026

This comment was marked as outdated.

Sign in to view

k1LoW added 3 commits March 28, 2026 01:12
@k1LoW
refactor: deduplicate escapeRegExp, remove custom min/max, and add se…
5c23171 
…arch debounce
@k1LoW
fix: address PR review comments for full-text search
44b9bb1 
- Skip fenced code blocks when tracking headings in search results
- Only clear pending scroll request when target heading is found
- Add explicit `uploaded` field to search API response
- Snapshot file list in handleSearch to avoid data races
@k1LoW
fix: format MarkdownViewer.tsx to pass oxfmt check
40c73a1
@k1LoW k1LoW requested a review from Copilot March 27, 2026 16:29
@github-actions

This comment has been minimized.

Sign in to view

This comment was marked as outdated.

Sign in to view

@k1LoW k1LoW requested a review from Copilot March 27, 2026 23:59
@github-actions

This comment has been minimized.

Sign in to view

This comment was marked as outdated.

Sign in to view

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 28, 2026

Code Metrics Report

main (f4cb788) #149 (a4f0bfb) +/-
Coverage 57.2% 57.5% +0.2%
Code to Test Ratio 1:0.5 1:0.5 -0.1
Test Execution Time 30s 28s -2s
Details 
  |                     | main (f4cb788) | #149 (a4f0bfb) |  +/-  |
  |---------------------|----------------|----------------|-------|
+ | Coverage            |          57.2% |          57.5% | +0.2% |
  |   Files             |             39 |             40 |    +1 |
  |   Lines             |           3272 |           3518 |  +246 |
+ |   Covered           |           1873 |           2023 |  +150 |
- | Code to Test Ratio  |          1:0.5 |          1:0.5 |  -0.1 |
  |   Code              |           5117 |           5632 |  +515 |
+ |   Test              |           2888 |           2950 |   +62 |
+ | Test Execution Time |            30s |            28s |   -2s |

Code coverage of files in pull request scope (71.8% → 70.3%)

Files Coverage +/- Status
internal/frontend/src/components/MarkdownViewer.tsx 35.1% -8.7% modified
internal/frontend/src/components/Sidebar.tsx 61.5% +2.3% modified
internal/frontend/src/hooks/useApi.ts 63.3% -9.8% modified
internal/frontend/src/utils/regex.ts 100.0% +100.0% added
internal/server/server.go 78.4% +0.3% modified

Reported by octocov

@k1LoW k1LoW requested a review from Copilot March 28, 2026 00:18
Copilot AI reviewed Mar 28, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

@k1LoW k1LoW merged commit e6baced into main Mar 28, 2026
7 checks passed
@k1LoW k1LoW deleted the fulltext branch March 28, 2026 00:33
@github-actions github-actions bot mentioned this pull request Mar 28, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

@k1LoW k1LoW

Labels

enhancement New feature or request tagpr:minor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Feature request: Full-text content search across files

2 participants

@k1LoW