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

[PR #313] [MERGED] refactor: post-#298 cleanup #325

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

📋 Pull Request Information

Original PR: https://github.com/openclaw/clawhub/pull/313
Author: @steipete
Created: 2/14/2026
Status: ✅ Merged
Merged: 2/14/2026
Merged by: @steipete

Base: main ← Head: refactor/post-298-cleanups

📝 Commits (4)

  • 98714bd refactor: consolidate slug + embedding helpers
  • 321056c refactor: batch ban/unban skill updates
  • d9b283e refactor: report batched ban/unban scheduling
  • 6011487 fix: unblock package typecheck

📊 Changes

10 files changed (+383 additions, -256 deletions)

View changed files

📝 convex/_generated/api.d.ts (+6 -0)
📝 convex/githubRestore.ts (+3 -8)
📝 convex/httpApiV1.ts (+38 -25)
➕ convex/lib/contentTypes.ts (+18 -0)
➕ convex/lib/embeddingVisibility.ts (+17 -0)
➕ convex/lib/reservedSlugs.ts (+131 -0)
📝 convex/skills.ts (+35 -135)
📝 convex/souls.ts (+5 -11)
📝 convex/users.ts (+129 -75)
📝 packages/clawdhub/src/cli/commands/auth.test.ts (+1 -2)

📄 Description

Refactors follow-ups after #298.

  • Reserved slugs: consolidate active-reservation selection, duplicate cleanup, reclaim/finalize/publish enforcement into convex/lib/reservedSlugs.ts.
  • Embedding visibility: single embeddingVisibilityFor helper used across skills/souls/users.
  • Restore: shared content-type guesser (convex/lib/contentTypes.ts).
  • HTTP v1: refactor users POST router auth+JSON parsing helpers (no behavior change).
  • Ban/unban: move skill hide/restore work into paginated internal batch mutations to reduce per-call load.

Gate: bun run lint && bun run build && 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/313 **Author:** [@steipete](https://github.com/steipete) **Created:** 2/14/2026 **Status:** ✅ Merged **Merged:** 2/14/2026 **Merged by:** [@steipete](https://github.com/steipete) **Base:** `main` ← **Head:** `refactor/post-298-cleanups` --- ### 📝 Commits (4) - [`98714bd`](https://github.com/openclaw/clawhub/commit/98714bdb9016ab3a65003129003b9cd9b6eba80c) refactor: consolidate slug + embedding helpers - [`321056c`](https://github.com/openclaw/clawhub/commit/321056c0ebd7debbf9c6a83555eb17e39a2ced96) refactor: batch ban/unban skill updates - [`d9b283e`](https://github.com/openclaw/clawhub/commit/d9b283ee5336adc44dab79be83a8ad9b244690be) refactor: report batched ban/unban scheduling - [`6011487`](https://github.com/openclaw/clawhub/commit/601148798206bcddef6651bed4c13dc64481cd99) fix: unblock package typecheck ### 📊 Changes **10 files changed** (+383 additions, -256 deletions) <details> <summary>View changed files</summary> 📝 `convex/_generated/api.d.ts` (+6 -0) 📝 `convex/githubRestore.ts` (+3 -8) 📝 `convex/httpApiV1.ts` (+38 -25) ➕ `convex/lib/contentTypes.ts` (+18 -0) ➕ `convex/lib/embeddingVisibility.ts` (+17 -0) ➕ `convex/lib/reservedSlugs.ts` (+131 -0) 📝 `convex/skills.ts` (+35 -135) 📝 `convex/souls.ts` (+5 -11) 📝 `convex/users.ts` (+129 -75) 📝 `packages/clawdhub/src/cli/commands/auth.test.ts` (+1 -2) </details> ### 📄 Description Refactors follow-ups after #298. - Reserved slugs: consolidate active-reservation selection, duplicate cleanup, reclaim/finalize/publish enforcement into convex/lib/reservedSlugs.ts. - Embedding visibility: single embeddingVisibilityFor helper used across skills/souls/users. - Restore: shared content-type guesser (convex/lib/contentTypes.ts). - HTTP v1: refactor users POST router auth+JSON parsing helpers (no behavior change). - Ban/unban: move skill hide/restore work into paginated internal batch mutations to reduce per-call load. Gate: bun run lint && bun run build && 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:54 -05:00
yindo closed this issue 2026-02-15 17:16:54 -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#325
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?