openclaw/clawhub
1
0
Fork 0
mirror of https://github.com/openclaw/clawhub.git synced 2026-07-22 03:35:29 -04:00
Code Issues 183 Packages Projects Releases 1 Wiki Activity

[PR #76] feat: display total skills count on /skills page #251

New Issue
Open
opened 2026-02-15 17:16:30 -05:00 by yindo · 0 comments
yindo commented 2026-02-15 17:16:30 -05:00
Owner
Copy Link
Copy Source

📋 Pull Request Information

Original PR: https://github.com/openclaw/clawhub/pull/76
Author: @rknoche6
Created: 1/31/2026
Status: 🔄 Open

Base: main ← Head: add-skills-count

📝 Commits (1)

  • eb1963b display total skills count on /skills page

📊 Changes

6 files changed (+59 additions, -4 deletions)

View changed files

📝 convex/crons.ts (+7 -0)
📝 convex/schema.ts (+7 -0)
📝 convex/skills.ts (+13 -1)
📝 convex/statsMaintenance.ts (+27 -0)
📝 src/lib/og.ts (+1 -1)
📝 src/routes/skills/index.tsx (+4 -2)

📄 Description

  • Added countPublicSkills query to convex/skills.ts
  • Updated SkillsIndex to fetch and display the count
  • Fixed missing undici dependency for e2e tests
  • Applied biome formatting
Screenshot 2026-01-31 at 17 33 31

Greptile Overview

Greptile Summary

This PR adds a new Convex query (countPublicSkills) and wires it into the /skills route to display a total count in the page header. It also adds undici as a dev dependency (and updates bun.lock) to fix missing dependency issues in e2e tests, and applies a small Biome-driven import reordering in src/lib/og.ts.

Main integration point is src/routes/skills/index.tsx, which now calls useQuery(api.skills.countPublicSkills) alongside the existing paginated public skills query to show the total.

Confidence Score: 3/5

  • This PR is likely safe to merge, but the new count query can become a scalability bottleneck as data grows.
  • UI change is small and dependency addition is straightforward, but countPublicSkills currently loads all active skills into memory to count them, which can degrade performance and potentially hit Convex query limits on larger datasets since it’s fetched on every /skills page load.
  • convex/skills.ts (countPublicSkills implementation)

(2/5) Greptile learns from your feedback when you react with thumbs up/down!

Context used:

  • Context from dashboard - AGENTS.md (source)

🔄 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/76 **Author:** [@rknoche6](https://github.com/rknoche6) **Created:** 1/31/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `add-skills-count` --- ### 📝 Commits (1) - [`eb1963b`](https://github.com/openclaw/clawhub/commit/eb1963bb99301a64b79076013f6c6f339b66438e) display total skills count on /skills page ### 📊 Changes **6 files changed** (+59 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `convex/crons.ts` (+7 -0) 📝 `convex/schema.ts` (+7 -0) 📝 `convex/skills.ts` (+13 -1) 📝 `convex/statsMaintenance.ts` (+27 -0) 📝 `src/lib/og.ts` (+1 -1) 📝 `src/routes/skills/index.tsx` (+4 -2) </details> ### 📄 Description - Added countPublicSkills query to convex/skills.ts - Updated SkillsIndex to fetch and display the count - Fixed missing undici dependency for e2e tests - Applied biome formatting <img width="1667" height="260" alt="Screenshot 2026-01-31 at 17 33 31" src="https://github.com/user-attachments/assets/915c2c01-5f5d-45bd-9fb7-01b5a3b9b94a" /> <!-- greptile_comment --> <h2>Greptile Overview</h2> <h3>Greptile Summary</h3> This PR adds a new Convex query (`countPublicSkills`) and wires it into the `/skills` route to display a total count in the page header. It also adds `undici` as a dev dependency (and updates `bun.lock`) to fix missing dependency issues in e2e tests, and applies a small Biome-driven import reordering in `src/lib/og.ts`. Main integration point is `src/routes/skills/index.tsx`, which now calls `useQuery(api.skills.countPublicSkills)` alongside the existing paginated public skills query to show the total. <h3>Confidence Score: 3/5</h3> - This PR is likely safe to merge, but the new count query can become a scalability bottleneck as data grows. - UI change is small and dependency addition is straightforward, but `countPublicSkills` currently loads all active skills into memory to count them, which can degrade performance and potentially hit Convex query limits on larger datasets since it’s fetched on every `/skills` page load. - convex/skills.ts (countPublicSkills implementation) <!-- greptile_other_comments_section --> <sub>(2/5) Greptile learns from your feedback when you react with thumbs up/down!</sub> **Context used:** - Context from `dashboard` - AGENTS.md ([source](https://app.greptile.com/review/custom-context?memory=a1d58d20-b4dd-4cbb-973a-9fd7824e1921)) <!-- /greptile_comment --> --- <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:30 -05:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
pull-request
Mirrored from GitHub Pull Request
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
yindo
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: openclaw/clawhub#251
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?