Skip to content

Add top-sellers-in-category skill#16

Open
norightsforlis wants to merge 3 commits into
joomcode:mainfrom
norightsforlis:add-top-sellers-in-category
Open

Add top-sellers-in-category skill#16
norightsforlis wants to merge 3 commits into
joomcode:mainfrom
norightsforlis:add-top-sellers-in-category

Conversation

@norightsforlis

Copy link
Copy Markdown
Contributor

Summary

Adds the top-sellers-in-category skill: Ranks the top sellers in a category by estimated average revenue — a single point-in-time leaderboard with full seller stats.

Lives in skills/top-sellers-in-category/SKILL.md and is registered in the README skills table. Public behavior and user-visible workflow only — no internal service names, schema, or implementation details.

Public-Safety Checklist

  • No credentials, tokens, keys, or secrets.
  • No internal filesystem paths.
  • No private Slack, Jira, Notion, admin, or service URLs.
  • No private service names or implementation details.
  • No customer data or private product data.
  • Skill directory name matches SKILL.md frontmatter name.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@profunctor

Copy link
Copy Markdown
Collaborator

KISS review: this skill references category-top-100 change tracker, seller-overview tracker, and brand-position tracker, but those skills are not present in this PR’s target branch.

That can make agents route users to unavailable workflows. Please remove these handoff promises or replace them with generic wording until those skills exist in the catalog.

@profunctor

Copy link
Copy Markdown
Collaborator

Review — top-sellers-in-category

Summary: Read-only skill that, given a free-text category, uses JoomPulse to rank a Mercado Livre (Brasil) category's sellers by estimated average revenue and returns a point-in-time leaderboard (avg sales/revenue, last-365-day sales, cancellation rate, trend, brands, product distribution, international shipping, listing-type counts), with per-seller links and an optional cards + medal-colored bar chart.

Findings:

  • minor — Description/body reference sibling skills that don't exist yet ("category-top-100 change tracker", "seller-overview tracker", "brand-position tracker"). Drop them or gate as "if available" until those PRs land, and use canonical slugs.
  • minor — "average revenue" vs "average monthly revenue" inconsistency. Frontmatter/README/intro say "average revenue"; Step 2 and the cards say "average monthly revenue"/"average ticket". The table header Receita méd. is also ambiguous on period. Pin the period and name it once.
  • nit — Prerequisites claim JoomPulse returns "medal, reputation" per seller and the chart colors by a medal field, but neither appears in the output table. Tighten the "when available" wording.
  • nit — Folder name matches name; no secrets, internal paths, or hardcoded IDs; read-only. Public-safety checklist holds.

No blockers. The dangling references and the revenue-period inconsistency are the two worth fixing before merge.

norightsforlis and others added 2 commits June 24, 2026 20:25
…lable

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…able leaderboard + optional period comparison (user-supplied previous table)

Replaces the separate category-top-100-change-tracker (PR joomcode#17, closed): the leaderboard is now downloadable and can be compared against a previous-period table the user supplies, showing per-seller movement + biggest movers.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants