[PR #16] [CLOSED] Fix skills search relevance ordering #212

Closed
opened 2026-02-15 17:16:18 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/openclaw/clawhub/pull/16
Author: @johntheyoung
Created: 1/18/2026
Status: Closed

Base: mainHead: fix/skills-search-relevance


📝 Commits (3)

  • 2d4c3c6 fix: align skills search relevance
  • cafff34 fix: tighten search matching
  • 74be9c7 chore: update convex api types

📊 Changes

8 files changed (+343 additions, -41 deletions)

View changed files

📝 convex/_generated/api.d.ts (+2 -0)
📝 convex/devSeed.ts (+10 -3)
convex/lib/searchText.test.ts (+32 -0)
convex/lib/searchText.ts (+25 -0)
📝 convex/search.ts (+85 -32)
e2e/search-exact.pw.test.ts (+97 -0)
📝 src/__tests__/skills-index.test.tsx (+83 -3)
📝 src/routes/skills/index.tsx (+9 -3)

📄 Description

Summary

  • keep /skills search results in relevance order
  • align /skills search behavior with root page (relevance-first)
  • show relevance sort and disable sort controls while searching
  • generate real embeddings for seeded skills when OPENAI_API_KEY is set

Screenshots

  • /skills with query shows Relevance sort + relevance-ordered results
  • /skills without query defaults to Newest sort

Testing

  • bun run lint
  • bun run test

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/openclaw/clawhub/pull/16 **Author:** [@johntheyoung](https://github.com/johntheyoung) **Created:** 1/18/2026 **Status:** ❌ Closed **Base:** `main` ← **Head:** `fix/skills-search-relevance` --- ### 📝 Commits (3) - [`2d4c3c6`](https://github.com/openclaw/clawhub/commit/2d4c3c6b360592f82566d38fa40f2635e6f84fc4) fix: align skills search relevance - [`cafff34`](https://github.com/openclaw/clawhub/commit/cafff341a882f0bdc2ebab0daa4db09b70546052) fix: tighten search matching - [`74be9c7`](https://github.com/openclaw/clawhub/commit/74be9c748964cb35e643c907ea5b7460e5a7b7aa) chore: update convex api types ### 📊 Changes **8 files changed** (+343 additions, -41 deletions) <details> <summary>View changed files</summary> 📝 `convex/_generated/api.d.ts` (+2 -0) 📝 `convex/devSeed.ts` (+10 -3) ➕ `convex/lib/searchText.test.ts` (+32 -0) ➕ `convex/lib/searchText.ts` (+25 -0) 📝 `convex/search.ts` (+85 -32) ➕ `e2e/search-exact.pw.test.ts` (+97 -0) 📝 `src/__tests__/skills-index.test.tsx` (+83 -3) 📝 `src/routes/skills/index.tsx` (+9 -3) </details> ### 📄 Description ## Summary - keep `/skills` search results in relevance order - align `/skills` search behavior with root page (relevance-first) - show relevance sort and disable sort controls while searching - generate real embeddings for seeded skills when `OPENAI_API_KEY` is set ## Screenshots - `/skills` with query shows `Relevance` sort + relevance-ordered results - `/skills` without query defaults to `Newest` sort ## Testing - bun run lint - bun run test --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 17:16:18 -05:00
yindo closed this issue 2026-02-15 17:16:18 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: openclaw/clawhub#212