Skip to content

Add category-opportunity-index skill#6

Open
norightsforlis wants to merge 2 commits into
joomcode:mainfrom
norightsforlis:add-category-opportunity-index
Open

Add category-opportunity-index skill#6
norightsforlis wants to merge 2 commits into
joomcode:mainfrom
norightsforlis:add-category-opportunity-index

Conversation

@norightsforlis

Copy link
Copy Markdown
Contributor

Summary

Adds the category-opportunity-index skill: Reports a category's opportunity index plus its monthly market stats (estimated GMV, sales, sellers, listings, average ticket) with a plain-language read on how attractive it is.

Lives in skills/category-opportunity-index/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: the new skill routes users to top-sellers, top-keywords, and category-monitor, but this public catalog/README does not include those skills yet. After installation, those handoffs can dead-end or encourage the agent to hallucinate unavailable capabilities.

Could we remove those references from the description/body, or add the referenced skills in the same release?

@profunctor

Copy link
Copy Markdown
Collaborator

Review — category-opportunity-index

Summary: Given one Mercado Livre (Brasil) category named in free text, reports JoomPulse's opportunity index (low/medium/high) plus current monthly stats (est. GMV, sales, active sellers, active listings, average ticket), pulls ~12 months of history for a trend line, and writes a short pt-BR interpretation with a "worth entering / enter with caution / not attractive" takeaway. Read-only, point-in-time snapshot.

Findings:

  • minor — References four sibling skills that don't exist on main (top-sellers, top-keywords, gap-analysis, category-monitor). This matches existing convention (current skills already reference a nonexistent gap-analysis), so these read as aspirational hand-offs rather than literal links — but the count of dangling references is growing. Trim or align to final slugs.
  • minor — The "four metric cards" (GMV, sales, sellers, average ticket) drop "active listings" relative to the 5-row indicators table. Confirm that's an intentional condensation, not an omission.
  • nit — Description is long (~190 words) but on par with ml-product-analysis; triggering coverage is good and distinct from per-product skills.

No bugs, broken queries, hardcoded IDs, wrong field names, or unsafe operations; stays at the abstract "use JoomPulse" level (no leaked cube/tool names), enforces read-only, mandates the disclaimer, handles missing data with . Structure, frontmatter, directory name == name, and README row all match. Solid and mergeable; only the dangling references are worth a comment.

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