fix(web): recall publishers outside browse window in search#2790
Merged
Patrick-Erichsen merged 6 commits intoJun 23, 2026
Merged
Conversation
Publisher search only scanned the top 500 by installs and dropped empty profiles, so handles like vincentkoc never appeared even when the user profile was public.
Add regression coverage for publishers with published skills that fall outside the top install browse window, matching the vyctorbrzezowski case.
vyctorbrzezowski
requested review from
a team and
Patrick-Erichsen
as code owners
Contributor
|
@vyctorbrzezowski is attempting to deploy a commit to the OpenClaw Foundation Team on Vercel. A member of the Team first needs to authorize it. |
Remove stats.skills expectation from listPublicPage search recall test; public list items only expose downloads and installs counts.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Deploy note
Merge alone does not fix production. This PR only changes Convex (
convex/publishers.ts); there are no frontend changes.After merge to
main, a maintainer must deploy the Convex backend (targetbackendorfull):Merging to
maindoes not auto-deploy. Until the backend deploy lands,/publisherssearch onclawhub.aiwill still show the pre-fix behavior documented below.Summary
/publishersonly scanned the top browse window (~500 publishers by downloads/installs) before applying the text filter, so matching handles outside that window never appeared.vyctorbrzezowskihas 5 skills and 46 installs in production but does not show up for?q=vyctoror?q=vyctorbrzezowski.Proof (production, before backend deploy)
Browser
?q=vyctorbrzezowski→ no publishers found?q=vincent→ onlyvincentchan(missingvincentkoc)Convex read-only (
wry-manatee-359.convex.cloud)listPublicPagehandlesvyctor[]vyctorbrzezowski[]vincent["vincentchan"]vincentkoc[]Profiles that exist but are missing from search:
vyctorbrzezowski→ 5 skills, 1 package, 46 installsvincentkoc→ public profile, 0 published skillsTest plan
VITE_CONVEX_URL=https://example.invalid bunx vitest run convex/publishers.test.tswry-manatee-359.convex.cloudvincentkoc,vyctorbrzezowski)https://clawhub.ai/publishers