Skip to content

feat: add install count formatting to skill display in search results#364

Merged
quuu merged 1 commit intomainfrom
popularity-contest
Feb 13, 2026
Merged

feat: add install count formatting to skill display in search results#364
quuu merged 1 commit intomainfrom
popularity-contest

Conversation

@markpython86
Copy link
Contributor

This pull request enhances the user experience of the skill search feature by displaying installation counts for each skill in a human-readable format. The main improvements include introducing a helper function to format install numbers and updating the search result output to show these counts both in the interactive prompt and in the summary listing.

User Interface Improvements:

  • Added a new formatInstalls function in src/find.ts to format install counts as "K" or "M" where appropriate (e.g., "1.2K installs", "3M installs").
  • Updated the interactive search prompt to display formatted install counts next to each skill, improving visibility of skill popularity.
  • Modified the summary output after a search to include formatted install counts for each skill listed.

Code Cleanup:

  • Removed unused parseOwnerRepo import from src/find.ts.

@vercel
Copy link

vercel bot commented Feb 13, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
v0-skills-hg Error Error Open in v0 Feb 13, 2026 7:32pm

import * as readline from 'readline';
import { runAdd, parseAddOptions } from './add.ts';
import { track } from './telemetry.ts';
import { isRepoPrivate, parseOwnerRepo } from './source-parser.ts';
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed unused functions

Copy link
Collaborator

@quuu quuu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sweet! luckily the install count was already being returned

@quuu quuu merged commit 4ff73ca into main Feb 13, 2026
6 of 7 checks passed
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