Skip to content

fix(cli): support skill search shorthand queries#405

Merged
christso merged 2 commits into
mainfrom
fix/skill-default-search
May 19, 2026
Merged

fix(cli): support skill search shorthand queries#405
christso merged 2 commits into
mainfrom
fix/skill-default-search

Conversation

@christso
Copy link
Copy Markdown
Contributor

@christso christso commented May 19, 2026

Summary

  • support allagents skill <query> shorthand for clearly search-shaped queries like pr-search and pr search
  • allow allagents skill search pr search to treat the remaining tokens as one multi-word query
  • rank skill search results by relevance instead of raw repo stars, and filter weak broad-match noise to better match gh skill search
  • map --agent-help skill pr-search and --agent-help skill pr search to the skill search help metadata

Manual E2E

Red E2E (before change)

bun run build
node dist/index.js skill 'pr search'

Observed: error: found 1 error ... Not a valid subcommand name

Green E2E (after change)

bun run build
node dist/index.js skill pr-search --page nope
node dist/index.js skill pr search --page nope
node dist/index.js skill 'pr search' --page nope
node dist/index.js --agent-help skill pr-search

Verified:

  • the three shorthand forms now route into skill search instead of failing subcommand parsing
  • explicit multi-word parsing reaches search validation (Error: --page must be an integer.)
  • --agent-help resolves shorthand invocations to the skill search metadata

Relevance validation

  • aligned src/core/skill-search.ts with the upstream gh search pipeline: pre-rank, enrich, filter, re-rank, then deduplicate by skill name
  • added unit coverage proving weak matches are filtered and exact/partial name matches outrank description-only matches
  • spot-checked live results before rate limiting: pr search now matches the same top set as gh skill search

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented May 19, 2026

Deploying allagents with  Cloudflare Pages  Cloudflare Pages

Latest commit: ece6faa
Status: ✅  Deploy successful!
Preview URL: https://9f8c61b1.allagents.pages.dev
Branch Preview URL: https://fix-skill-default-search.allagents.pages.dev

View logs

@christso christso merged commit 44a83d7 into main May 19, 2026
1 check passed
@christso christso deleted the fix/skill-default-search branch May 19, 2026 07:08
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.

1 participant