Skip to content

Chore: misc tune ups#4

Merged
ryan-roemer merged 2 commits into
mainfrom
chore/blog-post-misc
Jan 13, 2026
Merged

Chore: misc tune ups#4
ryan-roemer merged 2 commits into
mainfrom
chore/blog-post-misc

Conversation

@ryan-roemer

Copy link
Copy Markdown
Contributor
  • Add a debug timer.
  • Rename search fn.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR introduces a debug timer utility and renames the search function for better clarity. The changes add optional performance monitoring capabilities that can be enabled via a URL parameter.

Changes:

  • Add a new timing.js module with a debug timer utility that logs performance measurements when enabled
  • Rename search function to searchPosts throughout the codebase for better naming clarity
  • Integrate timing measurements for key operations (downloads, database creation, query embedding, vector search)

Reviewed changes

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

Show a summary per file
File Description
public/app/data/timing.js New module providing debug timer functionality with URL parameter control
public/app/data/search.js Renamed search to searchPosts, added timing for DB creation and query operations, removed unused performance global
public/app/data/index.js Updated export to use renamed searchPosts function
public/app/index.js Updated import and usage to call searchPosts instead of search
public/app/data/api.js Added timing measurements for data downloads using the new timer utility

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread public/app/data/search.js
@ryan-roemer ryan-roemer merged commit 43cc34e into main Jan 13, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants