feat: recommend Featured plugins and skills from search and adoption - #3717
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review completeClawSweeper finished reviewing this revision. The review result is being finalized. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Observed live validation after rebasing and deploying the published stack ( {
"environment": "DISPOSABLE LOCAL VALIDATION FIXTURES \u2014 not production recommendations",
"http": {
"plugin": {
"status": 200,
"cacheControl": "private, no-store"
},
"skill": {
"status": 200,
"cacheControl": "private, no-store"
}
},
"cliEvidenceMatches": true,
"unauthenticatedStatus": 401,
"reportReadsPreserveState": {
"pluginSearchObservations": 19,
"searchWeeklyDigests": 0,
"packageBadges": 26,
"skillBadges": 1
},
"catalogs": {
"plugin": {
"adoption": {
"generatedAt": 1789502704677,
"inspectedItems": 1,
"periodEnd": 1789502704677,
"periodStart": 1788912000000,
"rankingVersion": "unversioned",
"snapshotId": "pn7c84q0fxw6yej7a4qyh9atmx8efsbf",
"status": "available",
"totalItems": 1,
"truncated": false
},
"candidates": [
{
"adoption": {
"bookmarks": null,
"downloads": 40,
"generatedAt": 1789502704677,
"installs": 3,
"lifetimeInstalls": null,
"periodEnd": 1789502704677,
"periodStart": 1788912000000,
"rank": 1,
"rankingVersion": "unversioned",
"snapshotId": "pn7c84q0fxw6yej7a4qyh9atmx8efsbf",
"source": "package-trending",
"sourceObservedAt": null
},
"artifactKind": "plugin",
"category": null,
"displayName": "LOCAL validation calendar",
"eligibilityReasons": [],
"eligibleForFeatured": true,
"id": "plugin:claw893-local-calendar",
"isFeatured": false,
"isOfficial": false,
"name": "claw893-local-calendar",
"search": null,
"summary": null,
"support": "adoption-only",
"url": "/plugins/claw893-local-calendar",
"version": "1.0.0"
}
],
"searches7d": 0
},
"skill": {
"adoption": {
"generatedAt": 1789502705142,
"inspectedItems": 1,
"periodEnd": 1789502400000,
"periodStart": 1789416000000,
"rankingVersion": "skills-trending-v4",
"snapshotId": "claw893-local-observed",
"status": "available",
"totalItems": 1,
"truncated": false
},
"candidates": [
{
"adoption": {
"bookmarks": 2,
"downloads": 60,
"generatedAt": 1789502705142,
"installs": 4,
"lifetimeInstalls": null,
"periodEnd": 1789502400000,
"periodStart": 1789416000000,
"rank": 1,
"rankingVersion": "skills-trending-v4",
"snapshotId": "claw893-local-observed",
"source": "clawhub-rising",
"sourceObservedAt": null
},
"artifactKind": "skill",
"category": null,
"displayName": "LOCAL validation calendar",
"eligibilityReasons": [],
"eligibleForFeatured": true,
"id": "clawhub:xx71sg9xgqqp9j3zywanftxees8ee7vp",
"isFeatured": false,
"isOfficial": false,
"name": "claw893-local-calendar",
"nativeSkillId": "xx71sg9xgqqp9j3zywanftxees8ee7vp",
"search": null,
"summary": "Manage your calendar events and schedule meetings with clear reminders for your team.",
"support": "adoption-only",
"url": "/claw893-local-skill/skills/claw893-local-calendar",
"version": "1.0.0"
}
],
"searches7d": 0
}
}
} |
|
Codex review: needs maintainer review before merge. Reviewed September 15, 2026, 4:59 PM ET / 20:59 UTC (Revision 2). ClawSweeper reviewWhat this changesAdds staff-only Featured recommendations that combine search demand with existing adoption rankings across the dashboard, API, and admin CLI. Merge readiness✅ Ready for maintainer review Keep open: this collaborator-authored PR adds distinct functionality absent from current main. No blocking findings; the updated evidence resolves the previous screenshot-access concern. Priority: P2 Review scores
Verification
How this fits togetherClawHub’s staff intelligence tools help curators choose plugins and skills to feature. The report combines search aggregates with existing Trending snapshots, checks current eligibility, and returns advisory candidates without publishing selections. flowchart TD
A[Staff request] --> B[Staff access check]
B --> C[Recommendation report]
D[Search aggregates] --> C
E[Existing Trending snapshots] --> C
C --> F[Current eligibility checks]
F --> G[Dashboard API and CLI evidence]
Before mergeNone. Agent review detailsSecurityNone. Review metrics
Technical reviewBest possible solution: Keep a single advisory recommendation owner that preserves catalog-specific evidence and current eligibility while leaving Featured publication under existing staff controls. Do we have a high-confidence way to reproduce the issue? Not applicable: this adds a recommendation capability; the supplied real local browser and runtime evidence demonstrates adoption candidates with no completed-day search history. Is this the best way to solve the issue? Yes: shared report composition reuses existing rankings and eligibility checks; implementing separate scoring in the dashboard and CLI would create competing behavior. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against 22cc45be4fc3. LabelsLabel changes:
Label justifications:
EvidenceWhat I checked:
Likely related people:
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (1 earlier review cycle)
|
ClawHub UI ProofStatus: Existing search-demand dashboard (local fixture baseline) Plugin Featured evidence (local fixture) Skill Featured evidence (local fixture) Raw proof files: https://github.com/openclaw/clawhub/tree/qa-artifacts/clawhub-ui-proof/pr-3717/search-and-adoption-evidence |
|
Resolved both proof availability risks from the latest review. The three screenshots now live on the repository’s Proof index and screenshots · exact source blob comparison All 16 changed production/test/generated files have identical git blob IDs between the real running proof revision |
2347a08 to
a8e18d8
Compare



Search demand alone misses artifacts people are already adopting. This adds one advisory Featured report for plugins and skills, reusing each catalog’s existing Trending ranks and counts alongside the shared search aggregates. Staff can inspect the same candidates in the management dashboard, authenticated API, and admin CLI, with reasons, exact periods, freshness, current eligibility and excluded entries. It does not change public Trending or publish Featured selections.
Part of CLAW-893, stacked after scoped skill collection #3714 and homepage collection #3716.
Validation: the unified stack at
b5a6462c96519589d4bc8c5e4ea847375be4191dpasses 6,856 unit tests (3 skipped), all CLI/package suites, static/audit and types/build. Bounded independent code review returned no P0 findings. The only rebase adaptation is an English-language test fixture, matching the current public Trending eligibility rule.Before/after UI proof uses explicitly LOCAL validation data, not production recommendations. The baseline shows the original plugin-demand dashboard; the candidate demonstrates separate plugin/skill adoption evidence using its own labeled fixtures. Counts across these fixtures are not a product benchmark.
Live proof: Actual Convex actions, authenticated HTTP/admin CLI and browser agree: the plugin has 40 downloads/3 installs; the skill has 60 downloads/4 installs/2 bookmarks; both retain their catalog rank, periods and snapshot identity despite zero completed-day searches. Current eligibility is rechecked. Anonymous access remains denied; read-only report calls leave digest and Featured state untouched. Production dry run follows deployment and will use only real collected searches and adoption.
Production change: +878/−108 lines (tests and generated types reported separately); growth provides the shared adoption/recommendation owner, bounded API/CLI schema and review UI. No new scoring weights, collection store, ranking algorithm or publishing path.
Accessible proof and exact production-source blob comparison: evidence index. All three public PNGs returned HTTP200 and matched the inspected local image SHA256 bytes.