[PR #53] [MERGED] fix: resolve search timeout and improve skills page UI alignment #233

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

📋 Pull Request Information

Original PR: https://github.com/openclaw/clawhub/pull/53
Author: @GhadiSaab
Created: 1/27/2026
Status: Merged
Merged: 2/13/2026
Merged by: @steipete

Base: mainHead: fix/search-timeout-and-ui-alignment


📝 Commits (4)

  • 76718f3 fix: resolve search timeout and improve skills page UI alignment
  • 71f504c fix: resolve search timeout and improve skills page UI alignment
  • 24846c9 style: format skills index layout block
  • fde40e2 Merge commit '24846c9' into landpr-53-merge

📊 Changes

3 files changed (+141 additions, -118 deletions)

View changed files

📝 convex/lib/embeddings.ts (+33 -20)
📝 src/routes/skills/index.tsx (+97 -97)
📝 src/styles.css (+11 -1)

📄 Description

This PR addresses two issues in the ClawdHub skills page:

  1. Search Timeout Fix: Added a 10-second timeout using AbortController to the OpenAI embedding API call in convex/lib/embeddings.ts. This prevents the loading skills state from hanging indefinitely if the API is slow or unresponsive.
  2. UI Alignment Fix: Restructured the layout and updated CSS in src/routes/skills/index.tsx and src/styles.css to ensure the skill cards are visually aligned with the search toolbar in a centered container.
  3. TypeScript Fix: Resolved a type predicate error in convex/search.ts to ensure successful compilation.

Closes #50 #52


🔄 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/53 **Author:** [@GhadiSaab](https://github.com/GhadiSaab) **Created:** 1/27/2026 **Status:** ✅ Merged **Merged:** 2/13/2026 **Merged by:** [@steipete](https://github.com/steipete) **Base:** `main` ← **Head:** `fix/search-timeout-and-ui-alignment` --- ### 📝 Commits (4) - [`76718f3`](https://github.com/openclaw/clawhub/commit/76718f3352d872f0a95e547d282280d6d29e1675) fix: resolve search timeout and improve skills page UI alignment - [`71f504c`](https://github.com/openclaw/clawhub/commit/71f504ca9712e83cf6c8517ec3b7e349fecc2431) fix: resolve search timeout and improve skills page UI alignment - [`24846c9`](https://github.com/openclaw/clawhub/commit/24846c93d5f89af0c571226483cad21e9db5b59e) style: format skills index layout block - [`fde40e2`](https://github.com/openclaw/clawhub/commit/fde40e2e5b4061c2a3aa437065ebfb07f2a61a9f) Merge commit '24846c9' into landpr-53-merge ### 📊 Changes **3 files changed** (+141 additions, -118 deletions) <details> <summary>View changed files</summary> 📝 `convex/lib/embeddings.ts` (+33 -20) 📝 `src/routes/skills/index.tsx` (+97 -97) 📝 `src/styles.css` (+11 -1) </details> ### 📄 Description This PR addresses two issues in the ClawdHub skills page: 1. Search Timeout Fix: Added a 10-second timeout using AbortController to the OpenAI embedding API call in convex/lib/embeddings.ts. This prevents the loading skills state from hanging indefinitely if the API is slow or unresponsive. 2. UI Alignment Fix: Restructured the layout and updated CSS in src/routes/skills/index.tsx and src/styles.css to ensure the skill cards are visually aligned with the search toolbar in a centered container. 3. TypeScript Fix: Resolved a type predicate error in convex/search.ts to ensure successful compilation. Closes #50 #52 --- <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:24 -05:00
yindo closed this issue 2026-02-15 17:16:24 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: openclaw/clawhub#233