[PR #64] [MERGED] A few fixes for search. #239

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

📋 Pull Request Information

Original PR: https://github.com/openclaw/clawhub/pull/64
Author: @jamwt
Created: 1/28/2026
Status: Merged
Merged: 1/28/2026
Merged by: @thewilloftheshadow

Base: mainHead: fix_search


📝 Commits (8)

  • 856925b A few fixes for search.
  • dcdc87e Uh swap these labels back thx.
  • 476cf49 Lint.
  • b5ee488 Start to move toward action-based aggregations that bundle up larger stat sets.
  • 69df9a7 Merge remote-tracking branch 'origin/main' into fix_search
  • f97e1ec Use index for scanning stats records.
  • 09cc26b Re-add cron job.
  • 7417471 Merge branch 'main' into fix_search

📊 Changes

9 files changed (+399 additions, -26 deletions)

View changed files

📝 convex/crons.ts (+3 -3)
📝 convex/schema.ts (+7 -0)
📝 convex/search.ts (+8 -0)
📝 convex/skillStatEvents.ts (+289 -1)
📝 src/components/Header.tsx (+32 -4)
📝 src/routeTree.gen.ts (+15 -15)
📝 src/routes/index.tsx (+21 -3)
📝 src/routes/skills/index.tsx (+12 -0)
📝 src/routes/souls/index.tsx (+12 -0)

📄 Description

  • Focus the search box when you click "search" in the header
  • There were some bad codegen things -- trying to access ctx.db inside an action.
  • Change the stats aggregation strategy to speed things up even more and remove needless query invalidations
  • Various linting things

🔄 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/64 **Author:** [@jamwt](https://github.com/jamwt) **Created:** 1/28/2026 **Status:** ✅ Merged **Merged:** 1/28/2026 **Merged by:** [@thewilloftheshadow](https://github.com/thewilloftheshadow) **Base:** `main` ← **Head:** `fix_search` --- ### 📝 Commits (8) - [`856925b`](https://github.com/openclaw/clawhub/commit/856925bd72b50a699f6119846479b01aa1b3c097) A few fixes for search. - [`dcdc87e`](https://github.com/openclaw/clawhub/commit/dcdc87e83e53aab14b4d1d8528a1614966d9fe4c) Uh swap these labels back thx. - [`476cf49`](https://github.com/openclaw/clawhub/commit/476cf4957b42afd1f54221c536cdc605459f3fbb) Lint. - [`b5ee488`](https://github.com/openclaw/clawhub/commit/b5ee488a0c0b9df670a8532d05a692fc0fa53c38) Start to move toward action-based aggregations that bundle up larger stat sets. - [`69df9a7`](https://github.com/openclaw/clawhub/commit/69df9a789619150593f95cd8eb25065084786823) Merge remote-tracking branch 'origin/main' into fix_search - [`f97e1ec`](https://github.com/openclaw/clawhub/commit/f97e1ec15b295eb3678aa520f3f98483269fd591) Use index for scanning stats records. - [`09cc26b`](https://github.com/openclaw/clawhub/commit/09cc26beb7b34e95a070a7c2e5c87d159337aba7) Re-add cron job. - [`7417471`](https://github.com/openclaw/clawhub/commit/741747103ca819c4afeb95842b8f49f05ede69c6) Merge branch 'main' into fix_search ### 📊 Changes **9 files changed** (+399 additions, -26 deletions) <details> <summary>View changed files</summary> 📝 `convex/crons.ts` (+3 -3) 📝 `convex/schema.ts` (+7 -0) 📝 `convex/search.ts` (+8 -0) 📝 `convex/skillStatEvents.ts` (+289 -1) 📝 `src/components/Header.tsx` (+32 -4) 📝 `src/routeTree.gen.ts` (+15 -15) 📝 `src/routes/index.tsx` (+21 -3) 📝 `src/routes/skills/index.tsx` (+12 -0) 📝 `src/routes/souls/index.tsx` (+12 -0) </details> ### 📄 Description * Focus the search box when you click "search" in the header * There were some bad codegen things -- trying to access ctx.db inside an action. * Change the stats aggregation strategy to speed things up even more and remove needless query invalidations * Various linting things --- <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:26 -05:00
yindo closed this issue 2026-02-15 17:16:27 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: openclaw/clawhub#239