Skip to content

fix: resolve search timeout and improve skills page UI alignment #53

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
GhadiSaab wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix: resolve search timeout and improve skills page UI alignment #53

GhadiSaab wants to merge 1 commit into from
+57 −34

Conversation

@GhadiSaab
Copy link

@GhadiSaab GhadiSaab commented Jan 27, 2026

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

@GhadiSaab
fix: resolve search timeout and improve skills page UI alignment
76718f3 
- Added a 10s timeout to OpenAI embedding requests to prevent hanging searches.
- Fixed a TypeScript error in search.ts regarding entry hydration.
- Restructured skills page layout and CSS to ensure consistent alignment between the search toolbar and skill cards.
@vercel
Copy link
Contributor

vercel bot commented Jan 27, 2026

@GhadiSaab is attempting to deploy a commit to the Amantus Machina Team on Vercel.

A member of the Team first needs to authorize it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

all searches timing out

1 participant

@GhadiSaab